Passed
Push — master ( dc3f28...9cb9a1 )
by Andreas
16:51
created
lib/org/openpsa/sales/handler/deliverable/add.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -86,7 +86,7 @@
 block discarded – undo
86 86
      */
87 87
     public function _handler_add(Request $request, string $guid, array &$data)
88 88
     {
89
-        if (   !$request->request->has('product')
89
+        if (!$request->request->has('product')
90 90
             && !$request->request->has('org_openpsa_sales')) {
91 91
             throw new midcom_error('No product specified, aborting.');
92 92
         }
Please login to merge, or discard this patch.
lib/midcom/admin/folder/handler/approvals.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -21,7 +21,7 @@
 block discarded – undo
21 21
      */
22 22
     public function _handler_approval(Request $request, string $handler_id)
23 23
     {
24
-        if (   !$request->request->has('guid')
24
+        if (!$request->request->has('guid')
25 25
             || !$request->request->has('return_to')) {
26 26
             throw new midcom_error('Cannot process approval request, request is incomplete.');
27 27
         }
Please login to merge, or discard this patch.
lib/midcom/helper/nav/backend.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -279,7 +279,7 @@  discard block
 block discarded – undo
279 279
             }
280 280
 
281 281
             $this->_leaves[$id] = $leaf;
282
-            $this->_loaded_leaves[$node->id][$id] =& $this->_leaves[$id];
282
+            $this->_loaded_leaves[$node->id][$id] = & $this->_leaves[$id];
283 283
         }
284 284
     }
285 285
 
@@ -337,7 +337,7 @@  discard block
 block discarded – undo
337 337
                     continue;
338 338
                 }
339 339
 
340
-                if (   !$show_noentry
340
+                if (!$show_noentry
341 341
                     && self::$_nodes[$id]->noentry) {
342 342
                     // Hide "noentry" items
343 343
                     continue;
Please login to merge, or discard this patch.
lib/midcom/core/context.php 2 patches
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -165,7 +165,8 @@
 block discarded – undo
165 165
                 try {
166 166
                     self::$root_topic = midcom_db_topic::get_cached($guid);
167 167
                     return self::$root_topic;
168
-                } catch (midcom_error $e) {
168
+                }
169
+                catch (midcom_error $e) {
169 170
                     if ($e instanceof midcom_error_forbidden) {
170 171
                         throw $e;
171 172
                     }
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -254,7 +254,7 @@
 block discarded – undo
254 254
      */
255 255
     public function set_custom_key($key, &$value)
256 256
     {
257
-        $this->_data[MIDCOM_CONTEXT_CUSTOMDATA][$key] =& $value;
257
+        $this->_data[MIDCOM_CONTEXT_CUSTOMDATA][$key] = & $value;
258 258
     }
259 259
 
260 260
     public function show()
Please login to merge, or discard this patch.
lib/org/openpsa/directmarketing/style/show-campaign.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -50,14 +50,14 @@
 block discarded – undo
50 50
     </div>
51 51
 
52 52
     <script type="text/javascript">
53
-    $('#<?php echo $grid->get_identifier();?>').on('click', 'td.delete i', function(event) {
53
+    $('#<?php echo $grid->get_identifier(); ?>').on('click', 'td.delete i', function(event) {
54 54
         var guid = $(event.target).data('person-guid'),
55 55
             member_guid = $(event.target).data('member-guid'),
56 56
             post_data = {org_openpsa_ajax_mode: 'unsubscribe', org_openpsa_ajax_person_guid: guid};
57 57
 
58 58
         $(event.target).removeClass('fa-trash').addClass('fa-spin fa-spinner');
59 59
         $.post('&(node[MIDCOM_NAV_ABSOLUTEURL]);campaign/unsubscribe/ajax/' + member_guid + '/', post_data, function() {
60
-            $('#<?php echo $grid->get_identifier();?>').trigger('reloadGrid');
60
+            $('#<?php echo $grid->get_identifier(); ?>').trigger('reloadGrid');
61 61
             $('#org_openpsa_widgets_contact-' + guid).fadeOut('fast', function() {
62 62
                 $('#org_openpsa_widgets_contact-' + guid).remove();
63 63
             });
Please login to merge, or discard this patch.
src/midcom/datamanager/extension/type/urlnameType.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -22,7 +22,7 @@  discard block
 block discarded – undo
22 22
      */
23 23
     public function configureOptions(OptionsResolver $resolver)
24 24
     {
25
-        $resolver->setNormalizer('write_privilege', function (Options $options, $value) {
25
+        $resolver->setNormalizer('write_privilege', function(Options $options, $value) {
26 26
             return (array) $value + ['privilege' => 'midcom:urlname'];
27 27
         });
28 28
 
@@ -36,7 +36,7 @@  discard block
 block discarded – undo
36 36
             ]
37 37
         ]);
38 38
 
39
-        $resolver->setNormalizer('constraints', function (Options $options, $value) {
39
+        $resolver->setNormalizer('constraints', function(Options $options, $value) {
40 40
             $validator_options = [
41 41
                 'allow_catenate' => $options['type_config']['allow_catenate'],
42 42
                 'allow_unclean' => $options['type_config']['allow_unclean'],
Please login to merge, or discard this patch.
src/midcom/datamanager/storage/container/dbacontainer.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -36,7 +36,7 @@
 block discarded – undo
36 36
             }
37 37
             $config['name'] = $name;
38 38
             $field = $this->prepare_field($config);
39
-            if (   isset($config['default'])
39
+            if (isset($config['default'])
40 40
                 && (!$this->object->id || $field instanceof transientnode)) {
41 41
                 $field->set_value($config['default']);
42 42
             }
Please login to merge, or discard this patch.
lib/midcom/response/styled.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -55,7 +55,7 @@
 block discarded – undo
55 55
 
56 56
     private function set_context(midcom_core_context $context)
57 57
     {
58
-        if (   $context->get_key(MIDCOM_CONTEXT_PERMALINKGUID) === null
58
+        if ($context->get_key(MIDCOM_CONTEXT_PERMALINKGUID) === null
59 59
             || $context->get_key(MIDCOM_CONTEXT_PAGETITLE) == '') {
60 60
             // Retrieve Metadata
61 61
             $nav = new midcom_helper_nav();
Please login to merge, or discard this patch.
src/midcom/datamanager/validation/imageValidator.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -12,10 +12,10 @@
 block discarded – undo
12 12
 {
13 13
     public function validate($value, Constraint $constraint)
14 14
     {
15
-        if (   empty($value)
15
+        if (empty($value)
16 16
             || !empty($value['delete'])
17
-            || (   empty($value['file'])
18
-                && (   (empty($constraint->config['do_not_save_archival']) && empty($value['archival']))
17
+            || (empty($value['file'])
18
+                && ((empty($constraint->config['do_not_save_archival']) && empty($value['archival']))
19 19
                     || (!empty($constraint->config['do_not_save_archival']) && empty($value['main']))))) {
20 20
             $this->context->buildViolation('This value should not be blank.')
21 21
                 ->addViolation();
Please login to merge, or discard this patch.