Inspection of "updates" - mambax7/extcal - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( f62223...75b16c )

by Michael

created 2018-09-22 02:29 UTC

Status

Spacing +11 added lines, -11 removed lines patch added patch discarded remove patch

@@ -19,7 +19,7 @@  discard block
 block discarded – undo
 
 use XoopsModules\Extcal;
 
-require_once  dirname(__DIR__) . '/include/constantes.php';
+require_once  dirname(__DIR__).'/include/constantes.php';
 
 /**
  * @param $options
@@ -88,7 +88,7 @@  discard block
 block discarded – undo
         'nbEvent'      => $nbEvent,
         'nbDays'       => $nbDays,
     ];
-    $events   = $eventHandler->getEventsOnPeriode($criteres);
+    $events = $eventHandler->getEventsOnPeriode($criteres);
 
     //----------------------------
 
@@ -115,27 +115,27 @@  discard block
 block discarded – undo
 
     $cats = $catHandler->getAllCat($xoopsUser, 'extcal_cat_view');
 
-    $form = _MB_EXTCAL_DISPLAY . "&nbsp;\n";
-    $form .= '<input name="options[0]" size="5" maxlength="255" value="' . $options[0] . '" type="text">&nbsp;' . _MB_EXTCAL_EVENT . '<br>';
-    $form .= _MB_EXTCAL_TITLE_LENGTH . ' : <input name="options[1]" size="5" maxlength="255" value="' . $options[1] . '" type="text"><br>';
+    $form = _MB_EXTCAL_DISPLAY."&nbsp;\n";
+    $form .= '<input name="options[0]" size="5" maxlength="255" value="'.$options[0].'" type="text">&nbsp;'._MB_EXTCAL_EVENT.'<br>';
+    $form .= _MB_EXTCAL_TITLE_LENGTH.' : <input name="options[1]" size="5" maxlength="255" value="'.$options[1].'" type="text"><br>';
 
-    $form .= _MB_EXTCAL_UPCOMING_DAYS . ' : <input name="options[2]" size="5" maxlength="255" value="' . $options[2] . '" type="text"><br>';
+    $form .= _MB_EXTCAL_UPCOMING_DAYS.' : <input name="options[2]" size="5" maxlength="255" value="'.$options[2].'" type="text"><br>';
 
     array_shift($options);
     array_shift($options);
     array_shift($options);
 
-    $form .= _MB_EXTCAL_CAT_TO_USE . '<br><select name="options[]" multiple="multiple" size="5">';
+    $form .= _MB_EXTCAL_CAT_TO_USE.'<br><select name="options[]" multiple="multiple" size="5">';
     if (false === array_search(0, $options)) {
-        $form .= '<option value="0">' . _MB_EXTCAL_ALL_CAT . '</option>';
+        $form .= '<option value="0">'._MB_EXTCAL_ALL_CAT.'</option>';
     } else {
-        $form .= '<option value="0" selected="selected">' . _MB_EXTCAL_ALL_CAT . '</option>';
+        $form .= '<option value="0" selected="selected">'._MB_EXTCAL_ALL_CAT.'</option>';
     }
     foreach ($cats as $cat) {
         if (false === array_search($cat->getVar('cat_id'), $options)) {
-            $form .= '<option value="' . $cat->getVar('cat_id') . '">' . $cat->getVar('cat_name') . '</option>';
+            $form .= '<option value="'.$cat->getVar('cat_id').'">'.$cat->getVar('cat_name').'</option>';
         } else {
-            $form .= '<option value="' . $cat->getVar('cat_id') . '" selected="selected">' . $cat->getVar('cat_name') . '</option>';
+            $form .= '<option value="'.$cat->getVar('cat_id').'" selected="selected">'.$cat->getVar('cat_name').'</option>';
         }
     }
     $form .= '</select>';

Please login to merge, or discard this patch.

blocks/new_events.php 1 patch

Spacing +9 added lines, -9 removed lines patch added patch discarded remove patch

@@ -19,7 +19,7 @@  discard block
 block discarded – undo
 
 use XoopsModules\Extcal;
 
-require_once  dirname(__DIR__) . '/include/constantes.php';
+require_once  dirname(__DIR__).'/include/constantes.php';
 
 /**
  * @param $options
@@ -66,22 +66,22 @@  discard block
 block discarded – undo
 
     $cats = $catHandler->getAllCat($xoopsUser, 'extcal_cat_view');
 
-    $form = _MB_EXTCAL_DISPLAY . "&nbsp;\n";
-    $form .= '<input name="options[0]" size="5" maxlength="255" value="' . $options[0] . '" type="text">&nbsp;' . _MB_EXTCAL_EVENT . '<br>';
-    $form .= _MB_EXTCAL_TITLE_LENGTH . ' : <input name="options[1]" size="5" maxlength="255" value="' . $options[1] . '" type="text"><br>';
+    $form = _MB_EXTCAL_DISPLAY."&nbsp;\n";
+    $form .= '<input name="options[0]" size="5" maxlength="255" value="'.$options[0].'" type="text">&nbsp;'._MB_EXTCAL_EVENT.'<br>';
+    $form .= _MB_EXTCAL_TITLE_LENGTH.' : <input name="options[1]" size="5" maxlength="255" value="'.$options[1].'" type="text"><br>';
     array_shift($options);
     array_shift($options);
-    $form .= _MB_EXTCAL_CAT_TO_USE . '<br><select name="options[]" multiple="multiple" size="5">';
+    $form .= _MB_EXTCAL_CAT_TO_USE.'<br><select name="options[]" multiple="multiple" size="5">';
     if (false === array_search(0, $options)) {
-        $form .= '<option value="0">' . _MB_EXTCAL_ALL_CAT . '</option>';
+        $form .= '<option value="0">'._MB_EXTCAL_ALL_CAT.'</option>';
     } else {
-        $form .= '<option value="0" selected="selected">' . _MB_EXTCAL_ALL_CAT . '</option>';
+        $form .= '<option value="0" selected="selected">'._MB_EXTCAL_ALL_CAT.'</option>';
     }
     foreach ($cats as $cat) {
         if (false === array_search($cat->getVar('cat_id'), $options)) {
-            $form .= '<option value="' . $cat->getVar('cat_id') . '">' . $cat->getVar('cat_name') . '</option>';
+            $form .= '<option value="'.$cat->getVar('cat_id').'">'.$cat->getVar('cat_name').'</option>';
         } else {
-            $form .= '<option value="' . $cat->getVar('cat_id') . '" selected="selected">' . $cat->getVar('cat_name') . '</option>';
+            $form .= '<option value="'.$cat->getVar('cat_id').'" selected="selected">'.$cat->getVar('cat_name').'</option>';
         }
     }
     $form .= '</select>';

Please login to merge, or discard this patch.

blocks/spotlight_events.php 1 patch

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

@@ -1,6 +1,6 @@
 block discarded – undo
 <?php
 
-require_once  dirname(__DIR__) . '/include/constantes.php';
+require_once  dirname(__DIR__).'/include/constantes.php';
 
 //---------------------------------------------------------------------
 

Please login to merge, or discard this patch.

blocks/category_events.php 1 patch

Spacing +9 added lines, -9 removed lines patch added patch discarded remove patch

@@ -19,7 +19,7 @@  discard block
 block discarded – undo
 
 use XoopsModules\Extcal;
 
-require_once  dirname(__DIR__) . '/include/constantes.php';
+require_once  dirname(__DIR__).'/include/constantes.php';
 
 /******************************************************/
 /* Ajour JJD - Evenements par categries               */
