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

Incremental music sync log errors #270

Closed
mcarlton00 opened this issue Apr 9, 2020 · 2 comments · Fixed by #305 or #331
Closed

Incremental music sync log errors #270

mcarlton00 opened this issue Apr 9, 2020 · 2 comments · Fixed by #305 or #331
Labels
bug Something isn't working

Comments

@mcarlton00
Copy link
Member

Describe the bug

New music added to the server while music isn't synced -> Lots of errors in the logs

2020-04-08 20:06:13.443 T:140039749101312  NOTICE: JELLYFIN.objects.music -> ERROR::jellyfin_kodi/objects/music.py:392 'NoneType' object has no attribute '__getitem__'
                                            Traceback (most recent call last):
                                              File "jellyfin_kodi/objects/music.py", line 390, in song_artist_link
                                                temp_obj['ArtistId'] = self.jellyfin_db.get_item_by_id(*values(temp_obj, QUEM.get_item_obj))[0]
                                            TypeError: 'NoneType' object has no attribute '__getitem__'

Seems to be based around repeatedly trying to look up the item in the database when it doesn't exist.

System (please complete the following information):

  • Jellyfin Version: 10.5.3
  • Kodi Version: 18.6
  • Addon Version: 0.5.3
@mcarlton00 mcarlton00 added the bug Something isn't working label Apr 9, 2020
@mcarlton00
Copy link
Member Author

A lot of this is caused by the code surrounding the database. It's trying to sync everything, finding out that the library shouldn't be synced, but then not stopping the process so it throws a bunch of errors. There's also a lot of program flow being determined by try/except blocks around here:

try:
temp_obj['ArtistId'] = self.jellyfin_db.get_item_by_id(*values(temp_obj, QUEM.get_item_obj))[0]
except TypeError:
try:
self.artist(self.server.jellyfin.get_item(temp_obj['Id']), library=None)
temp_obj['ArtistId'] = self.jellyfin_db.get_item_by_id(*values(temp_obj, QUEM.get_item_obj))[0]
except Exception as error:
LOG.exception(error)
continue

@mcarlton00
Copy link
Member Author

mcarlton00 commented Jun 27, 2020

Still seeing this. It's harmless, but annoying

Edit: happening during the notification events from the server when metadata is changed, not during the incremental sync on startup.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
1 participant