functions.php ➔ lx_extract_keywords() - Code Metrics - Inspection of "Added support for Cyrillic, and other non-standard..." - XoopsModules25x/lexikon - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Pull Request — master (#12)

unknown

created 2017-09-30 10:19 UTC

functions.php ➔ lx_extract_keywords() D

↳ Parent: Project

Complexity

Conditions	15
Paths	144

Size

Total Lines	119
Code Lines	106

Duplication

Lines	63
Ratio	52.94 %

Importance

Changes

Metric	Value
cc	15
eloc	106
nc	144
nop	1
dl	63
loc	119
rs	4.597
c	0
b	0
f	0

How to fix Long Method Complexity

<?php

/**
 * $Id: functions.php v 1.0 8 May 2004 hsalazar Exp $
 * Module: Lexikon
 * Author: hsalazar
 * Additions and Modifications: Yerres
 * Licence: GNU
 */

if( ! defined( 'XOOPS_ROOT_PATH' ) ) die( 'XOOPS root path not defined' ) ;

/**
 * lx_getLinkedUnameFromId()
 *
 * @param integer $userid Userid of author etc
 * @param integer $name:  0 Use Usenamer 1 Use realname
/**
 * @param array $germany
 * @param array $island
 * @param array $italy
 */
function finale($germany, $island) {
    return "2:1";
}
 * @return
 **/
function lx_getLinkedUnameFromId($userid = 0, $name= 0) {
// Bad
class Router
{
    public function generate($path)
    {
        return $_SERVER['HOST'].$path;
    }
}

// Better
class Router
{
    private $host;

    public function __construct($host)
    {
        $this->host = $host;
    }

    public function generate($path)
    {
        return $this->host.$path;
    }
}

class Controller
{
    public function myAction(Request $request)
    {
        // Instead of
        $page = isset($_GET['page']) ? intval($_GET['page']) : 1;

        // Better (assuming you use the Symfony2 request)
        $page = $request->query->get('page', 1);
    }
}
    if (!is_numeric($userid)) {
        return $userid;
    }

    $userid = intval($userid);
    if ($userid > 0) {
        $member_handler = xoops_gethandler('member');
        $user = $member_handler->getUser($userid);

        if (is_object($user)) {
            $ts = MyTextSanitizer::getInstance();
            $username = $user->getVar('uname');
            $usernameu = $user->getVar('name');

            if ( ($name) && !empty($usernameu))  {
                $username = $user->getVar('name');
            }
            if ( !empty($usernameu)) {
                $linkeduser = "$usernameu [<a href='".XOOPS_URL."/userinfo.php?uid=".$userid."'>". $ts->htmlSpecialChars($username) ."</a>]";
            } else {
                $linkeduser = "<a href='".XOOPS_URL."/userinfo.php?uid=".$userid."'>". ucfirst($ts->htmlSpecialChars($username)) ."</a>";
            }

            return $linkeduser;
        }
    }

    return $GLOBALS['xoopsConfig']['anonymous'];
}

function lx_getuserForm($user) {

    global $xoopsDB, $xoopsConfig;
function myFunction($a, $b) {
    // Do something
}

    echo "<select name='author'>";
    echo "<option value='-1'>------</option>";
    $result = $xoopsDB->query("SELECT uid, uname FROM ".$xoopsDB->prefix("users")." ORDER BY uname");

    while (list($uid, $uname) = $xoopsDB->fetchRow($result)) {
        if ( $uid == $user ) {
            $opt_selected = "selected='selected'";
        } else {
            $opt_selected = "";
        }
        echo "<option value='".$uid."' $opt_selected>".$uname."</option>";
    }
    echo "</select></div>";
}

function lx_calculateTotals() {

    global $xoopsUser, $xoopsDB, $xoopsModule;
function myFunction($a, $b) {
    // Do something
}
    $groups = is_object($xoopsUser) ? $xoopsUser->getGroups() : XOOPS_GROUP_ANONYMOUS;
    $gperm_handler = xoops_gethandler('groupperm');

    $result01 = $xoopsDB -> query( "SELECT categoryID, total FROM " . $xoopsDB -> prefix( "lxcategories" ) . " " );
    list ( $totalcategories ) = $xoopsDB -> getRowsNum( $result01 );
<?php

function returnThreeValues() {
    return array('a', 'b', 'c');
}

list($a, $b, $c) = returnThreeValues();

print $a . " - " . $c;
    while (list ( $categoryID, $total ) = $xoopsDB -> fetchRow ( $result01 )) {
<?php

function returnThreeValues() {
    return array('a', 'b', 'c');
}

list($a, $b, $c) = returnThreeValues();

print $a . " - " . $c;
        if ($gperm_handler->checkRight('lexikon_view', $categoryID, $groups, $xoopsModule->getVar('mid'))) {
            $newcount = lx_countByCategory ( $categoryID );
            $xoopsDB -> queryF( "UPDATE " . $xoopsDB -> prefix( "lxcategories" ) . " SET total = '$newcount' WHERE categoryID = '$categoryID'");
        }
    }
}

function lx_countByCategory( $c ) {

    global $xoopsUser, $xoopsDB, $xoopsModule;
function myFunction($a, $b) {
    // Do something
}
    $groups = is_object($xoopsUser) ? $xoopsUser->getGroups() : XOOPS_GROUP_ANONYMOUS;
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
    $gperm_handler = xoops_gethandler('groupperm');
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
    $count = 0;
    $sql = $xoopsDB -> query( "SELECT entryID FROM " . $xoopsDB -> prefix( "lxentries" ) . " WHERE offline = '0' AND categoryID = '$c'" );
    while ( $myrow = $xoopsDB -> fetchArray( $sql ) ) {
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
        //if ($gperm_handler->checkRight('lexikon_view', $c, $groups, $xoopsModule->getVar('mid'))) {

        $count++;
        //}
    }

    return $count;
}

function lx_countCats () {

    global $xoopsUser, $xoopsModule;
function myFunction($a, $b) {
    // Do something
}
    $gperm_handler = xoops_gethandler('groupperm');
    $groups = (is_object($xoopsUser)) ? $xoopsUser -> getGroups() : XOOPS_GROUP_ANONYMOUS;
    $totalcats = $gperm_handler->getItemIds("lexikon_view", $groups, $xoopsModule->getVar('mid'));

    return count($totalcats);
}

function lx_countWords () {

    global $xoopsUser, $xoopsDB;
function myFunction($a, $b) {
    // Do something
}
    $gperm_handler = xoops_gethandler('groupperm');
    $groups = is_object($xoopsUser) ? $xoopsUser->getGroups() : XOOPS_GROUP_ANONYMOUS;
      $module_handler = xoops_gethandler('module');
    $module = $module_handler->getByDirname('lexikon');
    $module_id = $module->getVar('mid');
      $allowed_cats = $gperm_handler->getItemIds("lexikon_view", $groups, $module_id);
      $catids = implode(',', $allowed_cats);
      $catperms = " AND categoryID IN ($catids) ";

    $pubwords = $xoopsDB -> query( "SELECT * FROM " . $xoopsDB -> prefix( "lxentries" ) . " WHERE submit = '0' AND offline ='0' AND request = '0' ".$catperms." " );
    $publishedwords = $xoopsDB -> getRowsNum ( $pubwords );

    return $publishedwords;
}

// To display the list of categories
function lx_CatsArray(){
    global $xoopsDB, $xoopsModuleConfig,$xoopsUser,$xoopsModule;
function myFunction($a, $b) {
    // Do something
}
    $myts = MyTextSanitizer::getInstance();
    $groups = is_object($xoopsUser) ? $xoopsUser->getGroups() : XOOPS_GROUP_ANONYMOUS;
    $gperm_handler = xoops_gethandler('groupperm');
    $block0 = [];
    $count = 1;
    $resultcat = $xoopsDB -> query ( "SELECT categoryID, name, total, logourl FROM " . $xoopsDB -> prefix ( "lxcategories") . " ORDER BY weight ASC" );
    while (list( $catID, $name, $total, $logourl) = $xoopsDB->fetchRow($resultcat)) {
        if ($gperm_handler->checkRight('lexikon_view', $catID, $groups, $xoopsModule->getVar('mid'))) {
            $catlinks = [];
            $count++;
            if ($logourl && $logourl != "http://") {
                $logourl = $myts->htmlSpecialChars($logourl);
            } else {
                $logourl = '';
            }
            $xoopsModule = XoopsModule::getByDirname("lexikon");
            $catlinks['id'] = intval($catID);
            $catlinks['total'] = intval($total);
            $catlinks['linktext'] = $myts -> htmlSpecialChars( $name );
            $catlinks['image'] = $logourl;
            $catlinks['count'] = intval($count);

            $block0['categories'][] = $catlinks;
        }
    }

    return $block0;
}

function lx_alphaArray () {
    global $xoopsUser, $xoopsDB, $xoopsModule;
function myFunction($a, $b) {
    // Do something
}
    $gperm_handler = xoops_gethandler('groupperm');
    $groups = is_object($xoopsUser) ? $xoopsUser->getGroups() : XOOPS_GROUP_ANONYMOUS;
      $module_handler = xoops_gethandler('module');
    $module = $module_handler->getByDirname('lexikon');
    $module_id = $module->getVar('mid');
    $allowed_cats = $gperm_handler->getItemIds("lexikon_view", $groups, $module_id);
    $catids = implode(',', $allowed_cats);
      $catperms = " AND categoryID IN ($catids) ";
    $alpha = [];
    for ($a = 65; $a < (65+26); $a++ ) {

        $letterlinks = [];
        $initial = chr($a);
        $sql = $xoopsDB -> query ( "SELECT entryID FROM " . $xoopsDB -> prefix ( "lxentries") . " WHERE init = '$initial' AND submit = '0' AND offline ='0' AND request = '0' ".$catperms."");
        $howmany = $xoopsDB -> getRowsNum( $sql );
        $letterlinks['total'] = $howmany;
        $letterlinks['id'] = chr($a);
        $letterlinks['linktext'] = chr($a);

        $alpha['initial'][] = $letterlinks;
    }

    return $alpha;
}

/**
 * chr() with unicode support
 * I found this on this site http://en.php.net/chr
 * don't take credit for this.
 *
 */
function lx_uchr ($initials) {

    if (is_scalar($initials)) $initials= func_get_args();
    $str= '';
    foreach ($initials as $init) $str.= html_entity_decode('&#'.$init.';',ENT_NOQUOTES,'UTF-8');

    return $str;
}
/* sample */
/*

    echo lx_uchr(23383); echo '<br/>';
    echo lx_uchr(23383,215,23383); echo '<br/>';
    echo lx_uchr(array(23383,215,23383,215,23383)); echo '<br/>';
*/

// Functional links
function lx_serviceLinks ( $variable ) {

    global $xoopsUser, $xoopsDB, $xoopsModule, $xoopsModuleConfig, $xoopsConfig, $entrytype;
function myFunction($a, $b) {
    // Do something
}

    $module_handler  = xoops_gethandler('module');
    $moduleInfo = $module_handler->get($xoopsModule->getVar('mid'));
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
    $pathIcon16 = $xoopsModule->getInfo('icons16');

    $srvlinks = "";
    if ( $xoopsUser ) {
        if ( $xoopsUser->isAdmin() ) {
            $srvlinks .= "<a title=\""
                          . _EDIT
                          . "\" href=\"admin/entry.php?op=mod&entryID="
                          . $variable['id']
                          . "\" target=\"_blank\"><img src=\""
                          . $pathIcon16
                          . "/edit.png\" alt=\""
                          . _MD_LEXIKON_EDITTERM
                          . "\" style=\"width:16px; height:16px;\"></a>&nbsp;<a TITLE=\""
                          . _DELETE
                          . "\" href=\"admin/entry.php?op=del&entryID="
                          . $variable['id']
                          . "\" target=\"_self\"><img src=\""
                          . $pathIcon16
                          . "/delete.png\" alt=\""
                          . _MD_LEXIKON_DELTERM
                          . "\" style=\"width:16px; height:16px;\"></a>&nbsp;";
        }
    }
    if ( $entrytype != "1" ) {
        $srvlinks .= "<a title=\""
                      . _MD_LEXIKON_PRINTTERM
                      . "\" href=\"print.php?entryID="
                      . $variable['id']
                      . "\" target=\"_blank\"><img src=\""
                      . $pathIcon16
                      . "/printer.png\" alt=\""
                      . _MD_LEXIKON_PRINTTERM
                      . "\" style=\"width:16px; height:16px;\"></a>&nbsp;<a TITLE=\""
                      . _MD_LEXIKON_SENDTOFRIEND
                      . "\" href=\"mailto:?subject="
                      . sprintf(_MD_LEXIKON_INTENTRY,$xoopsConfig["sitename"])
                      . "&amp;body="
                      . sprintf(_MD_LEXIKON_INTENTRYFOUND, $xoopsConfig['sitename'])
                      . ": "
                      . XOOPS_URL
                      . "/modules/".$xoopsModule->dirname()
                      . "/entry.php?entryID="
                      . $variable['id']
                      . " \" target=\"_blank\"><img src=\""
                      . $pathIcon16
                      . "/mail_replay.png\" alt=\""
                      . _MD_LEXIKON_SENDTOFRIEND
                      . "\" style=\"width:16px; height:16px;\"></a>&nbsp;";
        if (( $xoopsModuleConfig['com_rule'] != 0 ) && (!empty($xoopsModuleConfig['com_anonpost']) || is_object($xoopsUser))) {
            $srvlinks .= "<a title=\""
                          . _COMMENTS
                          . "?\" href=\"comment_new.php?com_itemid="
                          . $variable['id']
                          . "\" target=\"_parent\"><img src=\"images/comments.gif\" alt=\""
                          . _COMMENTS
                          . "?\" style=\"width:16px; height:16px;\"></a>&nbsp;";
            }
    }

    return $srvlinks;
}
// entry footer
function lx_serviceLinksnew ( $variable ) {

    global $xoopsUser, $xoopsDB, $xoopsModule, $xoopsModuleConfig, $xoopsConfig, $myts;
function myFunction($a, $b) {
    // Do something
}
    $srvlinks2 = "<a title=\""
                  . _MD_LEXIKON_PRINTTERM
                  . "\" href=\"print.php?entryID="
                  . $variable['id']
                  . "\" target=\"_blank\"><img src=\"images/print.gif\" alt=\""
                  . _MD_LEXIKON_PRINTTERM
                  . "\" style=\"vertical-align: middle; width:16px; height:16px; margin: 2px 4px;\"> "
                  . _MD_LEXIKON_PRINTTERM2
                  . "</a>&nbsp; <a title=\""
                  . _MD_LEXIKON_SENDTOFRIEND
                  . "\" href=\"mailto:?subject="
                  . sprintf(_MD_LEXIKON_INTENTRY,$xoopsConfig["sitename"])
                  . "&amp;body="
                  . sprintf(_MD_LEXIKON_INTENTRYFOUND, $xoopsConfig['sitename'])
                  . ": ".$variable['term']
                  . " "
                  . XOOPS_URL
                  . "/modules/"
                  . $xoopsModule->dirname()
                  . "/entry.php?entryID="
                  . $variable['id']
                  . " \" target=\"_blank\"><img src=\"images/friend.gif\" alt=\""
                  . _MD_LEXIKON_SENDTOFRIEND
                  . "\" style=\"vertical-align: middle; width:16px; height:16px; margin: 2px 4px;\"> "
                  . _MD_LEXIKON_SENDTOFRIEND2
                  . "</a>&nbsp;";

    return $srvlinks2;
}

function lx_showSearchForm() {

    global $xoopsUser, $xoopsDB, $xoopsModule, $xoopsModuleConfig, $xoopsConfig;
function myFunction($a, $b) {
    // Do something
}
    $gperm_handler = xoops_gethandler('groupperm');
    $groups = is_object($xoopsUser) ? $xoopsUser->getGroups() : XOOPS_GROUP_ANONYMOUS;
    
    $searchform = "<table style=\"width:100%;\">";
    $searchform .= "<form name=\"op\" id=\"op\" action=\"search.php\" method=\"post\">";
    $searchform .= "<tr><td style=\"text-align: right; line-height: 200%; width:150px;\">";
    $searchform .= _MD_LEXIKON_LOOKON."</td><td style=\"width:10px;\">&nbsp;</td><td style=\"text-align: left;\">";
    $searchform .= "<select name=\"type\"><option value=\"1\">"._MD_LEXIKON_TERMS."</option><option value=\"2\">"._MD_LEXIKON_DEFINS."</option>";
    $searchform .= "<option SELECTED value=\"3\">"._MD_LEXIKON_TERMSDEFS."</option></select></td></tr>";

    if ($xoopsModuleConfig['multicats'] == 1) {
        $searchform .= "<tr><td style=\"text-align: right; line-height: 200%;\">"._MD_LEXIKON_CATEGORY."</td>";
        $searchform .= "<td>&nbsp;</td><td style=\"text-align: left;\">";
        $resultcat = $xoopsDB -> query ( "SELECT categoryID, name FROM " . $xoopsDB -> prefix ( "lxcategories") . " ORDER BY categoryID" );
        $searchform .= "<select name=\"categoryID\">";
        $searchform .= "<option value=\"0\">"._MD_LEXIKON_ALLOFTHEM."</option>";

        while (list( $categoryID, $name) = $xoopsDB->fetchRow($resultcat)) {
            if ($gperm_handler->checkRight('lexikon_view', intval($categoryID), $groups, $xoopsModule->getVar('mid'))) {
                $searchform .= "<option value=\"$categoryID\">$categoryID : $name</option>";
            }
        }
        $searchform .= "</select></td></tr>";
    }

    $searchform .= "<tr><td style=\"text-align: right; line-height: 200%;\">";
    $searchform .= _MD_LEXIKON_TERM."</td><td>&nbsp;</td><td style=\"text-align: left;\">";
    $searchform .= "<input type=\"text\" name=\"term\" class=\"searchBox\" /></td></tr><tr>";
    $searchform .= "<td>&nbsp;</td><td>&nbsp;</td><td><input type=\"submit\" class=\"btnDefault\" value=\""._MD_LEXIKON_SEARCH."\" />";
    $searchform .= "</td></tr></form></table>";

    return $searchform;
}

function lx_getHTMLHighlight($needle, $haystack, $hlS, $hlE) {

    $parts = explode(">", $haystack);
    foreach($parts as $key=>$part) {
        $pL = "";
        $pR = "";

        if (($pos = strpos($part, "<")) === false)
            $pL = $part;
        elseif($pos > 0) {
            $pL = substr($part, 0, $pos);
            $pR = substr($part, $pos, strlen($part));
        }
        if ($pL != "")
            $parts[$key] = preg_replace('|('.quotemeta($needle).')|iU', $hlS.'\\1'.$hlE, $pL) . $pR;
    }

    return(implode(">", $parts));
}

/* *******************************************************************************
 * Most of the following functions are modified functions from Herve's News Module
 * other functions are from  AMS by Novasmart/Mithrandir
 * others from Red Mexico Soft Rmdp
 * others from Xhelp 0.78 thanks to ackbarr and eric_juden
 * *******************************************************************************
 */

// Create the meta keywords based on content
function lx_extract_keywords($content) {
// Bad
class Router
{
    public function generate($path)
    {
        return $_SERVER['HOST'].$path;
    }
}

// Better
class Router
{
    private $host;

    public function __construct($host)
    {
        $this->host = $host;
    }

    public function generate($path)
    {
        return $this->host.$path;
    }
}

class Controller
{
    public function myAction(Request $request)
    {
        // Instead of
        $page = isset($_GET['page']) ? intval($_GET['page']) : 1;

        // Better (assuming you use the Symfony2 request)
        $page = $request->query->get('page', 1);
    }
}
    global $xoopsTpl, $xoTheme, $xoopsModule, $xoopsModuleConfig;
function myFunction($a, $b) {
    // Do something
}
    include_once XOOPS_ROOT_PATH.'/modules/lexikon/include/common.inc.php';
    $keywords_count = $xoopsModuleConfig['metakeywordsnum'];
    $tmp=array();
    if (isset($_SESSION['xoops_keywords_limit'])) {    // Search the "Minimum keyword length"
        $limit = $_SESSION['xoops_keywords_limit'];
    } else {
        $config_handler = xoops_gethandler('config');
        $xoopsConfigSearch = $config_handler->getConfigsByCat(XOOPS_CONF_SEARCH);
        $limit=$xoopsConfigSearch['keyword_min'];
        $_SESSION['xoops_keywords_limit']=$limit;
    }
    $myts = MyTextSanitizer::getInstance();
    $content = str_replace ("<br />", " ", $content);
    $content= $myts->undoHtmlSpecialChars($content);
    $content= strip_tags($content);
    $content=strtolower($content);
    $search_pattern=[
                    "&nbsp;",
                    "\t",
                    "\r\n",
                    "\r",
                    "\n",
                    ",",
                    ".",
                    "'",
                    ";",
                    ":",
                    ")",
                    "(",
                    '"',
                    '?',
                    '!',
                    '{',
                    '}',
                    '[',
                    ']',
                    '<',
                    '>',
                    '/',
                    '+',
                    '-',
                    '_',
                    '\\',
                    '*'
                    ];
    $replace_pattern=[
                    ' ',
                    ' ',
                    ' ',
                    ' ',
                    ' ',
                    ' ',
                    ' ',
                    ' ',
                    '',
                    '',
                    '',
                    '',
                    '',
                    '',
                    '',
                    '',
                    '',
                    '',
                    '',
                    '',
                    '',
                    '',
                    '',
                    '',
                    '',
                    '',
                    ''
                    ];
    $content = str_replace($search_pattern, $replace_pattern, $content);
    $keywords=explode(' ',$content);
    switch (META_KEYWORDS_ORDER) {
    case 1:    // Returns keywords in the same order that they were created in the text
            $keywords=array_unique($keywords);
        break;

    case 2:    // the keywords order is made according to the reverse keywords frequency (so the less frequent words appear in first in the list)
        $keywords=array_count_values($keywords);
        asort($keywords);
        $keywords=array_keys($keywords);
        break;

    case 3:    // Same as previous, the only difference is that the most frequent words will appear in first in the list
        $keywords=array_count_values($keywords);
        arsort($keywords);
        $keywords=array_keys($keywords);
        break;
    }
    foreach($keywords as $keyword) {
        if (strlen($keyword)>=$limit && !is_numeric($keyword)) {
            $tmp[]=$keyword;
        }
    }
    $tmp=array_slice($tmp,0,$keywords_count);
    if (count($tmp)>0) {
        if (isset($xoTheme) && is_object($xoTheme)) {
            $xoTheme->addMeta( 'meta', 'keywords', implode(',',$tmp));
        } else {    // Compatibility for old Xoops versions
            $xoopsTpl->assign('xoops_meta_keywords', implode(',',$tmp));
        }
    } else {
        if (!isset($config_handler) || !is_object($config_handler)) {
            $config_handler = xoops_gethandler('config');
        }
        $xoopsConfigMetaFooter = $config_handler->getConfigsByCat(XOOPS_CONF_METAFOOTER);
        if (isset($xoTheme) && is_object($xoTheme)) {
            $xoTheme->addMeta( 'meta', 'keywords', $xoopsConfigMetaFooter['meta_keywords']);
        } else {    // Compatibility for old Xoops versions
            $xoopsTpl->assign('xoops_meta_keywords', $xoopsConfigMetaFooter['meta_keywords']);
        }
    }
}

// Create meta description based on content
function lx_get_metadescription($content) {

    global $xoopsTpl, $xoTheme;
function myFunction($a, $b) {
    // Do something
}
    $myts = MyTextSanitizer::getInstance();
    $content= $myts->undoHtmlSpecialChars($myts->displayTarea($content));
    if (isset($xoTheme) && is_object($xoTheme)) {
        $xoTheme->addMeta( 'meta', 'description', strip_tags($content));
    } else {  // Compatibility for old Xoops versions
        $xoopsTpl->assign('xoops_meta_description', strip_tags($content));
    }
}

// Create pagetitles
function lx_create_pagetitle($article='', $topic='') {

    global $xoopsModule, $xoopsTpl;
function myFunction($a, $b) {
    // Do something
}
    $myts = MyTextSanitizer::getInstance();
    $content='';
    if (!empty($article)) $content .= strip_tags($myts->displayTarea($article));
    if (!empty($topic)) {
        if (xoops_trim($content)!='') {
            $content .= ' - '.strip_tags($myts->displayTarea($topic));
        } else {
            $content .= strip_tags($myts->displayTarea($topic));
        }
    }
    if (is_object($xoopsModule) && xoops_trim($xoopsModule->name())!='') {
        if (xoops_trim($content)!='') {
            $content.=' - '.strip_tags($myts->displayTarea($xoopsModule->name()));
        } else {
            $content.=strip_tags($myts->displayTarea($xoopsModule->name()));
        }
    }
    if ($content!='') {
        $xoopsTpl->assign('xoops_pagetitle', $content);
    }
}

// clear descriptions
function lx_html2text($document) {

    // PHP Manual:: function preg_replace $document should contain an HTML document.
    // This will remove HTML tags, javascript sections and white space. It will also
    // convert some common HTML entities to their text equivalent.

    $search       = ["'<script[^>]*?>.*?</script>'si",  // Strip out javascript
                     "'<[\/\!]*?[^<>]*?>'si",          // Strip out HTML tags
                     "'([\r\n])[\s]+'",                // Strip out white space
                     "'&(quot|#34);'i",                // Replace HTML entities
                     "'&(amp|#38);'i",
                     "'&(lt|#60);'i",
                     "'&(gt|#62);'i",
                     "'&(nbsp|#160);'i",
                     "'&(iexcl|#161);'i",
                     "'&(cent|#162);'i",
                     "'&(pound|#163);'i",
                     "'&(copy|#169);'i"];

    $replace       = ["",
                      "",
                      "\\1",
                      "\"",
                      "&",
                      "<",
                      ">",
                      " ",
                      chr(161),
                      chr(162),
                      chr(163),
                      chr(169)];

    $text = preg_replace($search, $replace, $document);

    $text = preg_replace_callback("&#(\d+)&", create_function('$matches',"return chr(\$matches[1]);"),$text);
// Instead of
$function = create_function('$a, $b', 'return $a + $b');

// Better use
$function = function($a, $b) { return $a + $b; }

    return $text;
}

//Retrieve moduleoptions equivalent to $Xoopsmoduleconfig
function lx_getmoduleoption($option, $repmodule='lexikon') {

    global $xoopsModuleConfig, $xoopsModule;
function myFunction($a, $b) {
    // Do something
}
    static $tbloptions= [];
    if (is_array($tbloptions) && array_key_exists($option,$tbloptions)) {
        return $tbloptions[$option];
    }

    $retval=false;
    if (isset($xoopsModuleConfig) && (is_object($xoopsModule) && $xoopsModule->getVar('dirname') == $repmodule && $xoopsModule->getVar('isactive'))) {
        if (isset($xoopsModuleConfig[$option])) {
            $retval= $xoopsModuleConfig[$option];
        }
    } else {
        $module_handler = xoops_gethandler('module');
        $module = $module_handler->getByDirname($repmodule);
        $config_handler = xoops_gethandler('config');
        if ($module) {
            $moduleConfig = $config_handler->getConfigsByCat(0, $module->getVar('mid'));
            if (isset($moduleConfig[$option])) {
                $retval= $moduleConfig[$option];
            }
        }
    }
    $tbloptions[$option]=$retval;

    return $retval;
}

/**
 * Is Xoops 2.3.x ?
 *
 * @return boolean need to say it ?
 */
function lx_isX23()

{
    $x23 = false;
    $xv = str_replace('XOOPS ','',XOOPS_VERSION);
    if(substr($xv,2,1) >= '3') {
        $x23 = true;
    }

    return $x23;
}

/**
 * Retreive an editor according to the module's option "form_options"
 * following function is from News modified by trabis
 */
function &lx_getWysiwygForm($caption, $name, $value = '', $width = '100%', $height = '400px', $supplemental='')

{
    $editor_option = strtolower(lx_getmoduleoption('form_options'));
    $editor = false;
    $editor_configs = [];
    $editor_configs['name'] = $name;
    $editor_configs['value'] = $value;
    $editor_configs['rows'] = 35;
    $editor_configs['cols'] = 60;
    $editor_configs['width'] = '100%';
    $editor_configs['height'] = '350px';
    $editor_configs['editor'] = $editor_option;

    if(lx_isX23()) {
        $editor = new XoopsFormEditor($caption, $name, $editor_configs);

        return $editor;
            }

    // Only for Xoops 2.0.x
    switch($editor_option) {
        case 'fckeditor':
            if ( is_readable(XOOPS_ROOT_PATH . '/class/fckeditor/formfckeditor.php'))    {
                require_once(XOOPS_ROOT_PATH . '/class/fckeditor/formfckeditor.php');
                $editor = new XoopsFormFckeditor($caption, $name, $value);
            }
            break;

        case 'htmlarea':
                if ( is_readable(XOOPS_ROOT_PATH . '/class/htmlarea/formhtmlarea.php'))    {
                require_once(XOOPS_ROOT_PATH . '/class/htmlarea/formhtmlarea.php');
                    $editor = new XoopsFormHtmlarea($caption, $name, $value);
            }
            break;

        case 'dhtmltextarea':
        case 'dhtml':
                $editor = new XoopsFormDhtmlTextArea($caption, $name, $value, 10, 50, $supplemental);
            break;

        case 'textarea':
            $editor = new XoopsFormTextArea($caption, $name, $value);
            break;

        case 'tinyeditor':
        case 'tinymce':
            if ( is_readable(XOOPS_ROOT_PATH.'/class/xoopseditor/tinyeditor/formtinyeditortextarea.php')) {
                require_once XOOPS_ROOT_PATH.'/class/xoopseditor/tinyeditor/formtinyeditortextarea.php';
                $editor = new XoopsFormTinyeditorTextArea(array('caption'=> $caption, 'name'=>$name, 'value'=>$value, 'width'=>'100%', 'height'=>'400px'));
            }
            break;

        case 'koivi':
                if ( is_readable(XOOPS_ROOT_PATH . '/class/wysiwyg/formwysiwygtextarea.php')) {
                require_once(XOOPS_ROOT_PATH . '/class/wysiwyg/formwysiwygtextarea.php');
                $editor = new XoopsFormWysiwygTextArea($caption, $name, $value, $width, $height, '');
            }
            break;
        }

        return $editor;
}

/**
 * linkterms: assign module header
 *
 * tooltips (c) dhtmlgoodies
 */
function lx_module_header() {

    global $xoopsTpl, $xoTheme, $xoopsModule, $xoopsModuleConfig, $lexikon_module_header;
function myFunction($a, $b) {
    // Do something
}
    if(isset($xoTheme) && is_object($xoTheme)) {
        $xoTheme -> addStylesheet( 'modules/lexikon/assets/css/style.css' );
        if ($xoopsModuleConfig['linkterms'] == 3) {
            $xoTheme -> addStylesheet( 'modules/lexikon/assets/css/linkterms.css' );
            $xoTheme->addScript( '/modules/lexikon/assets/js/tooltipscript2.js', array( 'type' => 'text/javascript' ) );
            }
        if ($xoopsModuleConfig['linkterms'] == 4) {
            $xoTheme->addScript( '/modules/lexikon/assets/js/popup.js', array( 'type' => 'text/javascript' ) );
            }
        if ($xoopsModuleConfig['linkterms'] == 5) {
            $xoTheme -> addStylesheet( 'modules/lexikon/assets/css/linkterms.css' );
            $xoTheme->addScript( '/modules/lexikon/assets/js/balloontooltip.js', array( 'type' => 'text/javascript' ) );
            }
        if ($xoopsModuleConfig['linkterms'] == 6) {
            $xoTheme -> addStylesheet( 'modules/lexikon/assets/css/linkterms.css' );
            $xoTheme->addScript( '/modules/lexikon/assets/js/shadowtooltip.js', array( 'type' => 'text/javascript' ) );
            }
    } else {
        $lexikon_url = XOOPS_URL.'/modules/'.$xoopsModule->getVar('dirname');
        if ($xoopsModuleConfig['linkterms'] == 3) {
            $lexikon_module_header = '<link rel="stylesheet" type="text/css" href="style.css" />
			<link rel="stylesheet" type="text/css" href="assets/css/linkterms.css" />
			<script src="'.$lexikon_url.'/assets/js/tooltipscript2.js" type="text/javascript"></script>';
            }
        if ($xoopsModuleConfig['linkterms'] == 4) {
            $lexikon_module_header = '<link rel="stylesheet" type="text/css" href="style.css" />
			<link rel="stylesheet" type="text/css" href="assets/css/linkterms.css" />
			<script src="'.$lexikon_url.'/assets/js/popup.js" type="text/javascript"></script>';
            }
        if ($xoopsModuleConfig['linkterms'] == 5) {
            $lexikon_module_header = '<link rel="stylesheet" type="text/css" href="style.css" />
			<link rel="stylesheet" type="text/css" href="assets/css/linkterms.css" />
			<script src="'.$lexikon_url.'/assets/js/balloontooltip.js" type="text/javascript"></script>';
            }
        if ($xoopsModuleConfig['linkterms'] == 6) {
            $lexikon_module_header = '<link rel="stylesheet" type="text/css" href="style.css" />
			<link rel="stylesheet" type="text/css" href="assets/css/linkterms.css" />
			<script src="'.$lexikon_url.'/assets/js/shadowtooltip.js" type="text/javascript"></script>';
            }
        }
}

/**
 * Validate userid
 */
function lx_val_user_data($uids) {

    global $xoopsDB,$xoopsUser, $xoopsUserIsAdmin;
function myFunction($a, $b) {
    // Do something
}

    if ($uids<=0) {
        return false;
    }
    if ($uids > 0) {
        $member_handler = xoops_gethandler('member');
        $user = $member_handler->getUser($uids);
        if (!is_object($user)) {
            return false;
        }
    }
    $result = $xoopsDB->query("SELECT * FROM ".$xoopsDB->prefix("users")." WHERE uid='$uids'");
    if ($xoopsDB->getRowsNum($result)<=0) {
        return false;
    }
    $row = $xoopsDB->fetchArray($result);

    return $row;
}

// Get all terms published by an author
function lx_AuthorProfile($uid ) {
// Bad
class Router
{
    public function generate($path)
    {
        return $_SERVER['HOST'].$path;
    }
}

// Better
class Router
{
    private $host;

    public function __construct($host)
    {
        $this->host = $host;
    }

    public function generate($path)
    {
        return $this->host.$path;
    }
}

class Controller
{
    public function myAction(Request $request)
    {
        // Instead of
        $page = isset($_GET['page']) ? intval($_GET['page']) : 1;

        // Better (assuming you use the Symfony2 request)
        $page = $request->query->get('page', 1);
    }
}
    include_once XOOPS_ROOT_PATH . '/class/pagenav.php';
    global $authortermstotal, $xoopsTpl, $xoopsDB, $xoopsUser, $xoopsModuleConfig;
function myFunction($a, $b) {
    // Do something
}
    $myts = MyTextSanitizer::getInstance();
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}
    //permissions
    $gperm_handler = xoops_gethandler('groupperm');
    $groups = is_object($xoopsUser) ? $xoopsUser->getGroups() : XOOPS_GROUP_ANONYMOUS;
    $module_handler = xoops_gethandler('module');
    $module = $module_handler->getByDirname('lexikon');
    $module_id = $module->getVar('mid');
    $allowed_cats = $gperm_handler->getItemIds("lexikon_view", $groups, $module_id);
    $catids = implode(',', $allowed_cats);
    $catperms = " AND categoryID IN ($catids) ";
        
    $start = isset( $_GET['start'] ) ? intval( $_GET['start'] ) : 0;
    $limit = $xoopsModuleConfig['indexperpage'];

    $sql = $xoopsDB -> query( "SELECT *
                              FROM " . $xoopsDB -> prefix( "lxentries" ) . "
                              WHERE uid='".intval($uid)."' AND  offline = '0' AND submit = '0' AND request = '0' ".$catperms."
                              ORDER BY term
                              LIMIT $start,$limit");

    while ($row=$xoopsDB->fetchArray($sql)) {

        $xoopsTpl->append('entries', array('id'=>$row['entryID'],'name'=>$row['term'],'date'=>date($xoopsModuleConfig['dateformat'], $row['datesub']),'counter'=>$row['counter']));
    }

    $navstring = '';
    $navstring .= "uid=".$uid."&start";
    $pagenav = new XoopsPageNav( $authortermstotal , $xoopsModuleConfig['indexperpage'], $start, $navstring);
    $authortermsarr['navbar'] = '<span style="text-align:right;">' . $pagenav -> renderNav(6) . '</span>';
foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

    if ($item->hasBar()) {
        $myArray['bar'] = $item->getBar();
    }

    // do something with $myArray
}
    $xoopsTpl -> assign( 'authortermsarr', $authortermsarr);
}

