fix: remove old lootcontainer flag

[luan]

Before this: QUICKLOOTCONTAINER wasn't being removed from previous container, so when we saved it in iologindata.cpp both the new container and the old were getting the attribute. This caused an inconsistent behavior where the item that ended up first in the table would win, if that item was the old one, the change in loot containers was then reverted.

This change always removed the attribute from the current container for the category before adding it to the new one, this makes it deterministically set the correct attribute.

Type of change

• Bug fix (non-breaking change which fixes an issue)

How Has This Been Tested

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

• Tried before and after

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

0.0% Coverage
0.0% Duplication