[WIP] Refactorisation d'une partie du code

[Arias800]

• Afficher les taglines sur les menu et a la bonne place sur le player.
• Utilisation de addFolder a la place de addHost.
• Refactorisation des métadonnées.
• Ne pas faire de requête si il n'y a pas de saison puisque de toute façon elle aboutira par une erreur.

[Arias800]

Todo list :

• Correction affichage des acteurs.
• Ajout affichage du budget.

[alKODIque]

Seulement pour préciser que les affiches d'acteurs sont présentes dans le menu Recherche Visuelle. C'est seulement dans la source Pastebin qu'il y a problème (acteurs et réalisateurs)

EDIT: et bien que l'imagette des acteurs soit présentes dans la recherche visuelle, lorsqu'on clique dessus, rien ne s'affiche. J'ai vérifié avant la modi et les filmographies s'affichent bien

[alKODIque]

Il y a aussi que maintenant, lorsqu'un épisode n'a pas d'imagette présente dans TMDb, on voit le fanart vStream:

[Arias800]

• Correction du cache pour les films.
• Gestion de la classification (affichage de la classification Française si disponible, sinon il prend la 1ere a disposition).

[Arias800]

• Modification et simplification de la fenêtre information de Vstream.
• Amélioration de la récupération de la durée en réduisant le nombre de conversion.
• Retrait d'un import inutile.

[Arias800]

• Les sagas sont désormais catégorisées comme tel et ont le droit à leurs propres table. Comme ça, la base de donnée est mieux structurer.

[alKODIque]

Salut 👋

Donc j'ai installé les nouvelles modifications que tu as fait aujourd'hui et j'ai fait plusieurs tests :)

Voici mes commentaires pour chacune des améliorations que tu as fait:

- Correction du cache pour les films. Pour moi ceci n'a pas eu d'effet. l'affichage de la pochette (ou fanart) ne remonte pas au niveau du lien lorsque je démarre un film depuis un widget. Je n'ai pas d'affiche de film non plus si j'appuis sur pause pendant la lecture. (depuis un widget)

• Modification et simplification de la fenêtre information de Vstream: J'utilise habituellement la fenêtre d'information du skin, donc il m'est difficile de comparer a ce qui se passait avant les modifs (je prévois remettre la version stable pour vérifier plus tard). Mais de ce que je vois présentement, je n'ai plus la pochette du média lorsque je change les paramètres pour utiliser la fenêtre d'information de vStream. (voir impression d'écran)

A noter aussi que contrairement a la fenêtre d'info du skin, il ne se passe rien lorsque j'appui sur un film des catégories similaires ou autres proposés par la fenêtre d'info par défaut. (Mais puisque j'utilise la fenêtre du skin, je n'ai pas vérifié si ce problème était présent avant)

- Correction affichage des acteurs:

Malheureusement, cette modif n'a pas fonctionné non plus pour moi.. Sous Pastebin, les acteurs et les réalisateurs ne possèdent plus d'affiche et À l'intérieur de la recherche visuelle, bien que je puisse voir l'imagette de l'acteur, rien ne se passe lorsque je clique sur un acteur. (est fonctionnel sans tes modifications, j'ai vérifié)

• Gestion de la classification:

Ici, j'ai tout de suite noté le changement. Je vois bien la classification et ce, lorsque j'utilise la fenêtre d'info du skin et aussi avec la fenêtre d'info par défaut de vStream

• Affichage des vues sagas

Ici, je n'ai pas observé de changement non plus MAIS je suis certaine que lorsque @sizanic vérifiera, il pourra corriger.. Je pense que le problème vient de comment sont codés les types de vues (types de média??) pour la section sagas de la source pastebin... (codé comme ''films''). Puisque tu as ajouté une nouvelle table pour les sagas, je devrais donc pouvoir utiliser les modes d'affichage proposé par mon skin pour les sagas lorsque la modif sera effectuée.

Présentement, même avec ta modif, seuls les modes d'affichage disponibles pour les films me sont proposés... Pour donner un exemple, pour les sagas, mon skin propose l'affichage ''Poster intégré''. - c'est similaire au mode de vue proposé pour les ''Saisons'' qui lui, est tout a fait fonctionnel a l'intérieur des sources vStream.

A noter: je n'ai pas trouver d'autres sources qui me propose un dossier saga qui est fonctionnel dans vStream... de ton côté, @Arias800 , sur l'image plus haut ou tu présentes les sagas, quelle source utilises-tu?

[alKODIque]

J'ai une question concernant la lecture des films :)

Est-ce que tu prévois aussi rendre plus clean le lien d'affichage comme tu as fait pour les séries?? Présentement, je vois toujours le lien direct, en bleu, lors de l'appui pause pour un film. Pour les épisodes, c'est vraiment génial. C'est pareil que pour la lecture d'un média en local :)

