Petopia Corrections - Current Patch Only

Message
Author
User avatar
Wain
The Insane
The Insane
Posts: 11651
Joined: Tue Jul 27, 2010 1:54 am
Gender: Male

Re: Petopia Corrections - Current Patch Only

#501 Unread post by Wain » Mon Apr 18, 2016 10:49 pm

Thanks! I'll look into those too. :)

I'm hoping a more dynamic version of the site (using php) will be in soon and I can start making live corrections on my own. Depending on how long that takes, I may hold off on these latest corrections until then. It'll be much easier if I can.
Shaman avatar by Spiritbinder.

TygerDarkstorm
 Community Resource
 Community Resource
Posts: 14480
Joined: Wed Sep 15, 2010 10:15 pm
Realm: Staghelm (Alliance), Wyrmrest Accord (Horde)
Gender: Female playing both genders
Location: WoW

Re: Petopia Corrections - Current Patch Only

#502 Unread post by TygerDarkstorm » Mon Apr 18, 2016 10:58 pm

Oh okay cool. That would be awesome if you could do that. ^_^ If that happens, I noticed that all of the small icons don't give hover blip thingers. So far just seems to be with the MoP and maybe later pets; I haven't looked into them all.

Follow me on Tumblr! @projectashley
Like Crochet? You can see what I make @ facebook.com/AshedCreations

Lord Godfrey wrote:Some people only want to watch the world burn. Others want to be the ones responsible for burning it...
[/center]

User avatar
Iowawolf
Master Hunter
Master Hunter
Posts: 1093
Joined: Sun Oct 06, 2013 7:53 am
Realm: Moon Guard, Hati's Realm
Gender: Male
Location: Trueshot Lodge with Hati

Re: Petopia Corrections - Current Patch Only

#503 Unread post by Iowawolf » Sun Apr 24, 2016 6:36 pm

Under the wolves page you have Frostfang in Storm Peaks as tamable but when I look on Wowhead they say it isn't and when you try tells you invalid target unless you need to be at war with them it is not tamable.
Image

User avatar
Wain
The Insane
The Insane
Posts: 11651
Joined: Tue Jul 27, 2010 1:54 am
Gender: Male

Re: Petopia Corrections - Current Patch Only

#504 Unread post by Wain » Sun Apr 24, 2016 9:45 pm

Iowawolf wrote:Under the wolves page you have Frostfang in Storm Peaks as tamable but when I look on Wowhead they say it isn't and when you try tells you invalid target unless you need to be at war with them it is not tamable.
Thanks for catching that. I corrected the spreadsheet, but the change isn't live yet.
Shaman avatar by Spiritbinder.

User avatar
WhyNotCats909
Posts: 40
Joined: Sat Jun 22, 2013 3:24 pm
Realm: Misha, Kul Tiras

Re: Petopia Corrections - Current Patch Only

#505 Unread post by WhyNotCats909 » Mon Apr 25, 2016 7:58 pm

Please don't hug the shiny kitties. The shiny kitties are angry all the time, and I wouldn't want Crazy Red Eyed Kitties to nom your face.

TygerDarkstorm
 Community Resource
 Community Resource
Posts: 14480
Joined: Wed Sep 15, 2010 10:15 pm
Realm: Staghelm (Alliance), Wyrmrest Accord (Horde)
Gender: Female playing both genders
Location: WoW

Re: Petopia Corrections - Current Patch Only

#506 Unread post by TygerDarkstorm » Fri Apr 29, 2016 12:25 am

For some of reason the bird of prey special ability links to the tenacity section of pet buffs.
http://www.wow-petopia.com/talents/skills.html#tenacity

Follow me on Tumblr! @projectashley
Like Crochet? You can see what I make @ facebook.com/AshedCreations

Lord Godfrey wrote:Some people only want to watch the world burn. Others want to be the ones responsible for burning it...
[/center]

User avatar
Vephriel
Illustrious Master Hunter
Illustrious Master Hunter
Posts: 15953
Joined: Wed Jan 06, 2010 8:07 pm
Realm: Wyrmrest Accord US
Gender: Female
Location: Alberta, Canada

