flack / openpsa

lib/org/openpsa/relatedto/handler/journal/entry.php

Spacing +2 added lines, -2 removed lines

@@ -33,7 +33,7 @@ 
     public function _handler_create(Request $request, string $guid)
     {
         $current_object = midcom::get()->dbfactory->get_object_by_guid($guid);
-        $entry= new org_openpsa_relatedto_journal_entry_dba();
+        $entry = new org_openpsa_relatedto_journal_entry_dba();
         $entry->linkGuid = $current_object->guid;
 
         midcom::get()->head->set_pagetitle($this->_l10n->get('add journal entry'));
@@ -52,7 +52,7 @@ 
         $workflow = $this->get_workflow('datamanager', ['controller' => $data['controller']]);
         if ($entry->can_do('midgard:delete')) {
             $delete = $this->get_workflow('delete', ['object' => $entry]);
-            $url = $this->router->generate('journal_entry_delete', ['guid' => $guid ]);
+            $url = $this->router->generate('journal_entry_delete', ['guid' => $guid]);
             $workflow->add_dialog_button($delete, $url);
         }
         return $workflow->run($request);

src/midcom/datamanager/template/form.php

Spacing +8 added lines, -8 removed lines

@@ -84,7 +84,7 @@ 
     public function form_end(FormView $view, array $data)
     {
         $string = '';
-        if (   !isset($data['render_rest'])
+        if (!isset($data['render_rest'])
             || $data['render_rest']) {
             $string .= $this->renderer->rest($view);
         }
@@ -223,7 +223,7 @@ 
             $data['attr']['class'] = 'shorttext';
         }
 
-        if (   !empty($data['value'])
+        if (!empty($data['value'])
             || is_numeric($data['value'])) {
             $data['attr']['value'] = $data['value'];
         }
@@ -265,7 +265,7 @@ 
     {
         if ($view->vars['readonly']) {
             $string = $this->get_view_renderer()->autocomplete_widget($view, $data);
-            return $string .  $this->renderer->widget($view['selection']);
+            return $string . $this->renderer->widget($view['selection']);
         }
 
         autocomplete::add_head_elements($data['handler_options']['creation_mode_enabled'], $data['handler_options']['sortable']);
@@ -275,7 +275,7 @@ 
         $jsinit .= "midcom_helper_datamanager2_autocomplete.create_dm2_widget('{$element_id}_search_input', {$data['min_chars']});\n";
 
         $string = '<fieldset ' . $this->renderer->block($view, 'widget_container_attributes') . '>';
-        $string .=  $this->renderer->widget($view['selection']);
+        $string .= $this->renderer->widget($view['selection']);
         $string .= ' ' . $this->renderer->widget($view['search_input']);
         $string .= '</fieldset>';
         return $string . $this->jsinit($jsinit);
@@ -324,7 +324,7 @@ 
         }
 
         $string = '<select ';
-        if (   $data['required']
+        if ($data['required']
             && null === $data['placeholder']
             && $data['placeholder_in_choices'] === false
             && $data['multiple'] === false) {
@@ -338,7 +338,7 @@ 
         $string .= $this->renderer->block($view, 'widget_attributes', $data) . '>';
         if (null !== $data['placeholder']) {
             $string .= '<option value=""';
-            if (   $data['required']
+            if ($data['required']
                 && empty($data['value'])
                 && "0" !== $data['value']) {
                 $string .= ' selected="selected"';
@@ -500,7 +500,7 @@ 
             $string .= '<label class="midcom_datamanager_photo_label">' . $this->renderer->humanize('delete photo') . ' ' . $this->renderer->widget($data['form']['delete']) . '</label>';
             $string .= '<ul>';
             foreach ($objects as $info) {
-                if (   $info['size_x']
+                if ($info['size_x']
                     && $info['size_y']) {
                     $size = "{$info['size_x']}&times;{$info['size_y']}";
                 } else {
@@ -525,7 +525,7 @@ 
         $string .= '</td></tr></table></div>';
         $string .= $this->renderer->row($data['form']['identifier']);
 
-        return $string . $this->jsinit('init_image_widget("' . $view->vars['id'] .'");');
+        return $string . $this->jsinit('init_image_widget("' . $view->vars['id'] . '");');
     }
 
     public function subform_widget(FormView $view, array $data)

lib/org/openpsa/user/style/show-person-account.php

Indentation +1 added lines, -1 removed lines

@@ -23,7 +23,7 @@
             echo '<span class="toolbar_label">' . $data['l10n_midcom']->get('delete') . '</span></a></li>';
             if (    midcom::get()->config->get('auth_allow_trusted') === true
                  && $data['person']->can_do('org.openpsa.user:su')) {
-                 echo '<li><a class="button" href="' . $data['router']->generate('account_su', ['guid' => $data['person']->guid]) . '">' . $data['l10n']->get('switch to user') . "</a></li>\n";
+                    echo '<li><a class="button" href="' . $data['router']->generate('account_su', ['guid' => $data['person']->guid]) . '">' . $data['l10n']->get('switch to user') . "</a></li>\n";
             }
             echo "</ul>\n";
         }

Spacing +4 added lines, -4 removed lines

@@ -11,7 +11,7 @@ 
             echo '<p>' . $data['l10n']->get('last login') . ': ' . $formatter->datetime($lastlogin) . "</p>\n";
         }
 
-        if (   $data['person']->guid == midcom::get()->auth->user->guid
+        if ($data['person']->guid == midcom::get()->auth->user->guid
             || midcom::get()->auth->can_user_do('org.openpsa.user:manage', null, org_openpsa_user_interface::class)) {
             $workflow = new midcom\workflow\datamanager;
             echo '<ul class="area_toolbar">';
@@ -20,9 +20,9 @@ 
                 'object' => $data['person'],
                 'label' => $data['l10n']->get('account')
             ]);
-            echo '<li><a href="' . $data['router']->generate('account_delete', ['guid' => $data['person']->guid]). '" ' . $workflow->render_attributes() . '"  class="button">';
+            echo '<li><a href="' . $data['router']->generate('account_delete', ['guid' => $data['person']->guid]) . '" ' . $workflow->render_attributes() . '"  class="button">';
             echo '<span class="toolbar_label">' . $data['l10n_midcom']->get('delete') . '</span></a></li>';
-            if (    midcom::get()->config->get('auth_allow_trusted') === true
+            if (midcom::get()->config->get('auth_allow_trusted') === true
                  && $data['person']->can_do('org.openpsa.user:su')) {
                  echo '<li><a class="button" href="' . $data['router']->generate('account_su', ['guid' => $data['person']->guid]) . '">' . $data['l10n']->get('switch to user') . "</a></li>\n";
             }
@@ -30,7 +30,7 @@ 
         }
     } else {
         echo '<p><span class="metadata">' . $data['l10n']->get("no account") . '</span></p>';
-        if (   $data['person']->guid == midcom::get()->auth->user->guid
+        if ($data['person']->guid == midcom::get()->auth->user->guid
             || midcom::get()->auth->can_user_do('org.openpsa.user:manage', null, org_openpsa_user_interface::class)) {
             $workflow = new midcom\workflow\datamanager;
             echo '<ul class="area_toolbar">';

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

Spacing +2 added lines, -2 removed lines

@@ -34,11 +34,11 @@
      */
     public function configureOptions(OptionsResolver $resolver)
     {
-        $resolver->setDefault('config', function (Options $options) {
+        $resolver->setDefault('config', function(Options $options) {
             return \midcom_baseclasses_components_configuration::get('midcom.datamanager', 'config');
         });
 
-        $resolver->setNormalizer('widget_config', function (Options $options, $value) {
+        $resolver->setNormalizer('widget_config', function(Options $options, $value) {
             $widget_defaults = [
                 'height' => 25,
                 'width' => 80,

lib/midgard/admin/asgard/navigation.php

Spacing +13 added lines, -13 removed lines

@@ -41,7 +41,7 @@ 
         $this->_component = 'midgard.admin.asgard';
 
         $this->_object = $object;
-        $this->_request_data =& $request_data;
+        $this->_request_data = & $request_data;
 
         $this->root_types = midcom_helper_reflector_tree::get_root_classes();
 
@@ -57,7 +57,7 @@ 
             // we go through the path bottom up and show the first root type we find
             foreach (array_reverse($this->_object_path) as $node) {
                 foreach ($this->root_types as $root_type) {
-                    if (   is_a($node, $root_type)
+                    if (is_a($node, $root_type)
                         || midcom_helper_reflector::is_same_class($root_type, $node->__midcom_class_name__)) {
                         $this->expanded_root_types[] = $root_type;
                         break;
@@ -69,7 +69,7 @@ 
 
     protected function _is_collapsed(string $type, int $total) : bool
     {
-        return (   $total > $this->_config->get('max_navigation_entries')
+        return ($total > $this->_config->get('max_navigation_entries')
                 && empty($_GET['show_all_' . $type]));
     }
 
@@ -85,7 +85,7 @@ 
         foreach ($ref->get_child_classes() as $class) {
             $qb = $ref->_child_objects_type_qb($class, $object, false);
 
-            if (   !$qb
+            if (!$qb
                 || !($count = $qb->count_unchecked())) {
                 continue;
             }
@@ -131,7 +131,7 @@ 
     {
         $qb = $ref->_root_objects_qb();
 
-        if (   !$qb
+        if (!$qb
             || !($total = $qb->count_unchecked())) {
             return;
         }
@@ -209,7 +209,7 @@ 
         }
 
         echo "<a href=\"" . midcom_connection::get_url('self') . "__mfa/asgard/object/{$mode}/{$object->guid}/\" title=\"GUID: {$object->guid}, ID: {$object->id}\">{$icon}{$label}</a>\n";
-        if (   $selected
+        if ($selected
             || $autoexpand) {
             $this->_list_child_elements($object, $level + 1);
         }
@@ -229,7 +229,7 @@ 
                 continue;
             }
 
-            if (   method_exists($class, 'navigation')
+            if (method_exists($class, 'navigation')
                 && ($this->_request_data['plugin_name'] == "asgard_{$component}")) {
                 $this->_request_data['expanded'] = true;
                 midcom_show_style('midgard_admin_asgard_navigation_section_header');
@@ -263,13 +263,13 @@ 
         if ($this->_is_selected($object)) {
             $css_class .= ' selected';
         }
-        if (   is_object($this->_object)
-            && (   $object->guid == $this->_object->guid
-                || (   is_a($this->_object, midcom_db_parameter::class)
+        if (is_object($this->_object)
+            && ($object->guid == $this->_object->guid
+                || (is_a($this->_object, midcom_db_parameter::class)
                     && $object->guid == $this->_object->parentguid))) {
             $css_class .= ' current';
         }
-        if ( !$object->can_do('midgard:update')) {
+        if (!$object->can_do('midgard:update')) {
             $css_class .= ' readonly';
         }
         return $css_class;
@@ -312,7 +312,7 @@ 
 
         // If the regular expression has been set, check which types should be shown
         if ($regexp !== '//') {
-            $label_mapping = array_filter($label_mapping, function ($root_type) use ($regexp, $exclude) {
+            $label_mapping = array_filter($label_mapping, function($root_type) use ($regexp, $exclude) {
                 return preg_match($regexp, $root_type) == $exclude;
             }, ARRAY_FILTER_USE_KEY);
         }
@@ -342,7 +342,7 @@ 
          * click if nothing is expanded
          */
         $types_shown = false;
-        if (    !empty($expanded_types)
+        if (!empty($expanded_types)
              && midgard_admin_asgard_plugin::get_preference('navigation_type') === 'dropdown') {
             $this->_draw_select_navigation();
             $types_shown = true;

lib/org/openpsa/mail/main.php

Spacing +1 added lines, -1 removed lines

@@ -39,7 +39,7 @@
  */
 class org_openpsa_mail
 {
-    use midcom_baseclasses_components_base {__get as base__get;}
+    use midcom_baseclasses_components_base {__get as base__get; }
 
     /**
      * Text body

lib/net/nehmer/blog/handler.php

Spacing +2 added lines, -2 removed lines

@@ -15,7 +15,7 @@ 
 {
     public function get_url(midcom_db_article $article, bool $allow_external = false) : string
     {
-        if (   $allow_external
+        if ($allow_external
             && $this->_config->get('link_to_external_url')
             && !empty($article->url)) {
             return $article->url;
@@ -67,7 +67,7 @@ 
 
         $qb->add_constraint('topic.guid', 'IN', $topic_guids);
 
-        if (   count($topic_guids) > 1
+        if (count($topic_guids) > 1
             && $list_from_folders_categories = $this->_config->get('list_from_folders_categories')) {
             $list_from_folders_categories = explode(',', $list_from_folders_categories);
 

lib/net/nemein/tag/handler.php

Braces +4 added lines, -2 removed lines

@@ -381,7 +381,8 @@ 
             try {
                 $tag = net_nemein_tag_tag_dba::get_cached($link->tag);
                 $link_object_map[$link->fromGuid][$tag->tag] = $link;
-            } catch (midcom_error $e) {
+            }
+            catch (midcom_error $e) {
                 $e->log();
             }
         }
@@ -410,7 +411,8 @@ 
                     $tmpobject = midcom::get()->dbfactory->convert_midgard_to_midcom($tmpobject);
                 }
                 $return[] = $tmpobject;
-            } catch (midcom_error $e) {
+            }
+            catch (midcom_error $e) {
                 $e->log();
             }
         }

lib/net/nehmer/blog/navigation.php

Spacing +5 added lines, -5 removed lines

@@ -64,14 +64,14 @@ 
 
     private function _add_pseudo_leaves(array &$leaves)
     {
-        if (   $this->_config->get('archive_enable')
+        if ($this->_config->get('archive_enable')
             && $this->_config->get('archive_in_navigation')) {
             $leaves["{$this->_topic->id}_ARCHIVE"] = [
                 MIDCOM_NAV_URL => "archive/",
                 MIDCOM_NAV_NAME => $this->_l10n->get('archive'),
             ];
         }
-        if (   $this->_config->get('rss_enable')
+        if ($this->_config->get('rss_enable')
             && $this->_config->get('feeds_in_navigation')) {
             $leaves[self::LEAFID_FEEDS] = [
                 MIDCOM_NAV_URL => "feeds/",
@@ -79,7 +79,7 @@ 
             ];
         }
 
-        if (   $this->_config->get('categories_in_navigation')
+        if ($this->_config->get('categories_in_navigation')
             && $this->_config->get('categories') != '') {
             $categories = explode(',', $this->_config->get('categories'));
             foreach ($categories as $category) {
@@ -90,7 +90,7 @@ 
             }
         }
 
-        if (   $this->_config->get('archive_years_in_navigation')
+        if ($this->_config->get('archive_years_in_navigation')
             && $this->_config->get('archive_years_enable')) {
             $qb = midcom_db_article::new_query_builder();
             $qb->add_constraint('topic', '=', $this->_topic->id);
@@ -121,7 +121,7 @@ 
      */
     public static function add_scheduling_constraints(midcom_core_querybuilder $qb, midcom_helper_configuration $config)
     {
-        if (   $config->get('enable_scheduled_publishing')
+        if ($config->get('enable_scheduled_publishing')
             && !midcom::get()->auth->admin) {
             // Show the article only if the publishing time has passed or the viewer
             // is the author