mprins / dokuwiki-plugin-backlinks

_test/general.test.php

Spacing +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
      * Simple test to make sure the plugin.info.txt is in correct format
      */
     public function test_plugininfo() {
-        $file = __DIR__ . '/../plugin.info.txt';
+        $file = __DIR__.'/../plugin.info.txt';
         $this->assertFileExists($file);
 
         $info = confToHash($file);

Indentation +3 added lines, -3 removed lines

@@ -55,8 +55,8 @@
     public function test_plugin_backlinks_isloaded() {
         global $plugin_controller;
         $this->assertTrue(
-                   in_array('backlinks', $plugin_controller->getList()),
-                   "backlinks plugin is loaded"
-                         );
+                    in_array('backlinks', $plugin_controller->getList()),
+                    "backlinks plugin is loaded"
+                            );
     }
 }

_test/syntax.test.php

Indentation +6 added lines, -6 removed lines

@@ -72,7 +72,7 @@ 
     public function testIndex() {
         $query = array('ross');
         $this->assertEquals(
-                 array('ross' => array(
+                    array('ross' => array(
                     'link' => '3',
                     'bob_ross_says' => '1',
                     'backlinks_syntax' => '2',
@@ -82,7 +82,7 @@ 
                     'include:link' => '3',
                     'exclude:link' => '3'
                     )),
-                 idx_lookup($query)
+                    idx_lookup($query)
         );
     }
 
@@ -94,7 +94,7 @@ 
             strpos($response->getContent(), 'A link to Bob Ross') !== false,
             'A link to Bob Ross was not in the output'
         );
-     }
+        }
 
     public function testStoryPage() {
         $request = new TestRequest();
@@ -119,18 +119,18 @@ 
         //look for id="plugin__backlinks"
         $this->assertEquals(1, pq('#plugin__backlinks', $doc)->length,
                             'There should be one backlinks element'
-                           );
+                            );
 
         $wikilinks = pq('#plugin__backlinks ul li', $doc);
         dbglog($wikilinks->text(), 'found backlinks');
         $this->assertEquals(4, $wikilinks->contents()->length,
                             'There should be 4 backlinks'
-                           );
+                            );
 
         $lastlink = pq('a:last',$wikilinks);
         dbglog($lastlink->text(),"last backlink");
         $this->assertEquals($lastlink->text(),'A link to Bob Ross',
                             'The last backlink should be a link to Bob Ross'
-                           );
+                            );
     }
 }

Spacing +7 added lines, -7 removed lines

@@ -28,12 +28,12 @@ 
     /**
      * copy data and add pages to the index.
      */