Re: Petopia Corrections - Current Patch Only

#507 Unread post by Vephriel » Wed Jun 08, 2016 8:17 pm

A very minor thing, and I only realized this because I've been working on my Isle of Thunder achievements, but I think Cera might be missing from the brown direhorn list (even though she's not tameable). Just for completionist's sake. :)

User avatar
Wain
The Insane
The Insane
Posts: 11651
Joined: Tue Jul 27, 2010 1:54 am
Gender: Male

Re: Petopia Corrections - Current Patch Only

#508 Unread post by Wain » Tue Jun 21, 2016 12:01 am

I now have direct access to the Petopia database to fix things.

So far I've added the missing beasts that were mentioned, set Frostfang as non-tameable and added a description for Reflective Plating.

Thanks for all those corrections :)


Next I'll see why the Crane and Porcupine pages don't dispay tooltips on abilities. Very odd.

Then I'll sort out the mess of Moth colours.
Shaman avatar by Spiritbinder.

User avatar
Wain
The Insane
The Insane
Posts: 11651
Joined: Tue Jul 27, 2010 1:54 am
Gender: Male

Re: Petopia Corrections - Current Patch Only

#509 Unread post by Wain » Tue Jun 21, 2016 1:04 am

TygerDarkstorm wrote:For some of reason the bird of prey special ability links to the tenacity section of pet buffs.
http://www.wow-petopia.com/talents/skills.html#tenacity
It turns out there were two "tenacity" anchors on the page, one for the "Tenacity" pet adaptation and another for the "Tenacity" ability of Birds of Prey.

I simply removed the grouping by adaptation and now all families are listed in the same table, alphabetically. Hopefully it should work now :)
Shaman avatar by Spiritbinder.

User avatar
Vephriel
Illustrious Master Hunter
Illustrious Master Hunter
Posts: 15953
Joined: Wed Jan 06, 2010 8:07 pm
Realm: Wyrmrest Accord US
Gender: Female
Location: Alberta, Canada

Re: Petopia Corrections - Current Patch Only

#510 Unread post by Vephriel » Tue Jun 21, 2016 1:48 am

Nice work Wain! :)

User avatar
Wain
The Insane
The Insane
Posts: 11651
Joined: Tue Jul 27, 2010 1:54 am
Gender: Male

Re: Petopia Corrections - Current Patch Only

#511 Unread post by Wain » Tue Jun 21, 2016 2:12 am

I fixed the tooltips issues for cranes, porcupines and bears.

Just the moths to go, for now, then I can work on getting the Legion stuff up.
Shaman avatar by Spiritbinder.

User avatar
Wain
The Insane
The Insane
Posts: 11651
Joined: Tue Jul 27, 2010 1:54 am
Gender: Male

Re: Petopia Corrections - Current Patch Only

#512 Unread post by Wain » Tue Jun 21, 2016 9:44 am

Moths should now be fixed. Hopefully all moths appear under their correct look.

The looks keys for some moths have changed, for consistency. Originally it didn't matter too much if they weren't consistent with the official skin names, but now we have all kinds of wing-body skin combos and there's literally 100 possible looks, so consistency becomes critical to avoid confusion like we had.

The old .html file versions have kept their old names, but refer to the proper colours in the php file versions. If that makes any sense. For example, mothskinyellow_green.html now refers to look.php?id=mothskinyellow-redwings .
Shaman avatar by Spiritbinder.

TygerDarkstorm
 Community Resource
 Community Resource
Posts: 14480
Joined: Wed Sep 15, 2010 10:15 pm
Realm: Staghelm (Alliance), Wyrmrest Accord (Horde)
Gender: Female playing both genders
Location: WoW

Re: Petopia Corrections - Current Patch Only

#513 Unread post by TygerDarkstorm » Tue Jun 21, 2016 3:50 pm

Thanks for all your hard work making those fixes, Wain. :) :hug:

Follow me on Tumblr! @projectashley
Like Crochet? You can see what I make @ facebook.com/AshedCreations

Lord Godfrey wrote:Some people only want to watch the world burn. Others want to be the ones responsible for burning it...
[/center]

Varethyn
Illustrious Master Hunter
Illustrious Master Hunter
Posts: 4473
Joined: Wed Dec 01, 2010 11:12 am
Realm: Azjol-Nerub EU

Re: Petopia Corrections - Current Patch Only

#514 Unread post by Varethyn » Tue Jun 28, 2016 4:30 am

Thanks indeed for all the effort you've been putting into this Wain :) But I have to prod you with one more thing: On the Oxen page the blue musken is currently listed under shoveltusks.

User avatar
Wain
The Insane
The Insane
Posts: 11651
Joined: Tue Jul 27, 2010 1:54 am
Gender: Male

Re: Petopia Corrections - Current Patch Only

#515 Unread post by Wain » Tue Jun 28, 2016 7:26 am

Thanks for pointing that out. The database entry actually seems to be correct, but there seems to be something weird going on with the way the page displays tables, in particular that one which is an unused look. I'm still trying to work it out, but it's very strange!
Shaman avatar by Spiritbinder.

User avatar
Wain
The Insane
The Insane
Posts: 11651
Joined: Tue Jul 27, 2010 1:54 am
Gender: Male

Re: Petopia Corrections - Current Patch Only

#516 Unread post by Wain » Tue Jun 28, 2016 7:49 am

There, should be fixed :)
Shaman avatar by Spiritbinder.

Varethyn
Illustrious Master Hunter
Illustrious Master Hunter
Posts: 4473
Joined: Wed Dec 01, 2010 11:12 am
Realm: Azjol-Nerub EU

Re: Petopia Corrections - Current Patch Only

#517 Unread post by Varethyn » Tue Jun 28, 2016 5:28 pm

Aha I see. You're welcome and it is displaying properly now, good job :)

User avatar
Kalliope
Illustrious Master Hunter
Illustrious Master Hunter
Posts: 14062
Joined: Wed Jan 06, 2010 4:40 am
Realm: Dethecus
Location: Thedas
Contact:

Re: Petopia Corrections - Current Patch Only

#518 Unread post by Kalliope » Tue Jun 28, 2016 8:21 pm

Flyra wrote:Really sorry to be a bother. Not sure if this is the right place. Duskstalker is actually level 8 and is listed as lvl 9 on Petopia.

http://www.wowhead.com/npc=14430/duskstalker
Quoting this for organizational purposes. :)

Image
Kalliope's Pantheon of Pets
YouTube Edition
Thanks to Serenith for the avatar and signature!


User avatar
Wain
The Insane
The Insane
Posts: 11651
Joined: Tue Jul 27, 2010 1:54 am
Gender: Male

Re: Petopia Corrections - Current Patch Only

#519 Unread post by Wain » Wed Jun 29, 2016 3:21 am

Duskstalker is fixed. Thanks :)
Shaman avatar by Spiritbinder.

User avatar
GormanGhaste
Illustrious Master Hunter
Illustrious Master Hunter
Posts: 4869
Joined: Tue Mar 09, 2010 2:00 pm
Realm: Uldaman (and Ravencrest)

Re: Petopia Corrections - Current Patch Only

#520 Unread post by GormanGhaste » Sun Jul 10, 2016 2:39 pm

Wain wrote:Moths should now be fixed. Hopefully all moths appear under their correct look.

The looks keys for some moths have changed, for consistency. Originally it didn't matter too much if they weren't consistent with the official skin names, but now we have all kinds of wing-body skin combos and there's literally 100 possible looks, so consistency becomes critical to avoid confusion like we had.

The old .html file versions have kept their old names, but refer to the proper colours in the php file versions. If that makes any sense. For example, mothskinyellow_green.html now refers to look.php?id=mothskinyellow-redwings .
Bloodspore Moths have red bodies and beige wings, but are currently displaying with both pic and text for beige bodies and greens wings.
Browncloaks Unite

Post Reply