Snd-R / komf

Komga and Kavita metadata fetcher
MIT License
324 stars 19 forks source link

Komelia destroys Config File & other Komelia Issues #170

Open ElryGH opened 1 month ago

ElryGH commented 1 month ago

At first i had my config file like this, fully configured and nicely formatted. image (1)

But after opening Komelia it did this to my file: image

It completly threw away important info and changed the formatting.

Also Komelia did not show the configured stuff correctly in the interface, the fields for multiple settings were wrong. bookCovers, seriesCovers were off, even tho it was on. overrideExistingCovers was on, even tho it was set to off. All metadataProviders that were marked as false were not listed.

Also the metadataProviders missing from the file, nearly all of those were added as links to manga (so the default is on). This should not be the case, if an provider is not in the file it should count that proivder as false (off).

Snd-R commented 1 month ago

That's how config updates coming from api always worked even when using userscript. When doing config write all it does is serialization of AppConfig.kt content structure and overriding existing file. Default values are not encoded. It would be rather difficult to preserve original config structure, let alone keeping the comments If you want to keep your config structure you'd need to not update it through API I'll keep this issue open in case I find a way to solve this, but this won't be fixed for the time being