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);
             }

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

lib/org/openpsa/expenses/hour/report.php

Spacing +1 added lines, -1 removed lines

@@ -113,7 +113,7 @@
     {
         if ($this->invoiceable) {
             $task = new org_openpsa_projects_task_dba($this->task);
-            $time_slot = (float)$task->get_parameter('org.openpsa.projects.projectbroker', 'minimum_slot');
+            $time_slot = (float) $task->get_parameter('org.openpsa.projects.projectbroker', 'minimum_slot');
             if (empty($time_slot)) {
                 $time_slot = (float) midcom_baseclasses_components_configuration::get('org.openpsa.expenses', 'config')->get('default_minimum_time_slot');
                 if (empty($time_slot)) {

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

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') {

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/midcom/admin/folder/handler/move.php

Spacing +3 added lines, -3 removed lines

@@ -27,7 +27,7 @@ 
     {
         $this->_object = midcom::get()->dbfactory->get_object_by_guid($guid);
 
-        if (   !($this->_object instanceof midcom_db_topic)
+        if (!($this->_object instanceof midcom_db_topic)
             && !($this->_object instanceof midcom_db_article)) {
             throw new midcom_error_notfound("Moving only topics and articles is supported.");
         }
@@ -66,7 +66,7 @@ 
     {
         $folder ??= midcom_core_context::get()->get_key(MIDCOM_CONTEXT_ROOTTOPIC);
 
-        if (   $this->_object instanceof midcom_db_topic
+        if ($this->_object instanceof midcom_db_topic
             && $folder->up == $this->_object->id) {
             $tree_disabled = true;
         }
@@ -79,7 +79,7 @@ 
             $selected = ' checked="checked"';
         }
 
-        if (   !($this->_object instanceof midcom_db_topic)
+        if (!($this->_object instanceof midcom_db_topic)
             && $folder->component !== $this->current_folder->component) {
             // Non-topic objects may only be moved under folders of same component
             $class = 'wrong_component';

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);
             }

lib/net/nemein/wiki/parser.php

Spacing +3 added lines, -3 removed lines

@@ -219,7 +219,7 @@ 
                 return $this->render_link($parts[1], $parts[2]) . $after;
 
             // Macro [something: <data>] (for example [abbr: BOFH - Bastard Operator From Hell] or [photo: <GUID>])
-            case (   preg_match('/^(.*?): (.*)/', $fulltext, $macro_parts)
+            case (preg_match('/^(.*?): (.*)/', $fulltext, $macro_parts)
                   && method_exists($this, "_run_macro_{$macro_parts[1]}")):
                 $method = "_run_macro_{$macro_parts[1]}";
                 return $this->$method($macro_parts[2], $match[0], $after);
@@ -247,7 +247,7 @@ 
             // No page matched, link to creation
             $folder = $wikipage_match['folder'] ?? $wikipage_match['latest_parent'];
 
-            if (   isset($folder[MIDCOM_NAV_OBJECT])
+            if (isset($folder[MIDCOM_NAV_OBJECT])
                 && $folder[MIDCOM_NAV_OBJECT]->can_do('midgard:create')) {
                 $workflow = $this->get_workflow('datamanager');
                 return "<a href=\"{$folder[MIDCOM_NAV_ABSOLUTEURL]}create/?wikiword={$wikipage_match['remaining_path']}\" " . $workflow->render_attributes() . " class=\"wiki_missing\" title=\"" . $this->_l10n->get('click to create') . "\">{$text}</a>";
@@ -282,7 +282,7 @@ 
                     $links[$parts[1]] = $parts[2];
                 }
                 // Ignore macros [something: <data>] (for example [abbr: BOFH - Bastard Operator From Hell] or [photo: <GUID>])
-                elseif (   !preg_match('/^(.*?): (.*)/', $fulltext, $macro_parts)
+                elseif (!preg_match('/^(.*?): (.*)/', $fulltext, $macro_parts)
                         || !method_exists($this, "_run_macro_{$macro_parts[1]}")) {
                     // MediaWiki-style link [wikipage] (no text)
                     $links[$fulltext] = $fulltext;