// Returns the author's IDs for authorslist
function lx_getAuthors($limit=0, $start=0) {

    global $xoopsDB ;
function myFunction($a, $b) {
    // Do something
}

    $ret = [];
    $sql = 'SELECT distinct(uid) as uid FROM '.$xoopsDB->prefix('lxentries').' WHERE offline = 0 ';
    $sql .= " ORDER BY uid";
    $result = $xoopsDB->query($sql);
    while ( $myrow = $xoopsDB->fetchArray($result) ) {
        $ret[] = $myrow['uid'];
    }

    return $ret;
}

// link to userprofile
function lx_getLinkedProfileFromId($userid) {
// Bad
class Router
{
    public function generate($path)
    {
        return $_SERVER['HOST'].$path;
    }
}

// Better
class Router
{
    private $host;

    public function __construct($host)
    {
        $this->host = $host;
    }

    public function generate($path)
    {
        return $this->host.$path;
    }
}

class Controller
{
    public function myAction(Request $request)
    {
        // Instead of
        $page = isset($_GET['page']) ? intval($_GET['page']) : 1;

        // Better (assuming you use the Symfony2 request)
        $page = $request->query->get('page', 1);
    }
}
    global $uid, $xoopsModule;
function myFunction($a, $b) {
    // Do something
}
    $userid = intval($uid);
    if ($userid > 0) {
        $member_handler = xoops_gethandler('member');
        $user = $member_handler->getUser($userid);
        if (is_object($user)) {
            $linkeduser = '<a title="'._MD_LEXIKON_AUTHORPROFILETEXT.'" href="'.XOOPS_URL.'/modules/'.$xoopsModule->dirname().'/profile.php?uid='.$uid.'">'. $user->getVar('uname').'</a>';
            //$linkeduser = XoopsUserUtility::getUnameFromId ( $uid );

            //$linkeduser .= '<div style=\'position:relative; right: 4px; top: 2px;\'><a title="'._MD_LEXIKON_AUTHORPROFILETEXT.'" href="'.XOOPS_URL.'/modules/'.$xoopsModule->dirname().'/profile.php?uid='.$uid.'">'._MD_LEXIKON_AUTHORPROFILETEXT.'</a></div>';

            return $linkeduser;
        }
    }

    return $GLOBALS['xoopsConfig']['anonymous'];
}

// functionset to assign terms with accentuated or umlaut initials to the adequate initial
function lx_removeAccents($string) {

    $chars['in'] = chr(128).chr(131).chr(138).chr(142).chr(154).chr(158)
foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

    if ($item->hasBar()) {
        $myArray['bar'] = $item->getBar();
    }

    // do something with $myArray
}
      .chr(159).chr(162).chr(165).chr(181).chr(192).chr(193).chr(194)
      .chr(195).chr(196).chr(197).chr(199).chr(200).chr(201).chr(202)
      .chr(203).chr(204).chr(205).chr(206).chr(207).chr(209).chr(210)
      .chr(211).chr(212).chr(213).chr(214).chr(216).chr(217).chr(218)
      .chr(219).chr(220).chr(221).chr(224).chr(225).chr(226).chr(227)
      .chr(228).chr(229).chr(231).chr(232).chr(233).chr(234).chr(235)
      .chr(236).chr(237).chr(238).chr(239).chr(241).chr(242).chr(243)
      .chr(244).chr(245).chr(246).chr(248).chr(249).chr(250).chr(251)
      .chr(252).chr(253).chr(255);
    $chars['out'] = "EfSZszYcYuAAAAAACEEEEIIIINOOOOOOUUUUYaaaaaaceeeeiiiinoooooouuuuyy";
    if (lx_seemsUtf8($string)) {
        $invalid_latin_chars = [
                                chr(197).chr(146) => 'OE',
                                chr(197).chr(147) => 'oe',
                                chr(197).chr(160) => 'S',
                                chr(197).chr(189) => 'Z',
                                chr(197).chr(161) => 's',
                                chr(197).chr(190) => 'z',
                                chr(226).chr(130).chr(172) => 'E'
                                ];
        $string = utf8_decode(strtr($string, $invalid_latin_chars));
    }
    $string = strtr($string, $chars['in'], $chars['out']);
    $double_chars['in'] = [
foreach ($collection as $item) {
    $myArray['foo'] = $item->getFoo();

    if ($item->hasBar()) {
        $myArray['bar'] = $item->getBar();
    }

    // do something with $myArray
}
                          chr(140),
                          chr(156),
                          chr(198),
                          chr(208),
                          chr(222),
                          chr(223),
                          chr(230),
                          chr(240),
                          chr(254)
                          ];
    $double_chars['out'] = [
                            'OE',
                            'oe',
                            'AE',
                            'DH',
                            'TH',
                            'ss',
                            'ae',
                            'dh',
                            'th'
                            ];
    $string = str_replace($double_chars['in'], $double_chars['out'], $string);

    return $string;
}

function lx_seemsUtf8($Str) { # by bmorel at ssi dot fr
    for ($i=0; $i<strlen($Str); $i++) {
        if (ord($Str[$i]) < 0x80) continue; # 0bbbbbbb

        elseif ((ord($Str[$i]) & 0xE0) == 0xC0) $n=1; # 110bbbbb
        elseif ((ord($Str[$i]) & 0xF0) == 0xE0) $n=2; # 1110bbbb
        elseif ((ord($Str[$i]) & 0xF8) == 0xF0) $n=3; # 11110bbb
        elseif ((ord($Str[$i]) & 0xFC) == 0xF8) $n=4; # 111110bb
        elseif ((ord($Str[$i]) & 0xFE) == 0xFC) $n=5; # 1111110b
        else return false; # Does not match any model
        for ($j=0; $j<$n; $j++) { # n bytes matching 10bbbbbb follow ?

            if ((++$i == strlen($Str)) || ((ord($Str[$i]) & 0xC0) != 0x80))
            return false;
        }
    }

    return true;
}

function lx_sanitizeFieldName($field) {

    $field = lx_removeAccents($field);
    $field = strtolower($field);
    $field = preg_replace('/&.+?;/', '', $field); // kill entities
    $field = preg_replace('/[^a-z0-9 _-]/', '', $field);
    $field = preg_replace('/\s+/', ' ', $field);
    $field = str_replace(' ', '-', $field);
    $field = preg_replace('|-+|', '-', $field);
    $field = trim($field, '-');

    return $field;
}

