smarcet / CalDAVClient

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/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

src/Facade/Requests/MakeCalendarRequestVO.php

Spacing +2 added lines, -2 removed lines

@@ -60,7 +60,7 @@ 
         $this->timezone      = $timezone;
         $this->uid           = md5(uniqid(mt_rand(), true));
 
-        if(is_null($this->timezone)){
+        if (is_null($this->timezone)) {
             $this->timezone = new DateTimeZone('UTC');
         }
     }
@@ -68,7 +68,7 @@ 
     /**
      * @param string $uid
      */
-    public function setUID($uid){
+    public function setUID($uid) {
         $this->uid = $uid;
     }
 

src/Facade/Utils/RequestFactory.php

Indentation +12 added lines, -12 removed lines

@@ -50,21 +50,21 @@ 
                 ];
             case HttpMethods::Put:
                $etag = $params[0];
-               if(!empty($etag)){
-                   return [
-                       Headers::ContentType  => ContentTypes::Calendar,
-                       Headers::IfMatch      => $etag
-                   ];
-               }
+                if(!empty($etag)){
+                    return [
+                        Headers::ContentType  => ContentTypes::Calendar,
+                        Headers::IfMatch      => $etag
+                    ];
+                }
         }
         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
         (
@@ -153,7 +153,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:
@@ -38,7 +38,7 @@ 
                 ];
             case HttpMethods::Delete:
                 $etag = $params[0];
-                if(!empty($etag)) {
+                if (!empty($etag)) {
                     return [
                         Headers::IfMatch => $etag,
                     ];
@@ -50,7 +50,7 @@ 
                 ];
             case HttpMethods::Put:
                $etag = $params[0];
-               if(!empty($etag)){
+               if (!empty($etag)) {
                    return [
                        Headers::ContentType  => ContentTypes::Calendar,
                        Headers::IfMatch      => $etag
@@ -65,11 +65,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
         );
@@ -80,9 +79,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),
@@ -95,9 +93,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])
@@ -110,9 +107,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]),
@@ -125,9 +121,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])
@@ -138,9 +133,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)
@@ -153,9 +147,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]),

src/Facade/CalDavClient.php

Spacing +38 added lines, -57 removed lines

@@ -137,15 +137,15 @@ 
      * @param Request $http_request
      * @return mixed|\Psr\Http\Message\ResponseInterface
      */
