Latest v2.5.1 breaks some previous features

What happened ?

After the latest tagged version of v2.5.1, while trying to fix #36, I found that it broke some useful features.

The options which I found not working are:

1. --use-chapter-title
2. --no-group-name

Using the same Reproducible command, explained at #36,

mangadex-downloader "https://mangadex.org/title/296cbc31-af1a-4b5b-a34b-fee2b4cad542/-oshi-no-ko" --language "en" --cover "original" --use-compressed-image --start-chapter 1 --end-chapter 6 --save-as raw --use-chapter-title --no-group-name

If run, results somewhat like this:

Where the above mentioned command options are ignored by the app:

If I try to download “Chapter 11” from the manga, which has 2 scanlator uploads, both are being downloaded with scanlator’s name prefixed in the folder name, but chapter title is still ignored.

So it’s basically forcing --group all to be an absolute option as well. But selecting a single group with --group XXX is still operational.

So, I get Downloaded Folders (for raw format) as: <Scanlator Group/Uploader Name> <Volume> <Chapter>, which is unexpected.

What did you expect to happen ?

I should’ve got folders/files prefixed as <Volume> <Chapter> - <Title>, as it did in the previous tagged versions.

OS version

Ubuntu 22.04

App version

mangadex-downloader v2.5.1 (https://github.com/mansuf/mangadex-downloader)
Python: 3.10.6
arch: x64
bundled executable: no

Installation origin

PyPI (Python Package Index)

Installation origin (other sources)

No response

Reproducible command

mangadex-downloader "https://mangadex.org/title/296cbc31-af1a-4b5b-a34b-fee2b4cad542/-oshi-no-ko" --language "en" --cover "original" --use-compressed-image --start-chapter 1 --end-chapter 6 --save-as raw --use-chapter-title --no-group-name

And for testing the chapter download with dual scanlator (chapter 11)

mangadex-downloader "https://mangadex.org/title/296cbc31-af1a-4b5b-a34b-fee2b4cad542/-oshi-no-ko" --language "en" --cover "original" --use-compressed-image --start-chapter 10 --end-chapter 12 --save-as raw --use-chapter-title --no-group-name

Additional context

No response

[mansuf]

Released in v2.5.2

[mansuf]

Side note: While figuring out this issue, if you could set the app to skip/ignore chapter links which redirects to non-mangadex links would be helpful (those are basically Licensed Uploader’s website links). I don’t know how the mangadex API works, but if there is a way to filter the links before downloading them, I’d appreciate if you could implement it too in the near future.

This has been implemented in v2.5.1. The app will look for images not referral links. If the chapter doesn’t have images it will simply skipped and will be not downloaded