// Verify that a term does not exist for submissions and requests (both user frontend and admin backend)
function lx_TermExists($term, $table) {

    global $xoopsDB;
function myFunction($a, $b) {
    // Do something
}
    $sql = sprintf('SELECT COUNT(*) FROM %s WHERE term = %s', $table, $xoopsDB->quoteString(addslashes($term)));
    $result = $xoopsDB->query($sql);
    list($count) = $xoopsDB->fetchRow($result);

    return($count);
}

// Static method to get author data block authors - from AMS
function lexikon_block_getAuthors($limit = 5, $sort = "count", $name = 'uname', $compute_method = "average") {
    $limit = intval($limit);
    if ($name != "uname")
        $name = "name"; //making sure that there is not invalid information in field value
    $ret = [];
    $db = XoopsDatabaseFactory::getDatabaseConnection();
    if ($sort == "count") {
        $sql = "SELECT u.".$name." AS name, u.uid , count( n.entryID ) AS count
              FROM ".$db->prefix("users")." u, ".$db->prefix("lxentries")." n
              WHERE u.uid = n.uid
              AND n.datesub > 0 AND n.datesub <= ".time()." AND n.offline = 0 AND n.submit = 0
              GROUP BY u.uid ORDER BY count DESC";
    } elseif ($sort == "read") {

        if ($compute_method == "average") {
            $compute = "sum( n.counter ) / count( n.entryID )";
        } else {
            $compute = "sum( n.counter )";
        }
        $sql = "SELECT u.".$name." AS name, u.uid , $compute AS count
              FROM ".$db->prefix("users")." u, ".$db->prefix("lxentries")." n
              WHERE u.uid = n.uid
              AND n.datesub > 0 AND n.datesub <= ".time()." AND n.offline = 0 AND n.submit = 0
              GROUP BY u.uid ORDER BY count DESC";
    }
    if (!$result = $db->query($sql, $limit))
function myFunction($a) {
    switch ($a) {
        case 'foo':
            $x = 1;
            break;

        case 'bar':
            $x = 2;
            break;
    }

    // $x is potentially undefined here.
    echo $x;
}
        return false;
        
    while ($row = $db->fetchArray($result)) {

        if ($name == "name" && $row['name'] == '')
            $row['name'] = XoopsUser::getUnameFromId($row['uid']);
        $row['count'] = round($row['count'], 0);
        $ret[] = $row;
    }

    return $ret;
}

