michael.heier/jellyfin
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #948 from cvium/revert_movie_matching

Browse Source 
Revert movie matching
This commit is contained in:
Bond-009 2019-02-19 16:31:01 -05:00 committed by GitHub
commit d978ae1996
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 33 additions and 47 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

80
Emby.Naming/Video/VideoListResolver.cs
View File

@ -175,64 +175,50 @@ namespace Emby.Naming.Video
return videos; return videos;
} }
var list = new List<VideoInfo>();
var folderName = Path.GetFileName(Path.GetDirectoryName(videos[0].Files[0].Path)); var folderName = Path.GetFileName(Path.GetDirectoryName(videos[0].Files[0].Path));
if (!string.IsNullOrEmpty(folderName))
if (!string.IsNullOrEmpty(folderName) && folderName.Length > 1)
{ {
var videosMatchingFolder = new List<VideoInfo>(); if (videos.All(i => i.Files.Count == 1 && IsEligibleForMultiVersion(folderName, i.Files[0].Path)))
foreach (VideoInfo video in videos)
{ {
// Only interested in single files if (HaveSameYear(videos))
if (video.Files.Count != 1)
{ {
continue; var ordered = videos.OrderBy(i => i.Name).ToList();
}
if (string.Equals(folderName, video.Name, StringComparison.OrdinalIgnoreCase)) list.Add(ordered[0]);
{
videosMatchingFolder.Add(video);
}
// Eg. My Movie == My Movie - Some Other Info, TODO doesn't seem like a robust test
else if (video.Name.StartsWith(folderName, StringComparison.OrdinalIgnoreCase) &&
video.Name.Substring(folderName.Length).TrimStart().StartsWith("-"))
{
videosMatchingFolder.Add(video);
}
}
// It is assumed that any non-matching files are random samples, trailers, extras etc. list[0].AlternateVersions = ordered.Skip(1).Select(i => i.Files[0]).ToList();
// So if there's at least one video file matching the folder name, skip the rest. list[0].Name = folderName;
if (videosMatchingFolder.Count > 0) list[0].Extras.AddRange(ordered.Skip(1).SelectMany(i => i.Extras));
{
var primary = videosMatchingFolder[0];
var remainingVideos = videosMatchingFolder.Skip(1);
var videoInfo = new VideoInfo
{
Name = folderName,
Year = primary.Year,
Files = primary.Files,
AlternateVersions = new List<VideoFileInfo>(),
Extras = primary.Extras
};
foreach (VideoInfo video in remainingVideos)
{
videoInfo.AlternateVersions.Add(video.Files.First());
videoInfo.Extras.AddRange(video.Extras);
}
return new[] { videoInfo }; return list;
}
} }
} }
return videos.GroupBy(v => new { v.Name, v.Year }).Select(group => new VideoInfo return videos;
}
private bool HaveSameYear(List<VideoInfo> videos)
{
return videos.Select(i => i.Year ?? -1).Distinct().Count() < 2;
}
private bool IsEligibleForMultiVersion(string folderName, string testFilename)
{
testFilename = Path.GetFileNameWithoutExtension(testFilename) ?? string.Empty;
if (testFilename.StartsWith(folderName, StringComparison.OrdinalIgnoreCase))
{ {
// Because of the grouping, we can grab the information from the first movie and make it primary testFilename = testFilename.Substring(folderName.Length).Trim();
// The remaining movie matches are 'alternate versions' return string.IsNullOrEmpty(testFilename) ||
Name = group.First().Name, testFilename.StartsWith("-") ||
Year = group.First().Year, string.IsNullOrWhiteSpace(Regex.Replace(testFilename, @"\[([^]]*)\]", string.Empty)) ;
Files = group.First().Files, }
AlternateVersions = group.Skip(1).Select(i => i.Files[0]).ToList(),
Extras = group.First().Extras.Concat(group.Skip(1).SelectMany(i => i.Extras)).ToList() return false;
});
} }
private List<VideoFileInfo> GetExtras(IEnumerable<VideoFileInfo> remainingFiles, List<string> baseNames) private List<VideoFileInfo> GetExtras(IEnumerable<VideoFileInfo> remainingFiles, List<string> baseNames)