antp.be forum

xenesys wrote:alors allocine ou le script qui bug ? ^^''

soulsnake wrote:J'ai remarqué un petit bug sur le script Allociné dans la fonction AnalyzePage, il manque un exit après ce bout de code :

Code: Select all

  if pos(UTF8Encode('0 résultat trouvé'),Line) <> 0 then
  begin
    if (GetOption('Type de Lancement') = 1) or (GetOption('Type de Lancement') = 2) then
    begin
      if Input(NomScript+' by ScorEpioN', retourneElement(0)+' pour '''+MovieName+''' :', MovieName) then
      begin
        if Pos(urlDomain, MovieName) > 0 then
          AnalyzeMoviePage(convertURL(MovieName))
        else
          AnalyzePage(urlAllocineSearch+retourneElement(2)+UrlEncode(TitrePlus(UTF8Encode(MovieName))));

  if pos(UTF8Encode('0 résultat trouvé'),Line) <> 0 then
  begin
    if (GetOption('Type de Lancement') = 1) or (GetOption('Type de Lancement') = 2) then
    begin
      if Input(NomScript+' by ScorEpioN', retourneElement(0)+' pour '''+MovieName+''' :', MovieName) then
      begin
        if Pos(urlDomain, MovieName) > 0 then
          AnalyzeMoviePage(convertURL(MovieName))
        else
          AnalyzePage(urlAllocineSearch+retourneElement(2)+UrlEncode(TitrePlus(UTF8Encode(MovieName))));
      end else
        exit;
      //showmessage(retourneElement(0)+' pour : '+MovieName);
      //exit;
    end else
    begin
      SetField(fieldChecked, 'False');
      exit;
    end;
  end;

soulsnake wrote:Aussi, concernant ceci :
Code:
SetField(fieldChecked, 'True');

Je l'aurais plutôt mis à la fin du traitement pour indiquer que le film a bien été traité plutôt que le mettre dans AnalyzePage seulement lorsque l'on a choisi un film dans la liste oh well.
Donc je l'aurais mis tout à la fin des fonctions AnalyzeMoviePage et AnalyzeSeriePage.

Pourtant de mon côté depuis la v118 j'ai ...

  if pos(UTF8Encode('0 résultat trouvé'),Line) <> 0 then
  begin
    if (GetOption('Type de Lancement') = 1) or (GetOption('Type de Lancement') = 2) then
    begin
      if Input(NomScript+' by ScorEpioN', retourneElement(0)+' pour '''+MovieName+''' :', MovieName) then
      begin
        if Pos(urlDomain, MovieName) > 0 then
          AnalyzeMoviePage(convertURL(MovieName))
        else
          AnalyzePage(urlAllocineSearch+retourneElement(2)+UrlEncode(TitrePlus(UTF8Encode(MovieName))));
      end;
      //showmessage(retourneElement(0)+' pour : '+MovieName);
      exit;
    end else
    begin
      SetField(fieldChecked, 'False');
      exit;
    end;
  end;

Quand je fais une recherche sur un titre avec 0 résultat comme "qsd" par exemple, je n'ai pas de problème.

Si on choisit un film dans la liste, il va forcément être traité ... ou bien alors, c'est que le script est complètement à la ramasse.

soulsnake wrote:Lorsque tu es en type de lancement 1 ou 2, essaie de rentrer n'importe quoi de sorte qu'on ne trouve pas de résultat 3 fois par exemple puis entre un nom de film existant la quatrième fois. A la fin tu auras 3 fois le message "No movie found" du fait des 3 tentatives. Ceci est dû au fait qu'il manque un exit et que le script continue alors qu'il doit s'arrêter après un nouveau appel à AnalysePage. Par contre les infos du film sont bien ramenées.

soulsnake wrote:Oui c'est juste mais on coche un film traité uniquement si on le choisit dans la liste. Si le film est trouvé directement, il n'est pas coché ce qui est surement un oubli. C'est pour ça que je propose de cocher le film à la fin du traitement comme ça plus de risque d'oubli.

Par contre je l'ai re-spécifié dans la partie "choisis dans la liste" car lorsque tu fais un 2 ème passage en choisissant l'option 4 du "type de lancement", les fiches n'ayant pas été traitées sont décochées

soulsnake wrote:Par exemple si tu as mal tapé le nom du film, tu t'en rends compte au second passage, tu le corriges et il n'y a pas forcement de liste de choix s'il n'y a qu'un résultat donc ton film n'est pas coché alors qu'il devrait l'être wink.

Raoul_Volfoni wrote:si vous constatez d'autres trucs dite le rapidement avant que je fournisse une mise à jour à Antoine.

antp wrote:Trop tard, c'est en ligne sur le serveur