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

Auto-update names after any actress name change #32

Closed
JavLuv opened this issue Sep 26, 2022 · 0 comments
Closed

Auto-update names after any actress name change #32

JavLuv opened this issue Sep 26, 2022 · 0 comments
Assignees
Labels
enhancement New feature or request

Comments

@JavLuv
Copy link
Owner

JavLuv commented Sep 26, 2022

If the user changes an actress name or merges actress entries, automatically launch a background command to scan all movies and update names appropriately.

The actress database will always be considered the authority, so the names in movies will reflect whatever is listed there. The actor alias data will be changed to reflect the actress database as well. In the case of an actress entry merger, duplicate entries will be located and eliminated. It may be simplest if the command does not distinguish any particular actress entry and simply updates all movie entries, making note of which ones actually change so the .nfo files can be saved at the end of the command.

When should this occur? After the user finishes the edit? What if it's in multiple parts, like swapping main and alternate names? Maybe after closing the current detail view would be safer then? Maybe don't update if the change set the name to empty, as that's not considered valid. In the case of a name change -easy enough. Launch the update when you click Merge.

Alternatively, this could be manually triggered. Not sure I like that, as the least surprising thing would be for the actress names in movie listings to automatically sync up to what's listed in the detail pages, I think.

I think I like the idea of launching this process after you either explicitly trigger it with Merge, or when you navigate away from the current detail page. Will have to keep track of an edit flag specifically for names.

@JavLuv JavLuv added the enhancement New feature or request label Sep 26, 2022
@JavLuv JavLuv self-assigned this Sep 26, 2022
@JavLuv JavLuv closed this as completed Sep 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant