Merge pull request #1326 from noesya/youtube-ignores-autoplay-param

Suppression automatique du paramètre autoplay des vidéos YouTube

app/services/video/provider/dailymotion.rb

@@ -2,7 +2,8 @@ class Video::Provider::Dailymotion < Video::Provider::Default
   DOMAINS = [
     'dailymotion.com', 
     'www.dailymotion.com', 
-    'dai.ly'
+    'dai.ly',
+    '*.dmcdn.net'
   ]
 
   def identifier
@@ -23,4 +24,9 @@ class Video::Provider::Dailymotion < Video::Provider::Default
   def iframe_url
     "https://www.dailymotion.com/embed/video/#{identifier}"
   end
+
+  # L'autoplay est à 1 uniquement parce que l'iframe n'est pas chargée
+  def embed_with_defaults
+    "#{iframe_url}?autoplay=1&quality=380"
+  end
 end

app/services/video/provider/vimeo.rb

@@ -22,4 +22,9 @@ class Video::Provider::Vimeo < Video::Provider::Default
   def iframe_url
     "https://player.vimeo.com/video/#{identifier}"
   end
+
+  # L'autoplay est à 1 uniquement parce que l'iframe n'est pas chargée
+  def embed_with_defaults
+    "#{iframe_url}?autoplay=1"
+  end
 end

app/services/video/provider/youtube.rb

@@ -7,8 +7,8 @@ class Video::Provider::Youtube < Video::Provider::Default
   ]
 
   def identifier
-    video_url.include?('youtu.be')  ? video_url.split('youtu.be/').last
-                                    : video_url.split('v=').last
+    video_url.include?('youtu.be')  ? identifier_path
+                                    : identifier_param
   end
 
   def csp_domains
@@ -29,4 +29,16 @@ class Video::Provider::Youtube < Video::Provider::Default
   def embed_with_defaults
     "#{iframe_url}?autoplay=1&modestbranding=1&rel=0"
   end
+
+  protected
+
+  def identifier_path
+    video_url.split('youtu.be/').last
+  end
+
+  def identifier_param
+    uri = URI(video_url)
+    params = CGI::parse(uri.query)
+    params['v'].first
+  end
 end