mambax7 / extcal

class/Form/FormRecurRules.php

Spacing +18 added lines, -18 removed lines

@@ -67,45 +67,45 @@ 
         $formObject = new \XoopsFormRadio('', 'rrule_freq', $this->_rrule_freq);
         $formObject->addOption('none', _MD_EXTCAL_NO_RECCUR_EVENT);
         $ret .= $formObject->render();
-        $ret .= '<br><br><fieldset><legend>' . _MD_EXTCAL_RECCUR_POLICY . '</legend><fieldset><legend>';
+        $ret .= '<br><br><fieldset><legend>'._MD_EXTCAL_RECCUR_POLICY.'</legend><fieldset><legend>';
 
         $formObject = new \XoopsFormRadio('', 'rrule_freq', $this->_rrule_freq);
         $formObject->addOption('daily', _MD_EXTCAL_DAILY);
         $ret .= $formObject->render();
-        $ret .= '</legend>' . _MD_EXTCAL_DURING . ' ';
+        $ret .= '</legend>'._MD_EXTCAL_DURING.' ';
 
         $formObject = new \XoopsFormText('', 'rrule_daily_interval', 3, 2, $this->_rrule_daily_interval);
         $ret        .= $formObject->render();
-        $ret        .= ' ' . _MD_EXTCAL_DAYS . '</fieldset><br><fieldset><legend>';
+        $ret        .= ' '._MD_EXTCAL_DAYS.'</fieldset><br><fieldset><legend>';
 
         $formObject = new \XoopsFormRadio('', 'rrule_freq', $this->_rrule_freq);
         $formObject->addOption('weekly', _MD_EXTCAL_WEEKLY);
         $ret .= $formObject->render();
-        $ret .= '</legend>' . _MD_EXTCAL_DURING . ' ';
+        $ret .= '</legend>'._MD_EXTCAL_DURING.' ';
 
         $formObject = new \XoopsFormText('', 'rrule_weekly_interval', 3, 2, $this->_rrule_weekly_interval);
         $ret        .= $formObject->render();
-        $ret        .= ' ' . _MD_EXTCAL_WEEKS . '<br>';
+        $ret        .= ' '._MD_EXTCAL_WEEKS.'<br>';
 
         $formObject = new \XoopsFormCheckBox('', 'rrule_weekly_bydays', $this->_rrule_weekly_bydays);
-        $formObject->addOption('MO', _MD_EXTCAL_MO2 . '&nbsp;');
-        $formObject->addOption('TU', _MD_EXTCAL_TU2 . '&nbsp;');
-        $formObject->addOption('WE', _MD_EXTCAL_WE2 . '&nbsp;');
-        $formObject->addOption('TH', _MD_EXTCAL_TH2 . '&nbsp;');
-        $formObject->addOption('FR', _MD_EXTCAL_FR2 . '&nbsp;');
-        $formObject->addOption('SA', _MD_EXTCAL_SA2 . '&nbsp;');
-        $formObject->addOption('SU', _MD_EXTCAL_SU2 . '&nbsp;');
+        $formObject->addOption('MO', _MD_EXTCAL_MO2.'&nbsp;');
+        $formObject->addOption('TU', _MD_EXTCAL_TU2.'&nbsp;');
+        $formObject->addOption('WE', _MD_EXTCAL_WE2.'&nbsp;');
+        $formObject->addOption('TH', _MD_EXTCAL_TH2.'&nbsp;');
+        $formObject->addOption('FR', _MD_EXTCAL_FR2.'&nbsp;');
+        $formObject->addOption('SA', _MD_EXTCAL_SA2.'&nbsp;');
+        $formObject->addOption('SU', _MD_EXTCAL_SU2.'&nbsp;');
         $ret .= $formObject->render();
         $ret .= '</fieldset><br><fieldset><legend>';
 
         $formObject = new \XoopsFormRadio('', 'rrule_freq', $this->_rrule_freq);
         $formObject->addOption('monthly', _MD_EXTCAL_MONTHLY);
         $ret .= $formObject->render();
-        $ret .= '</legend>' . _MD_EXTCAL_DURING . ' ';
+        $ret .= '</legend>'._MD_EXTCAL_DURING.' ';
 
         $formObject = new \XoopsFormText('', 'rrule_monthly_interval', 3, 2, $this->_rrule_monthly_interval);
         $ret        .= $formObject->render();
-        $ret        .= ' ' . _MD_EXTCAL_MONTH . ', ' . _MD_EXTCAL_ON . ' ';
+        $ret        .= ' '._MD_EXTCAL_MONTH.', '._MD_EXTCAL_ON.' ';
 
         $formObject = new \XoopsFormSelect('', 'rrule_monthly_byday', $this->_rrule_monthly_byday);
         $formObject->addOption('', '&nbsp;');
@@ -145,20 +145,20 @@ 
         $formObject->addOption('-1SA', _MD_EXTCAL_LAST_SA);
         $formObject->addOption('-1SU', _MD_EXTCAL_LAST_SU);
         $ret .= $formObject->render();
-        $ret .= ' ' . _MD_EXTCAL_OR_THE . ' ';
+        $ret .= ' '._MD_EXTCAL_OR_THE.' ';
 
         $formObject = new \XoopsFormText('', 'rrule_bymonthday', 3, 2, $this->_rrule_bymonthday);
         $ret        .= $formObject->render();
-        $ret        .= ' ' . _MD_EXTCAL_DAY_NUM_MONTH . '</fieldset><br><fieldset><legend>';
+        $ret        .= ' '._MD_EXTCAL_DAY_NUM_MONTH.'</fieldset><br><fieldset><legend>';
 
         $formObject = new \XoopsFormRadio('', 'rrule_freq', $this->_rrule_freq);
         $formObject->addOption('yearly', _MD_EXTCAL_YEARLY);
         $ret .= $formObject->render();
-        $ret .= '</legend>' . _MD_EXTCAL_DURING . ' ';
+        $ret .= '</legend>'._MD_EXTCAL_DURING.' ';
 
         $formObject = new \XoopsFormText('', 'rrule_yearly_interval', 3, 2, $this->_rrule_yearly_interval);
         $ret        .= $formObject->render();
-        $ret        .= ' ' . _MD_EXTCAL_YEARS . '<br>';
+        $ret        .= ' '._MD_EXTCAL_YEARS.'<br>';
 
         $formObject = new Extcal\Form\FormRRuleCheckBox('', 'rrule_yearly_bymonths', $this->_rrule_yearly_bymonths);
         $formObject->addOption('1', _MD_EXTCAL_JAN);

class/Form/FormDateTime.php

Spacing +4 added lines, -4 removed lines

