flack / openpsa

lib/midcom/style/dm2_config_recreate.php

Spacing +1 added lines, -1 removed lines

@@ -9,7 +9,7 @@
         continue;
     }
 
-    if (   !$object->can_do('midgard:update')
+    if (!$object->can_do('midgard:update')
         || !$object->can_do('midgard:attachments')) {
         echo sprintf(midcom::get()->i18n->get_string('not recreating object %s %s, reason %s', 'midcom'), $type, $object->guid, 'Insufficient privileges') . "\n";
         continue;

lib/midgard/admin/asgard/style/midgard_admin_asgard_welcome.php

Spacing +2 added lines, -2 removed lines

@@ -30,7 +30,7 @@ 
                     <?php
                     foreach ($type_choices as $value => $label) {
                         $selected = '';
-                        if (   isset($data['type_filter'])
+                        if (isset($data['type_filter'])
                             && $data['type_filter'] == $value) {
                             $selected = ' selected="selected"';
                         }
@@ -45,7 +45,7 @@ 
                     <?php
                     foreach ($revised_after_choices as $value => $label) {
                         $selected = '';
-                        if (   isset($data['revised_after'])
+                        if (isset($data['revised_after'])
                             && $data['revised_after'] == date('Y-m-d', $value)) {
                             $selected = ' selected="selected"';
                         }

lib/midcom/baseclasses/components/plugin.php

Spacing +1 added lines, -1 removed lines

@@ -18,7 +18,7 @@
     public function initialize(midcom_baseclasses_components_viewer $master, Router $router)
     {
         $this->router = $router;
-        $this->_request_data =& $master->_request_data;
+        $this->_request_data = & $master->_request_data;
         $this->_topic = $master->_topic;
         $this->_request_data['l10n'] = $this->_l10n;
 

lib/midcom/helper/imagepopup/handler/list.php

Spacing +1 added lines, -1 removed lines

@@ -33,7 +33,7 @@
         midcom::get()->auth->require_valid_user();
         midcom::get()->skip_page_style = true;
 
-        $this->add_stylesheet(MIDCOM_STATIC_URL ."/midcom.helper.imagepopup/styling.css", 'screen');
+        $this->add_stylesheet(MIDCOM_STATIC_URL . "/midcom.helper.imagepopup/styling.css", 'screen');
 
         $data['filetype'] = $filetype;
         $data['object'] = null;

src/midcom/datamanager/controller.php

Spacing +3 added lines, -3 removed lines

@@ -70,13 +70,13 @@ 
             }
         } elseif (!$this->form->isSubmitted()) {
             $this->form->handleRequest($request);
-            if (   $this->form->isSubmitted()
+            if ($this->form->isSubmitted()
                 && $button = $this->form->getClickedButton()) {
                 $operation = $button->getConfig()->getOption('operation');
             }
         }
 
-        if (   in_array($operation, [self::SAVE, self::NEXT])
+        if (in_array($operation, [self::SAVE, self::NEXT])
             && !$this->form->isValid()) {
             $operation = self::EDIT;
         }
@@ -159,7 +159,7 @@ 
                 if ($storage->get_last_operation() === self::SAVE) {
                     // Get GUIDs and such for new-created dependent objects
                     $this->dm->set_storage($storage->get_value(), $this->dm->get_schema()->get_name());
-                } elseif (   !$this->form->isValid()
+                } elseif (!$this->form->isValid()
                           && $storage->move_uploaded_files() > 0) {
                     $form = $this->dm->get_form($this->form->getName(), true);
                     $this->copy_errors($this->form, $form);

lib/org/openpsa/projects/style/show-frontpage.php

Spacing +1 added lines, -1 removed lines

@@ -68,7 +68,7 @@
             echo "        <td class=\"numeric\"><span title=\"" . $data['l10n']->get('ongoing') . "\">{$task_count['ongoing']}</span></td>\n";
             echo "        <td class=\"numeric\"><span title=\"" . $data['l10n']->get('on_hold') . "\">{$task_count['on_hold']}</span></td>\n";
             echo "        <td class=\"numeric\"><span title=\"" . $data['l10n']->get('closed') . "\">{$task_count['closed']}</span></td>\n";
-            echo "        <td class=\"numeric\"> " ;
+            echo "        <td class=\"numeric\"> ";
             echo "            <span title=\"" . $data['l10n']->get('reported') . "\">" . round($hours['reportedHours'], 2) . "</span>";
             if ($hours['plannedHours'] > 0) {
                 echo          " / <span title=\"" . $data['l10n']->get('planned hours') . "\">" . round($hours['plannedHours'], 2) . "</span>";

lib/midcom/admin/folder/management.php

Spacing +1 added lines, -1 removed lines

@@ -50,7 +50,7 @@
             }
 
             // Skip components beginning with midcom or midgard
-            if (   preg_match('/^(midcom|midgard)\./', $manifest->name)
+            if (preg_match('/^(midcom|midgard)\./', $manifest->name)
                 && $manifest->name != 'midcom.helper.search') {
                 continue;
             }

lib/org/openpsa/sales/style/show-deliverable.php

Spacing +3 added lines, -3 removed lines

@@ -3,7 +3,7 @@ 
 $state = $data['deliverable']->get_state();
 $formatter = $data['l10n']->get_formatter();
 $per_unit = $data['l10n']->get('per unit');
-if (   $data['product']
+if ($data['product']
     && $unit_option = org_openpsa_sales_viewer::get_unit_option($data['product']->unit)) {
     $per_unit = sprintf($data['l10n']->get('per %s'), $unit_option);
 }
@@ -115,7 +115,7 @@ 
 	    </div>
      <?php
     $tabs = [];
-    if (   $data['invoices_url']
+    if ($data['invoices_url']
         && $data['deliverable']->invoiced > 0) {
         $tabs[] = [
             'url' => $data['invoices_url'] . "list/deliverable/{$data['deliverable']->guid}/",
@@ -123,7 +123,7 @@ 
         ];
     }
 
-    if (   $data['projects_url']
+    if ($data['projects_url']
         && $data['deliverable']->state >= org_openpsa_sales_salesproject_deliverable_dba::STATE_ORDERED
         && $data['product']
         && $data['product']->orgOpenpsaObtype == org_openpsa_products_product_dba::TYPE_SERVICE) {

lib/org/openpsa/calendar/conflictmanager.php

Spacing +4 added lines, -4 removed lines

@@ -125,7 +125,7 @@ 
         }
         // TODO: Shared tasks need a separate check (different member object)
 
-        if (   !empty($this->busy_members)
+        if (!empty($this->busy_members)
             || !empty($this->busy_resources)) {
             //Unresolved conflicts (note return value is for conflicts not lack of them)
             midcom::get()->auth->drop_sudo();
@@ -188,7 +188,7 @@ 
         }
 
         if (array_key_exists($member->event, $modified_events)) {
-            $event =& $modified_events[$member->event];
+            $event = & $modified_events[$member->event];
             $set_as_modified = false;
         } else {
             try {
@@ -202,7 +202,7 @@ 
         }
         debug_add("overlap found in event {$event->title} (#{$event->id})");
 
-        if (   $event->tentative
+        if ($event->tentative
             && $rob_tentative) {
             debug_add('event is tentative, robbing resources');
             $event->resources = array_diff_key($event->resources, $this->_event->resources);
@@ -229,7 +229,7 @@ 
         }
         debug_add("overlap found in event {$event->title} (#{$event->id})");
 
-        if (   $event->tentative
+        if ($event->tentative
             && $rob_tentative) {
             debug_add('event is tentative, robbing participants');
             $event->participants = array_diff_key($event->participants, $this->_event->participants);