Petopia Corrections - Current Patch Only

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

Re: Petopia Corrections - Current Patch Only

Unread post by Wain »

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

Unread post by TygerDarkstorm »

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: 1171
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

Unread post by Iowawolf »

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: 13511
Joined: Tue Jul 27, 2010 1:54 am
Gender: Male

Re: Petopia Corrections - Current Patch Only

Unread post by Wain »

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

Unread post by WhyNotCats909 »

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

Unread post by TygerDarkstorm »

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: 16354
Joined: Wed Jan 06, 2010 8:07 pm
Realm: Wyrmrest Accord US
Gender: Female
Location: Canada

Re: Petopia Corrections - Current Patch Only

Unread post by Vephriel »

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: 13511
Joined: Tue Jul 27, 2010 1:54 am
Gender: Male

Re: Petopia Corrections - Current Patch Only

Unread post by Wain »

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: 13511
Joined: Tue Jul 27, 2010 1:54 am
Gender: Male

Re: Petopia Corrections - Current Patch Only

Unread post by Wain »

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: 16354
Joined: Wed Jan 06, 2010 8:07 pm
Realm: Wyrmrest Accord US
Gender: Female
Location: Canada

Re: Petopia Corrections - Current Patch Only

Unread post by Vephriel »

Nice work Wain! :)
User avatar
Wain
The Insane
The Insane
Posts: 13511
Joined: Tue Jul 27, 2010 1:54 am
Gender: Male

Re: Petopia Corrections - Current Patch Only

Unread post by Wain »

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: 13511
Joined: Tue Jul 27, 2010 1:54 am
Gender: Male

Re: Petopia Corrections - Current Patch Only

Unread post by Wain »

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

Unread post by TygerDarkstorm »

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]
User avatar
Varethyn
Petopia Artist
Posts: 5230
Joined: Wed Dec 01, 2010 11:12 am
Realm: Azjol-Nerub EU
Gender: Any

Re: Petopia Corrections - Current Patch Only

Unread post by Varethyn »

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.

Check out PetEmote here! | Visit my Flight Rising lair!

Rest in peace mum. I really did love and miss you, even if I never got to see you.

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

Re: Petopia Corrections - Current Patch Only

Unread post by Wain »

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: 13511
Joined: Tue Jul 27, 2010 1:54 am
Gender: Male

Re: Petopia Corrections - Current Patch Only

Unread post by Wain »

There, should be fixed :)
Shaman avatar by Spiritbinder.
User avatar
Varethyn
Petopia Artist
Posts: 5230
Joined: Wed Dec 01, 2010 11:12 am
Realm: Azjol-Nerub EU
Gender: Any

Re: Petopia Corrections - Current Patch Only

Unread post by Varethyn »

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

Check out PetEmote here! | Visit my Flight Rising lair!

Rest in peace mum. I really did love and miss you, even if I never got to see you.

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

Unread post by Kalliope »

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: 13511
Joined: Tue Jul 27, 2010 1:54 am
Gender: Male

Re: Petopia Corrections - Current Patch Only

Unread post by Wain »

Duskstalker is fixed. Thanks :)
Shaman avatar by Spiritbinder.
User avatar
GormanGhaste
 Community Resource
 Community Resource
Posts: 6460
Joined: Tue Mar 09, 2010 2:00 pm
Realm: Uldaman (and Ravencrest)

Re: Petopia Corrections - Current Patch Only

Unread post by GormanGhaste »

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.
Image
Post Reply