@@ -16,11 +16,11 @@ 
      */
     public function __construct($form, $startTS = 0, $endTS = 0)
     {
-        $startTS       = (int)$startTS;
+        $startTS       = (int) $startTS;
         $startTS       = ($startTS > 0) ? $startTS : time();
         $startDatetime = getdate($startTS);
 
-        $endTS       = (int)$endTS;
+        $endTS       = (int) $endTS;
         $endTS       = ($endTS > 0) ? $endTS : time();
         $endDatetime = getdate($endTS);
 
@@ -28,7 +28,7 @@ 
         for ($i = 0; $i < 24; ++$i) {
             for ($j = 0; $j < _EXTCAL_TS_MINUTE; $j += 15) {
                 $key             = ($i * _EXTCAL_TS_HOUR) + ($j * _EXTCAL_TS_MINUTE);
-                $timearray[$key] = (0 != $j) ? $i . ':' . $j : $i . ':0' . $j;
+                $timearray[$key] = (0 != $j) ? $i.':'.$j : $i.':0'.$j;
             }
         }
         ksort($timearray);
@@ -57,7 +57,7 @@ 
         $endElmtTray     = new \XoopsFormElementTray(_MD_EXTCAL_END_DATE, '<br>');
         $endDateElmtTray = new \XoopsFormElementTray('', '&nbsp;');
 
-        $endElmtTray->addElement( new \XoopsFormRadioYN(_MD_EXTCAL_EVENT_END, 'have_end', 1));
+        $endElmtTray->addElement(new \XoopsFormRadioYN(_MD_EXTCAL_EVENT_END, 'have_end', 1));
 
         $endDate = new \XoopsFormTextDateSelect('', 'event_end[date]', 15, $endTS);
         $endDate->setExtra('onBlur=\'validDate("event_start[date]", "event_start[time]", "event_end[date]", "event_end[time]");\'');

class/EventHandler.php

Spacing +112 added lines, -112 removed lines

@@ -23,7 +23,7 @@ 
 // defined('XOOPS_ROOT_PATH') || exit('Restricted access.');
 
 
-require_once __DIR__ . '/../include/constantes.php';
+require_once __DIR__.'/../include/constantes.php';
 
 
 
@@ -154,9 +154,9 @@ 
     {
         $user = $GLOBALS['xoopsUser'];
 
-        $criteriaCompo =  new \CriteriaCompo();
-        $criteriaCompo->add( new \Criteria('event_id', $eventId));
-        $criteriaCompo->add( new \Criteria('event_approved', 1));
+        $criteriaCompo = new \CriteriaCompo();
+        $criteriaCompo->add(new \Criteria('event_id', $eventId));
+        $criteriaCompo->add(new \Criteria('event_approved', 1));
         if (!$skipPerm) {
             $this->_addCatPermCriteria($criteriaCompo, $user);
         }
@@ -180,8 +180,8 @@ 
     {
         $user = $GLOBALS['xoopsUser'];
 
-        $criteriaCompo =  new \CriteriaCompo();
-        $criteriaCompo->add( new \Criteria('event_id', $eventId));
+        $criteriaCompo = new \CriteriaCompo();
+        $criteriaCompo->add(new \Criteria('event_id', $eventId));
         if (!$skipPerm) {
             $this->_addCatPermCriteria($criteriaCompo, $user);
         }
@@ -338,7 +338,7 @@ 
 
         //        while (list($k, $v) = each($events)) {
         foreach ($events as $k => $v) {
-            $ordre[] = (int)$v['event_start'];
+            $ordre[] = (int) $v['event_start'];
             $this->formatEventDate($v, $extcalConfig['event_date_week']);
             //$v['cat']['cat_light_color'] = $v['cat']['cat_color'];
             $v['cat']['cat_light_color'] = Extcal\Utility::getLighterColor($v['cat']['cat_color'], _EXTCAL_INFOBULLE_RGB_MIN, _EXTCAL_INFOBULLE_RGB_MAX);
@@ -382,7 +382,7 @@ 
             case _EXTCAL_EVENTS_CALENDAR_WEEK:
                 $criteriaCompo = $this->_getEventWeekCriteria($day, $month, $year, $cat, $nbDays);
                 if (!$extcalConfig['diplay_past_event_cal']) {
-                    $criteriaCompo->add( new \Criteria('event_end', time(), '>'));
+                    $criteriaCompo->add(new \Criteria('event_end', time(), '>'));
                 }
                 break;
 
@@ -390,7 +390,7 @@ 
             case _EXTCAL_EVENTS_AGENDA_WEEK:
                 $criteriaCompo = $this->_getEventWeekCriteria($day, $month, $year, $cat, $nbDays);
                 if (!$extcalConfig['diplay_past_event_list']) {
-                    $criteriaCompo->add( new \Criteria('event_end', time(), '>'));
+                    $criteriaCompo->add(new \Criteria('event_end', time(), '>'));
                 }
                 break;
 
@@ -398,7 +398,7 @@ 
                 $criteriaCompo = $this->_getEventMonthCriteria($month, $year, $cat);
 
                 if (!$extcalConfig['diplay_past_event_cal']) {
-                    $criteriaCompo->add( new \Criteria('event_end', time(), '>'));
+                    $criteriaCompo->add(new \Criteria('event_end', time(), '>'));
                 }
                 break;
 
@@ -406,7 +406,7 @@ 
                 $criteriaCompo = $this->_getEventMonthCriteria($month, $year, $cat);
 
                 if (!$extcalConfig['diplay_past_event_list']) {
-                    $criteriaCompo->add( new \Criteria('event_end', time(), '>'));
+                    $criteriaCompo->add(new \Criteria('event_end', time(), '>'));
                 }
                 break;
 
@@ -425,7 +425,7 @@ 
 
         }
         //--------------------------------------------------------------------------
-        $criteriaCompo->add( new \Criteria('event_isrecur', 0, '='));
+        $criteriaCompo->add(new \Criteria('event_isrecur', 0, '='));
         $criteriaCompo->setOrder($sens);
 
         $result = $this->getObjects($criteriaCompo);
@@ -462,7 +462,7 @@ 
         $user = $GLOBALS['xoopsUser'];
         //------------------------------------------------------
 
-        $criteriaCompo =  new \CriteriaCompo();
+        $criteriaCompo = new \CriteriaCompo();
 
         switch ($periode) {
             case _EXTCAL_EVENTS_WEEK:
@@ -480,7 +480,7 @@ 
                 $start = userTimeToServerTime(mktime(0, 0, 0, $month, 1, $year), $this->_extcalTime->_getUserTimeZone($user));
                 $end   = userTimeToServerTime(mktime(23, 59, 59, $month + 1, 1, $year) - _EXTCAL_TS_DAY, $this->_extcalTime->_getUserTimeZone($user));
 
-                $criteriaCompo->add( new \Criteria('event_start', $end, '<='));
+                $criteriaCompo->add(new \Criteria('event_start', $end, '<='));
                 //$criteriaCompo->add( new \Criteria('event_end', $start, '>='));
 
                 break;
@@ -500,7 +500,7 @@ 
         }
         $formatDate = $extcalConfig['event_date_week'];
         //--------------------------------------------------------------------------
-        $criteriaCompo->add( new \Criteria('event_isrecur', 1, '='));
+        $criteriaCompo->add(new \Criteria('event_isrecur', 1, '='));
         $criteriaCompo->setOrder($sens);
 
         $result = $this->getObjects($criteriaCompo);
@@ -543,7 +543,7 @@ 
 
         reset($period);
         foreach ($period as $dt) {
-            echo $dt->format("l d-m-Y H:i:s\n") . '<br>';
+            echo $dt->format("l d-m-Y H:i:s\n").'<br>';
         }
     }
 
@@ -579,7 +579,7 @@ 
      *
      * @return CriteriaCompo
      */
-    public function _getEventWeekCriteria($day, $month, $year, $cat=0, $nbDays = 7)
+    public function _getEventWeekCriteria($day, $month, $year, $cat = 0, $nbDays = 7)
     {
         $user = $GLOBALS['xoopsUser'];
 
@@ -601,7 +601,7 @@ 
      *
      * @return CriteriaCompo
      */
-    public function _getEventMonthCriteria($month, $year, $cat=0)
+    public function _getEventMonthCriteria($month, $year, $cat = 0)
     {
         $user = $GLOBALS['xoopsUser'];
 
@@ -653,26 +653,26 @@ 
      * @return \CriteriaCompo
      */
 
-    public function _getCriteriaCompo($start, $end, $cat=0, &$user)
+    public function _getCriteriaCompo($start, $end, $cat = 0, &$user)
     {
-        $criteriaNoRecur =  new \CriteriaCompo();
-        $criteriaNoRecur->add( new \Criteria('event_start', $end, '<='));
-        $criteriaNoRecur->add( new \Criteria('event_end', $start, '>='));
-        $criteriaNoRecur->add( new \Criteria('event_isrecur', 0));
+        $criteriaNoRecur = new \CriteriaCompo();
+        $criteriaNoRecur->add(new \Criteria('event_start', $end, '<='));
+        $criteriaNoRecur->add(new \Criteria('event_end', $start, '>='));
+        $criteriaNoRecur->add(new \Criteria('event_isrecur', 0));
 
-        $criteriaRecur =  new \CriteriaCompo();
-        $criteriaRecur->add( new \Criteria('event_recur_start', $end, '<='));
-        $criteriaRecur->add( new \Criteria('event_recur_end', $start, '>='));
-        $criteriaRecur->add( new \Criteria('event_isrecur', 1));
+        $criteriaRecur = new \CriteriaCompo();
+        $criteriaRecur->add(new \Criteria('event_recur_start', $end, '<='));
+        $criteriaRecur->add(new \Criteria('event_recur_end', $start, '>='));
+        $criteriaRecur->add(new \Criteria('event_isrecur', 1));
 
-        $criteriaCompoDate =  new \CriteriaCompo();
+        $criteriaCompoDate = new \CriteriaCompo();
         $criteriaCompoDate->add($criteriaNoRecur, 'OR');
         $criteriaCompoDate->add($criteriaRecur, 'OR');
 
-        $criteriaCompo =  new \CriteriaCompo();
+        $criteriaCompo = new \CriteriaCompo();
         $criteriaCompo->add($criteriaCompoDate);
 
-        $criteriaCompo->add( new \Criteria('event_approved', 1));
+        $criteriaCompo->add(new \Criteria('event_approved', 1));
         $this->_addCatSelectCriteria($criteriaCompo, $cat);
         $this->_addCatPermCriteria($criteriaCompo, $user);
         $criteriaCompo->setSort('event_start');
@@ -688,13 +688,13 @@ 
      *
      * @return CriteriaCompo
      */
-    public function _getCalendarCriteriaCompo($start, $end, $cat=0, &$user)
+    public function _getCalendarCriteriaCompo($start, $end, $cat = 0, &$user)
     {
         global $extcalConfig;
         $criteriaCompo = $this->_getCriteriaCompo($start, $end, $cat, $user);
         //if (!$this->_extcalConfig['diplay_past_event_cal']) {
         if (!$extcalConfig['diplay_past_event_cal']) {
-            $criteriaCompo->add( new \Criteria('event_end', time(), '>'));
+            $criteriaCompo->add(new \Criteria('event_end', time(), '>'));
         }
 
         return $criteriaCompo;
@@ -708,13 +708,13 @@ 
      *
      * @return CriteriaCompo
      */
-    public function _getListCriteriaCompo($start, $end, $cat=0, &$user)
+    public function _getListCriteriaCompo($start, $end, $cat = 0, &$user)
     {
         global $extcalConfig;
         $criteriaCompo = $this->_getCriteriaCompo($start, $end, $cat, $user);
         // if (!$this->_extcalConfig['diplay_past_event_list']) {
         if (!$extcalConfig['diplay_past_event_list']) {
-            $criteriaCompo->add( new \Criteria('event_end', time(), '>'));
+            $criteriaCompo->add(new \Criteria('event_end', time(), '>'));
         }
 
         return $criteriaCompo;
@@ -732,22 +732,22 @@ 
     {
         $now = time();
 
-        $criteriaNoRecur =  new \CriteriaCompo();
-        $criteriaNoRecur->add( new \Criteria('event_start', $now, '>='));
-        $criteriaNoRecur->add( new \Criteria('event_isrecur', 0));
+        $criteriaNoRecur = new \CriteriaCompo();
+        $criteriaNoRecur->add(new \Criteria('event_start', $now, '>='));
+        $criteriaNoRecur->add(new \Criteria('event_isrecur', 0));
 
-        $criteriaRecur =  new \CriteriaCompo();
-        $criteriaRecur->add( new \Criteria('event_recur_start', $now, '>='));
-        $criteriaRecur->add( new \Criteria('event_isrecur', 1));
+        $criteriaRecur = new \CriteriaCompo();
+        $criteriaRecur->add(new \Criteria('event_recur_start', $now, '>='));
+        $criteriaRecur->add(new \Criteria('event_isrecur', 1));
 
-        $criteriaCompoDate =  new \CriteriaCompo();
+        $criteriaCompoDate = new \CriteriaCompo();
         $criteriaCompoDate->add($criteriaNoRecur, 'OR');
         $criteriaCompoDate->add($criteriaRecur, 'OR');
 
-        $criteriaCompo =  new \CriteriaCompo();
+        $criteriaCompo = new \CriteriaCompo();
         $criteriaCompo->add($criteriaCompoDate);
 
-        $criteriaCompo->add( new \Criteria('event_approved', 1));
+        $criteriaCompo->add(new \Criteria('event_approved', 1));
         $this->_addCatSelectCriteria($criteriaCompo, $cat);
         $this->_addCatPermCriteria($criteriaCompo, $GLOBALS['xoopsUser']);
 
@@ -783,12 +783,12 @@ 
         $dayStart = mktime(0, 0, 0, $month, $day, $year);
         $dayEnd   = mktime(0, 0, 0, $month, $day + 1, $year);
 
-        $criteriaCompo =  new \CriteriaCompo();
+        $criteriaCompo = new \CriteriaCompo();
         $this->_addCatSelectCriteria($criteriaCompo, $cat);
         $this->_addCatPermCriteria($criteriaCompo, $GLOBALS['xoopsUser']);
-        $criteriaCompo->add( new \Criteria('event_end', $dayStart, '>='));
-        $criteriaCompo->add( new \Criteria('event_start', $dayEnd, '<'));
-        $criteriaCompo->add( new \Criteria('event_approved', 1));
+        $criteriaCompo->add(new \Criteria('event_end', $dayStart, '>='));
+        $criteriaCompo->add(new \Criteria('event_start', $dayEnd, '<'));
+        $criteriaCompo->add(new \Criteria('event_approved', 1));
         $criteriaCompo->setSort('event_start');
         $criteriaCompo->setLimit($nbEvent);
 
@@ -807,12 +807,12 @@ 
      */
     public function getNewEvent($start, $limit, $cat = 0, $skipPerm = false)
     {
-        $criteriaCompo =  new \CriteriaCompo();
+        $criteriaCompo = new \CriteriaCompo();
         $this->_addCatSelectCriteria($criteriaCompo, $cat);
         if (!$skipPerm) {
             $this->_addCatPermCriteria($criteriaCompo, $GLOBALS['xoopsUser']);
         }
-        $criteriaCompo->add( new \Criteria('event_approved', 1));
+        $criteriaCompo->add(new \Criteria('event_approved', 1));
         $criteriaCompo->setSort('event_id');
         $criteriaCompo->setOrder('DESC');
         $criteriaCompo->setStart($start);
@@ -826,9 +826,9 @@ 
      */
     public function getCountNewEvent()
     {
-        $criteriaCompo =  new \CriteriaCompo();
+        $criteriaCompo = new \CriteriaCompo();
         $this->_addCatSelectCriteria($criteriaCompo, 0);
-        $criteriaCompo->add( new \Criteria('event_approved', 1));
+        $criteriaCompo->add(new \Criteria('event_approved', 1));
         $criteriaCompo->setSort('event_id');
 
         return $this->getCount($criteriaCompo);
@@ -844,11 +844,11 @@ 
      */
     public function getRandomEvent($nbEvent, $cat = 0)
     {
-        $criteriaCompo =  new \CriteriaCompo();
+        $criteriaCompo = new \CriteriaCompo();
         $this->_addCatSelectCriteria($criteriaCompo, $cat);
         $this->_addCatPermCriteria($criteriaCompo, $GLOBALS['xoopsUser']);
-        $criteriaCompo->add( new \Criteria('event_start', time(), '>='));
-        $criteriaCompo->add( new \Criteria('event_approved', 1));
+        $criteriaCompo->add(new \Criteria('event_start', time(), '>='));
+        $criteriaCompo->add(new \Criteria('event_approved', 1));
         $criteriaCompo->setSort('RAND()');
         $criteriaCompo->setLimit($nbEvent);
 
@@ -860,8 +860,8 @@ 
      */
     public function getPendingEvent()
     {
-        $criteriaCompo =  new \CriteriaCompo();
-        $criteriaCompo->add( new \Criteria('event_approved', 0));
+        $criteriaCompo = new \CriteriaCompo();
+        $criteriaCompo->add(new \Criteria('event_approved', 0));
         $criteriaCompo->setSort('event_start');
 
         return $this->getObjects($criteriaCompo);
@@ -876,15 +876,15 @@ 
         $authorizedAccessCats = $this->_extcalPerm->getAuthorizedCat($user, 'extcal_cat_view');
         $count                = count($authorizedAccessCats);
         if ($count > 0) {
-            $in = '(' . $authorizedAccessCats[0];
+            $in = '('.$authorizedAccessCats[0];
             array_shift($authorizedAccessCats);
             foreach ($authorizedAccessCats as $authorizedAccessCat) {
-                $in .= ',' . $authorizedAccessCat;
+                $in .= ','.$authorizedAccessCat;
             }
             $in .= ')';
-            $criteria->add( new \Criteria('cat_id', $in, 'IN'));
+            $criteria->add(new \Criteria('cat_id', $in, 'IN'));
         } else {
-            $criteria->add( new \Criteria('cat_id', '(0)', 'IN'));
+            $criteria->add(new \Criteria('cat_id', '(0)', 'IN'));
         }
     }
 
@@ -892,20 +892,20 @@ 
      * @param $criteria
      * @param $cats
      */
-    public function _addCatSelectCriteria(&$criteria, $cats=null)
+    public function _addCatSelectCriteria(&$criteria, $cats = null)
     {
         if (!is_array($cats) && $cats > 0) {
-            $criteria->add( new \Criteria('cat_id', $cats));
+            $criteria->add(new \Criteria('cat_id', $cats));
         }
         if (is_array($cats)) {
             if (false === array_search(0, $cats)) {
-                $in = '(' . current($cats);
+                $in = '('.current($cats);
                 array_shift($cats);
                 foreach ($cats as $cat) {
-                    $in .= ',' . $cat;
+                    $in .= ','.$cat;
                 }
                 $in .= ')';
-                $criteria->add( new \Criteria('cat_id', $in, 'IN'));
+                $criteria->add(new \Criteria('cat_id', $in, 'IN'));
             }
         }
     }
@@ -921,7 +921,7 @@ 
     public function getEventForm($siteSide = 'user', $mode = 'new', $data = null)
     {
         global $xoopsModuleConfig;
-        $catHandler   = Extcal\Helper::getInstance()->getHandler(_EXTCAL_CLN_CAT);
+        $catHandler = Extcal\Helper::getInstance()->getHandler(_EXTCAL_CLN_CAT);
         $fileHandler = Extcal\Helper::getInstance()->getHandler(_EXTCAL_CLN_FILE);
 
         /***************************************************/
@@ -943,7 +943,7 @@ 
             if ('clone' === $mode) {
                 $data['event_id'] = 0;
                 $event->setVar('event_id', 0);
-                $newTitle = $event->getVar('event_title') . ' (' . _MD_EXTCAL_CLONE_OF . $data['event_id'] . ')';
+                $newTitle = $event->getVar('event_title').' ('._MD_EXTCAL_CLONE_OF.$data['event_id'].')';
                 $event->setVar('event_title', $newTitle);
             }
 
@@ -1117,7 +1117,7 @@ 
 
         //-----------------------------------------------
         // Title
-        $form->addElement( new \XoopsFormText(_MD_EXTCAL_TITLE, 'event_title', 80, 255, $title), true);
+        $form->addElement(new \XoopsFormText(_MD_EXTCAL_TITLE, 'event_title', 80, 255, $title), true);
         //-----------------------------------------------
         // Category select
         $catSelect = new \XoopsFormSelect(_MD_EXTCAL_CATEGORY, 'cat_id', $cat);
@@ -1127,7 +1127,7 @@ 
         $form->addElement($catSelect, true);
         //-----------------------------------------------------------
 
-        $file_path = __DIR__ . '/../assets/css/images';
+        $file_path = __DIR__.'/../assets/css/images';
         $tf        = \XoopsLists::getImgListAsArray($file_path);
         array_unshift($tf, _MD_EXTCAL_NONE);
         $xfIcones = new \XoopsFormSelect(_MD_EXTCAL_ICONE, 'event_icone', $event_icone, '');
@@ -1137,7 +1137,7 @@ 
         //etablissement
         $etablissementHandler = Extcal\Helper::getInstance()->getHandler(_EXTCAL_CLN_ETABLISSEMENT);
         $etablissement_select = new \XoopsFormSelect(_MD_EXTCAL_ETABLISSEMENT, 'event_etablissement', $event_etablissement);
-        $criteria             =  new \CriteriaCompo();
+        $criteria             = new \CriteriaCompo();
         $criteria->setSort('nom');
         $criteria->setOrder('ASC');
 
@@ -1191,19 +1191,19 @@ 
         //Price and monnaie
         $monnaie_price = new \XoopsFormElementTray(_MD_EXTCAL_PRICE, '');
         //price
-        $monnaie_price->addElement( new \XoopsFormText('', 'event_price', 20, 255, $event_price));
+        $monnaie_price->addElement(new \XoopsFormText('', 'event_price', 20, 255, $event_price));
         //monnaie
         $monnaie = new \XoopsFormLabel(_MD_EXTCAL_DEVISE2, '');
         $monnaie_price->addElement($monnaie);
         $form->addElement($monnaie_price);
         //----------------------------------------------------------------
-        $form->addElement( new \XoopsFormText(_MD_EXTCAL_ORGANISATEUR, 'event_organisateur', 80, 255, $organisateur), false);
+        $form->addElement(new \XoopsFormText(_MD_EXTCAL_ORGANISATEUR, 'event_organisateur', 80, 255, $organisateur), false);
         // Contact
-        $form->addElement( new \XoopsFormText(_MD_EXTCAL_CONTACT, 'event_contact', 80, 255, $contact), false);
+        $form->addElement(new \XoopsFormText(_MD_EXTCAL_CONTACT, 'event_contact', 80, 255, $contact), false);
         // Url
-        $form->addElement( new \XoopsFormText(_MD_EXTCAL_URL, 'event_url', 80, 255, $url), false);
+        $form->addElement(new \XoopsFormText(_MD_EXTCAL_URL, 'event_url', 80, 255, $url), false);
         // Email
-        $form->addElement( new \XoopsFormText(_MD_EXTCAL_EMAIL, 'event_email', 80, 255, $email), false);
+        $form->addElement(new \XoopsFormText(_MD_EXTCAL_EMAIL, 'event_email', 80, 255, $email), false);
 
         // Address
         if (class_exists('XoopsFormEditor')) {
@@ -1232,23 +1232,23 @@ 
         if (count($files) > 0) {
             $eventFiles = new Extcal\Form\FormFileCheckBox('', 'filetokeep');
             foreach ($files as $file) {
-                $name = $file['file_nicename'] . ' (<i>' . $file['file_mimetype'] . '</i>) ' . $file['formated_file_size'];
+                $name = $file['file_nicename'].' (<i>'.$file['file_mimetype'].'</i>) '.$file['formated_file_size'];
                 $eventFiles->addOption($file['file_id'], $name);
             }
             $fileElmtTray->addElement($eventFiles);
         }
-        $fileElmtTray->addElement( new \XoopsFormFile(_MD_EXTCAL_FILE_ATTACHEMENT, 'event_file', 3145728));
+        $fileElmtTray->addElement(new \XoopsFormFile(_MD_EXTCAL_FILE_ATTACHEMENT, 'event_file', 3145728));
         $form->addElement($fileElmtTray);
 
         if (isset($data['event_id'])) {
-            $form->addElement( new \XoopsFormHidden('event_id', $data['event_id']), false);
+            $form->addElement(new \XoopsFormHidden('event_id', $data['event_id']), false);
         }
         //Hack Kraven0
         ///////////////////////////////////////////////////////////////////////////////
         //Picture1
         $file_tray = new \XoopsFormElementTray(sprintf(_MD_EXTCAL_FORM_IMG, 1), '');
         if (!empty($event_picture1)) {
-            $file_tray->addElement( new \XoopsFormLabel('', "<img src='" . XOOPS_URL . '/uploads/extcal/' . $event_picture1 . "' name='image' id='image' alt=''><br><br>"));
+            $file_tray->addElement(new \XoopsFormLabel('', "<img src='".XOOPS_URL.'/uploads/extcal/'.$event_picture1."' name='image' id='image' alt=''><br><br>"));
             $check_del_img = new \XoopsFormCheckBox('', 'delimg_1');
             $check_del_img->addOption(1, _MD_EXTCAL_DEL_IMG);
             $file_tray->addElement($check_del_img);
@@ -1259,16 +1259,16 @@ 
         }
         $file_img->setExtra("size ='40'");
         $file_tray->addElement($file_img);
-        $msg        = sprintf(_MD_EXTCAL_IMG_CONFIG, (int)(400728 / 1000), 500, 500);
-        $file_label = new \XoopsFormLabel('', '<br>' . $msg);
+        $msg        = sprintf(_MD_EXTCAL_IMG_CONFIG, (int) (400728 / 1000), 500, 500);
+        $file_label = new \XoopsFormLabel('', '<br>'.$msg);
         $file_tray->addElement($file_label);
         $form->addElement($file_tray);
-        $form->addElement( new \XoopsFormHidden('file1', $event_picture1));
+        $form->addElement(new \XoopsFormHidden('file1', $event_picture1));
         unset($file_img, $file_tray);
         //Picture2
         $file_tray = new \XoopsFormElementTray(sprintf(_MD_EXTCAL_FORM_IMG, 2), '');
         if (!empty($event_picture2)) {
-            $file_tray->addElement( new \XoopsFormLabel('', "<img src='" . XOOPS_URL . '/uploads/extcal/' . $event_picture2 . "' name='image' id='image' alt=''><br><br>"));
+            $file_tray->addElement(new \XoopsFormLabel('', "<img src='".XOOPS_URL.'/uploads/extcal/'.$event_picture2."' name='image' id='image' alt=''><br><br>"));
             $check_del_img = new \XoopsFormCheckBox('', 'delimg_2');
             $check_del_img->addOption(1, _MD_EXTCAL_DEL_IMG);
             $file_tray->addElement($check_del_img);
@@ -1279,18 +1279,18 @@ 
         }
         $file_img->setExtra("size ='40'");
         $file_tray->addElement($file_img);
-        $msg        = sprintf(_MD_EXTCAL_IMG_CONFIG, (int)(400728 / 1000), 500, 500);
-        $file_label = new \XoopsFormLabel('', '<br>' . $msg);
+        $msg        = sprintf(_MD_EXTCAL_IMG_CONFIG, (int) (400728 / 1000), 500, 500);
+        $file_label = new \XoopsFormLabel('', '<br>'.$msg);
         $file_tray->addElement($file_label);
         $form->addElement($file_tray);
-        $form->addElement( new \XoopsFormHidden('file2', $event_picture2));
+        $form->addElement(new \XoopsFormHidden('file2', $event_picture2));
         unset($file_img, $file_tray);
         ///////////////////////////////////////////////////////////////////////////////
 
         $buttonElmtTray = new \XoopsFormElementTray('', '&nbsp;');
-        $buttonElmtTray->addElement( new \XoopsFormButton('', 'form_submit', _SUBMIT, 'submit'), false);
+        $buttonElmtTray->addElement(new \XoopsFormButton('', 'form_submit', _SUBMIT, 'submit'), false);
         if ('user' === $siteSide) {
-            $buttonElmtTray->addElement( new \XoopsFormButton('', 'form_preview', _MD_EXTCAL_PREVIEW, 'submit'), false);
+            $buttonElmtTray->addElement(new \XoopsFormButton('', 'form_preview', _MD_EXTCAL_PREVIEW, 'submit'), false);
         }
         $form->addElement($buttonElmtTray);
 
@@ -1347,7 +1347,7 @@ 
                 $recurRules = 'weekly|';
                 $recurRules .= $parm['rrule_weekly_interval'];
                 foreach ($parm['rrule_weekly_bydays'] as $day) {
-                    $recurRules .= '|' . $day;
+                    $recurRules .= '|'.$day;
                 }
 
                 break;
@@ -1357,11 +1357,11 @@ 
                     $parm['rrule_monthly_interval'] = 0;
                 }
                 $recurRules = 'monthly|';
-                $recurRules .= $parm['rrule_monthly_interval'] . '|';
+                $recurRules .= $parm['rrule_monthly_interval'].'|';
                 if ('' != $parm['rrule_monthly_byday']) {
                     $recurRules .= $parm['rrule_monthly_byday'];
                 } else {
-                    $recurRules .= 'MD' . $parm['rrule_bymonthday'];
+                    $recurRules .= 'MD'.$parm['rrule_bymonthday'];
                 }
 
                 break;
@@ -1395,9 +1395,9 @@ 
 
                 $recurRules = 'yearly|';
                 $recurRules .= $parm['rrule_yearly_interval'];
-                $recurRules .= '|' . $parm['rrule_yearly_byday'];
+                $recurRules .= '|'.$parm['rrule_yearly_byday'];
                 foreach ($parm['rrule_yearly_bymonths'] as $month) {
-                    $recurRules .= '|' . $month;
+                    $recurRules .= '|'.$month;
                 }
 
                 break;
@@ -2327,10 +2327,10 @@ 
         global $xoopsDB;
 
         //echo "<hr>{$andor}-{$limit}-{$offset}-{$userId}-{$user}<br>{$criteresPlus}";
-        $tEvent = $xoopsDB->prefix('extcal_event') . ' AS te';
-        $tCat   = $xoopsDB->prefix('extcal_cat') . ' AS tc';
+        $tEvent = $xoopsDB->prefix('extcal_event').' AS te';
+        $tCat   = $xoopsDB->prefix('extcal_cat').' AS tc';
 
-        $sql = 'SELECT te.*, tc.cat_name , tc.cat_color, ' . 'year(FROM_UNIXTIME(event_start)) AS year,' . 'month(FROM_UNIXTIME(event_start)) AS month,' . 'day(FROM_UNIXTIME(event_start)) AS day' . " FROM {$tEvent}, {$tCat}";
+        $sql = 'SELECT te.*, tc.cat_name , tc.cat_color, '.'year(FROM_UNIXTIME(event_start)) AS year,'.'month(FROM_UNIXTIME(event_start)) AS month,'.'day(FROM_UNIXTIME(event_start)) AS day'." FROM {$tEvent}, {$tCat}";
         //---------------------------------------------------
         $tw   = [];
         $tw[] = 'te.cat_id = tc.cat_id';
@@ -2339,7 +2339,7 @@ 
         $authorizedAccessCats = $this->_extcalPerm->getAuthorizedCat($user, 'extcal_cat_view');
         $inCat                = 'te.cat_id IN (0)';
         if (count($authorizedAccessCats) > 0) {
-            $inCat = 'te.cat_id IN (' . implode(',', $authorizedAccessCats) . ')';
+            $inCat = 'te.cat_id IN ('.implode(',', $authorizedAccessCats).')';
         }
         //echo $tw[count($tw)-1];
 
@@ -2375,12 +2375,12 @@ 
                 'te.event_address',
                 'tc.cat_name',
             ];
-            $t       = [];
+            $t = [];
             for ($i = 0, $count = count($queryarray); $i < $count; ++$i) {
                 $t1[] = " %1\$s LIKE '#{$queryarray[$i]}#' ";
             }
 
-            $flt = '(' . implode(" {$andor} ", $t1) . ')';
+            $flt = '('.implode(" {$andor} ", $t1).')';
 
             $t = [];
             for ($h = 0, $count = count($tFields); $h < $count; ++$h) {
@@ -2389,10 +2389,10 @@ 
 
             $filtre = implode(' OR ', $t);
             $filtre = str_replace('#', '%', $filtre);
-            $tw[]   = '(' . $filtre . ')';
+            $tw[]   = '('.$filtre.')';
         }
 
-        $sql .= ' WHERE ' . implode(' AND ', $tw);
+        $sql .= ' WHERE '.implode(' AND ', $tw);
         //------------------------------------------------------------
         if (count($orderBy) > 0) {
             $t = [];
@@ -2402,7 +2402,7 @@ 
                 }
             }
             if (count($t) > 0) {
-                $sql .= ' ORDER BY ' . implode(',', $t);
+                $sql .= ' ORDER BY '.implode(',', $t);
             }
         }
 
@@ -2435,7 +2435,7 @@ 
         $i = 0;
         while ($myrow = $xoopsDB->fetchArray($result)) {
             $ret[$i]['image'] = 'assets/images/icons/extcal.gif';
-            $ret[$i]['link']  = 'event.php?event=' . $myrow['event_id'];
+            $ret[$i]['link']  = 'event.php?event='.$myrow['event_id'];
             $ret[$i]['title'] = $myrow['event_title'];
             $ret[$i]['time']  = $myrow['event_submitdate'];
             $ret[$i]['uid']   = $myrow['event_submitter'];
@@ -2475,23 +2475,23 @@ 
         //        }
         $tEvent = $xoopsDB->prefix('extcal_event');
         $tCat   = $xoopsDB->prefix('extcal_cat');
-        $sql    = "SELECT {$tEvent}.*, {$tCat}.cat_name AS categorie, {$tCat}.cat_color " . " FROM {$tEvent}, {$tCat}" . " WHERE {$tEvent}.cat_id = {$tCat}.cat_id AND event_approved = '1'";
+        $sql    = "SELECT {$tEvent}.*, {$tCat}.cat_name AS categorie, {$tCat}.cat_color "." FROM {$tEvent}, {$tCat}"." WHERE {$tEvent}.cat_id = {$tCat}.cat_id AND event_approved = '1'";
 
         $authorizedAccessCats = $this->_extcalPerm->getAuthorizedCat($user, 'extcal_cat_view');
         $count                = count($authorizedAccessCats);
         if ($count > 0) {
-            $in = '(' . $authorizedAccessCats[0];
+            $in = '('.$authorizedAccessCats[0];
             array_shift($authorizedAccessCats);
             foreach ($authorizedAccessCats as $authorizedAccessCat) {
-                $in .= ',' . $authorizedAccessCat;
+                $in .= ','.$authorizedAccessCat;
             }
             $in .= ')';
         } else {
             $in = '(0)';
         }
-        $sql .= " AND {$tEvent}.cat_id IN " . $in . '';
+        $sql .= " AND {$tEvent}.cat_id IN ".$in.'';
         if (0 != $userId) {
-            $sql .= " AND event_submitter = '" . $userId . "'";
+            $sql .= " AND event_submitter = '".$userId."'";
         }
 
         //echoArray($queryarray,false);
@@ -2511,7 +2511,7 @@ 
                 $t1[] = " %1\$s LIKE '#{$queryarray[$i]}#' ";
             }
 
-            $flt = '(' . implode(" {$andor} ", $t1) . ')';
+            $flt = '('.implode(" {$andor} ", $t1).')';
 
             $t = [];
             for ($h = 0, $count = count($tFields); $h < $count; ++$h) {
@@ -2520,11 +2520,11 @@ 
 
             $filtre = implode(' OR ', $t);
             $filtre = str_replace('#', '%', $filtre);
-            $sql    .= " AND ($filtre)";
+            $sql .= " AND ($filtre)";
         }
 
         if ('' != $criteresPlus) {
-            $sql .= ' AND ' . $criteresPlus;
+            $sql .= ' AND '.$criteresPlus;
         }
         $sql .= ' ORDER BY event_id DESC';
 
@@ -2534,7 +2534,7 @@ 
         if ($xoopsSearch) {
             while ($myrow = $xoopsDB->fetchArray($result)) {
                 $ret[$i]['image'] = 'assets/images/icons/extcal.gif';
-                $ret[$i]['link']  = 'event.php?event=' . $myrow['event_id'];
+                $ret[$i]['link']  = 'event.php?event='.$myrow['event_id'];
                 $ret[$i]['title'] = $myrow['event_title'];
                 $ret[$i]['time']  = $myrow['event_submitdate'];
                 $ret[$i]['uid']   = $myrow['event_submitter'];

class/Category.php

Spacing +2 added lines, -2 removed lines

@@ -20,8 +20,8 @@
 // defined('XOOPS_ROOT_PATH') || exit('Restricted access.');
 
  // require_once __DIR__ . '/ExtcalPersistableObjectHandler.php';
-require_once __DIR__ . '/perm.php';
-require_once __DIR__ . '/time.php';
+require_once __DIR__.'/perm.php';
+require_once __DIR__.'/time.php';
 
 /**
  * Class Category.

class/ExtcalPersistableObjectHandler.php

Spacing +46 added lines, -46 removed lines

@@ -100,12 +100,12 @@ 
     public function get($id = null, $fields = null, $as_object = true) //get($id, $as_object = true)
     {
         if (is_array($this->keyName)) {
-            $criteria =  new \CriteriaCompo();
+            $criteria = new \CriteriaCompo();
             for ($i = 0, $iMax = count($this->keyName); $i < $iMax; ++$i) {
-                $criteria->add( new \Criteria($this->keyName[$i], (int)$id[$i]));
+                $criteria->add(new \Criteria($this->keyName[$i], (int) $id[$i]));
             }
         } else {
-            $criteria =  new \Criteria($this->keyName, (int)$id);
+            $criteria = new \Criteria($this->keyName, (int) $id);
         }
         $criteria->setLimit(1);
         $objectArray = $this->getObjects($criteria, false, true);
@@ -129,11 +129,11 @@ 
     {
         $ret   = [];
         $limit = $start = 0;
-        $sql   = 'SELECT * FROM ' . $this->table;
+        $sql   = 'SELECT * FROM '.$this->table;
         if (isset($criteria) && is_subclass_of($criteria, 'CriteriaElement')) {
-            $sql .= ' ' . $criteria->renderWhere();
+            $sql .= ' '.$criteria->renderWhere();
             if ('' != $criteria->getSort()) {
-                $sql .= ' ORDER BY ' . $criteria->getSort() . ' ' . $criteria->getOrder();
+                $sql .= ' ORDER BY '.$criteria->getSort().' '.$criteria->getOrder();
             }
             $limit = $criteria->getLimit();
             $start = $criteria->getStart();
@@ -168,7 +168,7 @@ 
                     $ret[] = $obj;
                 } else {
                     $row  = [];
-                    $vars =& $obj->getVars();
+                    $vars = & $obj->getVars();
                     foreach (array_keys($vars) as $i) {
                         $row[$i] = $obj->getVar($i);
                     }
@@ -179,7 +179,7 @@ 
                     $ret[$myrow[$this->keyName]] = $obj;
                 } else {
                     $row  = [];
-                    $vars =& $obj->getVars();
+                    $vars = & $obj->getVars();
                     foreach (array_keys($vars) as $i) {
                         $row[$i] = $obj->getVar($i);
                     }
@@ -205,22 +205,22 @@ 
     {
         $ret = [];
         if (null === $criteria) {
-            $criteria =  new \CriteriaCompo();
+            $criteria = new \CriteriaCompo();
         }
 
         if ('' == $criteria->getSort()) {
             $criteria->setSort($this->identifierName);
         }
 
-        $sql = 'SELECT ' . $this->keyName;
+        $sql = 'SELECT '.$this->keyName;
         if (!empty($this->identifierName)) {
-            $sql .= ', ' . $this->identifierName;
+            $sql .= ', '.$this->identifierName;
         }
-        $sql .= ' FROM ' . $this->table;
+        $sql .= ' FROM '.$this->table;
         if (isset($criteria) && is_subclass_of($criteria, 'CriteriaElement')) {
-            $sql .= ' ' . $criteria->renderWhere();
+            $sql .= ' '.$criteria->renderWhere();
             if ('' != $criteria->getSort()) {
-                $sql .= ' ORDER BY ' . $criteria->getSort() . ' ' . $criteria->getOrder();
+                $sql .= ' ORDER BY '.$criteria->getSort().' '.$criteria->getOrder();
             }
             $limit = $criteria->getLimit();
             $start = $criteria->getStart();
@@ -253,12 +253,12 @@ 
         if (isset($criteria) && is_subclass_of($criteria, 'CriteriaElement')) {
             if ('' != $criteria->groupby) {
                 $groupby = true;
-                $field   = $criteria->groupby . ', '; //Not entirely secure unless you KNOW that no criteria's groupby clause is going to be mis-used
+                $field   = $criteria->groupby.', '; //Not entirely secure unless you KNOW that no criteria's groupby clause is going to be mis-used
             }
         }
-        $sql = 'SELECT ' . $field . 'COUNT(*) FROM ' . $this->table;
+        $sql = 'SELECT '.$field.'COUNT(*) FROM '.$this->table;
         if (isset($criteria) && is_subclass_of($criteria, 'CriteriaElement')) {
-            $sql .= ' ' . $criteria->renderWhere();
+            $sql .= ' '.$criteria->renderWhere();
             if ('' != $criteria->groupby) {
                 $sql .= $criteria->getGroupby();
             }
@@ -294,13 +294,13 @@ 
         if (is_array($this->keyName)) {
             $clause = [];
             for ($i = 0, $iMax = count($this->keyName); $i < $iMax; ++$i) {
-                $clause[] = $this->keyName[$i] . ' = ' . $id[$i];
+                $clause[] = $this->keyName[$i].' = '.$id[$i];
             }
             $whereclause = implode(' AND ', $clause);
         } else {
-            $whereclause = $this->keyName . ' = ' . $id;
+            $whereclause = $this->keyName.' = '.$id;
         }
-        $sql = 'DELETE FROM ' . $this->table . ' WHERE ' . $whereclause;
+        $sql = 'DELETE FROM '.$this->table.' WHERE '.$whereclause;
         if (false !== $force) {
             $result = $this->db->queryF($sql);
         } else {
@@ -331,7 +331,7 @@ 
             }
 
             if (!($obj instanceof $this->className && class_exists($this->className))) {
-                $obj->setErrors(get_class($obj) . ' Differs from ' . $this->className);
+                $obj->setErrors(get_class($obj).' Differs from '.$this->className);
 
                 return false;
             }
@@ -342,7 +342,7 @@ 
 
         foreach ($obj->cleanVars as $k => $v) {
             if (XOBJ_DTYPE_INT == $obj->vars[$k]['data_type']) {
-                $cleanvars[$k] = (int)$v;
+                $cleanvars[$k] = (int) $v;
             } elseif (is_array($v)) {
                 $cleanvars[$k] = $this->db->quoteString(implode(',', $v));
             } else {
@@ -352,12 +352,12 @@ 
         if ($obj->isNew()) {
             if (!is_array($this->keyName)) {
                 if ($cleanvars[$this->keyName] < 1) {
-                    $cleanvars[$this->keyName] = $this->db->genId($this->table . '_' . $this->keyName . '_seq');
+                    $cleanvars[$this->keyName] = $this->db->genId($this->table.'_'.$this->keyName.'_seq');
                 }
             }
-            $sql = 'INSERT INTO ' . $this->table . ' (' . implode(',', array_keys($cleanvars)) . ') VALUES (' . implode(',', array_values($cleanvars)) . ')';
+            $sql = 'INSERT INTO '.$this->table.' ('.implode(',', array_keys($cleanvars)).') VALUES ('.implode(',', array_values($cleanvars)).')';
         } else {
-            $sql = 'UPDATE ' . $this->table . ' SET';
+            $sql = 'UPDATE '.$this->table.' SET';
             foreach ($cleanvars as $key => $value) {
                 if ((!is_array($this->keyName) && $key == $this->keyName)
                     || (is_array($this->keyName)
@@ -367,7 +367,7 @@ 
                 if (isset($notfirst)) {
                     $sql .= ',';
                 }
-                $sql      .= ' ' . $key . ' = ' . $value;
+                $sql .= ' '.$key.' = '.$value;
                 $notfirst = true;
             }
             if (is_array($this->keyName)) {
@@ -376,12 +376,12 @@ 
                     if ($i > 0) {
                         $whereclause .= ' AND ';
                     }
-                    $whereclause .= $this->keyName[$i] . ' = ' . $obj->getVar($this->keyName[$i]);
+                    $whereclause .= $this->keyName[$i].' = '.$obj->getVar($this->keyName[$i]);
                 }
             } else {
-                $whereclause = $this->keyName . ' = ' . $obj->getVar($this->keyName);
+                $whereclause = $this->keyName.' = '.$obj->getVar($this->keyName);
             }
-            $sql .= ' WHERE ' . $whereclause;
+            $sql .= ' WHERE '.$whereclause;
         }
         if (false !== $force) {
             $result = $this->db->queryF($sql);
@@ -410,7 +410,7 @@ 
      */
     public function updateAll($fieldname, $fieldvalue, \CriteriaElement $criteria = null, $force = false)
     {
-        $setClause = $fieldname . ' = ';
+        $setClause = $fieldname.' = ';
         if (is_numeric($fieldvalue)) {
             $setClause .= $fieldvalue;
         } elseif (is_array($fieldvalue)) {
@@ -418,9 +418,9 @@ 
         } else {
             $setClause .= $this->db->quoteString($fieldvalue);
         }
-        $sql = 'UPDATE ' . $this->table . ' SET ' . $setClause;
+        $sql = 'UPDATE '.$this->table.' SET '.$setClause;
         if (isset($criteria) && is_subclass_of($criteria, 'CriteriaElement')) {
-            $sql .= ' ' . $criteria->renderWhere();
+            $sql .= ' '.$criteria->renderWhere();
         }
         if (false !== $force) {
             $result = $this->db->queryF($sql);
@@ -444,9 +444,9 @@ 
      */
     public function updateFieldValue($fieldname, $fieldvalue, $criteria = null, $force = true)
     {
-        $sql = 'UPDATE ' . $this->table . ' SET ' . $fieldname . ' = ' . $fieldvalue;
+        $sql = 'UPDATE '.$this->table.' SET '.$fieldname.' = '.$fieldvalue;
         if (isset($criteria) && is_subclass_of($criteria, 'CriteriaElement')) {
-            $sql .= ' ' . $criteria->renderWhere();
+            $sql .= ' '.$criteria->renderWhere();
         }
         if (false !== $force) {
             $result = $this->db->queryF($sql);
@@ -473,8 +473,8 @@ 
     public function deleteAll(\CriteriaElement $criteria = null, $force = true, $asObject = false)
     {
         if (isset($criteria) && is_subclass_of($criteria, 'CriteriaElement')) {
-            $sql = 'DELETE FROM ' . $this->table;
-            $sql .= ' ' . $criteria->renderWhere();
+            $sql = 'DELETE FROM '.$this->table;
+            $sql .= ' '.$criteria->renderWhere();
             if (!$this->db->query($sql)) {
                 return false;
             }
@@ -608,8 +608,8 @@ 
      */
     public function updateCounter($fieldname, $criteria, $op = '+')
     {
-        $sql    = 'UPDATE ' . $this->table . ' SET ' . $fieldname . ' = ' . $fieldname . $op . '1';
-        $sql    .= ' ' . $criteria->renderWhere();
+        $sql    = 'UPDATE '.$this->table.' SET '.$fieldname.' = '.$fieldname.$op.'1';
+        $sql .= ' '.$criteria->renderWhere();
         $result = $this->db->queryF($sql);
         if (!$result) {
             return false;
@@ -631,12 +631,12 @@ 
         if (isset($criteria) && is_subclass_of($criteria, 'CriteriaElement')) {
             if ('' != $criteria->groupby) {
                 $groupby = true;
-                $field   = $criteria->groupby . ', '; //Not entirely secure unless you KNOW that no criteria's groupby clause is going to be mis-used
+                $field   = $criteria->groupby.', '; //Not entirely secure unless you KNOW that no criteria's groupby clause is going to be mis-used
             }
         }
-        $sql = 'SELECT ' . $field . "SUM($sum) FROM " . $this->table;
+        $sql = 'SELECT '.$field."SUM($sum) FROM ".$this->table;
         if (isset($criteria) && is_subclass_of($criteria, 'CriteriaElement')) {
-            $sql .= ' ' . $criteria->renderWhere();
+            $sql .= ' '.$criteria->renderWhere();
             if ('' != $criteria->groupby) {
                 $sql .= $criteria->getGroupby();
             }
@@ -672,12 +672,12 @@ 
         if (isset($criteria) && is_subclass_of($criteria, 'CriteriaElement')) {
             if ('' != $criteria->groupby) {
                 $groupby = true;
-                $field   = $criteria->groupby . ', '; //Not entirely secure unless you KNOW that no criteria's groupby clause is going to be mis-used
+                $field   = $criteria->groupby.', '; //Not entirely secure unless you KNOW that no criteria's groupby clause is going to be mis-used
             }
         }
-        $sql = 'SELECT ' . $field . "MAX($max) FROM " . $this->table;
+        $sql = 'SELECT '.$field."MAX($max) FROM ".$this->table;
         if (isset($criteria) && is_subclass_of($criteria, 'CriteriaElement')) {
-            $sql .= ' ' . $criteria->renderWhere();
+            $sql .= ' '.$criteria->renderWhere();
             if ('' != $criteria->groupby) {
                 $sql .= $criteria->getGroupby();
             }
@@ -710,9 +710,9 @@ 
     {
         $field = '';
 
-        $sql = 'SELECT ' . $field . "AVG($avg) FROM " . $this->table;
+        $sql = 'SELECT '.$field."AVG($avg) FROM ".$this->table;
         if (isset($criteria) && is_subclass_of($criteria, 'CriteriaElement')) {
-            $sql .= ' ' . $criteria->renderWhere();
+            $sql .= ' '.$criteria->renderWhere();
         }
         $result = $this->db->query($sql);
         if (!$result) {

class/EtablissementHandler.php

Spacing +3 added lines, -3 removed lines

@@ -20,7 +20,7 @@ 
 //Kraven 30
 // defined('XOOPS_ROOT_PATH') || exit('Restricted access.');
 
-require_once XOOPS_ROOT_PATH . '/kernel/object.php';
+require_once XOOPS_ROOT_PATH.'/kernel/object.php';
 
 
 /**
@@ -46,8 +46,8 @@ 
     {
         $user = $GLOBALS['xoopsUser'];
 
-        $criteriaCompo =  new \CriteriaCompo();
-        $criteriaCompo->add( new \Criteria('id', $etablissementId));
+        $criteriaCompo = new \CriteriaCompo();
+        $criteriaCompo->add(new \Criteria('id', $etablissementId));
 
         if (!$skipPerm) {
             $this->_addCatPermCriteria($criteriaCompo, $user);

class/Helper.php

Spacing +1 added lines, -1 removed lines

@@ -70,7 +70,7 @@
     {
         $ret   = false;
         $db    = \XoopsDatabaseFactory::getDatabaseConnection();
-        $class = '\\XoopsModules\\' . ucfirst(strtolower(basename(dirname(__DIR__)))) . '\\' . $name . 'Handler';
+        $class = '\\XoopsModules\\'.ucfirst(strtolower(basename(dirname(__DIR__)))).'\\'.$name.'Handler';
         $ret   = new $class($db);
         return $ret;
     }

xoops_version.php

Spacing +12 added lines, -12 removed lines

@@ -21,11 +21,11 @@ 
 
 // defined('XOOPS_ROOT_PATH') || die('XOOPS Root Path not defined');
 
-include __DIR__ . '/preloads/autoloader.php';
+include __DIR__.'/preloads/autoloader.php';
 
-require_once __DIR__ . '/include/constantes.php';
-require_once __DIR__ . '/include/agenda_fnc.php';
-require_once __DIR__ . '/class/config.php';
+require_once __DIR__.'/include/constantes.php';
+require_once __DIR__.'/include/agenda_fnc.php';
+require_once __DIR__.'/class/config.php';
 //$loc_de = setlocale (LC_ALL, 'french');
 
 //echo "local :" .  setlocale(LC_TIME, $xoopsConfig['language'])."</ br>";
@@ -76,7 +76,7 @@ 
     && 'extcal' === $GLOBALS['xoopsModule']->getVar('dirname')) {
     $user = isset($GLOBALS['xoopsUser']) ? $GLOBALS['xoopsUser'] : null;
 //    $catHandler = xoops_getModuleHandler(_EXTCAL_CLS_CAT, _EXTCAL_MODULE);
-    $catHandler   = Extcal\Helper::getInstance()->getHandler(_EXTCAL_CLN_CAT);
+    $catHandler = Extcal\Helper::getInstance()->getHandler(_EXTCAL_CLN_CAT);
     if ($catHandler->haveSubmitRight($user)) {
         $modversion['sub'][0]['name'] = _MI_EXTCAL_SUBMIT_EVENT;
         $modversion['sub'][0]['url'] = _EXTCAL_FILE_NEW_EVENT;
@@ -97,12 +97,12 @@ 
 
 // Tables created by sql file (without prefix!)
 $modversion['tables'] = [
-    $moduleDirName . '_' . 'cat',
-    $moduleDirName . '_' . 'event',
-    $moduleDirName . '_' . 'eventmember',
-    $moduleDirName . '_' . 'eventnotmember',
-    $moduleDirName . '_' . 'file',
-    $moduleDirName . '_' . 'etablissement'
+    $moduleDirName.'_'.'cat',
+    $moduleDirName.'_'.'event',
+    $moduleDirName.'_'.'eventmember',
+    $moduleDirName.'_'.'eventnotmember',
+    $moduleDirName.'_'.'file',
+    $moduleDirName.'_'.'etablissement'
 ];
 
 // SQL
@@ -514,7 +514,7 @@ 
 $modversion['config'][$i]['default']     = 5;
 
 ++$i;
-$modversion['config'][$i]['name']        = 'break' . $i;
+$modversion['config'][$i]['name']        = 'break'.$i;
 $modversion['config'][$i]['title']       = '_MI_EXTCAL_SHOW_OTHEROPTIONS';
 $modversion['config'][$i]['description'] = '';
 $modversion['config'][$i]['formtype']    = 'line_break';

admin/admin_header.php

Spacing +3 added lines, -3 removed lines

@@ -19,11 +19,11 @@
 
 use XoopsModules\Extcal;
 
-require_once __DIR__ . '/../../../include/cp_header.php';
+require_once __DIR__.'/../../../include/cp_header.php';
 require_once $GLOBALS['xoops']->path('www/class/xoopsformloader.php');
-require_once __DIR__ . '/../include/constantes.php';
+require_once __DIR__.'/../include/constantes.php';
 
-include __DIR__ . '/../preloads/autoloader.php';
+include __DIR__.'/../preloads/autoloader.php';
 
 $moduleDirName = basename(dirname(__DIR__));