flack / openpsa

src/midcom/routing/parser.php

Spacing +2 added lines, -2 removed lines

@@ -46,7 +46,7 @@ 
         $url = $this->context->get_key(MIDCOM_CONTEXT_URI);
 
         $prefix_length = strlen(midcom_connection::get_url('prefix'));
-        if (   $prefix_length > 1
+        if ($prefix_length > 1
             && str_starts_with($url, midcom_connection::get_url('prefix'))) {
             $url = substr($url, $prefix_length);
         }
@@ -71,7 +71,7 @@ 
      */
     public function find_urlmethod() : ?string
     {
-        if (   empty($this->argv)
+        if (empty($this->argv)
             || !str_starts_with($this->argv[0], 'midcom-')) {
             return null;
         }

src/midcom/datamanager/extension/transformer/imageTransformer.php

Spacing +1 added lines, -1 removed lines

@@ -47,7 +47,7 @@
         }
 
         $result = [];
-        if (   !empty($array['file'])
+        if (!empty($array['file'])
             || !empty($array['identifier']) && str_starts_with($array['identifier'], 'tmpfile-')) {
             $result['file'] = parent::reverseTransform($array);
         } elseif (!empty($array['objects'])) {

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

Spacing +2 added lines, -2 removed lines

@@ -30,7 +30,7 @@ 
     private function load_controller(org_openpsa_invoices_billing_data_dba $bd) : controller
     {
         $schemadb = schemadb::from_path($this->_config->get('schemadb_billing_data'));
-        $vat =& $schemadb->get('default')->get_field('vat');
+        $vat = & $schemadb->get('default')->get_field('vat');
         // Fill VAT select
         if ($options = $this->get_vat_options($this->_config->get('vat_percentages'))) {
             $vat['type_config']['options'] = $options;
@@ -68,7 +68,7 @@ 
         $data['controller'] = $this->load_controller($bd);
 
         $workflow = $this->get_workflow('datamanager', ['controller' => $data['controller']]);
-        if (   $mode == 'edit'
+        if ($mode == 'edit'
             && $bd->can_do('midgard:delete')) {
             $delete = $this->get_workflow('delete', [
                 'object' => $bd,

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

Braces +4 added lines, -2 removed lines

@@ -79,7 +79,8 @@ 
                 try {
                     $dms['group']->set_storage($group);
                     org_openpsa_products_viewer::index($dms['group'], $indexer, $topic, $config);
-                } catch (midcom_error $e) {
+                }
+                catch (midcom_error $e) {
                     $e->log(MIDCOM_LOG_WARN);
                 }
             }
@@ -91,7 +92,8 @@ 
                     try {
                         $dms['product']->set_storage($product);
                         org_openpsa_products_viewer::index($dms['product'], $indexer, $topic, $config);
-                    } catch (midcom_error $e) {
+                    }
+                    catch (midcom_error $e) {
                         $e->log(MIDCOM_LOG_WARN);
                     }
                 }

Spacing +1 added lines, -1 removed lines

@@ -52,7 +52,7 @@
      */
     public function _on_reindex($topic, midcom_helper_configuration $config, midcom_services_indexer $indexer)
     {
-        if (   !$config->get('index_products')
+        if (!$config->get('index_products')
             && !$config->get('index_groups')) {
             debug_add("No indexing to groups and products, skipping", MIDCOM_LOG_WARN);
             return true;

lib/org/openpsa/notifications/notifier/email.php

Spacing +1 added lines, -1 removed lines

@@ -38,7 +38,7 @@
             $mail->from = $config->get('default_sender') ?: '"OpenPSA Notifier" <noreply@' . $_SERVER['SERVER_NAME'] . '>';
         }
 
-        $mail->subject = $message['title'] ??  'org.openpsa.notifications message (no title provided)';
+        $mail->subject = $message['title'] ?? 'org.openpsa.notifications message (no title provided)';
         if (array_key_exists('attachments', $message)) {
             $mail->attachments = $message['attachments'];
         }

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

Spacing +1 added lines, -1 removed lines

@@ -40,7 +40,7 @@
 
     public function resolve_object_link(midcom_db_topic $topic, midcom_core_dbaobject $object) : ?string
     {
-        if (   $object instanceof org_openpsa_documents_document_dba
+        if ($object instanceof org_openpsa_documents_document_dba
             && $object->topic == $topic->id) {
             return "document/{$object->guid}/";
         }

lib/net/nemein/wiki/style/view-wikipage-raw.php

Spacing +1 added lines, -1 removed lines

@@ -6,6 +6,6 @@
     &(view["content"]:h);
     <?php
 } else {
-    echo "<p class=\"stub\">" . $data['l10n']->get('this page is stub')."</p>";
+    echo "<p class=\"stub\">" . $data['l10n']->get('this page is stub') . "</p>";
 }
 ?>
\ No newline at end of file

lib/org/openpsa/invoices/style/admin-read.php

Spacing +1 added lines, -1 removed lines

@@ -34,7 +34,7 @@
         <div class="value"><?php echo $formatter->date($invoice->deliverydate); ?></div></div>
     <?php } ?>
 
-    <div class="field"><div class="title"><?php echo $data['l10n_midcom']->get('description');?>: </div>
+    <div class="field"><div class="title"><?php echo $data['l10n_midcom']->get('description'); ?>: </div>
     <div class="description value">&(view['description']:h);</div></div>
 
     <?php

Braces +2 added lines, -1 removed lines

@@ -8,7 +8,8 @@
     if ($customer->orgOpenpsaObtype < org_openpsa_contacts_group_dba::ORGANIZATION) {
         $customer = false;
     }
-} catch (midcom_error) {
+}
+catch (midcom_error) {
     $customer = false;
 }
 

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

Indentation +2 added lines, -2 removed lines

@@ -74,12 +74,12 @@
                         <tr>
                           <td class="title"><?php echo $data['l10n']->get('cost'); ?></td>
                           <?php
-                          if ($data['deliverable']->costType == '%') { ?>
+                            if ($data['deliverable']->costType == '%') { ?>
                               <td class="numeric"><?php echo $view['costPerUnit']; ?> %</td>
                               <td class="numeric">&nbsp;</td>
                               <td class="numeric">&nbsp;</td>
                           <?php
-                          } else { ?>
+                            } else { ?>
                               <td class="numeric"><?php echo $formatter->number($data['deliverable']->costPerUnit); ?></td>
                               <td class="numeric"><?php echo $view['plannedUnits']; ?></td>
                               <td class="numeric"><?php echo $view['units']; ?></td>

Spacing +4 added lines, -4 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);
 }
@@ -109,7 +109,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}/",
@@ -117,7 +117,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) {
@@ -172,7 +172,7 @@ 
             }
             echo "    </tbody>\n";
             echo "</table>\n";
-            if (   $at_entries[0]->status == midcom_services_at_entry_dba::SCHEDULED
+            if ($at_entries[0]->status == midcom_services_at_entry_dba::SCHEDULED
                 && midcom::get()->auth->can_user_do('midgard:create', class: org_openpsa_invoices_invoice_dba::class)) {
                 $label = $data['l10n']->get('generate now');
                 $process_link = $data['router']->generate('deliverable_run_cycle', ['guid' => $data['deliverable']->guid]);