XoopsModules\Extcal\Utility

Complexity

Total Complexity

55

Size/Duplication

Total Lines	471
Duplicated Lines	51.38 %

Coupling/Cohesion

Components	1
Dependencies	5

Importance

Changes

0

17 Methods

Rating	Name	Duplication	Size	Complexity
A	extcal_getEvent()	13	13	2
C	extcal_loadImg()	54	54	15
A	getListCategories()	18	18	3
B	getCheckeCategories()	27	27	3
A	getListOrderBy()	23	23	2
A	getListAndOr()	11	11	1
A	getList()	15	15	3
B	getDateBetweenDates()	0	38	1
A	echoDateArray()	0	6	2
A	ext_echoArray()	9	9	2
A	ext_echo()	0	7	2
A	ext_echoTSN()	0	6	1
A	ext_echoTSU()	8	8	2
A	ext_convert_date()	13	13	3
A	ext_DateAdd()	8	8	1
C	ext_DateAdd2()	43	43	11
A	getLighterColor()	0	8	1

<?php namespace XoopsModules\Extcal;

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 27 and the first side effect is on line 18.

/**
 * classGenerator
 * walls_watermarks.
 *
 * You may not change or alter any portion of this comment or credits
 * of supporting developers from this source code or any supporting source code
 * which is considered copyrighted (c) material of the original comment or credit authors.
 * This program is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
 *
 *
 *
 * L'utilisation de ce formulaire d'adminitration suppose
 * que la classe correspondante de la table a été générées avec classGenerator
 **/
require_once XOOPS_ROOT_PATH . '/class/uploader.php';

use Xmf\Request;
use XoopsModules\Extcal;
use XoopsModules\Extcal\Common;

/**
 * Class Utility
 */
class Utility
{
    use common\VersionChecks; //checkVerXoops, checkVerPhp Traits

    use common\ServerStats; // getServerStats Trait

    use common\FilesManagement; // Files Management Trait

    //--------------- Custom module methods -----------------------------
    /**
     * @param $eventId
     *
     * @return array
     */
    public static function extcal_getEvent($eventId)

This method seems to be duplicated in your project.

    {
        $eventHandler = Extcal\Helper::getInstance()->getHandler(_EXTCAL_CLN_EVENT);
        $event        = $eventHandler->getEvent($eventId);
        $t            = $event->getVars();
        $data         = [];
        //        while (list($key, $val) = each($t)) {

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

        foreach ($t as $key => $val) {
            $data[$key] = $val['value'];
        }

        return $data;
    }

    /**
     * @param $REQUEST
     * @param $event_picture1
     * @param $event_picture2
     */
    public static function extcal_loadImg(&$REQUEST, &$event_picture1, &$event_picture2)

This method seems to be duplicated in your project.

    {
        ///////////////////////////////////////////////////////////////////////////////
        $uploaddir_event = XOOPS_ROOT_PATH . '/uploads/extcal/';
        $uploadurl_event = XOOPS_URL . '/uploads/extcal/';

$uploadurl_event is not used, you could remove the assignment.

        //$picture = '';

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

        for ($j = 1; $j < 3; ++$j) {
            $delimg = @$REQUEST['delimg_' . $j . ''];
            $delimg = isset($delimg) ? (int)$delimg : 0;
            if (0 == $delimg && !empty($REQUEST['xoops_upload_file'][$j])) {
                $upload = new \XoopsMediaUploader($uploaddir_event, [
                    'image/gif',
                    'image/jpeg',
                    'image/pjpeg',
                    'image/x-png',
                    'image/png',
                    'image/jpg',
                ], 3145728, null, null);
                if ($upload->fetchMedia($REQUEST['xoops_upload_file'][$j])) {
                    $upload->setPrefix('event_');
                    $upload->fetchMedia($REQUEST['xoops_upload_file'][$j]);
                    if (!$upload->upload()) {
                        $errors = $upload->getErrors();
                        redirect_header('javascript:history.go(-1)', 3, $errors);
                    } else {
                        if (1 == $j) {
                            $event_picture1 = $upload->getSavedFileName();
                        } elseif (2 == $j) {
                            $event_picture2 = $upload->getSavedFileName();
                        }
                    }
                } elseif (!empty($REQUEST['file' . $j])) {
                    if (1 == $j) {
                        $event_picture1 = $REQUEST['file' . $j];
                    } elseif (2 == $j) {
                        $event_picture2 = $REQUEST['file' . $j];
                    }
                }
            } else {
                $url_event = XOOPS_ROOT_PATH . '/uploads/extcal/' . $REQUEST['file' . $j];
                if (1 == $j) {
                    $event_picture1 = '';
                } elseif (2 == $j) {
                    $event_picture2 = '';
                }
                if (is_file($url_event)) {
                    chmod($url_event, 0777);
                    unlink($url_event);
                }
            }
        }
        //exit;
        ///////////////////////////////////////////////////////////////////////////////
    }

    /*******************************************************************
     *
     ******************************************************************
     * @param        $cat
     * @param bool   $addNone
     * @param string $name
     * @return XoopsFormSelect

Should the return type not be \XoopsFormSelect?

     */
    public static function getListCategories($cat, $addNone = true, $name = 'cat')

This method seems to be duplicated in your project.

    {
        global $xoopsUser;

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

        // Category selectbox
        $catHandler   = Extcal\Helper::getInstance()->getHandler(_EXTCAL_CLN_CAT);

        $catsList  = $catHandler->getAllCat($xoopsUser);
        $catSelect = new \XoopsFormSelect('', $name, $cat);
        if ($addNone) {
            $catSelect->addOption(0, ' ');
        }

        foreach ($catsList as $catList) {
            $catSelect->addOption($catList->getVar('cat_id'), $catList->getVar('cat_name'));
        }

        return $catSelect;
    }

    /*******************************************************************
     *
     ******************************************************************
     * @param string $name
     * @param        $cat
     * @return array
     */
    public static function getCheckeCategories($name = 'cat', $cat)

The parameter $name is not used and could be removed.

This method seems to be duplicated in your project.

    {
        global $xoopsUser;

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

        // Category selectbox
        //<option style="background-color:#00FFFF;">VARCHAR</option>

         $catHandler   = Extcal\Helper::getInstance()->getHandler(_EXTCAL_CLN_CAT);
        $catsList   = $catHandler->getAllCat($xoopsUser);

        $t = [];
        foreach ($catsList as $catList) {
            $cat_id    = $catList->getVar('cat_id');
            $name      = $catList->getVar('cat_name');
            $cat_color = $catList->getVar('cat_color');
            $checked   = in_array($cat_id, $cat) ? 'checked' : '';
            $cat       = ''
                         . "<div style='float:left; margin-left:5px;'>"
                         . "<input type='checkbox' name='{$name}[{$cat_id}]' value='1' {$checked}>"
                         . "<div style='absolute:left;height:12px; width:6px; background-color:#{$cat_color}; border:1px solid black; float:left; margin-right:5px;' ></div>"
                         . " {$name}"
                         . '</div>';

            $t[] = $cat;
        }

        return $t;
    }

    /*******************************************************************
     *
     ******************************************************************
     * @param string $name
     * @param string $caption
     * @param        $defaut
     * @param bool   $addNone
     * @return XoopsFormSelect

Should the return type not be \XoopsFormSelect?

     */
    public static function getListOrderBy($name = 'orderby', $caption = '', $defaut, $addNone = false)

This method seems to be duplicated in your project.

    {
        global $xoopsUser;

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

        $select = new \XoopsFormSelect($caption, $name, $defaut);
        if ($addNone) {
            $select->addOption('', '');
        }

        $select->addOption('year ASC', _MD_EXTCAL_YEAR . ' ' . _MD_EXTCAL_ORDER_BY_ASC);
        $select->addOption('year DESC', _MD_EXTCAL_YEAR . ' ' . _MD_EXTCAL_ORDER_BY_DESC);

        $select->addOption('month ASC', _MD_EXTCAL_MONTH . ' ' . _MD_EXTCAL_ORDER_BY_ASC);
        $select->addOption('month DESC', _MD_EXTCAL_MONTH . ' ' . _MD_EXTCAL_ORDER_BY_DESC);

        $select->addOption('event_title ASC', _MD_EXTCAL_ALPHA . ' ' . _MD_EXTCAL_ORDER_BY_ASC);
        $select->addOption('event_title DESC', _MD_EXTCAL_ALPHA . ' ' . _MD_EXTCAL_ORDER_BY_DESC);

        $select->addOption('cat_name ASC', _MD_EXTCAL_CATEGORY . ' ' . _MD_EXTCAL_ORDER_BY_ASC);
        $select->addOption('cat_name DESC', _MD_EXTCAL_CATEGORY . ' ' . _MD_EXTCAL_ORDER_BY_DESC);

        return $select;
    }

    /*******************************************************************
     *
     ******************************************************************
     * @param string $name
     * @param string $caption
     * @param        $defaut
     * @return XoopsFormSelect

Should the return type not be \XoopsFormSelect?

     */
    public static function getListAndOr($name = 'andor', $caption = '', $defaut)

This method seems to be duplicated in your project.

    {
        global $xoopsUser;

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

        $select = new \XoopsFormSelect($caption, $name, $defaut);

        $select->addOption('AND', _MD_EXTCAL_AND);
        $select->addOption('OR', _MD_EXTCAL_OR);

        return $select;
    }

    /*******************************************************************
     *
     ******************************************************************
     * @param        $name
     * @param        $caption
     * @param        $defaut
     * @param        $options
     * @param string $sep
     * @return XoopsFormSelect

Should the return type not be \XoopsFormSelect?

     */
    public static function getList($name, $caption, $defaut, $options, $sep = ';')

This method seems to be duplicated in your project.

    {
        global $xoopsUser;

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

        $select = new \XoopsFormSelect($caption, $name, $defaut);
        if (!is_array($options)) {
            $options = explode($sep, $options);
        }

        for ($h = 0, $count = count($options); $h < $count; ++$h) {
            $select->addOption($h, $options[$h]);
        }

        return $select;
    }

    /*******************************************************************
     *
     ******************************************************************
     * @param        $ts
     * @param        $startMonth
     * @param        $endMonth
     * @param string $mode
     * @return DateTime
     */
    public static function getDateBetweenDates($ts, $startMonth, $endMonth, $mode = 'w')

The parameter $mode is not used and could be removed.

    {
        $d = new DateTime($periodStart);

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

        $d->setTimestamp($ts);

        //echo "<br>affichage des periodes : <br>";
        $begin = new DateTime();
        $begin->setTimestamp($startMonth);
        //echo $begin->format("d/m/Y à H\hi:s").'<br>'; // 03/10/2007 à 19h39:53

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

        $end = new DateTime();
        $end->setTimestamp($endMonth);
        //echo $end->format("d/m/Y à H\hi:s").'<br>'; // 03/10/2007 à 19h39:53

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

        //echo "<hr>";
        $interval = DateInterval::createFromDateString('next sunday');
        $period   = new DatePeriod($begin, $interval, $end);

$period is not used, you could remove the assignment.

        //echoDateArray($period);

        //echo "<hr>{$interval}";
        return $d;

        //echo mktime($heure, $minute, $seconde, $mois, $jour, $an);

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

        //
        //   $jour = date('d', $ts);

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

        //   $mois = date('m', $ts);

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

        //   $an = date('Y', $ts);

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

        //   $heure = date('H', $ts);

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

        //   $minute = date('i', $ts);

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

        //   $seconde = date('s', $ts);

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

        //   $d->setDate($heure,$minute,$seconde,$mois,$jour,$an);

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

        // <?php
        // $interval = DateInterval::createFromDateString('next sunday');

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

        // $period = new DatePeriod($begin, $interval, $end);

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

        // foreach ($period as $dt) {

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

        //   echo $dt->format( "l Y-m-d H:i:s\n" );

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

    }

    /*

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

    Sunday 2009-11-01 00:00:00
    Sunday 2009-11-08 00:00:00
    Sunday 2009-11-15 00:00:00
    Sunday 2009-11-22 00:00:00
    Sunday 2009-11-29 00:00:00
    Sunday 2009-12-06 00:00:00
    ...
    */
    /**
     * @param $period
     */
    public static function echoDateArray($period)
    {
        foreach ($period as $dt) {
            echo $dt->format("l Y-m-d H:i:s\n") . '<br>';
        }
    }

    /*****************************************************************/
    /**
     * @param        $t
     * @param string $msg
     */
    public static function ext_echoArray($t, $msg = '')

This method seems to be duplicated in your project.

    {
        if ('' != $msg) {
            echo "<hr>{$msg}<hr>";
        }

        $txt = print_r($t, true);
        echo '<pre>Number of items: ' . count($t) . "<br>{$txt}</pre>";
    }

