flack / openpsa

lib/org/openpsa/mypage/handler/workingon.php

Braces +2 added lines, -1 removed lines

@@ -92,7 +92,8 @@
                         $customer = new org_openpsa_contacts_group_dba($customer_id);
                         $customer_label = $customer->official;
                         $customer = $customer_id;
-                    } catch (midcom_error $e) {
+                    }
+                    catch (midcom_error $e) {
                     }
                 }
             }

lib/org/openpsa/documents/directory.php

Spacing +2 added lines, -2 removed lines

@@ -20,7 +20,7 @@ 
         $ownerwg = $this->get_parameter('org.openpsa.core', 'orgOpenpsaOwnerWg');
         $accesstype = $this->get_parameter('org.openpsa.core', 'orgOpenpsaAccesstype');
 
-        if (   $ownerwg
+        if ($ownerwg
             && $accesstype) {
             // Sync the object's ACL properties into MidCOM ACL system
             $sync = new org_openpsa_core_acl_synchronizer();
@@ -41,7 +41,7 @@ 
     private function _update_parent_timestamp()
     {
         $parent = $this->get_parent();
-        if (   $parent
+        if ($parent
             && $parent->component == 'org.openpsa.documents') {
             midcom::get()->auth->request_sudo('org.openpsa.documents');
 

lib/org/openpsa/httplib/exec/test_post.php

Spacing +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
 
 } else {
     $client = new org_openpsa_httplib();
-    if (   !empty($_REQUEST['username'])
+    if (!empty($_REQUEST['username'])
         && !empty($_REQUEST['password'])) {
         $client->basicauth['user'] = $_REQUEST['username'];
         $client->basicauth['password'] = $_REQUEST['password'];

lib/org/openpsa/mail/exec/test.php

Spacing +1 added lines, -1 removed lines

@@ -1,7 +1,7 @@
 <?php
 midcom::get()->auth->require_admin_user();
 
-if (   empty($_POST['to'])
+if (empty($_POST['to'])
     || empty($_POST['from'])) {
     ?>
     <h2>Send test email</h2>

lib/org/openpsa/notifications/main.php

Braces +2 added lines, -1 removed lines

@@ -52,7 +52,8 @@
         // TODO: Should we sudo here to ensure getting correct prefs regardless of ACLs?
         try {
             $recipient = midcom_db_person::get_cached($recipient);
-        } catch (midcom_error $e) {
+        }
+        catch (midcom_error $e) {
             return false;
         }
 

Spacing +1 added lines, -1 removed lines

@@ -102,7 +102,7 @@
 
         // If user has preference for this message, we use that
         $personal_preferences = $recipient->list_parameters('org.openpsa.notifications');
-        if (   !empty($personal_preferences)
+        if (!empty($personal_preferences)
             && array_key_exists("{$component}:{$action}", $personal_preferences)) {
             return $personal_preferences[$action];
         }

lib/org/openpsa/invoices/scheduler.php

Braces +2 added lines, -1 removed lines

@@ -126,7 +126,8 @@
         $salesproject = org_openpsa_sales_salesproject_dba::get_cached($this->_deliverable->salesproject);
         try {
             $owner = midcom_db_person::get_cached($salesproject->owner);
-        } catch (midcom_error $e) {
+        }
+        catch (midcom_error $e) {
             $e->log();
             return;
         }

Spacing +2 added lines, -2 removed lines

@@ -95,7 +95,7 @@ 
         }
 
         // TODO: Warehouse management: create new order
-        if (   $this->_deliverable->end < $next_cycle_start
+        if ($this->_deliverable->end < $next_cycle_start
             && $this->_deliverable->end != 0) {
             debug_add('Do not register next cycle, the contract ends before');
             return $this->_deliverable->end_subscription();
@@ -328,7 +328,7 @@ 
         }
 
         //If previous cycle was run at the end of the month, the new one should be at the end of the month as well
-        if (   $date->format('t') == $date->format('j')
+        if ($date->format('t') == $date->format('j')
             && $new_date->format('t') != $new_date->format('j')) {
             $new_date->setDate((int) $new_date->format('Y'), (int) $new_date->format('m'), (int) $new_date->format('t'));
         }

lib/org/openpsa/sales/handler/deliverable/view.php

Braces +2 added lines, -1 removed lines

@@ -51,7 +51,8 @@
         }
         try {
             $this->_request_data['product'] = org_openpsa_products_product_dba::get_cached($this->_deliverable->product);
-        } catch (midcom_error $e) {
+        }
+        catch (midcom_error $e) {
             $this->_request_data['product'] = false;
         }
     }

lib/org/openpsa/sales/handler/list.php

Braces +2 added lines, -1 removed lines

@@ -113,7 +113,8 @@
         try {
             $this->_request_data['customer'] = new org_openpsa_contacts_group_dba($guid);
             $qb->add_constraint('customer', '=', $this->_request_data['customer']->id);
-        } catch (midcom_error $e) {
+        }
+        catch (midcom_error $e) {
             $this->_request_data['customer'] = new org_openpsa_contacts_person_dba($guid);
             $qb->add_constraint('customerContact', '=', $this->_request_data['customer']->id);
         }

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

Braces +2 added lines, -1 removed lines

@@ -159,7 +159,8 @@
 
         try {
             $object = new $classname($guid);
-        } catch (midcom_error $e) {
+        }
+        catch (midcom_error $e) {
             $handler->print_error($type . " {$guid} not found, error " . $e->getMessage());
             return false;
         }

Spacing +4 added lines, -4 removed lines

@@ -77,11 +77,11 @@ 
 
     public function resolve_object_link(midcom_db_topic $topic, midcom_core_dbaobject $object) : ?string
     {
-        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}/";
         }
@@ -110,8 +110,8 @@ 
             // We have a feed URL, but we should check if it is GeoRSS as well
             $items = net_nemein_rss_fetch::raw_fetch($data['rss_url'])->get_items();
 
-            if (   !empty($items)
-                && (   $items[0]->get_latitude()
+            if (!empty($items)
+                && ($items[0]->get_latitude()
                     || $items[0]->get_longitude())) {
                 // This is a GeoRSS feed
                 $data['georss_url'] = $data['rss_url'];