@@ -92,22 +92,22 @@  discard block
 block discarded – undo
 
     $cats = $catHandler->getAllCat($xoopsUser, 'extcal_cat_view');
 
-    $form = _MB_EXTCAL_DISPLAY . "&nbsp;\n";
-    $form .= '<input name="options[0]" size="5" maxlength="255" value="' . $options[0] . '" type="text">&nbsp;' . _MB_EXTCAL_EVENT . '<br>';
-    $form .= _MB_EXTCAL_TITLE_LENGTH . ' : <input name="options[1]" size="5" maxlength="255" value="' . $options[1] . '" type="text"><br>';
+    $form = _MB_EXTCAL_DISPLAY."&nbsp;\n";
+    $form .= '<input name="options[0]" size="5" maxlength="255" value="'.$options[0].'" type="text">&nbsp;'._MB_EXTCAL_EVENT.'<br>';
+    $form .= _MB_EXTCAL_TITLE_LENGTH.' : <input name="options[1]" size="5" maxlength="255" value="'.$options[1].'" type="text"><br>';
     array_shift($options);
     array_shift($options);
-    $form .= _MB_EXTCAL_CAT_TO_USE . '<br><select name="options[]" multiple="multiple" size="5">';
+    $form .= _MB_EXTCAL_CAT_TO_USE.'<br><select name="options[]" multiple="multiple" size="5">';
     if (false === array_search(0, $options)) {
-        $form .= '<option value="0">' . _MB_EXTCAL_ALL_CAT . '</option>';
+        $form .= '<option value="0">'._MB_EXTCAL_ALL_CAT.'</option>';
     } else {
-        $form .= '<option value="0" selected="selected">' . _MB_EXTCAL_ALL_CAT . '</option>';
+        $form .= '<option value="0" selected="selected">'._MB_EXTCAL_ALL_CAT.'</option>';
     }
     foreach ($cats as $cat) {
         if (false === array_search($cat->getVar('cat_id'), $options)) {
-            $form .= '<option value="' . $cat->getVar('cat_id') . '">' . $cat->getVar('cat_name') . '</option>';
+            $form .= '<option value="'.$cat->getVar('cat_id').'">'.$cat->getVar('cat_name').'</option>';
         } else {
-            $form .= '<option value="' . $cat->getVar('cat_id') . '" selected="selected">' . $cat->getVar('cat_name') . '</option>';
+            $form .= '<option value="'.$cat->getVar('cat_id').'" selected="selected">'.$cat->getVar('cat_name').'</option>';
         }
     }
     $form .= '</select>';

Please login to merge, or discard this patch.

blocks/random_events.php 1 patch

Spacing +9 added lines, -9 removed lines patch added patch discarded remove patch

@@ -19,7 +19,7 @@  discard block
 block discarded – undo
 
 use XoopsModules\Extcal;
 
