fix: broken LinOps for entity and block NBT

[PierreSchwang]

Overview

Fixes #3414

Description

regression introduced in #3406
the special empty ops value seems to be kinda required and switching that to a compound broke several codecs. Switched it back to a EndTag and implemented the remaining mergeToMap methods. I don't really know how I got the error leading to the linked PR - I just know that some codec attempted to put the empty (LinEndTag) into a compound tag which LinBus obviously prevents and errors. I should've filtered all EndTags so they hopefully don't get put into any compounds. Couldn't get the error to reproduce on my original testing server at least.

And the test cases from the issue do work as well.

Submitter Checklist

• Make sure you are opening from a topic branch (/feature/fix/docs/ branch (right side)) and not your main branch.
• Ensure that the pull request title represents the desired changelog entry.
• New public fields and methods are annotated with @since TODO.
• I read and followed the contribution guidelines.

[Stoupy51]

Fix tested and working right! Lovely thank you

[DoogiePC]

The commit doesn't work with any coloured signs

[PierreSchwang]

The commit doesn't work with any coloured signs

How did you create the sign? I can't reproduce that with a sign that I right-clicked with blue dye and a glow ink sac

[DoogiePC]

The commit doesn't work with any coloured signs

How did you create the sign? I can't reproduce that with a sign that I right-clicked with blue dye and a glow ink sac

Just the usual way of &atest
normal Minecraft colour codes

[PierreSchwang]

those are not "normal" color codes nor is that possible without other plugins. share the /data get block <x> <y> <z> output for the colored sign got it

[DoogiePC]

/data get block

[PierreSchwang]

should be fixed now