Roomify / bat

src/Calendar/CalendarResponse.php

Braces +1 added lines, -2 removed lines

@@ -126,8 +126,7 @@
       unset($this->included_set[$unit->getUnitId()]);
       $this->addMiss($unit, $reason, $constraint);
       return TRUE;
-    }
-    else {
+    } else {
       return FALSE;
     }
   }

src/Store/DrupalDBStore.php

Braces +2 added lines, -4 removed lines

@@ -110,8 +110,7 @@ 
       for ($i = 0; $i<=59; $i++) {
         if ($i <= 9) {
           $index = 'm0'.$i;
-        }
-        else {
+        } else {
           $index = 'm'.$i;
         }
         $db_events[$data['unit_id']][Event::BAT_MINUTE][$data['year']][$data['month']][$data['day']][$data['hour']][$index] = $data[$index];
@@ -190,8 +189,7 @@ 
           }
         }
       }
-    }
-    catch (\Exception $e) {
+    } catch (\Exception $e) {
       $saved = FALSE;
       $transaction->rollback();
       watchdog_exception('BAT Event Save Exception', $e);

src/Constraint/MinMaxDaysConstraint.php

Braces +6 added lines, -12 removed lines

@@ -70,8 +70,7 @@ 
             $calendar_response->removeFromMatched($included_set[$unit_id]['unit'], CalendarResponse::CONSTRAINT, $this);
 
             $this->affected_units[$unit_id] = $included_set[$unit_id]['unit'];
-          }
-          elseif (is_numeric($this->max_days) && $diff > $this->max_days) {
+          } elseif (is_numeric($this->max_days) && $diff > $this->max_days) {
             $calendar_response->removeFromMatched($included_set[$unit_id]['unit'], CalendarResponse::CONSTRAINT, $this);
 
             $this->affected_units[$unit_id] = $included_set[$unit_id]['unit'];
@@ -122,8 +121,7 @@ 
     if ($day_of_the_week) {
       if ($start_date && $end_date) {
         $text = t('From @start_date to @end_date, if booking starts on @day_of_the_week', $args);
-      }
-      else {
+      } else {
         $text = t('If booking starts on @day_of_the_week', $args);
       }
     }
@@ -132,8 +130,7 @@ 
     if ($minimum_stay || $maximum_stay) {
       if (empty($text)) {
         $text = t('The stay') . ' ';
-      }
-      else {
+      } else {
         $text .=   ' ' . t('the stay') . ' ';
       }
     }
@@ -141,15 +138,12 @@ 
       // Special case when min stay and max stay are the same.
       if ($minimum_stay == $maximum_stay) {
         $text .= t('must be for @minimum_stay', $args);
-      }
-      else {
+      } else {
         $text .= t('must be at least @minimum_stay and at most @maximum_stay', $args);
       }
-    }
-    elseif ($minimum_stay) {
+    } elseif ($minimum_stay) {
       $text .= t('must be for at least @minimum_stay', $args);
-    }
-    elseif ($maximum_stay) {
+    } elseif ($maximum_stay) {
       $text .= t('cannot be more than @maximum_stay', $args);
     }
 

src/Store/SqlLiteDBStore.php

Braces +2 added lines, -4 removed lines

@@ -110,8 +110,7 @@ 
       for ($i = 0; $i<=59; $i++) {
         if ($i <= 9) {
           $index = 'm0'.$i;
-        }
-        else {
+        } else {
           $index = 'm'.$i;
         }
         $db_events[$data['unit_id']][Event::BAT_MINUTE][$data['year']][$data['month']][$data['day']][$data['hour']][$index] = $data[$index];
@@ -190,8 +189,7 @@ 
           }
         }
       }