-    public static function setUpBeforeClass(){
+    public static function setUpBeforeClass() {
         parent::setUpBeforeClass();
         global $conf;
         $conf['allowdebug'] = 1;
 
-        TestUtils::rcopy(TMP_DIR, dirname(__FILE__) . '/data/');
+        TestUtils::rcopy(TMP_DIR, dirname(__FILE__).'/data/');
 
         dbglog("\nset up class syntax_plugin_backlinks_test");
     }
@@ -52,13 +52,13 @@ 
 
         $verbose = false;
         $force = false;
-        foreach($data as $val) {
+        foreach ($data as $val) {
             idx_addPage($val['id'], $verbose, $force);
         }
         //idx_addPage('bob_ross_says', $verbose, $force);
         //idx_addPage('link', $verbose, $force);
         //idx_addPage('backlinks_syntax', $verbose, $force);
-        if($conf['allowdebug']) {
+        if ($conf['allowdebug']) {
             touch(DOKU_TMP_DATA.'cache/debug.log');
         }
     }
@@ -68,7 +68,7 @@ 
 
         global $conf;
         // try to get the debug log after running the test, print and clear
-        if($conf['allowdebug']) {
+        if ($conf['allowdebug']) {
             print "\n";
             readfile(DOKU_TMP_DATA.'cache/debug.log');
             unlink(DOKU_TMP_DATA.'cache/debug.log');
@@ -137,8 +137,8 @@ 
                             'There should be 4 backlinks'
                            );
 
-        $lastlink = pq('a:last',$wikilinks);
-        dbglog($lastlink->text(),"last backlink");
+        $lastlink = pq('a:last', $wikilinks);
+        dbglog($lastlink->text(), "last backlink");
         $this->assertEquals(
                             $lastlink->text(),
                             'A link to Bob Ross',

_test/syntax_exclude.test.php

Spacing +7 added lines, -7 removed lines

@@ -28,12 +28,12 @@ 
     /**
      * copy data and add pages to the index.
      */
-    public static function setUpBeforeClass(){
+    public static function setUpBeforeClass() {
         parent::setUpBeforeClass();
         global $conf;
         $conf['allowdebug'] = 1;
 
-        TestUtils::rcopy(TMP_DIR, dirname(__FILE__) . '/data/');
+        TestUtils::rcopy(TMP_DIR, dirname(__FILE__).'/data/');
 
         dbglog("\nset up class syntax_plugin_backlinks_test");
     }
@@ -52,13 +52,13 @@ 
 
         $verbose = false;
         $force = false;
-        foreach($data as $val) {
+        foreach ($data as $val) {
             idx_addPage($val['id'], $verbose, $force);
         }
         //idx_addPage('bob_ross_says', $verbose, $force);
         //idx_addPage('link', $verbose, $force);
         //idx_addPage('backlinks_syntax', $verbose, $force);
-        if($conf['allowdebug']) {
+        if ($conf['allowdebug']) {
             touch(DOKU_TMP_DATA.'cache/debug.log');
         }
     }
@@ -68,7 +68,7 @@ 
 
         global $conf;
         // try to get the debug log after running the test, print and clear
-        if($conf['allowdebug']) {
+        if ($conf['allowdebug']) {
             print "\n";
             readfile(DOKU_TMP_DATA.'cache/debug.log');
             unlink(DOKU_TMP_DATA.'cache/debug.log');
@@ -137,8 +137,8 @@ 
                             'There should be 4 backlinks'
                            );
 
-        $lastlink = pq('a:last',$wikilinks);
-        dbglog($lastlink->text(),"last backlink");
+        $lastlink = pq('a:last', $wikilinks);
+        dbglog($lastlink->text(), "last backlink");
         $this->assertEquals(
                             $lastlink->text(),
                             'A link to Bob Ross',

Indentation +3 added lines, -3 removed lines

@@ -88,7 +88,7 @@ 
                             1,
                             pq('#plugin__backlinks', $doc)->length,
                             'There should be one backlinks element'
-                           );
+                            );
 
         $wikilinks = pq('#plugin__backlinks ul li', $doc);
         dbglog($wikilinks->text(), 'found backlinks');
@@ -96,7 +96,7 @@ 
                             1,
                             $wikilinks->contents()->length,
                             'There should be 1 backlink'
-                           );
+                            );
 
         $lastlink = pq('a:last',$wikilinks);
         dbglog($lastlink->text(),"last backlink");
@@ -104,6 +104,6 @@ 
                             $lastlink->text(),
                             'An included link to Bob Ross',
                             'The last backlink should be "An included link to Bob Ross"'
-                           );
+                            );
     }
 }

_test/syntax_include.test.php

Indentation +3 added lines, -3 removed lines

@@ -88,7 +88,7 @@ 
                             1,
                             pq('#plugin__backlinks', $doc)->length,
                             'There should be one backlinks element'
-                           );
+                            );
 
         $wikilinks = pq('#plugin__backlinks ul li', $doc);
         dbglog($wikilinks->text(), 'found backlinks');
@@ -96,7 +96,7 @@ 
                             1,
                             $wikilinks->contents()->length,
                             'There should be 1 backlink'
-                           );
+                            );
 
         $lastlink = pq('a:last',$wikilinks);
         dbglog($lastlink->text(),"last backlink");
@@ -104,6 +104,6 @@ 
                             $lastlink->text(),
                             'An included link to Bob Ross',
                             'The last backlink should be "An included link to Bob Ross"'
