luvies / TaledensInvManagerUpdated

The code base for my unofficial update of Taleden's Inventory Manager
25 stars 28 forks source link

Add custom data tagging support #5

Open luvies opened 6 years ago

luvies commented 6 years ago

I plan to add support for using the custom data of blocks to add tags. This is how I plan for it to work currently:

pckroon commented 6 years ago

Item tags in the name will still work, and will override anything in the custom data

This is going to be tricky. What's the outcome of the following? Name: [TIM components:200] Custom: SteelPlates:1000

luvies commented 6 years ago

Hmm. I will have to sort out the levels of priorities. While it makes sense to give the SteelPlates priority for being more explicit, it also makes sense to give the Component priority since the name tag feel like it should be more important.

I think that maybe the best resolution is to treat name tags as a complete override of the custom data, which in this case would mean the Component would win out. The reason for this, is that the names feel like more explicit and more of a short-term system than custom data. It also means that whatever is in the name is reflected exactly in the inventory, with the custom data only adding to it rather than modifying it.

Flow86 commented 6 years ago

that would be awesome to have the information in custom (too?) since it will be saved into blueprints