From 75b69987b3562be62f41763a99bbf0684a603374 Mon Sep 17 00:00:00 2001
From: ArthurHoaro <arthur@hoa.ro>
Date: Wed, 15 Jul 2015 12:04:22 +0200
Subject: PLUGIN readityourself

Add an icon for each link (linklist) for ReadItYourself
---
 plugins/readityourself/book-open.png       | Bin 0 -> 568 bytes
 plugins/readityourself/config.php.dist     |   3 +++
 plugins/readityourself/readityourself.html |   1 +
 plugins/readityourself/readityourself.php  |  35 +++++++++++++++++++++++++++++
 4 files changed, 39 insertions(+)
 create mode 100644 plugins/readityourself/book-open.png
 create mode 100644 plugins/readityourself/config.php.dist
 create mode 100644 plugins/readityourself/readityourself.html
 create mode 100644 plugins/readityourself/readityourself.php

diff --git a/plugins/readityourself/book-open.png b/plugins/readityourself/book-open.png
new file mode 100644
index 00000000..36513d7b
Binary files /dev/null and b/plugins/readityourself/book-open.png differ
diff --git a/plugins/readityourself/config.php.dist b/plugins/readityourself/config.php.dist
new file mode 100644
index 00000000..d6b5cb85
--- /dev/null
+++ b/plugins/readityourself/config.php.dist
@@ -0,0 +1,3 @@
+<?php
+
+$GLOBALS['plugins']['READITYOUSELF_URL'] = 'http://someurl.com';
\ No newline at end of file
diff --git a/plugins/readityourself/readityourself.html b/plugins/readityourself/readityourself.html
new file mode 100644
index 00000000..fa0210f7
--- /dev/null
+++ b/plugins/readityourself/readityourself.html
@@ -0,0 +1 @@
+<span><a href="%s?url=%s"><img width="13" height="13" src="%s/readityourself/book-open.png" title="Read with Readityourself" /></a></span>
\ No newline at end of file
diff --git a/plugins/readityourself/readityourself.php b/plugins/readityourself/readityourself.php
new file mode 100644
index 00000000..7bad906d
--- /dev/null
+++ b/plugins/readityourself/readityourself.php
@@ -0,0 +1,35 @@
+<?php
+
+// If we're talking about https://github.com/memiks/readityourself
+// it seems kinda dead.
+// Not tested.
+
+// don't raise unnecessary warnings
+if (is_file(PluginManager::$PLUGINS_PATH . '/readityourself/config.php')) {
+    include PluginManager::$PLUGINS_PATH . '/readityourself/config.php';
+}
+
+if (!isset($GLOBALS['plugins']['READITYOUSELF_URL'])) {
+    header('Content-Type: text/plain; charset=utf-8');
+    echo 'ReadItYourself plugin error: '. PHP_EOL;
+    echo '  Please copy "plugins/readityourself/config.php.dist" to config.php and configure your readityourself URL.'. PHP_EOL;
+    echo '  You can also define "$GLOBALS[\'plugins\'][\'READITYOUSELF_URL\']" in your global Shaarli config.php file.';
+    exit;
+}
+
+/**
+ * Add readityourself icon to link_plugin when rendering linklist.
+ *
+ * @param $data - linklist data.
+ * @return mixed - linklist data with readityourself plugin.
+ */
+function hook_readityourself_render_linklist($data) {
+    $readityourself_html = file_get_contents(PluginManager::$PLUGINS_PATH . '/readityourself/readityourself.html');
+
+    foreach ($data['links'] as &$value) {
+        $readityourself = sprintf($readityourself_html, $GLOBALS['plugins']['READITYOUSELF_URL'], $value['url'], PluginManager::$PLUGINS_PATH);
+        $value['link_plugin'][] = $readityourself;
+    }
+
+    return $data;
+}
-- 
cgit v1.2.3


From b11c8f25dfe3c684f2f39352e3acdeb660a069ac Mon Sep 17 00:00:00 2001
From: ArthurHoaro <arthur@hoa.ro>
Date: Sun, 8 Nov 2015 12:16:55 +0100
Subject: unit tests for readityourself plugin + remove hard error

---
 plugins/readityourself/readityourself.php  | 22 ++++++---
 tests/plugins/PluginReadityourselfTest.php | 75 ++++++++++++++++++++++++++++++
 2 files changed, 90 insertions(+), 7 deletions(-)
 create mode 100644 tests/plugins/PluginReadityourselfTest.php

