flack / openpsa

lib/midcom/helper/misc.php

Braces +2 added lines, -1 removed lines

@@ -55,7 +55,8 @@
     {
         try {
             return eval("return [{$data}\n];");
-        } catch (ParseError $e) {
+        }
+        catch (ParseError $e) {
             throw new midcom_error('Failed to parse config data: ' . $e->getMessage() . ' in ' . $path . ' line ' . $e->getLine());
         }
     }

src/midcom/datamanager/extension/type/captchaType.php

Spacing +2 added lines, -2 removed lines

@@ -43,11 +43,11 @@
     public function buildForm(FormBuilderInterface $builder, array $options)
     {
         $session_key = md5($builder->getForm()->getName() . '_session_key');
-        $builder->addEventListener(FormEvents::POST_SUBMIT, function (FormEvent $event) use ($session_key) {
+        $builder->addEventListener(FormEvents::POST_SUBMIT, function(FormEvent $event) use ($session_key) {
             $value = $event->getForm()->getData();
             $session = new midcom_services_session('midcom_datamanager_captcha');
 
-            if (   !$session->exists($session_key)
+            if (!$session->exists($session_key)
                 || $value != $session->get($session_key)) {
                 $event->getForm()->addError(new FormError($this->l10n->get('captcha validation failed')));
             }

src/midcom/datamanager/extension/formExtension.php

Spacing +1 added lines, -1 removed lines

@@ -65,7 +65,7 @@
         $view->vars['readonly'] = $options['readonly'];
 
         if ($options['write_privilege'] !== null) {
-            if (   array_key_exists('group', $options['write_privilege'])
+            if (array_key_exists('group', $options['write_privilege'])
                 && !$this->auth->is_group_member($options['write_privilege']['group'])) {
                 $view->vars['readonly'] = true;
             }

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