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

Addon doesnt work (?) since Kodi 19 #27

Closed
Dinth opened this issue Feb 21, 2021 · 7 comments
Closed

Addon doesnt work (?) since Kodi 19 #27

Dinth opened this issue Feb 21, 2021 · 7 comments

Comments

@Dinth
Copy link

Dinth commented Feb 21, 2021

Hi. I was happy to see this addon be one of the first subtitle addon upgraded to Kodi 19/Python 3 a while ago. Unfortunately, i have never managed to get Python3 version to work since then.
Logging in works fine, same with search, but when i try to download any subtitle im getting an error.
Looking at kodi.log:
CUtil::SplitParams("C:\XXX\Kodi\iserdata\addon_data\service.subtitles.napisy24pl\temp\subs.zip", C:\XXX\Kodi\userdata\addon_data\service.subtitles.napisy24pl\temp") - end of string while searching for ) or "

@CaTzil
Copy link
Owner

CaTzil commented Feb 21, 2021

Hi @Dinth
Thank you for reporting an issue, 🙏🏼
Yesterday, I've tried to fix it, seems that it related to unzipping the archive.
I didn't able to find a working method to extract the subs from zip files

@pogo1975
Copy link

hi @CaTzil i worked around this problem
find the attached changed files, i think it is required to include urllib i addon.xml but i do not know this for certain,

Think that required changes are only in NapisyUtils.py but this is my firs ever "hack" of python script, so it needs checking, code is borrowed from another plugin "subscene"

Regards
Jarek
changes.zip

@CaTzil
Copy link
Owner

CaTzil commented Feb 28, 2021 via email

@pogo1975
Copy link

as much as i can check it is working, downloaded couple of subtitles for diffrent tv shows without error

@CaTzil
Copy link
Owner

CaTzil commented Feb 28, 2021 via email

@CaTzil
Copy link
Owner

CaTzil commented Mar 1, 2021

I've tested your code, it throws

Create - unsupported protocol(archive) in archive://

image

I've tried similar solution (which i've copied from one of the addons) with zip protocol, and it works, but only once.

Maybe, there is some issue with my setup, I will try to setup a regular dev env, if it solves the basic usage I will upload a fix.

@CaTzil
Copy link
Owner

CaTzil commented Mar 1, 2021

Fixed in v3.0.2

xbmc/repo-scripts#1894

@CaTzil CaTzil closed this as completed Mar 1, 2021
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

No branches or pull requests

3 participants