[BUGFIX] Tiny refactoring of metadata handling with subentries

[markusweigelt]

• Ignore metadata replacement for subentries
• Remove metadata with empty values

Fixes #1308

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 0.00%. Comparing base (4629cf2) to head (8881d10).
Report is 1 commits behind head on main.

Additional details and impacted files

@@     Coverage Diff      @@
##   main   #1478   +/-   ##
============================
============================

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[sebastian-meyer]

@chrizzor Since I just asked you to prioritize #1308, could you please have a look at this pull request? Maybe this already fixes the issue.

[sebastian-meyer]

Did you test this with hierarchical as well as non-hierarchical metadata mappings?

Maybe I am confused, but as far as I understand, metadata values are always handled as arrays internally (because even non-hierarchical metadata fields can be multi-valued), so trying to distinguish hierarchical and non-hierarchical metadata by simply checking for data type "array" is not enough.

So, can you please confirm that this works for non-hierarchical, hierarchical, and multi-valued metadata fields?

[sebastian-meyer]

Scratch that! I just saw that you check for arrays inside the array of metadata values. My bad!