search

DemiCloud / universal-media-metadata

MIT License
0 stars 0 forks source link

add dynamic mapping to simplify schema to template mapping #3

Closed Gorian closed 1 year ago

Gorian commented 1 year ago

right now, the schema to template mapping is a simple 1:1. This is inconvenient as any change of scheme means an addition to the mapping, even if if the template doesn't have to change. So you could theoretically end up with this:

{
  "opf":
  {
    "0.0.1": "0.0.1",
    "0.0.2": "0.0.1",
    "0.0.3": "0.0.1",
    "0.0.4": "0.0.1",
    "0.0.5": "0.0.1",
    "0.0.6": "0.0.2",
    "0.0.7": "0.0.2"
  }
}

We should instead add some logic to just check if there's any updated mapping, so this would be a valid (and equivalent) mapping object:

{
  "opf":
  {
    "0.0.1": "0.0.1",
    "0.0.6": "0.0.2",
  }
}
Gorian commented 1 year ago

Resolved by https://github.com/DemiCloud/universal-media-metadata/commit/9095dae5f099c66a60ab578afc43db55bafd5a51