flack / openpsa

lib/org/openpsa/contacts/handler/group/action.php

Braces +2 added lines, -1 removed lines

@@ -30,7 +30,8 @@
                 $member->require_do('midgard:update');
                 $member->extra = $_POST['title'];
                 $response->status = $member->update();
-            } catch (midcom_error $e) {
+            }
+            catch (midcom_error $e) {
                 $e->log();
             }
             $response->message = midcom_connection::get_error_string();

Spacing +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
         $response = new midcom_response_json;
         $response->status = false;
 
-        if (   !empty($_POST['guid'])
+        if (!empty($_POST['guid'])
             && array_key_exists('title', $_POST)) {
             try {
                 $member = new midcom_db_member($_POST['guid']);

lib/org/openpsa/contacts/handler/search.php

Spacing +5 added lines, -5 removed lines

@@ -80,7 +80,7 @@ 
 
         // Handle automatic wildcards
         $auto_wildcards = $this->_config->get('auto_wildcards');
-        if (   $auto_wildcards
+        if ($auto_wildcards
             && strpos($this->_query_string_processed, '%') === false) {
             switch ($auto_wildcards) {
                 case 'both':
@@ -127,7 +127,7 @@ 
      */
     public function _show_search_type($handler_id, array &$data)
     {
-        if (   $this->_view == 'foaf'
+        if ($this->_view == 'foaf'
             && sizeof($this->_persons) > 0) {
             midcom_show_style('foaf-header');
             foreach ($this->_persons as $person) {
@@ -159,11 +159,11 @@ 
             return $this->_prepare_json_reply();
         }
 
-        if (   count($this->_groups) == 1
+        if (count($this->_groups) == 1
             && count($this->_persons) == 0) {
             return new midcom_response_relocate('group/' . $this->_groups[0]->guid . '/');
         }
-        if (   count($this->_groups) == 0
+        if (count($this->_groups) == 0
             && count($this->_persons) == 1) {
             return new midcom_response_relocate('person/' . $this->_persons[0]->guid . '/');
         }
@@ -238,7 +238,7 @@ 
 
         midcom_show_style('search-header');
 
-        if (   count($this->_groups) == 0
+        if (count($this->_groups) == 0
             && count($this->_persons) == 0
             && $this->_query_string) {
             //No results at all (from any of the queries)

lib/org/openpsa/contacts/person.php

Spacing +2 added lines, -2 removed lines

@@ -22,7 +22,7 @@ 
 
     public function __construct($identifier = null)
     {
-        if (   midcom::get()->config->get('person_class') != 'midgard_person'
+        if (midcom::get()->config->get('person_class') != 'midgard_person'
             && midcom::get()->config->get('person_class') != 'openpsa_person') {
             $this->__mgdschema_class_name__ = midcom::get()->config->get('person_class');
         }
@@ -31,7 +31,7 @@ 
 
     public function __set($name, $value)
     {
-        if (   $name == 'homepage'
+        if ($name == 'homepage'
             && !empty($value)
             && $value != $this->homepage) {
             $this->_url_changed = true;

lib/org/openpsa/qbpager/main.php

Spacing +7 added lines, -7 removed lines

@@ -61,16 +61,16 @@ 
     private function _check_page_vars()
     {
         $page_var = $this->_prefix . 'page';
-        $results_var =  $this->_prefix . 'results';
+        $results_var = $this->_prefix . 'results';
         if (!empty($_REQUEST[$page_var])) {
             debug_add("{$page_var} has value: {$_REQUEST[$page_var]}");
-            $this->_current_page = (int)$_REQUEST[$page_var];
+            $this->_current_page = (int) $_REQUEST[$page_var];
         }
         if (!empty($_REQUEST[$results_var])) {
             debug_add("{$results_var} has value: {$_REQUEST[$results_var]}");
-            $this->results_per_page = (int)$_REQUEST[$results_var];
+            $this->results_per_page = (int) $_REQUEST[$results_var];
         }
-        $this->_offset = ($this->_current_page-1)*$this->results_per_page;
+        $this->_offset = ($this->_current_page - 1) * $this->results_per_page;
         if ($this->_offset < 0) {
             $this->_offset = 0;
         }
@@ -292,7 +292,7 @@ 
         return $this->_midcom_qb->add_constraint($param, $op, $val);
     }
 
-    public function add_order($param, $sort='ASC')
+    public function add_order($param, $sort = 'ASC')
     {
         if (!$this->_sanity_check()) {
             return false;
@@ -329,7 +329,7 @@ 
         if (!$this->_sanity_check()) {
             return false;
         }
-        if (   !$this->count
+        if (!$this->count
             || $this->_count_mode != 'count') {
             $this->count = $this->_midcom_qb_count->count();
         }
@@ -342,7 +342,7 @@ 
         if (!$this->_sanity_check()) {
             return false;
         }
-        if (   !$this->count
+        if (!$this->count
             || $this->_count_mode != 'count_unchecked') {
             $this->count = $this->_midcom_qb_count->count_unchecked();
         }

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

Braces +4 added lines, -2 removed lines

@@ -25,7 +25,8 @@ 
         midcom::get()->auth->request_sudo($this->_component);
         try {
             $person = new midcom_db_person($args['guid']);
-        } catch (midcom_error $e) {
+        }
+        catch (midcom_error $e) {
             $handler->print_error('Person with guid #' . $args['guid'] . ' does not exist');
             midcom::get()->auth->drop_sudo();
             return false;
@@ -33,7 +34,8 @@ 
         $accounthelper = new org_openpsa_user_accounthelper($person);
         try {
             $accounthelper->reopen_account();
-        } catch (midcom_error $e) {
+        }
+        catch (midcom_error $e) {
             $handler->print_error($e->getMessage());
             midcom::get()->auth->drop_sudo();
             return false;

lib/org/openpsa/projects/handler/task/resourcing.php

Spacing +2 added lines, -2 removed lines

@@ -58,7 +58,7 @@ 
         $this->_task = new org_openpsa_projects_task_dba($args[0]);
         $this->_task->require_do('midgard:create');
 
-        if (   array_key_exists('org_openpsa_projects_prospects', $_POST)
+        if (array_key_exists('org_openpsa_projects_prospects', $_POST)
             && $_POST['save']) {
             $qb = org_openpsa_projects_task_resource_dba::new_query_builder();
             $qb->add_constraint('guid', 'IN', array_keys($_POST['org_openpsa_projects_prospects']));
@@ -89,7 +89,7 @@ 
                     // create relatedto
                     org_openpsa_relatedto_plugin::create($event, 'org.openpsa.calendar', $this->_task, 'org.openpsa.projects');
                 }
-                if (   $update_prospect
+                if ($update_prospect
                     && !$prospect->update()) {
                     debug_add('Failed to update prospect: ' . midcom_connection::get_error_string(), MIDCOM_LOG_ERROR);
                 }

lib/org/openpsa/projects/task/status.php

Spacing +1 added lines, -1 removed lines

@@ -63,7 +63,7 @@
     public function _on_created()
     {
         //Remove the resource if necessary
-        if (   $this->type == self::DECLINED
+        if ($this->type == self::DECLINED
             && $this->targetPerson) {
             $qb = org_openpsa_projects_task_resource_dba::new_query_builder();
             $qb->add_constraint('task', '=', $this->task);

Braces +2 added lines, -1 removed lines

@@ -91,7 +91,8 @@
                 if ($recipient->guid == $this->metadata->creator) {
                     return org_openpsa_projects_workflow::accept($task, 0, $this->comment);
                 }
-            } catch (midcom_error $e) {
+            }
+            catch (midcom_error $e) {
                 $e->log();
             }
         }

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

Braces +2 added lines, -1 removed lines

@@ -124,7 +124,8 @@
     {
         try {
             $task = new org_openpsa_projects_task_dba($args['task']);
-        } catch (midcom_error $e) {
+        }
+        catch (midcom_error $e) {
             $e->log();
             return false;
         }

Spacing +1 added lines, -1 removed lines

@@ -51,7 +51,7 @@
      */
     private function _find_suspects_event(midcom_core_dbaobject $object, $defaults, array &$links_array)
     {
-        if (   !is_array($object->participants)
+        if (!is_array($object->participants)
             || count($object->participants) < 1) {
             //We have invalid list or zero participants, abort
             return;

lib/org/openpsa/projects/project.php

Spacing +1 added lines, -1 removed lines

@@ -231,7 +231,7 @@
             $new_status = $this->_find_status($this->_status_map[$this->status_type], $status_types);
         }
 
-        if (   !is_null($new_status)
+        if (!is_null($new_status)
             && $this->status != $new_status) {
             $this->status = $new_status;
             $update_required = true;