Skip to content

Skip staff entries without names - #71

Merged
LinFor merged 1 commit into
LinFor:masterfrom
Weegley:fix/skip-empty-person-names
Jul 22, 2026
Merged

Skip staff entries without names#71
LinFor merged 1 commit into
LinFor:masterfrom
Weegley:fix/skip-empty-person-names

Conversation

@Weegley

@Weegley Weegley commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Summary

Skip Kinopoisk staff entries that do not contain either a Russian or English name.

Jellyfin rejects PersonInfo objects with an empty Name, which currently causes metadata refreshes to fail with:

The value cannot be an empty string. (Parameter 'person.Name')

Changes

  • Filter out staff entries with empty or whitespace-only names.
  • Keep valid Russian and English fallback names.
  • Preserve person sort order after filtering.
  • Add a regression test.

Testing

  • All 13 automated tests pass.

@LinFor
LinFor merged commit 78b0f03 into LinFor:master Jul 22, 2026
1 check passed
@Weegley
Weegley deleted the fix/skip-empty-person-names branch July 22, 2026 20:44
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.

2 participants