Impression lors de la lecture film:

[alKODIque]

A titre d'info, pour que tu puisses comparer l'affichage de la fenêtre d'info avec la classification pour un film:

Fenêtre du skin:

Fenêtre par défaut vStream:

A noter aussi que lorsque je descend plus bas, dans la fenêtre d'information du skin, les options sont fonctionnelles, contrairement a la fenêtre d'info par défaut de vStream. (Je sais que c'est l'addon TMDb helper qui prend la relève par contre) :

EXEMPLE VIDÉO:

https://user-images.githubusercontent.com/49578130/130332530-9b423293-8d75-48f8-946d-831124b861ef.mp4

Comme je disais, TMDb helper prend la relève pour trouver les liens disponible du film (Poltergeist !!! dans mon exemple vidéo) mais lorsque le lien est trouvé, l'affiche du film de départ demeure maintenant dans le cache.. C'est donc une mauvaise affiche qui demeure également lors de la lecture.

• Sur ma vidéo, je suis parti de la fenêtre d'info du film ''Unearth'', je suis descendu vers les films similaire et j'ai appuyé sur ''Poltergeist 111''. Voici ce que j'obtiens:

• J'ai bien les liens du film Poltergeist, mais j'ai l'affiche du film ''Unearth''.
• Lors de la lecture, ça demeure aussi:

[Arias800]

• Pour la fenêtre de skin de Vstream il y a deux fichiers que j'ai modifiés en local que je n'ai pas mis dans le pull requests (simple oublie de ma part). Donc les problèmes liés a ça peuvent venir de là.
• Pour la recherche visuelle, effectivement j'avais oublié de mettre a jour ce fichier.
• Pour les sagas, je suis passé par Pastebin. Donc à creuser pour voir l'origine du problème.
• Pour les titres du film, faut voir. Je ne promets rien, mais je vais voir ce que je peux faire.
• Pour le dernier point, c’est à creuser. Mon skin ne permet pas d'utiliser correctement cette fonction. Donc je tesetterai avec le tien pour voir.

Vu qu'il n'y a pas de bug bloquant, je pense que je vais merge le pull requests. Un derniers avis ?

[Arias800]

Note : Les fichiers manquants ont directement été mis dans la beta.

[alKODIque]

ok! donc en installant la beta, tout y est?

[Arias800]

Oui tout est dans la bêta (y compris ces fichiers). Et au pire si besoin je peux le revert.

[alKODIque]

je vais installer

[Arias800]

J'ai fait une petite liste des choses qui reste à faire, mais normalement il n'y a plus rien de bloquant.

[alKODIque]

Pour l'affichage du budget, j'ai bien l'info ave mon skin, même a l'interieur de vStream. C'est peut-etre mon skin ou tmdb helper qui prend la relève? (certains medias en ont pas mais c'est parce que l'info n'est pas répertoriée..)

Pour les sagas, je suis certaine que c'est pas un bug arctic. Je vois bien les vues supplémentaires proposées pour les dossiers sagas dans d'autres addons.

merci pour tout!

[Arias800]

Pour le budget j'ai oublié de l'enlever de la liste. C'est une fonction de SkinHelper et pas de Kodi.

Pour les sagas, c’est surtout un mémo pour me rappeler que c'est reproduisible avec Artic (car sur mon skin je n'ai pas ce problème).

Tu peux regarder si l'affichage des acteurs est corrigé, car sur la recherche visuelle ça marche. Mais comme je ne sais pas ce qui s'affichait de base sur Pastebin, je ne sais pas si c'est redevenu normal.

[alKODIque]

les acteurs et réalisateurs ne s'affichent toujours pas.. J'ai pris soins de tout nettoyer mes caches, y compris texture13.db et mes thumbnails

J'ai pas skinhelper.. Mon skin utilise skin Variablees et titres les métadonnées de tmdb helper... donc ce doit être ca, car j'ai pas vérifié sur la fenêtre d'info par défaut de vstream pour le budget

[Arias800]

En tout cas, Kodi ne prends pas en charge le budget, donc ça viens forcément d'un addon externe. Pour les acteurs, je m'en doutais, je voulais juste en être sûre.

[alKODIque]

sur ton skin, pour les sagas, sur l'image que tu as mis, je cois que c'est une vue Films et non une vue saga... car si c'était une vue spécifique saga, on ne verrait pas la durée prévue.. C'est bien Amber??

[Arias800]