-require_once  dirname(__DIR__) . '/include/constantes.php';
+require_once  dirname(__DIR__).'/include/constantes.php';
 
 /**
  * @param $options
@@ -65,22 +65,22 @@  discard block
 block discarded – undo
 
     $cats = $catHandler->getAllCat($xoopsUser, 'extcal_cat_view');
 
-    $form = _MB_EXTCAL_DISPLAY . "&nbsp;\n";
-    $form .= '<input name="options[0]" size="5" maxlength="255" value="' . $options[0] . '" type="text">&nbsp;' . _MB_EXTCAL_EVENT . '<br>';
-    $form .= _MB_EXTCAL_TITLE_LENGTH . ' : <input name="options[1]" size="5" maxlength="255" value="' . $options[1] . '" type="text"><br>';
+    $form = _MB_EXTCAL_DISPLAY."&nbsp;\n";
+    $form .= '<input name="options[0]" size="5" maxlength="255" value="'.$options[0].'" type="text">&nbsp;'._MB_EXTCAL_EVENT.'<br>';
+    $form .= _MB_EXTCAL_TITLE_LENGTH.' : <input name="options[1]" size="5" maxlength="255" value="'.$options[1].'" type="text"><br>';
     array_shift($options);
     array_shift($options);
-    $form .= _MB_EXTCAL_CAT_TO_USE . '<br><select name="options[]" multiple="multiple" size="5">';
+    $form .= _MB_EXTCAL_CAT_TO_USE.'<br><select name="options[]" multiple="multiple" size="5">';
     if (false === array_search(0, $options)) {
-        $form .= '<option value="0">' . _MB_EXTCAL_ALL_CAT . '</option>';
+        $form .= '<option value="0">'._MB_EXTCAL_ALL_CAT.'</option>';
     } else {
-        $form .= '<option value="0" selected="selected">' . _MB_EXTCAL_ALL_CAT . '</option>';
+        $form .= '<option value="0" selected="selected">'._MB_EXTCAL_ALL_CAT.'</option>';
     }
     foreach ($cats as $cat) {
         if (false === array_search($cat->getVar('cat_id'), $options)) {
-            $form .= '<option value="' . $cat->getVar('cat_id') . '">' . $cat->getVar('cat_name') . '</option>';
+            $form .= '<option value="'.$cat->getVar('cat_id').'">'.$cat->getVar('cat_name').'</option>';
         } else {
-            $form .= '<option value="' . $cat->getVar('cat_id') . '" selected="selected">' . $cat->getVar('cat_name') . '</option>';
+            $form .= '<option value="'.$cat->getVar('cat_id').'" selected="selected">'.$cat->getVar('cat_name').'</option>';
         }
     }
     $form .= '</select>';

Please login to merge, or discard this patch.

blocks/day_events.php 1 patch

Spacing +9 added lines, -9 removed lines patch added patch discarded remove patch

@@ -19,7 +19,7 @@  discard block
 block discarded – undo
 
 use XoopsModules\Extcal;
 
-require_once  dirname(__DIR__) . '/include/constantes.php';
+require_once  dirname(__DIR__).'/include/constantes.php';
 
 /**
  * @param $options
@@ -69,23 +69,23 @@  discard block
 block discarded – undo
 
     $cats = $catHandler->getAllCat($xoopsUser, 'extcal_cat_view');
 
-    $form = _MB_EXTCAL_DISPLAY . "&nbsp;\n";
-    $form .= '<input name="options[0]" size="5" maxlength="255" value="' . $options[0] . '" type="text">&nbsp;' . _MB_EXTCAL_EVENT . '<br>';
-    $form .= _MB_EXTCAL_TITLE_LENGTH . ' : <input name="options[1]" size="5" maxlength="255" value="' . $options[1] . '" type="text"><br>';
+    $form = _MB_EXTCAL_DISPLAY."&nbsp;\n";
+    $form .= '<input name="options[0]" size="5" maxlength="255" value="'.$options[0].'" type="text">&nbsp;'._MB_EXTCAL_EVENT.'<br>';
+    $form .= _MB_EXTCAL_TITLE_LENGTH.' : <input name="options[1]" size="5" maxlength="255" value="'.$options[1].'" type="text"><br>';
     array_shift($options);
     array_shift($options);
-    $form .= _MB_EXTCAL_CAT_TO_USE . '<br><select name="options[]" multiple="multiple" size="5">';
+    $form .= _MB_EXTCAL_CAT_TO_USE.'<br><select name="options[]" multiple="multiple" size="5">';
     if (false === array_search(0, $options)) {
-        $form .= '<option value="0">' . _MB_EXTCAL_ALL_CAT . '</option>';
+        $form .= '<option value="0">'._MB_EXTCAL_ALL_CAT.'</option>';
     } else {
-        $form .= '<option value="0" selected="selected">' . _MB_EXTCAL_ALL_CAT . '</option>';
+        $form .= '<option value="0" selected="selected">'._MB_EXTCAL_ALL_CAT.'</option>';
     }
     /** @var Extcal\Category $cat */
     foreach ($cats as $cat) {
         if (false === array_search($cat->getVar('cat_id'), $options)) {
-            $form .= '<option value="' . $cat->getVar('cat_id') . '">' . $cat->getVar('cat_name') . '</option>';
+            $form .= '<option value="'.$cat->getVar('cat_id').'">'.$cat->getVar('cat_name').'</option>';
         } else {
-            $form .= '<option value="' . $cat->getVar('cat_id') . '" selected="selected">' . $cat->getVar('cat_name') . '</option>';
+            $form .= '<option value="'.$cat->getVar('cat_id').'" selected="selected">'.$cat->getVar('cat_name').'</option>';
         }
     }
     $form .= '</select>';

