ZeroQI / Absolute-Series-Scanner

Seasons, absolute mode, Subfolders...
1.01k stars 155 forks source link

No files found on dir/dir when containing same string #198

Closed whinette closed 5 years ago

whinette commented 5 years ago

At least that's the only hunch I have right now.

=============================================================================================================================================================
Library: 'Anime', root: '/storage/SHOU/pub/plex/tv/Anime', path: 'Urusei Yatsura Collection', files: '0', dirs: '1'
Plex scan start: 2018-12-23 18:42:47,226996
plexignore_files: '[]', plexignore_dirs: '[]'
=============================================================================================================================================================
[folder] Urusei Yatsura Collection/Urusei Yatsura [anidb-377]
[no files detected]

After adding another folder:


=============================================================================================================================================================
Library: 'Anime', root: '/storage/SHOU/pub/plex/tv/Anime', path: 'Urusei Yatsura Collection/Kanketsu Hen [anidb-382]', files: '1', dirs: '0'
Manual scan start: 2018-12-23 19:07:18,671926
plexignore_files: '[]', plexignore_dirs: '[]'
=============================================================================================================================================================
[file] Urusei Yatsura Collection/Kanketsu Hen [anidb-382]/Urusei Yatsura Movie 05 ~ The Final Chapter [DVDRip 960x720 x264 AAC].mkv
=============================================================================================================================================================
Manual scan end: 2018-12-23 19:07:18,673471
=============================================================================================================================================================
Library: 'Anime', root: '/storage/SHOU/pub/plex/tv/Anime', path: 'Urusei Yatsura Collection/Urusei Yatsura [anidb-377]', files: '197', dirs: '0'
Manual scan start: 2018-12-23 19:07:22,488429
plexignore_files: '[]', plexignore_dirs: '[]'
=============================================================================================================================================================
[file] Urusei Yatsura Collection/Urusei Yatsura [anidb-377]/Urusei Yatsura - 0x01 - Urusei Yatsura All-Star All-Out Attack.mkv

Files are found. Pretty weird, uh?

Full log attached: Urusei Yatsura Collection.filelist.log

ZeroQI commented 5 years ago

Weird, it depends, as the scanner doesn't run again if it considers the folder hasn't changed... Reproduced the issue, not linked to folder named but single folder in grouping folder

EndOfLine369 commented 5 years ago

You can see form the above log clips it changes from a Plex scanned folder to a manual grouping folder scan.

Makes sense to me on this behavior. You should not really have a grouping folder for a single series. Those should all be in the library root.

ZeroQI commented 5 years ago

The scanner development in incredibly complex but we need it to be the most flexible possible

From the logs, the scanner was called for grouping folder then as no file present exited line 531, and not called by plex directly even though dirs had 1 folder...

If that is the case, and if a file in grouping folder allow detecting properly a single folder inside a grouping folder, then we can add a test for 1 folder in grouping folder situation and not return

whinette commented 5 years ago

This is not consistent, on some others folder I have this structure root/show/Season 1/files and the files are found. I may have time end of the week, will look into it then.

ZeroQI commented 5 years ago

season folders are taken out with regex so not really same situation but same folder depth.

ZeroQI commented 5 years ago

working on it but super tricky to resolve.

ZeroQI commented 5 years ago

Did you had time to test?

whinette commented 5 years ago

No, I did not see the notification. Will do this week!

whinette commented 5 years ago

Sorry for the delay. It works well. :)

ZeroQI commented 5 years ago

This took forever. Seem so easy with so little code changed but took weeks to figure out... Worst is, I am pretty sure it used to work...