flack / openpsa

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/main.php

Spacing +1 added lines, -1 removed lines

@@ -115,7 +115,7 @@
         $request->addHeader('User-Agent: Midgard/' . substr(mgd_version(), 0, 4));
 
         // Handle basic auth
-        if (   !empty($username)
+        if (!empty($username)
             && !empty($password)) {
             // Set basic auth
             $request->addHeader('Authorization: Basic ' . base64_encode($username . ':' . $password));

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/backend.php

Spacing +1 added lines, -1 removed lines

@@ -79,7 +79,7 @@
             return false;
         }
 
-        if (   is_string($this->error)
+        if (is_string($this->error)
             && !empty($this->error)) {
             return $this->error;
         }

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/mail/template.php

Spacing +1 added lines, -1 removed lines

@@ -111,7 +111,7 @@
         foreach ($this->_parameters as $key => $value) {
             $this->_patterns[] = "/__({$key})__/";
 
-            if (   is_array($value)
+            if (is_array($value)
                 || is_object($value)) {
                 $this->_patterns[] = "/__({$key})_([^ \.>\"-]*?)__/";
             }

lib/org/openpsa/invoices/handler/goto.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
             return $this->fail($this->_l10n->get('no invoice number was handed over'));
         }
 
-        $invoicenumber = (int) $_GET['query'] ;
+        $invoicenumber = (int) $_GET['query'];
 
         if ($invoice = org_openpsa_invoices_invoice_dba::get_by_number($invoicenumber)) {
             return new midcom_response_relocate('invoice/' . $invoice->guid . '/');

lib/org/openpsa/contacts/handler/duplicates/person.php

Spacing +10 added lines, -10 removed lines

@@ -43,7 +43,7 @@ 
 
     private function load_next()
     {
-        $i =& $this->_request_data['loop_i'];
+        $i = & $this->_request_data['loop_i'];
         while ($i < 100) {
             debug_add("Loop iteration {$i}");
             $qb = new midgard_query_builder('midgard_parameter');
@@ -71,7 +71,7 @@ 
                 continue;
             }
             // Make sure we actually have enough rights to do this
-            if (   !$person1->can_do('midgard:update')
+            if (!$person1->can_do('midgard:update')
                 || !$person1->can_do('midgard:delete')
                 || !$person2->can_do('midgard:update')
                 || !$person2->can_do('midgard:delete')) {
@@ -80,14 +80,14 @@ 
                 continue;
             }
             // Extra sanity check (in case of semi-successful not-duplicate mark)
-            if (   $person1->get_parameter('org.openpsa.contacts.duplicates:not_duplicate', $person2->guid)
+            if ($person1->get_parameter('org.openpsa.contacts.duplicates:not_duplicate', $person2->guid)
                 || $person2->get_parameter('org.openpsa.contacts.duplicates:not_duplicate', $person1->guid)) {
                 debug_add("It seems these two (#{$person1->id} and #{$person2->id}) have also marked as not duplicates, some cleanup might be a good thing", MIDCOM_LOG_WARN);
                 $i++;
                 continue;
             }
 
-            $this->_request_data['probability'] = (float)$param->value;
+            $this->_request_data['probability'] = (float) $param->value;
             $this->_request_data['person1'] = $person1;
             $this->_request_data['person2'] = $person2;
             break;
@@ -96,7 +96,7 @@ 
 
     private function process_submit()
     {
-        if (   !empty($_POST['org_openpsa_contacts_handler_duplicates_person_keep'])
+        if (!empty($_POST['org_openpsa_contacts_handler_duplicates_person_keep'])
             && !empty($_POST['org_openpsa_contacts_handler_duplicates_person_options'])
             && count($_POST['org_openpsa_contacts_handler_duplicates_person_options']) == 2) {
             $option1 = new org_openpsa_contacts_person_dba($_POST['org_openpsa_contacts_handler_duplicates_person_options'][1]);
@@ -105,7 +105,7 @@ 
             if ($keep == 'both') {
                 $option1->require_do('midgard:update');
                 $option2->require_do('midgard:update');
-                if (   $option1->set_parameter('org.openpsa.contacts.duplicates:not_duplicate', $option2->guid, time())
+                if ($option1->set_parameter('org.openpsa.contacts.duplicates:not_duplicate', $option2->guid, time())
                     && $option2->set_parameter('org.openpsa.contacts.duplicates:not_duplicate', $option1->guid, time())) {
                     // Clear the possible duplicate parameters
                     $option1->delete_parameter('org.openpsa.contacts.duplicates:possible_duplicate', $option2->guid);
@@ -123,11 +123,11 @@ 
                 }
             } else {
                 if ($keep == $option1->guid) {
-                    $person1 =& $option1;
-                    $person2 =& $option2;
+                    $person1 = & $option1;
+                    $person2 = & $option2;
                 } elseif ($keep == $option2->guid) {
-                    $person1 =& $option2;
-                    $person2 =& $option1;
+                    $person1 = & $option2;
+                    $person2 = & $option1;
                 } else {
                     throw new midcom_error('Something weird happened (basically we got bogus data)');
                 }

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

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']);