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

[Bug Report] Tagger used with multiple StashBoxes will lose Performer StashIDs #4212

Closed
scruffynerf opened this issue Oct 16, 2023 · 1 comment · Fixed by #4215
Closed

[Bug Report] Tagger used with multiple StashBoxes will lose Performer StashIDs #4212

scruffynerf opened this issue Oct 16, 2023 · 1 comment · Fixed by #4215
Labels
bug report Bug reports that are not yet verified

Comments

@scruffynerf
Copy link

scruffynerf commented Oct 16, 2023

If you use Performer Tagger, and scrape a performer, and the matched performer already exists with other StashIDs from other stashboxes, hitting the save will cause the loss of the other existing StashIDs, and only the new one is saved.

To Reproduce
Scrape a Performer using StashBox B, and if the Performer already has a StashID from Stashbox A...
It will match, and allow you to save the new StashID.
It then loses any existing StashIDs, and only keeps the new one.

Expected behavior
It should keep any existing stashids as well as add the new one.

Stash Version: (from Settings -> About):
Existed for a while, including up to v.23.1

Confirmed by multiple people.

Corrected by me, as I misunderstood the latest report.
I know @DingDongSoLong4 fixed the scene tagger issues in #4092
and pointed to this other similar issue as long standing (https://discord.com/channels/559159668438728723/559159910550732809/1152724118299496570)

@scruffynerf scruffynerf added the bug report Bug reports that are not yet verified label Oct 16, 2023
@DingDongSoLong4
Copy link
Collaborator

This also happens in the Performer Tagger (not the bulk performer tagger), when updating an existing performer.

A relevant bit of code:

performerData.stash_ids = [
{
endpoint,
stash_id: remoteSiteID,
},
];

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug report Bug reports that are not yet verified
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants