XoopsModules25x/apcal

Method ImageMagickVersion has 27 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function ImageMagickVersion($returnRAW = false)
    {
        static $versionstring = null;
        if (is_null($versionstring)) {
            $versionstring = array(0 => false, 1 => false);

Method GetAllFilesInSubfolders has 27 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public static function GetAllFilesInSubfolders($dirname)
    {
        $AllFiles = array();
        $dirname  = rtrim(realpath($dirname), '/\\');
        if ($dirhandle = @opendir($dirname)) {

Similar blocks of code found in 2 locations. Consider refactoring.

Open

        foreach ($cal->long_event_legends as $bit => $legend) {
            $tmpl->addVar('LongEventLegends', 'BIT_MASK', 1 << ($bit - 1));
            $tmpl->addVar('LongEventLegends', 'LEGEND_ALT', _APCAL_MB_ALLDAY_EVENT . " $bit");
            $tmpl->addVar('LongEventLegends', 'LEGEND', $legend);
            $tmpl->addVar('LongEventLegends', 'SKINPATH', $cal->images_url);

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

            foreach ($this->long_event_legends as $bit => $legend) {
                $tmpl->addVar('LongEventLegends', 'BIT_MASK', 1 << ($bit - 1));
                $tmpl->addVar('LongEventLegends', 'LEGEND_ALT', _APCAL_MB_ALLDAY_EVENT . " $bit");
                $tmpl->addVar('LongEventLegends', 'LEGEND', $legend);
                $tmpl->addVar('LongEventLegends', 'SKINPATH', $this->images_url);

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

                function nextMonth() {
                    var date = new Date(cal.date);
                    date.setDate(date.getDate() + step);
                    cal.setDate(date);
                };

• Read up
  Read up

Similar blocks of code found in 2 locations. Consider refactoring.

Open

                function prevMonth() {
                    var date = new Date(cal.date);
                    date.setDate(date.getDate() - step);
                    cal.setDate(date);
                };

• Read up
  Read up

Function focus has 26 lines of code (exceeds 25 allowed). Consider refactoring.

Open

        focus: function () {
            this.wrapper.style.zIndex = hs.zIndexCounter += 2;
            // blur others
            for (var i = 0; i < hs.expanders.length; i++) {
                if (hs.expanders[i] && i == hs.focusKey) {

Method get_vtimezones_str has 26 lines of code (exceeds 25 allowed). Consider refactoring.

Open

        public function get_vtimezones_str($timezones)
        {
            if (empty($timezones)) {
                return "BEGIN:VTIMEZONE\r
TZID:GMT\r

Method apcal_thedays_schedule_edit has 26 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    function apcal_thedays_schedule_edit($options)
    {
        global $xoopsDB, $xoopsConfig;

        $moduleDirName = empty($options[0]) ? basename(dirname(__DIR__)) : $options[0];

Method load has 26 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function load($lpData, &$hdrLen)
    {
        $hdrLen = 0;

        $this->m_lpVer = substr($lpData, 0, 6);

Method apcal_todays_schedule_edit has 26 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    function apcal_todays_schedule_edit($options)
    {
        global $xoopsDB, $xoopsConfig;

        $moduleDirName = empty($options[0]) ? basename(dirname(__DIR__)) : $options[0];

Method Frame has 26 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    public function Frame(&$gdimg, $frame_width, $edge_width, $hexcolor_frame, $hexcolor1, $hexcolor2)
    {
        $frame_width    = ($frame_width ?: 5);
        $edge_width     = ($edge_width ?: 1);
        $hexcolor_frame = ($hexcolor_frame ?: 'CCCCCC');

Consider simplifying this complex logical expression.

Open

            if (!hs.container) {

                hs.ieLt7 = hs.ie && hs.uaVersion < 7;
                hs.ieLt9 = hs.ie && hs.uaVersion < 9;

Method get_todays_time_description has 8 arguments (exceeds 4 allowed). Consider refactoring.

Open

            $start,
            $end,
            $ynj,
            $justify = true,
            $admission = true,

Consider simplifying this complex logical expression.

Open

if ((!isset($_GET['action']) || $_GET['action'] === '') && isset($_GET['cid']) && !is_numeric($_GET['cid'])) {
    $cat_title = addslashes($_GET['cid']);
    $cat       = $GLOBALS['xoopsDB']->queryF("SELECT cid FROM {$GLOBALS['xoopsDB']->prefix('apcal_cat')} WHERE cat_shorttitle LIKE '$cat_title' LIMIT 0,1");

    if ($cat && $GLOBALS['xoopsDB']->getRowsNum($cat)) {

Consider simplifying this complex logical expression.

Open

                            if ($this->iar && ((int)$this->w > 0) && ((int)$this->h > 0)) {
                                list($nw, $nh) = phpthumb_functions::TranslateWHbyAngle($this->w, $this->h, $this->ra);
                                $nw          = ((round($nw) != 0) ? round($nw) : '');
                                $nh          = ((round($nh) != 0) ? round($nh) : '');
                                $commandline .= ' -' . $IMresizeParameter . ' ' . phpthumb_functions::escapeshellarg_replacement($nw . 'x' . $nh . '!');

Identical blocks of code found in 4 locations. Consider refactoring.

Open

        previousOrNext: function (el, op) {
            var exp = hs.getExpander(el);
            if (exp) return hs.transit(exp.getAdjacentAnchor(op), exp);
            else return false;
        },

• Read up
  Read up

Identical blocks of code found in 4 locations. Consider refactoring.

Open

        previousOrNext: function (el, op) {
            var exp = hs.getExpander(el);
            if (exp) return hs.transit(exp.getAdjacentAnchor(op), exp);
            else return false;
        },

• Read up
  Read up

Identical blocks of code found in 4 locations. Consider refactoring.

Open

        previousOrNext: function (el, op) {
            var exp = hs.getExpander(el);
            if (exp) return hs.transit(exp.getAdjacentAnchor(op), exp);
            else return false;
        },

• Read up
  Read up

Identical blocks of code found in 4 locations. Consider refactoring.

Open

        previousOrNext: function (el, op) {
            var exp = hs.getExpander(el);
            if (exp) return hs.transit(exp.getAdjacentAnchor(op), exp);
            else return false;
        },

• Read up
  Read up