Kodi-vStream / venom-xbmc-addons

Addon de Streaming français pour Xbmc
GNU General Public License v3.0
495 stars 275 forks source link

Proposition d'améliorations sur vStream/Compte/TMDB #3902

Closed doctornono closed 2 years ago

doctornono commented 2 years ago

Bonjour à tous,

J'utilise la section Comptes/tmdb de vStream mais les actions de Noter un film, Ajouter/retirer un film des favoris ou de la watchlist (dans le menu contextuel tmdb) ne fonctionnent pas.

Voilà, un petit correctif qui rend ces actions fonctionnelles.

vStream TMDB

def getPostUrl(self, url, post):
        # Execute une requete POST vers l'API
        # Utile pour :
        #       - Noter film
        #       - Ajouter/Retirer le film des favoris
        #       - Ajouter/Retirer le film de la watchlist
        # Appelé depuis le menu contextuel TMDB
        #
        # Paramètres :
        #   url : le complément de l'url de l'api
        #           ex : '/movie/{movie_id}/rating' pour noter un film
        #           ou : '/account/{account_id}/favorite' pour ajouter aux favoris
        #   post : json qui sera envoyé Request Body
        #           ex :   {
        #                   "value": 8.5
        #                  }

        from urllib import request
        session_id = self.ADDON.getSetting('tmdb_session')

        urlapi = self.URL + url +'?api_key='+self.ADDON.getSetting('api_tmdb')+'&session_id='+ session_id

        req = request.Request(urlapi, method="POST")
        req.add_header('Content-Type', 'application/json')
        data = json.dumps(post)
        data = data.encode()
        r = request.urlopen(req, data=data)         
        response = r.read()
        r.close()
        data = json.loads(response)
        return data
sizanic commented 2 years ago

Merci pour ta proposition, on va regarder ça 👍🏻

doctornono commented 2 years ago

Tu as vu, je te spamme ici maintenant ;-) Si ca plait, je peux voir pour ajouter la création d'une liste tmdb et l'ajout de films à une liste utilisateur

sizanic commented 2 years ago

Oui, j'ai vu. Et j'ai vu que tu n'as pris en compte aucune de mes propositions 😄

1- utiliser dialog().numeric de comaddon 2- passer par une pull Request

Mais c'est pas grave, avance comme tu veux 👍🏻

doctornono commented 2 years ago

J'ai modifié par dialog.numeric. Concernant le pull Request, je ne connais rien à github...

sizanic commented 2 years ago

Pour la pull Request, tu te rends sur le fichier : https://github.com/Kodi-vStream/venom-xbmc-addons/blob/Beta/plugin.video.vstream/resources/sites/themoviedb_org.py

Tu passes en édition avec le stylo, et tu modifies directement le fichier. Cela va créer tout seul une Pull Request : ta modification est soumise aux développeurs qui peuvent l'accepter ou la modifier ou apporter des commentaires dessus.

doctornono commented 2 years ago

Voilà, c'est fait , merci pour l'info

sizanic commented 2 years ago

Je ne suis pas sûr 🤔 Rien ici : https://github.com/Kodi-vStream/venom-xbmc-addons/pulls

TmpName commented 2 years ago

Oui on peut voir ton fork, mais la derniere modif visible c'est celle de Sizanic pour Uptobox.

Dommage, now ca va etre plus compliqué ^^ Tu peux editer le fichier directement dessus https://github.com/doctornono/venom-xbmc-addons

Puis apres "pull request" "new pull requests"

doctornono commented 2 years ago

image

Voilà, j'ai fait comme demandé @TmpName. C'est bon comme ça ?

TmpName commented 2 years ago

Yep, parfait ^^

sizanic commented 2 years ago

Traité, je clos 👍🏻 Encore merci @doctornono