Added config option keep-drops which allows players to keep items t…

[Fisher2911]

…hey find in the arena

Summary

• This is a…
  • Bug fix
  • [X ] Feature addition
  • Documentation
  • Refactoring
  • Minor / simple change (like a typo)
  • Other
• Describe this change in 1-2 sentences:

It adds the config option keep-drops which allows players to keep items from the arena after is is complete.

Problem

It is just an extra feature that could be helpful for people who want mob drops or other items to be kept.

• GitHub issue (optional):
  Config option to keep picked up items #750

Solution

I made it so that when the player joins the arena, PDC is set to all their items to identify them. Then, when the player leaves, the items with the PDC are removed from the inventory, and the rest of the items are stored. Then, after the player's inventory has been reset, the extra items are added back to their inventory.

Action

It would be helpful if you could review it to make sure I didn't miss any areas where players' inventories are cleared and restored.

[garbagemule]

Thanks for submitting an issue and a pull request!

I'm not entirely sure what the proper way to accommodate something like this would be, but I'd be happy to review the PR after a cleanup. There are way too many formatter-related changes that should not be included. The best approach is probably to disable all formatter automations in your IDE and change/add just the lines necessary for the implementation to work.

[garbagemule]

Closing this to clear out stale PRs. There's still way too much noise in this PR. The overall idea is fine, but the implementation is a bit too "creative", and the execution is messy. I think an easier way around this would be to build on top of the rewards system and just add drops as rewards when they get picked up, similar to how keep-exp works. Worth a try!