sephrat 16 Posted October 26, 2022 Posted October 26, 2022 I have a TV library with two folders. Option "Automatically merge series that are spread across multiple folders" is ticked. I have a series that's spanned across two folders in the same library. That series appears only once when querying /Items. My issue is that internally, Emby has two series ID for that series, as can be seen on the episodes: their SeriesId differ. When querying for the series list, one of those IDs is hidden and we only see the other one in Emby response. My request to you would be to provide a new option to not merge series so that every individual series shows up in the response. Current behaviour: { "Items": [ { "Id": "A", "Path": "/path/1", "Type": "Series", .... } ], } Requested behavior with this additional option: { "Items": [ { "Id": "A", "Path": "/path/1", "Type": "Series", .... }, { "Id": "B", "Path": "/path/2", "Type": "Series", .... }, .... ], } Is this possible? Another accepted alternative would be that you expose the merged IDs on the Series entity so we know all possible IDs of a merged series. { "Items": [ { "Id": "A", "Type": "Series", "MergedIds": [ "B", "C" ], .... } ], } On a side note, querying /Items with additional filter Ids=B does return the series (with correct ID). { "Items": [ { "Id": "B", "Path": "/path/2", "Type": "Series", .... } ], } But removing that filter hides that series ID.
Luke 42077 Posted October 26, 2022 Posted October 26, 2022 Quote Requested behavior with this additional option: { "Items": [ { "Id": "A", "Path": "/path/1", "Type": "Series", .... }, { "Id": "B", "Path": "/path/2", "Type": "Series", .... }, .... ], } HI, wouldn't this prevent the feature from working entirely?
Luke 42077 Posted October 26, 2022 Posted October 26, 2022 Quote Another accepted alternative would be that you expose the merged IDs on the Series entity so we know all possible IDs of a merged series. { "Items": [ { "Id": "A", "Type": "Series", "MergedIds": [ "B", "C" ], .... } ], } Yes this makes more sense. 1
sephrat 16 Posted October 27, 2022 Author Posted October 27, 2022 12 hours ago, Luke said: HI, wouldn't this prevent the feature from working entirely? Not if you add an option such as "UnmergeSeries" to have that new behaviour, as to not break current API behaviour. Thank you for looking into it.
Luke 42077 Posted October 27, 2022 Posted October 27, 2022 53 minutes ago, sephrat said: Not if you add an option such as "UnmergeSeries" to have that new behaviour, as to not break current API behaviour. Thank you for looking into it. Sure, yes. Not having an unmerge button is a bit of a hole in the current behavior.
sephrat 16 Posted October 27, 2022 Author Posted October 27, 2022 Sorry but just to be sure there's no misunderstanding, I'm not talking about a button but an additional option to the Items endpoint. So that /Items keeps returning the same behaviour { "Items": [ { "Id": "A", "Path": "/path/1", "Type": "Series", .... } ], } But with the new option, /Items?UnmergeSeries=True returns: { "Items": [ { "Id": "A", "Path": "/path/1", "Type": "Series", .... }, { "Id": "B", "Path": "/path/2", "Type": "Series", .... }, .... ], }
horstepipe 422 Posted October 27, 2022 Posted October 27, 2022 @Luke the latest beta release note Quote Fix regressions with tv series folder parsing is not about this issue, is it? Best regards
Luke 42077 Posted October 27, 2022 Posted October 27, 2022 3 hours ago, horstepipe said: @Luke the latest beta release note is not about this issue, is it? Best regards No it's not. 1
horstepipe 422 Posted November 3, 2022 Posted November 3, 2022 @TeamB may I ask how you solve that problem in the Emby4Kodi plug-in? best regards
TeamB 2438 Posted November 3, 2022 Posted November 3, 2022 6 minutes ago, horstepipe said: @TeamB may I ask how you solve that problem in the Emby4Kodi plug-in? best regards emby4kodi is @quickmic
quickmic 1703 Posted November 3, 2022 Posted November 3, 2022 (edited) 2 hours ago, horstepipe said: @TeamB may I ask how you solve that problem in the Emby4Kodi plug-in? best regards e4k always merges series/seasons located on different folders. It uses the PresentationUniqueKey for detection. This key is similar even if the IDs are different. Edited November 3, 2022 by quickmic
horstepipe 422 Posted November 3, 2022 Posted November 3, 2022 45 minutes ago, quickmic said: e4k always merges series/seasons located on different folders. It uses the PresentationUniqueKey for detection. This key is similar even if the IDs are different. @sephrat Would that be possible for Ombi or will this need a bigger development effort?
adminExitium 355 Posted November 4, 2022 Posted November 4, 2022 On 26/10/2022 at 23:43, Luke said: Yes this makes more sense. I wonder if it would be better to add PresentationUniqueKey as a filter for the Items endpoint and it can return all Item objects that have the same key. Seeing something like MergedIds within an Item object seems weird somehow.
horstepipe 422 Posted November 16, 2022 Posted November 16, 2022 (edited) @Luke any chance that this will be added in this beta circle? This is mandatory for Ombi to work properly if one uses multiple tv show directories with the same show. Best regards Edited November 16, 2022 by horstepipe
Luke 42077 Posted November 16, 2022 Posted November 16, 2022 I don't have an ETA, sorry, but it's certainly possible. 1
horstepipe 422 Posted November 16, 2022 Posted November 16, 2022 38 minutes ago, Luke said: I don't have an ETA, sorry, but it's certainly possible. Thanks. So I expect this is going to take awhile. Could you or anybody else provide a method to find all shows whose seasons/episodes are in two or more folders so I can manually merge them? Doing this completely by hand would be a pita.
horstepipe 422 Posted May 16, 2023 Posted May 16, 2023 On 11/16/2022 at 6:35 PM, Luke said: I don't have an ETA, sorry, but it's certainly possible. anything new here?
Luke 42077 Posted May 16, 2023 Posted May 16, 2023 5 hours ago, horstepipe said: anything new here? I don't have an update yet, sorry. 1
Recommended Posts
Create an account or sign in to comment
You need to be a member in order to leave a comment
Create an account
Sign up for a new account in our community. It's easy!
Register a new accountSign in
Already have an account? Sign in here.
Sign In Now