@@ -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; |