flack / openpsa

lib/net/nemein/wiki/resolver.php

Spacing +2 added lines, -2 removed lines

@@ -150,7 +150,7 @@ 
             $localpath = $path;
             $matches['latest_parent'] = $folder_tree['/'];
             $missing_levels = 0;
-            while (   $localpath
+            while ($localpath
                    && $localpath != '/') {
                 $localpath = dirname($localpath);
                 $missing_levels++;
@@ -177,7 +177,7 @@ 
 
         $root_folder = $folder;
         $max = 100;
-        while (   $folder[MIDCOM_NAV_COMPONENT] == 'net.nemein.wiki'
+        while ($folder[MIDCOM_NAV_COMPONENT] == 'net.nemein.wiki'
                && (($parent = $folder[MIDCOM_NAV_NODEID]) != -1)
                && $max > 0) {
             $root_folder = $folder;

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

Spacing +4 added lines, -4 removed lines

@@ -54,7 +54,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->deleteItem($parent_entry[MIDCOM_NAV_ID] . '-leaves');
                 }
@@ -69,7 +69,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);
@@ -79,7 +79,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]);
@@ -103,7 +103,7 @@ 
             if ($object instanceof midcom_db_topic) {
                 return $nav->get_node($object->id);
             }
-            if (   ($node = $nav->find_closest_topic($object))
+            if (($node = $nav->find_closest_topic($object))
                 && $nodeobject = $nav->get_node($node->id)) {
                 return $nav->get_leaf($nodeobject[MIDCOM_NAV_ID] . '-' . $object->id);
             }

Braces +2 added lines, -1 removed lines

@@ -107,7 +107,8 @@
                 && $nodeobject = $nav->get_node($node->id)) {
                 return $nav->get_leaf($nodeobject[MIDCOM_NAV_ID] . '-' . $object->id);
             }
-        } catch (midcom_error $e) {
+        }
+        catch (midcom_error $e) {
             $e->log();
         }
         return null;

lib/org/openpsa/user/style/show-group.php

Indentation +1 added lines, -1 removed lines

@@ -1,7 +1,7 @@
 <div class="content-with-sidebar">
     <div class="main">
 	    <?php
-	    $data['view']->display_view(true);
+        $data['view']->display_view(true);
         $prefix = midcom_core_context::get()->get_key(MIDCOM_CONTEXT_ANCHORPREFIX);
         midcom::get()->dynamic_load($prefix . 'members/' . $data['group']->guid . '/');
         ?>

lib/org/openpsa/user/handler/group/privileges.php

Spacing +2 added lines, -2 removed lines

@@ -23,9 +23,9 @@
         $schemadb = schemadb::from_path($this->_config->get('schemadb_acl'));
 
         // Get the calendar root event
-        if (   class_exists('org_openpsa_calendar_interface')
+        if (class_exists('org_openpsa_calendar_interface')
             && $root_event = org_openpsa_calendar_interface::find_root_event()) {
-            $field =& $schemadb->get('default')->get_field('calendar');
+            $field = & $schemadb->get('default')->get_field('calendar');
             $field['type_config']['privilege_object'] = $root_event;
             $field['type_config']['assignee'] = 'group:' . $group->guid;
         }

src/midcom/workflow/delete.php

Spacing +1 added lines, -1 removed lines

@@ -87,7 +87,7 @@
             ])
             ->setRequired('object')
             ->setAllowedTypes('object', midcom_core_dbaobject::class)
-            ->setNormalizer('label', function ($options, $value) {
+            ->setNormalizer('label', function($options, $value) {
                 return $value ?? midcom_helper_reflector::get_object_title($options['object']);
             });
     }

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

Spacing +3 added lines, -3 removed lines

@@ -66,7 +66,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');
@@ -192,7 +192,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;
@@ -210,7 +210,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

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

lib/midcom/core/nullcomponent/handler/index.php

Spacing +1 added lines, -1 removed lines

@@ -24,7 +24,7 @@
         midcom::get()->metadata->set_request_metadata($this->_topic->metadata->revised, $this->_topic->guid);
         $data['node'] = $this->_topic;
 
-        if (   $this->_topic->id == midcom_core_context::get()->get_key(MIDCOM_CONTEXT_ROOTTOPIC)->id
+        if ($this->_topic->id == midcom_core_context::get()->get_key(MIDCOM_CONTEXT_ROOTTOPIC)->id
             && !midcom::get()->config->get('midcom_root_component')) {
             // if we have an ad hoc root topic, give user some options to do the initial setup
             return new midcom_response_relocate($this->router->generate('wizard'));

src/midcom/httpkernel/subscriber.php

Spacing +2 added lines, -2 removed lines

@@ -64,7 +64,7 @@ 
     public function on_request(RequestEvent $event)
     {
         $request = $event->getRequest();
-        if (   $event->isMainRequest()
+        if ($event->isMainRequest()
             && $response = $this->initialize($request)) {
             $event->setResponse($response);
             return;
@@ -98,7 +98,7 @@ 
         $i = array_search('__request_data__', $arguments, true);
         if ($i !== false) {
             $context = $event->getRequest()->attributes->get('context');
-            $arguments[$i] =& $context->get_custom_key('request_data');
+            $arguments[$i] = & $context->get_custom_key('request_data');
             $event->setArguments($arguments);
         }
     }

lib/midcom/core/account.php

Spacing +1 added lines, -1 removed lines

@@ -192,7 +192,7 @@
             return false;
         }
 
-        if (   !empty($this->_old_username)
+        if (!empty($this->_old_username)
             && $this->_old_username !== $new_username) {
             $history = @unserialize($this->_person->get_parameter('midcom', 'username_history')) ?: [];
             $history[time()] = ['old' => $this->_old_username, 'new' => $new_username];