Passed
Push — master ( bc24ea...ec5c87 )
by Andreas
26:16
created
lib/net/nehmer/blog/viewer.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -84,7 +84,7 @@  discard block
 block discarded – undo
84 84
             net_nemein_rss_manage::add_toolbar_buttons($this->_node_toolbar, $this->_topic->can_do('midgard:create'));
85 85
         }
86 86
 
87
-        if (   $this->_topic->can_do('midgard:update')
87
+        if ($this->_topic->can_do('midgard:update')
88 88
             && $this->_topic->can_do('midcom:component_config')) {
89 89
             $buttons[] = $workflow->get_button('config/', [
90 90
                 MIDCOM_TOOLBAR_LABEL => $this->_l10n_midcom->get('component configuration'),
@@ -114,7 +114,7 @@  discard block
 block discarded – undo
114 114
             $this->_request_data['categories'] = explode(',', $this->_config->get('categories'));
115 115
 
116 116
             foreach ($this->_request_data['schemadb']->all() as $schema) {
117
-                if (   $schema->has_field('categories')
117
+                if ($schema->has_field('categories')
118 118
                     && $schema->get_field('categories')['type'] == 'select') {
119 119
                     // TODO: Merge schema local and component config categories?
120 120
                     $options = array_combine($this->_request_data['categories'], $this->_request_data['categories']);
Please login to merge, or discard this patch.
lib/org/openpsa/invoices/billing/data.php 3 patches
Indentation   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -61,7 +61,7 @@
 block discarded – undo
61 61
     public static function get_by_object(org_openpsa_invoices_interfaces_customer $object) : self
62 62
     {
63 63
         if (   !($bd = self::get_billing_data(org_openpsa_contacts_group_dba::class, $object->customer))
64
-               // check if the customerContact is set and has invoice_data
64
+                // check if the customerContact is set and has invoice_data
65 65
             && !($bd = self::get_billing_data(org_openpsa_contacts_person_dba::class, $object->customerContact))) {
66 66
             $bd = new self();
67 67
             $due = midcom_baseclasses_components_configuration::get('org.openpsa.invoices', 'config')->get('default_due_days');
Please login to merge, or discard this patch.
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -60,7 +60,7 @@
 block discarded – undo
60 60
      */
61 61
     public static function get_by_object(org_openpsa_invoices_interfaces_customer $object) : self
62 62
     {
63
-        if (   !($bd = self::get_billing_data(org_openpsa_contacts_group_dba::class, $object->customer))
63
+        if (!($bd = self::get_billing_data(org_openpsa_contacts_group_dba::class, $object->customer))
64 64
                // check if the customerContact is set and has invoice_data
65 65
             && !($bd = self::get_billing_data(org_openpsa_contacts_person_dba::class, $object->customerContact))) {
66 66
             $bd = new self();
Please login to merge, or discard this patch.
Braces   +6 added lines, -3 removed lines patch added patch discarded remove patch
@@ -96,7 +96,8 @@  discard block
 block discarded – undo
96 96
             // if the property useContactAddress is set
97 97
             $billing_data[0]->set_address();
98 98
             return $billing_data[0];
99
-        } catch (midcom_error $e) {
99
+        }
100
+        catch (midcom_error $e) {
100 101
             $e->log();
101 102
             return null;
102 103
         }
@@ -141,10 +142,12 @@  discard block
 block discarded – undo
141 142
     {
142 143
         try {
143 144
             return new org_openpsa_contacts_person_dba($this->linkGuid);
144
-        } catch (midcom_error $e) {
145
+        }
146
+        catch (midcom_error $e) {
145 147
             try {
146 148
                 return new org_openpsa_contacts_group_dba($this->linkGuid);
147
-            } catch (midcom_error $e) {
149
+            }
150
+            catch (midcom_error $e) {
148 151
                 debug_add("Failed to load contact with GUID: " . $this->linkGuid . " - last error:" . $e->getMessage(), MIDCOM_LOG_ERROR);
149 152
                 return false;
150 153
             }
Please login to merge, or discard this patch.
lib/net/nehmer/static/handler/admin.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -26,7 +26,7 @@
 block discarded – undo
26 26
 
27 27
     private function load_controller() : controller
28 28
     {
29
-        if (    $this->_config->get('simple_name_handling')
29
+        if ($this->_config->get('simple_name_handling')
30 30
              && !midcom::get()->auth->admin) {
31 31
             foreach ($this->_request_data['schemadb']->all() as $schema) {
32 32
                 $schema->get_field('name')['readonly'] = true;
Please login to merge, or discard this patch.
lib/midcom/services/toolbars.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -314,7 +314,7 @@
 block discarded – undo
314 314
 
315 315
     private function add_head_elements(bool $centralized = false) : bool
316 316
     {
317
-        if (   !midcom::get()->auth->user
317
+        if (!midcom::get()->auth->user
318 318
             || !midcom::get()->config->get('toolbars_enable_centralized')
319 319
             || !midcom::get()->auth->can_user_do('midcom:centralized_toolbar', null, __CLASS__)) {
320 320
             return false;
Please login to merge, or discard this patch.
lib/midcom/services/metadata.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -109,20 +109,20 @@  discard block
 block discarded – undo
109 109
         }
110 110
 
111 111
         // Approval attributes
112
-        if (   midcom::get()->config->get('metadata_approval')
112
+        if (midcom::get()->config->get('metadata_approval')
113 113
             && !$object->metadata->is_approved()) {
114 114
             $css_classes[] = 'unapproved';
115 115
         }
116 116
 
117 117
         // Hiding and scheduling attributes
118
-        if (   (   !midcom::get()->config->get('show_hidden_objects')
118
+        if ((!midcom::get()->config->get('show_hidden_objects')
119 119
                 || midcom::get()->config->get('metadata_scheduling'))
120 120
             && !$object->metadata->is_visible()) {
121 121
             $css_classes[] = 'hidden';
122 122
         }
123 123
 
124 124
         // Folder's class
125
-        if (   $object instanceof midcom_db_topic
125
+        if ($object instanceof midcom_db_topic
126 126
             && $page_class = $object->get_parameter('midcom.services.metadata', 'page_class')) {
127 127
             $css_classes[] = $page_class;
128 128
         }
@@ -193,7 +193,7 @@  discard block
 block discarded – undo
193 193
     private function _add_opengraph_metadata(midcom_helper_metadata $view_metadata)
194 194
     {
195 195
         $opengraph_type = $view_metadata->object->get_parameter('midcom.helper.metadata', 'opengraph_type');
196
-        if (   $opengraph_type
196
+        if ($opengraph_type
197 197
             && $opengraph_type != 'none') {
198 198
             $request_metadata = $this->get_request_metadata();
199 199
 
Please login to merge, or discard this patch.
lib/net/nehmer/comments/handler/admin.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -111,7 +111,7 @@
 block discarded – undo
111 111
      */
112 112
     public function _handler_moderate_ajax(Request $request, string $status, array &$data)
113 113
     {
114
-        if (   !$request->request->has('action')
114
+        if (!$request->request->has('action')
115 115
             || !$request->request->has('guid')) {
116 116
             throw new midcom_error_notfound('Incomplete POST data');
117 117
         }
Please login to merge, or discard this patch.
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.