-    }
-    catch (\Exception $e) {
+    } catch (\Exception $e) {
       $saved = FALSE;
       $transaction->rollback();
       watchdog_exception('BAT Event Save Exception', $e);

src/Constraint/CheckInDayConstraint.php

Braces +1 added lines, -2 removed lines

@@ -102,8 +102,7 @@
     if ($day_of_the_week) {
       if ($start_date && $end_date) {
         $text = 'From @start_date to @end_date, if booking starts on @day_of_the_week';
-      }
-      else {
+      } else {
         $text = 'If booking starts on @day_of_the_week';
       }
 

src/Calendar/AbstractCalendar.php

Braces +11 added lines, -22 removed lines

@@ -128,8 +128,7 @@ 
         // Unit is in a state that is within the set of valid states so add to result set
         $units[$unit] = $unit;
         $response->addMatch($keyed_units[$unit], CalendarResponse::VALID_STATE);
-      }
-      else {
+      } else {
         $response->addMiss($keyed_units[$unit], CalendarResponse::INVALID_STATE);
       }
 
@@ -223,8 +222,7 @@ 
           foreach ($days as $day => $value) {
             $result[$year][$month][$day] = ((int)$db_events[$unit][Event::BAT_DAY][$year][$month][$day] == 0 ? $keyed_units[$unit]->getDefaultValue() : (int)$db_events[$unit][Event::BAT_DAY][$year][$month][$day]);
           }
-        }
-        else {
+        } else {
           foreach ($days as $day => $value) {
             $result[$year][$month][$day] = $keyed_units[$unit]->getDefaultValue();
           }
@@ -254,8 +252,7 @@ 
           foreach ($hours as $hour => $value) {
             if (isset($db_events[$unit][Event::BAT_HOUR][$year][$month][$day][$hour])) {
               $result[$year][$month][$day][$hour] = ((int) $db_events[$unit][Event::BAT_HOUR][$year][$month][$day][$hour] == 0 ? $keyed_units[$unit]->getDefaultValue() : (int) $db_events[$unit][Event::BAT_HOUR][$year][$month][$day][$hour]);
-            }
-            else {
+            } else {
               // If nothing from db - then revert to the defaults
               $result[$year][$month][$day][$hour] = (int) $keyed_units[$unit]->getDefaultValue();
             }
@@ -301,8 +298,7 @@ 
             foreach ($minutes as $minute => $value) {
               if (isset($db_events[$unit][Event::BAT_MINUTE][$year][$month][$day][$hour][$minute])) {
                 $result[$year][$month][$day][$hour][$minute] = ((int) $db_events[$unit][Event::BAT_MINUTE][$year][$month][$day][$hour][$minute] == 0 ? $keyed_units[$unit]->getDefaultValue() : (int) $db_events[$unit][Event::BAT_MINUTE][$year][$month][$day][$hour][$minute]);
-              }
-              else {
+              } else {
                 // If nothing from db - then revert to the defaults
                 $result[$year][$month][$day][$hour][$minute] = (int) $keyed_units[$unit]->getDefaultValue();
               }
@@ -377,8 +373,7 @@ 
                     if ($current_value === $minute_value) {
                       // We are still in minutes and going through so add a minute
                       $end_event->add(new \DateInterval('PT1M'));
-                    }
-                    elseif (($current_value != $minute_value) && ($current_value !== NULL)) {
+                    } elseif (($current_value != $minute_value) && ($current_value !== NULL)) {
                       // Value just switched - let us wrap up with current event and start a new one
                       $normalized_events[$unit_id][] = new Event($start_event, $end_event, $this->getUnit($unit_id), $current_value);
                       $start_event = clone($end_event->add(new \DateInterval('PT1M')));
@@ -392,12 +387,10 @@ 
                     }
                     $current_value = $minute_value;
                   }
-                }
-                elseif ($current_value === $hour_value) {
+                } elseif ($current_value === $hour_value) {
                   // We are in hours and can add something
                   $end_event->add(new \DateInterval('PT1H'));
-                }
-                elseif (($current_value != $hour_value) && ($current_value !== NULL)) {
+                } elseif (($current_value != $hour_value) && ($current_value !== NULL)) {
                   // Value just switched - let us wrap up with current event and start a new one
                   $normalized_events[$unit_id][] = new Event($start_event, $end_event, $this->getUnit($unit_id), $current_value);
                   // Start event becomes the end event with a minute added
@@ -414,12 +407,10 @@ 
                   $current_value = $hour_value;
                 }
               }
