flack / openpsa

lib/midcom/services/auth/acl.php

Spacing +2 added lines, -2 removed lines

@@ -568,10 +568,10 @@
         }
 
         //owner privileges override everything but person privileges, so we have to cross-check those here
-        if (   $privilegename != 'midgard:owner'
+        if ($privilegename != 'midgard:owner'
             && $last_scope < MIDCOM_PRIVILEGE_SCOPE_OWNER) {
             $owner_privileges = $this->get_owner_default_privileges();
-            if (    array_key_exists($privilegename, $owner_privileges)
+            if (array_key_exists($privilegename, $owner_privileges)
                  && $this->_load_content_privilege('midgard:owner', $guid, $class, $user_id)
                  && self::$_content_privileges_cache[$cache_id]['midgard:owner']) {
                 self::$_content_privileges_cache[$cache_id][$privilegename] = ($owner_privileges[$privilegename] == MIDCOM_PRIVILEGE_ALLOW);

lib/midcom/services/cache/module/nap.php

Spacing +4 added lines, -4 removed lines

@@ -71,7 +71,7 @@ 
             // Get parent from DB and compare to catch moves
             if ($parent = $napobject[MIDCOM_NAV_OBJECT]->get_parent()) {
                 $parent_entry = $this->get_guid($parent->guid);
-                if (   $parent_entry
+                if ($parent_entry
                     && $parent_entry[MIDCOM_NAV_ID] != $cached_node_id) {
                     $this->backend->delete($parent_entry[MIDCOM_NAV_ID] . '-leaves');
                 }
@@ -86,7 +86,7 @@ 
             $parent_id = $napobject[MIDCOM_NAV_NODEID];
             $parent_entry = $this->get_node($parent_id);
 
-            if (   $parent_entry
+            if ($parent_entry
                 && array_key_exists(MIDCOM_NAV_SUBNODES, $parent_entry)) {
                 unset($parent_entry[MIDCOM_NAV_SUBNODES]);
                 $this->put_node($parent_id, $parent_entry);
@@ -96,7 +96,7 @@ 
             if ($parent = $napobject[MIDCOM_NAV_OBJECT]->get_parent()) {
                 $parent_entry_from_object = $this->get_guid($parent->guid);
 
-                if (    !empty($parent_entry_from_object[MIDCOM_NAV_ID])
+                if (!empty($parent_entry_from_object[MIDCOM_NAV_ID])
                      && !empty($parent_entry[MIDCOM_NAV_ID])
                      && $parent_entry_from_object[MIDCOM_NAV_ID] != $parent_entry[MIDCOM_NAV_ID]) {
                     unset($parent_entry_from_object[MIDCOM_NAV_SUBNODES]);
@@ -122,7 +122,7 @@ 
             $nav = new midcom_helper_nav;
             if ($object instanceof midcom_db_topic) {
                 $napobject = $nav->get_node($object->id);
-            } elseif (   ($node = $nav->find_closest_topic($object))
+            } elseif (($node = $nav->find_closest_topic($object))
                       && $nodeobject = $nav->get_node($node->id)) {
                 $napobject = $nav->get_leaf($nodeobject[MIDCOM_NAV_ID] . '-' . $object->id);
             }

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/midcom/debug.php

Spacing +4 added lines, -4 removed lines

@@ -88,9 +88,9 @@ 
 
         while ($bt) {
             $caller = array_shift($bt);
-            if (   !array_key_exists('class', $caller)
+            if (!array_key_exists('class', $caller)
                 || $caller['class'] != midcom_debug::class) {
-                if (   !array_key_exists('function', $bt[0])
+                if (!array_key_exists('function', $bt[0])
                     || $bt[0]['function'] != 'require') {
                     $caller = array_shift($bt);
                 }
@@ -102,11 +102,11 @@ 
         if (array_key_exists('class', $caller)) {
             $record['extra']['caller'] .= $caller['class'] . '::';
         }
-        if (   array_key_exists('function', $caller)
+        if (array_key_exists('function', $caller)
             && !str_starts_with($caller['function'], 'debug_')) {
             $record['extra']['caller'] .= $caller['function'];
         } else {
-            $record['extra']['caller'] .= $caller['file'] . ' (' . $caller['line']. ')';
+            $record['extra']['caller'] .= $caller['file'] . ' (' . $caller['line'] . ')';
         }
         return $record;
     }

lib/midcom/services/rcs/backend/rcs.php

Spacing +1 added lines, -1 removed lines

@@ -91,7 +91,7 @@
                 $revisions[$history['revision']] = $history;
 
                 $i += 3;
-                while (   $i < $total
+                while ($i < $total
                         && !str_starts_with($lines[$i], '----')
                         && !str_starts_with($lines[$i], '=====')) {
                     $i++;

lib/midcom/helper/nav/main.php

Spacing +2 added lines, -2 removed lines

@@ -422,7 +422,7 @@ 
                     $entry = "<span class=\"{$current_class}\">{$entry}</span>";
                 }
             } else {
-                if (   !empty($data['napobject'][MIDCOM_NAV_GUID])
+                if (!empty($data['napobject'][MIDCOM_NAV_GUID])
                     && in_array($data['napobject'][MIDCOM_NAV_GUID], $skip_guids)) {
                     continue;
                 }
@@ -537,7 +537,7 @@ 
                     $id = "custom-{$key}";
 
                     $url = "{$prefix}{$entry[MIDCOM_NAV_URL]}";
-                    if (   str_starts_with($entry[MIDCOM_NAV_URL], '/')
+                    if (str_starts_with($entry[MIDCOM_NAV_URL], '/')
                         || preg_match('|^https?://|', $entry[MIDCOM_NAV_URL])) {
                         $url = $entry[MIDCOM_NAV_URL];
                     }

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/midcom/config/main.php

Spacing +3 added lines, -3 removed lines

@@ -427,7 +427,7 @@ 
         // show_unapproved_objects). Disabled by default. Unsafe to Link Prefetching!
         'metadata_approval' => false,
         'metadata_scheduling' => false,
-        'metadata_lock_timeout' => 60,    // Time in minutes
+        'metadata_lock_timeout' => 60, // Time in minutes
         'staging2live_staging' => false,
 
         // Set the datamanager schema used by the Metadata Service
@@ -518,12 +518,12 @@ 
             return $default;
         }
 
-        if (   $key === 'auth_type'
+        if ($key === 'auth_type'
             && !in_array($this->_merged_config[$key], ['Plaintext', 'Legacy', 'SHA256'])) {
             throw new midcom_error('Unsupported authentication type');
         }
         // Check the midcom_config site prefix for absolute local urls
-        if (   $key === 'midcom_site_url'
+        if ($key === 'midcom_site_url'
             && str_starts_with($this->_merged_config[$key], '/')) {
             $this->_merged_config[$key] = midcom::get()->get_page_prefix() . substr($this->_merged_config[$key], 1);
         }