-
Notifications
You must be signed in to change notification settings - Fork 198
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Feature: Add librarySectionTitle for playlist items #1461
base: master
Are you sure you want to change the base?
Conversation
This was already present on Also: What about |
https://github.com/pkkid/python-plexapi/blob/master/plexapi%2Fvideo.py#L61 https://github.com/pkkid/python-plexapi/blob/master/plexapi%2Faudio.py#L74 Do those not work in playlists? |
This comment was marked as resolved.
This comment was marked as resolved.
I know |
Ok, misread, I see what you mean, Episode and Movie extend from Video class, so yes. they should work. |
Github mobile app doesn't do permalinks. |
There is one problem, while assigning I've used such trickery for now: librarySectionTitle = p.librarySectionTitle if p.__dict__.get("librarySectionTitle") is not None else None or (easier to the eye) librarySectionTitle = p._data.attrib.get("librarySectionTitle") |
Now that you bring it up, any action on a playlist item from a different server will fail (i.e. editing, playing, etc.) because of the incorrect |
yes. removeItems fails with 404 errors, as it loads from wrong server. |
building a proper |
Description
Similarly to #1335, add
librarySectionTitle
, this is present on local playlist entries.The
source
attribute should be onThe diff of local/remove items can be seen here:
Type of change
Please delete options that are not relevant.
Checklist: