search

skylex / xbmc-betaseries

Kodi Plugin which allows to mark your episodes as watched on BetaSeries.com
GNU General Public License v2.0
24 stars 16 forks source link

Refonte totale pour passage Python3 #26

Open kugan49 opened 3 years ago

kugan49 commented 3 years ago

J'adore cette extension mais comme elle n'avait pas été migrée pour python 3 j'en ai profité pour la refondre complètement :

respect des PEP et des préconisation de Kodi sur la forme je me suis inspiré (un peu) de l'add-on trakt pour l'organisation des modules Dites moi si ca vous intéresse de conserver ce projet, ou je peux demander dans le site betaseries pour le reprendre sinon

(je n'avais pas utilisé la bonne source)

skylex commented 3 years ago

En effet, le code a bien changé ! :) Par contre ton dépôt ne contient que le code du plugin "BetaSeries Agent", donc la merge request irait écraser les autres plugins (repository et subtitles). De plus, tu as changé l'id du plugin "service.betaseries.com" en "script.betaseries", ce qui ferait que les mises à jour automatiques ne pourront pas se faire pour les personnes utilisant le plugin actuel. Peux-tu modifier cela ?

kugan49 commented 3 years ago

J'avais tout enlevé et renommé l'addon car j'avais envisagé de demander d'ajouter juste cet addon dans le repository officiel kodi.

C'est corrigé

PCCV commented 3 years ago

Hello Kugan49, J'avais prévu de migrer en python3 mais tu m'as coupé l'herbe sous le pied... ceci dit, je n'aurais pas été aussi loin avec les préconisations. J'ai voulu tester avec Kodi 19, mais j'ai un échec d'installation avec le fichier "service.betaseries.com-2.0.0" pour cause de structure invalide. Saurais-tu à quoi cela est dû?

En tout cas, beau travail et bel effort.

kugan49 commented 3 years ago

Salut, désolé j'avais oublié une balise dans le xml des addons. Bon tests

PCCV commented 3 years ago

Bonjour kugan49,

Voici mes premiers retours : 1 - Pour que l'on puisse faire la mise à jour à partir de la version précédente sans entrer à nouveau les identifiants et en conservant les options déjà choisies, il faudrait renommer le service de "script.betaseries" à "service.betaseries.com" dans "addon.xml", "default.py", "kodiUtilities.py" "player.py" et "utilities.py"

2 - Dans "serviceapi.py" ligne 186: changer (showtitle.encode("ascii", "xmlcharrefreplace")).replace(" ", "+") par showtitle.replace(" ", "+").encode("ascii", "xmlcharrefreplace")

et lignes 368 & 395: changer showtitle.encode("ascii", "xmlcharrefreplace").replace(" ", "+") par showtitle.replace(" ", "+").encode("ascii", "xmlcharrefreplace")

3 - Je n'ai pas regardé pourquoi mais dans le log j'ai 3 fois: [script.betaseries] resources.lib.player: Scan finished, all episodes updated

Je continue à tester. j'avais commencé à modifier l'addon pour qu'il soit compatible avec Netflix et Disney+, je vais voir ce qu'il reste a faire pour cela.

kugan49 commented 3 years ago

Salut,

j'ai corrigé les 2 premiers points. Pour le point 3 : je ne vois pas pourquoi tu l'as 3 fois la fonction est appelé au chargement du service et sur notification de kodi : "VideoLibrary.OnScanFinished", tu aurais 2 fois cette notification ?

Si tu veux qu'on travaille ensemble pour Netflix et Disney+ redis moi

PCCV commented 3 years ago

Pour le point 3 : je ne vois pas pourquoi tu l'as 3 fois la fonction est appelé au chargement du service et sur notification de kodi : "VideoLibrary.OnScanFinished", tu aurais 2 fois cette notification ?

J'ai installé watchdog avec les options "Clean on startup" et "Scan on startup" est-ce que cela pourrait venir là ?

Si tu veux qu'on travaille ensemble pour Netflix et Disney+ redis moi

Je vais reprendre ce que j'avais commencé pour Netflix et voir pour Disney+. J'aimerai aussi que cela puisse fonctionner avec le replay de catchupTV mais cela dépend bcp des chaines. je te proposerai qqchose.

En tout cas ton code est super clean, c'est du beau travail.

kugan49 commented 3 years ago

Merci, ca fait plaisir que tu t'y retrouves. Pour les 3 logs c'st bien watchdog puisqu'il scan en continu et une fois terminé kodi lance une notification "VideoLibrary.OnScanFinished"

On est près pour merge alors ?

PCCV commented 3 years ago

Bonjour,

On est près pour merge alors ?

Ca y est, c'est mergé. J'ai galéré hier (ça se voit au nb de commit):

Il faudra vérifier que tout fonctionne pour l'installation ainsi que pour la MàJ.

skylex commented 3 years ago

Bravo c'est très propre comme code ! Il reste juste une dernière chose à changer pour que je puisse merge : remplacer les URL https://raw.github.com/PCCV pour rester sur le repo actuel.

PCCV commented 3 years ago

remplacer les URL https://raw.github.com/PCCV pour rester sur le repo actuel.

j'ai fait une branche pour Kodi18, je laisse les liens ou je les change aussi ?

skylex commented 3 years ago

Oui pareil, je vais créer la branche leia pour que tu puisses faire une merge request dessus. Merci

jaumard commented 3 years ago

Hey :) du coup il faut installer quoi et ou? un petite doc serais top Car la je vois que rien est merge ni sur quel repo je dois récupérer l'addon. J'ai essayé d'installé https://github.com/PCCV/xbmc-betaseries branche master mais je me retrouve avec l'écran des settings sans aucun texte juste les checkbox :D et rien pour télécharger les sous titres.

Merci merci :)

rmathot commented 3 years ago

Bonjour, est-ce que je peux filer un coup de main niveau code/testing/debugging pour faire avancer cette PR? Comme j'aime bien cette extension, j'ai un peu de temps à donner :)