Please login to merge, or discard this patch.

view_new-event.php 1 patch

Spacing +5 added lines, -5 removed lines patch added patch discarded remove patch

@@ -1,10 +1,10 @@  discard block
 block discarded – undo
 <?php
 
-require_once dirname(dirname(__DIR__)) . '/mainfile.php';
-require_once __DIR__ . '/include/constantes.php';
+require_once dirname(dirname(__DIR__)).'/mainfile.php';
+require_once __DIR__.'/include/constantes.php';
 $params                                  = ['view' => _EXTCAL_NAV_NEW_EVENT, 'file' => _EXTCAL_FILE_NEW_EVENT];
 $GLOBALS['xoopsOption']['template_main'] = "extcal_view_{$params['view']}.tpl";
-require_once __DIR__ . '/header.php';
+require_once __DIR__.'/header.php';
 
 /* ========================================================================== */
 //Extcal\Utility::echoArray($_GET);
@@ -16,7 +16,7 @@  discard block
 block discarded – undo
 
 //exit;
 if (count($permHandler->getAuthorizedCat($xoopsUser, 'extcal_cat_submit')) > 0) {
-    require_once XOOPS_ROOT_PATH . '/header.php';
+    require_once XOOPS_ROOT_PATH.'/header.php';
 
     // Title of the page
     $xoopsTpl->assign('xoops_pagetitle', _MI_EXTCAL_SUBMIT_EVENT);
@@ -58,7 +58,7 @@  discard block
 block discarded – undo
     $xoopsTpl->assign('lang', $lang);
     $xoopsTpl->assign('view', 'newevent');
 
-    require_once XOOPS_ROOT_PATH . '/footer.php';
+    require_once XOOPS_ROOT_PATH.'/footer.php';
 } else {
     redirect_header('index.php', 3);
 }

Please login to merge, or discard this patch.

include/onupdate.php 1 patch

Spacing +12 added lines, -12 removed lines patch added patch discarded remove patch

