zlainsama / CosmeticArmorReworked

This mod allows you to wear two sets of armor, one for display, one for function.
https://minecraft.curseforge.com/projects/cosmetic-armor-reworked
Other
17 stars 18 forks source link

Disappearing Armor 1.12.2 #39

Closed The-Artificer closed 4 years ago

The-Artificer commented 4 years ago

If a player is wearing armor in their armor slots, and then hides it, not only will it disappear visually initially, but eventually it will be irretrievably deleted. The regular armor slots don't seem to always drop properly on death, and they will often be deleted on relog to the server while hidden as well. The version used is [CosmeticArmorReworked-1.12.2-v4a.jar]. This occurs with any armor, including vanilla armors. It should additionally be noted that both Armourer's Workshop and Moreplayermodels are present in the modpack as well as CosmeticArmor. This issue has been encountered by multiple players with no identifiable common thread besides the mod and modpack itself.

The-Artificer commented 4 years ago

I have realized that this issue could possibly simply be due to durability running out on the items without players knowing and the item breaking. I will test this theory with spawned unbreakable armor to see if the issue is still replicable.

zlainsama commented 4 years ago

If you put armors in your regular armor slots, they are still being used normally even if you hide them. The cosmetic slots added by my mod are separate from your regular armor slots, and the visual effect of having different armors(or hiding them) on your character using those slots is actually only taking place on the client side, other mods and the server itself shouldn't even notice anything unusual unless they are specifically looking for them through a special API provided by my mod.

The-Artificer commented 4 years ago

I have conducted some further testing, and have encountered multiple strange issues with cosmetic armor's intended functionality. The specific modfile is "CosmeticArmorReworked-1.12.2-v4a.jar". The issues themselves appear to be as follows:

  1. Regular armor slots while hidden not only do not display on the player(As should happen), but additionally disappear within the inventory as well. I understand this is a clientside visual display matter, but it is very confusing and misleading for users, and also means that people can't see the current durability on their armor.
  2. Equipped hidden armor is always deleted from regular armor slots when switching between survival and creative modes. In fact, creative mode armor slots simply eat and won't store or save any armor placed into them whatsoever while hidden. These items cannot be retrieved.

To be clear, all issues mentioned here are purely in regards to vanilla armor slots, the only difference is them being hidden. None of these issues take place if the armor is not hidden, and to further clarify, all cosmetic slots themselves are empty.

zlainsama commented 4 years ago

Does this happen in singleplayer or a server? If this happens in singleplayer, can you find out what other mods are causing incompatibility? As it works fine on it's own. If this happens only on a server, is the server a normal forge server? Or others like spongeforge? Does this happen on a normal forge server? If not, can you find out what plugins are causing incompatibility?

The-Artificer commented 4 years ago

It happens on a normal forge server. A private one for my friends, using this list of mods: https://gist.githubusercontent.com/The-Artificer/efb774980ba6ff219eb15bcc802a83eb/raw/32581c38edbb6583e3cc98d6140d83f94ea139ac/modlist.txt

On Sun, Feb 16, 2020 at 2:50 PM Lain MultipleInstances < notifications@github.com> wrote:

Does this happen in singleplayer or a server? If this happens in singleplayer, can you find out what other mods are causing incompatibility? As it works fine on it's own. If this happens only on a server, is the server a normal forge server? Or others like spongeforge? Does this happen on a normal forge server? If not, can you find out what plugins are causing incompatibility?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/zlainsama/CosmeticArmorReworked/issues/39?email_source=notifications&email_token=ALDDUOFMCS42X572E3NT6Z3RDGYMFA5CNFSM4KV4K2N2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEL4TQ5I#issuecomment-586758261, or unsubscribe https://github.com/notifications/unsubscribe-auth/ALDDUOB6BHSUE7ZBRAENQRLRDGYMFANCNFSM4KV4K2NQ .

zlainsama commented 4 years ago

I have found the cause, it happens because Armourer's Workshop is doing the same thing as my mod, I might be able to fix it by altering some event handlers order.

zlainsama commented 4 years ago

Fixed in 1.12.2-v4b.