-    private function makeRequest(Request $http_request){
-        try{
+    private function makeRequest(Request $http_request) {
+        try {
             return $this->client->send($http_request, [
                 'auth'    => [$this->user, $this->password, $this->authtype],
                 'timeout' => $this->timeout
             ]);
         }
-        catch (ClientException $ex){
-            switch($ex->getCode()){
+        catch (ClientException $ex) {
+            switch ($ex->getCode()) {
                 case 401:
                     throw new UserUnAuthorizedException();
                     break;
@@ -173,9 +173,9 @@ 
 
         $res     = $http_response->hasHeader(self::DAVHeader);
         $options = [];
-        if($res){
+        if ($res) {
             $val = $http_response->getHeaderLine(self::DAVHeader);
-            if(!empty($val)){
+            if (!empty($val)) {
                 $options = explode(', ', $val);
             }
         }
@@ -189,14 +189,13 @@ 
     public function getUserPrincipal()
     {
         $http_response = $this->makeRequest(
-            RequestFactory::createPropFindRequest
-            (
+            RequestFactory::createPropFindRequest(
                 $this->server_url,
                 CalDAVRequestFactory::getInstance()->build(CalDAVRequestFactory::PrincipalRequestType)->getContent()
             )
         );
 
-        return new UserPrincipalResponse($this->server_url, (string)$http_response->getBody(), $http_response->getStatusCode());
+        return new UserPrincipalResponse($this->server_url, (string) $http_response->getBody(), $http_response->getStatusCode());
     }
 
     /**
@@ -206,14 +205,13 @@ 
     public function getCalendarHome($principal_url)
     {
         $http_response = $this->makeRequest(
-            RequestFactory::createPropFindRequest
-            (
+            RequestFactory::createPropFindRequest(
                 $principal_url,
                 CalDAVRequestFactory::getInstance()->build(CalDAVRequestFactory::CalendarHomeRequestType)->getContent()
             )
         );
 
-        return new CalendarHomesResponse($this->server_url, (string)$http_response->getBody(), $http_response->getStatusCode());
+        return new CalendarHomesResponse($this->server_url, (string) $http_response->getBody(), $http_response->getStatusCode());
     }
 
     /**
@@ -227,8 +225,7 @@ 
         $uid           = $vo->getUID();
         $resource_url  = $calendar_home_set.$uid;
         $http_response = $this->makeRequest(
-            RequestFactory::createMakeCalendarRequest
-            (
+            RequestFactory::createMakeCalendarRequest(
                 $resource_url,
                 CalDAVRequestFactory::getInstance()->build(CalDAVRequestFactory::CalendarCreateRequestType, [$vo])->getContent()
             )
@@ -244,14 +241,13 @@ 
     public function getCalendars($calendar_home_set_url)
     {
         $http_response = $this->makeRequest(
-            RequestFactory::createPropFindRequest
-            (
+            RequestFactory::createPropFindRequest(
                 $calendar_home_set_url,
                 CalDAVRequestFactory::getInstance()->build(CalDAVRequestFactory::CalendarsRequestType)->getContent()
             )
         );
 
-        return new GetCalendarsResponse($this->server_url, (string)$http_response->getBody(), $http_response->getStatusCode());
+        return new GetCalendarsResponse($this->server_url, (string) $http_response->getBody(), $http_response->getStatusCode());
     }
 
     /**
@@ -261,15 +257,14 @@ 
     public function getCalendar($calendar_url)
     {
         $http_response = $this->makeRequest(
-            RequestFactory::createPropFindRequest
-            (
+            RequestFactory::createPropFindRequest(
                 $calendar_url,
                 CalDAVRequestFactory::getInstance()->build(CalDAVRequestFactory::CalendarRequestType)->getContent(),
                 0
             )
         );
 
-        return new GetCalendarResponse($this->server_url, (string)$http_response->getBody(), $http_response->getStatusCode());
+        return new GetCalendarResponse($this->server_url, (string) $http_response->getBody(), $http_response->getStatusCode());
     }
 
 
@@ -282,14 +277,13 @@ 
     {
 
         $http_response = $this->makeRequest(
-            RequestFactory::createReportRequest
-            (
+            RequestFactory::createReportRequest(
                 $calendar_url,
                 CalDAVRequestFactory::getInstance()->build(CalDAVRequestFactory::CalendarSyncRequestType, [$sync_token])->getContent()
             )
         );
 
-        return new CalendarSyncInfoResponse($this->server_url, (string)$http_response->getBody(), $http_response->getStatusCode());
+        return new CalendarSyncInfoResponse($this->server_url, (string) $http_response->getBody(), $http_response->getStatusCode());
     }
 
     /**
@@ -302,19 +296,17 @@ 
         $uid           = $vo->getUID();
         $resource_url  = $calendar_url.$uid.self::SchedulingInformationSuffix;
         $http_response = $this->makeRequest(
-            RequestFactory::createPutRequest
-            (
+            RequestFactory::createPutRequest(
                 $resource_url,
                 CalDAVRequestFactory::getInstance()->build(CalDAVRequestFactory::EventCreateRequestType, [$vo])->getContent()
             )
         );
         $etag = $http_response->hasHeader(self::ETagHeader) ? $http_response->getHeaderLine(self::ETagHeader) : null;
-        return new EventCreatedResponse
-        (
+        return new EventCreatedResponse(
             $uid,
             $etag,
             $resource_url,
-            (string)$http_response->getBody(),
+            (string) $http_response->getBody(),
             $http_response->getStatusCode()
         );
     }
@@ -330,20 +322,18 @@ 
         $uid           = $vo->getUID();
         $resource_url  = $calendar_url.$uid.self::SchedulingInformationSuffix;
         $http_response = $this->makeRequest(
-            RequestFactory::createPutRequest
-            (
+            RequestFactory::createPutRequest(
                 $resource_url,
                 CalDAVRequestFactory::getInstance()->build(CalDAVRequestFactory::EventUpdateRequestType, [$vo])->getContent(),
                 $etag
             )
         );
         $etag = $http_response->hasHeader(self::ETagHeader) ? $http_response->getHeaderLine(self::ETagHeader) : null;
-        return new EventUpdatedResponse
-        (
+        return new EventUpdatedResponse(
             $uid,
             $etag,
             $resource_url,
-            (string)$http_response->getBody(),
+            (string) $http_response->getBody(),
             $http_response->getStatusCode()
         );
     }
@@ -357,16 +347,14 @@ 
     public function deleteEvent($calendar_url, $uid, $etag = null)
     {
         $http_response = $this->makeRequest(
-            RequestFactory::createDeleteRequest
-            (
+            RequestFactory::createDeleteRequest(
                 $calendar_url.$uid.self::SchedulingInformationSuffix,
                 $etag
             )
         );
 
-        return new EventDeletedResponse
-        (
-            (string)$http_response->getBody(), $http_response->getStatusCode()
+        return new EventDeletedResponse(
+            (string) $http_response->getBody(), $http_response->getStatusCode()
         );
     }
 
@@ -374,15 +362,14 @@ 
      * @param string $event_url
      * @return string
      */
-    public function getEventVCardBy($event_url){
+    public function getEventVCardBy($event_url) {
         $http_response = $this->makeRequest(
-            RequestFactory::createGetRequest
-            (
+            RequestFactory::createGetRequest(
                 $event_url
             )
         );
 
-        $ical = (string)$http_response->getBody();
+        $ical = (string) $http_response->getBody();
         return $ical;
     }
 
@@ -394,17 +381,15 @@ 
     public function getEventsBy($calendar_url, array $events_urls)
     {
         $http_response = $this->makeRequest(
-            RequestFactory::createReportRequest
-            (
+            RequestFactory::createReportRequest(
                 $calendar_url,
                 CalDAVRequestFactory::getInstance()->build(CalDAVRequestFactory::CalendarMultiGetRequestType, [$events_urls])->getContent()
             )
         );
 
-        return new ResourceCollectionResponse
-        (
+        return new ResourceCollectionResponse(
             $this->server_url,
-            (string)$http_response->getBody(),
+            (string) $http_response->getBody(),
             $http_response->getStatusCode()
         );
     }
@@ -418,17 +403,15 @@ 
     {
 
         $http_response = $this->makeRequest(
-            RequestFactory::createReportRequest
-            (
+            RequestFactory::createReportRequest(
                 $calendar_url,
                 CalDAVRequestFactory::getInstance()->build(CalDAVRequestFactory::CalendarQueryRequestType, [$filter])->getContent()
             )
         );
 
-        return new ResourceCollectionResponse
-        (
+        return new ResourceCollectionResponse(
             $this->server_url,
-            (string)$http_response->getBody(),
+            (string) $http_response->getBody(),
             $http_response->getStatusCode()
         );
     }
@@ -441,16 +424,14 @@ 
     public function deleteCalendar($calendar_url, $etag = null)
     {
         $http_response = $this->makeRequest(
-            RequestFactory::createDeleteRequest
-            (
+            RequestFactory::createDeleteRequest(
                 $calendar_url,
                 $etag
             )
         );
 
-        return new CalendarDeletedResponse
-        (
-            $this->server_url, (string)$http_response->getBody(), $http_response->getStatusCode()
+        return new CalendarDeletedResponse(
+            $this->server_url, (string) $http_response->getBody(), $http_response->getStatusCode()
         );
     }
 }

Braces +1 added lines, -2 removed lines

@@ -143,8 +143,7 @@
                 'auth'    => [$this->user, $this->password, $this->authtype],
                 'timeout' => $this->timeout
             ]);
-        }
-        catch (ClientException $ex){
+        } catch (ClientException $ex){
             switch($ex->getCode()){
                 case 401:
                     throw new UserUnAuthorizedException();