@@ -22,7 +22,7 @@  discard block
 block discarded – undo
 
 if ((!defined('XOOPS_ROOT_PATH')) || !($GLOBALS['xoopsUser'] instanceof \XoopsUser)
     || !$GLOBALS['xoopsUser']->IsAdmin()) {
-    exit('Restricted access' . PHP_EOL);
+    exit('Restricted access'.PHP_EOL);
 }
 
 /**
@@ -75,7 +75,7 @@  discard block
 block discarded – undo
         return true;
     }
 
-    $fld = XOOPS_ROOT_PATH . '/modules/' . $module->getVar('dirname') . '/versions/';
+    $fld = XOOPS_ROOT_PATH.'/modules/'.$module->getVar('dirname').'/versions/';
     $cls = 'Extcal_%1$s';
 
     $version = [
@@ -94,7 +94,7 @@  discard block
 block discarded – undo
     foreach ($version as $key => $val) {
         if ($previousVersion < $val) {
             $name = sprintf($cls, $key);
-            $f    = $fld . $name . '.php';
+            $f    = $fld.$name.'.php';
             //ext_echo ("<hr>{$f}<hr>");
             if (is_readable($f)) {
                 echo "mise à jour version : {$key} = {$val}<br>";
@@ -118,14 +118,14 @@  discard block
 block discarded – undo
         //delete old HTML templates
         if (count($configurator['templateFolders']) > 0) {
             foreach ($configurator['templateFolders'] as $folder) {
-                $templateFolder = $GLOBALS['xoops']->path('modules/' . $moduleDirName . $folder);
+                $templateFolder = $GLOBALS['xoops']->path('modules/'.$moduleDirName.$folder);
                 if (is_dir($templateFolder)) {
                     $templateList = array_diff(scandir($templateFolder, SCANDIR_SORT_NONE), ['..', '.']);
                     foreach ($templateList as $k => $v) {
-                        $fileInfo = new \SplFileInfo($templateFolder . $v);
+                        $fileInfo = new \SplFileInfo($templateFolder.$v);
                         if ('html' === $fileInfo->getExtension() && 'index.html' !== $fileInfo->getFilename()) {
-                            if (file_exists($templateFolder . $v)) {
-                                unlink($templateFolder . $v);
+                            if (file_exists($templateFolder.$v)) {
+                                unlink($templateFolder.$v);
                             }
                         }
                     }
@@ -135,9 +135,9 @@  discard block
 block discarded – undo
 
         //  ---  COPY blank.png FILES ---------------
         if (count($configurator['copyFiles']) > 0) {
-            $file =  dirname(__DIR__) . '/assets/images/blank.png';
+            $file = dirname(__DIR__).'/assets/images/blank.png';
             foreach (array_keys($configurator['copyFiles']) as $i) {
-                $dest = $configurator['copyFiles'][$i] . '/blank.png';
+                $dest = $configurator['copyFiles'][$i].'/blank.png';
                 $utility::copyFile($file, $dest);
             }
         }
@@ -146,7 +146,7 @@  discard block
 block discarded – undo
         if (count($configurator['oldFiles']) > 0) {
             //    foreach (array_keys($GLOBALS['uploadFolders']) as $i) {
             foreach (array_keys($configurator['oldFiles']) as $i) {
-                $tempFile = $GLOBALS['xoops']->path('modules/' . $moduleDirName . $configurator['oldFiles'][$i]);
+                $tempFile = $GLOBALS['xoops']->path('modules/'.$moduleDirName.$configurator['oldFiles'][$i]);
                 if (is_file($tempFile)) {
                     unlink($tempFile);
                 }
@@ -156,14 +156,14 @@  discard block
 block discarded – undo
         //---------------------
 
         //delete .html entries from the tpl table
-        $sql = 'DELETE FROM ' . $xoopsDB->prefix('tplfile') . " WHERE `tpl_module` = '" . $module->getVar('dirname', 'n') . "' AND `tpl_file` LIKE '%.html%'";
+        $sql = 'DELETE FROM '.$xoopsDB->prefix('tplfile')." WHERE `tpl_module` = '".$module->getVar('dirname', 'n')."' AND `tpl_file` LIKE '%.html%'";
         $xoopsDB->queryF($sql);
 
         // Load class XoopsFile ====================
         xoops_load('XoopsFile');
 
         //delete /images directory ============
-        $imagesDirectory = $GLOBALS['xoops']->path('modules/' . $module->getVar('dirname', 'n') . '/images/');
+        $imagesDirectory = $GLOBALS['xoops']->path('modules/'.$module->getVar('dirname', 'n').'/images/');
         $folderHandler   = \XoopsFile::getHandler('folder', $imagesDirectory);
         $folderHandler->delete($imagesDirectory);
     }

Please login to merge, or discard this patch.

include/agenda_fnc.php 1 patch

Spacing +7 added lines, -7 removed lines patch added patch discarded remove patch

@@ -21,7 +21,7 @@  discard block
 block discarded – undo
 define('_EXTCAL_FORMAT_AGENDA_KEYD', 'Y-m-d');
 define('_EXTCAL_FORMAT_AGENDA_KEYT', 'H:i');
 
-require_once __DIR__ . '/constantes.php';
+require_once __DIR__.'/constantes.php';
 // require_once  dirname(__DIR__) . '/class/Utility.php';
 
 $moduleDirName = basename(dirname(__DIR__));
@@ -75,7 +75,7 @@  discard block
 block discarded – undo
         $tj[$kj]['dayWeek'] = date('w', $tsj);
         $tj[$kj]['jour']    = $jName[$tj[$kj]['dayWeek']]; //date('l', $tsj);
         if (0 == $tj[$kj]['dayWeek']) {
-            $tj[$kj]['bg'] = "background='" . XOOPS_URL . "/modules/extcal/assets/images/trame.png'";
+            $tj[$kj]['bg'] = "background='".XOOPS_URL."/modules/extcal/assets/images/trame.png'";
         } else {
             $tj[$kj]['bg'] = '';
         }
@@ -155,16 +155,16 @@  discard block
 block discarded – undo
         $kd     = date(_EXTCAL_FORMAT_AGENDA_KEYD, $ts);
         $hour   = date('H', $ts);
         $minute = date('i', $ts);
-        $m      = (int)($minute / $mPlage) * $mPlage;
+        $m      = (int) ($minute / $mPlage) * $mPlage;
         //      echo "--->{$minute} / {$mPlage} = {$m}<br>";
-        $sMinute = (($m < 10) ? '0' . $m : $m);
+        $sMinute = (($m < 10) ? '0'.$m : $m);
         //$kt = date(_EXTCAL_FORMAT_AGENDA_KEYT, $ts);
         if ($hour < $hStart) {
             $kt = $tk0;
         } elseif ($hour >= ($hEnd + 1)) {
             $kt = $tk1;
         } else {
-            $kt = $hour . ':' . $sMinute;
+            $kt = $hour.':'.$sMinute;
         }
 
         $tAgenda[$kt]['jours'][$kd]['events'][] = $e;
@@ -313,7 +313,7 @@  discard block
 block discarded – undo
             return false;
     }
 
-    $file   = XOOPS_ROOT_PATH . '/language/' . $xoopsConfig['language'] . '/' . $f;
+    $file   = XOOPS_ROOT_PATH.'/language/'.$xoopsConfig['language'].'/'.$f;
     $prefix = (defined($name) ? '_MI' : '_MD');
     require_once $file;
 }
@@ -454,7 +454,7 @@  discard block
 block discarded – undo
     //    while (list($k, $v) = each($tNavBar)) {
     foreach ($tNavBar as $k => $v) {
         if (isset($tWeight[$k])) {
-            $ordre[] = (int)$tWeight[$k]; //ordre defini dans les option du module
+            $ordre[] = (int) $tWeight[$k]; //ordre defini dans les option du module
         } else {
             $ordre[] = $v['weight']; // ordre par defaut ddefini dans le tableau $tNavBar
         }

Please login to merge, or discard this patch.

		@@ -22,7 +22,7 @@ discard block
		block discarded – undo
22	22
23	23	if ((!defined('XOOPS_ROOT_PATH')) \|\| !($GLOBALS['xoopsUser'] instanceof \XoopsUser)
24	24	\|\| !$GLOBALS['xoopsUser']->IsAdmin()) {
25		- exit('Restricted access' . PHP_EOL);
	25	+ exit('Restricted access'.PHP_EOL);
26	26	}
27	27
28	28	/**
		@@ -75,7 +75,7 @@ discard block
		block discarded – undo
75	75	return true;
76	76	}
77	77
78		- $fld = XOOPS_ROOT_PATH . '/modules/' . $module->getVar('dirname') . '/versions/';
	78	+ $fld = XOOPS_ROOT_PATH.'/modules/'.$module->getVar('dirname').'/versions/';
79	79	$cls = 'Extcal_%1$s';
80	80
81	81	$version = [
		@@ -94,7 +94,7 @@ discard block
		block discarded – undo
94	94	foreach ($version as $key => $val) {
95	95	if ($previousVersion < $val) {
96	96	$name = sprintf($cls, $key);
97		- $f = $fld . $name . '.php';
	97	+ $f = $fld.$name.'.php';
98	98	//ext_echo ("<hr>{$f}<hr>");
99	99	if (is_readable($f)) {
100	100	echo "mise à jour version : {$key} = {$val}<br>";
		@@ -118,14 +118,14 @@ discard block
		block discarded – undo
118	118	//delete old HTML templates
119	119	if (count($configurator['templateFolders']) > 0) {
120	120	foreach ($configurator['templateFolders'] as $folder) {
121		- $templateFolder = $GLOBALS['xoops']->path('modules/' . $moduleDirName . $folder);
	121	+ $templateFolder = $GLOBALS['xoops']->path('modules/'.$moduleDirName.$folder);
122	122	if (is_dir($templateFolder)) {
123	123	$templateList = array_diff(scandir($templateFolder, SCANDIR_SORT_NONE), ['..', '.']);
124	124	foreach ($templateList as $k => $v) {
125		- $fileInfo = new \SplFileInfo($templateFolder . $v);
	125	+ $fileInfo = new \SplFileInfo($templateFolder.$v);
126	126	if ('html' === $fileInfo->getExtension() && 'index.html' !== $fileInfo->getFilename()) {
127		- if (file_exists($templateFolder . $v)) {
128		- unlink($templateFolder . $v);
	127	+ if (file_exists($templateFolder.$v)) {
	128	+ unlink($templateFolder.$v);
129	129	}
130	130	}
131	131	}
		@@ -135,9 +135,9 @@ discard block
		block discarded – undo
135	135
136	136	// --- COPY blank.png FILES ---------------
137	137	if (count($configurator['copyFiles']) > 0) {
138		- $file = dirname(__DIR__) . '/assets/images/blank.png';
	138	+ $file = dirname(__DIR__).'/assets/images/blank.png';
139	139	foreach (array_keys($configurator['copyFiles']) as $i) {
140		- $dest = $configurator['copyFiles'][$i] . '/blank.png';
	140	+ $dest = $configurator['copyFiles'][$i].'/blank.png';
141	141	$utility::copyFile($file, $dest);
142	142	}
143	143	}
		@@ -146,7 +146,7 @@ discard block
		block discarded – undo
146	146	if (count($configurator['oldFiles']) > 0) {
147	147	// foreach (array_keys($GLOBALS['uploadFolders']) as $i) {
148	148	foreach (array_keys($configurator['oldFiles']) as $i) {
149		- $tempFile = $GLOBALS['xoops']->path('modules/' . $moduleDirName . $configurator['oldFiles'][$i]);
	149	+ $tempFile = $GLOBALS['xoops']->path('modules/'.$moduleDirName.$configurator['oldFiles'][$i]);
150	150	if (is_file($tempFile)) {
151	151	unlink($tempFile);
152	152	}
		@@ -156,14 +156,14 @@ discard block
		block discarded – undo
156	156	//---------------------
157	157
158	158	//delete .html entries from the tpl table
159		- $sql = 'DELETE FROM ' . $xoopsDB->prefix('tplfile') . " WHERE `tpl_module` = '" . $module->getVar('dirname', 'n') . "' AND `tpl_file` LIKE '%.html%'";
	159	+ $sql = 'DELETE FROM '.$xoopsDB->prefix('tplfile')." WHERE `tpl_module` = '".$module->getVar('dirname', 'n')."' AND `tpl_file` LIKE '%.html%'";
160	160	$xoopsDB->queryF($sql);
161	161
162	162	// Load class XoopsFile ====================
163	163	xoops_load('XoopsFile');
164	164
165	165	//delete /images directory ============
166		- $imagesDirectory = $GLOBALS['xoops']->path('modules/' . $module->getVar('dirname', 'n') . '/images/');
	166	+ $imagesDirectory = $GLOBALS['xoops']->path('modules/'.$module->getVar('dirname', 'n').'/images/');
167	167	$folderHandler = \XoopsFile::getHandler('folder', $imagesDirectory);
168	168	$folderHandler->delete($imagesDirectory);
169	169	}

		@@ -19,7 +19,7 @@ discard block
		block discarded – undo
19	19
20	20	use XoopsModules\Extcal;
21	21
22		-require_once dirname(__DIR__) . '/include/constantes.php';
	22	+require_once dirname(__DIR__).'/include/constantes.php';
23	23
24	24	/**
25	25	* @param $options
		@@ -88,7 +88,7 @@ discard block
		block discarded – undo
88	88	'nbEvent' => $nbEvent,
89	89	'nbDays' => $nbDays,
90	90	];
91		- $events = $eventHandler->getEventsOnPeriode($criteres);
	91	+ $events = $eventHandler->getEventsOnPeriode($criteres);
92	92
93	93	//----------------------------
94	94
		@@ -115,27 +115,27 @@ discard block
		block discarded – undo
115	115
116	116	$cats = $catHandler->getAllCat($xoopsUser, 'extcal_cat_view');
117	117
118		- $form = _MB_EXTCAL_DISPLAY . " \n";
119		- $form .= '<input name="options[0]" size="5" maxlength="255" value="' . $options[0] . '" type="text"> ' . _MB_EXTCAL_EVENT . '<br>';
120		- $form .= _MB_EXTCAL_TITLE_LENGTH . ' : <input name="options[1]" size="5" maxlength="255" value="' . $options[1] . '" type="text"><br>';
	118	+ $form = _MB_EXTCAL_DISPLAY." \n";
	119	+ $form .= '<input name="options[0]" size="5" maxlength="255" value="'.$options[0].'" type="text"> '._MB_EXTCAL_EVENT.'<br>';
	120	+ $form .= _MB_EXTCAL_TITLE_LENGTH.' : <input name="options[1]" size="5" maxlength="255" value="'.$options[1].'" type="text"><br>';
121	121
122		- $form .= _MB_EXTCAL_UPCOMING_DAYS . ' : <input name="options[2]" size="5" maxlength="255" value="' . $options[2] . '" type="text"><br>';
	122	+ $form .= _MB_EXTCAL_UPCOMING_DAYS.' : <input name="options[2]" size="5" maxlength="255" value="'.$options[2].'" type="text"><br>';
123	123
124	124	array_shift($options);
125	125	array_shift($options);
126	126	array_shift($options);
127	127
128		- $form .= _MB_EXTCAL_CAT_TO_USE . '<br><select name="options[]" multiple="multiple" size="5">';
	128	+ $form .= _MB_EXTCAL_CAT_TO_USE.'<br><select name="options[]" multiple="multiple" size="5">';
129	129	if (false === array_search(0, $options)) {
130		- $form .= '<option value="0">' . _MB_EXTCAL_ALL_CAT . '</option>';
	130	+ $form .= '<option value="0">'._MB_EXTCAL_ALL_CAT.'</option>';
131	131	} else {
132		- $form .= '<option value="0" selected="selected">' . _MB_EXTCAL_ALL_CAT . '</option>';
	132	+ $form .= '<option value="0" selected="selected">'._MB_EXTCAL_ALL_CAT.'</option>';
133	133	}
134	134	foreach ($cats as $cat) {
135	135	if (false === array_search($cat->getVar('cat_id'), $options)) {
136		- $form .= '<option value="' . $cat->getVar('cat_id') . '">' . $cat->getVar('cat_name') . '</option>';
	136	+ $form .= '<option value="'.$cat->getVar('cat_id').'">'.$cat->getVar('cat_name').'</option>';
137	137	} else {
138		- $form .= '<option value="' . $cat->getVar('cat_id') . '" selected="selected">' . $cat->getVar('cat_name') . '</option>';
	138	+ $form .= '<option value="'.$cat->getVar('cat_id').'" selected="selected">'.$cat->getVar('cat_name').'</option>';
139	139	}
140	140	}
141	141	$form .= '</select>';

		@@ -1,6 +1,6 @@
		block discarded – undo
1	1	<?php
2	2
3		-require_once dirname(__DIR__) . '/include/constantes.php';
	3	+require_once dirname(__DIR__).'/include/constantes.php';
4	4
5	5	//---------------------------------------------------------------------
6	6

		@@ -1,10 +1,10 @@ discard block
		block discarded – undo
1	1	<?php
2	2
3		-require_once dirname(dirname(__DIR__)) . '/mainfile.php';
4		-require_once __DIR__ . '/include/constantes.php';
	3	+require_once dirname(dirname(__DIR__)).'/mainfile.php';
	4	+require_once __DIR__.'/include/constantes.php';
5	5	$params = ['view' => _EXTCAL_NAV_NEW_EVENT, 'file' => _EXTCAL_FILE_NEW_EVENT];
6	6	$GLOBALS['xoopsOption']['template_main'] = "extcal_view_{$params['view']}.tpl";
7		-require_once __DIR__ . '/header.php';
	7	+require_once __DIR__.'/header.php';
8	8
9	9	/* ========================================================================== */
10	10	//Extcal\Utility::echoArray($_GET);
		@@ -16,7 +16,7 @@ discard block
		block discarded – undo
