smirgol / plugin.video.crunchyroll

Watch videos from the anime platform Crunchyroll.com on Kodi
GNU Affero General Public License v3.0
45 stars 11 forks source link

Version multiprofile don't work #65

Closed KevinSatizabal closed 2 months ago

KevinSatizabal commented 2 months ago

Excuse my English.

The addon don't work with multiprofile

SO: Debian Bookworm Kodi version: kodi 21 Nexus Addon Version: 3.3.0.16 The add-on doesn't work with multiple profiles.

OS: Debian Bookworm Kodi version: Kodi 21 Nexus Add-on Version: 3.3.0.16

Steps to reproduce:

Install the new version.
Open the add-on.

Possible Solution:

To resolve this error, modify line 188 in the file resource/lib/utils.py as follows:

Change:

return path + image

TO: return str(path) + str(image)

PD:

The first time you set the profile, it already works with the old version.

LOG:

EXCEPTION Thrown (PythonToCppException) : -->Python callback/script returned the following error<--

smirgol commented 2 months ago

Hm that's really interesting, wondering what the underlying issue is. It fetched the data from the API, which includes the wallpaper. So the data should not be None at this point. I've added a None check for the image, but I really would love to understand why it happens in the first place. :-/

xtero commented 2 months ago

On the main profile, it looks like if we don't set the wallpaper, the field does not exist. At least, I had the issue on my own account.

rjousse18 commented 2 months ago

We will probably delete wallpaper feature as it's not really useful, as it's discussed here

smirgol commented 2 months ago

should be fixed with the None check I've added.