Inspection of "PSR-4" - FriendsOfPear/Calendar - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#2)

by tsms

created 2019-03-12 05:14 UTC

Status

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

@@ -14,18 +14,18 @@
 block discarded – undo
 // Build the month
 $Calendar = new Calendar_Month_Weekdays($_GET['jahr'], $_GET['monat']);
 
-echo ( '<p>The current month is '
+echo ('<p>The current month is '
         .$Calendar->thisMonth().' of year '.$Calendar->thisYear().'</p>');
 
 $Uri = & new Calendar_Decorator_Uri($Calendar);
-$Uri->setFragments('jahr','monat');
+$Uri->setFragments('jahr', 'monat');
 // $Uri->setSeperator('/'); // Default is &
 // $Uri->setScalar(); // Omit variable names
-echo ( "<pre>Previous Uri:\t".$Uri->prev('month')."\n" );
-echo ( "This Uri:\t".$Uri->this('month')."\n" );
-echo ( "Next Uri:\t".$Uri->next('month')."\n</pre>" );
+echo ("<pre>Previous Uri:\t".$Uri->prev('month')."\n");
+echo ("This Uri:\t".$Uri->this('month')."\n");
+echo ("Next Uri:\t".$Uri->next('month')."\n</pre>");
 ?>
 <p>
-<a href="<?php echo($_SERVER['PHP_SELF'].'?'.$Uri->prev('month'));?>">Prev</a> :
-<a href="<?php echo($_SERVER['PHP_SELF'].'?'.$Uri->next('month'));?>">Next</a>
+<a href="<?php echo($_SERVER['PHP_SELF'].'?'.$Uri->prev('month')); ?>">Prev</a> :
+<a href="<?php echo($_SERVER['PHP_SELF'].'?'.$Uri->next('month')); ?>">Next</a>
 </p>
\ No newline at end of file

Please login to merge, or discard this patch.

Braces +6 added lines, -2 removed lines patch added patch discarded remove patch

@@ -8,8 +8,12 @@
 block discarded – undo
 require_once CALENDAR_ROOT.'Month/Weekdays.php';
 require_once CALENDAR_ROOT.'Decorator/Uri.php';
 
-if (!isset($_GET['jahr'])) $_GET['jahr'] = date('Y');
-if (!isset($_GET['monat'])) $_GET['monat'] = date('m');
+if (!isset($_GET['jahr'])) {
+    $_GET['jahr'] = date('Y');
+}
+if (!isset($_GET['monat'])) {
+    $_GET['monat'] = date('m');
+}
 
 // Build the month
 $Calendar = new Calendar_Month_Weekdays($_GET['jahr'], $_GET['monat']);

Please login to merge, or discard this patch.

docs/examples/11.php 1 patch

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

@@ -3,8 +3,8 @@  discard block
 block discarded – undo
 * Description: demonstrates a decorator used to "attach a payload" to a selection
 * to make it available when iterating over calendar children
 */
-if ( !@include 'Calendar/Calendar.php' ) {
-    define('CALENDAR_ROOT','../../');
+if (!@include 'Calendar/Calendar.php') {
+    define('CALENDAR_ROOT', '../../');
 }
 require_once CALENDAR_ROOT.'Day.php';
 require_once CALENDAR_ROOT.'Hour.php';
@@ -25,7 +25,7 @@  discard block
 block discarded – undo
 }
 
 // Create a day to view the hours for
-$Day = & new Calendar_Day(2003,10,24);
+$Day = & new Calendar_Day(2003, 10, 24);
 
 // A sample query to get the data for today (NOT ACTUALLY USED HERE)
 $sql = "
@@ -39,18 +39,18 @@  discard block
 block discarded – undo
             eventtime < '".$Day->nextDay(TRUE)."';";
 
 // An array simulating data from a database
-$result = array (
-    array('eventtime'=>mktime(9,0,0,10,24,2003),'entry'=>'Meeting with sales team'),
-    array('eventtime'=>mktime(11,0,0,10,24,2003),'entry'=>'Conference call with Widget Inc.'),
-    array('eventtime'=>mktime(15,0,0,10,24,2003),'entry'=>'Presentation to board of directors')
+$result = array(
+    array('eventtime'=>mktime(9, 0, 0, 10, 24, 2003), 'entry'=>'Meeting with sales team'),
+    array('eventtime'=>mktime(11, 0, 0, 10, 24, 2003), 'entry'=>'Conference call with Widget Inc.'),
+    array('eventtime'=>mktime(15, 0, 0, 10, 24, 2003), 'entry'=>'Presentation to board of directors')
     );
 
 // An array to place selected hours in
 $selection = array();
 
 // Loop through the "database result"
-foreach ( $result as $row ) {
-    $Hour = new Calendar_Hour(2000,1,1,1); // Create Hour with dummy values
+foreach ($result as $row) {
+    $Hour = new Calendar_Hour(2000, 1, 1, 1); // Create Hour with dummy values
     $Hour->setTimeStamp($row['eventtime']); // Set the real time with setTimeStamp
 
     // Create the decorator, passing it the Hour
@@ -74,36 +74,36 @@  discard block
 block discarded – undo
 <body>
 <h1>Passing a Selection "Payload" using a Decorator</h1>
 <table>
-<caption><b>Your Schedule for <?php echo ( date('D nS F Y',$Day->thisDay(TRUE)) ); ?></b></caption>
+<caption><b>Your Schedule for <?php echo (date('D nS F Y', $Day->thisDay(TRUE))); ?></b></caption>
 <tr>
 <th width="5%">Time</th>
 <th>Entry</th>
 </tr>
 <?php
-while ( $Hour = & $Day->fetch() ) {
+while ($Hour = & $Day->fetch()) {
 
     $hour = $Hour->thisHour();
     $minute = $Hour->thisMinute();
 
     // Office hours only...
-    if ( $hour >= 8 && $hour <= 18 ) {
-        echo ( "<tr>\n" );
-        echo ( "<td>$hour:$minute</td>\n" );
+    if ($hour >= 8 && $hour <= 18) {
+        echo ("<tr>\n");
+        echo ("<td>$hour:$minute</td>\n");
 
         // If the hour is selected, call the decorator method...
-        if ( $Hour->isSelected() ) {
-            echo ( "<td bgcolor=\"silver\">".$Hour->getEntry()."</td>\n" );
+        if ($Hour->isSelected()) {
+            echo ("<td bgcolor=\"silver\">".$Hour->getEntry()."</td>\n");
         } else {
-            echo ( "<td>&nbsp;</td>\n" );
+            echo ("<td>&nbsp;</td>\n");
         }
-        echo ( "</tr>\n" );
+        echo ("</tr>\n");
     }
 }
 ?>
 </table>
 <p>The query to fetch this data, with help from PEAR::Calendar, might be;</p>
 <pre>
-<?php echo ( $sql ); ?>
+<?php echo ($sql); ?>
 </pre>
 </body>
 </html>
\ No newline at end of file

Please login to merge, or discard this patch.

docs/examples/23.php 1 patch

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

@@ -25,12 +25,12 @@  discard block
 block discarded – undo
 echo "<hr>Creating: new Calendar_Day(date('Y'), date('n'), date('d'));<br />";
 $Calendar = new Calendar_Day(date('Y'), date('n'), date('d'));
 
-echo '<hr>Previous month is: '.Calendar_Util_Textual::prevMonthName($Calendar,'two').'<br />';
-echo 'This month is: '.Calendar_Util_Textual::thisMonthName($Calendar,'short').'<br />';
+echo '<hr>Previous month is: '.Calendar_Util_Textual::prevMonthName($Calendar, 'two').'<br />';
+echo 'This month is: '.Calendar_Util_Textual::thisMonthName($Calendar, 'short').'<br />';
 echo 'Next month is: '.Calendar_Util_Textual::nextMonthName($Calendar).'<br /><hr />';
 echo 'Previous day is: '.Calendar_Util_Textual::prevDayName($Calendar).'<br />';
-echo 'This day is: '.Calendar_Util_Textual::thisDayName($Calendar,'short').'<br />';
-echo 'Next day is: '.Calendar_Util_Textual::nextDayName($Calendar,'one').'<br /><hr />';
+echo 'This day is: '.Calendar_Util_Textual::thisDayName($Calendar, 'short').'<br />';
+echo 'Next day is: '.Calendar_Util_Textual::nextDayName($Calendar, 'one').'<br /><hr />';
 
 echo "Creating: new Calendar_Month_Weekdays(date('Y'), date('n'), 6); - Saturday is first day of week<br />";
 $Calendar = new Calendar_Month_Weekdays(date('Y'), date('n'), 6);
@@ -41,7 +41,7 @@  discard block
 block discarded – undo
 <caption><?php echo Calendar_Util_Textual::thisMonthName($Calendar).' '.$Calendar->thisYear(); ?></caption>
 <tr>
 <?php
-$dayheaders = Calendar_Util_Textual::orderedWeekdays($Calendar,'short');
+$dayheaders = Calendar_Util_Textual::orderedWeekdays($Calendar, 'short');
 foreach ($dayheaders as $dayheader) {
     echo '<th>'.$dayheader.'</th>';
 }

Please login to merge, or discard this patch.

src/Week.php 2 patches

Switch Indentation +58 added lines, -58 removed lines patch added patch discarded remove patch

@@ -346,22 +346,22 @@  discard block
 block discarded – undo
     function prevWeek($format = 'n_in_month')
     {
         switch (strtolower($format)) {
-        case 'int':
-        case 'n_in_month':
-            return ($this->firstWeek) ? null : $this->thisWeek('n_in_month') -1;
-        case 'n_in_year':
-            return $this->cE->getWeekNInYear(
-                $this->cE->stampToYear($this->prevWeek),
-                $this->cE->stampToMonth($this->prevWeek),
-                $this->cE->stampToDay($this->prevWeek));
-        case 'array':
-            return $this->toArray($this->prevWeek);
-        case 'object':
-            include_once CALENDAR_ROOT.'Factory.php';
-            return Factory::createByTimestamp('Week', $this->prevWeek);
-        case 'timestamp':
-        default:
-            return $this->prevWeek;
+            case 'int':
+            case 'n_in_month':
+                return ($this->firstWeek) ? null : $this->thisWeek('n_in_month') -1;
+            case 'n_in_year':
+                return $this->cE->getWeekNInYear(
+                    $this->cE->stampToYear($this->prevWeek),
+                    $this->cE->stampToMonth($this->prevWeek),
+                    $this->cE->stampToDay($this->prevWeek));
+            case 'array':
+                return $this->toArray($this->prevWeek);
+            case 'object':
+                include_once CALENDAR_ROOT.'Factory.php';
+                return Factory::createByTimestamp('Week', $this->prevWeek);
+            case 'timestamp':
+            default:
+                return $this->prevWeek;
         }
     }
 
@@ -376,35 +376,35 @@  discard block
 block discarded – undo
     function thisWeek($format = 'n_in_month')
     {
         switch (strtolower($format)) {
-        case 'int':
-        case 'n_in_month':
-            if ($this->firstWeek) {
-                return 1;
-            }
-            if ($this->lastWeek) {
-                return $this->cE->getWeeksInMonth(
+            case 'int':
+            case 'n_in_month':
+                if ($this->firstWeek) {
+                    return 1;
+                }
+                if ($this->lastWeek) {
+                    return $this->cE->getWeeksInMonth(
+                        $this->thisYear(),
+                        $this->thisMonth(),
+                        $this->firstDay);
+                }
+                return $this->cE->getWeekNInMonth(
                     $this->thisYear(),
                     $this->thisMonth(),
+                    $this->thisDay(),
                     $this->firstDay);
-            }
-            return $this->cE->getWeekNInMonth(
-                $this->thisYear(),
-                $this->thisMonth(),
-                $this->thisDay(),
-                $this->firstDay);
-        case 'n_in_year':
-            return $this->cE->getWeekNInYear(
-                $this->cE->stampToYear($this->thisWeek),
-                $this->cE->stampToMonth($this->thisWeek),
-                $this->cE->stampToDay($this->thisWeek));
-        case 'array':
-            return $this->toArray($this->thisWeek);
-        case 'object':
-            include_once CALENDAR_ROOT.'Factory.php';
-            return Factory::createByTimestamp('Week', $this->thisWeek);
-        case 'timestamp':
-        default:
-            return $this->thisWeek;
+            case 'n_in_year':
+                return $this->cE->getWeekNInYear(
+                    $this->cE->stampToYear($this->thisWeek),
+                    $this->cE->stampToMonth($this->thisWeek),
+                    $this->cE->stampToDay($this->thisWeek));
+            case 'array':
+                return $this->toArray($this->thisWeek);
+            case 'object':
+                include_once CALENDAR_ROOT.'Factory.php';
+                return Factory::createByTimestamp('Week', $this->thisWeek);
+            case 'timestamp':
+            default:
+                return $this->thisWeek;
         }
     }
 
@@ -419,22 +419,22 @@  discard block
 block discarded – undo
     function nextWeek($format = 'n_in_month')
     {
         switch (strtolower($format)) {
-        case 'int':
-        case 'n_in_month':
-            return ($this->lastWeek) ? null : $this->thisWeek('n_in_month') +1;
-        case 'n_in_year':
-            return $this->cE->getWeekNInYear(
-                $this->cE->stampToYear($this->nextWeek),
-                $this->cE->stampToMonth($this->nextWeek),
-                $this->cE->stampToDay($this->nextWeek));
-        case 'array':
-            return $this->toArray($this->nextWeek);
-        case 'object':
-            include_once CALENDAR_ROOT.'Factory.php';
-            return Factory::createByTimestamp('Week', $this->nextWeek);
-        case 'timestamp':
-        default:
-            return $this->nextWeek;
+            case 'int':
+            case 'n_in_month':
+                return ($this->lastWeek) ? null : $this->thisWeek('n_in_month') +1;
+            case 'n_in_year':
+                return $this->cE->getWeekNInYear(
+                    $this->cE->stampToYear($this->nextWeek),
+                    $this->cE->stampToMonth($this->nextWeek),
+                    $this->cE->stampToDay($this->nextWeek));
+            case 'array':
+                return $this->toArray($this->nextWeek);
+            case 'object':
+                include_once CALENDAR_ROOT.'Factory.php';
+                return Factory::createByTimestamp('Week', $this->nextWeek);
+            case 'timestamp':
+            default:
+                return $this->nextWeek;
         }
     }
 

Please login to merge, or discard this patch.

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

@@ -223,7 +223,7 @@  discard block
 block discarded – undo
             $this->thisDay()
         );
 
-        for ($i=1; $i <= $end; $i++) {
+        for ($i = 1; $i <= $end; $i++) {
             $stamp = $this->cE->dateToStamp($year, $month, $day++);
             $this->children[$i] = new Day(
                 $this->cE->stampToYear($stamp),
@@ -235,13 +235,13 @@  discard block
 block discarded – undo
         //set empty days (@see Calendar_Month_Weeks::build())
         if ($this->firstWeek) {
             $eBefore = $this->tableHelper->getEmptyDaysBefore();
-            for ($i=1; $i <= $eBefore; $i++) {
+            for ($i = 1; $i <= $eBefore; $i++) {
                 $this->children[$i]->setEmpty();
             }
         }
         if ($this->lastWeek) {
             $eAfter = $this->tableHelper->getEmptyDaysAfterOffset();
-            for ($i = $eAfter+1; $i <= $end; $i++) {
+            for ($i = $eAfter + 1; $i <= $end; $i++) {
                 $this->children[$i]->setEmpty();
             }
         }
@@ -348,7 +348,7 @@  discard block
 block discarded – undo
         switch (strtolower($format)) {
         case 'int':
         case 'n_in_month':
-            return ($this->firstWeek) ? null : $this->thisWeek('n_in_month') -1;
+            return ($this->firstWeek) ? null : $this->thisWeek('n_in_month') - 1;
         case 'n_in_year':
             return $this->cE->getWeekNInYear(
                 $this->cE->stampToYear($this->prevWeek),
@@ -421,7 +421,7 @@  discard block
 block discarded – undo
         switch (strtolower($format)) {
         case 'int':
         case 'n_in_month':
-            return ($this->lastWeek) ? null : $this->thisWeek('n_in_month') +1;
+            return ($this->lastWeek) ? null : $this->thisWeek('n_in_month') + 1;
         case 'n_in_year':
             return $this->cE->getWeekNInYear(
                 $this->cE->stampToYear($this->nextWeek),

Please login to merge, or discard this patch.

src/Hour.php 2 patches

Indentation +2 added lines, -2 removed lines patch added patch discarded remove patch

@@ -98,11 +98,11 @@
 block discarded – undo
     {
         include_once CALENDAR_ROOT.'Minute.php';
         $mIH = $this->cE->getMinutesInHour($this->year, $this->month, $this->day,
-                           $this->hour);
+                            $this->hour);
         for ($i=0; $i < $mIH; $i++) {
             $this->children[$i] =
                 new Minute($this->year, $this->month, $this->day,
-                           $this->hour, $i);
+                            $this->hour, $i);
         }
         if (count($sDates) > 0) {
             $this->setSelection($sDates);

Please login to merge, or discard this patch.

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

@@ -99,7 +99,7 @@
 block discarded – undo
         include_once CALENDAR_ROOT.'Minute.php';
         $mIH = $this->cE->getMinutesInHour($this->year, $this->month, $this->day,
                            $this->hour);
-        for ($i=0; $i < $mIH; $i++) {
+        for ($i = 0; $i < $mIH; $i++) {
             $this->children[$i] =
                 new Minute($this->year, $this->month, $this->day,
                            $this->hour, $i);

Please login to merge, or discard this patch.

src/Engine/CalendarEngineInterface.php 1 patch

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

@@ -186,7 +186,7 @@  discard block
 block discarded – undo
      * @return int (e.g. 12)
      * @access protected
      */
-    function getMonthsInYear($y=null)
+    function getMonthsInYear($y = null)
     {
     }
 
@@ -213,7 +213,7 @@  discard block
 block discarded – undo
      * @return int
      * @access protected
      */
-    function getFirstDayInMonth ($y, $m)
+    function getFirstDayInMonth($y, $m)
     {
     }
 
@@ -227,7 +227,7 @@  discard block
 block discarded – undo
      * @return int (e.g. 7)
      * @access protected
      */
-    function getDaysInWeek($y=null, $m=null, $d=null)
+    function getDaysInWeek($y = null, $m = null, $d = null)
     {
     }
 
@@ -256,7 +256,7 @@  discard block
 block discarded – undo
      * @return int week number
      * @access protected
      */
-    function getWeekNInMonth($y, $m, $d, $firstDay=1)
+    function getWeekNInMonth($y, $m, $d, $firstDay = 1)
     {
     }
 
@@ -297,7 +297,7 @@  discard block
 block discarded – undo
      * @return array list of numeric values of days in week, beginning 0
      * @access protected
      */
-    function getWeekDays($y=null, $m=null, $d=null)
+    function getWeekDays($y = null, $m = null, $d = null)
     {
     }
 
@@ -313,7 +313,7 @@  discard block
 block discarded – undo
      * @see getWeekDays
      * @access protected
      */
-    function getFirstDayOfWeek($y=null, $m=null, $d=null)
+    function getFirstDayOfWeek($y = null, $m = null, $d = null)
     {
     }
 
@@ -327,7 +327,7 @@  discard block
 block discarded – undo
      * @return int (e.g. 24)
      * @access protected
      */
-    function getHoursInDay($y=null,$m=null,$d=null)
+    function getHoursInDay($y = null, $m = null, $d = null)
     {
     }
 
@@ -342,7 +342,7 @@  discard block
 block discarded – undo
      * @return int
      * @access protected
      */
-    function getMinutesInHour($y=null,$m=null,$d=null,$h=null)
+    function getMinutesInHour($y = null, $m = null, $d = null, $h = null)
     {
     }
 
@@ -358,7 +358,7 @@  discard block
 block discarded – undo
      * @return int
      * @access protected
      */
-    function getSecondsInMinute($y=null,$m=null,$d=null,$h=null,$i=null)
+    function getSecondsInMinute($y = null, $m = null, $d = null, $h = null, $i = null)
     {
     }
 

Please login to merge, or discard this patch.

src/Engine/UnixTS.php 1 patch

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

@@ -77,7 +77,7 @@  discard block
 block discarded – undo
     function stampCollection($stamp)
     {
         static $stamps = array();
-        if ( !isset($stamps[$stamp]) ) {
+        if (!isset($stamps[$stamp])) {
             $date = @date('Y n j H i s t W w', $stamp);
             $stamps[$stamp] = sscanf($date, "%d %d %d %d %d %d %d %d %d");
         }
@@ -181,7 +181,7 @@  discard block
 block discarded – undo
      * @return int Unix timestamp
      * @access protected
      */
-    function dateToStamp($y, $m, $d, $h=0, $i=0, $s=0)
+    function dateToStamp($y, $m, $d, $h = 0, $i = 0, $s = 0)
     {
         static $dates = array();
         if (!isset($dates[$y][$m][$d][$h][$i][$s])) {
@@ -220,7 +220,7 @@  discard block
 block discarded – undo
      * @return int (12)
      * @access protected
      */
-    function getMonthsInYear($y=null)
+    function getMonthsInYear($y = null)
     {
         return 12;
     }
@@ -268,7 +268,7 @@  discard block
 block discarded – undo
      * @return int (7)
      * @access protected
      */
-    function getDaysInWeek($y=null, $m=null, $d=null)
+    function getDaysInWeek($y = null, $m = null, $d = null)
     {
         return 7;
     }
@@ -301,9 +301,9 @@  discard block
 block discarded – undo
      * @return int week number
      * @access protected
      */
-    function getWeekNInMonth($y, $m, $d, $firstDay=1)
+    function getWeekNInMonth($y, $m, $d, $firstDay = 1)
     {
-        $weekEnd = (0 == $firstDay) ? $this->getDaysInWeek()-1 : $firstDay-1;
+        $weekEnd = (0 == $firstDay) ? $this->getDaysInWeek() - 1 : $firstDay - 1;
         $end_of_week = 1;
         while (@date('w', @mktime(0, 0, 0, $m, $end_of_week, $y)) != $weekEnd) {
             ++$end_of_week; //find first weekend of the month
@@ -371,7 +371,7 @@  discard block
 block discarded – undo
      * @return array (0,1,2,3,4,5,6) 1 = Monday
      * @access protected
      */
-    function getWeekDays($y=null, $m=null, $d=null)
+    function getWeekDays($y = null, $m = null, $d = null)
     {
         return array(0, 1, 2, 3, 4, 5, 6);
     }
@@ -386,7 +386,7 @@  discard block
 block discarded – undo
      * @return int (default 1 = Monday)
      * @access protected
      */
-    function getFirstDayOfWeek($y=null, $m=null, $d=null)
+    function getFirstDayOfWeek($y = null, $m = null, $d = null)
     {
         return 1;
     }
@@ -401,7 +401,7 @@  discard block
 block discarded – undo
      * @return int (24)
      * @access protected
      */
-    function getHoursInDay($y=null, $m=null, $d=null)
+    function getHoursInDay($y = null, $m = null, $d = null)
     {
         return 24;
     }
@@ -417,7 +417,7 @@  discard block
 block discarded – undo
      * @return int (60)
      * @access protected
      */
-    function getMinutesInHour($y=null, $m=null, $d=null, $h=null)
+    function getMinutesInHour($y = null, $m = null, $d = null, $h = null)
     {
         return 60;
     }
@@ -434,7 +434,7 @@  discard block
 block discarded – undo
      * @return int (60)
      * @access protected
      */
-    function getSecondsInMinute($y=null, $m=null, $d=null, $h=null, $i=null)
+    function getSecondsInMinute($y = null, $m = null, $d = null, $h = null, $i = null)
     {
         return 60;
     }
@@ -455,7 +455,7 @@  discard block
 block discarded – undo
             $today = sscanf($today_date, '%d %d %d');
         }
         $date = UnixTS::stampCollection($stamp);
-        return (   $date[2] == $today[2]
+        return ($date[2] == $today[2]
                 && $date[1] == $today[1]
                 && $date[0] == $today[0]
         );

Please login to merge, or discard this patch.

src/Engine/PearDate.php 1 patch

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

@@ -167,7 +167,7 @@  discard block
 block discarded – undo
      * @return string iso-8601 datetime
      * @access protected
      */
-    function dateToStamp($y, $m, $d, $h=0, $i=0, $s=0)
+    function dateToStamp($y, $m, $d, $h = 0, $i = 0, $s = 0)
     {
         $r = array();
         PearDate::adjustDate($y, $m, $d, $h, $i, $s);
@@ -218,8 +218,8 @@  discard block
 block discarded – undo
             $d += floor($h / 24);
             $h %= 24;
         }
-        for(; $m < 1; $y--, $m+=12);
-        for(; $m > 12; $y++, $m-=12);
+        for (; $m < 1; $y--, $m += 12);
+        for (; $m > 12; $y++, $m -= 12);
 
         while ($d < 1) {
             if ($m > 1) {
@@ -230,7 +230,7 @@  discard block
 block discarded – undo
             }
             $d += Date_Calc::daysInMonth($m, $y);
         }
-        for ($max_days = Date_Calc::daysInMonth($m, $y); $d > $max_days; ) {
+        for ($max_days = Date_Calc::daysInMonth($m, $y); $d > $max_days;) {
             $d -= $max_days;
             if ($m < 12) {
                 $m++;
@@ -271,7 +271,7 @@  discard block
 block discarded – undo
      * @return int (12)
      * @access protected
      */
-    function getMonthsInYear($y=null)
+    function getMonthsInYear($y = null)
     {
         return 12;
     }
@@ -315,7 +315,7 @@  discard block
 block discarded – undo
      * @return int (7)
      * @access protected
      */
-    function getDaysInWeek($y=null, $m=null, $d=null)
+    function getDaysInWeek($y = null, $m = null, $d = null)
     {
         return 7;
     }
@@ -348,9 +348,9 @@  discard block
 block discarded – undo
      * @return int week number
      * @access protected
      */
-    function getWeekNInMonth($y, $m, $d, $firstDay=1)
+    function getWeekNInMonth($y, $m, $d, $firstDay = 1)
     {
-        $weekEnd = ($firstDay == 0) ? $this->getDaysInWeek()-1 : $firstDay-1;
+        $weekEnd = ($firstDay == 0) ? $this->getDaysInWeek() - 1 : $firstDay - 1;
         $end_of_week = (int)Date_Calc::nextDayOfWeek($weekEnd, 1, $m, $y, '%e', true);
         $w = 1;
         while ($d > $end_of_week) {
@@ -370,7 +370,7 @@  discard block
 block discarded – undo
      * @return int weeks number
      * @access protected
      */
-    function getWeeksInMonth($y, $m, $firstDay=1)
+    function getWeeksInMonth($y, $m, $firstDay = 1)
     {
         $FDOM = Date_Calc::firstOfMonthWeekday($m, $y);
         if ($FDOM == 0) {
@@ -413,7 +413,7 @@  discard block
 block discarded – undo
      * @return array (0, 1, 2, 3, 4, 5, 6) 1 = Monday
      * @access protected
      */
-    function getWeekDays($y=null, $m=null, $d=null)
+    function getWeekDays($y = null, $m = null, $d = null)
     {
         return array(0, 1, 2, 3, 4, 5, 6);
     }
@@ -428,7 +428,7 @@  discard block
 block discarded – undo
      * @return int (default 1 = Monday)
      * @access protected
      */
-    function getFirstDayOfWeek($y=null, $m=null, $d=null)
+    function getFirstDayOfWeek($y = null, $m = null, $d = null)
     {
         return 1;
     }
@@ -443,7 +443,7 @@  discard block
 block discarded – undo
      * @return int (24)
      * @access protected
      */
-    function getHoursInDay($y=null,$m=null,$d=null)
+    function getHoursInDay($y = null, $m = null, $d = null)
     {
         return 24;
     }
@@ -459,7 +459,7 @@  discard block
 block discarded – undo
      * @return int (60)
      * @access protected
      */
-    function getMinutesInHour($y=null,$m=null,$d=null,$h=null)
+    function getMinutesInHour($y = null, $m = null, $d = null, $h = null)
     {
         return 60;
     }
@@ -476,7 +476,7 @@  discard block
 block discarded – undo
      * @return int (60)
      * @access protected
      */
-    function getSecondsInMinute($y=null,$m=null,$d=null,$h=null,$i=null)
+    function getSecondsInMinute($y = null, $m = null, $d = null, $h = null, $i = null)
     {
         return 60;
     }
@@ -496,7 +496,7 @@  discard block
 block discarded – undo
             $today = new Date();
         }
         $date = PearDate::stampCollection($stamp);
-        return (   $date->day == $today->getDay()
+        return ($date->day == $today->getDay()
                 && $date->month == $today->getMonth()
                 && $date->year == $today->getYear()
         );

Please login to merge, or discard this patch.

src/Minute.php 1 patch

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

@@ -100,7 +100,7 @@
 block discarded – undo
         include_once CALENDAR_ROOT.'Second.php';
         $sIM = $this->cE->getSecondsInMinute($this->year, $this->month,
                 $this->day, $this->hour, $this->minute);
-        for ($i=0; $i < $sIM; $i++) {
+        for ($i = 0; $i < $sIM; $i++) {
             $this->children[$i] = new Second($this->year, $this->month,
                 $this->day, $this->hour, $this->minute, $i);
         }

Please login to merge, or discard this patch.

		@@ -223,7 +223,7 @@ discard block
		block discarded – undo
223	223	$this->thisDay()
224	224	);
225	225
226		- for ($i=1; $i <= $end; $i++) {
	226	+ for ($i = 1; $i <= $end; $i++) {
227	227	$stamp = $this->cE->dateToStamp($year, $month, $day++);
228	228	$this->children[$i] = new Day(
229	229	$this->cE->stampToYear($stamp),
		@@ -235,13 +235,13 @@ discard block
		block discarded – undo
235	235	//set empty days (@see Calendar_Month_Weeks::build())
236	236	if ($this->firstWeek) {
237	237	$eBefore = $this->tableHelper->getEmptyDaysBefore();
238		- for ($i=1; $i <= $eBefore; $i++) {
	238	+ for ($i = 1; $i <= $eBefore; $i++) {
239	239	$this->children[$i]->setEmpty();
240	240	}
241	241	}
242	242	if ($this->lastWeek) {
243	243	$eAfter = $this->tableHelper->getEmptyDaysAfterOffset();
244		- for ($i = $eAfter+1; $i <= $end; $i++) {
	244	+ for ($i = $eAfter + 1; $i <= $end; $i++) {
245	245	$this->children[$i]->setEmpty();
246	246	}
247	247	}
		@@ -348,7 +348,7 @@ discard block
		block discarded – undo
348	348	switch (strtolower($format)) {
349	349	case 'int':
350	350	case 'n_in_month':
351		- return ($this->firstWeek) ? null : $this->thisWeek('n_in_month') -1;
	351	+ return ($this->firstWeek) ? null : $this->thisWeek('n_in_month') - 1;
352	352	case 'n_in_year':
353	353	return $this->cE->getWeekNInYear(
354	354	$this->cE->stampToYear($this->prevWeek),
		@@ -421,7 +421,7 @@ discard block
		block discarded – undo
421	421	switch (strtolower($format)) {
422	422	case 'int':
423	423	case 'n_in_month':
424		- return ($this->lastWeek) ? null : $this->thisWeek('n_in_month') +1;
	424	+ return ($this->lastWeek) ? null : $this->thisWeek('n_in_month') + 1;
425	425	case 'n_in_year':
426	426	return $this->cE->getWeekNInYear(
427	427	$this->cE->stampToYear($this->nextWeek),

		@@ -186,7 +186,7 @@ discard block
		block discarded – undo
186	186	* @return int (e.g. 12)
187	187	* @access protected
188	188	*/
189		- function getMonthsInYear($y=null)
	189	+ function getMonthsInYear($y = null)
190	190	{
191	191	}
192	192
		@@ -213,7 +213,7 @@ discard block
		block discarded – undo
213	213	* @return int
214	214	* @access protected
215	215	*/
216		- function getFirstDayInMonth ($y, $m)
	216	+ function getFirstDayInMonth($y, $m)
217	217	{
218	218	}
219	219
		@@ -227,7 +227,7 @@ discard block
		block discarded – undo
227	227	* @return int (e.g. 7)
228	228	* @access protected
229	229	*/
230		- function getDaysInWeek($y=null, $m=null, $d=null)
	230	+ function getDaysInWeek($y = null, $m = null, $d = null)
231	231	{
232	232	}
233	233
		@@ -256,7 +256,7 @@ discard block
		block discarded – undo
256	256	* @return int week number
257	257	* @access protected
258	258	*/
259		- function getWeekNInMonth($y, $m, $d, $firstDay=1)
	259	+ function getWeekNInMonth($y, $m, $d, $firstDay = 1)
260	260	{
261	261	}
262	262
		@@ -297,7 +297,7 @@ discard block
		block discarded – undo
297	297	* @return array list of numeric values of days in week, beginning 0
298	298	* @access protected
299	299	*/
300		- function getWeekDays($y=null, $m=null, $d=null)
	300	+ function getWeekDays($y = null, $m = null, $d = null)
301	301	{
302	302	}
303	303
		@@ -313,7 +313,7 @@ discard block
		block discarded – undo
313	313	* @see getWeekDays
314	314	* @access protected
315	315	*/
316		- function getFirstDayOfWeek($y=null, $m=null, $d=null)
	316	+ function getFirstDayOfWeek($y = null, $m = null, $d = null)
317	317	{
318	318	}
319	319
		@@ -327,7 +327,7 @@ discard block
		block discarded – undo
327	327	* @return int (e.g. 24)
328	328	* @access protected
329	329	*/
330		- function getHoursInDay($y=null,$m=null,$d=null)
	330	+ function getHoursInDay($y = null, $m = null, $d = null)
331	331	{
332	332	}
333	333
		@@ -342,7 +342,7 @@ discard block
		block discarded – undo
342	342	* @return int
343	343	* @access protected
344	344	*/
345		- function getMinutesInHour($y=null,$m=null,$d=null,$h=null)
	345	+ function getMinutesInHour($y = null, $m = null, $d = null, $h = null)
346	346	{
347	347	}
348	348
		@@ -358,7 +358,7 @@ discard block
		block discarded – undo
358	358	* @return int
359	359	* @access protected
360	360	*/
361		- function getSecondsInMinute($y=null,$m=null,$d=null,$h=null,$i=null)
	361	+ function getSecondsInMinute($y = null, $m = null, $d = null, $h = null, $i = null)
362	362	{
363	363	}
364	364

		@@ -100,7 +100,7 @@
		block discarded – undo
100	100	include_once CALENDAR_ROOT.'Second.php';
101	101	$sIM = $this->cE->getSecondsInMinute($this->year, $this->month,
102	102	$this->day, $this->hour, $this->minute);
103		- for ($i=0; $i < $sIM; $i++) {
	103	+ for ($i = 0; $i < $sIM; $i++) {
104	104	$this->children[$i] = new Second($this->year, $this->month,
105	105	$this->day, $this->hour, $this->minute, $i);
106	106	}

		@@ -14,18 +14,18 @@
		block discarded – undo
14	14	// Build the month
15	15	$Calendar = new Calendar_Month_Weekdays($_GET['jahr'], $_GET['monat']);
16	16
17		-echo ( '<p>The current month is '
	17	+echo ('<p>The current month is '
18	18	.$Calendar->thisMonth().' of year '.$Calendar->thisYear().'</p>');
19	19
20	20	$Uri = & new Calendar_Decorator_Uri($Calendar);
21		-$Uri->setFragments('jahr','monat');
	21	+$Uri->setFragments('jahr', 'monat');
22	22	// $Uri->setSeperator('/'); // Default is &
23	23	// $Uri->setScalar(); // Omit variable names
24		-echo ( "<pre>Previous Uri:\t".$Uri->prev('month')."\n" );
25		-echo ( "This Uri:\t".$Uri->this('month')."\n" );
26		-echo ( "Next Uri:\t".$Uri->next('month')."\n</pre>" );
	24	+echo ("<pre>Previous Uri:\t".$Uri->prev('month')."\n");
	25	+echo ("This Uri:\t".$Uri->this('month')."\n");
	26	+echo ("Next Uri:\t".$Uri->next('month')."\n</pre>");
27	27	?>
28	28	<p>
29		-<a href="<?php echo($_SERVER['PHP_SELF'].'?'.$Uri->prev('month'));?>">Prev</a> :
30		-<a href="<?php echo($_SERVER['PHP_SELF'].'?'.$Uri->next('month'));?>">Next</a>
	29	+<a href="<?php echo($_SERVER['PHP_SELF'].'?'.$Uri->prev('month')); ?>">Prev</a> :
	30	+<a href="<?php echo($_SERVER['PHP_SELF'].'?'.$Uri->next('month')); ?>">Next</a>
31	31	</p>
32	32	\ No newline at end of file

		@@ -8,8 +8,12 @@
		block discarded – undo
8	8	require_once CALENDAR_ROOT.'Month/Weekdays.php';
9	9	require_once CALENDAR_ROOT.'Decorator/Uri.php';
10	10
11		-if (!isset($_GET['jahr'])) $_GET['jahr'] = date('Y');
12		-if (!isset($_GET['monat'])) $_GET['monat'] = date('m');
	11	+if (!isset($_GET['jahr'])) {
	12	+ $_GET['jahr'] = date('Y');
	13	+}
	14	+if (!isset($_GET['monat'])) {
	15	+ $_GET['monat'] = date('m');
	16	+}
13	17
14	18	// Build the month
15	19	$Calendar = new Calendar_Month_Weekdays($_GET['jahr'], $_GET['monat']);

		@@ -3,8 +3,8 @@ discard block
		block discarded – undo
3	3	* Description: demonstrates a decorator used to "attach a payload" to a selection
4	4	* to make it available when iterating over calendar children
5	5	*/
6		-if ( !@include 'Calendar/Calendar.php' ) {
7		- define('CALENDAR_ROOT','../../');
	6	+if (!@include 'Calendar/Calendar.php') {
	7	+ define('CALENDAR_ROOT', '../../');
8	8	}
9	9	require_once CALENDAR_ROOT.'Day.php';
10	10	require_once CALENDAR_ROOT.'Hour.php';
		@@ -25,7 +25,7 @@ discard block
		block discarded – undo
25	25	}
26	26
27	27	// Create a day to view the hours for
28		-$Day = & new Calendar_Day(2003,10,24);
	28	+$Day = & new Calendar_Day(2003, 10, 24);
29	29
30	30	// A sample query to get the data for today (NOT ACTUALLY USED HERE)
31	31	$sql = "
		@@ -39,18 +39,18 @@ discard block
		block discarded – undo
39	39	eventtime < '".$Day->nextDay(TRUE)."';";
40	40
41	41	// An array simulating data from a database
42		-$result = array (
43		- array('eventtime'=>mktime(9,0,0,10,24,2003),'entry'=>'Meeting with sales team'),
44		- array('eventtime'=>mktime(11,0,0,10,24,2003),'entry'=>'Conference call with Widget Inc.'),
45		- array('eventtime'=>mktime(15,0,0,10,24,2003),'entry'=>'Presentation to board of directors')
	42	+$result = array(
	43	+ array('eventtime'=>mktime(9, 0, 0, 10, 24, 2003), 'entry'=>'Meeting with sales team'),
	44	+ array('eventtime'=>mktime(11, 0, 0, 10, 24, 2003), 'entry'=>'Conference call with Widget Inc.'),
	45	+ array('eventtime'=>mktime(15, 0, 0, 10, 24, 2003), 'entry'=>'Presentation to board of directors')
46	46	);
47	47
48	48	// An array to place selected hours in
49	49	$selection = array();
50	50
51	51	// Loop through the "database result"
52		-foreach ( $result as $row ) {
53		- $Hour = new Calendar_Hour(2000,1,1,1); // Create Hour with dummy values
	52	+foreach ($result as $row) {
	53	+ $Hour = new Calendar_Hour(2000, 1, 1, 1); // Create Hour with dummy values
54	54	$Hour->setTimeStamp($row['eventtime']); // Set the real time with setTimeStamp
55	55
56	56	// Create the decorator, passing it the Hour
		@@ -74,36 +74,36 @@ discard block
		block discarded – undo
74	74	<body>
75	75	<h1>Passing a Selection "Payload" using a Decorator</h1>
76	76	<table>
77		-<caption><b>Your Schedule for <?php echo ( date('D nS F Y',$Day->thisDay(TRUE)) ); ?></b></caption>
	77	+<caption><b>Your Schedule for <?php echo (date('D nS F Y', $Day->thisDay(TRUE))); ?></b></caption>
78	78	<tr>
79	79	<th width="5%">Time</th>
80	80	<th>Entry</th>
81	81	</tr>
82	82	<?php
83		-while ( $Hour = & $Day->fetch() ) {
	83	+while ($Hour = & $Day->fetch()) {
84	84
85	85	$hour = $Hour->thisHour();
86	86	$minute = $Hour->thisMinute();
87	87
88	88	// Office hours only...
89		- if ( $hour >= 8 && $hour <= 18 ) {
90		- echo ( "<tr>\n" );
91		- echo ( "<td>$hour:$minute</td>\n" );
	89	+ if ($hour >= 8 && $hour <= 18) {
	90	+ echo ("<tr>\n");
	91	+ echo ("<td>$hour:$minute</td>\n");
92	92
93	93	// If the hour is selected, call the decorator method...
94		- if ( $Hour->isSelected() ) {
95		- echo ( "<td bgcolor=\"silver\">".$Hour->getEntry()."</td>\n" );
	94	+ if ($Hour->isSelected()) {
	95	+ echo ("<td bgcolor=\"silver\">".$Hour->getEntry()."</td>\n");
96	96	} else {
97		- echo ( "<td> </td>\n" );
	97	+ echo ("<td> </td>\n");
98	98	}
99		- echo ( "</tr>\n" );
	99	+ echo ("</tr>\n");
100	100	}
101	101	}
102	102	?>
103	103	</table>
104	104	<p>The query to fetch this data, with help from PEAR::Calendar, might be;</p>
105	105	<pre>
106		-<?php echo ( $sql ); ?>
	106	+<?php echo ($sql); ?>
107	107	</pre>
108	108	</body>
109	109	</html>
110	110	\ No newline at end of file

		@@ -25,12 +25,12 @@ discard block
		block discarded – undo
25	25	echo "<hr>Creating: new Calendar_Day(date('Y'), date('n'), date('d'));<br />";
26	26	$Calendar = new Calendar_Day(date('Y'), date('n'), date('d'));
27	27
28		-echo '<hr>Previous month is: '.Calendar_Util_Textual::prevMonthName($Calendar,'two').'<br />';
29		-echo 'This month is: '.Calendar_Util_Textual::thisMonthName($Calendar,'short').'<br />';
	28	+echo '<hr>Previous month is: '.Calendar_Util_Textual::prevMonthName($Calendar, 'two').'<br />';
	29	+echo 'This month is: '.Calendar_Util_Textual::thisMonthName($Calendar, 'short').'<br />';
30	30	echo 'Next month is: '.Calendar_Util_Textual::nextMonthName($Calendar).'<br /><hr />';
31	31	echo 'Previous day is: '.Calendar_Util_Textual::prevDayName($Calendar).'<br />';
32		-echo 'This day is: '.Calendar_Util_Textual::thisDayName($Calendar,'short').'<br />';
33		-echo 'Next day is: '.Calendar_Util_Textual::nextDayName($Calendar,'one').'<br /><hr />';
	32	+echo 'This day is: '.Calendar_Util_Textual::thisDayName($Calendar, 'short').'<br />';
	33	+echo 'Next day is: '.Calendar_Util_Textual::nextDayName($Calendar, 'one').'<br /><hr />';
34	34
35	35	echo "Creating: new Calendar_Month_Weekdays(date('Y'), date('n'), 6); - Saturday is first day of week<br />";
36	36	$Calendar = new Calendar_Month_Weekdays(date('Y'), date('n'), 6);
		@@ -41,7 +41,7 @@ discard block
		block discarded – undo
41	41	<caption><?php echo Calendar_Util_Textual::thisMonthName($Calendar).' '.$Calendar->thisYear(); ?></caption>
42	42	<tr>
43	43	<?php
44		-$dayheaders = Calendar_Util_Textual::orderedWeekdays($Calendar,'short');
	44	+$dayheaders = Calendar_Util_Textual::orderedWeekdays($Calendar, 'short');
45	45	foreach ($dayheaders as $dayheader) {
46	46	echo '<th>'.$dayheader.'</th>';
47	47	}

		@@ -346,22 +346,22 @@ discard block
		block discarded – undo
346	346	function prevWeek($format = 'n_in_month')
347	347	{
348	348	switch (strtolower($format)) {
349		- case 'int':
350		- case 'n_in_month':
351		- return ($this->firstWeek) ? null : $this->thisWeek('n_in_month') -1;
352		- case 'n_in_year':
353		- return $this->cE->getWeekNInYear(
354		- $this->cE->stampToYear($this->prevWeek),
355		- $this->cE->stampToMonth($this->prevWeek),
356		- $this->cE->stampToDay($this->prevWeek));
357		- case 'array':
358		- return $this->toArray($this->prevWeek);
359		- case 'object':
360		- include_once CALENDAR_ROOT.'Factory.php';
361		- return Factory::createByTimestamp('Week', $this->prevWeek);
362		- case 'timestamp':
363		- default:
364		- return $this->prevWeek;
	349	+ case 'int':
	350	+ case 'n_in_month':
	351	+ return ($this->firstWeek) ? null : $this->thisWeek('n_in_month') -1;
	352	+ case 'n_in_year':
	353	+ return $this->cE->getWeekNInYear(
	354	+ $this->cE->stampToYear($this->prevWeek),
	355	+ $this->cE->stampToMonth($this->prevWeek),
	356	+ $this->cE->stampToDay($this->prevWeek));
	357	+ case 'array':
	358	+ return $this->toArray($this->prevWeek);
	359	+ case 'object':
	360	+ include_once CALENDAR_ROOT.'Factory.php';
	361	+ return Factory::createByTimestamp('Week', $this->prevWeek);
	362	+ case 'timestamp':
	363	+ default:
	364	+ return $this->prevWeek;
365	365	}
366	366	}
367	367
		@@ -376,35 +376,35 @@ discard block
		block discarded – undo
376	376	function thisWeek($format = 'n_in_month')
377	377	{
378	378	switch (strtolower($format)) {
379		- case 'int':
380		- case 'n_in_month':
381		- if ($this->firstWeek) {
382		- return 1;
383		- }
384		- if ($this->lastWeek) {
385		- return $this->cE->getWeeksInMonth(
	379	+ case 'int':
	380	+ case 'n_in_month':
	381	+ if ($this->firstWeek) {
	382	+ return 1;
	383	+ }
	384	+ if ($this->lastWeek) {
	385	+ return $this->cE->getWeeksInMonth(
	386	+ $this->thisYear(),
	387	+ $this->thisMonth(),
	388	+ $this->firstDay);
	389	+ }
	390	+ return $this->cE->getWeekNInMonth(
386	391	$this->thisYear(),
387	392	$this->thisMonth(),
	393	+ $this->thisDay(),
388	394	$this->firstDay);
389		- }
390		- return $this->cE->getWeekNInMonth(
391		- $this->thisYear(),
392		- $this->thisMonth(),
393		- $this->thisDay(),
394		- $this->firstDay);
395		- case 'n_in_year':
396		- return $this->cE->getWeekNInYear(
397		- $this->cE->stampToYear($this->thisWeek),
398		- $this->cE->stampToMonth($this->thisWeek),
399		- $this->cE->stampToDay($this->thisWeek));
400		- case 'array':
401		- return $this->toArray($this->thisWeek);
402		- case 'object':
403		- include_once CALENDAR_ROOT.'Factory.php';
404		- return Factory::createByTimestamp('Week', $this->thisWeek);
405		- case 'timestamp':
406		- default:
407		- return $this->thisWeek;
	395	+ case 'n_in_year':
	396	+ return $this->cE->getWeekNInYear(
	397	+ $this->cE->stampToYear($this->thisWeek),
	398	+ $this->cE->stampToMonth($this->thisWeek),
	399	+ $this->cE->stampToDay($this->thisWeek));
	400	+ case 'array':
	401	+ return $this->toArray($this->thisWeek);
	402	+ case 'object':
	403	+ include_once CALENDAR_ROOT.'Factory.php';
	404	+ return Factory::createByTimestamp('Week', $this->thisWeek);
	405	+ case 'timestamp':
	406	+ default:
	407	+ return $this->thisWeek;
408	408	}
409	409	}
410	410
		@@ -419,22 +419,22 @@ discard block
		block discarded – undo
419	419	function nextWeek($format = 'n_in_month')
420	420	{
421	421	switch (strtolower($format)) {
422		- case 'int':
423		- case 'n_in_month':
424		- return ($this->lastWeek) ? null : $this->thisWeek('n_in_month') +1;
425		- case 'n_in_year':
426		- return $this->cE->getWeekNInYear(
427		- $this->cE->stampToYear($this->nextWeek),
428		- $this->cE->stampToMonth($this->nextWeek),
429		- $this->cE->stampToDay($this->nextWeek));
430		- case 'array':
431		- return $this->toArray($this->nextWeek);
432		- case 'object':
433		- include_once CALENDAR_ROOT.'Factory.php';
434		- return Factory::createByTimestamp('Week', $this->nextWeek);
435		- case 'timestamp':
436		- default:
437		- return $this->nextWeek;
	422	+ case 'int':
	423	+ case 'n_in_month':
	424	+ return ($this->lastWeek) ? null : $this->thisWeek('n_in_month') +1;
	425	+ case 'n_in_year':
	426	+ return $this->cE->getWeekNInYear(
	427	+ $this->cE->stampToYear($this->nextWeek),
	428	+ $this->cE->stampToMonth($this->nextWeek),
	429	+ $this->cE->stampToDay($this->nextWeek));
	430	+ case 'array':
	431	+ return $this->toArray($this->nextWeek);
	432	+ case 'object':
	433	+ include_once CALENDAR_ROOT.'Factory.php';
	434	+ return Factory::createByTimestamp('Week', $this->nextWeek);
	435	+ case 'timestamp':
	436	+ default:
	437	+ return $this->nextWeek;
438	438	}
439	439	}
440	440

		@@ -98,11 +98,11 @@
		block discarded – undo
98	98	{
99	99	include_once CALENDAR_ROOT.'Minute.php';
100	100	$mIH = $this->cE->getMinutesInHour($this->year, $this->month, $this->day,
101		- $this->hour);
	101	+ $this->hour);
102	102	for ($i=0; $i < $mIH; $i++) {
103	103	$this->children[$i] =
104	104	new Minute($this->year, $this->month, $this->day,
105		- $this->hour, $i);
	105	+ $this->hour, $i);
106	106	}
107	107	if (count($sDates) > 0) {
108	108	$this->setSelection($sDates);

		@@ -99,7 +99,7 @@
		block discarded – undo
99	99	include_once CALENDAR_ROOT.'Minute.php';
100	100	$mIH = $this->cE->getMinutesInHour($this->year, $this->month, $this->day,
101	101	$this->hour);
102		- for ($i=0; $i < $mIH; $i++) {
	102	+ for ($i = 0; $i < $mIH; $i++) {
103	103	$this->children[$i] =
104	104	new Minute($this->year, $this->month, $this->day,
105	105	$this->hour, $i);

		@@ -77,7 +77,7 @@ discard block
		block discarded – undo
77	77	function stampCollection($stamp)
78	78	{
79	79	static $stamps = array();
80		- if ( !isset($stamps[$stamp]) ) {
	80	+ if (!isset($stamps[$stamp])) {
81	81	$date = @date('Y n j H i s t W w', $stamp);
82	82	$stamps[$stamp] = sscanf($date, "%d %d %d %d %d %d %d %d %d");
83	83	}
		@@ -181,7 +181,7 @@ discard block
		block discarded – undo
181	181	* @return int Unix timestamp
182	182	* @access protected
183	183	*/
184		- function dateToStamp($y, $m, $d, $h=0, $i=0, $s=0)
	184	+ function dateToStamp($y, $m, $d, $h = 0, $i = 0, $s = 0)
185	185	{
186	186	static $dates = array();
187	187	if (!isset($dates[$y][$m][$d][$h][$i][$s])) {
		@@ -220,7 +220,7 @@ discard block
		block discarded – undo
220	220	* @return int (12)
221	221	* @access protected
222	222	*/
223		- function getMonthsInYear($y=null)
	223	+ function getMonthsInYear($y = null)
224	224	{
225	225	return 12;
226	226	}
		@@ -268,7 +268,7 @@ discard block
		block discarded – undo
268	268	* @return int (7)
269	269	* @access protected
270	270	*/
271		- function getDaysInWeek($y=null, $m=null, $d=null)
	271	+ function getDaysInWeek($y = null, $m = null, $d = null)
272	272	{
273	273	return 7;
274	274	}
		@@ -301,9 +301,9 @@ discard block
		block discarded – undo
301	301	* @return int week number
302	302	* @access protected
303	303	*/
304		- function getWeekNInMonth($y, $m, $d, $firstDay=1)
	304	+ function getWeekNInMonth($y, $m, $d, $firstDay = 1)
305	305	{
306		- $weekEnd = (0 == $firstDay) ? $this->getDaysInWeek()-1 : $firstDay-1;
	306	+ $weekEnd = (0 == $firstDay) ? $this->getDaysInWeek() - 1 : $firstDay - 1;
307	307	$end_of_week = 1;
308	308	while (@date('w', @mktime(0, 0, 0, $m, $end_of_week, $y)) != $weekEnd) {
309	309	++$end_of_week; //find first weekend of the month
		@@ -371,7 +371,7 @@ discard block
		block discarded – undo
371	371	* @return array (0,1,2,3,4,5,6) 1 = Monday
372	372	* @access protected
373	373	*/
374		- function getWeekDays($y=null, $m=null, $d=null)
	374	+ function getWeekDays($y = null, $m = null, $d = null)
375	375	{
376	376	return array(0, 1, 2, 3, 4, 5, 6);
377	377	}
		@@ -386,7 +386,7 @@ discard block
		block discarded – undo
386	386	* @return int (default 1 = Monday)
387	387	* @access protected
388	388	*/
389		- function getFirstDayOfWeek($y=null, $m=null, $d=null)
	389	+ function getFirstDayOfWeek($y = null, $m = null, $d = null)
390	390	{
391	391	return 1;
392	392	}
		@@ -401,7 +401,7 @@ discard block
		block discarded – undo
401	401	* @return int (24)
402	402	* @access protected
403	403	*/
404		- function getHoursInDay($y=null, $m=null, $d=null)
	404	+ function getHoursInDay($y = null, $m = null, $d = null)
405	405	{
406	406	return 24;
407	407	}
		@@ -417,7 +417,7 @@ discard block
		block discarded – undo
417	417	* @return int (60)
418	418	* @access protected
419	419	*/
420		- function getMinutesInHour($y=null, $m=null, $d=null, $h=null)
	420	+ function getMinutesInHour($y = null, $m = null, $d = null, $h = null)
421	421	{
422	422	return 60;
423	423	}
		@@ -434,7 +434,7 @@ discard block
		block discarded – undo
434	434	* @return int (60)
435	435	* @access protected
436	436	*/
437		- function getSecondsInMinute($y=null, $m=null, $d=null, $h=null, $i=null)
	437	+ function getSecondsInMinute($y = null, $m = null, $d = null, $h = null, $i = null)
438	438	{
439	439	return 60;
440	440	}
		@@ -455,7 +455,7 @@ discard block
		block discarded – undo
455	455	$today = sscanf($today_date, '%d %d %d');
456	456	}
457	457	$date = UnixTS::stampCollection($stamp);
458		- return ( $date[2] == $today[2]
	458	+ return ($date[2] == $today[2]
459	459	&& $date[1] == $today[1]
460	460	&& $date[0] == $today[0]
461	461	);

		@@ -167,7 +167,7 @@ discard block
		block discarded – undo
167	167	* @return string iso-8601 datetime
168	168	* @access protected
169	169	*/
170		- function dateToStamp($y, $m, $d, $h=0, $i=0, $s=0)
	170	+ function dateToStamp($y, $m, $d, $h = 0, $i = 0, $s = 0)
171	171	{
172	172	$r = array();
173	173	PearDate::adjustDate($y, $m, $d, $h, $i, $s);
		@@ -218,8 +218,8 @@ discard block
		block discarded – undo
218	218	$d += floor($h / 24);
219	219	$h %= 24;
220	220	}
221		- for(; $m < 1; $y--, $m+=12);
222		- for(; $m > 12; $y++, $m-=12);
	221	+ for (; $m < 1; $y--, $m += 12);
	222	+ for (; $m > 12; $y++, $m -= 12);
223	223
224	224	while ($d < 1) {
225	225	if ($m > 1) {
		@@ -230,7 +230,7 @@ discard block
		block discarded – undo
230	230	}
231	231	$d += Date_Calc::daysInMonth($m, $y);
232	232	}
233		- for ($max_days = Date_Calc::daysInMonth($m, $y); $d > $max_days; ) {
	233	+ for ($max_days = Date_Calc::daysInMonth($m, $y); $d > $max_days;) {
234	234	$d -= $max_days;
235	235	if ($m < 12) {
236	236	$m++;
		@@ -271,7 +271,7 @@ discard block
		block discarded – undo
271	271	* @return int (12)
272	272	* @access protected
273	273	*/
274		- function getMonthsInYear($y=null)
	274	+ function getMonthsInYear($y = null)
275	275	{
276	276	return 12;
277	277	}
		@@ -315,7 +315,7 @@ discard block
		block discarded – undo
315	315	* @return int (7)
316	316	* @access protected
317	317	*/
318		- function getDaysInWeek($y=null, $m=null, $d=null)
	318	+ function getDaysInWeek($y = null, $m = null, $d = null)
319	319	{
320	320	return 7;
321	321	}
		@@ -348,9 +348,9 @@ discard block
		block discarded – undo
348	348	* @return int week number
349	349	* @access protected
350	350	*/
351		- function getWeekNInMonth($y, $m, $d, $firstDay=1)
	351	+ function getWeekNInMonth($y, $m, $d, $firstDay = 1)
352	352	{
353		- $weekEnd = ($firstDay == 0) ? $this->getDaysInWeek()-1 : $firstDay-1;
	353	+ $weekEnd = ($firstDay == 0) ? $this->getDaysInWeek() - 1 : $firstDay - 1;
354	354	$end_of_week = (int)Date_Calc::nextDayOfWeek($weekEnd, 1, $m, $y, '%e', true);
355	355	$w = 1;
356	356	while ($d > $end_of_week) {
		@@ -370,7 +370,7 @@ discard block
		block discarded – undo
370	370	* @return int weeks number
371	371	* @access protected
372	372	*/
373		- function getWeeksInMonth($y, $m, $firstDay=1)
	373	+ function getWeeksInMonth($y, $m, $firstDay = 1)
374	374	{
375	375	$FDOM = Date_Calc::firstOfMonthWeekday($m, $y);
376	376	if ($FDOM == 0) {
		@@ -413,7 +413,7 @@ discard block
		block discarded – undo
413	413	* @return array (0, 1, 2, 3, 4, 5, 6) 1 = Monday
414	414	* @access protected
415	415	*/
416		- function getWeekDays($y=null, $m=null, $d=null)
	416	+ function getWeekDays($y = null, $m = null, $d = null)
417	417	{
418	418	return array(0, 1, 2, 3, 4, 5, 6);
419	419	}
		@@ -428,7 +428,7 @@ discard block
		block discarded – undo
428	428	* @return int (default 1 = Monday)
429	429	* @access protected
430	430	*/
431		- function getFirstDayOfWeek($y=null, $m=null, $d=null)
	431	+ function getFirstDayOfWeek($y = null, $m = null, $d = null)
432	432	{
433	433	return 1;
434	434	}
		@@ -443,7 +443,7 @@ discard block
		block discarded – undo
443	443	* @return int (24)
444	444	* @access protected
445	445	*/
446		- function getHoursInDay($y=null,$m=null,$d=null)
	446	+ function getHoursInDay($y = null, $m = null, $d = null)
447	447	{
448	448	return 24;
449	449	}
		@@ -459,7 +459,7 @@ discard block
		block discarded – undo
459	459	* @return int (60)
460	460	* @access protected
461	461	*/
462		- function getMinutesInHour($y=null,$m=null,$d=null,$h=null)
	462	+ function getMinutesInHour($y = null, $m = null, $d = null, $h = null)
463	463	{
464	464	return 60;
465	465	}
		@@ -476,7 +476,7 @@ discard block
		block discarded – undo
476	476	* @return int (60)
477	477	* @access protected
478	478	*/
479		- function getSecondsInMinute($y=null,$m=null,$d=null,$h=null,$i=null)
	479	+ function getSecondsInMinute($y = null, $m = null, $d = null, $h = null, $i = null)
480	480	{
481	481	return 60;
482	482	}
		@@ -496,7 +496,7 @@ discard block
		block discarded – undo
496	496	$today = new Date();
497	497	}
498	498	$date = PearDate::stampCollection($stamp);
499		- return ( $date->day == $today->getDay()
	499	+ return ($date->day == $today->getDay()
500	500	&& $date->month == $today->getMonth()
501	501	&& $date->year == $today->getYear()
502	502	);

FriendsOfPear / Calendar

Pull Request — master (#2)

Status

Category

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

Braces +6 added lines, -2 removed lines patch added patch discarded remove patch

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

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

Switch Indentation +58 added lines, -58 removed lines patch added patch discarded remove patch

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

Indentation +2 added lines, -2 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 +12 added lines, -12 removed lines patch added patch discarded remove patch

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

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