flack / openpsa

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/widget/password.php

Spacing +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
         $elements = array();
         $attributes = array_merge($attributes, array('class' => 'shorttext', 'id' => $this->name . '_input'));
 
-        $menu ='<label><input type="radio" name="org_openpsa_user_person_account_password_switch" value="0" checked="checked"/> ' . $this->_l10n->get("generate_password") . '
+        $menu = '<label><input type="radio" name="org_openpsa_user_person_account_password_switch" value="0" checked="checked"/> ' . $this->_l10n->get("generate_password") . '
             </label>
             <label>
                 <input type="radio" name="org_openpsa_user_person_account_password_switch" value="1"/> ' . $this->_l10n->get("own_password") . '

lib/org/openpsa/user/handler/person/view.php

Spacing +2 added lines, -2 removed lines

@@ -43,7 +43,7 @@ 
         $this->add_breadcrumb('', $this->_person->get_label());
 
         $auth = midcom::get()->auth;
-        if (   $this->_person->id == midcom_connection::get_user()
+        if ($this->_person->id == midcom_connection::get_user()
             || $auth->can_user_do('org.openpsa.user:manage', null, 'org_openpsa_user_interface')) {
             $buttons = array();
             $workflow = $this->get_workflow('datamanager2');
@@ -56,7 +56,7 @@ 
                 $delete_workflow = $this->get_workflow('delete', array('object' => $this->_person));
                 $buttons[] = $delete_workflow->get_button("delete/{$this->_person->guid}/");
             }
-            if (   midcom_connection::is_user($this->_person)
+            if (midcom_connection::is_user($this->_person)
                 && $this->_person->can_do('midgard:privileges')) {
                 $buttons[] = $workflow->get_button("privileges/{$this->_person->guid}/", array(
                     MIDCOM_TOOLBAR_LABEL => $this->_l10n->get("permissions"),

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/hour/report.php

Braces +2 added lines, -1 removed lines

@@ -69,7 +69,8 @@
         try {
             $parent = new org_openpsa_projects_task_dba($this->task);
             $parent->update_cache();
-        } catch (midcom_error $e) {
+        }
+        catch (midcom_error $e) {
             return false;
         }
         if ($start) {

Spacing +1 added lines, -1 removed lines

@@ -90,7 +90,7 @@
     {
         if ($this->invoiceable) {
             $task = new org_openpsa_projects_task_dba($this->task);
-            $time_slot = (float)$task->get_parameter('org.openpsa.projects.projectbroker', 'minimum_slot');
+            $time_slot = (float) $task->get_parameter('org.openpsa.projects.projectbroker', 'minimum_slot');
             if (empty($time_slot)) {
                 $time_slot = (float) midcom_baseclasses_components_configuration::get('org.openpsa.projects', 'config')->get('default_minimum_time_slot');
                 if (empty($time_slot)) {