flack / openpsa

src/midcom/datamanager/template/view.php

Spacing +1 added lines, -1 removed lines

@@ -143,7 +143,7 @@
 
     public function form_widget_simple(FormView $view, array $data)
     {
-        if (   !empty($data['value'])
+        if (!empty($data['value'])
             || is_numeric($data['value'])) {
             return $this->escape($data['value']);
         }

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

Braces +2 added lines, -1 removed lines

@@ -116,7 +116,8 @@
                 try {
                     $object = $options['widget_config']['class']::get_cached($identifier);
                     $preset[$identifier] = autocomplete_helper::create_item_label($object, $options['widget_config']['result_headers'], $options['widget_config']['titlefield']);
-                } catch (midcom_error $e) {
+                }
+                catch (midcom_error $e) {
                     $e->log();
                 }
             }

Spacing +3 added lines, -3 removed lines

@@ -33,7 +33,7 @@ 
     public function configureOptions(OptionsResolver $resolver)
     {
         $resolver->setDefault('error_bubbling', false);
-        $resolver->setNormalizer('widget_config', function (Options $options, $value) {
+        $resolver->setNormalizer('widget_config', function(Options $options, $value) {
             $widget_defaults = [
                 'creation_mode_enabled' => false,
                 'class' => null,
@@ -66,7 +66,7 @@ 
 
             return helper::normalize($widget_defaults, $value);
         });
-        $resolver->setNormalizer('type_config', function (Options $options, $value) {
+        $resolver->setNormalizer('type_config', function(Options $options, $value) {
             $type_defaults = [
                 'options' => [],
                 'method' => 'GET',
@@ -108,7 +108,7 @@ 
     public function finishView(FormView $view, FormInterface $form, array $options)
     {
         $preset = [];
-        if (   !empty($view->children['selection']->vars['data'])
+        if (!empty($view->children['selection']->vars['data'])
             && !empty($options['widget_config']['class'])) {
             foreach (array_filter((array) $view->children['selection']->vars['data']) as $identifier) {
                 if ($options['widget_config']['id_field'] == 'id') {

lib/midcom/core/dbaproxy.php

Braces +2 added lines, -1 removed lines

@@ -64,7 +64,8 @@
         try {
             $this->__object = $this->__midcom_class_name__::get_cached($this->__identifier);
             return true;
-        } catch (midcom_error $e) {
+        }
+        catch (midcom_error $e) {
             $e->log();
         }
         return false;

Spacing +2 added lines, -2 removed lines

@@ -22,11 +22,11 @@
     /**
      * MidCOM DBA object identifier, can be ID or GUID
      */
-    private string|int $__identifier;
+    private string | int $__identifier;
 
     private bool $__tried_to_load = false;
 
-    public function __construct(string|int $identifier, string $classname)
+    public function __construct(string | int $identifier, string $classname)
     {
         $this->__midcom_class_name__ = $classname;
         $this->__identifier = $identifier;

lib/midcom/services/auth/main.php

Braces +2 added lines, -1 removed lines

@@ -575,7 +575,8 @@
                     $param = $param->__object;
                 }
                 $this->_group_cache[$id] = new midcom_core_group($param);
-            } catch (midcom_error $e) {
+            }
+            catch (midcom_error $e) {
                 debug_add("Group with identifier {$id} could not be loaded: " . $e->getMessage(), MIDCOM_LOG_WARN);
                 $this->_group_cache[$id] = null;
             }

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

Spacing +4 added lines, -4 removed lines

@@ -64,7 +64,7 @@ 
 
     protected function add_string_field(string $key, string $type)
     {
-        if (   $key == 'component'
+        if ($key == 'component'
             && $type == midcom_db_topic::class) {
             $this->_add_component_dropdown($key);
             return;
@@ -134,12 +134,12 @@ 
         parent::add_longtext_field($key);
 
         // Check the user preference and configuration
-        if (   in_array($key, ['content', 'description'])
+        if (in_array($key, ['content', 'description'])
             && midgard_admin_asgard_plugin::get_preference('tinymce_enabled')) {
             $this->schema['fields'][$key]['widget'] = 'tinymce';
         }
 
-        if (   in_array($key, ['value', 'code'])
+        if (in_array($key, ['value', 'code'])
             && midgard_admin_asgard_plugin::get_preference('codemirror_enabled')) {
             $this->schema['fields'][$key]['widget'] = 'codemirror';
         }
@@ -257,7 +257,7 @@ 
         if ($score1 > $score2) {
             return 1;
         }
-        if (   $score1 < 3
+        if ($score1 < 3
             || $score1 > 6) {
             return strnatcmp($first, $second);
         }

lib/midgard/admin/asgard/handler/component/configuration.php

Spacing +2 added lines, -2 removed lines

@@ -277,7 +277,7 @@ 
     {
         $success = true;
         foreach ($this->_request_data['config']->_global as $global_key => $global_value) {
-            if (   isset($config[$global_key])
+            if (isset($config[$global_key])
                 && $config[$global_key] != $global_value) {
                 continue;
                 // Skip the ones we will set next
@@ -290,7 +290,7 @@ 
         }
 
         foreach ($config as $key => $value) {
-            if (   is_array($value)
+            if (is_array($value)
                 || is_object($value)) {
                 /**
                  * See http://trac.midgard-project.org/ticket/1442

lib/midgard/admin/asgard/schemadb/config.php

Spacing +2 added lines, -2 removed lines

@@ -61,11 +61,11 @@
                 $fields[$key] = $this->detect_schema($key, $value);
             }
 
-            if (   !isset($this->config->_local[$key])
+            if (!isset($this->config->_local[$key])
                 || $this->config->_local[$key] == $this->config->_global[$key]) {
                 // No local configuration setting, note to user that this is the global value
                 $fields[$key]['title'] = $schema->get_l10n()->get($fields[$key]['title']);
-                $fields[$key]['title'] .= " <span class=\"global\">(" . midcom::get()->i18n->get_string('global value', 'midgard.admin.asgard') .")</span>";
+                $fields[$key]['title'] .= " <span class=\"global\">(" . midcom::get()->i18n->get_string('global value', 'midgard.admin.asgard') . ")</span>";
             }
         }
 

lib/midcom/admin/help/help.php

Spacing +3 added lines, -3 removed lines

@@ -179,9 +179,9 @@ 
         $files = [];
         $pattern = $component_dir . '*.{' . $this->_i18n->get_current_language() . ',' . $this->_i18n->get_fallback_language() . '}.txt';
 
-        foreach (glob($pattern, GLOB_NOSORT|GLOB_BRACE) as $path) {
+        foreach (glob($pattern, GLOB_NOSORT | GLOB_BRACE) as $path) {
             $entry = basename($path);
-            if (   str_starts_with($entry, 'index')
+            if (str_starts_with($entry, 'index')
                 || str_starts_with($entry, 'handler')
                 || str_starts_with($entry, 'urlmethod')) {
                 // Ignore dotfiles, handlers & index.lang.txt
@@ -232,7 +232,7 @@ 
         $data = [];
 
         $exec_path = midcom::get()->componentloader->path_to_snippetpath($component) . '/exec/';
-        if (   !is_dir($exec_path)
+        if (!is_dir($exec_path)
             || !is_readable($exec_path)) {
             // Directory not accessible, skip loading it
             return $data;

lib/org/openpsa/calendar/event.php

Spacing +2 added lines, -2 removed lines

@@ -141,7 +141,7 @@ 
         }
 
         //Check up
-        if (   !$this->up
+        if (!$this->up
             && $this->title != '__org_openpsa_calendar') {
             $root_event = org_openpsa_calendar_interface::find_root_event();
             $this->up = $root_event->id;
@@ -166,7 +166,7 @@ 
 
     private function _check_timerange() : bool
     {
-        if (   !$this->start
+        if (!$this->start
             || !$this->end) {
             debug_add('Event must have start and end timestamps');
             midcom_connection::set_error(MGD_ERR_RANGE);