X-Git-Url: http://git.euphorik.ch/?a=blobdiff_plain;f=src%2Fmodules%2Fallocine.rb;h=1a28000f705b5de3bfeb01e8a6686d309e4f0b3e;hb=2cf96462fb75a5e95cf70ac288e8ffd08aad29de;hp=b4b65f0a5cdebbc6417bf16ae0722194f5c7f48b;hpb=40778efb4943c2f728258e865210fa169dfb3d15;p=pompage.git

diff --git a/src/modules/allocine.rb b/src/modules/allocine.rb
index b4b65f0..1a28000 100644
--- a/src/modules/allocine.rb
+++ b/src/modules/allocine.rb
@@ -20,7 +20,7 @@ module Net
    end
 end
 
-# un module d'importation pour le site "www.allocine.fr"
+# Un module d'importation pour le site "www.allocine.fr".
 class Allocine  < Basemodule
    
    def createConnexion
@@ -37,7 +37,7 @@ class Allocine  < Basemodule
       @connexion = nil
    end
    
-   # voir le parent.
+   # Voir le parent.
    def rechercherFilm(titre)
       createConnexion
       
@@ -68,10 +68,15 @@ class Allocine  < Basemodule
       
       reponses = {}
       unless titre.nil? or titre.empty?
-         r = html.scan(/<a href="\/film\/fichefilm_gen_cfilm=(\d+)\.html" class="link1">(.*?)<\/a>(?:<\/h4><h5 style="color: #666666">&nbsp;(.*?)<\/h5>){0,1}(?:<h4><br \/><\/h4>){0,1}(?:<h4 style="color: #666666"> de (.*?)<\/h4>){0,1}(?:<h4 style="color: #666666">&nbsp;avec (.*?)<\/h4>){0,1}(?:<h4 style="color: #666666">&nbsp;\((.*?)\)<\/h4>){0,1}/)
-
+         r = html.scan(/<h4><a href="\/film\/fichefilm_gen_cfilm=(\d+)\.html".*?>(.*?)<\/a><\/h4>(?:<h5><a .*?>&nbsp;(.*?)<\/a><\/h5>)?(?:<div.*?><h4.*?>(\d*?){4}<\/h4><\/div>)?(?:<div.*?><h5.*?>de (.*?)<\/h5><\/div>)?(?:<div.*?><h5.*?>avec (.*?)<\/h5><\/div>)?/)
+         # f[0] : l'id allocine du film
+         # f[1] : nom
+         # f[2] : nom original
+         # f[3] : année
+         # f[4] : réalisateur
+         # f[5] : acteurs
          r.each{|f|
-            reponses[f[1].virerBalisesHTML + (f[2] != nil ? " " + f[2].virerBalisesHTML : "") + (f[3] != nil ? " de " + f[3].virerBalisesHTML : "") + (f[4] != nil ? " avec " + f[4].virerBalisesHTML : "") + (f[5] != nil ? " (" + f[5].virerBalisesHTML + ")" : "")] = f[0]
+            reponses[f[1].virerBalisesHTML + (f[2] != nil ? " " + f[2].virerBalisesHTML : "") + (f[4] != nil ? " de " + f[4].virerBalisesHTML : "") + (f[5] != nil ? " avec " + f[5].virerBalisesHTML : "") + (f[3] != nil ? " (" + f[3].virerBalisesHTML + ")" : "")] = f[0]
          }
       end
       reponses
@@ -82,11 +87,11 @@ class Allocine  < Basemodule
       loadPath("/film/fichefilm_gen_cfilm=#{id}.html", film)
    end
    
+   # Voir le parent.
    def loadURL(url, film)
       loadPath(URI.parse(url).path, film)
    end
    
-   # Voir le parent.
    def loadPath(path, film)
       createConnexion
 
@@ -103,9 +108,10 @@ class Allocine  < Basemodule
       
       puts "Pompage de #{film.titre} (#{film.fichiers[0]}) ..."
       
-      #convertit le code latin-1 en UTF8      
+      # convertit le code latin-1 en UTF8      
       html = Iconv.iconv("UTF-8", "ISO-8859-1", html)[0]
             
+      # pompage de l'image dans un thread séparé
       if take?("image") && ($force || !film.possedeImage?)
          /<img src="(.*?)" border="0" alt="" class="affichette" \/>/ =~ html
          unless $1.nil?
@@ -166,7 +172,7 @@ class Allocine  < Basemodule
          film.pays.clear
          /<h4>Film (.*?)\.&nbsp;<\/h4>/ =~ html
          $1.split(',').each{|pays|
-            film.pays << Pays::ajouter(pays) unless pays.nil?
+            film.pays << Pays::ajouter(pays.strip) unless pays.nil?
          } unless $1.nil? 
       end
       
@@ -212,9 +218,10 @@ class Allocine  < Basemodule
       end
       
       fermerConnexion
-   end   
+   end
    
    def finish
+      # on attend que tous les threads de pompage d'image aient terminé
       @threadImage.join if defined? @threadImage
    end
 end
\ No newline at end of file