-            }
-            elseif ($current_value === $value) {
+            } elseif ($current_value === $value) {
               // We are adding a whole day so the end event gets moved to the end of the day we are adding
               $end_event = new \DateTime($year . '-' . $month . '-' . substr($day, 1) . ' ' . '23:59');
-            }
-            elseif (($current_value !== $value) && ($current_value !== NULL)) {
+            } elseif (($current_value !== $value) && ($current_value !== NULL)) {
               // Value just switched - let us wrap up with current event and start a new one
               $normalized_events[$unit_id][] = new Event($start_event, $end_event, $this->getUnit($unit_id), $current_value);
               // Start event becomes the end event with a minute added
@@ -454,8 +445,7 @@ 
           if ($event->endsLater($end_date)) {
             $event->setEndDate($end_date);
           }
-        }
-        else {
+        } else {
           // Event completely not in range so unset it
           unset($normalized_events[$unit_id][$key]);
         }
@@ -486,8 +476,7 @@ 
           $e++;
           $flipped[$e][$value][$datum] = $datum;
           $old_value = $value;
-        }
-        else {
+        } else {
           $flipped[$e][$value][$datum] = $datum;
         }
       }

src/Event/AbstractEvent.php

Doc Comments +6 added lines, -5 removed lines

@@ -90,7 +90,7 @@ 
   /**
    * Returns the start date.
    *
-   * @return DateTime
+   * @return \DateTime
    */
   public function getStartDate() {
     return clone($this->start_date);
@@ -107,7 +107,7 @@ 
   /**
    * Set the start date.
    *
-   * @param DateTime $start_date
+   * @param \DateTime $start_date
    */
   public function setStartDate(\DateTime $start_date) {
     $this->start_date = clone($start_date);
@@ -116,7 +116,7 @@ 
   /**
    * Returns the end date.
    *
-   * @return DateTime
+   * @return \DateTime
    */
   public function getEndDate() {
     return clone($this->end_date);
@@ -133,7 +133,7 @@ 
   /**
    * Set the end date.
    *
-   * @param DateTime $end_date
+   * @param \DateTime $end_date
    */
   public function setEndDate(\DateTime $end_date) {
     $this->end_date = clone($end_date);
@@ -451,6 +451,7 @@ 
   /**
    * Transforms the event in a breakdown of days, hours and minutes with associated states.
    *
+   * @param EventItemizer $itemizer
    * @return array
    */
   public function itemize($itemizer, $granularity = AbstractEvent::BAT_HOURLY) {
@@ -461,7 +462,7 @@ 
   /**
    * Saves an event using the Store object
    *
-   * @param \Roomify\Bat\\Store\Store $store
+   * @param Store $store
    * @param string $granularity
    *
    * @return boolean

Braces +1 added lines, -2 removed lines

@@ -342,8 +342,7 @@
     if ($this->dateIsEarlier($start) &&
       ($this->dateIsInRange($end) || $this->dateIsLater($end))) {
       $overlaps = TRUE;
-    }
-    elseif ($this->dateIsInRange($start) &&
+    } elseif ($this->dateIsInRange($start) &&
       ($this->dateIsInRange($end) || $this->dateIsLater($end))) {
       $overlaps = TRUE;
     }

src/Event/EventItemizer.php

Braces +4 added lines, -8 removed lines

@@ -147,8 +147,7 @@ 
           // If we are in the same month the end date is the end date of the event
           if ($this->event->isSameMonth()) {
             $dayrange = new \DatePeriod($this->event->getStartDate(), $dayinterval, new \DateTime($this->event->getEndDate()->format("Y-n-j 23:59:59")));
-          }
-          else { // alternatively it is the last day of the start month
+          } else { // alternatively it is the last day of the start month
             $dayrange = new \DatePeriod($this->event->getStartDate(), $dayinterval, $this->event->endMonthDate($this->event->getStartDate()));
           }
           foreach ($dayrange as $day) {
@@ -214,8 +213,7 @@ 
         $itemized[EventItemizer::BAT_HOUR][$sy][$sm]['d' . $sd] = $itemized_same_day[EventItemizer::BAT_HOUR][$sy][$sm]['d' . $sd];
         $itemized[EventItemizer::BAT_MINUTE][$sy][$sm]['d' . $sd] = $itemized_same_day[EventItemizer::BAT_MINUTE][$sy][$sm]['d' . $sd];
       }
-    }
-    else {
+    } else {
       // Deal with the start day unless it starts on midnight precisely at which point the whole day is booked
       if (!($this->event->getStartDate()->format('H:i') == '00:00')) {
         $start_period = new \DatePeriod($start_date, $interval, new \DateTime($start_date->format("Y-n-j 23:59:59")));
@@ -223,8 +221,7 @@ 
         $itemized[EventItemizer::BAT_DAY][$sy][$sm]['d' . $sd] = -1;
         $itemized[EventItemizer::BAT_HOUR][$sy][$sm]['d' . $sd] = $itemized_start[EventItemizer::BAT_HOUR][$sy][$sm]['d' . $sd];
         $itemized[EventItemizer::BAT_MINUTE][$sy][$sm]['d' . $sd] = $itemized_start[EventItemizer::BAT_MINUTE][$sy][$sm]['d' . $sd];
-      }
-      else {
+      } else {
         // Just set an empty hour and minute
         $itemized[EventItemizer::BAT_HOUR][$sy][$sm]['d' . $sd] = array();
         $itemized[EventItemizer::BAT_MINUTE][$sy][$sm]['d' . $sd] = array();
@@ -270,8 +267,7 @@ 
         // Not a real hour - leave as is and move on
         $counter = 0;
         $start_minute = 0;
-      }
-      elseif ($counter == 60 && $start_minute == 0) {
+      } elseif ($counter == 60 && $start_minute == 0) {
         // Did a real whole hour so initialize the hour
         $itemized[EventItemizer::BAT_HOUR][$minute->format('Y')][$minute->format('n')]['d' . $minute->format('j')]['h' . $minute->format('G')] = $this->event->getValue();
         // We have a whole hour so get rid of the minute info

src/Event/EventInterface.php

Doc Comments +6 added lines, -3 removed lines

@@ -25,6 +25,7 @@ 
    * Set the unit id.
    *
    * @param int $unit_id
+   * @return void
    */
   public function setUnitId($unit_id);
 
@@ -46,6 +47,7 @@ 
    * Sets the StartDate
    *
    * @param \DateTime
+   * @return void
    */
   public function setStartDate(\DateTime $start_date);
 
@@ -53,6 +55,7 @@ 
    * Sets the EndDate
    *
    * @param \DateTime
+   * @return void
    */
   public function setEndDate(\DateTime $end_date);
 
@@ -67,6 +70,7 @@ 
    * Sets the event value.
    *
    * @param int $value
+   * @return void
    */
   public function setValue($value);
 
@@ -215,7 +219,7 @@ 
 
   /**
    * Returns TRUE if the date supplied is the first month of the event
-   * @param $date
+   * @param \DateTime $date
    *
    * @return bool
    */
@@ -231,7 +235,7 @@ 
 
   /**
    * Returns TRUE if the date supplied is the last month of the event
-   * @param $date
+   * @param \DateTime $date
    *
    * @return bool
    */
@@ -283,7 +287,6 @@ 
   /**
    * Returns the json version of this event.
    *
-   * @param \Roomify\Bat\Event\EventInterface $event
    *
    * @return mixed
    */