-
-
Notifications
You must be signed in to change notification settings - Fork 2.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix(Core/Player): Allow to swap bags when they are filled #17086
Conversation
Need check if bag can hold all items e.g smaller bag and also if it’s appropriate e.g soul bag mining bag etc |
I tested swapping big filled bag to small filled bag, don't work. So that is good. |
Interestingly I've tried switching a felcloth bag with an herb bag in this situation, and it gave this error, |
I also tested filled mining bag (20slot), to normal bag (20slot), to mining bag. Works too |
As for swapping mining bag onto bag with normal items on it, it gives the same error bench got. So that is good I guess caught here azerothcore-wotlk/src/server/game/Entities/Player/PlayerStorage.cpp Lines 3861 to 3865 in aea5401
|
Bank-wise, I've swapped bags between existing bags in the bank: the items don't leave the bag they were in before the swap. I've swapped bags between a bag in my backpack and one in the bank, the item in the bank bag stays in the bank bag in the position post-swap. If too many items are in the bank bag to swap into the new bag, it gives the error |
If you see me forget it, please at least link the issue for me in the comment so I don't have to go looking for it. I don't work for cc 🤷 |
Changes Proposed:
This PR proposes changes to:
If your pull request promotes complex changes that require a detailed explanation, please describe them in detail specifying what your solution is and what is it meant to address.
Issues Addressed:
SOURCE:
The changes have been validated through:
Tests Performed:
This PR has been:
How to Test the Changes:
Known Issues and TODO List:
How to Test AzerothCore PRs
When a PR is ready to be tested, it will be marked as [WAITING TO BE TESTED].
You can help by testing PRs and writing your feedback here on the PR's page on GitHub. Follow the instructions here:
http://www.azerothcore.org/wiki/How-to-test-a-PR
REMEMBER: when testing a PR that changes something generic (i.e. a part of code that handles more than one specific thing), the tester should not only check that the PR does its job (e.g. fixing spell XXX) but especially check that the PR does not cause any regression (i.e. introducing new bugs).
For example: if a PR fixes spell X by changing a part of code that handles spells X, Y, and Z, we should not only test X, but we should test Y and Z as well.