Merge pull request #1234 from virtualtam/lint

[github/shaarli/Shaarli.git] / plugins / playvideos / playvideos.php

diff --git a/plugins/playvideos/playvideos.php b/plugins/playvideos/playvideos.php
index 9918453c3f614e9e61660afafb0913a548593c2a..c6d6b0cc6815e4f5c2547efdc395a67ccde08e3d 100644 (file)
--- a/plugins/playvideos/playvideos.php
+++ b/plugins/playvideos/playvideos.php
@@ -1,14 +1,30 @@
 <?php
+/**
+ * Plugin PlayVideos
+ *
+ * Add a button in the toolbar allowing to watch all videos.
+ * Note: this plugin adds jQuery.
+ */
 
 /**
  * When linklist is displayed, add play videos to header's toolbar.
  *
  * @param array $data - header data.
+ *
  * @return mixed - header data with playvideos toolbar item.
  */
-function hook_playvideos_render_header($data) {
+function hook_playvideos_render_header($data)
+{
     if ($data['_PAGE_'] == Router::$PAGE_LINKLIST) {
-        $data['buttons_toolbar'][] = file_get_contents(PluginManager::$PLUGINS_PATH . '/playvideos/playvideos.html');
+        $playvideo = array(
+            'attr' => array(
+                'href' => '#',
+                'title' => t('Video player'),
+                'id' => 'playvideos',
+            ),
+            'html' => '► '. t('Play Videos')
+        );
+        $data['buttons_toolbar'][] = $playvideo;
     }
 
     return $data;
@@ -18,13 +34,24 @@ function hook_playvideos_render_header($data) {
  * When linklist is displayed, include playvideos JS files.
  *
  * @param array $data - footer data.
+ *
  * @return mixed - footer data with playvideos JS files added.
  */
-function hook_playvideos_render_footer($data) {
+function hook_playvideos_render_footer($data)
+{
     if ($data['_PAGE_'] == Router::$PAGE_LINKLIST) {
         $data['js_files'][] = PluginManager::$PLUGINS_PATH . '/playvideos/jquery-1.11.2.min.js';
         $data['js_files'][] = PluginManager::$PLUGINS_PATH . '/playvideos/youtube_playlist.js';
     }
 
     return $data;
-}
\ No newline at end of file
+}
+
+/**
+ * This function is never called, but contains translation calls for GNU gettext extraction.
+ */
+function playvideos_dummy_translation()
+{
+    // meta
+    t('Add a button in the toolbar allowing to watch all videos.');
+}