b_marquee_tplleaguestats()

Complexity

Conditions	7
Paths	20

Size

Total Lines	61
Code Lines	36

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
function b_marquee_tplleaguestats($limit, $dateFormat, $itemsSize)
{
//    require_once XOOPS_ROOT_PATH . '/modules/marquee/class/Utility.php';

    //######################## SETTINGS ######################
    $displaySeason  = false; // display season name?
    $hour           = 1; // GMT+1  -> var = 1
    $useItemSize    = false; // use marquee $itemsize value?
    $overwriteLimit = true; // overwrite marquee's limit settings?
    $newLimit       = 6; // new limit (valid only if
    //     overwrite_limit_settings = true)
    $overwriteDateformat = true; // overwrite marquee's dateformat?
    $newDateformat       = 'd/m/Y'; // new dateformat (valid only if
    //     overwrite_dateformat_settings=true)
    //######################## SETTINGS ######################

    global $xoopsDB;

Use of global functionality is not recommended; it makes your code harder to test, and less reusable.

    if ($overwriteLimit) {

The condition $overwriteLimit is always true.

        $limit = $newLimit;
    }
    if ($overwriteDateformat) {

The condition $overwriteDateformat is always true.

        $dateFormat = $newDateformat;
    }

    $block  = [];
    $myts   = \MyTextSanitizer::getInstance();

The type MyTextSanitizer was not found. Maybe you did not declare it correctly or list all dependencies?

    $sql    = 'SELECT H.OpponentName as home, A.OpponentName as away, M.LeagueMatchHomeGoals as home_p, M.LeagueMatchAwayGoals as away_p,
                  M.LeagueMatchDate as date, S.SeasonName as season
           FROM ' . $xoopsDB->prefix('tplls_leaguematches') . ' M
           LEFT JOIN ' . $xoopsDB->prefix('tplls_opponents') . ' AS H ON M.LeagueMatchHomeID = H.OpponentID
           LEFT JOIN ' . $xoopsDB->prefix('tplls_opponents') . ' AS A ON M.LeagueMatchAwayID = A.OpponentID
           LEFT JOIN ' . $xoopsDB->prefix('tplls_seasonnames') . " AS S ON M.LeagueMatchSeasonID = S.SeasonID
           ORDER BY M.LeagueMatchDate DESC
           LIMIT 0,$limit";
    $result = $xoopsDB->query($sql);
    while (false !== ($myrow = $xoopsDB->fetchArray($result))) {
        $title = $myts->htmlSpecialChars($myrow['home']) . ' - ' . $myts->htmlSpecialChars($myrow['away']) . ' ' . $myts->htmlSpecialChars($myrow['home_p']) . '-' . $myts->htmlSpecialChars($myrow['away_p']);

        if ($useItemSize && $itemsSize > 0) {
            $title = xoops_substr($title, 0, $itemsSize + 3);

The function xoops_substr was not found. Maybe you did not declare it correctly or list all dependencies?

        }

        $arrDate = explode('-', $myrow['date']);

        $season = '';

        if ($displaySeason) {
            $season = $myrow['season'];
        }

        $block[] = [
            'date'     => formatTimestamp(mktime($hour, 0, 0, $arrDate[1], $arrDate[2], $arrDate[0]), $dateFormat),

The function formatTimestamp was not found. Maybe you did not declare it correctly or list all dependencies?

$arrDate[0] of type string is incompatible with the type integer expected by parameter $year of mktime().

$arrDate[1] of type string is incompatible with the type integer expected by parameter $month of mktime().

$arrDate[2] of type string is incompatible with the type integer expected by parameter $day of mktime().

            'category' => $season,
            'author'   => '',
            'title'    => $title,
            'link'     => '<a href="' . XOOPS_URL . '/modules/tplleaguestats">' . $title . '</a>'

The constant XOOPS_URL was not found. Maybe you did not declare it correctly or list all dependencies?

        ];
    }

    return $block;
}