16	16
17	17	//exit;
18	18	if (count($permHandler->getAuthorizedCat($xoopsUser, 'extcal_cat_submit')) > 0) {
19		- require_once XOOPS_ROOT_PATH . '/header.php';
	19	+ require_once XOOPS_ROOT_PATH.'/header.php';
20	20
21	21	// Title of the page
22	22	$xoopsTpl->assign('xoops_pagetitle', _MI_EXTCAL_SUBMIT_EVENT);
		@@ -58,7 +58,7 @@ discard block
		block discarded – undo
58	58	$xoopsTpl->assign('lang', $lang);
59	59	$xoopsTpl->assign('view', 'newevent');
60	60
61		- require_once XOOPS_ROOT_PATH . '/footer.php';
	61	+ require_once XOOPS_ROOT_PATH.'/footer.php';
62	62	} else {
63	63	redirect_header('index.php', 3);
64	64	}

		@@ -21,7 +21,7 @@ discard block
		block discarded – undo
21	21	define('_EXTCAL_FORMAT_AGENDA_KEYD', 'Y-m-d');
22	22	define('_EXTCAL_FORMAT_AGENDA_KEYT', 'H:i');
23	23
24		-require_once __DIR__ . '/constantes.php';
	24	+require_once __DIR__.'/constantes.php';
