mirror of
https://github.com/advplyr/audiobookshelf.git
synced 2026-05-30 23:40:40 +02:00
[Bug]: Author/Series with non-ASCII names are listed once for every book when updated #1681
Closed
opened 2026-04-24 23:54:36 +02:00 by adam
·
9 comments
No Branch/Tag Specified
master
book_tags_genres_dedupe
episode_download_fallback
Issue-4540-SortBy-StartedDate-and-FinishedDate
episode_meta_tagging
fix_authorize_race_condition
redirect_transcode_requests
progress_updated_sort
fix_ereader_socket_event
fix_change_empty_root_password
fix_podcast_session_track_index
fix_set_token
session_modal_user
localize_durations
fix_oidc_create_user
jwt_auth_refactor
fix_scanner_deleting_single_file_books
fix_mediaprogress_updatedat_2
experimental_next_client
podcast_episode_duration
episode-timestamps-clickable
book_author_secondary_sort_title
podcast_useragents
pathexists_user_access
fix_pathexists_join
book_author_secondary_sort
clean_duplicate_mediaprogress
sanitize_html_description
trix_prevent_attachments
check_path_api_fix
fix_mediaprogress_updatedat
increase_express_json_limit
fix_dockerfile_nunicode
search_episodes
audiobook_tools_update
episode_secondary_sorts
hls_stream_url_update
new_session_track_endpoint
audiobook_tools_enhancements
watcher_rescans_update
player_track_tooltip
fix_exclude_prefixes_crash
socket_item_events
fix_podcast_episode_scanner_promise
new_stats_controller
count_cache_for_userpermissions
parsing-opf-v3
validate_migration_files
fix-quick-match-all-crash
fix-chapter-end-sleep-timer
stringify_sequelize_query
remove-col-ambiguity
fix_next_prev_edit_description
details_trim_whitespace
fix_content_url_basepath
fix_logger_fatal
progress_bar_visibility
batch-edit-populate-map-details
feed_generator_updates
bookmark-modal-updates
migrate-library-item-in-scanner
migrate-new-library-items
migrate-podcasts-new-library-item-2
migrate-podcasts-new-library-item
fix-remove-episode-from-playlist
playback-session-use-new-library-item
refactor-library-item
fix-heatmap-caption
feed-episodes-upsert
share-media-player-media-session-api
remove-old-playlist
remove_old_collection_object
plugin-implementation-demo
feed_migration
refactor-feeds-from-item
fix_remove_authors_no_books
v2.17.3-fk-constraints-migration
migrations-first-upgrade
sqlite_2
feature/nuxt-target-server
waveform
sqlite
playlists
video
v2.35.1
v2.35.0
v2.34.0
v2.33.2
v2.33.1
v2.33.0
v2.32.1
v2.32.0
v2.31.0
v2.30.0
v2.29.0
v2.28.0
v2.27.0
v2.26.3
v2.26.2
v2.26.1
v2.26.0
v2.25.1
v2.25.0
v2.24.0
v2.23.0
v2.22.0
v2.21.0
v2.20.0
v2.19.5
v2.19.4
v2.19.3
v2.19.2
v2.19.1
v2.19.0
v2.18.1
v2.18.0
v2.17.7
v2.17.6
v2.17.5
v2.17.4
v2.17.3
v2.17.2
v2.17.1
v2.17.0
v2.16.2
v2.16.1
v2.16.0
v2.15.1
v2.15.0
v2.14.0
v2.13.4
v2.13.3
v2.13.2
v2.13.1
v2.13.0
v2.12.3
v2.12.2
v2.12.1
v2.12.0
v2.11.0
v2.10.1
v2.10.0
v2.9.0
v2.8.1
v2.8.0
v2.7.2
v2.7.1
v2.7.0
v2.6.0
v2.5.0
v2.4.4
v2.4.3
v2.4.2
v2.4.1
v2.4.0
v2.3.5
v2.3.4
v2.3.3
v2.3.2
v2.3.1
v2.3.0
v2.2.23
v2.2.22
v2.2.21
v2.2.20
v2.2.19
v2.2.18
v2.2.17
v2.2.16
v2.2.15
v2.2.14
v2.2.13
v2.2.12
v2.2.11
v2.2.10
v2.2.9
v2.2.8
v2.2.7
v2.2.6
v2.2.5
v2.2.4
v2.2.3
v2.2.2
v2.2.1
v2.2.0
v2.1.5
v2.1.4
v2.1.3
v2.1.2
v2.1.1
v2.1.0
v2.0.24
v2.0.23
v2.0.22
v2.0.21
v2.0.20
v2.0.19
v2.0.18
v2.0.17
v2.0.16
v2.0.15
v2.0.14
v2.0.13
v2.0.12
v2.0.11
v2.0.10
v2.0.9
v2.0.8
v2.0.7
v2.0.6
v2.0.5
v2.0.4
v2.0.3
v2.0.2
v2.0.1
v1.7.2
v1.7.1
v1.7.0
v1.6.0
v1.5.5
v1.5.0
v1.4.11
v1.4.9
v1.4.7
v1.4.6
v1.4.4
v1.4.2
v1.4.0
v1.4.1
v1.3.4
v1.3.3
v1.3.1
v1.2.8
v1.2.6
v1.2.5
v1.2.4
v1.2.1
v1.1.15
v1.1.14
v1.1.13
v1.1.12
v1.1.11
v1.1.10
v1.1.9
v1.1.8
v1.0.0
0.9.61-beta.0
0.9.61-beta
Labels
Clear labels
authentication
backlog
bug
chapter editor
config-issue
ebooks
encoding/embedding
enhancement
help wanted
listening sessions & progress
planned
possible plugin
progress sync
pull-request
sorting/filtering/searching
unable to reproduce
upload
users & permissions
waiting
Mirrored from GitHub Pull Request
No Label
bug
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
adam (Adam Melkus)
Clear assignees
No Assignees
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: starred/audiobookshelf#1681
Reference in New Issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @thatguy7 on GitHub (Jan 21, 2024).
Describe the issue
When the author's name is in cyrillic letters, it seems like ABS does not reliably process it.
As can be seen, e.g. I have 10 books by Анджей Сапковский (Andrzej Sapkowski, red), but instead of listing the author once with 10 books, the author is listed 10 times, once for each book.
I set the author's name using multple-selection and mapping the authors field, just to be sure it's the same for all of them.
When I changed the author Василий Маханенко (Vasily Mahanenko, blue) to the latin-lettered version, all his books were linked to a single author entry.
Interestingly, this bug does not seem to always happen - there clearly are multiple authors with more than 1 book in the screenshot.
The books by Sapkowski and Mahanenko are a single series each, wheres the books by Dostoevsky (green) are not part of a series.
Steps to reproduce the issue
UPDATE:
This error occurs only, when the tag is edited post import in ABS:
Pick any number of books and add the same cyrillic name to the authors tag, it will create a new author for every book.
Audiobookshelf version
v2.7.2
How are you running audiobookshelf?
Docker
@nichwall commented on GitHub (Jan 21, 2024):
Have you verified that every author is identical and there are no differences in the Unicode or hidden characters?
This issue came up a few months ago in Discord, and was resolved once the user fixed the Unicode.
"Might be that ABS handles that fine, but й (U+0439) and й (U+0306 and U+0438) look the same for example"
Beginning of conversation: https://discord.com/channels/942908292873723984/942908295579062335/1176217664918663229
@thatguy7 commented on GitHub (Jan 21, 2024):
Thanks for pointing me to the conversation, I will report back once I had a look.
@nichwall commented on GitHub (Jan 21, 2024):
One way you can check is by comparing the
metadata.jsonfor the library items. That will show you what ABS is using, depending on whether you got author names from metadata tags, the filesystem, or an online match, or a combination of sources@thatguy7 commented on GitHub (Jan 22, 2024):
I checked the metadata.json. They seem to be the same for all of the books.
This is the й (U+0439), if I use the other one, it does not find the files. Same goes for the other replicated entries.
As was discussed in the linked conversation on Discord I can fix this issue by writing the author tag to file, removing the items from my library and rescanning.
This means the problem occurs when I use ABS to set the author - when it is read on import things are fine. This explains why it worked for some authors, but not for all: If the tag was set correctly on import, it works fine. If I modify the value after import, it does not. Here it does not matter wether I edit the tag to the same value for all books or not - if the name is cyrillic, it will create a new author every time. If I put a latin name, it works as expected.
@thatguy7 commented on GitHub (Jan 22, 2024):
I updated the issue, but to summarize the state so far:
Problem
Authors with cyrillic name are listed multiple times in the authors tab of the UI - but only if the authors tag is edited.
Steps to reproduce
Pick any number of books and add the same cyrillic name to the authors tag; ABS will create a new author for every book.
Workaround
After editing the author tag, write the metadata to file, remove the edited books from your library and add them back in.
@thatguy7 commented on GitHub (Jan 26, 2024):
I just realised the behavior is the same for the series field. If I set the series using the web-gui it creates a new series for every book with one book in the series.
According to the discord discussion, this happens with other non-latin languages as well.
@thatguy7 commented on GitHub (Sep 13, 2024):
I have dug a bit deeper and the problem arises in these functions:
https://github.com/advplyr/audiobookshelf/blob/01fbea02f19bb19b5d2e8629fdbe4662d278e872/server/models/Author.js#L55
https://github.com/advplyr/audiobookshelf/blob/01fbea02f19bb19b5d2e8629fdbe4662d278e872/server/models/Series.js#L41
If I run the following query on the DB:
SELECT LOWER(name), name, lastFirst FROM authors;, the output looks like:Notice how the ASCII characters were properly set to lower case but the non-ASCII characters remained unaffected.
This is a limitation of SQLite.
The easy workaround is checking if
authorName(orseriesName) contains non-ASCII chars and avoiding thelowerfunction in that case. This could be used for the check: https://stackoverflow.com/questions/13522782/how-can-i-tell-if-a-string-has-any-non-ascii-characters-in-itIt would be better to figure out how to get SQLite to do behave as expected.
@nichwall commented on GitHub (Sep 13, 2024):
Related discussion here https://github.com/advplyr/audiobookshelf/issues/2678 and in the referenced PRs
@github-actions[bot] commented on GitHub (Oct 5, 2024):
Fixed in v2.14.0.