Ignore empty cast names

Falke-Design commented 6 months ago

Check if this PR fulfills these requirements:

[x] My changes respect the Kodi add-on development rules
[x] I have read the CONTRIBUTING document.
[x] I made sure there wasn't another Pull Request opened for the same update/change
[x] I have successfully tested my changes locally

Types of changes

[ ] New feature (non-breaking change which adds functionality)
[ ] Feature change (non-breaking change which changes behaviour of an existing functionality)
[ ] Improvement (non-breaking change which improves functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Refactor (non-breaking performance or readability improvements)

Description

Fixes: #1655

"Error TypeError: Actor: name property must not be empty" occurs.

This happens if the Cast has some empty entries. For example the movie Misfit:

{
   "Title":"Misfit",
   "Cast":[
      "Selina Mour",
      "Lion Wasczyk",
      "Lisa-Marie Koroll",
      "Jonathan Elias Weiske",
      "Vivien Wulf",
      "Meriel Hinsching",
      "",
      "Moritz Schirdewahn",
      "Simon Will",
      "Sylvie Meis"
   ],
...

Falke-Design commented 6 months ago

Attention while testing: the cache needs to be cleared else the info data will be not parsed again. And then the error will still occur grafik

CastagnaIT commented 6 months ago

take a look on my new PR solution opened, and take a look at the differences, also take in account that the "person" list is not used by "cast" only but from each REFERENCE_MAPPINGS value, so the change is more small and not require specifics additional for loop and data replacing as done here

CastagnaIT / plugin.video.netflix