diff --git a/plugins/readityourself/readityourself.php b/plugins/readityourself/readityourself.php
index 7bad906d..ee7579c0 100644
--- a/plugins/readityourself/readityourself.php
+++ b/plugins/readityourself/readityourself.php
@@ -1,5 +1,9 @@
 <?php
 
+/**
+ * Plugin readityourself
+ */
+
 // If we're talking about https://github.com/memiks/readityourself
 // it seems kinda dead.
 // Not tested.
@@ -10,20 +14,24 @@ if (is_file(PluginManager::$PLUGINS_PATH . '/readityourself/config.php')) {
 }
 
 if (!isset($GLOBALS['plugins']['READITYOUSELF_URL'])) {
-    header('Content-Type: text/plain; charset=utf-8');
-    echo 'ReadItYourself plugin error: '. PHP_EOL;
-    echo '  Please copy "plugins/readityourself/config.php.dist" to config.php and configure your readityourself URL.'. PHP_EOL;
-    echo '  You can also define "$GLOBALS[\'plugins\'][\'READITYOUSELF_URL\']" in your global Shaarli config.php file.';
-    exit;
+    $GLOBALS['plugins']['errors'][] = 'Wallabag plugin error: '.
+        'Please define "$GLOBALS[\'plugins\'][\'WALLABAG_URL\']" '.
+        'in "plugins/wallabag/config.php" or in your Shaarli config.php file.';
 }
 
 /**
  * Add readityourself icon to link_plugin when rendering linklist.
  *
- * @param $data - linklist data.
+ * @param mixed $data - linklist data.
+ *
  * @return mixed - linklist data with readityourself plugin.
  */
-function hook_readityourself_render_linklist($data) {
+function hook_readityourself_render_linklist($data)
+{
+    if (!isset($GLOBALS['plugins']['READITYOUSELF_URL'])) {
+        return $data;
+    }
+
     $readityourself_html = file_get_contents(PluginManager::$PLUGINS_PATH . '/readityourself/readityourself.html');
 
     foreach ($data['links'] as &$value) {
diff --git a/tests/plugins/PluginReadityourselfTest.php b/tests/plugins/PluginReadityourselfTest.php
new file mode 100644
index 00000000..8bf17bf1
--- /dev/null
+++ b/tests/plugins/PluginReadityourselfTest.php
@@ -0,0 +1,75 @@
+<?php
+
+/**
+ * PluginReadityourselfTest.php.php
+ */
+
+require_once 'plugins/readityourself/readityourself.php';
+
+/**
+ * Class PluginWallabagTest
+ * Unit test for the Wallabag plugin
+ */
+class PluginReadityourselfTest extends PHPUnit_Framework_TestCase
+{
+    /**
+     * Reset plugin path
+     */
+    function setUp()
+    {
+        PluginManager::$PLUGINS_PATH = 'plugins';
+    }
+
+    /**
+     * Test render_linklist hook.
+     */
+    function testReadityourselfLinklist()
+    {
+        $GLOBALS['plugins']['READITYOUSELF_URL'] = 'value';
+        $str = 'http://randomstr.com/test';
+        $data = array(
+            'title' => $str,
+            'links' => array(
+                array(
+                    'url' => $str,
+                )
+            )
+        );
+
+        $data = hook_readityourself_render_linklist($data);
+        $link = $data['links'][0];
+        // data shouldn't be altered
+        $this->assertEquals($str, $data['title']);
+        $this->assertEquals($str, $link['url']);
+
+        // plugin data
+        $this->assertEquals(1, count($link['link_plugin']));
+        $this->assertNotFalse(strpos($link['link_plugin'][0], $str));
+    }
+
+    /**
+     * Test without config: nothing should happened.
+     */
+    function testReadityourselfLinklistWithoutConfig()
+    {
+        unset($GLOBALS['plugins']['READITYOUSELF_URL']);
+        $str = 'http://randomstr.com/test';
+        $data = array(
+            'title' => $str,
+            'links' => array(
+                array(
+                    'url' => $str,
+                )
+            )
+        );
+
+        $data = hook_readityourself_render_linklist($data);
+        $link = $data['links'][0];
+        // data shouldn't be altered
+        $this->assertEquals($str, $data['title']);
+        $this->assertEquals($str, $link['url']);
+
+        // plugin data
+        $this->assertArrayNotHasKey('link_plugin', $link);
+    }
+}
-- 
cgit v1.2.3