-                           );
+                            );
     }
 }

Spacing +7 added lines, -7 removed lines

@@ -28,12 +28,12 @@ 
     /**
      * copy data and add pages to the index.
      */
-    public static function setUpBeforeClass(){
+    public static function setUpBeforeClass() {
         parent::setUpBeforeClass();
         global $conf;
         $conf['allowdebug'] = 1;
 
-        TestUtils::rcopy(TMP_DIR, dirname(__FILE__) . '/data/');
+        TestUtils::rcopy(TMP_DIR, dirname(__FILE__).'/data/');
 
         dbglog("\nset up class syntax_plugin_backlinks_test");
     }
@@ -52,13 +52,13 @@ 
 
         $verbose = false;
         $force = false;
-        foreach($data as $val) {
+        foreach ($data as $val) {
             idx_addPage($val['id'], $verbose, $force);
         }
         //idx_addPage('bob_ross_says', $verbose, $force);
         //idx_addPage('link', $verbose, $force);
         //idx_addPage('backlinks_syntax', $verbose, $force);
-        if($conf['allowdebug']) {
+        if ($conf['allowdebug']) {
             touch(DOKU_TMP_DATA.'cache/debug.log');
         }
     }
@@ -68,7 +68,7 @@ 
 
         global $conf;
         // try to get the debug log after running the test, print and clear
-        if($conf['allowdebug']) {
+        if ($conf['allowdebug']) {
             print "\n";
             readfile(DOKU_TMP_DATA.'cache/debug.log');
             unlink(DOKU_TMP_DATA.'cache/debug.log');
@@ -137,8 +137,8 @@ 
                             'There should be 4 backlinks'
                            );
 
-        $lastlink = pq('a:last',$wikilinks);
-        dbglog($lastlink->text(),"last backlink");
+        $lastlink = pq('a:last', $wikilinks);
+        dbglog($lastlink->text(), "last backlink");
         $this->assertEquals(
                             $lastlink->text(),
                             'A link to Bob Ross',

syntax.php

Spacing +20 added lines, -20 removed lines

@@ -32,7 +32,7 @@ 
      * Needs to return one of the mode types defined in $PARSER_MODES in parser.php
      * @see DokuWiki_Syntax_Plugin::getType()
      */
-    function getType()  { return 'substition'; }
+    function getType() { return 'substition'; }
 
     /**
      * @see DokuWiki_Syntax_Plugin::getPType()
@@ -42,7 +42,7 @@ 
     /**
      * @see Doku_Parser_Mode::getSort()
      */
-    function getSort()  { return 304; }
+    function getSort() { return 304; }
 
     /**
      * Connect pattern to lexer.
@@ -56,7 +56,7 @@ 
      * Handler to prepare matched data for the rendering process.
      * @see DokuWiki_Syntax_Plugin::handle()
      */
-    function handle($match, $state, $pos, Doku_Handler $handler){
+    function handle($match, $state, $pos, Doku_Handler $handler) {
 
         // Take the id of the source
         // It can be a rendering of a sidebar
@@ -69,17 +69,17 @@ 
         }
 
         // strip {{backlinks> from start and }} from end
-        $match = substr($match,12,-2);
+        $match = substr($match, 12, -2);
 
-        if(strstr($match, "#")){
+        if (strstr($match, "#")) {
             $includeNS = substr(strstr($match, "#", FALSE), 1);
-            $match= strstr($match, "#", TRUE);
+            $match = strstr($match, "#", TRUE);
         }
 
         $match = ($match == '.') ? $id : $match;
 
-        if(strstr($match,".:")) {
-            resolve_pageid(getNS($id),$match,$exists);
+        if (strstr($match, ".:")) {
+            resolve_pageid(getNS($id), $match, $exists);
         }
 
         return (array($match, $includeNS));
@@ -92,7 +92,7 @@ 
     function render($mode, Doku_Renderer $renderer, $data) {
         global $lang;
 
-        if($mode == 'xhtml'){
+        if ($mode == 'xhtml') {
             $renderer->info['cache'] = false;
 
             @require_once(DOKU_INC.'inc/fulltext.php');
@@ -100,19 +100,19 @@ 
 
             dbglog($backlinks, "backlinks: all backlinks to: $data[0]");
 
-            $renderer->doc .= '<div id="plugin__backlinks">' . DW_LF;
+            $renderer->doc .= '<div id="plugin__backlinks">'.DW_LF;
 
             $filterNS = $data[1];
-            if(!empty($backlinks) && !empty($filterNS)) {
+            if (!empty($backlinks) && !empty($filterNS)) {
                 if (stripos($filterNS, "!", 0) === 0) {
                     $filterNS = substr($filterNS, 1);
                     dbglog($filterNS, "backlinks: exluding all of namespace: $filterNS");
-                    $backlinks= array_filter($backlinks, function($ns) use($filterNS) {
+                    $backlinks = array_filter($backlinks, function($ns) use($filterNS) {
                         return stripos($ns, $filterNS, 0) !== 0;
                     });
                 } else {
                     dbglog($filterNS, "backlinks: including namespace: $filterNS only");
-                    $backlinks= array_filter($backlinks, function($ns) use($filterNS) {
+                    $backlinks = array_filter($backlinks, function($ns) use($filterNS) {
                         return stripos($ns, $filterNS, 0) === 0;
                     });
                 }
@@ -120,24 +120,24 @@ 
 
             dbglog($backlinks, "backlinks: all backlinks to be rendered");
 
-            if(!empty($backlinks)) {
+            if (!empty($backlinks)) {
 
                 $renderer->doc .= '<ul class="idx">';
 
-                foreach($backlinks as $backlink){
+                foreach ($backlinks as $backlink) {
                     $name = p_get_metadata($backlink, 'title');
-                    if(empty($name)) $name = $backlink;
+                    if (empty($name)) $name = $backlink;
                     $renderer->doc .= '<li><div class="li">';
                     $renderer->doc .= html_wikilink(':'.$backlink, $name);
-                    $renderer->doc .= '</div></li>' . DW_LF;
+                    $renderer->doc .= '</div></li>'.DW_LF;
                 }
 
-                $renderer->doc .= '</ul>' . DW_LF;
+                $renderer->doc .= '</ul>'.DW_LF;
             } else {
-                $renderer->doc .= "<strong>Plugin Backlinks: " . $lang['nothingfound'] . "</strong>" . DW_LF;
+                $renderer->doc .= "<strong>Plugin Backlinks: ".$lang['nothingfound']."</strong>".DW_LF;
             }
 
-            $renderer->doc .= '</div>' . DW_LF;
+            $renderer->doc .= '</div>'.DW_LF;
 
             return true;
         }

Braces +12 added lines, -4 removed lines

@@ -13,10 +13,16 @@ 
  * @author  Michael Klier <chi@chimeric.de>
  * @author  Mark C. Prins <mprins@users.sf.net>
  */
-if (!defined('DOKU_INC')) die();
+if (!defined('DOKU_INC')) {
+    die();
+}
 
-if (!defined('DOKU_PLUGIN')) define('DOKU_PLUGIN', DOKU_INC.'lib/plugins/');
-if (!defined('DW_LF')) define('DW_LF', "\n");
+if (!defined('DOKU_PLUGIN')) {
+    define('DOKU_PLUGIN', DOKU_INC.'lib/plugins/');
+}
+if (!defined('DW_LF')) {
+    define('DW_LF', "\n");
+}
 
 require_once(DOKU_PLUGIN.'syntax.php');
 require_once(DOKU_INC.'inc/parserutils.php');
@@ -126,7 +132,9 @@ 
 
                 foreach($backlinks as $backlink){
                     $name = p_get_metadata($backlink, 'title');
-                    if(empty($name)) $name = $backlink;
+                    if(empty($name)) {
+                        $name = $backlink;
+                    }
                     $renderer->doc .= '<li><div class="li">';
                     $renderer->doc .= html_wikilink(':'.$backlink, $name);
                     $renderer->doc .= '</div></li>' . DW_LF;