25	25	// require_once dirname(__DIR__) . '/class/Utility.php';
26	26
27	27	$moduleDirName = basename(dirname(__DIR__));
		@@ -75,7 +75,7 @@ discard block
		block discarded – undo
75	75	$tj[$kj]['dayWeek'] = date('w', $tsj);
76	76	$tj[$kj]['jour'] = $jName[$tj[$kj]['dayWeek']]; //date('l', $tsj);
77	77	if (0 == $tj[$kj]['dayWeek']) {
78		- $tj[$kj]['bg'] = "background='" . XOOPS_URL . "/modules/extcal/assets/images/trame.png'";
	78	+ $tj[$kj]['bg'] = "background='".XOOPS_URL."/modules/extcal/assets/images/trame.png'";
79	79	} else {
80	80	$tj[$kj]['bg'] = '';
81	81	}
		@@ -155,16 +155,16 @@ discard block
		block discarded – undo
155	155	$kd = date(_EXTCAL_FORMAT_AGENDA_KEYD, $ts);
156	156	$hour = date('H', $ts);
157	157	$minute = date('i', $ts);
158		- $m = (int)($minute / $mPlage) * $mPlage;
	158	+ $m = (int) ($minute / $mPlage) * $mPlage;
159	159	// echo "--->{$minute} / {$mPlage} = {$m}<br>";
160		- $sMinute = (($m < 10) ? '0' . $m : $m);
	160	+ $sMinute = (($m < 10) ? '0'.$m : $m);
