mambax7 / extcal

include/notification.inc.php

Spacing +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
         $eventHandler = xoops_getModuleHandler(_EXTCAL_CLS_EVENT, _EXTCAL_MODULE);
         $event        = $eventHandler->getEvent($itemId, 0, true);
         $item['name'] = $event->getVar('event_title');
-        $item['url']  = XOOPS_URL . '/modules/extcal/event.php?event=' . $event->getVar('event_id');
+        $item['url']  = XOOPS_URL.'/modules/extcal/event.php?event='.$event->getVar('event_id');
 
         return $item;
     }

class/ExtcalPersistableObjectHandler.php

Doc Comments +1 added lines, -1 removed lines

@@ -536,7 +536,7 @@
 
     /**
      * @param        $objects
-     * @param array  $externalKeys
+     * @param string[]  $externalKeys
      * @param string $format
      *
      * @return array

Spacing +42 added lines, -42 removed lines

@@ -83,10 +83,10 @@ 
         if (is_array($this->keyName)) {
             $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);
@@ -110,11 +110,11 @@ 
     {
         $ret   = array();
         $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();
@@ -193,15 +193,15 @@ 
             $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();
@@ -234,12 +234,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();
             }
@@ -275,13 +275,13 @@ 
         if (is_array($this->keyName)) {
             $clause = array();
             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 {
@@ -312,7 +312,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;
             }
@@ -323,7 +323,7 @@ 
 
         foreach ($obj->cleanVars as $k => $v) {
             if ($obj->vars[$k]['data_type'] == XOBJ_DTYPE_INT) {
-                $cleanvars[$k] = (int)$v;
+                $cleanvars[$k] = (int) $v;
             } elseif (is_array($v)) {
                 $cleanvars[$k] = $this->db->quoteString(implode(',', $v));
             } else {
@@ -333,12 +333,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)
@@ -349,7 +349,7 @@ 
                 if (isset($notfirst)) {
                     $sql .= ',';
                 }
-                $sql .= ' ' . $key . ' = ' . $value;
+                $sql .= ' '.$key.' = '.$value;
                 $notfirst = true;
             }
             if (is_array($this->keyName)) {
@@ -358,12 +358,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);
@@ -392,7 +392,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)) {
@@ -400,9 +400,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);
@@ -426,9 +426,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);
@@ -455,8 +455,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;
             }
@@ -590,8 +590,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;
@@ -613,12 +613,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();
             }
@@ -654,12 +654,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();
             }
@@ -692,9 +692,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/pear/Calendar/Decorator.php

Doc Comments +2 added lines, -2 removed lines

@@ -136,7 +136,7 @@ 
     /**
      * Returns the date as an associative array (helper method).
      *
-     * @param mixed $stamp timestamp (leave empty for current timestamp)
+     * @param integer $stamp timestamp (leave empty for current timestamp)
      *
      * @return array
      */
@@ -242,7 +242,7 @@ 
      *
      * @param array $sDates array containing Calendar objects to select (optional)
      *
-     * @return bool
+     * @return boolean|null
      * @abstract
      */
     public function build($sDates = array())

class/pear/Calendar/Engine/Interface.php

Doc Comments +1 added lines, -1 removed lines

@@ -346,7 +346,7 @@
      *
      * @param int timestamp (depending on implementation)
      *
-     * @return bool
+     * @return boolean|null
      */
     public function isToday($stamp)
     {

event_notmember.php

Indentation +3 added lines, -3 removed lines

@@ -34,9 +34,9 @@
         if ($_POST['mode'] === 'add') {
             $event = $eventHandler->getEvent((int) $_POST['event'], $xoopsUser);
             $eventNotMemberHandler->createEventNotMember(array(
-                                                             'event_id' => (int) $_POST['event'],
-                                                             'uid' => $xoopsUser->getVar('uid'),
-                                                         ));
+                                                                'event_id' => (int) $_POST['event'],
+                                                                'uid' => $xoopsUser->getVar('uid'),
+                                                            ));
             sendMail2member($mode, $event_id, $member_uid, _MD_EXTCAL_SUBJECT_3, _MD_EXTCAL_MSG_3);
             $rediredtMessage = _MD_EXTCAL_WHOSNOT_GOING_ADDED_TO_EVENT;
 

view_calendar-month.php

Indentation +2 added lines, -2 removed lines

@@ -65,8 +65,8 @@
 // Flag current day
 $selectedDays = array(
     new Calendar_Day(date('Y', xoops_getUserTimestamp(time(), $extcalTimeHandler->_getUserTimeZone($xoopsUser))),
-                     date('n', xoops_getUserTimestamp(time(), $extcalTimeHandler->_getUserTimeZone($xoopsUser))),
-                     date('j', xoops_getUserTimestamp(time(), $extcalTimeHandler->_getUserTimeZone($xoopsUser)))),
+                        date('n', xoops_getUserTimestamp(time(), $extcalTimeHandler->_getUserTimeZone($xoopsUser))),
+                        date('j', xoops_getUserTimestamp(time(), $extcalTimeHandler->_getUserTimeZone($xoopsUser)))),
 );
 
 // Build calendar object

event_member.php

Indentation +3 added lines, -3 removed lines

