#musicbrainz-picard-development

/

      • outsidecontext[m
        João Sousa: yes, I think preserving the case would be the right thing. But actually there is already an implementation for this in MP4 and ID3
      • Check the code for `cademap`. It stores the original casing on loading and uses it again on saving.
      • That means please carefully investigate where the actual problem lies
      • I suspect that it is not working for tags with defined mapping, like label, and only applies to freeform tags unknown to picard
      • Thinking about this there could also be an argument made for changing the casing for defined tags. Not fully sure. Do we maybe need an option?
      • Reading case insensitive definitely makes sense, though
      • Protopia[m] has quit
      • stevilknevil[m] has quit
      • JooSousa[m]
        <outsidecontext[m> "Check the code for `cademap`. It..." <- Correct, the code does store the case in __casemap when loading, however when saving, if it is a label already mapped it saves to the mapped case. In the solution im working on, a label is checked if it exists in __casemap before the "default mappings" checks occur
      • outsidecontext[m
        Just make sure this only happens for tags that are stored as free text tags
      • JooSousa[m]
        <outsidecontext[m> "Just make sure this only happens..." <- you mean freeform tags?
      • BobSwift[m] has quit
      • aerozol[m] has quit