Bestride Mount Manager

Bestride Mount Manager

57.6k Downloads

Option "Automatically enable new mounts upon learning them" Bug

Veldryn opened this issue ยท 0 comments

commented

Environment

  • BeStride version: 1.0.2.1 and the versions before that
  • WoW version: 9.0.1

Steps to reproduce

I have the option "Automatically enable new mounts upon learning them" unchecked. I do not wish any new mounts be added automatically to my Mount profiles.
Now, if I learn a new mount...

Expected Behaviour

...the new mount should not get called automatically and should not get its checkbox checked in the "Mounts" tab.

The new mount doesn't get called automatically. This works exactly as it should.
But: there's a display issue. New mounts learned get their checkboxes checked in the mounts list, although they should not.

Debug Output

There's no bugs in the bugsack concerning this. Code works fine.

Together with a friend we have identified the error in the code, though. It's really just a visual thing, the boxes in the Mounts tab getting checked upon learning new mounts, no matter how I chose to set the "Automatically enable new mounts" option.

The error is in Bestride.lua, line 341
self.db.profile.mounts[mountType][mountID] = true
--> it must be false instead.

I have been playing with this fix for the last months and it worked fine - any new mounts appeared in the list just fine and stayed unchecked. Please let me know if you can acknowledge the bug and the fix, since you understand your code much better than I do. It would be lovely if you could put the fix in the next version for everyone.