mprins / dokuwiki-plugin-backlinks

syntax.php

<?php
/**
 * DokuWiki Syntax Plugin Backlinks.
 *
 * Shows a list of pages that link back to a given page.
 *
 * Syntax:  {{backlinks>[pagename][#filterNS|!#filterNS]}}
 *
 *   [pagename] - a valid wiki pagename or a . for the current page
 *   [filterNS] - a valid,absolute namespace name, optionally prepended with ! to exclude
 *
 * @license GPL 2 (http://www.gnu.org/licenses/gpl.html)
 * @author  Michael Klier <chi@chimeric.de>
 * @author  Mark C. Prins <mprins@users.sf.net>
 */
if(!defined('DOKU_INC')) die();

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');

/**
 * All DokuWiki plugins to extend the parser/rendering mechanism
 * need to inherit from this class
 */
class syntax_plugin_backlinks extends DokuWiki_Syntax_Plugin {
    /**
     * Syntax Type.
     *
     * Needs to return one of the mode types defined in $PARSER_MODES in parser.php
     * @see DokuWiki_Syntax_Plugin::getType()
     */
    function getType()  { return 'substition'; }

It is generally recommended to explicitly declare the visibility for methods.

    /**
     * @see DokuWiki_Syntax_Plugin::getPType()
     */
    function getPType() { return 'block'; }

It is generally recommended to explicitly declare the visibility for methods.

    /**
     * @see Doku_Parser_Mode::getSort()
     */
    function getSort()  { return 304; }

It is generally recommended to explicitly declare the visibility for methods.

    /**
     * Connect pattern to lexer.
     * @see Doku_Parser_Mode::connectTo()
     */
    function connectTo($mode) {

It is generally recommended to explicitly declare the visibility for methods.

        $this->Lexer->addSpecialPattern('\{\{backlinks>.+?\}\}', $mode, 'plugin_backlinks');
    }

    /**
     * Handler to prepare matched data for the rendering process.
     * @see DokuWiki_Syntax_Plugin::handle()
     */
    function handle($match, $state, $pos, Doku_Handler $handler){

It is generally recommended to explicitly declare the visibility for methods.

        // Take the id of the source
        // It can be a rendering of a sidebar
        global $INFO;
        global $ID;
        $id = $ID;
        // If it's a sidebar, get the original id.
        if ($INFO != null) {
            $id = $INFO['id'];
        }

        $match = substr($match,12,-2); //strip {{backlinks> from start and }} from end

        if(strstr($match, "#")){
            $includeNS = substr(strstr($match, "#", FALSE), 1);
            $match= strstr($match, "#", TRUE);
        }

        $match = ($match == '.') ? $id : $match;

        if(strstr($match,".:")) {
            resolve_pageid(getNS($id),$match,$exists);

The variable $exists does not exist. Did you forget to declare it?

        }

        return (array($match, $includeNS));

The variable $includeNS does not seem to be defined for all execution paths leading up to this point.

    }

    /**
     * Handles the actual output creation.
     * @see DokuWiki_Syntax_Plugin::render()
     */
    function render($mode, Doku_Renderer $renderer, $data) {

It is generally recommended to explicitly declare the visibility for methods.

        global $lang;

        if($mode == 'xhtml'){
            $renderer->info['cache'] = false;

            @require_once(DOKU_INC.'inc/fulltext.php');

It seems like you do not handle an error condition here. This can introduce security issues, and is generally not recommended.

            $backlinks = ft_backlinks($data[0]);

            dbglog($backlinks, "backlinks: all backlinks to: $data[0]");

            $renderer->doc .= '<div id="plugin__backlinks">' . DW_LF;

            $filterNS = $data[1];
            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) {
                        return stripos($ns, $filterNS, 0) !== 0;
                    });
                } else {
                    dbglog($filterNS, "backlinks: including namespace: $filterNS only");
                    $backlinks= array_filter($backlinks, function($ns) use($filterNS) {
                        return stripos($ns, $filterNS, 0) === 0;
                    });
                }
            }

            dbglog($backlinks, "backlinks: all backlinks to be rendered");

            if(!empty($backlinks)) {

                $renderer->doc .= '<ul class="idx">';

                foreach($backlinks as $backlink){
                    $name = p_get_metadata($backlink, 'title');
                    if(empty($name)) $name = $backlink;
                    $renderer->doc .= '<li><div class="li">';
                    //$renderer->doc .= html_wikilink(':'.$backlink, $name);

53% of this comment could be valid code. Did you maybe forget this after debugging?

                    $renderer->doc .= '<a href="'.wl(':'.$backlink).'">'.$name.'</a>';
                    $renderer->doc .= '</div></li>' . DW_LF;
                }

                $renderer->doc .= '</ul>' . DW_LF;
            } else {
                $renderer->doc .= "<strong>Plugin Backlinks: " . $lang['nothingfound'] . "</strong>" . DW_LF;
            }

            $renderer->doc .= '</div>' . DW_LF;

            return true;
        }
        return false;
    }
}