1. Introduction
Link copied
Over the years of developing Xu-Fu's Pet Guide, many ideas and suggestions accumulated. I have tried a few different tools like Trello or even Confluence to keep track of them together with my development notes, but these tools are way too clunky for this small project. And they all lack transparency.
This section aims to solve both problems: all my notes in one place, and making them visible to you, the hopefully curious reader!
You will find this article changing constantly. If a feature or bug suddenly vanishes from this list, it usually means that it weng live :-)
2. Bugs
Link copied
- Wailing Critters => Pet table when not logged in shows different pets than when logged in. Why!?
- PO file on Home.php => title for "Article creators" is hardcoded
- message when comment reaches gold status working or not?
- gold comment trigger will trigger email in the language of that person who made the deciding upvote. Not the person who made the comment. com_vote in ajax
- in profiles, btag and discord can be way too long. needs a way to display it properly
asdf
3. Project: Battletable 2.0 (40%)
Link copied
The battle tables were the first thing created on Xu-Fu. They are mostly static and not up to the standards of the current page.
This project is to fully re-create them, to integrate the user system, user collections and to add many more features.
3.1 Info
Link copied
Logic of substitute finder:
no requirements => all substitutes are fine, based on skills and family
only a breed => creator pet, only show the selected breed(s). For substitutes, ignore breeds
only a stat => use the stat to determine which creator pet breed is OK. and use the stat for substitutes and their breeds
breed and stats => for the creators pet, only show the breed he activated. for substitutes, use the stat to show them or not.
3.2 Open Tasks
Link copied
- Required breed output:
> if only 1 breed is possible, format so the lone commata is not visible - code marker btbr01
> create tooltip showing all possible breeds and which ones are OK to use, which ones are NOK
- check against collection and output it graphically:
> show owned pets under "Your pet":
If 1: "Yes - HB"
If more: "Yes (2)" with tooltip.
> test if owned pet does NOT meet requirements (breed, level etc.) and output orangy border
> make the border around the whole pet card, not just the lighter part (include column left)
> check if border colors work with every family color background
- Rematch strings
> Store pet IDs and breed strings somewhere hidden for all 3 pets.
> When changing substitute, have the content changed and have the rematch string changed accordingly with the data stored in those hidden fields
- level pet slot
- Star rating tooltip needs a single "star" icon
- An option in user settings to select the priority of strategies, which order is more important, maybe tags to exclude entirely so they never show up?
- strategies pull info from Pets database, need to move to PetsUser and PetsNPC
- URLs in creator text are not formatted correctly https://wow-petguide.com/index.php?Strategy=1218
- Recreate additional info section (see UI draft)
- question mark with breeds to show breed info, which breeds are available for this pet, and which ones you own, if they are verified for this strategy or not and why not.
- add interactivity to above tooltips to rate breeds and gather info from users if a certain breed worked or not
- in case no breed fits the requirements, show some kind of info text about it.
- update print_pettable to use the new logic for substitutes. "Any Fox" etc. doesn't exist anymore then.
3.3 Known Bugs
Link copied
- Tooltip of favourite button not positioned correctly
3.4 Optional features
Link copied
- Counter for how many visits a strategy has received
- "My Strategies" page for strat creators. show # visits, comments, votes and favs of own strategies.
- Info section explaining how everything work
3.5 Release plan
Link copied
- check if functions check_specialpet and get_specialpet are still required and kill them if not
- give every strategy a user as creator
- remove all votes /ratings from strategies
- move all "Any Fox" etc. to direct pets
- remove "Any Fox" etc. entries from PetsUser DB
- thanks to Grainne and Remte and everyone else who helped with the substitute finder
- give thanks to Simca for the breed info from petbattle breed ID
- give thanks to the creator of the pet xp thread for the info: https://www.warcraftpets.com/community/forum/viewtopic.php?t=8829
4. Project: Clever Alternatives (0%)
Link copied
The strategy alternatives should not have a limit. They should not have a fixed order. The order should be dynamic and follow a logic that could include user settings, pets collected, strategy upvotes, favourits and additional factors.
The interface should use AJAX and give a clearer preview of the pets used.
5. Project: User Strategies (0%)
Link copied
Users should be able to create and add their own strategies.
6. Project: Pet Cards (0%)
Link copied
Basically, replace wowhead tooltips with xu-fu pet tooltips that show more value (breeds, stats etc.)
7. Project: Blog 2.0 (0%)
Link copied
The blog should support different topics and different users posting updates. The blog should be sortable by topic / date. It should show a quick overview and should be searchable. There should be the option to go to a single blog post and view that in full screen.
8. Miscellaneous Tasks, Bugs and Suggestions
Link copied
8.1 Suggestions
Link copied
By Irayna: Add XP Values to link list on left (like EU/US appearances of Menagerie). To see which fights give more XP than usual (espcially for legion world quests)
By Flatluigi: 21.03.2018: minor sort of request: for the pets that don't have a level/can't battle, take them out of the calculations for number of pets at max level/at rare quality
By Remte: I'd like to have fresh comment responses notifications cleared out when I visit them independently.
By Panseit: message system, put an icon next to the user profile icon when new messages are there.
By Irayna: When viewing other peoples collection, add a column for your own pets to compare the collections.
8.2 Tasks and To-Dos
Link copied
- add protocol entries to admin pages
- Petimport - adm_petimport.php:
- check at beginning if user pets are not verified with "cageable or not" and go through them.
- user pets: option to set cageable or not
- Spells: entire API import missing
- battle tag in user profiles needs EU / US switch
- my comments section sometimes does not filter the comments correctly
- option to claim old comments
- crawl of pet collection: if a character is saved, use that one. If that saved char is not available anymore, crawl all characters from bnet again and pick most recently active one and save that one.
9. Possible Future Projects
Link copied
- SEO
- comment-section: show more info about the threads in tooltip? maybe the top comment + 2 or 3 additional ones with [..] and a "X more messages" ?
- comment-section: for "Votes", maybe add together all votes from the thread from your comments? Otherwise a +50 subcomment might be hidden because it's not the first one
- Email notifications
- Tools: Top X pets of all strategies
- "Your profile has been viewed X times."
- Add friend (what effects?)
- specific user icons and backgrounds for admins / mods /translators
- sticky comments
- user setting how many comments to display per page (currently set site wide)
- better brute force protection (pw reset)
- rework all tooltips to use tooltipster
- comment moderation features directly on page, including display of reports
- allow bnet connected accounts to unlink from bnet
- comment statistics (how many, how often per month, average upvotes etc.)
- Flavour growls:
"your collection was updated and you got these new pets in the last week!"
after X days of registration, still no password / battle.net connected - send reminder message "hey do this for x"
10. Finished Projects
Link copied
10.1 Project: Article Editor (100%)
Link copied
The page hosts many different articles and supports localization into 9 languages. This editor makes it easy for other users to add, edit or translate articles.
It is fully integrated into the user system, UAC, and allows on-page editing.
1
You must be logged in to vote on commentsMeows#2580
wrote on 2022-01-10 00:19:35
https://cdnjs.cloudflare.com/ajax/libs/jquery/2.1.4/jquery.min.js
https://ajax.googleapis.com/ajax/libs/jquery/2.1.4/jquery.min.js
https://cdn.jsdelivr.net/npm/[email protected]/dist/jquery.min.js
is some alternative CDNs.
Not sure if its related, but I got a dual homed PC, with both IPv4 and IPv6, but something is off using the code from jquery.com, but not the other CDNs.
(edited)
1
You must be logged in to vote on commentsAranesh
wrote on 2022-05-09 11:41:11
1
You must be logged in to vote on commentsToenail
wrote on 2022-02-04 22:49:30
edit: i just saw the upcoming project that may be related to my idea. my bad (edited)
1
You must be logged in to vote on commentsAranesh
wrote on 2022-05-09 11:39:42
https://www.wow-petguide.com/Compare
Enjoy!
1
You must be logged in to vote on commentsBarbasol#1361
wrote on 2021-11-29 17:32:05
It'd also be helpful to show the pet levels when you have more than 1 pet. So you could tell without hovering if you have all level 1s, or possibly a level 25 in there. I guess for space reasons maybe a level range would work. E.g. "Your pet: 3* (1-25)". Hope that makes sense. Thanks!
2
You must be logged in to vote on commentsAranesh
wrote on 2021-12-01 06:51:44
This is a perfect place to leave feedback and make suggestions ^^
The substitutes were supposed to behave exactly like you say, with pets you have leveled to 25 and rare showing up first, others later. I took a look at that and found a bug that caused this to not work properly. It should be fixed now, have a look please and tell me if that's what you were looking for :)
I could add the level range in brackets as well but don't feel that it's needed because of the color coding. If you have at least one of the duplicates on 25 rare, the bar will be green and that should be enough to do the strategy. Otherwise it's yellow. This also takes into account multiple uses of pets (e.g. a strategy asking for 2 or 3x the same pet).
What do you think, does that do the trick or do you still think the level range would help?
1
You must be logged in to vote on commentsBarbasol#1361
wrote on 2021-12-01 13:20:14
I'm not always seeing the sorting. For instance, on this one, the rabbits still seem pretty random, with the blue 25s actually being last:
Link
However, on this one with snails, the sorting is better:
Link
The first alternates there for me are rare 25s which is good. Then a 1 green, 6 gray, then 23 white (so kinda random) but then the last 2 are missing which seems correct. Maybe you aren't sorting the non-25s but descending level would be nice as you can then use stones to level or make blue.
I think you are right about the level range not being needed. The color coding (which I didn't exactly realize there) and the level sorting should do the trick!
One last thing: when viewing alternate strategies I'll see Red/yellow for missing/low level pets even when I might have a great substitute. Not sure what to suggest about this. Maybe a checkbox in the Strategy Alternates Browser to show substitutes? Or maybe automatically show substitutes if the specific pet isn't leveled, but with an asterisk in the corner? Not sure what is possible or best but thought I'd mention it. Thanks! (edited)
1
You must be logged in to vote on commentsBarbasol#1361
wrote on 2021-12-01 13:35:10
Edit: Well this one does rabbits fine, but others still seem off:
https://www.wow-petguide.com/Strategy/4455/Crab_People-Critter (edited)
1
You must be logged in to vote on commentsAranesh
wrote on 2021-12-02 06:01:53
All pets in category B) are in a random order, meaning sometimes level 1 pets come before level 25 because both are not rare quality.
I'll see if I can optimize that to prioritize lvl 25 pets to be first there :)
EDIT: Done, better now?
-------
Substitues for alternatives are a very tricky topic.
I can't add that by default, because substitutes are in most cases no perfect match. For example, a creator might select a HS breed pet because it needs exactly >280 speed. A substitute with HS breed might only have 278 speed, making the strategy always fail. From the information the creator gives me (this pet in HS), I can't know what speed *exactly* is needed and which substitutes would need to be excluded.
This would be solved by strategy creators adding hard requirements (>280 speed instead of "HS breed") but most don't do that, or even know the exact requirement.
I see two options to improve this:
- Check if substitute pets are *exactly* the same. Not the case very often, some rabbits, snakes and so on but as long as they have really exactly the same stats, no problem. The sub finder currently doesn't do that
- An advanced setting as you suggest to regulate how strict/lenient the sub finder is. Something that likely not many would use :D
Both are... quite heavy in dev time so I haven't really dabbled with that, so far :S (edited)
2
You must be logged in to vote on commentsBarbasol#1361
wrote on 2021-12-03 12:43:20
And it does seem to favor descending level order a lot of the time. I've been doing Family Familiar where leveling is possible and it's very satisfying to pick an alternate lvl 23 pet and still have it work (though it is risky... most reliable for the 3rd slot pet). A lot easier now.
I appreciate your thoughts on substitutes. Dev time vs usefulness is very understandable! The sub finder seems to already classify subs as green/yellow (based on abilities/speed I assume). I would say just tapping into that is good enough. Just to be able to see that when viewing all strats at once to know which ones already have green subs available. But dev time is always a consideration.
Thanks again for all your work!
0
You must be logged in to vote on commentsWh15k3y_ wrote on 2020-09-27 08:35:36
1
You must be logged in to vote on commentsAranesh
wrote on 2020-09-27 15:50:00
0
You must be logged in to vote on commentsWh15k3y_ wrote on 2020-11-30 10:15:49
1
You must be logged in to vote on commentsAranesh
wrote on 2020-11-30 12:08:07
Best I can offer is trying without adblocker on the page and if that works it, perhaps just whitelist the page. When you are logged in, 99% of the ads are hidden anyway.
0
You must be logged in to vote on commentsBeardsly wrote on 2020-09-07 09:52:02
1
You must be logged in to vote on commentsShenk
wrote on 2020-09-07 11:39:24
1
You must be logged in to vote on commentsAranesh
wrote on 2020-09-07 13:57:52
Just go to a section and hit the blue Load Pet Table button - once loaded, a rematch export button is available
1
You must be logged in to vote on commentsThoizor
wrote on 2020-06-20 15:10:06
EDIT: Am leveling battle pets atm, so the numbers mentioned above are no longer correct. There does remain a difference of 16 pets at max level between both sites, as well as a difference of 7 uncommon pets. (edited)
1
You must be logged in to vote on commentsCoJo
wrote on 2020-06-17 15:30:25
1
You must be logged in to vote on commentsAranesh
wrote on 2020-06-17 16:05:34
2
You must be logged in to vote on commentsGráinne
wrote on 2020-06-17 16:27:30
Sir Galveston https://wow-petguide.com/8361
Jarrun https://wow-petguide.com/5043
Ahhh. US vs EU.
I had the US page up when I saw those. On EU I get the right pages. (edited)
1
You must be logged in to vote on commentsCoJo
wrote on 2020-06-17 19:27:31
1
You must be logged in to vote on commentsAranesh
wrote on 2020-06-18 04:44:40
Both of these should be fixed now :) Thanks for bringing it up!
1
You must be logged in to vote on commentsWalkingDead11
wrote on 2020-04-03 23:18:25
2
You must be logged in to vote on commentsAranesh
wrote on 2020-04-04 05:59:20
The sad but simple answer, though, is: There is no mobile version of the page. It was never designed to support mobile :/ I try to make tweaks here and there but lack the knowledge (and time) to make it really mobile friendly.
I'll log a bug ticket for me to look into the alternatives tab a bit more, maybe I can come up with a solution. Thanks for reporting it and sorry I don't have a better answer for you!
1
You must be logged in to vote on commentsShenk
wrote on 2020-04-04 10:00:40
2
You must be logged in to vote on commentsGráinne
wrote on 2020-04-04 10:52:29
The only time these strats are useful is when you are actually at your PC. Why would people use a mobile browser when sitting at their keyboard? What's the condition I'm not seeing?
1
You must be logged in to vote on commentsShenk
wrote on 2020-04-04 12:36:38
1
You must be logged in to vote on commentsDragonsAfterDark
wrote on 2020-04-04 15:31:16
1
You must be logged in to vote on commentsShenk
wrote on 2020-04-04 16:25:05
1
You must be logged in to vote on commentsGráinne
wrote on 2020-04-04 16:25:25
1
You must be logged in to vote on commentsShenk
wrote on 2020-04-04 16:31:46
2
You must be logged in to vote on commentsAranesh
wrote on 2020-04-05 04:26:07
I'm in the very, very early stages of learning how to do mobile friendliness :/
1
You must be logged in to vote on commentsBlocky
wrote on 2020-01-14 06:28:50
Error messages:
This error message: "Use of undefined constant language - assumed 'language' (this will throw an Error in a future version of PHP) in /kunden/426045_85652/webseiten/Petguide/classes/ajax/bt_rmsteps.php on line 152
This error message: "Use of undefined constant language - assumed 'language' (this will throw an Error in a future version of PHP) in /kunden/426045_85652/webseiten/Petguide/classes/ajax/bt_rmsteps.php on line 179
2
You must be logged in to vote on commentsAranesh
wrote on 2020-01-15 03:42:26
3
You must be logged in to vote on commentsBlocky
wrote on 2020-01-16 16:03:37
1
You must be logged in to vote on commentsAranesh
wrote on 2020-01-17 11:48:08
1
You must be logged in to vote on commentsJestersTear
wrote on 2019-12-05 03:21:42
For the pet dungeons, instead of just spitting out the optimal strategy for each encounter based on our pet list, make a dungeon optimal strategy list all at once? What I mean is I look up, let's say, Gnomeregon, and it tells me for optimal strategies to use the same pet in 5 fights. Maybe something where it starts at the last fight and works backwards, so your best pets are used for the last fight (let's say it's fight #10), then it goes back to fight 9 and looks for an optimal strategy that doesn't use the 3 pets you'll be using in the final fight unless you have 2+ of one of them. This way you don't get to fight 8 and it says to use Ikky, but all 3 are either dead or too wounded from fights 2, 4 and 5 to be used again.
(edited)
2
You must be logged in to vote on commentsAranesh
wrote on 2019-12-05 13:17:43
That's definitely planned :) It's currenty listed as "Upcoming Project: New Dungeon Tool" and what you are suggesting is *exactly* what I'm planning to do ^^
Hope I'll be through this current database rework soon so I can jump on cooler stuff like this one.
Thanks for the suggestion! If you have any other ideas, always happy to hear them
3
You must be logged in to vote on commentsJestersTear
wrote on 2019-12-05 16:25:54
1
You must be logged in to vote on commentsZoroarkRonan
wrote on 2019-09-07 21:36:11
2
You must be logged in to vote on commentsGráinne
wrote on 2019-09-07 21:37:24
1
You must be logged in to vote on commentsragath
wrote on 2019-05-20 14:15:45
1
You must be logged in to vote on commentsAranesh
wrote on 2019-05-21 05:31:12
I know about both and what they do, but have never used them to work on a software product. I'm very much a novice in this field and in the creation of this page I made many mistakes that make it very hard to move it to a repository, mostly security details (hardcoded passwords yay!). The effort to make everything ready for such a move is probably quite high, and a big unknown to me. So it's part not wanting to invest the time and part not wanting people to see the code behind this page that keeps me from doing it ^^'
1
You must be logged in to vote on commentsragath
wrote on 2019-05-21 20:15:48
1
You must be logged in to vote on commentsshybobcat
wrote on 2019-03-20 01:40:39
https://www.wow-petguide.com/index.php?Strategy=1352
Would also like to add that this site is amazing! I’m very impressed with both how it looks and especially how well done the coding behind it! Thank you!
2
You must be logged in to vote on commentsRemte
wrote on 2019-03-20 02:58:56
I reuploaded the pet and told our master tinkerer, and it seems it's okay now. Hopefully it was not a signal of something more severe ^^
1
You must be logged in to vote on commentsshybobcat
wrote on 2019-03-20 19:45:07
2
You must be logged in to vote on commentsEkire
wrote on 2018-12-28 12:53:36
1
You must be logged in to vote on commentsEkire
wrote on 2018-12-30 11:43:38
1
You must be logged in to vote on commentsRemte
wrote on 2018-12-06 14:30:06
If this is not worth it, just expand the duplicate warning to all sections of the page… though it could be problem when someone decides to apply a strat the other way i.e. from globals to specific ones :P
1
You must be logged in to vote on commentsSeira
wrote on 2018-10-16 08:14:47
1
You must be logged in to vote on commentsgsanta
wrote on 2018-10-20 06:51:03
1
You must be logged in to vote on commentsSeira
wrote on 2018-10-21 06:53:45
1
You must be logged in to vote on commentsAmber1019
wrote on 2018-10-13 11:52:12
Another thing I've thought would be great would be adding a "date posted" and/or "date updated" to strategy pages. That way if/when Blizzard nerfs and makes changes to battle pets again, we'll be able to see when the strategy was posted, and whether or not it's been updated since the changes Blizzard made, so we know to alert others in the comments about it or to just look into finding a more recent strategy.
1
You must be logged in to vote on commentsNIteowl1970
wrote on 2018-09-26 15:08:39
0
You must be logged in to vote on commentsPike wrote on 2018-08-24 19:28:37
If I could give one little bit of feedback. I copy the turn instructions and paste them into the Rematch notes for each saved team. Previously the copied notes would paste with each turn on one line (with the turn number and description separated by whitespace). However, sometime between mid-Legion and today the page structure has changed so that copy and paste now loses the formatting and each cell is on its own line (so the turn number is on its own line then the description on the next line and so on).
I'd love to see the old table structure return (maybe just for the popout?) although tbh its not a major problem to manually fix the notes after pasting.
Thanks again for the site.
1
You must be logged in to vote on commentsAranesh
wrote on 2018-09-10 05:36:45
0
You must be logged in to vote on commentsPike wrote on 2018-09-22 10:28:03
1
You must be logged in to vote on commentsNebs
wrote on 2018-09-22 03:58:02
Have seen few suggestions similar to mine, relating high XP battles leveling pet fights.
I was thinking of a list of fights for Pet battle week (bonus event) when you get that lovely +200% XP, focusing to those battles that give high XP.
So my idea was to have a list categorized by areas, where you can find those battles that give high XP, so most beneficial to do on the bonus week. It could be a pet leveling strat at the same time even on the weeks where the bonus XP isn't on.
1
You must be logged in to vote on commentsCole#13497
wrote on 2018-07-27 16:19:40
Allow users to link pets when typing comments so you could use your links you want for wowhead on your own site as well. Also the links would allow you to say prevent people from link beast pets on the Aquatic Assault page etc. Additionally, it could eventually even help you import strategies from comments.
2
You must be logged in to vote on commentsAranesh
wrote on 2018-07-28 04:55:51
1
You must be logged in to vote on commentsCole#13497
wrote on 2018-07-30 12:42:11
1
You must be logged in to vote on commentsgsanta
wrote on 2018-05-11 07:22:47
Just wondering how Battletable 2.0 is coming along. Looking forward to the new features and hadn't seen any noticeable updates on this Development Notes page lately. Thanks for all the team's great work on the site. I love it.
1
You must be logged in to vote on commentsAranesh
wrote on 2018-05-15 03:19:27
There are smaller changes here and there. I did put it to 70% completed recently because the main parts are done, and I think that's a fair estimate :D There's only some minor additions missing like a system for tags, or the coordinates of tamers. I'm getting there! Outside of weekends I barely find the time to work on it nowadays, unfortunately. But I think at least this part will be ready well before BfA comes out.
1
You must be logged in to vote on commentsgsanta
wrote on 2018-05-15 07:50:41
1
You must be logged in to vote on commentsTa
wrote on 2022-02-01 12:38:18
New Comment: