mambax7 / smartfaq

include/functions.render.php

<?php

For compatibility and reusability of your code, PSR1 recommends that a file should introduce either new symbols (like classes, functions, etc.) or have side-effects (like outputting something, or including other files), but not both at the same time. The first symbol is defined on line 15 and the first side effect is on line 14.

/**
 * CBB 4.0, or newbb, the forum module for XOOPS project
 *
 * @copyright   XOOPS Project (http://xoops.org)
 * @license     http://www.fsf.org/copyleft/gpl.html GNU public license
 * @author      Taiwen Jiang (phppp or D.J.) <phppp@users.sourceforge.net>
 * @since       4.00
 * @package     module::newbb
 */

// defined('XOOPS_ROOT_PATH') || exit('XOOPS root path not defined');

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

defined('NEWBB_FUNCTIONS_INI') || include __DIR__ . '/functions.ini.php';
define('NEWBB_FUNCTIONS_RENDER_LOADED', true);

if (!defined('NEWBB_FUNCTIONS_RENDER')):
    define('NEWBB_FUNCTIONS_RENDER', 1);

    /*
     * Sorry, we have to use the stupid solution unless there is an option in MyTextSanitizer:: htmlspecialchars();
     */
    /**
     * @param $text
     * @return mixed
     */
    function sf_htmlSpecialChars($text)
    {
        return preg_replace(array('/&/i', '/ /i'), array('&', ' '), htmlspecialchars($text));
    }

    /**
     * @param      $text
     * @param  int $html
     * @param  int $smiley
     * @param  int $xcode
     * @param  int $image
     * @param  int $br
     * @return mixed
     */
    function &sf_displayTarea(&$text, $html = 0, $smiley = 1, $xcode = 1, $image = 1, $br = 1)
    {
        global $myts;

        if ($html != 1) {
            // html not allowed
            $text = sf_htmlSpecialChars($text);
        }
        $text = $myts->codePreConv($text, $xcode); // Ryuji_edit(2003-11-18)

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

        $text = $myts->makeClickable($text);
        if ($smiley != 0) {
            // process smiley
            $text = $myts->smiley($text);
        }
        if ($xcode != 0) {
            // decode xcode
            if ($image != 0) {
                // image allowed
                $text =& $myts->xoopsCodeDecode($text);
            } else {
                // image not allowed
                $text =& $myts->xoopsCodeDecode($text, 0);
            }
        }
        if ($br != 0) {
            $text =& $myts->nl2Br($text);
        }
        $text = $myts->codeConv($text, $xcode, $image);    // Ryuji_edit(2003-11-18)

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

        return $text;
    }

    /**
     * @param $document
     * @return string
     */
    function sf_html2text($document)
    {
        $text = strip_tags($document);

        return $text;
    }

    /**
     * Display forrum button
     *
     * @param          $link
     * @param          $button  image/button name, without extension
     * @param  string  $alt     alt message
     * @param  boolean $asImage true for image mode; false for text mode
     * @param  string  $extra   extra attribute for the button
     * @return mixed
     */
    function sf_getButton($link, $button, $alt = '', $asImage = true, $extra = "class='forum_button'")
    {
        $button = "<input type='button' name='{$button}' {$extra} value='{$alt}' onclick='window.location.href={$link}' />";
        if (empty($asImage)) {
            $button = "<a href='{$link}' title='{$alt}' {$extra}>" . sf_displayImage($button, $alt, true) . '</a>';
        }

        return $button;
    }

    /**
     * Display forrum images
     *
     * @param  string  $image   image name, without extension
     * @param  string  $alt     alt message
     * @param  boolean $display true for return image anchor; faulse for assign to $xoopsTpl
     * @param  string  $extra   extra attribute for the image
     * @return mixed
     */
    function sf_displayImage($image, $alt = '', $display = true, $extra = "class='forum_icon'")
    {
        $iconHandler = sf_getIconHandler();
        // START hacked by irmtfan
        // to show text links instead of buttons - func_num_args()==2 => only when $image, $alt is set and optional $display not set
        global $xoopsModuleConfig;
        if (func_num_args() == 2) {
            // overall setting
            if (!empty($xoopsModuleConfig['display_text_links'])) {
                $display = false;
            }
            // if set for each link => overwrite $display
            if (isset($xoopsModuleConfig['display_text_each_link'][$image])) {
                $display = empty($xoopsModuleConfig['display_text_each_link'][$image]);
            }
        }
        // END hacked by irmtfan
        if (empty($display)) {
            return $iconHandler->assignImage($image, $alt, $extra);
        } else {
            return $iconHandler->getImage($image, $alt, $extra);
        }
    }

    /**
     * @return NewbbIconHandler
     */
    function sf_getIconHandler()
    {
        global $xoTheme, $xoopsConfig;
        static $iconHandler;

        if (isset($iconHandler)) {
            return $iconHandler;
        }

        if (!class_exists('NewbbIconHandler')) {
            require_once dirname(__DIR__) . '/class/icon.php';
        }

        $iconHandler           = NewbbIconHandler::instance();
        $iconHandler->template = $xoTheme->template;
        $iconHandler->init($xoopsConfig['language']);

        return $iconHandler;
    }

endif;