OpenBagTwo / EnderChest

syncing and linking for all your Minecraft instances
https://openbagtwo.github.io/EnderChest/
GNU General Public License v3.0
3 stars 2 forks source link

Instance with no groups (or tags) will not link to a crafted shulker box #114

Closed OpenBagTwo closed 10 months ago

OpenBagTwo commented 10 months ago

Summary

A shulker box crafted with the default ("") tag-match will not* match an instance created without any groups or tags

Environment and Version

Steps to Reproduce

After onboarding an instance with no tags or groups (freshly created via PrismLauncher with no group assignment), going through the enderchest craft shulkerbox wizard:

Filters match the instance: Tagless
==> Do you wish to continue?
==> [Y/n] y
==> Tags?
==> e.g.vanilla-plus, multiplayer, modded, dev, april-fools
==> (or multiple using comma-separated lists or wildcards)
==> Note: tag-matching is not case-sensitive.
==> [*] 
Filters do not match any known instance.
==> Do you wish to continue?
==> [y/N] n
==> Tags?
==> e.g.vanilla-plus, multiplayer, modded, dev, april-fools
==> (or multiple using comma-separated lists or wildcards)
==> Note: tag-matching is not case-sensitive.
==> [*] *
Filters do not match any known instance.
==> Do you wish to continue?
==> [y/N]

where the entry in the enderchest.cfg looks like:

[Tagless]
root = instances/Tagless/.minecraft
minecraft-version = 1.20.1
modloader = Some Loader
groups = 
tags = 

Furthermore, running enderchest inventory minecraft Tagless returns the message:

The instance Tagless (instances/Tagless/.minecraft) does not link to any shulker boxes in this chest

even though I have "global" shulker boxes that should match

Workaround

Add a "placeholder" tag to the instance manually, in enderchest.cfg

Severity

Gamebreaking. In the case of a "global" shulker box, this could go completely missed until there were already a ton of conflicts.

Desired Outcome

Fix

Urgency

By next minor release (https://github.com/OpenBagTwo/EnderChest/milestone/6)

Notes

This issue is arising because there's no check here for instance.tags being empty:

https://github.com/OpenBagTwo/EnderChest/blob/6829ffae78386b939188ef814bd77465de0ebfe9/enderchest/shulker_box.py#L215-L222