smarcet / CalDAVClient

src/Facade/Requests/CalendarCreateRequest.php

Spacing +3 added lines, -5 removed lines

@@ -59,17 +59,15 @@
                     'attributes'  => ['name' => 'VEVENT']
                 ]
             ],
-            '{urn:ietf:params:xml:ns:caldav}calendar-timezone' => new Cdata
-            (
-                ICalTimeZoneBuilder::build
-                (
+            '{urn:ietf:params:xml:ns:caldav}calendar-timezone' => new Cdata(
+                ICalTimeZoneBuilder::build(
                     $this->vo->getTimezone(),
                     $this->vo->getDisplayName()
                 )->render()
             )
         ];
 
-        if(!empty($this->vo->getDescription()))
+        if (!empty($this->vo->getDescription()))
             $props['{urn:ietf:params:xml:ns:caldav}calendar-description'] = $this->vo->getDescription();
 
         return $service->write('{urn:ietf:params:xml:ns:caldav}mkcalendar',

Braces +3 added lines, -2 removed lines

@@ -69,8 +69,9 @@
             )
         ];
 
-        if(!empty($this->vo->getDescription()))
-            $props['{urn:ietf:params:xml:ns:caldav}calendar-description'] = $this->vo->getDescription();
+        if(!empty($this->vo->getDescription())) {
+                    $props['{urn:ietf:params:xml:ns:caldav}calendar-description'] = $this->vo->getDescription();
+        }
 
         return $service->write('{urn:ietf:params:xml:ns:caldav}mkcalendar',
             [

src/Facade/Requests/MakeCalendarRequestVO.php

Spacing +1 added lines, -1 removed lines

@@ -54,7 +54,7 @@
         $this->description   = $description;
         $this->timezone      = $timezone;
 
-        if(is_null($this->timezone)){
+        if (is_null($this->timezone)) {
             $this->timezone = new DateTimeZone('UTC');
         }
     }

src/Facade/Utils/ICalTimeZoneBuilder.php

Spacing +26 added lines, -31 removed lines

@@ -31,11 +31,11 @@ 
      * @param DateTimeZone $time_zone
      * @return array
      */
-    private static function calculateTimeRangeForTransitions(DateTimeZone $time_zone){
+    private static function calculateTimeRangeForTransitions(DateTimeZone $time_zone) {
 
         $now           = new  DateTime('now', $time_zone);
         $year          = $now->format('Y');
-        return [new DateTime('1/1/'.$year, $time_zone),  new DateTime('1/1/'.($year + 1), $time_zone)];
+        return [new DateTime('1/1/'.$year, $time_zone), new DateTime('1/1/'.($year + 1), $time_zone)];
     }
 
     /**
@@ -43,11 +43,11 @@ 
      * @param int $former_offset
      * @return DateTime
      */
-    private static function convertStartDateFromUTC2Local(array $trans, $former_offset){
+    private static function convertStartDateFromUTC2Local(array $trans, $former_offset) {
         $dt     = new DateTime($trans['time'], new DateTimeZone('UTC'));
         $hours  = abs($former_offset);
         // START TIME IS ON UTC and should be converted to local using former offset
-        if($former_offset >= 0 )
+        if ($former_offset >= 0)
             $dt->add(new DateInterval("PT{$hours}H"));
         else
             $dt->sub(new DateInterval("PT{$hours}H"));
@@ -59,12 +59,11 @@ 
      * @param DateTime $dt
      * @return RecurrenceRule
      */
-    private static function calculateRecurrenceRule(DateTime $dt){
-        $r_rule        = new RecurrenceRule();
+    private static function calculateRecurrenceRule(DateTime $dt) {
+        $r_rule = new RecurrenceRule();
         $r_rule->setFreq(RecurrenceRule::FREQ_YEARLY);
         $r_rule->setByMonth(intval($dt->format('m')));
-        $r_rule->setByDay
-        (
+        $r_rule->setByDay(
             self::translate2ByDay($dt)
         );
         return $r_rule;
@@ -74,7 +73,7 @@ 
      * @param $offset
      * @return string
      */
-    private static function calculateOffsetFrom($offset){
+    private static function calculateOffsetFrom($offset) {
         return sprintf('%s%02d%02d', $offset >= 0 ? '+' : '-', abs($offset), abs(($offset - floor($offset)) * 60));
     }
 
@@ -82,7 +81,7 @@ 
      * @param  $offset
      * @return string
      */
-    private static function calculateOffsetTo($offset){
+    private static function calculateOffsetTo($offset) {
         return sprintf('%s%02d%02d', $offset >= 0 ? '+' : '-', abs($offset), abs(($offset - floor($offset)) * 60));
     }
     /**
@@ -91,7 +90,7 @@ 
      * @param bool $with_calendar_envelope
      * @return Calendar|Timezone
      */
-    public static function build(DateTimeZone $time_zone, $calendar_prod_id, $with_calendar_envelope = true){
+    public static function build(DateTimeZone $time_zone, $calendar_prod_id, $with_calendar_envelope = true) {
 
         // get all transitions for one current year and next
         list($start_range, $end_range) = self::calculateTimeRangeForTransitions($time_zone);
@@ -111,11 +110,11 @@ 
 
             // daylight saving time definition
             if ($trans['isdst']) {
-                $current_time_zone_rule = new TimezoneRule(TimezoneRule::TYPE_DAYLIGHT);;
+                $current_time_zone_rule = new TimezoneRule(TimezoneRule::TYPE_DAYLIGHT); ;
             }
             // standard time definition
             else {
-                $current_time_zone_rule = new TimezoneRule(TimezoneRule::TYPE_STANDARD);;
+                $current_time_zone_rule = new TimezoneRule(TimezoneRule::TYPE_STANDARD); ;
             }
 
             if ($current_time_zone_rule) {
@@ -136,7 +135,7 @@ 
             }
 
         }
-        if($with_calendar_envelope) {
+        if ($with_calendar_envelope) {
             $vCalendar = new Calendar(sprintf("'-//%s//EN'", $calendar_prod_id));
             $vCalendar->setTimezone($vTimezone);
             return $vCalendar;
@@ -156,25 +155,25 @@ 
      * @param DateTime $dt)
      * @return string
      */
-    private static function translate2ByDay(DateTime $dt){
-        $day_name = substr(strtoupper($dt->format('D')), 0,2);
+    private static function translate2ByDay(DateTime $dt) {
+        $day_name = substr(strtoupper($dt->format('D')), 0, 2);
         $ordinals = ['first', 'second', 'third', 'fourth', 'last'];
         $day_nbr  = 0;
         $is_last  = false;
 
-        foreach($ordinals as $idx => $ord){
+        foreach ($ordinals as $idx => $ord) {
             $dt_n = self::buildOrdinalDateTime($ord, $dt);
-            if($dt_n->format('Y-m-d') == $dt->format('Y-m-d')){
-                $day_nbr = $idx  + 1;
-                if($ord == 'last'){
+            if ($dt_n->format('Y-m-d') == $dt->format('Y-m-d')) {
+                $day_nbr = $idx + 1;
+                if ($ord == 'last') {
                     $is_last = true;
-                    $day_nbr  = 1;
+                    $day_nbr = 1;
                 }
                 break;
             }
         }
 
-        return sprintf('%s%s%s', $is_last? '-':'', $day_nbr, $day_name);
+        return sprintf('%s%s%s', $is_last ? '-' : '', $day_nbr, $day_name);
     }
 
     /**
@@ -182,15 +181,11 @@ 
      * @param DateTime $dt
      * @return DateTime
      */
-    private static function buildOrdinalDateTime($ord, DateTime $dt){
-        return new DateTime
-        (
-            date
-            ("Y-m-d",
-                strtotime
-                (
-                    sprintf
-                    (
+    private static function buildOrdinalDateTime($ord, DateTime $dt) {
+        return new DateTime(
+            date("Y-m-d",
+                strtotime(
+                    sprintf(
                         self::GetOrdinalDayQuery,
                         $ord,
                         $dt->format('l'),

Braces +5 added lines, -4 removed lines

@@ -47,10 +47,11 @@
         $dt     = new DateTime($trans['time'], new DateTimeZone('UTC'));
         $hours  = abs($former_offset);
         // START TIME IS ON UTC and should be converted to local using former offset
-        if($former_offset >= 0 )
-            $dt->add(new DateInterval("PT{$hours}H"));
-        else
-            $dt->sub(new DateInterval("PT{$hours}H"));
+        if($former_offset >= 0 ) {
+                    $dt->add(new DateInterval("PT{$hours}H"));
+        } else {
+                    $dt->sub(new DateInterval("PT{$hours}H"));
+        }
 
         return $dt;
     }

src/Facade/Utils/RequestFactory.php

Indentation +16 added lines, -16 removed lines

@@ -50,26 +50,26 @@ 
             break;
             case HttpMethods::Put:
                $etag = $params[0];
-               if(empty($etag)){
-                   return [
-                       Headers::ContentType  => ContentTypes::Calendar,
-                       Headers::IfNotMatch   => '*',
-                   ];
-               }
-               return [
-                   Headers::ContentType  => ContentTypes::Calendar,
-                   Headers::IfMatch      => $etag
-               ];
+                if(empty($etag)){
+                    return [
+                        Headers::ContentType  => ContentTypes::Calendar,
+                        Headers::IfNotMatch   => '*',
+                    ];
+                }
+                return [
+                    Headers::ContentType  => ContentTypes::Calendar,
+                    Headers::IfMatch      => $etag
+                ];
             break;
         }
         return [];
     }
     /**
-    * @param string $url
-    * @param string $body
-    * @param int $depth
-    * @return Request
-    */
+     * @param string $url
+     * @param string $body
+     * @param int $depth
+     * @return Request
+     */
     public static function createPropFindRequest($url , $body, $depth = 1){
         return new Request
         (
@@ -158,7 +158,7 @@ 
      * @param string $etag
      * @return Request
      */
-     public static function createPutRequest($url, $body, $etag = null){
+        public static function createPutRequest($url, $body, $etag = null){
         return new Request
         (
             HttpMethods::Put,

Spacing +19 added lines, -26 removed lines

@@ -26,8 +26,8 @@ 
      * @param array $params
      * @return array
      */
-    private static function createHeadersFor($http_method, array $params = []){
-        switch ($http_method){
+    private static function createHeadersFor($http_method, array $params = []) {
+        switch ($http_method) {
             case HttpMethods::PropFind:
             case HttpMethods::Options:
             case HttpMethods::Report:
@@ -39,7 +39,7 @@ 
             break;
             case HttpMethods::Delete:
                 $etag = $params[0];
-                if(!empty($etag))
+                if (!empty($etag))
                 return [
                     Headers::IfMatch =>  $params[0],
                 ];
@@ -53,7 +53,7 @@ 
             break;
             case HttpMethods::Put:
                $etag = $params[0];
-               if(empty($etag)){
+               if (empty($etag)) {
                    return [
                        Headers::ContentType  => ContentTypes::Calendar,
                        Headers::IfNotMatch   => '*',
@@ -73,11 +73,10 @@ 
     * @param int $depth
     * @return Request
     */
-    public static function createPropFindRequest($url , $body, $depth = 1){
-        return new Request
-        (
+    public static function createPropFindRequest($url, $body, $depth = 1) {
+        return new Request(
             HttpMethods::PropFind,
-            $url ,
+            $url,
             self::createHeadersFor(HttpMethods::PropFind, [$depth]),
             $body
         );
@@ -88,9 +87,8 @@ 
      * @param string $body
      * @return Request
      */
-    public static function createMakeCalendarRequest($url , $body){
-        return new Request
-        (
+    public static function createMakeCalendarRequest($url, $body) {
+        return new Request(
             HttpMethods::MakeCalendar,
             $url,
             self::createHeadersFor(HttpMethods::MakeCalendar),
@@ -103,9 +101,8 @@ 
      * @param int $depth
      * @return Request
      */
-    public static function createOptionsRequest($url, $depth = 1){
-        return new Request
-        (
+    public static function createOptionsRequest($url, $depth = 1) {
+        return new Request(
             HttpMethods::Options,
             $url,
             self::createHeadersFor(HttpMethods::Options, [$depth])
@@ -118,9 +115,8 @@ 
      * @param int $depth
      * @return Request
      */
-    public static function createReportRequest($url , $body, $depth = 1){
-        return new Request
-        (
+    public static function createReportRequest($url, $body, $depth = 1) {
+        return new Request(
             HttpMethods::Report,
             $url,
             self::createHeadersFor(HttpMethods::Report, [$depth]),
@@ -133,9 +129,8 @@ 
      * @param string $etag
      * @return Request
      */
-    public static function createDeleteRequest($url , $etag){
-        return new Request
-        (
+    public static function createDeleteRequest($url, $etag) {
+        return new Request(
             HttpMethods::Delete,
             $url,
             self::createHeadersFor(HttpMethods::Delete, [$etag])
@@ -146,9 +141,8 @@ 
      * @param string $url
      * @return Request
      */
-    public static function createGetRequest($url){
-        return new Request
-        (
+    public static function createGetRequest($url) {
+        return new Request(
             HttpMethods::Get,
             $url,
             self::createHeadersFor(HttpMethods::Get)
@@ -161,9 +155,8 @@ 
      * @param string $etag
      * @return Request
      */
-     public static function createPutRequest($url, $body, $etag = null){
-        return new Request
-        (
+     public static function createPutRequest($url, $body, $etag = null) {
+        return new Request(
             HttpMethods::Put,
             $url,
             self::createHeadersFor(HttpMethods::Put, [$etag]),

Braces +3 added lines, -2 removed lines

@@ -39,10 +39,11 @@
             break;
             case HttpMethods::Delete:
                 $etag = $params[0];
-                if(!empty($etag))
-                return [
+                if(!empty($etag)) {
+                                return [
                     Headers::IfMatch =>  $params[0],
                 ];
+                }
                 return [];
             break;
             case HttpMethods::MakeCalendar:

src/Facade/Responses/GenericMultiCalDAVResponse.php

Spacing +3 added lines, -3 removed lines

@@ -25,9 +25,9 @@ 
 
     protected function parse()
     {
-        if(isset($this->content['response'])){
+        if (isset($this->content['response'])) {
 
-            if(isset($this->content['response']['propstat'])) {
+            if (isset($this->content['response']['propstat'])) {
                 // its a collection with one single element
                 $single_resource = $this->buildSingleResponse();
                 $single_resource->setContent(['response' => $this->content['response']]);
@@ -48,7 +48,7 @@ 
     /**
      * @return GenericSinglePROPFINDCalDAVResponse[]
      */
-    public function getResponses(){
+    public function getResponses() {
         return $this->responses;
     }
 

src/Facade/Responses/VCardEntityResponse.php

Spacing +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
     /**
      * @return string
      */
-    public function getVCard(){
+    public function getVCard() {
         return isset($this->found_props['calendar-data']) ? $this->found_props['calendar-data'] : null;
     }
 }
\ No newline at end of file

src/Facade/Responses/CalendarDeletedResponse.php

Spacing +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
     /**
      * @return bool
      */
-    public function isSuccessFull(){
+    public function isSuccessFull() {
         return $this->code == HttpResponse::HttpCodeNoContent;
     }
 }
\ No newline at end of file