161	161	//$kt = date(_EXTCAL_FORMAT_AGENDA_KEYT, $ts);
162	162	if ($hour < $hStart) {
163	163	$kt = $tk0;
164	164	} elseif ($hour >= ($hEnd + 1)) {
165	165	$kt = $tk1;
166	166	} else {
167		- $kt = $hour . ':' . $sMinute;
	167	+ $kt = $hour.':'.$sMinute;
168	168	}
169	169
170	170	$tAgenda[$kt]['jours'][$kd]['events'][] = $e;
		@@ -313,7 +313,7 @@ discard block
		block discarded – undo
313	313	return false;
314	314	}
315	315
316		- $file = XOOPS_ROOT_PATH . '/language/' . $xoopsConfig['language'] . '/' . $f;
	316	+ $file = XOOPS_ROOT_PATH.'/language/'.$xoopsConfig['language'].'/'.$f;
317	317	$prefix = (defined($name) ? '_MI' : '_MD');
318	318	require_once $file;
319	319	}
		@@ -454,7 +454,7 @@ discard block
		block discarded – undo
454	454	// while (list($k, $v) = each($tNavBar)) {
455	455	foreach ($tNavBar as $k => $v) {
456	456	if (isset($tWeight[$k])) {
457		- $ordre[] = (int)$tWeight[$k]; //ordre defini dans les option du module
	457	+ $ordre[] = (int) $tWeight[$k]; //ordre defini dans les option du module
458	458	} else {
459	459	$ordre[] = $v['weight']; // ordre par defaut ddefini dans le tableau $tNavBar
460	460	}

mambax7 / extcal

Push — master ( f62223...75b16c )

Status

Category

Spacing +11 added lines, -11 removed lines patch added patch discarded remove patch

Spacing +9 added lines, -9 removed lines patch added patch discarded remove patch

Spacing +1 added lines, -1 removed lines patch added patch discarded remove patch

Spacing +9 added lines, -9 removed lines patch added patch discarded remove patch

Spacing +9 added lines, -9 removed lines patch added patch discarded remove patch

Spacing +9 added lines, -9 removed lines patch added patch discarded remove patch

Spacing +5 added lines, -5 removed lines patch added patch discarded remove patch

Spacing +12 added lines, -12 removed lines patch added patch discarded remove patch

Spacing +7 added lines, -7 removed lines patch added patch discarded remove patch