arkayenro / arkinventory

A World of Warcraft Inventory mod for Retail, Burning Crusade, and Classic
108 stars 14 forks source link

[BUG] Re-assignment of Mount Type Causes Error Message #2005

Open Calesta opened 2 weeks ago

Calesta commented 2 weeks ago

What version number you are using?

3.11.01

What game client are you playing? Retail, Wrath, Classic / Live, PTR, Beta

Retail

What language is the game client set to?

English

Describe the bug

I have the 'Include as Ground mounts selections' option selected but the Mechanized Lumber Extractor wasn't showing up in my Ground mount list. I tried manually setting it to 'Type' Ground and received an error message (see attached screenshot). It didn't show up in my Ground mount list and it disappeared from my Flying mount list!

Luckily, it seems to be something related to the current session as I logged in the next day on the same character and the mount had returned to my Flying mount list.

I'm reporting the bug so that maybe you can squash it as you're able, but I'm also interested if there's a known issue of Ground mounts not showing up in the Flying mount list with that option selected. Should I open a separate issue for that?

To Reproduce See above.

Screenshots Mechanized

Additional context Thanks for your continued efforts on making everything functional given all the sweeping changes. I've been seeing the regular alpha updates every day or so with fixes and/or enhancements. It's very much appreciated. I just downloaded the latest alpha to try the mount 'Mode' option you added. It sounds very much like something I inquired about while testing in the Beta client.

arkayenro commented 1 week ago

i think the error may have already been fixed in 3.11.01 alpha 3.

the mounts only show up in their own type window, you just select them in their own area and it will use what you have selected in there if you tick that option

Calesta commented 1 week ago

Thanks for clearing up how that's intended to work. I won't worry about it not showing up in the ground mount list. ;)

I installed the latest version (3.11.01 alpha 3) and tried to change the mount type again and received a different error message.

Mechanized2

So 'capabilities' seems to have an argument now but not 'options'. Whatever that means. ;)

arkayenro commented 1 week ago

just means i buggered up the config option code, should be fixed in the next alpha