Debut de la refactorisation du traitement des url pour determiner la nature du partage.

src/Muzich/CoreBundle/Entity/Element.php

@@ -82,6 +82,12 @@ class Element
   
   const DATA_PLAYLIST_AUTHOR = "data_playlist_author";
   
+  
+  const TYPE_TRACK = 'track';
+  const TYPE_ALBUM = 'album';
+  const TYPE_PLAYLIST =  'playlist';
+  
+  
   /**
    * @ORM\Id
    * @ORM\Column(type="integer")
@@ -999,4 +1005,33 @@ class Element
     return $this->hasTagsProposition();
   }
   
+  // TODO: Pas booo ! Ecrit comme ca pour le moment lors de la refacto de UrlAnalyzer
+  public function getCleanedUrl()
+  {
+    // Procèdures de nettoyages après constat d'erreurs
+    $url = $this->getUrl();
+    if ($this->getType() == 'deezer.com')
+    {
+      $url = urldecode($url);  
+    }
+    
+    $base_url = $this->getType();
+    if ($this->getType() == 'spotify.com')
+    {
+      $base_url = 'open.spotify.com';
+    }
+    
+    $url = str_replace('www.', '', $url);
+    $url = str_replace('http://'.$base_url, '', $url);
+    $url = str_replace('https://'.$base_url, '', $url);
+    
+    if ($this->getType() == 'deezer.com')
+    {
+      $url = str_replace(' ', '-', $url);
+      $url = strtolower($url);
+    }
+    
+    return $url;
+  }
+  
 }

src/Muzich/CoreBundle/Factory/ElementFactory.php

@@ -6,6 +6,7 @@ use Muzich\CoreBundle\Entity\Element;
 use Symfony\Component\DependencyInjection\Container;
 use Doctrine\ORM\EntityManager;
 use Muzich\CoreBundle\lib\Api\Connector as ApiConnector;
+use Muzich\CoreBundle\lib\Element\UrlAnalyzer;
 
 /**
  *
@@ -18,6 +19,9 @@ abstract class ElementFactory
   protected $container;
   protected $entity_manager;
   protected $api_connector;
+  protected $url_analyzer;
+  
+  protected $url_matchs = array();
   
   /**
    *
@@ -30,6 +34,7 @@ abstract class ElementFactory
     $this->container = $container;
     $this->entity_manager = $entity_manager;
     $this->api_connector = new ApiConnector($element);
+    $this->url_analyzer = new UrlAnalyzer($element, $this->url_matchs);
   }
   
   protected function getApiConnector()
@@ -37,6 +42,15 @@ abstract class ElementFactory
     return $this->api_connector;
   }
   
+  public function setUrlDatas()
+  {
+    if ($this->url_analyzer->haveMatch())
+    {
+      $this->element->setData(Element::DATA_TYPE  , $this->url_analyzer->getType());
+      $this->element->setData(Element::DATA_REF_ID, $this->url_analyzer->getRefId());
+    }
+  }
+  
   /**
    * Retourne l'url relative dans le site
    * 

src/Muzich/CoreBundle/Factory/Elements/Jamendocom.php

@@ -4,6 +4,7 @@ namespace Muzich\CoreBundle\Factory\Elements;
 
 use Muzich\CoreBundle\Factory\ElementFactory;
 use Muzich\CoreBundle\Entity\Element;
+use Muzich\CoreBundle\Factory\UrlMatchs;
 
 /**
  * 
@@ -13,43 +14,10 @@ use Muzich\CoreBundle\Entity\Element;
 class Jamendocom extends ElementFactory
 {
   
-  protected function proceedTypeAndId()
+  public function __construct(Element $element, Container $container, EntityManager $entity_manager)
   {
-    $url_clean = $this->getCleanedUrl();
-        
-    // album
-    // http://www.jamendo.com/fr/album/3409
-    $type   = null;
-    $ref_id = null;
-    if (preg_match("#^\/[a-zA-Z0-9_-]+\/album\/([0-9]+)#", $url_clean, $chaines))
-    {
-      $type   = 'album';
-      $ref_id = $chaines[1];
-    }
-    // track
-    // http://www.jamendo.com/fr/track/894974
-    else if (preg_match("#^\/[a-zA-Z0-9_-]+\/track\/([0-9]+)#", $url_clean, $chaines))
-    {
-      $type = 'track';
-      $ref_id = $chaines[1];
-    }
-    // album new ver
-    // http://www.jamendo.com/fr/list/a45666/proceed-positron...
-    else if (preg_match("#^\/[a-zA-Z0-9_-]+\/list\/a([0-9]+)\/.#", $url_clean, $chaines))
-    {
-      $type   = 'album';
-      $ref_id = $chaines[1];
-    }
-    // track new ver
-    // http://www.jamendo.com/fr/track/347602/come-come
-    else if (preg_match("#^\/[a-zA-Z0-9_-]+\/track\/([0-9]+)\/.#", $url_clean, $chaines))
-    {
-      $type = 'track';
-      $ref_id = $chaines[1];
-    }
-    
-    $this->element->setData(Element::DATA_TYPE  , $type);
-    $this->element->setData(Element::DATA_REF_ID, $ref_id);
+    parent::__construct($element, $container, $entity_manager);
+    $this->url_matchs = UrlMatchs::$jamendo;
   }
   
   public function getStreamData()
@@ -106,9 +74,6 @@ class Jamendocom extends ElementFactory
    */
   public function retrieveDatas()
   {
-    // On determine le type et l'url
-    $this->proceedTypeAndId();
-    
     $type = $this->element->getData(Element::DATA_TYPE);
     $ref_id = $this->element->getData(Element::DATA_REF_ID);
     

src/Muzich/CoreBundle/Factory/Elements/Soundcloudcom.php

@@ -47,159 +47,6 @@ class Soundcloudcom extends ElementFactory
       $this->setElementDatasWithApi();
     }
     
