Question

In my model, a Song is linked to a Type. A Type can be Youtube, Soundcloud, Deezer, etc..

When the link value has been validated by my validator, I want to set the style_id value with the correct Type.

What is the best way to do it ?

Answer 1

I think the best way is to perform the check twice:

• first time: using the validator, so you know it’s one of these video provider and then return the video link (not the id)
• second time: redefine the setLink() so it takes the link, extract the id and save both the link and the style_id

How to do that.

Create a custom lib, like lib/videoProvider.class.php. This is kind of prototyped class to valid & retrieve id from a video provider. It, of course, needs improvements.

class videoProvider
{
  private $url;
  private $providers = array('youtube','deezer','soundcloud');
  private $youtubePattern = '%^# Match any youtube URL
      (?:https?://)?  # Optional scheme. Either http or https
      (?:www\.)?      # Optional www subdomain
      (?:             # Group host alternatives
        youtu\.be/    # Either youtu.be,
      | youtube\.com  # or youtube.com
        (?:           # Group path alternatives
          /embed/     # Either /embed/
        | /v/         # or /v/
        | /watch\?v=  # or /watch\?v=
        )             # End path alternatives.
      )               # End host alternatives.
      ([\w-]{10,12})  # Allow 10-12 for 11 char youtube id.
      $%x';
  private $deezerPattern = '/\d+/';
  private $soundcloudPattern = '[\w-]+/[\w-]+$';

  public function __construct($url)
  {
    $this->url = $url;
  }

  /**
   * @return true / false
   */
  private function checkYoutube()
  {
    return preg_match($this->youtubePattern, $this->url) ? true : false;
  }

  /**
   * @return true / false
   */
  private function checkDeezer()
  {
     // A Deezer URL has this format : http://www.deezer.com/track/61340079

     return preg_match($this->deezerPattern, $this->url) ? true : false;
  }

  /**
   * @return true / false
   */
  private function checkSoundcloud()
  {
     // A Soundcloud URL has this format : http://soundcloud.com/[A-Z Artist]/[A-Z Title]

     return preg_match($this->soundcloudPattern, $this->url) ? true : false;
  }

  /**
   * @return true / false
   */
  public function isValid()
  {
    // check all video provider as you do in your validator
    // so it will return true if it find one, otherwise false

    foreach ($this->providers as $provider)
    {
      $function = 'check'.ucfirst($provider);

      if (true === $this->$function())
      {
        return true;
      }
    }

    return false;
  }

  /**
   * @return string
   */
  public function getId()
  {
    if ($this->checkYoutube() && preg_match($this->youtubePattern, $this->url, $matches))
    {
      return $matches[1];
    }

    if ($this->checkDeezer() && preg_match($this->deezerPattern, $this->url, $matches))
    {
      return $matches[1];
    }

    if ($this->checkSoundcloud() && preg_match($this->deezerPattern, $this->url, $matches))
    {
      return $matches[1];
    }
  }

  /**
   * @return string
   */
  public function getProvider()
  {
    if ($this->checkYoutube())
    {
      return 'youtube';
    }

    if ($this->checkDeezer())
    {
      return 'deezer';
    }

    if ($this->checkSoundcloud())
    {
      return 'soundcloud';
    }
  }
}

Then in the doClean of your validator, you just need to call this class, like that:

$videoProvider = new videoProvider($url);
if (false === $videoProvider->isValid())
{
  throw new sfValidatorError($this, 'invalid', array('value' => $url));
}

return $url;

Finally, the setLink in Song.class.php should now be:

public function setLink($value)
{
  // only perform this tweak if the value is a http link
  if (preg_match('/^http/i', $value))
  {
    $videoProvider = new videoProvider($value);

    // define url id
    parent::_set('link', $videoProvider->getId());

    // define type id
    $provider = $videoProvider->getProvider();
    $type     = Doctrine_Core::getTable('Type')->findOneByName($provider);

    parent::_set('type_id', $type->getId());
  }
}

This is a first draft that must be tested and improved (test if getId() returns an id and not false, same for getProvider, etc …)