Passed
Push — master ( 3657f2...b17fdb )
by Andreas
24:28 queued 05:07
created
src/midcom/templating/loader.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -196,7 +196,7 @@
 block discarded – undo
196 196
 
197 197
     private function resolve_includes(string $content) : string
198 198
     {
199
-        return preg_replace_callback("/<\\(([a-zA-Z0-9 _-]+)\\)>/", function (array $matches) {
199
+        return preg_replace_callback("/<\\(([a-zA-Z0-9 _-]+)\\)>/", function(array $matches) {
200 200
             $element = $matches[1];
201 201
 
202 202
             switch ($element) {
Please login to merge, or discard this patch.
lib/midgard/admin/asgard/handler/object/permissions.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -92,7 +92,7 @@  discard block
 block discarded – undo
92 92
         $tmp = $this->_object;
93 93
 
94 94
         $i = 0;
95
-        while (   !empty($tmp->guid)
95
+        while (!empty($tmp->guid)
96 96
                && !is_a($tmp, midcom_db_topic::class)
97 97
                && $i < 100) {
98 98
             // Get the parent; wishing eventually to get a topic
@@ -137,7 +137,7 @@  discard block
 block discarded – undo
137 137
 
138 138
         $assignees = $this->load_assignees();
139 139
         $this->process_assignees($assignees, $schemadb);
140
-        $assignee_field =& $schemadb->get('privileges')->get_field('add_assignee');
140
+        $assignee_field = & $schemadb->get('privileges')->get_field('add_assignee');
141 141
 
142 142
         if (!$this->additional_assignee) {
143 143
             // Populate additional assignee selector
@@ -236,7 +236,7 @@  discard block
 block discarded – undo
236 236
                 if ($assignee = midcom::get()->auth->get_assignee($privilege->assignee)) {
237 237
                     $label = $assignee->name;
238 238
                 } else {
239
-                    $label = $this->_l10n->get('ghost assignee for '. $privilege->assignee);
239
+                    $label = $this->_l10n->get('ghost assignee for ' . $privilege->assignee);
240 240
                 }
241 241
             }
242 242
 
Please login to merge, or discard this patch.
lib/midcom/services/auth/backend.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -81,7 +81,7 @@  discard block
 block discarded – undo
81 81
             return null;
82 82
         }
83 83
 
84
-        if (   midcom::get()->config->get('auth_check_client_ip')
84
+        if (midcom::get()->config->get('auth_check_client_ip')
85 85
             && $data['clientip'] != $request->getClientIp()) {
86 86
             debug_add("The session had mismatching client IP.", MIDCOM_LOG_INFO);
87 87
             debug_add("Expected {$data['clientip']}, got {$request->getClientIp()}.");
@@ -95,7 +95,7 @@  discard block
 block discarded – undo
95 95
             return null;
96 96
         }
97 97
 
98
-        if (   !$this->check_timestamp($data['timestamp'], $user)
98
+        if (!$this->check_timestamp($data['timestamp'], $user)
99 99
             || !$this->authenticate($user->username, '', true)) {
100 100
             $this->logout($user);
101 101
             return null;
@@ -141,7 +141,7 @@  discard block
 block discarded – undo
141 141
         $user = midcom_connection::login($username, $password, $trusted);
142 142
 
143 143
         if (!$user) {
144
-            debug_add("Failed to authenticate the given user: ". midcom_connection::get_error_string(),
144
+            debug_add("Failed to authenticate the given user: " . midcom_connection::get_error_string(),
145 145
                     MIDCOM_LOG_INFO);
146 146
             return null;
147 147
         }
Please login to merge, or discard this patch.
lib/midcom/services/cache/main.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -98,7 +98,7 @@
 block discarded – undo
98 98
     {
99 99
         if (!isset($this->_modules[$name])) {
100 100
             $this->_modules[$name] = $module;
101
-            $this->$name =& $this->_modules[$name];
101
+            $this->$name = & $this->_modules[$name];
102 102
         }
103 103
     }
104 104
 
Please login to merge, or discard this patch.
lib/net/nehmer/comments/comment.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -218,8 +218,8 @@  discard block
 block discarded – undo
218 218
     {
219 219
         $config = midcom_baseclasses_components_configuration::get('net.nehmer.comments', 'config');
220 220
 
221
-        if (   $config->get('ratings_enable')
222
-            && (    $config->get('ratings_cache_to_object')
221
+        if ($config->get('ratings_enable')
222
+            && ($config->get('ratings_cache_to_object')
223 223
                  || $config->get('comment_count_cache_to_object'))) {
224 224
             // Handle ratings
225 225
             $comments = self::list_by_objectguid($this->objectguid);
@@ -268,7 +268,7 @@  discard block
 block discarded – undo
268 268
 
269 269
     private function _send_notifications()
270 270
     {
271
-        if (   empty($this->title)
271
+        if (empty($this->title)
272 272
             && empty($this->content)) {
273 273
             // No need to send notifications about empty rating entries
274 274
             return;
Please login to merge, or discard this patch.
lib/net/nehmer/comments/handler/view.php 1 patch
Spacing   +10 added lines, -10 removed lines patch added patch discarded remove patch
@@ -98,8 +98,8 @@  discard block
 block discarded – undo
98 98
         if (!$this->_schemadb) {
99 99
             $this->_schemadb = schemadb::from_path($this->_config->get('schemadb'));
100 100
 
101
-            if (   $this->_config->get('use_captcha')
102
-                || (   !midcom::get()->auth->user
101
+            if ($this->_config->get('use_captcha')
102
+                || (!midcom::get()->auth->user
103 103
                     && $this->_config->get('use_captcha_if_anonymous'))) {
104 104
                 $fields = $this->_schemadb->get('comment')->get('fields');
105 105
                 $fields['captcha'] = [
@@ -112,7 +112,7 @@  discard block
 block discarded – undo
112 112
                 $this->_schemadb->get('comment')->set('fields', $fields);
113 113
             }
114 114
 
115
-            if (   $this->_config->get('ratings_enable')
115
+            if ($this->_config->get('ratings_enable')
116 116
                 && $this->_schemadb->get('comment')->has_field('rating')) {
117 117
                 $this->_schemadb->get('comment')->get_field('rating')['hidden'] = false;
118 118
             }
@@ -191,7 +191,7 @@  discard block
 block discarded – undo
191 191
             $this->_comments = $this->_comments->execute();
192 192
         }
193 193
 
194
-        if (   midcom::get()->auth->user
194
+        if (midcom::get()->auth->user
195 195
             || $this->_config->get('allow_anonymous')) {
196 196
             $this->_init_post_controller($request);
197 197
             if ($response = $this->_process_post($request)) {
@@ -222,7 +222,7 @@  discard block
 block discarded – undo
222 222
      */
223 223
     private function _process_post(Request $request)
224 224
     {
225
-        if (   !midcom::get()->auth->user
225
+        if (!midcom::get()->auth->user
226 226
             && !midcom::get()->auth->request_sudo('net.nehmer.comments')) {
227 227
             throw new midcom_error('We were anonymous but could not acquire SUDO privileges, aborting');
228 228
         }
@@ -235,7 +235,7 @@  discard block
 block discarded – undo
235 235
                 }
236 236
 
237 237
                 $formdata = $this->_post_controller->get_form_values();
238
-                if (   $formdata['subscribe']
238
+                if ($formdata['subscribe']
239 239
                     && midcom::get()->auth->user) {
240 240
                     // User wants to subscribe to receive notifications about this comments thread
241 241
 
@@ -269,7 +269,7 @@  discard block
 block discarded – undo
269 269
      */
270 270
     private function _get_last_modified() : int
271 271
     {
272
-        return array_reduce($this->_comments, function ($carry, net_nehmer_comments_comment $item) {
272
+        return array_reduce($this->_comments, function($carry, net_nehmer_comments_comment $item) {
273 273
             return max($item->metadata->revised, $carry);
274 274
         }, 0);
275 275
     }
@@ -291,8 +291,8 @@  discard block
 block discarded – undo
291 291
                 $data['comment_toolbar'] = $this->populate_post_toolbar($comment);
292 292
                 midcom_show_style('comments-item');
293 293
 
294
-                if (   midcom::get()->auth->admin
295
-                    || (   midcom::get()->auth->user
294
+                if (midcom::get()->auth->admin
295
+                    || (midcom::get()->auth->user
296 296
                         && $comment->can_do('midgard:delete'))) {
297 297
                     midcom_show_style('comments-admintoolbar');
298 298
                 }
@@ -302,7 +302,7 @@  discard block
 block discarded – undo
302 302
             midcom_show_style('comments-nonefound');
303 303
         }
304 304
 
305
-        if (   midcom::get()->auth->user
305
+        if (midcom::get()->auth->user
306 306
             || $this->_config->get('allow_anonymous')) {
307 307
             midcom_show_style('post-comment');
308 308
         } else {
Please login to merge, or discard this patch.
lib/net/nehmer/comments/handler/moderate.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -54,7 +54,7 @@
 block discarded – undo
54 54
         $message['content'] = '';
55 55
         $logs = $comment->get_logs();
56 56
         if (!empty($logs)) {
57
-            $message['content'] .= $this->_l10n->get('moderation history').":\n\n";
57
+            $message['content'] .= $this->_l10n->get('moderation history') . ":\n\n";
58 58
             foreach ($logs as $time => $log) {
59 59
                 $reported = $this->_l10n->get_formatter()->datetime(strtotime("{$time}Z"));
60 60
                 $message['content'] .= $this->_l10n->get(sprintf('%s: %s by %s (from %s)', "$reported:\n", $this->_l10n->get($log['action']), $log['reporter'], $log['ip'])) . "\n\n";
Please login to merge, or discard this patch.
lib/net/nehmer/blog/handler/view.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -48,7 +48,7 @@  discard block
 block discarded – undo
48 48
             ]);
49 49
         }
50 50
 
51
-        if (   $this->_article->topic === $this->_topic->id
51
+        if ($this->_article->topic === $this->_topic->id
52 52
             && $this->_article->can_do('midgard:delete')) {
53 53
             $delete = $this->get_workflow('delete', ['object' => $this->_article]);
54 54
             $buttons[] = $delete->get_button("delete/{$this->_article->guid}/");
@@ -83,7 +83,7 @@  discard block
 block discarded – undo
83 83
         if ($this->_config->get('comments_enable')) {
84 84
             if ($node = net_nehmer_comments_interface::get_node($this->_topic, $this->_config->get('comments_topic'))) {
85 85
                 $this->_request_data['comments_url'] = $node[MIDCOM_NAV_RELATIVEURL] . "comment/{$this->_article->guid}";
86
-                if (   $this->_topic->can_do('midgard:update')
86
+                if ($this->_topic->can_do('midgard:update')
87 87
                     && $this->_topic->can_do('net.nehmer.comments:moderation')) {
88 88
                     net_nehmer_comments_viewer::add_head_elements();
89 89
                 }
Please login to merge, or discard this patch.
lib/org/openpsa/directmarketing/importer/csv.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -55,11 +55,11 @@
 block discarded – undo
55 55
         foreach (array_filter($csv_line) as $field => $value) {
56 56
             // Process the row accordingly
57 57
             $field_matching = $this->_settings['fields'][$field];
58
-            if (   $field_matching
58
+            if ($field_matching
59 59
                 && strstr($field_matching, ':')) {
60 60
                 [$schemadb, $schema_field] = explode(':', $field_matching);
61 61
 
62
-                if (   !array_key_exists($schemadb, $this->_schemadbs)
62
+                if (!array_key_exists($schemadb, $this->_schemadbs)
63 63
                     || !$this->_schemadbs[$schemadb]->get('default')->has_field($schema_field)) {
64 64
                     // Invalid matching, skip
65 65
                     continue;
Please login to merge, or discard this patch.