-    ////$this->element->setData(Element::DATA_REF_ID, $this->element->getUrl());
-    //
-    //// récupération de données avec l'API
-    //if ($match)
-    //{
-    //  
-    //  
-    //  
-    //  
-    //  
-    //  // La première étape consiste a résoudre l'url
-    //  $ch = curl_init('http://api.soundcloud.com/resolve.json?url='.$this->element->getUrl().'&client_id=39946ea18e3d78d64c0ac95a025794e1');
-    //
-    //  $options = array(
-    //    CURLOPT_RETURNTRANSFER => true,
-    //    CURLOPT_HTTPHEADER => array('Content-type: application/json')
-    //  );
-    //
-    //  curl_setopt_array( $ch, $options );
-    //  $result = json_decode(curl_exec($ch));
-    //
-    //  if (isset($result->errors))
-    //  {
-    //    if (count($result->errors))
-    //    {
-    //      return;  
-    //    }
-    //  }
-    //  
-    //  if (!isset($result->location))
-    //  {
-    //    return;
-    //  }
-    //  
-    //  if (!$result->location)
-    //  {
-    //    return;
-    //  }
-    //  
-    //  $getjsonurl = $result->location;
-    //  // On a maintenant la bonne url pour demander les infos
-    //  $ch = curl_init($getjsonurl);
-    //  curl_setopt_array($ch, $options);
-    //  $result = json_decode(curl_exec($ch), true);
-    //  
-    //  // En premier lieux il nous faut être sur d'avoir le droit d'utiliser le lecteur exportable
-    //  $sharing = false;
-    //  if (array_key_exists('sharing', $result) && array_key_exists('embeddable_by', $result))
-    //  {
-    //    if ($result['sharing'] == 'public' && ($result['embeddable_by'] == 'all' || $result['embeddable_by'] == 'me'))
-    //    {
-    //      $sharing = true;
-    //    }
-    //  }
-    //  
-    //  if ($sharing)
-    //  {
-    //    if (array_key_exists('id', $result) )
-    //    {
-    //      $this->element->setData(Element::DATA_REF_ID, $result['id']);
-    //    }
-    //    
-    //    if (array_key_exists('uri', $result) )
-    //    {
-    //      $this->element->setData(Element::DATA_NORMALIZED_URL, $result['uri']);
-    //    }
-    //
-    //    if (array_key_exists('artwork_url', $result) )
-    //    {
-    //      if ($result['artwork_url'])
-    //      {
-    //        $this->element->setData(Element::DATA_THUMB_URL, $result['artwork_url']);
-    //      }
-    //      else
-    //      {
-    //        if (array_key_exists('user', $result) )
-    //        {
-    //          if (array_key_exists('avatar_url', $result['user']) )
-    //          {
-    //            if ($result['user']['avatar_url'])
-    //            {
-    //              $this->element->setData(Element::DATA_THUMB_URL, $result['user']['avatar_url']);
-    //            }
-    //          }
-    //        }
-    //      }
-    //    }
-    //    
-    //    if (array_key_exists('kind', $result) )
-    //    {
-    //      $this->element->setData(Element::DATA_TYPE, $result['kind']);
-    //    }
-    //
-    //    if (array_key_exists('downloadable', $result) )
-    //    {
-    //      $this->element->setData(Element::DATA_DOWNLOAD, $result['downloadable']);
-    //      // FIXME
-    //      $this->element->setData(Element::DATA_DOWNLOAD_URL, $this->element->getUrl().'/download');
-    //    }
-    //
-    //    if (array_key_exists('title', $result) )
-    //    {
-    //      $this->element->setData(Element::DATA_TITLE, $result['title']);
-    //    }
-    //
-    //    if (array_key_exists('user', $result) )
-    //    {
-    //      $this->element->setData(Element::DATA_ARTIST, $result['user']['username']);
-    //    }
-    //    
-    //    $genres = '';
-    //    if (array_key_exists('genre', $result) )
-    //    {
-    //      if (strlen($result['genre']))
-    //      {
-    //        $genres = $result['genre'];
-    //      }
-    //    }
-    //    
-    //    $tags_list = '';
-    //    if (array_key_exists('tag_list', $result) )
-    //    {
-    //      if (strlen($result['tag_list']))
-    //      {
-    //        $tags_list = $result['tag_list'];
-    //      }
-    //    }
-    //    
-    //    $tags_string = $genres.' '.$tags_list.' '.str_replace(' ', '-', $genres);
-    //    $tags_like = array();
-    //    if (strlen($tags_string))
-    //    {
-    //      $tag_like = new TagLike($this->entity_manager);
-    //      foreach (explode(' ', $tags_string) as $word)
-    //      {
-    //        $similar_tags = $tag_like->getSimilarTags($word, ($this->element->getOwner())?$this->element->getOwner()->getId():null);
-    //        if (count($similar_tags))
-    //        {
-    //          if ($similar_tags['same_found'])
-    //          {
-    //            $tags_like[] = $similar_tags['tags'][0]['name'];
-    //          }
-    //        }
-    //      }
-    //      $tags_like[] = $genres;
-    //      if (count($tags_like))
-    //      {
-    //        $this->element->setData(Element::DATA_TAGS, array_unique($tags_like));
-    //      }
-    //    }
-    //    
-    //  }
-    //}
   }
   
   protected function setElementDatasWithApi()

