Passed
Push — master ( f70f67...05b81b )
by Andreas
28:08
created
lib/midgard/admin/asgard/toolbar.php 1 patch
Spacing   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -142,7 +142,7 @@  discard block
 block discarded – undo
142 142
             ];
143 143
         }
144 144
 
145
-        if (   midcom::get()->config->get('midcom_services_rcs_enable')
145
+        if (midcom::get()->config->get('midcom_services_rcs_enable')
146 146
             && $object->can_do('midgard:update')
147 147
             && $object->_use_rcs) {
148 148
             $buttons[] = [
@@ -203,7 +203,7 @@  discard block
 block discarded – undo
203 203
     private function get_toolbar_update_items(midcom_core_dbaobject $object) : array
204 204
     {
205 205
         $buttons = [];
206
-        if (   $object instanceof midcom_db_topic
206
+        if ($object instanceof midcom_db_topic
207 207
             && $object->component
208 208
             && $object->can_do('midcom:component_config')) {
209 209
             $buttons[] = [
@@ -241,7 +241,7 @@  discard block
 block discarded – undo
241 241
             MIDCOM_TOOLBAR_ENABLED => $object->can_do('midgard:privileges'),
242 242
         ];
243 243
 
244
-        if (   midcom::get()->componentloader->is_installed('midcom.helper.replicator')
244
+        if (midcom::get()->componentloader->is_installed('midcom.helper.replicator')
245 245
             && midcom::get()->auth->admin) {
246 246
             $buttons[] = [
247 247
                 MIDCOM_TOOLBAR_URL => "__mfa/asgard_midcom.helper.replicator/object/{$object->guid}/",
Please login to merge, or discard this patch.
src/midcom/httpkernel/subscriber.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -90,7 +90,7 @@
 block discarded – undo
90 90
         $i = array_search('__request_data__', $arguments, true);
91 91
         if ($i !== false) {
92 92
             $context = $event->getRequest()->attributes->get('context');
93
-            $arguments[$i] =& $context->get_custom_key('request_data');
93
+            $arguments[$i] = & $context->get_custom_key('request_data');
94 94
             $event->setArguments($arguments);
95 95
         }
96 96
     }
Please login to merge, or discard this patch.
lib/midcom/baseclasses/components/viewer.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -246,7 +246,7 @@
 block discarded – undo
246 246
      */
247 247
     public function prepare_handler(midcom_baseclasses_components_handler $handler, array &$parameters)
248 248
     {
249
-        $this->parameters =& $parameters;
249
+        $this->parameters = & $parameters;
250 250
 
251 251
         $parameters['handler'] = explode('::', $parameters['_controller'], 2);
252 252
 
Please login to merge, or discard this patch.
lib/org/openpsa/projects/handler/task/resourcing.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -58,7 +58,7 @@  discard block
 block discarded – undo
58 58
             return new midcom_response_relocate($this->router->generate('task_view', ['guid' => $guid]));
59 59
         }
60 60
 
61
-        if (   $request->request->has('save')
61
+        if ($request->request->has('save')
62 62
             && $prospects = $request->request->all('org_openpsa_projects_prospects')) {
63 63
             $qb = org_openpsa_projects_task_resource_dba::new_query_builder();
64 64
             $qb->add_constraint('guid', 'IN', array_keys($prospects));
@@ -89,7 +89,7 @@  discard block
 block discarded – undo
89 89
                     // create relatedto
90 90
                     org_openpsa_relatedto_plugin::create($event, 'org.openpsa.calendar', $this->_task, 'org.openpsa.projects');
91 91
                 }
92
-                if (   $update_prospect
92
+                if ($update_prospect
93 93
                     && !$prospect->update()) {
94 94
                     debug_add('Failed to update prospect: ' . midcom_connection::get_error_string(), MIDCOM_LOG_ERROR);
95 95
                 }
Please login to merge, or discard this patch.
lib/org/openpsa/invoices/scheduler.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -95,7 +95,7 @@  discard block
 block discarded – undo
95 95
         }
96 96
 
97 97
         // TODO: Warehouse management: create new order
98
-        if (   $this->_deliverable->end < $next_cycle_start
98
+        if ($this->_deliverable->end < $next_cycle_start
99 99
             && $this->_deliverable->end != 0) {
100 100
             debug_add('Do not register next cycle, the contract ends before');
101 101
             return $this->_deliverable->end_subscription();
@@ -328,7 +328,7 @@  discard block
 block discarded – undo
328 328
         }
329 329
 
330 330
         //If previous cycle was run at the end of the month, the new one should be at the end of the month as well
331
-        if (   $date->format('t') == $date->format('j')
331
+        if ($date->format('t') == $date->format('j')
332 332
             && $new_date->format('t') != $new_date->format('j')) {
333 333
             $new_date->setDate((int) $new_date->format('Y'), (int) $new_date->format('m'), (int) $new_date->format('t'));
334 334
         }
Please login to merge, or discard this patch.
lib/org/openpsa/documents/handler/document/admin.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -90,7 +90,7 @@
 block discarded – undo
90 90
 
91 91
         $this->_controller = $this->load_controller();
92 92
 
93
-        if (   $data['enable_versioning']
93
+        if ($data['enable_versioning']
94 94
             && $request->request->count() > 0) {
95 95
             $this->_backup_attachment($request);
96 96
         }
Please login to merge, or discard this patch.
lib/org/openpsa/core/queryfilter.php 1 patch
Spacing   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -67,7 +67,7 @@
 block discarded – undo
67 67
         $user = midcom::get()->auth->user->get_storage();
68 68
 
69 69
         if ($request->request->get('unset_filter') == $filter_id) {
70
-            if (   $user->get_parameter("org_openpsa_core_filter", $filter_id)
70
+            if ($user->get_parameter("org_openpsa_core_filter", $filter_id)
71 71
                 && !$user->delete_parameter("org_openpsa_core_filter", $filter_id)) {
72 72
                 $message_content = sprintf(
73 73
                     $l10n->get('the handed filter for %s could not be set as parameter'),
Please login to merge, or discard this patch.
lib/org/openpsa/reports/handler/invoices/report.php 1 patch
Spacing   +5 added lines, -5 removed lines patch added patch discarded remove patch
@@ -56,7 +56,7 @@  discard block
 block discarded – undo
56 56
 
57 57
     private function _get_invoices_for_subscription(org_openpsa_sales_salesproject_deliverable_dba $deliverable, midcom_services_at_entry_dba $at_entry) : array
58 58
     {
59
-        if (   $deliverable->invoiceByActualUnits
59
+        if ($deliverable->invoiceByActualUnits
60 60
             && $at_entry->arguments['cycle'] > 1) {
61 61
             $months = midcom_baseclasses_components_configuration::get('org.openpsa.sales', 'config')->get('subscription_profit_months');
62 62
             $cutoff = (new DateTime)->modify('-' . $months . ' months');
@@ -81,8 +81,8 @@  discard block
 block discarded – undo
81 81
         $time = $at_entry->start;
82 82
         $scheduler = new org_openpsa_invoices_scheduler($deliverable);
83 83
 
84
-        while (   $time < $this->_request_data['end']
85
-               && (   $time < $deliverable->end
84
+        while ($time < $this->_request_data['end']
85
+               && ($time < $deliverable->end
86 86
                    || $deliverable->continuous)) {
87 87
             $invoices[] = $this->get_invoice_for_deliverable($deliverable, $invoice_sum, $time, $calculation_base);
88 88
 
@@ -124,8 +124,8 @@  discard block
 block discarded – undo
124 124
         foreach ($qb->execute() as $at_entry) {
125 125
             try {
126 126
                 $deliverable = org_openpsa_sales_salesproject_deliverable_dba::get_cached($at_entry->arguments['deliverable']);
127
-                if (   $deliverable->continuous
128
-                    || (   $deliverable->start < $this->_request_data['end']
127
+                if ($deliverable->continuous
128
+                    || ($deliverable->start < $this->_request_data['end']
129 129
                         && $deliverable->end > $this->_request_data['start'])) {
130 130
                     $invoices = array_merge($invoices, $this->_get_invoices_for_subscription($deliverable, $at_entry));
131 131
                 }
Please login to merge, or discard this patch.
lib/org/openpsa/directmarketing/campaign/ruleresolver.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -270,7 +270,7 @@  discard block
 block discarded – undo
270 270
      */
271 271
     private function add_misc_rule(array $rule, string $class, string $person_property)
272 272
     {
273
-        $persons = [ 0 => -1];
273
+        $persons = [0 => -1];
274 274
         $match = $rule['match'];
275 275
         $constraint_match = "IN";
276 276
         if ($rule['match'] == '<>') {
@@ -342,7 +342,7 @@  discard block
 block discarded – undo
342 342
         $ret = [];
343 343
 
344 344
         foreach ($object->get_properties() as $property) {
345
-            if (   $property[0] == '_'
345
+            if ($property[0] == '_'
346 346
                 || in_array($property, $skip_properties)) {
347 347
                 // Skip private or otherwise invalid properties
348 348
                 continue;
Please login to merge, or discard this patch.