/**
 * close all unclosed xhtml tags *Test*
 *
 * @param string $html
 * @return string
 * @author Milian Wolff <mail -at- milianw.de>
 */
function lx_closetags2($html){

  // put all opened tags into an array
  preg_match_all("#<([a-z]+)( .*)?(?!/)>#iU",$html,$result);
  $openedtags=$result[1];

  // put all closed tags into an array
  preg_match_all("#</([a-z]+)>#iU",$html,$result);
  $closedtags=$result[1];
  $len_opened = count($openedtags);
  // all tags are closed
  if(count($closedtags) == $len_opened){
    return $html;
  }

  $openedtags = array_reverse($openedtags);
  // close tags
  for($i=0;$i < $len_opened;$i++) {
    if (!in_array($openedtags[$i],$closedtags)){
      $html .= '</'.$openedtags[$i].'>';
    } else {
      unset($closedtags[array_search($openedtags[$i],$closedtags)]);
    }
  }

  return $html;
}

/**
 * @author   Monte Ohrt <monte at ohrt dot com>, modified by Amos Robinson
 *           <amos dot robinson at gmail dot com>
 */
function lx_close_tags($string) {

    // match opened tags
    if(preg_match_all('/<([a-z\:\-]+)[^\/]>/', $string, $start_tags)) {
        $start_tags = $start_tags[1];
        // match closed tags
        if(preg_match_all('/<\/([a-z]+)>/', $string, $end_tags)) {
            $complete_tags = array();
            $end_tags = $end_tags[1];
    
            foreach($start_tags as $key => $val) {
                $posb = array_search($val, $end_tags);
                   if(is_integer($posb)) {
                      unset($end_tags[$posb]);
                   } else {
                    $complete_tags[] = $val;
                }
            }
        } else {
            $complete_tags = $start_tags;
        }
        
        $complete_tags = array_reverse($complete_tags);
        for($i = 0; $i < count($complete_tags); $i++) {
for ($i=0; $i<count($array); $i++) { // calls count() on each iteration
}

// Better
for ($i=0, $c=count($array); $i<$c; $i++) { // calls count() just once
}
            $string .= '</' . $complete_tags[$i] . '>';
        }
    }

    return $string;
}

