@@ -142,7 +142,7 @@ discard block |
||
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 |
||
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 |
||
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}/", |
@@ -90,7 +90,7 @@ |
||
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 | } |
@@ -246,7 +246,7 @@ |
||
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 |
@@ -58,7 +58,7 @@ discard block |
||
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 |
||
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 | } |
@@ -95,7 +95,7 @@ discard block |
||
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 |
||
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 | } |
@@ -90,7 +90,7 @@ |
||
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 | } |
@@ -67,7 +67,7 @@ |
||
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'), |
@@ -56,7 +56,7 @@ discard block |
||
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 |
||
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 |
||
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 | } |
@@ -270,7 +270,7 @@ discard block |
||
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 |
||
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; |