@@ -59,9 +59,9 @@
                 $rediredtMessage = _MD_EXTCAL_MAX_MEMBER_REACHED;
             } else {
                 $eventMemberHandler->createEventmember(array(
-                                                           'event_id' => (int) $_POST['event'],
-                                                           'uid' => $xoopsUser->getVar('uid'),
-                                                       ));
+                                                            'event_id' => (int) $_POST['event'],
+                                                            'uid' => $xoopsUser->getVar('uid'),
+                                                        ));
                 sendMail2member($mode, $event_id, $member_uid, _MD_EXTCAL_SUBJECT_1, _MD_EXTCAL_MSG_1);
                 $rediredtMessage = _MD_EXTCAL_WHOS_GOING_ADDED_TO_EVENT;
             }

class/pear/Calendar/docs/examples/13.php

Switch Indentation +26 added lines, -26 removed lines

@@ -39,32 +39,32 @@
 }
 
 switch (@$_GET['view']) {
-    default:
-        $_GET['view'] = 'calendar_year';
-    case 'calendar_year':
-        require_once CALENDAR_ROOT.'Year.php';
-        $c = new Calendar_Year($_GET['y']);
-        break;
-    case 'calendar_month':
-        require_once CALENDAR_ROOT.'Month.php';
-        $c = new Calendar_Month($_GET['y'], $_GET['m']);
-        break;
-    case 'calendar_day':
-        require_once CALENDAR_ROOT.'Day.php';
-        $c = new Calendar_Day($_GET['y'], $_GET['m'], $_GET['d']);
-        break;
-    case 'calendar_hour':
-        require_once CALENDAR_ROOT.'Hour.php';
-        $c = new Calendar_Hour($_GET['y'], $_GET['m'], $_GET['d'], $_GET['h']);
-        break;
-    case 'calendar_minute':
-        require_once CALENDAR_ROOT.'Minute.php';
-        $c = new Calendar_Minute($_GET['y'], $_GET['m'], $_GET['d'], $_GET['h'], $_GET['i']);
-        break;
-    case 'calendar_second':
-        require_once CALENDAR_ROOT.'Second.php';
-        $c = new Calendar_Second($_GET['y'], $_GET['m'], $_GET['d'], $_GET['h'], $_GET['i'], $_GET['s']);
-        break;
+        default:
+            $_GET['view'] = 'calendar_year';
+        case 'calendar_year':
+            require_once CALENDAR_ROOT.'Year.php';
+            $c = new Calendar_Year($_GET['y']);
+            break;
+        case 'calendar_month':
+            require_once CALENDAR_ROOT.'Month.php';
+            $c = new Calendar_Month($_GET['y'], $_GET['m']);
+            break;
+        case 'calendar_day':
+            require_once CALENDAR_ROOT.'Day.php';
+            $c = new Calendar_Day($_GET['y'], $_GET['m'], $_GET['d']);
+            break;
+        case 'calendar_hour':
+            require_once CALENDAR_ROOT.'Hour.php';
+            $c = new Calendar_Hour($_GET['y'], $_GET['m'], $_GET['d'], $_GET['h']);
+            break;
+        case 'calendar_minute':
+            require_once CALENDAR_ROOT.'Minute.php';
+            $c = new Calendar_Minute($_GET['y'], $_GET['m'], $_GET['d'], $_GET['h'], $_GET['i']);
+            break;
+        case 'calendar_second':
+            require_once CALENDAR_ROOT.'Second.php';
+            $c = new Calendar_Second($_GET['y'], $_GET['m'], $_GET['d'], $_GET['h'], $_GET['i'], $_GET['s']);
+            break;
 }
 
 // Convert timestamp to human readable date

class/pear/Calendar/docs/examples/6.php

Indentation +4 added lines, -4 removed lines

@@ -85,10 +85,10 @@ 
                     echo "<td></td>\n";
                 } elseif ($Day->isSelected()) {
                     echo '<td><anchor><strong><u>'.$Day->thisDay()."</u></strong>\n<go href=\"".$_SERVER['PHP_SELF'].'?viewday=true&amp;y='.$Day->thisYear().'&amp;m='
-                             .$Day->thisMonth().'&amp;d='.$Day->thisDay()."&amp;mime=wml\" />\n</anchor></td>\n";
+                                .$Day->thisMonth().'&amp;d='.$Day->thisDay()."&amp;mime=wml\" />\n</anchor></td>\n";
                 } else {
                     echo '<td><anchor>'.$Day->thisDay()."\n<go href=\"?viewday=true&amp;y=".$Day->thisYear().'&amp;m='.$Day->thisMonth().'&amp;d='.$Day->thisDay()
-                             ."&amp;mime=wml\" /></anchor></td>\n";
+                                ."&amp;mime=wml\" /></anchor></td>\n";
                 }
                 if ($Day->isLast()) {
                     echo "</tr>\n";
@@ -181,10 +181,10 @@ 
                 echo "<td></td>\n";
             } elseif ($Day->isSelected()) {
                 echo '<td><a href="'.$_SERVER['PHP_SELF'].'?viewday=true&amp;y='.$Day->thisYear().'&amp;m='.$Day->thisMonth().'&amp;d='.$Day->thisDay().'&amp;wml"><strong><u>'
-                         .$Day->thisDay()."</u></strong></a></td>\n";
+                            .$Day->thisDay()."</u></strong></a></td>\n";
             } else {
                 echo '<td><a href="'.$_SERVER['PHP_SELF'].'?viewday=true&amp;y='.$Day->thisYear().'&amp;m='.$Day->thisMonth().'&amp;d='.$Day->thisDay().'">'.$Day->thisDay()
-                         ."</a></td>\n";
+                            ."</a></td>\n";
             }
             if ($Day->isLast()) {
                 echo "</tr>\n";