@@ -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 | } |
@@ -39,7 +39,7 @@ |
||
39 | 39 | */ |
40 | 40 | public function read_login_data(Request $request) : ?array |
41 | 41 | { |
42 | - if ( !$request->request->has('midcom_services_auth_frontend_form_submit') |
|
42 | + if (!$request->request->has('midcom_services_auth_frontend_form_submit') |
|
43 | 43 | || !$request->request->has('username') |
44 | 44 | || !$request->request->has('password')) { |
45 | 45 | return null; |
@@ -44,7 +44,7 @@ discard block |
||
44 | 44 | $workflow = $this->get_workflow('datamanager', ['controller' => $data['controller']]); |
45 | 45 | $response = $workflow->run($request); |
46 | 46 | |
47 | - if ( $workflow->get_state() == 'save' |
|
47 | + if ($workflow->get_state() == 'save' |
|
48 | 48 | && $this->create_account($this->person, $data["controller"]->get_form_values())) { |
49 | 49 | midcom::get()->uimessages->add($this->_l10n->get('org.openpsa.user'), sprintf($this->_l10n->get('person %s created'), $this->person->name)); |
50 | 50 | } |
@@ -77,7 +77,7 @@ discard block |
||
77 | 77 | $person = new midcom_db_person($guid); |
78 | 78 | $accounthelper = new org_openpsa_user_accounthelper($person); |
79 | 79 | if ($accounthelper->welcome_email()) { |
80 | - midcom::get()->uimessages->add($this->_l10n->get($this->_component), sprintf($this->_l10n->get('password reset and mail to %s sent'), $person->email )); |
|
80 | + midcom::get()->uimessages->add($this->_l10n->get($this->_component), sprintf($this->_l10n->get('password reset and mail to %s sent'), $person->email)); |
|
81 | 81 | } else { |
82 | 82 | midcom::get()->uimessages->add($this->_l10n->get($this->_component), $accounthelper->errstr, 'error'); |
83 | 83 | } |
@@ -133,7 +133,7 @@ discard block |
||
133 | 133 | $accounthelper = new org_openpsa_user_accounthelper($this->person); |
134 | 134 | |
135 | 135 | // Update account |
136 | - if ( !$accounthelper->set_account($controller->get_form_values()["username"], $password) |
|
136 | + if (!$accounthelper->set_account($controller->get_form_values()["username"], $password) |
|
137 | 137 | && midcom_connection::get_error() != MGD_ERR_OK) { |
138 | 138 | // Failure, give a message |
139 | 139 | midcom::get()->uimessages->add($this->_l10n->get('org.openpsa.user'), $this->_l10n->get("failed to update the user account, reason") . ': ' . midcom_connection::get_error_string(), 'error'); |