flack / openpsa

lib/org/openpsa/sales/validator.php

Spacing +2 added lines, -2 removed lines

@@ -20,7 +20,7 @@ 
         if ($result === true) {
             $result = [];
         }
-        if (   empty($fields['end'])
+        if (empty($fields['end'])
             && empty($fields['continuous'])) {
             $result['end'] = midcom::get()->i18n->get_string('select either end date or continuous', 'org.openpsa.sales');
         }
@@ -33,7 +33,7 @@ 
     public function validate_units(array $fields)
     {
         $result = [];
-        if (   empty($fields['invoiceByActualUnits'])
+        if (empty($fields['invoiceByActualUnits'])
             && empty($fields['plannedUnits'])) {
             $result['plannedUnits'] = midcom::get()->i18n->get_string('select either planned units or invoice by actual units', 'org.openpsa.sales');
             return $result;

lib/org/openpsa/invoices/invoice.php

Spacing +1 added lines, -1 removed lines

@@ -237,7 +237,7 @@
                 $invoice_item->pricePerUnit = $deliverable->pricePerUnit;
                 $invoice_item->deliverable = $deliverable->id;
                 //calculate price
-                if (   $deliverable->invoiceByActualUnits
+                if ($deliverable->invoiceByActualUnits
                     || $deliverable->plannedUnits == 0) {
                     $invoice_item->units = $hours;
                 } else {

lib/midcom/core/account.php

Spacing +1 added lines, -1 removed lines

@@ -194,7 +194,7 @@
             return false;
         }
 
-        if (   !empty($this->_old_username)
+        if (!empty($this->_old_username)
             && $this->_old_username !== $new_username) {
             if (!$history = @unserialize($this->_person->get_parameter('midcom', 'username_history'))) {
                 $history = [];

lib/org/openpsa/expenses/hour/report.php

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)) {

lib/org/openpsa/core/siteconfig.php

Spacing +1 added lines, -1 removed lines

@@ -174,7 +174,7 @@
         }
         if (!midcom::get()->auth->admin) {
             $user_id = midcom::get()->auth->acl->get_user_id();
-            if (   !$this->data[$type . '_guid']
+            if (!$this->data[$type . '_guid']
                 || !midcom::get()->auth->acl->can_do_byguid('midgard:read', $this->data[$type . '_guid'], midcom_db_topic::class, $user_id)) {
                 return null;
             }

lib/org/openpsa/widgets/ui.php

Spacing +3 added lines, -3 removed lines

@@ -39,8 +39,8 @@ 
             $config = array_merge($defaults, $config);
 
             $node_url = $siteconfig->get_node_full_url($component);
-            if (   $node_url
-                && (   !$user_id
+            if ($node_url
+                && (!$user_id
                     || midcom::get()->auth->acl->can_do_byguid('midgard:read', $siteconfig->get_node_guid($component), midcom_db_topic::class, $user_id))) {
                 $providers[] = [
                     'placeholder' => midcom::get()->i18n->get_string('search title', $component),
@@ -115,7 +115,7 @@ 
         echo '<div id="tabs">';
         echo "\n<ul>\n";
         foreach ($tabdata as $key => $tab) {
-            echo "<li><a id='key_" . $key ."' class='tabs_link' href='" . $prefix . $tab['url'] . "' ><span> " . $tab['title'] . "</span></a></li>";
+            echo "<li><a id='key_" . $key . "' class='tabs_link' href='" . $prefix . $tab['url'] . "' ><span> " . $tab['title'] . "</span></a></li>";
         }
         echo "\n</ul>\n";
         echo "</div>\n";

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

Spacing +2 added lines, -2 removed lines

@@ -17,7 +17,7 @@ 
     public function _on_watched_dba_delete(midcom_core_dbaobject $object)
     {
         $qb = org_openpsa_directmarketing_campaign_member_dba::new_query_builder();
-        if (   $object instanceof midcom_db_person
+        if ($object instanceof midcom_db_person
             || $object instanceof org_openpsa_contacts_person_dba) {
             $qb->add_constraint('person', '=', $object->id);
         } elseif ($object instanceof org_openpsa_directmarketing_campaign_dba) {
@@ -42,7 +42,7 @@ 
      */
     public function background_send_message(array $args, midcom_baseclasses_components_cron_handler $handler)
     {
-        if (   !isset($args['url_base'])
+        if (!isset($args['url_base'])
             || !isset($args['batch'])) {
             $handler->print_error('url_base or batch number not set, aborting');
             return false;

lib/org/openpsa/products/viewer.php

Spacing +3 added lines, -3 removed lines

@@ -32,11 +32,11 @@
         $object = $dm->get_storage()->get_value();
 
         $document = $indexer->new_document($dm);
-        if (   $config->get('enable_scheduling')
+        if ($config->get('enable_scheduling')
             && is_a($object, org_openpsa_products_product_dba::class)) {
             // Check start/end for products
-            if (   $object->start > time()
-                || (   $object->end != 0
+            if ($object->start > time()
+                || ($object->end != 0
                     && $object->end < time())) {
                 // Not in market, remove from index
                 $indexer->delete($document->RI);

lib/midcom/services/permalinks/main.php

Spacing +1 added lines, -1 removed lines

@@ -93,7 +93,7 @@
         if ($object instanceof midcom_db_attachment) {
             // Faster linking to attachments
             $parent = $object->get_parent();
-            if (   is_a($parent, midcom_db_topic::class)
+            if (is_a($parent, midcom_db_topic::class)
                 && $nav->is_node_in_tree($parent->id, $nav->get_root_node())) {
                 $napobj = $nav->get_node($parent->id);
                 return $napobj[MIDCOM_NAV_FULLURL] . $object->name;