ZeroQI / Absolute-Series-Scanner

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

First letter of a series title gets removed #6

Closed WastedMeerkat closed 8 years ago

WastedMeerkat commented 8 years ago

For example, "Charlotte" becomes "harlotte", "Overlord" becomes "verlord". I have no idea why it's happening, and it only seems to be on some things with no clear pattern as far as I can tell. Tried taking a look at the python source but this definitely looks like something more easily solved by the original author, heh.

ZeroQI commented 8 years ago

Happens with alpha release in gits? Can you post custom log entries? I need to reproduce the issue.

WastedMeerkat commented 8 years ago

I'm actually not sure how to get the custom logs working, but here's the default log in the meantime: https://dl.dropboxusercontent.com/u/4072938/Text%20Files/Plex%20Media%20Scanner.log

This line sticks out to me:

Nov 03, 2015 18:27:38:557 [11168] DEBUG - Hint show didn't match ('iamond no Ace' != 'Diamond No Ace') for DB media item 29180

Perhaps there's something going wrong when it looks up the title in AniDB or something?

And there's also a different error like this:

Nov 03, 2015 18:27:32:503 [11168] ERROR - Error in Python: Running scanner: Traceback (most recent call last): File "D:\Users\WastedMeerkat\AppData\Local\Plex Media Server\Scanners\Series\Absolute Series Scanner.py", line 383, in Scan add_episode_into_plex(mediaList, files, file, root, path, show, season, int(ep), title, year, int(ep2) if ep2 and ep2.isdigit() else None, rx, tvdb_mapping) File "D:\Users\WastedMeerkat\AppData\Local\Plex Media Server\Scanners\Series\Absolute Series Scanner.py", line 203, in add_episode_into_plex if not keep_zero_size_files and str(os.path.getsize(file))=="0": return #do not keep dummy files by default unless this file present in Logs folder File "C:\Program Files (x86)\Plex\Plex Media Server\python27.zip\genericpath.py", line 49, in getsize return os.stat(filename).st_size WindowsError: (3, 'The system cannot find the path specified', "D:\User backup\Downloads\uTorrent\Completed.Anime\2015-2 Spring\Danna ga Nani wo Itteiru ka Wakaranai Ken 2 Sure-me[Kaitou] Danna ga Nani wo Itteiru ka Wakaranai Ken 2 Sure-me (I Can't Understand What My Husband Is Saying 2nd Thread) - 01 [720p][10bit][811C80FE].mkv")

Here's a list of the shows that are having their titles trimmed: Assassination Classroom Charlotte Classroom Crisis Danna ga Nani wo Itteiru ka Wakaranai Ken Diamond no Ace Durarara!!x2 Shou Gangsta God Eater Jitsu wa Watashi wa Overlord Ranpo Kitan - Game of Laplace Rokka no Yuusha Ushio to Tora Working!!!

Cephia commented 8 years ago

same thing happening here, log below

log.docx

ZeroQI commented 8 years ago

what is wrong with this path: "D:\User backup\Downloads\uTorrent\Completed.Anime\2015-2 Spring\Danna ga Nani wo Itteiru ka Wakaranai Ken 2 Sure-me[Kaitou] Danna ga Nani wo Itteiru ka Wakaranai Ken 2 Sure-me (I Can't Understand What My Husband Is Saying 2nd Thread) - 01 [720p][10bit][811C80FE].mkv" ?

Does the issue happens with the alpha scanner https://gist.github.com/ZeroQI/26d26f91af04c876b1a0 ?

WastedMeerkat commented 8 years ago

Wait, it should be "D:\User backup\Downloads\uTorrent\Completed\.Anime\". Where'd you copy that path from? Also, I'm not sure, I'll try it out later.

Oh wait a second, could regular expressions be reading that as an escaped period somehow?

ZeroQI commented 8 years ago

Does the issue happens with the alpha scanner https://gist.github.com/ZeroQI/26d26f91af04c876b1a0 ?

Xalaxis commented 8 years ago

The same issue does not occur with the alpha scanner for me, however it picks up my "Extras" Folders as a season which niggles me more than just a little bit. I'm happy to provide any logs you require.

ZeroQI commented 8 years ago

Give me all names of folders that are seen as season so i can reproduce and solve. Glad the alpha solve some bugs for you. Will correct the rest On 15 Jan 2016 20:56, "Matthew Wolffsohn" notifications@github.com wrote:

The same issue does not occur with the alpha scanner for me, however it picks up my "Extras" Folders as a season which niggles me more than just a little bit. I'm happy to provide any logs you require.

— Reply to this email directly or view it on GitHub https://github.com/ZeroQI/Absolute-Series-Scanner/issues/6#issuecomment-172090059 .

Xalaxis commented 8 years ago

"Extras" and "TV Preview" (Which should be inside the "Extras" folder, so you mightn't want to blacklist it). Many thanks for the quick response!

Xalaxis commented 8 years ago

capture

ZeroQI commented 8 years ago

Can you attach scanner logs by any chance?

Cephia commented 8 years ago

Thanks Zero, alpha release fixed the missing letters just fine.

also seeing Extras marked out in its own folder here

Xalaxis commented 8 years ago

Plex Media Scanner (custom ASS) - Anime (Temporary work-around).txt I think that's the right one?

Xalaxis commented 8 years ago

Okay, so this is a little strange. I just refreshed the content for the library and got a new log that stated that it was 'Ignoring extras' (Which didn't change the library showing extras) And also removed the first letter from each show again in the log (whilst still using the alpha scanner) But did not affect the current items in the library. Plex Media Scanner (custom ASS) - Anime.txt

I'm going to recreate the library from scratch and send a clean log for you

Xalaxis commented 8 years ago

Ah, it seems to have joined all the scans I did together. Here's a clean scan (although it starts near the end of the complete log, apologies) Plex Media Scanner (custom ASS) - Anime.txt

ZeroQI commented 8 years ago

I see the issue, and i think i have a fix. Line 215 replace "subdir" with "os.path.basename(subdir)":

for rx in ignore_dirs_rx: # if initial scan and root folder if re.match(rx, os.path.basename(subdir), re.IGNORECASE): subdirs.remove(subdir); Log("\"%s\" match ignore_dirs_rx: \"%s\"" % (path, rx)); break #skip dirs to be ignored

Xalaxis commented 8 years ago

Made the appropriate change - Testing now.

Xalaxis commented 8 years ago

I can confirm this fixes the issue :smile: