Sort does not work with specialty bags (ammo, quiver, etc...)

Question

Sort does not work with specialty bags (ammo, quiver, etc...)

D34d3nd opened this issue 5 years ago · 10 comments

D34d3nd commented 5 years ago

Running Software (issues missing this information will be deleted):

Addon version: 8.2.10
Server patch: 1.13.2 (31727)

Have you read the changelog? (please don't waste our time)
Yes, always read the changelog.

Describe the bug
With a specialty bag (ammo pouch), sorting does not work. Blizzard warning message "Only bullets can be placed in that"

To Reproduce
Steps to reproduce the behaviour:

Open Bagnon pressing the 'b' or whatever button is mapped to open bags
Make sure you have an ammo pouch with bullets. Full slot or missing slots in the ammo pouch does not matter. Have not tested with quiver or soul shard bag for locks.
Click on the 'broom' icon to sort
See error

Expected behaviour
Bags sorted including sorting of ammo pouch

Screenshots

Error Logs
No error logs

Additional context
Nothing additional

BjarkeM · Answer 1 · 2021-08-09T15:41:01.000Z

This is still a bug on version 9.1.0 for TBC classic 2.5.1

any pointers as for where I can look to fix this in the code?

Jaliborc · Answer 2 · 2019-09-10T00:22:58.000Z

Can you send a screenshot of your bags? because it is working for me

D34d3nd · Answer 3 · 2019-09-10T00:26:49.000Z

From: João Cardoso <[email protected]> Sent: Monday, September 9, 2019 7:23 PM To: tullamods/Bagnon <[email protected]> Cc: D34d3nd <[email protected]>; Author <[email protected]> Subject: Re: [tullamods/Bagnon] Sort does not work with specialty bags (ammo, quiver, etc...) (#977) Can you send a screenshot of your bags? because it is working for me — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#977> , or mute the thread <https://github.com/notifications/unsubscribe-auth/AA64GZOPF724PHJNSJXEO63QI3SGHANCNFSM4IVBEYDQ> . <https://github.com/notifications/beacon/AA64GZP3J362D5HKQOWOFQDQI3SGHA5CNFSM4IVBEYD2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6JNKTQ.gif>

Jaliborc · Answer 4 · 2019-09-10T23:00:15.000Z

If you get rid of one of the bullet stacks (putting it at the bank, using, deleting). Does the issue still happen?

D34d3nd · Answer 5 · 2019-09-11T01:40:25.000Z

Works just fine after I put all my arrows and quiver in the bank. Sorting works.

So, a couple things have changed since yesterday.

I no longer have an ammo pouch and ammo. I have a quiver and arrows. Same issue though but I get "Only arrows can go in that"
I am no longer using the starter ammo pouch for dwarves. See last bullet-point where I say I now have a quiver.
I have tried different options in Bagnon like 'Reverse Bag Order', 'Bag Break', and 'Reverse Slot Order'. Each one or any combination seems to have the same effect where it will refuse to sort.
I have tried placing the ammo pouch from before and now the quiver in different bag slots to see if it made a difference. No difference. Same error message.

Also, I have noticed, after I get the Blizzard red error message that only bullets or arrows can be placed in that, every time I manually move something in my bag, I see the message again. Its almost like the process is still running maybe?

Is there some type of debug or verbose logging that I can enable to possibly catch the output when I run this? I have tried '/console scriptErrors 1' and then a reload of the UI, but it does not seem to be throwing any LUA error popups.

D34d3nd · Answer 6 · 2019-09-11T01:53:26.000Z

One other note. I have just tried disabling all addons 1 by 1 so the only addons enable are Bagnon, Bagnon Config, Bagnon Guild Bank, and Bagnon Void Storage.

Issue persists. This did not help.

Jaliborc · Answer 7 · 2019-09-11T18:22:58.000Z

Could you try depositing items other than bullets/arrows in the bank one by one and see if one of them causes the issue?
Might be trying to put a specific item in the specialized bag.

D34d3nd · Answer 8 · 2019-09-13T02:33:00.000Z

Ok, so I have not had a chance to try depositing everything in the bank and doing it one by one due to lack of bank space and what not. Although, I did see a new/updated addon on Curse called ‘tdPack2’ https://www.curseforge.com/wow/addons/tdpack2 I gave it a whirl and wow, it works fantastic. It sorted my bags without issue no matter if I had empty quiver slots, arrows not in my quiver but in my bags (it automatically put them into the quiver for me), and it was fairly quick. I will test some more tomorrow, but not sure if you can maybe look at the code in that addon to see what the difference is in the way it is handling specialty bags, but it seems like a winner so far for me. From: João Cardoso <[email protected]> Sent: Wednesday, September 11, 2019 1:23 PM To: tullamods/Bagnon <[email protected]> Cc: D34d3nd <[email protected]>; Author <[email protected]> Subject: Re: [tullamods/Bagnon] Sort does not work with specialty bags (ammo, quiver, etc...) (#977) Could you try depositing items other than bullets/arrows in the bank one by one and see if one of them causes the issue? Might be trying to put a specific item in the specialized bag. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#977> , or mute the thread <https://github.com/notifications/unsubscribe-auth/AA64GZJZGYEDMCJWSMLT453QJEZQPANCNFSM4IVBEYDQ> . <https://github.com/notifications/beacon/AA64GZJP3UYHBFXGQZDVQ4DQJEZQPA5CNFSM4IVBEYD2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6PNWEY.gif>

Jaliborc · Answer 9 · 2019-09-14T12:19:54.000Z

I think I figured it out. It was the enchant reagents that it was trying to place in the ammo bag.

D34d3nd · Answer 10 · 2019-09-14T16:05:56.000Z

Awesome, thanks for all the work. I will check it out once you post an update. From: João Cardoso <[email protected]> Sent: Saturday, September 14, 2019 7:20 AM To: tullamods/Bagnon <[email protected]> Cc: D34d3nd <[email protected]>; Author <[email protected]> Subject: Re: [tullamods/Bagnon] Sort does not work with specialty bags (ammo, quiver, etc...) (#977) I think I figured it out. It was the enchant reagents that it was trying to place in the ammo bag. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#977> , or mute the thread <https://github.com/notifications/unsubscribe-auth/AA64GZLQVJPXEVE3AQRLETTQJTJGXANCNFSM4IVBEYDQ> . <https://github.com/notifications/beacon/AA64GZOJOY3E6F56I2ZKMYLQJTJGXA5CNFSM4IVBEYD2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6W2V4A.gif>

Share to