/**
 * Smarty plugin
 * @package Smarty
 * @subpackage plugins
 */
/**
 * Smarty truncate_tagsafe modifier plugin
 *
 * Type:     modifier<br>
 * Name:     truncate_tagsafe<br>
 * Purpose:  Truncate a string to a certain length if necessary,
 *           optionally splitting in the middle of a word, and
 *           appending the $etc string or inserting $etc into the middle.
 *           Makes sure no tags are left half-open or half-closed (e.g. "Banana in a <a...")
 * @author   Monte Ohrt <monte at ohrt dot com>, modified by Amos Robinson
 *           <amos dot robinson at gmail dot com>
 * used in Block entries_scrolling.php
 */
function lx_truncate_tagsafe($string, $length = 80, $etc = '...', $break_words = false) {

    if ($length == 0) return '';
    if (strlen($string) > $length) {
        $length -= strlen($etc);
        if (!$break_words) {
            $string = preg_replace('/\s+?(\S+)?$/', '', substr($string, 0, $length+1));
            $string = preg_replace('/<[^>]*$/', '', $string);
            $string = lx_close_tags($string);
        }

        return $string . $etc;
    } else {
        return $string;
    }
}

function lexikon_summary() {
global $xoopsDB;
function myFunction($a, $b) {
    // Do something
}

    $summary = [];

    $result01 = $xoopsDB -> query( "SELECT COUNT(*)
                                   FROM " . $xoopsDB -> prefix( "lxcategories" ) . " " );
    list( $totalcategories ) = $xoopsDB -> fetchRow( $result01 );

    $result02 = $xoopsDB -> query( "SELECT COUNT(*)
                                   FROM " . $xoopsDB -> prefix( "lxentries" ) . "
                                   WHERE submit = 0" );
    list( $totalpublished ) = $xoopsDB -> fetchRow( $result02 );

    $result03 = $xoopsDB -> query( "SELECT COUNT(*)
                                   FROM " . $xoopsDB -> prefix( "lxentries" ) . "
                                   WHERE submit = '1' AND request = '0' " );
    list( $totalsubmitted ) = $xoopsDB -> fetchRow( $result03 );

    $result04 = $xoopsDB -> query( "SELECT COUNT(*)
                                   FROM " . $xoopsDB -> prefix( "lxentries" ) . "
                                   WHERE submit = '1' AND request = '1' " );
    list( $totalrequested ) = $xoopsDB -> fetchRow( $result04 );

// Recuperer les valeurs dans la base de donnees

$summary['publishedEntries']   = $totalpublished ? $totalpublished : '0';
$summary['availableCategories']   = $totalcategories ? $totalcategories : '0';
$summary['submittedEntries'] = $totalsubmitted ? $totalsubmitted : '0';
$summary['requestedEntries'] = $totalrequested ? $totalrequested : '0';

    //print_r($summary);
    return $summary;

} // end function


1		<?php
		0 ignored issues – show Coding Style Compatibility introduced 2016-06-01 03:48 UTC by Report Bug Copy Issue Report 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 19 and the first side effect is on line 10. The PSR-1: Basic Coding Standard recommends that a file should either introduce new symbols, that is classes, functions, constants or similar, or have side effects. Side effects are anything that executes logic, like for example printing output, changing ini settings or writing to a file. The idea behind this recommendation is that merely auto-loading a class should not change the state of an application. It also promotes a cleaner style of programming and makes your code less prone to errors, because the logic is not spread out all over the place. To learn more about the PSR-1, please see the PHP-FIG site on the PSR-1. Loading history...
2		/**
3		* $Id: functions.php v 1.0 8 May 2004 hsalazar Exp $
4		* Module: Lexikon
5		* Author: hsalazar
6		* Additions and Modifications: Yerres
7		* Licence: GNU
8		*/
9
10		if( ! defined( 'XOOPS_ROOT_PATH' ) ) die( 'XOOPS root path not defined' ) ;
11
12		/**
13		* lx_getLinkedUnameFromId()
14		*
15		* @param integer $userid Userid of author etc
16		* @param integer $name: 0 Use Usenamer 1 Use realname
		0 ignored issues – show Bug introduced 2016-01-18 10:48 UTC by Report Bug Copy Issue Report There is no parameter named `$name:`. Was it maybe removed? This check looks for PHPDoc comments describing methods or function parameters that do not exist on the corresponding method or function. Consider the following example. The parameter `$italy` is not defined by the method `finale(...)`. /** * @param array $germany * @param array $island * @param array $italy */ function finale($germany, $island) { return "2:1"; } The most likely cause is that the parameter was removed, but the annotation was not. Loading history...
17		* @return
18		**/
19	View Code Duplication	function lx_getLinkedUnameFromId($userid = 0, $name= 0) {
		0 ignored issues – show Documentation introduced 2016-01-18 10:48 UTC by Report Bug Copy Issue Report The return type could not be reliably inferred; please add a `@return` annotation. Our type inference engine in quite powerful, but sometimes the code does not provide enough clues to go by. In these cases we request you to add a `@return` annotation as described here. Loading history... Duplication introduced 2017-08-07 08:39 UTC by Report Bug Copy Issue Report This function seems to be duplicated in your project. Duplicated code is one of the most pungent code smells. If you need to duplicate the same code in three or more different places, we strongly encourage you to look into extracting the code into a single class or operation. You can also find more detailed suggestions in the “Code” section of your repository. Loading history... Coding Style introduced 2016-01-18 10:48 UTC by Report Bug Copy Issue Report `lx_getLinkedUnameFromId` uses the super-global variable `$GLOBALS` which is generally not recommended. Instead of super-globals, we recommend to explicitly inject the dependencies of your class. This makes your code less dependent on global state and it becomes generally more testable: // Bad class Router { public function generate($path) { return $_SERVER['HOST'].$path; } } // Better class Router { private $host; public function __construct($host) { $this->host = $host; } public function generate($path) { return $this->host.$path; } } class Controller { public function myAction(Request $request) { // Instead of $page = isset($_GET['page']) ? intval($_GET['page']) : 1; // Better (assuming you use the Symfony2 request) $page = $request->query->get('page', 1); } } Loading history...
20		if (!is_numeric($userid)) {
21		return $userid;
22		}
23
24		$userid = intval($userid);
25		if ($userid > 0) {
26		$member_handler = xoops_gethandler('member');
27		$user = $member_handler->getUser($userid);
28
29		if (is_object($user)) {
30		$ts = MyTextSanitizer::getInstance();
31		$username = $user->getVar('uname');
32		$usernameu = $user->getVar('name');
33
34		if ( ($name) && !empty($usernameu)) {
35		$username = $user->getVar('name');
36		}
37		if ( !empty($usernameu)) {

XoopsModules25x / lexikon

Pull Request — master (#12)

functions.php ➔ lx_extract_keywords() D

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

1. Pass all data via parameters

2. Create a class that maintains your state

Available Fixes

1. Pass all data via parameters

2. Create a class that maintains your state

Duplication Side-by-Side

Filter issues like