flack / openpsa

lib/midcom/services/auth/main.php

Spacing +3 added lines, -3 removed lines

@@ -203,7 +203,7 @@ 
             return true;
         }
         if ($user === null) {
-            $user =& $this->user;
+            $user = & $this->user;
         }
 
         if ($user == 'EVERYONE') {
@@ -274,7 +274,7 @@ 
      *
      * It always returns true for administrative users.
      */
-    public function is_group_member(midcom_core_group|string $group, ?midcom_core_user $user = null) : bool
+    public function is_group_member(midcom_core_group | string $group, ?midcom_core_user $user = null) : bool
     {
         if ($this->is_admin($user)) {
             // Administrators always have access.
@@ -519,7 +519,7 @@ 
      * (group:...), any valid identifier for the midcom_core_group
      * constructor or a valid object of that type.
      */
-    public function get_group(string|int|midcom_db_group|midgard_group $id) : ?midcom_core_group
+    public function get_group(string | int | midcom_db_group | midgard_group $id) : ?midcom_core_group
     {
         $param = $id;
 

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

lib/org/openpsa/expenses/handler/hours/admin.php

Spacing +1 added lines, -1 removed lines

@@ -133,7 +133,7 @@
         return new midcom_response_relocate($relocate);
     }
 
-    private function parse_input(ParameterBag $input, string $action) : int|bool
+    private function parse_input(ParameterBag $input, string $action) : int | bool
     {
         if (!in_array($action, ['invoiceable', 'invoice', 'task'])) {
             throw new midcom_error('passed action ' . $action . ' is unknown');

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;
         }