Passed
Push — master ( 805652...eccbd5 )
by Andreas
26:13
created
lib/midgard/admin/asgard/handler/component/configuration.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -277,7 +277,7 @@  discard block
 block discarded – undo
277 277
     {
278 278
         $success = true;
279 279
         foreach ($this->_request_data['config']->_global as $global_key => $global_value) {
280
-            if (   isset($config[$global_key])
280
+            if (isset($config[$global_key])
281 281
                 && $config[$global_key] != $global_value) {
282 282
                 continue;
283 283
                 // Skip the ones we will set next
@@ -290,7 +290,7 @@  discard block
 block discarded – undo
290 290
         }
291 291
 
292 292
         foreach ($config as $key => $value) {
293
-            if (   is_array($value)
293
+            if (is_array($value)
294 294
                 || is_object($value)) {
295 295
                 /**
296 296
                  * See http://trac.midgard-project.org/ticket/1442
Please login to merge, or discard this patch.
lib/midgard/admin/asgard/schemadb/config.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -61,11 +61,11 @@
 block discarded – undo
61 61
                 $fields[$key] = $this->detect_schema($key, $value);
62 62
             }
63 63
 
64
-            if (   !isset($this->config->_local[$key])
64
+            if (!isset($this->config->_local[$key])
65 65
                 || $this->config->_local[$key] == $this->config->_global[$key]) {
66 66
                 // No local configuration setting, note to user that this is the global value
67 67
                 $fields[$key]['title'] = $schema->get_l10n()->get($fields[$key]['title']);
68
-                $fields[$key]['title'] .= " <span class=\"global\">(" . midcom::get()->i18n->get_string('global value', 'midgard.admin.asgard') .")</span>";
68
+                $fields[$key]['title'] .= " <span class=\"global\">(" . midcom::get()->i18n->get_string('global value', 'midgard.admin.asgard') . ")</span>";
69 69
             }
70 70
         }
71 71
 
Please login to merge, or discard this patch.
lib/midcom/admin/help/help.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -179,9 +179,9 @@  discard block
 block discarded – undo
179 179
         $files = [];
180 180
         $pattern = $component_dir . '*.{' . $this->_i18n->get_current_language() . ',' . $this->_i18n->get_fallback_language() . '}.txt';
181 181
 
182
-        foreach (glob($pattern, GLOB_NOSORT|GLOB_BRACE) as $path) {
182
+        foreach (glob($pattern, GLOB_NOSORT | GLOB_BRACE) as $path) {
183 183
             $entry = basename($path);
184
-            if (   str_starts_with($entry, 'index')
184
+            if (str_starts_with($entry, 'index')
185 185
                 || str_starts_with($entry, 'handler')
186 186
                 || str_starts_with($entry, 'urlmethod')) {
187 187
                 // Ignore dotfiles, handlers & index.lang.txt
@@ -232,7 +232,7 @@  discard block
 block discarded – undo
232 232
         $data = [];
233 233
 
234 234
         $exec_path = midcom::get()->componentloader->path_to_snippetpath($component) . '/exec/';
235
-        if (   !is_dir($exec_path)
235
+        if (!is_dir($exec_path)
236 236
             || !is_readable($exec_path)) {
237 237
             // Directory not accessible, skip loading it
238 238
             return $data;
Please login to merge, or discard this patch.
lib/org/openpsa/calendar/event.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -141,7 +141,7 @@  discard block
 block discarded – undo
141 141
         }
142 142
 
143 143
         //Check up
144
-        if (   !$this->up
144
+        if (!$this->up
145 145
             && $this->title != '__org_openpsa_calendar') {
146 146
             $root_event = org_openpsa_calendar_interface::find_root_event();
147 147
             $this->up = $root_event->id;
@@ -166,7 +166,7 @@  discard block
 block discarded – undo
166 166
 
167 167
     private function _check_timerange() : bool
168 168
     {
169
-        if (   !$this->start
169
+        if (!$this->start
170 170
             || !$this->end) {
171 171
             debug_add('Event must have start and end timestamps');
172 172
             midcom_connection::set_error(MGD_ERR_RANGE);
Please login to merge, or discard this patch.
lib/midcom/helper/_dbfactory.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -49,7 +49,7 @@  discard block
 block discarded – undo
49 49
             throw new midcom_error_midgard($e, $guid);
50 50
         }
51 51
         $person_class = midcom::get()->config->get('person_class');
52
-        if (   get_class($tmp) == 'midgard_person'
52
+        if (get_class($tmp) == 'midgard_person'
53 53
             && $person_class != 'midgard_person') {
54 54
             $tmp = new $person_class($guid);
55 55
         }
@@ -79,7 +79,7 @@  discard block
 block discarded – undo
79 79
         }
80 80
         $object = new $classname($src);
81 81
         $cache[$classname][$object->guid] = $object;
82
-        $cache[$classname][$object->id] =& $cache[$classname][$object->guid];
82
+        $cache[$classname][$object->id] = & $cache[$classname][$object->guid];
83 83
         return $cache[$classname][$object->guid];
84 84
     }
85 85
 
@@ -127,7 +127,7 @@  discard block
 block discarded – undo
127 127
             return true;
128 128
         }
129 129
 
130
-        if (   isset($object->__object)
130
+        if (isset($object->__object)
131 131
             && is_object($object->__object)
132 132
             && $object->__object instanceof $class) {
133 133
             // Decorator whose MgdSchema object matches
@@ -150,7 +150,7 @@  discard block
 block discarded – undo
150 150
             return $this->get_parent_data_uncached($object);
151 151
         });
152 152
 
153
-        if (   empty($parent_guid)
153
+        if (empty($parent_guid)
154 154
             || $parent_guid === $object->guid) {
155 155
             return null;
156 156
         }
Please login to merge, or discard this patch.
lib/net/nehmer/blog/handler/index.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -91,7 +91,7 @@
 block discarded – undo
91 91
         midcom::get()->head->set_pagetitle($this->_request_data['page_title']);
92 92
 
93 93
         // Activate correct leaf
94
-        if (   $this->_config->get('show_navigation_pseudo_leaves')
94
+        if ($this->_config->get('show_navigation_pseudo_leaves')
95 95
             && in_array($this->category, $this->_request_data['categories'])) {
96 96
             $this->set_active_leaf($this->_topic->id . '_CAT_' . $this->category);
97 97
         }
Please login to merge, or discard this patch.
lib/net/nehmer/blog/handler/archive.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -131,8 +131,8 @@  discard block
 block discarded – undo
131 131
         $year_data = [];
132 132
         $first_post = $this->_compute_welcome_first_post();
133 133
         $this->_request_data['first_post'] = $first_post;
134
-        $this->_request_data['total_count'] =& $total_count;
135
-        $this->_request_data['year_data'] =& $year_data;
134
+        $this->_request_data['total_count'] = & $total_count;
135
+        $this->_request_data['year_data'] = & $year_data;
136 136
         if (!$first_post) {
137 137
             return;
138 138
         }
@@ -263,7 +263,7 @@  discard block
 block discarded – undo
263 263
         switch ($handler_id) {
264 264
             case 'archive-year-category':
265 265
                 $category = trim(strip_tags($args[1]));
266
-                if (   $data['datamanager']->get_schema('default')->has_field('categories')
266
+                if ($data['datamanager']->get_schema('default')->has_field('categories')
267 267
                     && !$data['datamanager']->get_schema('default')->get_field('categories')['type_config']['allow_multiple']) {
268 268
                     $qb->add_constraint('extra1', '=', (string) $category);
269 269
                 } else {
@@ -352,7 +352,7 @@  discard block
 block discarded – undo
352 352
             throw new midcom_error_notfound("The year '{$year}' is not a valid year identifier.");
353 353
         }
354 354
 
355
-        if (   $month < 1
355
+        if ($month < 1
356 356
             || $month > 12) {
357 357
             throw new midcom_error_notfound("The year {$month} is not a valid year identifier.");
358 358
         }
Please login to merge, or discard this patch.
lib/net/nehmer/blog/style/index-item.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -12,7 +12,7 @@  discard block
 block discarded – undo
12 12
 $published = sprintf($data['l10n']->get('posted on %s.'), "<abbr title=\"" . strftime('%Y-%m-%dT%H:%M:%S%z', $data['article']->metadata->published) . "\">" . $published . "</abbr>");
13 13
 
14 14
 if (array_key_exists('base_ajax_comments_url', $data)) {
15
-    $published .= ' <a href="#switch_comments" onClick="showAjaxComments(this, \''.$data['article']->guid.'\'); return false;">'
15
+    $published .= ' <a href="#switch_comments" onClick="showAjaxComments(this, \'' . $data['article']->guid . '\'); return false;">'
16 16
                     . sprintf($data['l10n']->get('%s comments'), net_nehmer_comments_comment::count_by_objectguid($data['article']->guid))
17 17
                     . "</a>.";
18 18
 } elseif ($data['config']->get('comments_enable')) {
@@ -28,7 +28,7 @@  discard block
 block discarded – undo
28 28
         &(published:h);
29 29
             <?php
30 30
             if ($data['linked']) {
31
-                echo $data['l10n']->get('to') ." <a href=\"{$data['node'][MIDCOM_NAV_ABSOLUTEURL]}\">{$data['node'][MIDCOM_NAV_NAME]}</a>\n";
31
+                echo $data['l10n']->get('to') . " <a href=\"{$data['node'][MIDCOM_NAV_ABSOLUTEURL]}\">{$data['node'][MIDCOM_NAV_NAME]}</a>\n";
32 32
             }
33 33
             ?>
34 34
     </p>
Please login to merge, or discard this patch.
lib/midcom/admin/folder/handler/move.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -35,7 +35,7 @@  discard block
 block discarded – undo
35 35
     {
36 36
         $this->_object = midcom::get()->dbfactory->get_object_by_guid($guid);
37 37
 
38
-        if (   !is_a($this->_object, midcom_db_topic::class)
38
+        if (!is_a($this->_object, midcom_db_topic::class)
39 39
             && !is_a($this->_object, midcom_db_article::class)) {
40 40
             throw new midcom_error_notfound("Moving only topics and articles is supported.");
41 41
         }
@@ -76,7 +76,7 @@  discard block
 block discarded – undo
76 76
             $folder = midcom_core_context::get()->get_key(MIDCOM_CONTEXT_ROOTTOPIC);
77 77
         }
78 78
 
79
-        if (   is_a($this->_object, midcom_db_topic::class)
79
+        if (is_a($this->_object, midcom_db_topic::class)
80 80
             && $folder->up == $this->_object->id) {
81 81
             $tree_disabled = true;
82 82
         }
@@ -89,7 +89,7 @@  discard block
 block discarded – undo
89 89
             $selected = ' checked="checked"';
90 90
         }
91 91
 
92
-        if (   !is_a($this->_object, midcom_db_topic::class)
92
+        if (!is_a($this->_object, midcom_db_topic::class)
93 93
             && $folder->component !== $this->current_folder->component) {
94 94
             // Non-topic objects may only be moved under folders of same component
95 95
             $class = 'wrong_component';
Please login to merge, or discard this patch.