src/Muzich/CoreBundle/Factory/UrlMatchs.php

@@ -0,0 +1,23 @@
+<?php
+
+namespace Muzich\CoreBundle\Factory;
+
+use Muzich\CoreBundle\Entity\Element;
+
+class UrlMatchs
+{
+  public static $jamendo = array(
+    Element::TYPE_TRACK => array(
+      // http://www.jamendo.com/fr/track/894974
+      "#^\/[a-zA-Z0-9_-]+\/track\/([0-9]+)#" => 1,
+      // http://www.jamendo.com/fr/track/347602/come-come
+      "#^\/[a-zA-Z0-9_-]+\/track\/([0-9]+)\/.#" => 1
+    ),
+    Element::TYPE_ALBUM => array(
+      // http://www.jamendo.com/fr/album/3409
+      "#^\/[a-zA-Z0-9_-]+\/album\/([0-9]+)#" => 1,
+      // http://www.jamendo.com/fr/list/a45666/proceed-positron...
+      "#^\/[a-zA-Z0-9_-]+\/list\/a([0-9]+)\/.#" => 1
+    )
+  );
+}

src/Muzich/CoreBundle/Managers/ElementManager.php

@@ -136,6 +136,7 @@ class ElementManager
     {
       $site_factory = $this->getFactory();
       // On récupères les datas de l'élément
+      $site_factory->setUrlDatas();
       $site_factory->retrieveDatas();
       // On procède a la construction de nos informations
       $site_factory->proceedEmbedCode();

src/Muzich/CoreBundle/Tests/ElementFactory/UrlAnalyzerTest.php

@@ -0,0 +1,49 @@
+<?php
+
+namespace Muzich\CoreBundle\Tests\Api;
+
+use Muzich\CoreBundle\lib\Element\UrlAnalyzer;
+use Muzich\CoreBundle\Entity\Element;
+use Muzich\CoreBundle\Factory\Elements\Jamendocom;
+use Muzich\CoreBundle\Factory\UrlMatchs;
+
+class UrlAnalyzerTest extends \PHPUnit_Framework_TestCase
+{
+  
+  protected function getNewElement($type, $url)
+  {
+    $element = new Element();
+    $element->setUrl($url);
+    $element->setType($type);
+    return $element;
+  }
+  
+  public function testJamendo()
+  {
+    $url_analyzer = new UrlAnalyzer($this->getNewElement('jamendo.com', 'http://www.jamendo.com/fr/track/894974'), UrlMatchs::$jamendo);
+    $this->assertTrue($url_analyzer->haveMatch());
+    $this->assertEquals(Element::TYPE_TRACK, $url_analyzer->getType());
+    $this->assertEquals('894974', $url_analyzer->getRefId());
+    
+    $url_analyzer = new UrlAnalyzer($this->getNewElement('jamendo.com', 'http://www.jamendo.com/fr/track/347602/come-come'), UrlMatchs::$jamendo);
+    $this->assertTrue($url_analyzer->haveMatch());
+    $this->assertEquals(Element::TYPE_TRACK, $url_analyzer->getType());
+    $this->assertEquals('347602', $url_analyzer->getRefId());
+    
+    $url_analyzer = new UrlAnalyzer($this->getNewElement('jamendo.com', 'http://www.jamendo.com/fr/album/3409'), UrlMatchs::$jamendo);
+    $this->assertTrue($url_analyzer->haveMatch());
+    $this->assertEquals(Element::TYPE_ALBUM, $url_analyzer->getType());
+    $this->assertEquals('3409', $url_analyzer->getRefId());
+    
+    $url_analyzer = new UrlAnalyzer($this->getNewElement('jamendo.com', 'http://www.jamendo.com/fr/list/a45666/proceed-positron...'), UrlMatchs::$jamendo);
+    $this->assertTrue($url_analyzer->haveMatch());
+    $this->assertEquals(Element::TYPE_ALBUM, $url_analyzer->getType());
+    $this->assertEquals('45666', $url_analyzer->getRefId());
+    
+    $url_analyzer = new UrlAnalyzer($this->getNewElement('jamendo.com', 'http://www.jamendo.com/fr/playlist/2134354'), UrlMatchs::$jamendo);
+    $this->assertFalse($url_analyzer->haveMatch());
+    $this->assertEquals(null, $url_analyzer->getType());
+    $this->assertEquals(null, $url_analyzer->getRefId());
+  }
+  
+}

src/Muzich/CoreBundle/lib/Element/UrlAnalyzer.php

@@ -0,0 +1,72 @@
+<?php
+
+namespace Muzich\CoreBundle\lib\Element;
+
+use Muzich\CoreBundle\Entity\Element;
+
+class UrlAnalyzer
+{
+  
+  protected $url;
+  protected $match_rules;
+  
+  // TODO: Ce serait bien que le match_rules soit dans un Element fille
+  public function __construct(Element $element, $match_rules = array())
+  {
+    $this->url = $element->getCleanedUrl();
+    $this->match_rules = $match_rules;
+  }
+  
+  protected function getMatchRules()
+  {
+    foreach ($this->match_rules as $type => $rules)
+    {
+      foreach ($rules as $expression => $ref_id_position)
+      {
+        if (preg_match($expression, $this->url, $preg_result))
+        {
+          if (array_key_exists($ref_id_position, $preg_result))
+          {
+            return array(
+              Element::DATA_TYPE => $type,
+              Element::DATA_REF_ID => $preg_result[$ref_id_position]
+            );
+          }
+        }
+      }
+    }
+    
+    return false;
+  }
+  
+  public function haveMatch()
+  {
+    if ($this->getMatchRules() !== false)
+    {
+      return true;
+    }
+    
+    return false;
+  }
+  
+  public function getRefId()
+  {
+    if (($match_rules = $this->getMatchRules()))
+    {
+      return $match_rules[Element::DATA_REF_ID];
+    }
+    
+    return null;
+  }
+  
+  public function getType()
+  {
+    if (($match_rules = $this->getMatchRules()))
+    {
+      return $match_rules[Element::DATA_TYPE];
+    }
+    
+    return null;
+  }
+  
+}