flack / openpsa

lib/org/openpsa/contacts/midcom/interfaces.php

Spacing +5 added lines, -5 removed lines

@@ -84,12 +84,12 @@ 
 
     public function resolve_object_link(midcom_db_topic $topic, midcom_core_dbaobject $object)
     {
-        if (   $object instanceof org_openpsa_contacts_group_dba
+        if ($object instanceof org_openpsa_contacts_group_dba
             || $object instanceof midcom_db_group)
         {
             return "group/{$object->guid}/";
         }
-        if (   $object instanceof org_openpsa_contacts_person_dba
+        if ($object instanceof org_openpsa_contacts_person_dba
             || $object instanceof midcom_db_person)
         {
             return "person/{$object->guid}/";
@@ -132,7 +132,7 @@ 
 
             if (count($items) > 0)
             {
-                if (   $items[0]->get_latitude()
+                if ($items[0]->get_latitude()
                     || $items[0]->get_longitude())
                 {
                     // This is a GeoRSS feed
@@ -143,7 +143,7 @@ 
 
         $hcards = @$hkit->getByURL('hcard', $url);
 
-        if (   is_array($hcards)
+        if (is_array($hcards)
             && count($hcards) > 0)
         {
             // We have found hCard data here
@@ -217,7 +217,7 @@ 
             {
                 $latitude = (float) $icbm_parts[0];
                 $longitude = (float) $icbm_parts[1];
-                if (   abs($latitude) < 90
+                if (abs($latitude) < 90
                     && abs($longitude) < 180)
                 {
                     $location = new org_routamc_positioning_location_dba();

Braces +5 added lines, -10 removed lines

@@ -59,8 +59,7 @@ 
             {
                 $root_groups[$name] = $group;
             }
-        }
-        else
+        } else
         {
             debug_add("OpenPSA Contacts root group could not be found", MIDCOM_LOG_WARN);
 
@@ -168,8 +167,7 @@ 
             try
             {
                 $person = new org_openpsa_contacts_person_dba($args['person']);
-            }
-            catch (midcom_error $e)
+            } catch (midcom_error $e)
             {
                 $handler->print_error("Person {$args['person']} not found, error " . $e->getMessage());
                 return false;
@@ -180,15 +178,13 @@ 
                 return false;
             }
             return $this->_check_person_url($person);
-        }
-        else if (array_key_exists('group', $args))
+        } else if (array_key_exists('group', $args))
         {
             // Handling for groups
             try
             {
                 $group = new org_openpsa_contacts_group_dba($args['group']);
-            }
-            catch (midcom_error $e)
+            } catch (midcom_error $e)
             {
                 $handler->print_error("Group {$args['group']} not found, error " . $e->getMessage());
                 return false;
@@ -255,8 +251,7 @@ 
         {
             // GeoRSS subscription is a good way to keep track of person's location
             $person->set_parameter('org.routamc.positioning:georss', 'georss_url', $data['georss_url']);
-        }
-        else if (array_key_exists('icbm', $data))
+        } else if (array_key_exists('icbm', $data))
         {
             // Instead of using the ICBM position data directly we can subscribe to it so we get modifications too
             $person->set_parameter('org.routamc.positioning:html', 'icbm_url', $person->homepage);

lib/org/openpsa/contacts/group.php

Spacing +4 added lines, -6 removed lines

@@ -14,8 +14,7 @@ 
     public $__midcom_class_name__ = __CLASS__;
     public $__mgdschema_class_name__ = 'org_openpsa_organization';
 
-    public $autodelete_dependents = array
-    (
+    public $autodelete_dependents = array(
         'org_openpsa_contacts_member_dba' => 'gid'
     );
 
@@ -32,7 +31,7 @@ 
 
     public function __set($name, $value)
     {
-        if (   $name == 'homepage'
+        if ($name == 'homepage'
             && !empty($value)
             && $value != $this->homepage)
         {
@@ -83,7 +82,7 @@ 
 
     public function __get($property)
     {
-        if (   $property == 'invoice_label'
+        if ($property == 'invoice_label'
             || $property == 'postal_label')
         {
             if (!isset($this->_address_extras[$property]))
@@ -124,8 +123,7 @@ 
     {
         if ($this->_register_prober)
         {
-            $args = array
-            (
+            $args = array(
                 'group' => $this->guid,
             );
             midcom_services_at_interface::register(time() + 60, 'org.openpsa.contacts', 'check_url', $args);

Braces +1 added lines, -2 removed lines

@@ -102,8 +102,7 @@
             if (!empty($this->name))
             {
                 $this->official = $this->name;
-            }
-            else
+            } else
             {
                 $this->official = "Group #{$this->id}";
             }

lib/org/openpsa/contacts/duplicates/check.php

Spacing +12 added lines, -16 removed lines

@@ -48,8 +48,7 @@ 
         //Search for all potential duplicates (more detailed checking is done later)
         $check_persons = $this->get_person_candidates($person);
 
-        $person = $this->normalize_fields(array
-        (
+        $person = $this->normalize_fields(array(
             'firstname' => $person->firstname,
             'lastname' => $person->lastname,
             'email' => $person->email,
@@ -122,8 +121,7 @@ 
      */
     function p_duplicate_person(array $person1, array $person2)
     {
-        $ret = array
-        (
+        $ret = array(
             'p' => 0,
             'email_match' => false,
             'handphone_match' => false,
@@ -144,7 +142,7 @@ 
             // if user's lastname is part of the email address, check to see if the difference is only in the domain part
             $email1 = preg_replace('/@.+/', '', $person1['email']);
             $email2 = preg_replace('/@.+/', '', $person2['email']);
-            if (   strpos($email1, $person1['lastname']) !== false
+            if (strpos($email1, $person1['lastname']) !== false
                 && $email1 == $email2)
             {
                 $ret['email_match'] = true;
@@ -203,7 +201,7 @@ 
 
     private function match($property, array $data1, array $data2)
     {
-        if (   !empty($data1[$property])
+        if (!empty($data1[$property])
             && $data1[$property] == $data2[$property])
         {
             return true;
@@ -243,8 +241,7 @@ 
 
         $check_groups = $this->get_group_candidates($group);
 
-        $group = $this->normalize_fields(array
-        (
+        $group = $this->normalize_fields(array(
             'official' => $group->official,
             'street' => $group->street,
             'phone' => $group->phone,
@@ -303,8 +300,7 @@ 
      */
     function p_duplicate_group(array $group1, array $group2)
     {
-        $ret = array
-        (
+        $ret = array(
             'p' => 0,
             'homepage_match' => false,
             'phone_match' => false,
@@ -370,7 +366,7 @@ 
         $persons = $this->get_person_candidates();
 
         $params = array();
-        $params['objects'] =& $persons;
+        $params['objects'] = & $persons;
         $params['mode'] = 'person';
 
         array_walk($persons, array($this, 'check_all_arraywalk'), $params);
@@ -442,7 +438,7 @@ 
                 continue;
             }
 
-            if (   $obj1->get_parameter('org.openpsa.contacts.duplicates:not_duplicate', $obj2->guid)
+            if ($obj1->get_parameter('org.openpsa.contacts.duplicates:not_duplicate', $obj2->guid)
                 || $obj2->get_parameter('org.openpsa.contacts.duplicates:not_duplicate', $obj1->guid))
             {
                 // Not-duplicate parameter found, returning zero probability
@@ -472,7 +468,7 @@ 
 
         $groups = $this->get_group_candidates();
         $params = array();
-        $params['objects'] =& $groups;
+        $params['objects'] = & $groups;
         $params['mode'] = 'group';
         array_walk($groups, array($this, 'check_all_arraywalk'), $params);
 
@@ -517,7 +513,7 @@ 
                 }
             }
         }
-        debug_add("Done on " . time() . ", took: " . (time() - $time_start) .  " seconds");
+        debug_add("Done on " . time() . ", took: " . (time() - $time_start) . " seconds");
         if ($output)
         {
             echo "INFO: DONE with persons. Elapsed time " . (time() - $time_start) . " seconds<br/>\n";
@@ -557,10 +553,10 @@ 
             }
         }
 
-        debug_add("Done on " . time() . ", took: " . (time()-$time_start) .  " seconds");
+        debug_add("Done on " . time() . ", took: " . (time() - $time_start) . " seconds");
         if ($output)
         {
-            echo "INFO: DONE with groups. Elapsed time: " . (time()-$time_start) ." seconds<br/>\n";
+            echo "INFO: DONE with groups. Elapsed time: " . (time() - $time_start) . " seconds<br/>\n";
             flush();
         }
     }

Braces +2 added lines, -4 removed lines

@@ -138,8 +138,7 @@ 
         {
             $ret['email_match'] = true;
             $ret['p'] += 1;
-        }
-        else if (!empty($person1['lastname']))
+        } else if (!empty($person1['lastname']))
         {
             // if user's lastname is part of the email address, check to see if the difference is only in the domain part
             $email1 = preg_replace('/@.+/', '', $person1['email']);
@@ -435,8 +434,7 @@ 
             {
                 $obj1 = $class::get_cached($arr1['guid']);
                 $obj2 = $class::get_cached($arr2['guid']);
-            }
-            catch (midcom_error $e)
+            } catch (midcom_error $e)
             {
                 $e->log();
                 continue;

lib/org/openpsa/contacts/duplicates/merge.php

Spacing +7 added lines, -8 removed lines

@@ -46,7 +46,7 @@ 
      */
     public function merge($obj1, $obj2, $merge_mode)
     {
-        if (   $merge_mode !== 'all'
+        if ($merge_mode !== 'all'
             && $merge_mode !== 'future')
         {
             debug_add("invalid mode {$merge_mode}", MIDCOM_LOG_ERROR);
@@ -97,10 +97,10 @@ 
                         {
                             $dup = $this->check_duplicate($results, $result, $conf['duplicate_check']);
 
-                            if (   is_object($dup)
+                            if (is_object($dup)
                                 || $dup === false)
                             {
-                                if (   !is_object($dup)
+                                if (!is_object($dup)
                                     || !array_key_exists($dup->id, $todelete))
                                 {
                                     $todelete[$result->id] = $result;
@@ -111,7 +111,7 @@ 
                         }
                     }
                 }
-                if (   $needs_update
+                if ($needs_update
                     && !$result->update())
                 {
                     throw new midcom_error("Failed to update {$classname} #{$result->id}, errstr: " . midcom_connection::get_error_string());
@@ -170,7 +170,7 @@ 
 
         foreach ($results as $result)
         {
-            if (   $result->$field === $object->$field
+            if ($result->$field === $object->$field
                 && $result->id !== $object->id)
             {
                 return $result;
@@ -182,8 +182,7 @@ 
     private function merge_persons($person1, $person2)
     {
         // Copy fields missing from person1 and present in person2 over
-        $skip_properties = array
-        (
+        $skip_properties = array(
             'id' => true,
             'guid' => true,
         );
@@ -191,7 +190,7 @@ 
         foreach ($person2 as $property => $value)
         {
             // Copy only simple properties not marked to be skipped missing from person1
-            if (   empty($person2->$property)
+            if (empty($person2->$property)
                 || !empty($person1->$property)
                 || isset($skip_properties[$property])
                 || !is_scalar($value))

lib/org/openpsa/calendar/viewer.php

Spacing +2 added lines, -4 removed lines

@@ -74,10 +74,8 @@
      */
     public function get_calendar_options()
     {
-        $options = array
-        (
-            'businessHours' => array
-            (
+        $options = array(
+            'businessHours' => array(
                 'start' => $this->_config->get('day_start_time') . ':00',
                 'end' => $this->_config->get('day_end_time') . ':00',
                 'dow' => array(1, 2, 3, 4, 5)

lib/org/openpsa/calendar/event.php

Spacing +6 added lines, -6 removed lines

@@ -166,7 +166,7 @@ 
         }
 
         //Check up
-        if (   !$this->up
+        if (!$this->up
             && $this->title != '__org_openpsa_calendar')
         {
             $root_event = org_openpsa_calendar_interface::find_root_event();
@@ -202,9 +202,9 @@ 
     private function _check_timerange()
     {
         //Force types
-        $this->start = (int)$this->start;
-        $this->end = (int)$this->end;
-        if (   !$this->start
+        $this->start = (int) $this->start;
+        $this->end = (int) $this->end;
+        if (!$this->start
             || !$this->end)
         {
             debug_add('Event must have start and end timestamps');
@@ -216,7 +216,7 @@ 
          * Force start and end seconds to 1 and 0 respectively
          * (to avoid stupid one second overlaps)
          */
-        $this->start = mktime(  date('G', $this->start),
+        $this->start = mktime(date('G', $this->start),
                                 date('i', $this->start),
                                 1,
                                 date('n', $this->start),
@@ -325,7 +325,7 @@ 
      */
     private function _suspects_classes_present()
     {
-        return (   class_exists('org_openpsa_relatedto_dba')
+        return (class_exists('org_openpsa_relatedto_dba')
                 && class_exists('org_openpsa_relatedto_suspect'));
     }
 

Braces +3 added lines, -6 removed lines

@@ -310,8 +310,7 @@ 
             if ($linkdata['link']->create())
             {
                 debug_add("saved link to task #{$linkdata['other_obj']->id} (link id #{$linkdata['link']->id})", MIDCOM_LOG_INFO);
-            }
-            else
+            } else
             {
                 debug_add("could not save link to task #{$linkdata['other_obj']->id}, errstr" . midcom_connection::get_error_string(), MIDCOM_LOG_WARN);
             }
@@ -360,8 +359,7 @@ 
             if ($linkdata['link']->create())
             {
                 debug_add("saved sales link to {$linkdata['other_obj']->guid} (link id #{$linkdata['link']->id})", MIDCOM_LOG_INFO);
-            }
-            else
+            } else
             {
                 debug_add("could not save sales link to {$linkdata['other_obj']->guid}, errstr" . midcom_connection::get_error_string(), MIDCOM_LOG_WARN);
             }
@@ -420,8 +418,7 @@ 
         if ($this->orgOpenpsaAccesstype == org_openpsa_core_acl::ACCESS_PRIVATE)
         {
             $this->set_privilege('org.openpsa.calendar:read', 'EVERYONE', MIDCOM_PRIVILEGE_DENY);
-        }
-        else
+        } else
         {
             $this->set_privilege('org.openpsa.calendar:read', 'EVERYONE', MIDCOM_PRIVILEGE_ALLOW);
         }

lib/org/openpsa/calendar/event/resource.php

Braces +1 added lines, -2 removed lines

@@ -45,8 +45,7 @@
         try
         {
             $resource = org_openpsa_calendar_resource_dba::get_cached($this->resource);
-        }
-        catch (midcom_error $e)
+        } catch (midcom_error $e)
         {
             debug_add("Cannot fetch resource #{$this->resource} returning false", MIDCOM_LOG_INFO);
             return false;

lib/org/openpsa/calendar/event/member.php

Spacing +11 added lines, -14 removed lines

@@ -73,7 +73,7 @@ 
             $event = new org_openpsa_calendar_event_dba($this->eid);
         }
 
-        if (    $recipient->id == midcom_connection::get_user()
+        if ($recipient->id == midcom_connection::get_user()
              && !$event->send_notify_me)
         {
             //Do not send notification to current user
@@ -119,7 +119,7 @@ 
                 return false;
         }
 
-        if (   $type == 'cancel'
+        if ($type == 'cancel'
             || $type == 'remove')
         {
             // TODO: Create iCal export with correct delete commands
@@ -129,10 +129,8 @@ 
             $generator = midcom::get()->serviceloader->load('midcom_core_service_urlgenerator');
             $encoder = new org_openpsa_calendar_vcal;
             $encoder->add_event($event);
-            $message['attachments'] = array
-            (
-                array
-                (
+            $message['attachments'] = array(
+                array(
                     'name' => $generator->from_string(sprintf('%s on %s', $event->title, date('Ymd_Hi', $event->start))) . '.ics',
                     'mimetype' => 'text/calendar',
                     'content' => (string) $encoder,
@@ -208,7 +206,7 @@ 
         {
             $ymd = date('Ymd', $stamp);
             debug_add("making sure date {$ymd} has at least one event");
-            $stamp = mktime(0, 0, 1, date('m', $stamp), date('d', $stamp)+1, date('Y', $stamp));
+            $stamp = mktime(0, 0, 1, date('m', $stamp), date('d', $stamp) + 1, date('Y', $stamp));
             if (array_key_exists($ymd, $events_by_date))
             {
                 continue;
@@ -227,13 +225,13 @@ 
             $workday_starts = 8;
             $workday_ends = 16;
 
-            $workday_starts_ts = mktime($workday_starts, 0, 0, (int)$ymd_matches[2], (int)$ymd_matches[3], (int)$ymd_matches[1]);
-            $workday_ends_ts = mktime($workday_ends, 0, 0, (int)$ymd_matches[2], (int)$ymd_matches[3], (int)$ymd_matches[1]);
+            $workday_starts_ts = mktime($workday_starts, 0, 0, (int) $ymd_matches[2], (int) $ymd_matches[3], (int) $ymd_matches[1]);
+            $workday_ends_ts = mktime($workday_ends, 0, 0, (int) $ymd_matches[2], (int) $ymd_matches[3], (int) $ymd_matches[1]);
             $last_end_time = false;
             $last_event = false;
             foreach ($events as $event)
             {
-                if (   $event->end <= $workday_starts_ts
+                if ($event->end <= $workday_starts_ts
                     || $event->start >= $workday_ends_ts)
                 {
                     // We need not to consider this event, it is outside the defined workday
@@ -271,8 +269,8 @@ 
             {
                 $last_end_time = $workday_starts_ts;
             }
-            if (   $last_end_time < $workday_ends_ts
-                && (($workday_ends_ts- $last_end_time) >= $amount))
+            if ($last_end_time < $workday_ends_ts
+                && (($workday_ends_ts - $last_end_time) >= $amount))
             {
                 $slots[] = self::_create_slot($last_end_time, $workday_ends_ts, $last_event);
             }
@@ -283,8 +281,7 @@ 
 
     private static function _create_slot($start, $end, $previous, $next = false)
     {
-        return array
-        (
+        return array(
             'start' => $start,
             'end' => $end,
             'previous' => $previous,

Braces +4 added lines, -8 removed lines

@@ -123,8 +123,7 @@ 
             || $type == 'remove')
         {
             // TODO: Create iCal export with correct delete commands
-        }
-        else
+        } else
         {
             $generator = midcom::get()->serviceloader->load('midcom_core_service_urlgenerator');
             $encoder = new org_openpsa_calendar_vcal;
@@ -158,8 +157,7 @@ 
                 debug_add('person #' . $person->id . 'has no email address, aborting');
                 return false;
             }
-        }
-        catch (midcom_error $e)
+        } catch (midcom_error $e)
         {
             return false;
         }
@@ -190,8 +188,7 @@ 
             try
             {
                 $event = org_openpsa_calendar_event_dba::get_cached($eid);
-            }
-            catch (midcom_error $e)
+            } catch (midcom_error $e)
             {
                 continue;
             }
@@ -247,8 +244,7 @@ 
                     {
                         // First event of the day starts after we have started working, use work start time as last end time.
                         $last_end_time = $workday_starts_ts;
-                    }
-                    else
+                    } else
                     {
                         // Make the first event of the day the last end time and skip rest of the checks
                         $last_end_time = $event->end;

lib/org/openpsa/calendar/vcal.php

Spacing +1 added lines, -2 removed lines

@@ -99,8 +99,7 @@
                 // Attendee must have email address of valid format, these must also be unique.
                 $person->email = preg_replace('/[^0-9_\x61-\x7a]/i', '_', strtolower($person->name)) . '_is_not@openpsa2.org';
             }
-            $parameters = array
-            (
+            $parameters = array(
                 'ROLE' => 'REQ-PARTICIPANT',
                 'CUTYPE' => 'INDIVIDUAL',
                 'PARTSTAT' => 'ACCEPTED',

Braces +1 added lines, -2 removed lines

@@ -68,8 +68,7 @@
         if ($event->busy)
         {
             $vevent->TRANSP = 'OPAQUE';
-        }
-        else
+        } else
         {
             $vevent->TRANSP = 'TRANSPARENT';
         }