flack / openpsa

lib/midcom/response/relocate.php

Spacing +1 added lines, -1 removed lines

@@ -27,7 +27,7 @@
      */
     public function setTargetUrl(string $url) : static
     {
-        if (   !str_starts_with($url, "/")
+        if (!str_starts_with($url, "/")
             && !preg_match('|^https?://|', $url)) {
             $prefix = midcom_core_context::get()->get_key(MIDCOM_CONTEXT_ANCHORPREFIX) ?: '/';
             $url = $prefix . $url;

src/midcom/grid/provider.php

Spacing +4 added lines, -4 removed lines

@@ -75,7 +75,7 @@ 
      */
     private array $_search = [];
 
-    public function __construct(array|client $source, string $datatype = 'json')
+    public function __construct(array | client $source, string $datatype = 'json')
     {
         $this->_datatype = $datatype;
         if ($source instanceof client) {
@@ -202,7 +202,7 @@ 
 
     private function _convert_to_localdata() : string
     {
-        return "var " . $this->_grid->get_identifier() . '_entries = ' .  json_encode($this->get_rows()) . ";\n";
+        return "var " . $this->_grid->get_identifier() . '_entries = ' . json_encode($this->get_rows()) . ";\n";
     }
 
     private function _render_json()
@@ -233,7 +233,7 @@ 
             $this->_sort_field = $query['sidx'];
             $this->_sort_direction = strtoupper($query['sord'] ?? 'ASC');
         }
-        if (   !empty($query['_search'])
+        if (!empty($query['_search'])
             && $query['_search'] === 'true') {
             foreach ($query as $field => $value) {
                 if (in_array($field, ['_search', 'nd', 'page', 'rows', 'sidx', 'sord'])) {
@@ -256,7 +256,7 @@ 
 
         $this->_total_rows = $query->count();
 
-        if (   $this->_datatype == 'json'
+        if ($this->_datatype == 'json'
             && !empty($this->_results_per_page)) {
             $query->set_limit($this->_results_per_page);
             if (!empty($this->_offset)) {

lib/midcom/db/attachment.php

Spacing +2 added lines, -2 removed lines

@@ -155,7 +155,7 @@ 
         return "{$cacheroot}/{$subdir}/{$this->guid}/{$this->name}";
     }
 
-    public static function get_url(midgard_attachment|midcom_db_attachment|string $attachment, ?string $name = null) : string
+    public static function get_url(midgard_attachment | midcom_db_attachment | string $attachment, ?string $name = null) : string
     {
         if (is_string($attachment)) {
             $guid = $attachment;
@@ -301,7 +301,7 @@ 
     public function update_cache()
     {
         // Check if the attachment can be read anonymously
-        if (   midcom::get()->config->get('attachment_cache_enabled')
+        if (midcom::get()->config->get('attachment_cache_enabled')
             && !$this->can_do('midgard:read', 'EVERYONE')) {
             // Not public file, ensure it is removed
             $this->remove_from_cache();

lib/org/openpsa/user/handler/person/create.php

Spacing +1 added lines, -1 removed lines

@@ -59,7 +59,7 @@
 
     public function _handler_create(Request $request, array &$data, ?string $guid = null)
     {
-        midcom::get()->auth->require_user_do('org.openpsa.user:manage', class: org_openpsa_user_interface::class);
+        midcom::get()->auth->require_user_do('org.openpsa.user:manage', class : org_openpsa_user_interface::class);
 
         if ($guid !== null) {
             // Get the organization

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

Spacing +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
 
     public function _handler_create(Request $request, ?string $guid = null)
     {
-        midcom::get()->auth->require_user_do('midgard:create', class: org_openpsa_contacts_person_dba::class);
+        midcom::get()->auth->require_user_do('midgard:create', class : org_openpsa_contacts_person_dba::class);
 
         $defaults = [];
         if (!empty($guid)) {

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

Spacing +2 added lines, -2 removed lines

@@ -37,7 +37,7 @@ 
 
     public function _handler_new(Request $request, ?string $guid = null)
     {
-        midcom::get()->auth->require_user_do('midgard:create', class: org_openpsa_sales_salesproject_dba::class);
+        midcom::get()->auth->require_user_do('midgard:create', class : org_openpsa_sales_salesproject_dba::class);
 
         $this->_salesproject = new org_openpsa_sales_salesproject_dba;
 
@@ -48,7 +48,7 @@ 
         $schemadb = schemadb::from_path($this->_config->get('schemadb_salesproject'));
 
         if ($guid) {
-            $field =& $schemadb->get('default')->get_field('customer');
+            $field = & $schemadb->get('default')->get_field('customer');
             try {
                 $customer = new org_openpsa_contacts_group_dba($guid);
                 $field['type_config']['options'] = [0 => '', $customer->id => $customer->official];

Braces +4 added lines, -2 removed lines

@@ -54,7 +54,8 @@ 
                 $field['type_config']['options'] = [0 => '', $customer->id => $customer->official];
 
                 $defaults['customer'] = $customer->id;
-            } catch (midcom_error) {
+            }
+            catch (midcom_error) {
                 $customer = new org_openpsa_contacts_person_dba($guid);
                 $defaults['customerContact'] = $customer->id;
                 $field['type_config']['options'] = $this->list_groups([$customer->id => true]);
@@ -126,7 +127,8 @@ 
             try {
                 $company = new org_openpsa_contacts_group_dba($company_id);
                 $ret[$company->id] = $company->get_label();
-            } catch (midcom_error $e) {
+            }
+            catch (midcom_error $e) {
                 $e->log();
             }
         }

lib/org/openpsa/directmarketing/handler/import.php

Spacing +2 added lines, -2 removed lines

@@ -138,7 +138,7 @@ 
 
         if ($contacts = $importer->parse($input)) {
             $this->_request_data['import_status'] = $importer->import_subscribers($contacts, $this->_request_data['campaign']);
-            if (   $this->_request_data['import_status']['subscribed_new'] > 0
+            if ($this->_request_data['import_status']['subscribed_new'] > 0
                 || $this->_request_data['import_status']['already_subscribed'] > 0) {
                 $this->_import_success = true;
             }
@@ -195,7 +195,7 @@ 
                 $read_rows = 0;
                 $handle = fopen($data['tmp_file'], 'r');
                 $total_columns = 0;
-                while (   $read_rows < 2
+                while ($read_rows < 2
                        && $csv_line = fgetcsv($handle, 1000, $data['separator'])) {
                     if ($total_columns == 0) {
                         $total_columns = count($csv_line);

lib/org/openpsa/sales/validator.php

Spacing +2 added lines, -2 removed lines

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

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

Spacing +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
             $schema->get_field('next_cycle')['hidden'] = false;
             $schema->get_field('next_cycle')['default'] = $entry->start;
             $schema->get_field('at_entry')['default'] = $entry->id;
-        } elseif (   ($this->_deliverable->continuous || $this->_deliverable->end > time())
+        } elseif (($this->_deliverable->continuous || $this->_deliverable->end > time())
                   && $this->_deliverable->state == org_openpsa_sales_salesproject_deliverable_dba::STATE_STARTED) {
             $schema->get_field('next_cycle')['hidden'] = false;
         }