Ouais, c’est Amber, et oui, c’est possible que ce ne sois pas une vue pour les sagas (puisque le skin est entrain d'énormément changer en ce moment).

[alKODIque]

(Remarque... ca me dérange pas trop pour la vue saga, on peut quand-meme voir l'info et ce type de vue films me convient aussi)

[alKODIque]

oui c'est fou comment le skin a changé depuis que ce dev l'a pris en main (AMber) et j'y pense, j'ai essayé avec Arctic Zephir Reloaded et je n'avais pas non plus la vue sagas proposée ... faudrait voir avec tes modifs

[Arias800]

Comment il faut activer le mode de vue "saga" sur Arctic Horizon ?

[alKODIque]

c'est pas un mode qui s'active manuellement.. C'est avec un appui-long sur la saga que tu peux changer le type de vue, et lorsque c'est une saga, le choix ''Poster intégré'' est tout simplement proposé. Sinon, il est possible de changer les types de vue par défaut à l'intérieur de l'addon Skin Variable et ce, pour chaque type de média proposé par un addon.

Autre point: je viens de vérifier la recherche visuelle et la filmographie des acteur est revenue fonctionnelle! cool :))

[alKODIque]

Comme tu vois, les sagas sont reconnues comme (movies), on le voit en haut lorsque j'appui-long sur une saga:

[Arias800]

Je viens de voir ça. Je vais essayer de voir comment le skin fonctionne pour corriger ça.

[alKODIque]

Pour les sagas, a l'intérieur de l'addon TMDb Helper, voici ce qui est proposé: (sets): Les choix de vues sont différents

[Arias800]

J'ai vu la liste des "types" de vues, par déduction, j’ai deviné que c'était sets. Il faut juste que je trouve comment il comprend le type.

[alKODIque]

J'avais déja essayé de modifié dans le xml de skin variables et d'inscrire (season) avec le bon numéro de vue, pour vstream (avant que tu fasses tes modif) et ca avait pas fonctionné.. lol.. j'ai abandonné car je m'y connais pas

[alKODIque]

Par curiosité, je viens de créer un noeud vidéo avec Library node editor vers les sagas dans pastebin et j'ai pas non plus la proposition de vue.. :(

[alKODIque]

Pour mon problème de cache de la mauvaise affiche noté plus haut (ma vidéo), le problème est présent avant tes modifications.. je viens d'essayer sur mon pc avec une ancienne version vStream et j'ai le même problème.

Aussi, je pense que le problème ne vient pas dutout de vStream... puisqu'il s'agit de la fenêtre d'information du skin et que tmdb helper prends la relève pour trouver les liens, je crois que c'est tmdb helper qui prend en cache la mauvaise affiche.

je vais aviser le développeur

[Arias800]

Changement dans la beta :

• Les images pour les acteurs ou réalisateurs sur Pastebin, c'est corriger.
• Les vues spécifiques aux sagas fonctionne (testé sur pastebin).

[alKODIque]

Wahouu! Tu es génial ❤

Les sagas:

EDIT: Tu as vu l'info en haut?? bizarre, ça indique Returning serie''... et clearart d'une série (fourni par mon skin par contre. Donc surement un bug de AH)

Les acteurs:

[alKODIque]

