flack / openpsa

lib/midcom/response/relocate.php

Spacing +1 added lines, -1 removed lines

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

lib/org/openpsa/contacts/handler/search.php

Spacing +1 added lines, -1 removed lines

@@ -82,7 +82,7 @@
 
         // Handle automatic wildcards
         $auto_wildcards = $this->_config->get('auto_wildcards');
-        if (   $auto_wildcards
+        if ($auto_wildcards
             && !str_contains($this->_query_string_processed, '%')) {
             switch ($auto_wildcards) {
                 case 'both':

lib/org/openpsa/directmarketing/support/bounce_detector.php

Spacing +2 added lines, -2 removed lines

@@ -45,7 +45,7 @@ 
 }
 $recipient = $_ENV["RECIPIENT"];
 $token_length = strlen($recipient) - strlen($prefix) - strlen($suffix);
-if (   $token_length <= 0
+if ($token_length <= 0
     || !str_starts_with($recipient, $prefix)
     || !str_ends_with($recipient, $suffix)) {
     error_log("Recipient address does not match address template: $recipient");
@@ -61,7 +61,7 @@ 
         fclose($fh);
         exit(0);
     }
-} elseif (   str_starts_with($BOUNCE_LOGGER, "http://")
+} elseif (str_starts_with($BOUNCE_LOGGER, "http://")
           || str_starts_with($BOUNCE_LOGGER, "https://")) {
     $client = new org_openpsa_httplib();
     $client->post($BOUNCE_LOGGER, ["token" => $token]);

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/midgard/admin/asgard/handler/welcome.php

Spacing +2 added lines, -2 removed lines

@@ -33,7 +33,7 @@ 
                 $qb = new midcom_core_querybuilder($midcom_dba_classname);
                 $qb->add_constraint('metadata.revised', '>=', $since);
 
-                if (   $only_mine
+                if ($only_mine
                     && midcom::get()->auth->user) {
                     $qb->add_constraint('metadata.authors', 'LIKE', '|' . midcom::get()->auth->user->guid . '|');
                 }
@@ -198,7 +198,7 @@ 
                 continue;
             }
 
-            if (   $object->can_do('midgard:update')
+            if ($object->can_do('midgard:update')
                 && $object->can_do('midcom:approve')) {
                 $object->metadata->approve();
                 midcom::get()->uimessages->add($this->_l10n->get('midgard.admin.asgard'), sprintf($this->_l10n->get('object %s approved'), $object->guid));

Braces +6 added lines, -3 removed lines

@@ -112,7 +112,8 @@ 
             $row['approved'] = $object->is_approved() ? $object->metadata->approved : '';
             try {
                 $row['index_title'] = $reflector->get_object_label($object) ?: '[' . $this->_l10n->get('no title') . ']';
-            } catch (Exception $e) {
+            }
+            catch (Exception $e) {
                 $row['index_title'] = 'ERROR: ' . $e->getMessage();
             }
             $link = $this->router->generate('object_' . $this->_request_data['default_mode'], ['guid' => $object->guid]);
@@ -177,7 +178,8 @@ 
         foreach ($guids as $guid) {
             try {
                 $object = midcom::get()->dbfactory->get_object_by_guid($guid);
-            } catch (midcom_error $e) {
+            }
+            catch (midcom_error $e) {
                 continue;
             }
 
@@ -192,7 +194,8 @@ 
         foreach ($guids as $guid) {
             try {
                 $object = midcom::get()->dbfactory->get_object_by_guid($guid);
-            } catch (midcom_error $e) {
+            }
+            catch (midcom_error $e) {
                 continue;
             }
 

lib/midgard/admin/user/style/midgard-admin-user-group-folders.php

Braces +2 added lines, -1 removed lines

@@ -17,7 +17,8 @@
         foreach ($data['objects'] as $guid => $privs) {
             try {
                 $object = new midcom_db_topic($guid);
-            } catch (midcom_error $e) {
+            }
+            catch (midcom_error $e) {
                 continue;
             }
             echo "<tr>\n";

lib/org/openpsa/calendar/event/member.php

Spacing +3 added lines, -3 removed lines

@@ -68,7 +68,7 @@ 
             $event = new org_openpsa_calendar_event_dba($this->eid);
         }
 
-        if (    $recipient->id == midcom_connection::get_user()
+        if ($recipient->id == midcom_connection::get_user()
              && !$event->send_notify_me) {
             //Do not send notification to current user
             debug_add('event->send_notify_me is false and recipient is current user, aborting notify');
@@ -194,7 +194,7 @@ 
 
             if (isset($events_by_date[$ymd])) {
                 foreach ($events_by_date[$ymd] as $event) {
-                    if (   $event->end <= $workday_starts_ts
+                    if ($event->end <= $workday_starts_ts
                         || $event->start >= $workday_ends_ts) {
                         // We need not consider this event, it is outside the defined workday
                         continue;
@@ -212,7 +212,7 @@ 
                 }
             }
             // End of day slot
-            if (   $last_end_time < $workday_ends_ts
+            if ($last_end_time < $workday_ends_ts
                 && (($workday_ends_ts - $last_end_time) >= $amount)) {
                 $slots[] = self::_create_slot($last_end_time, $workday_ends_ts, $last_event);
             }

Braces +2 added lines, -1 removed lines

@@ -143,7 +143,8 @@
                 debug_add('person #' . $person->id . 'has no email address, aborting');
                 return null;
             }
-        } catch (midcom_error $e) {
+        }
+        catch (midcom_error $e) {
             return null;
         }
 

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,