metafacture / metafacture-core

Core package of the Metafacture tool suite for metadata processing.
https://metafacture.org
Apache License 2.0
71 stars 34 forks source link

Add duplicate-Warning to field-merger #359

Closed TobiasNx closed 2 years ago

TobiasNx commented 3 years ago

Object keys in JSON should be unique but the do not have to. Since that recommendation allows duplicate object keys they are not invalid. But a duplicate can hint to an error in an metafacture morph/fix.

For oersi @fsteeg has build a fieldMerger-module that merges dupilcates, this module could be used to highlight the duplicates too and return a Warning.

TobiasNx commented 3 years ago

Otherwise this could be just an option in the JSON Encoder, that returns a warning if duplicate fields exist. Perhaps this is also anachronistic if FIX is in fully working in record mode.

TobiasNx commented 2 years ago

since moving to new fix this is not needed anymore