Je vois que tu as aussi corrigé le type ''Saisons'' pour les séries ;) Avant, les saisons étaient traitées comme ''Épisodes'' (Je n'avais pas mentionné car j'avais un peu peur d'être trop pointilleuse sur ces détails hihi)

[Arias800]

Je l'ai vu après, que les saisons n'était pas catégoriser comme tel. Donc tant qu'à faire, autant le corriger.

Pour le "Returning serie" ça ne le fait pas sur tous. Est-ce que ça s'affiche si un nouveau film est prévus ? En tout cas je ne suis pas sûr que ce sois corrigeable.

[alKODIque]

Ca le fait juste si une série a le même nom qu'une saga... Je suis certaine que c'est un bug de tmdb helper.. je viens de vider ses caches et même a ça, a l'intérieur même de tmdb helper, j'ai des erreur de clearart.

Pour les sagas, tu veux dire que si le film n'est pas encore sorti mais fera parti de la saga?? Sur pastebin, je pense pas que ce soit possible car il doit impérativement avoir un lien existant pour que le film soit présent. Ce n'est pas juste des métadonnées... On pourrait savoir ça si il y avait un dossier Sagas à l'intérieur de la recherche visuelle par contre..

[Arias800]

En sois pastebin exploite les données de TMDB. Donc si TMDB retourne un status "en cours" pour la saga. Alors Vstream affichera ça, peux importe le contenu du past.

[alKODIque]

Tu es certain de ça?? Car pour chaque clic dans pastebin, il doit impérativement avoir un lien. Dailleurs, j'ai du ajouter à ma db la nouvelle saga ''Fear Streat, car elle n'était pas présente.

[alKODIque]

De plus, pour qu'une saga soit répertoriée dans cette source, il doit y avoir au minimum 3 films... donc on retrouve pas Deadpool ou encore Antman... et toutes les sagas de 2 films sont manquantes :(

[Arias800]

Il y a des choses qui sont liée au contenu en lui-même, et d'autres aux métas. Les métas ne prennent pas en compte le contenu réel. À titre de comparaison, le souci peut être le même que pour le nombre totales de saison. Par exemple Blacklist est marqué comme ayant 9 saisons, alors que seulement 8 sont sorties pour le moment. Ça vient du fait que c'est la valeur retournée par TMDB. Le status de la saga vient de TMDB et, comme le nombre total de saison, il ignore le contenu réel.

À partir du moment où le type de contenu est identifié (donc ça c'est pastebin qui le gère comme tu as expliqué au-dessus), Vstream se contente d'afficher le contenu que TMDB lui renvois.

Pour la saga Alien je suis allé voir et ça doit être liée au bug du clearart, car sur TMDB la saga n'a pas de status (donc ça ne devrait pas être marqué "returning").

[alKODIque]

ok!! Je croyais que tu voulais dire que l'affiche devrait être présente même si le film n'est pas sorti... mais je comprend que tu parle de l'information inscrite en haut d'écran... exemple ''en production'', ou quelque chose de la sorte... ca, c'est possible. en effet!!

J'ai vu qu'une autre source a aussi un dossier sagas.. c'est dommage qu'il ne soit pas fonctionnel, on pourrait comparer les résultats :)

Dans tous les cas, merci pour le super boulot que tu as fait, c'est génial 👍

[alKODIque]

J'ai un bel exemple ici avec la saga Indiana Jones...

Via Pastebin, on ne voit que les 4 films déjà sortis, avec une indication ''canceled'' - donc information provenant probablement de la série du même nom:

Dans l'addon TMDB helper, on voit l'affiche du 5e film a venir - mais il y a aussi un bug loll... car juste en haut de cet affiche, on peut lire ''par épisode croissant'' 😆😂

[Arias800]

Ouais, c'est le genre de chose qui n'est pas forcément corrigeable. Il faut que je regarde.

J'ai fait deux modifications, maintenant les métas persistent jusqu'au bout (ce qui n'était pas le cas avant puisque la durée du programme n'était pas pris en compte). Mais maintenant c'est corriger.

Et j'ai ajouté une version nettoyer du titre pour les films.

[alKODIque]

Hooo yes Ça fonctionne !!

Ta correction a bel et bien nettoyé le lien lors de la lecture de films, tout comme pour les épisodes ❤❤❤

[alKODIque]

J'ai remarqué un petit bug... même si vStream est maintenant presque parfait 😆

J'ai certaines pochettes de films qui n'apparaissent plus.. c'est le cas, par exemple, de Illusions (1993) - et il y en a d'autre... de ce que j'ai constaté, chaque fois que j'en ai ciblé, il s'agit d'un film ou l'affiche n'est pas présente en langue française sur le site TMDb (mais dans une autre langue elle y est)..

[Arias800]

Ca fait partie des bug que j'ai rajouté a la liste. Il y a aussi des menus dans le menu contextuel qui apparaît en double et sous Android les vues ne sont pas constante (il faut que je rajoute la possibilité de forcé les vues pour les saisons et les épisodes).

[alKODIque]

Hoo Une autre correction!

Maintenant, l'image d'un film remonte également au niveau du lien. Et le BIG changement est que le problème que j'expliquait avec ma vidéo (https://github.com/Kodi-vStream/venom-xbmc-addons/pull/3354#issuecomment-903162525) ou une mauvaise affiche persistait et bien c'est RÉSOLU!! 🔥✨

Maintenant, si je choisi dans la liste des films similaires, dans ma fenêtre d'information, j'ai la bonne pochette qui suit le lien ❤ Et ce, aussi lorsque je suis a l'intérieur d'un widget

[alKODIque]

Il me semble que tu en as parlé, suis pas certaine:

Je note qu'il indique 1 saison supplémentaire pour beaucoup de séries.. une saison qui n'existe pas. un exemple ici avec The Strain. Il est indiqué 5 saisons. Mais il n'en existe que 4:

[Arias800]

J'ai corrigé :

• Le contexte menu
• Les affiches qui n'apparaissaient plus quand TMDB ne retournais rien.
• Le nombre total de saisons qui était effectivement pas récupéré de la bonne manière.
• J'ai rajouté des options pour les vues saisons et épisodes dans les paramètres de Vstream. Ça permets de faire en sortent qu'elles soient constantes.