Passed
Push — master ( f0a80d...74d5c1 )
by Andreas
18:22
created
lib/midcom/services/i18n/main.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -101,7 +101,7 @@
 block discarded – undo
101 101
         $rawdata = base64_decode($request->cookies->get('midcom_services_i18n'));
102 102
         $array = unserialize($rawdata);
103 103
 
104
-        if (   !array_key_exists('language', $array)
104
+        if (!array_key_exists('language', $array)
105 105
             || !array_key_exists('charset', $array)) {
106 106
             debug_add("Rejecting cookie, it seems invalid.");
107 107
             return false;
Please login to merge, or discard this patch.
lib/org/openpsa/slideshow/handler/edit.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -61,7 +61,7 @@  discard block
 block discarded – undo
61 61
         $images = $qb->execute();
62 62
         $failed = 0;
63 63
         foreach ($images as $image) {
64
-            if (   !$image->generate_image('thumbnail', $this->_config->get('thumbnail_filter'))
64
+            if (!$image->generate_image('thumbnail', $this->_config->get('thumbnail_filter'))
65 65
                 || !$image->generate_image('image', $this->_config->get('image_filter'))) {
66 66
                 $failed++;
67 67
             }
@@ -241,7 +241,7 @@  discard block
 block discarded – undo
241 241
         $attachment->title = $title;
242 242
         $attachment->mimetype = $file['type'];
243 243
         $attachment->parentguid = $image->guid;
244
-        if (   !$attachment->create()
244
+        if (!$attachment->create()
245 245
             || !$attachment->copy_from_file($file['tmp_name'])) {
246 246
             throw new midcom_error('Failed to create attachment: ' . midcom_connection::get_error_string());
247 247
         }
Please login to merge, or discard this patch.
lib/midcom/helper/reflector/nameresolver.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -36,7 +36,7 @@  discard block
 block discarded – undo
36 36
         if ($name_property === null) {
37 37
             $name_property = midcom_helper_reflector::get_name_property($this->_object);
38 38
         }
39
-        if (    empty($name_property)
39
+        if (empty($name_property)
40 40
             || !midcom_helper_reflector::get($this->_object)->property_exists($name_property)) {
41 41
             // Could not resolve valid property
42 42
             return null;
@@ -132,7 +132,7 @@  discard block
 block discarded – undo
132 132
 
133 133
             $qb->add_constraint($child_name_property, '=', $name);
134 134
             if ($qb->count()) {
135
-                debug_add("Name clash in sibling class {$schema_type} for " . get_class($this->_object) . " #{$this->_object->id} (path '" . midcom_helper_reflector_tree::resolve_path($this->_object, '/') . "')" );
135
+                debug_add("Name clash in sibling class {$schema_type} for " . get_class($this->_object) . " #{$this->_object->id} (path '" . midcom_helper_reflector_tree::resolve_path($this->_object, '/') . "')");
136 136
                 return false;
137 137
             }
138 138
         }
@@ -209,7 +209,7 @@  discard block
 block discarded – undo
209 209
         } while (!$this->name_is_unique());
210 210
 
211 211
         // Get a copy of the current, usable name
212
-        $ret = (string)$this->_object->{$name_prop};
212
+        $ret = (string) $this->_object->{$name_prop};
213 213
         // Restore the original name
214 214
         $this->_object->{$name_prop} = $original_name;
215 215
         return $ret;
Please login to merge, or discard this patch.
lib/midcom/application.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -97,7 +97,7 @@
 block discarded – undo
97 97
             $loader->load($this->getProjectDir() . '/config/services.yml');
98 98
         }
99 99
         if ($classes = midcom::get_registered_service_classes()) {
100
-            $loader->load(function (ContainerBuilder $container) use ($classes) {
100
+            $loader->load(function(ContainerBuilder $container) use ($classes) {
101 101
                 foreach ($classes as $id => $class) {
102 102
                     $container->findDefinition($id)->setClass($class);
103 103
                 }
Please login to merge, or discard this patch.
lib/midcom/helper/style.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -152,7 +152,7 @@
 block discarded – undo
152 152
         $preparsed = midcom_helper_formatter::compile($style);
153 153
 
154 154
         if (midcom_core_context::get()->has_custom_key('request_data')) {
155
-            $data =& midcom_core_context::get()->get_custom_key('request_data');
155
+            $data = & midcom_core_context::get()->get_custom_key('request_data');
156 156
         }
157 157
 
158 158
         try {
Please login to merge, or discard this patch.
lib/midcom/helper/formatter.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -52,7 +52,7 @@  discard block
 block discarded – undo
52 52
      */
53 53
     public static function compile(string $content) : string
54 54
     {
55
-        return preg_replace_callback("%&\(([^)]*)\);%i", function ($variable)
55
+        return preg_replace_callback("%&\(([^)]*)\);%i", function($variable)
56 56
         {
57 57
             $variable_parts = explode(':', $variable[1]);
58 58
             $variable = '$' . $variable_parts[0];
@@ -62,7 +62,7 @@  discard block
 block discarded – undo
62 62
                 $variable = $parts[0] . '->' . $parts[1];
63 63
             }
64 64
 
65
-            if (    isset($variable_parts[1])
65
+            if (isset($variable_parts[1])
66 66
                 && array_key_exists($variable_parts[1], self::$_filters)) {
67 67
                     switch ($variable_parts[1]) {
68 68
                         case 's':
Please login to merge, or discard this patch.
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.