    /*****************************************************************/
    /**
     * @param        $line
     * @param string $msg
     */
    public static function ext_echo($line, $msg = '')
    {
        if ('' != $msg) {
            echo "<hr>{$msg}<hr>";
        }
        echo $line . '<br>';
    }

    /*****************************************************************/
    /**
     * @param        $tsName
     * @param string $msg
     */
    public static function ext_echoTSN($tsName, $msg = '')
    {
        global $$tsName;

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

        $ts = $$tsName;
        static::ext_echoTSU($ts, $tsName, $msg = '');
    }

    /*****************************************************************/
    /**
     * @param        $ts
     * @param        $tsName
     * @param string $msg
     */
    public static function ext_echoTSU($ts, $tsName, $msg = '')

This method seems to be duplicated in your project.

    {
        if ('' != $msg) {
            echo "<hr>{$msg}<hr>";
        }

        echo 'date --->' . $tsName . ' = ' . $ts . ' - ' . date('d-m-Y H:m:s', $ts) . '<br>';
    }

    /*****************************************************************/
    /*****************************************************************/
    /**
     * @param        $date
     * @param string $sep
     *
     * @return int

Should the return type not be integer|null?

     */
    public static function ext_convert_date($date, $sep = '-')

This method seems to be duplicated in your project.

    {
        $lstSep = '/ .';

        for ($h = 0, $count = strlen($lstSep); $h < $count; ++$h) {

++$h; does not seem to be reachable.

            $sep2replace = substr($lstSep, $h, 1);
            if (strpos($date, $sep2replace)) {
                $date = str_replace($sep2replace, $sep, $date);
            }

            return strtotime($date);
        }
    }

    /**
     * @param     $givendate
     * @param int $day
     * @param int $mth
     * @param int $yr
     *
     * @return int
     */
    public static function ext_DateAdd($givendate, $day = 0, $mth = 0, $yr = 0)

This method seems to be duplicated in your project.

    {
        //$cd = strtotime($givendate);

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

        $cd      = $givendate;
        $newdate = date('Y-m-d h:i:s', mktime(date('h', $cd), date('i', $cd), date('s', $cd), date('m', $cd) + $mth, date('d', $cd) + $day, date('Y', $cd) + $yr));

        return strtotime($newdate);
    }

    /**
     * @param $date
     * @param $number
     * @param $interval
     *
     * @return int
     */
    public static function ext_DateAdd2($date, $number, $interval = 'd')

This method seems to be duplicated in your project.

    {
        $date_time_array = getdate($date);
        $hours           = $date_time_array['hours'];
        $minutes         = $date_time_array['minutes'];
        $seconds         = $date_time_array['seconds'];
        $month           = $date_time_array['mon'];
        $day             = $date_time_array['mday'];
        $year            = $date_time_array['year'];

        switch ($interval) {

            case 'yyyy':
                $year += $number;
                break;
            case 'q':
                $year += ($number * 3);
                break;
            case 'm':
                $month += $number;
                break;
            case 'y':
            case 'd':
            case 'w':
                $day += $number;
                break;
            case 'ww':
                $day += ($number * 7);
                break;
            case 'h':
                $hours += $number;
                break;
            case 'n':
                $minutes += $number;
                break;
            case 's':
                $seconds += $number;
                break;
        }
        $timestamp = mktime($hours, $minutes, $seconds, $month, $day, $year);

        return $timestamp;
    }

    // function date_diff($date1, $date2) {

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

    //     $current = $date1;

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

    //     $datetime2 = date_create($date2);

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

    //     $count = 0;

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

    //     while (date_create($current) < $datetime2) {

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

    //         $current = gmdate("Y-m-d", strtotime("+1 day", strtotime($current)));

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

    //         ++$count;
    //     }
    //     return $count;
    // }

    /**************************************************************************/
    /**
     * @param $color
     * @param $plancher
     * @param $plafond
     *
     * @return string
     */
    public static function getLighterColor($color, $plancher, $plafond)
    {
        require_once __DIR__ . '/ColorTools.php';

        //$ct = new ColorTools();

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

        //return $ct->eclaircir($color,$plancher,$plafond);

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

        return ColorTools::eclaircir($color, $plancher, $plafond);
    }
    /**************************************************************************/

}