Skip to content
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

Actor editing #510

Draft
wants to merge 10 commits into
base: master
Choose a base branch
from
Draft

Actor editing #510

wants to merge 10 commits into from

Conversation

blacktwin
Copy link
Collaborator

Addressing Issue #462

addActor() and removeActor() methods are now added to video.Movie and video.Show.

Copy link
Collaborator

@JonnyWong16 JonnyWong16 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Question: Does adding an actor with the same name that already exists in a different movie use the same role id and filter?

plexapi/video.py Outdated Show resolved Hide resolved
plexapi/video.py Outdated Show resolved Hide resolved
@blacktwin
Copy link
Collaborator Author

Question: Does adding an actor with the same name that already exists in a different movie use the same role id and filter?

Yes.

@blacktwin
Copy link
Collaborator Author

blacktwin commented Jun 6, 2020

addActor() can also be used to edit any existing actor's thumb or role. The name is the only required argument to make changes. name and whatever other attribute someone would want to change.

index argument is an assumption on my part. I was able to create a new actor and their position in the webUI was not first. Not 100% if it was as commanded or if Plex ordered for me. More testing for the index is needed.

@blacktwin blacktwin marked this pull request as draft June 6, 2020 18:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants