Snd-R / komf

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

MangaDex provider to use series title #169

Closed xcymax closed 1 month ago

xcymax commented 1 month ago

As per #142 This behaviour of taking the first matching alternative title instead of the main one really confused me before digging deeper.

With

     postProcessing:
        seriesTitle: true
        seriesTitleLanguage: "en"
        fallbackToAltTitle: false

I expected it to use the en series title only but instead it ignores it and sets to the first en altTitle if it exists - even with fallbackToAltTitle: false

https://github.com/Snd-R/komf/issues/142#issuecomment-2041135950

Also Komf ignores main title completely and only takes altTitles since main title often tagged wrong.

I think if the metadata source is wrong then let it be wrong 😛

Anyway requesting this behaviour be reverted or at least configurable to use main titles again thanks.

Snd-R commented 1 month ago

Komf doesn't distinguish between main and alt titles. When doing an update it will take first matching title as main and treat the rest as alternative titles. Mangadex provider only takes alternative titles at the moment.

I've tried to see if anything changed regarding this issue on mangadex. I went through a dozen random manga including popular and new titles. Only 1/10 had correct language tag as ja-ro. Most of romanized titles still use en language tag I'll only reconsider this when most or at least half of the titles on Mangadex use correct language tag