@@ -68,7 +68,7 @@ discard block |
||
68 | 68 | $event = new org_openpsa_calendar_event_dba($this->eid); |
69 | 69 | } |
70 | 70 | |
71 | - if ( $recipient->id == midcom_connection::get_user() |
|
71 | + if ($recipient->id == midcom_connection::get_user() |
|
72 | 72 | && !$event->send_notify_me) { |
73 | 73 | //Do not send notification to current user |
74 | 74 | debug_add('event->send_notify_me is false and recipient is current user, aborting notify'); |
@@ -194,7 +194,7 @@ discard block |
||
194 | 194 | |
195 | 195 | if (isset($events_by_date[$ymd])) { |
196 | 196 | foreach ($events_by_date[$ymd] as $event) { |
197 | - if ( $event->end <= $workday_starts_ts |
|
197 | + if ($event->end <= $workday_starts_ts |
|
198 | 198 | || $event->start >= $workday_ends_ts) { |
199 | 199 | // We need not consider this event, it is outside the defined workday |
200 | 200 | continue; |
@@ -212,7 +212,7 @@ discard block |
||
212 | 212 | } |
213 | 213 | } |
214 | 214 | // End of day slot |
215 | - if ( $last_end_time < $workday_ends_ts |
|
215 | + if ($last_end_time < $workday_ends_ts |
|
216 | 216 | && (($workday_ends_ts - $last_end_time) >= $amount)) { |
217 | 217 | $slots[] = self::_create_slot($last_end_time, $workday_ends_ts, $last_event); |
218 | 218 | } |
@@ -143,7 +143,8 @@ |
||
143 | 143 | debug_add('person #' . $person->id . 'has no email address, aborting'); |
144 | 144 | return null; |
145 | 145 | } |
146 | - } catch (midcom_error $e) { |
|
146 | + } |
|
147 | + catch (midcom_error $e) { |
|
147 | 148 | return null; |
148 | 149 | } |
149 | 150 |
@@ -30,7 +30,7 @@ discard block |
||
30 | 30 | private function load_controller(org_openpsa_invoices_billing_data_dba $bd) : controller |
31 | 31 | { |
32 | 32 | $schemadb = schemadb::from_path($this->_config->get('schemadb_billing_data')); |
33 | - $vat =& $schemadb->get('default')->get_field('vat'); |
|
33 | + $vat = & $schemadb->get('default')->get_field('vat'); |
|
34 | 34 | // Fill VAT select |
35 | 35 | if ($options = $this->get_vat_options($this->_config->get('vat_percentages'))) { |
36 | 36 | $vat['type_config']['options'] = $options; |
@@ -68,7 +68,7 @@ discard block |
||
68 | 68 | $data['controller'] = $this->load_controller($bd); |
69 | 69 | |
70 | 70 | $workflow = $this->get_workflow('datamanager', ['controller' => $data['controller']]); |
71 | - if ( $mode == 'edit' |
|
71 | + if ($mode == 'edit' |
|
72 | 72 | && $bd->can_do('midgard:delete')) { |
73 | 73 | $delete = $this->get_workflow('delete', [ |
74 | 74 | 'object' => $bd, |
@@ -79,7 +79,8 @@ discard block |
||
79 | 79 | try { |
80 | 80 | $dms['group']->set_storage($group); |
81 | 81 | org_openpsa_products_viewer::index($dms['group'], $indexer, $topic, $config); |
82 | - } catch (midcom_error $e) { |
|
82 | + } |
|
83 | + catch (midcom_error $e) { |
|
83 | 84 | $e->log(MIDCOM_LOG_WARN); |
84 | 85 | } |
85 | 86 | } |
@@ -91,7 +92,8 @@ discard block |
||
91 | 92 | try { |
92 | 93 | $dms['product']->set_storage($product); |
93 | 94 | org_openpsa_products_viewer::index($dms['product'], $indexer, $topic, $config); |
94 | - } catch (midcom_error $e) { |
|
95 | + } |
|
96 | + catch (midcom_error $e) { |
|
95 | 97 | $e->log(MIDCOM_LOG_WARN); |
96 | 98 | } |
97 | 99 | } |
@@ -52,7 +52,7 @@ |
||
52 | 52 | */ |
53 | 53 | public function _on_reindex($topic, midcom_helper_configuration $config, midcom_services_indexer &$indexer) |
54 | 54 | { |
55 | - if ( !$config->get('index_products') |
|
55 | + if (!$config->get('index_products') |
|
56 | 56 | && !$config->get('index_groups')) { |
57 | 57 | debug_add("No indexing to groups and products, skipping", MIDCOM_LOG_WARN); |
58 | 58 | return true; |
@@ -38,7 +38,7 @@ |
||
38 | 38 | $mail->from = $config->get('default_sender') ?: '"OpenPSA Notifier" <noreply@' . $_SERVER['SERVER_NAME'] . '>'; |
39 | 39 | } |
40 | 40 | |
41 | - $mail->subject = $message['title'] ?? 'org.openpsa.notifications message (no title provided)'; |
|
41 | + $mail->subject = $message['title'] ?? 'org.openpsa.notifications message (no title provided)'; |
|
42 | 42 | if (array_key_exists('attachments', $message)) { |
43 | 43 | $mail->attachments = $message['attachments']; |
44 | 44 | } |
@@ -14,7 +14,7 @@ |
||
14 | 14 | echo "\n<li class=\"separator\"></li>"; |
15 | 15 | echo "\n<li class=\"page splitter\">...</li>"; |
16 | 16 | } |
17 | - if ( $i > 0 |
|
17 | + if ($i > 0 |
|
18 | 18 | && $i < $total_links) { |
19 | 19 | echo "\n<li class=\"separator\"></li>"; |
20 | 20 | } |
@@ -78,7 +78,7 @@ discard block |
||
78 | 78 | } |
79 | 79 | |
80 | 80 | $result_cache[$cache_key] = []; |
81 | - $ret =& $result_cache[$cache_key]; |
|
81 | + $ret = & $result_cache[$cache_key]; |
|
82 | 82 | |
83 | 83 | if (empty($up)) { |
84 | 84 | // TODO: use reflection to see what kind of property this is ? |
@@ -94,7 +94,7 @@ discard block |
||
94 | 94 | |
95 | 95 | $value_properties = array_unique(array_merge($label_fields, [$keyproperty, 'id'])); |
96 | 96 | |
97 | - $mc = self::new_collector('up', (int)$up); |
|
97 | + $mc = self::new_collector('up', (int) $up); |
|
98 | 98 | if ($order_by_score) { |
99 | 99 | $mc->add_order('metadata.score', 'DESC'); |
100 | 100 | } |
@@ -40,7 +40,7 @@ discard block |
||
40 | 40 | */ |
41 | 41 | public function read_login_data(Request $request) : ?array |
42 | 42 | { |
43 | - if ( !$request->request->has('midcom_services_auth_frontend_form_submit') |
|
43 | + if (!$request->request->has('midcom_services_auth_frontend_form_submit') |
|
44 | 44 | || !$request->request->has('username') |
45 | 45 | || !$request->request->has('password')) { |
46 | 46 | return null; |
@@ -76,7 +76,7 @@ discard block |
||
76 | 76 | { |
77 | 77 | // Store the submitted form if the session expired, but user wants to save the data |
78 | 78 | if (!empty($_POST)) { |
79 | - $data =& midcom_core_context::get()->get_custom_key('request_data'); |
|
79 | + $data = & midcom_core_context::get()->get_custom_key('request_data'); |
|
80 | 80 | |
81 | 81 | $data['restored_form_data'] = []; |
82 | 82 |
@@ -40,7 +40,7 @@ |
||
40 | 40 | |
41 | 41 | public function resolve_object_link(midcom_db_topic $topic, midcom_core_dbaobject $object) : ?string |
42 | 42 | { |
43 | - if ( $object instanceof org_openpsa_documents_document_dba |
|
43 | + if ($object instanceof org_openpsa_documents_document_dba |
|
44 | 44 | && $object->topic == $topic->id) { |
45 | 45 | return "document/{$object->guid}/"; |
46 | 46 | } |
@@ -36,7 +36,7 @@ |
||
36 | 36 | echo "<form action=\"{$data['expenses_url']}\" method='post'><div>"; |
37 | 37 | $current_user = midcom::get()->auth->user->get_storage(); |
38 | 38 | echo "<input type=\"hidden\" name=\"person[]\" value=\"{$current_user->id}\" />"; |
39 | - echo "<input type=\"submit\" value=\"".sprintf($data['l10n']->get('see all %s hours'), round($total_hours, 2))."\" />"; |
|
39 | + echo "<input type=\"submit\" value=\"" . sprintf($data['l10n']->get('see all %s hours'), round($total_hours, 2)) . "\" />"; |
|
40 | 40 | echo "</div></form>"; |
41 | 41 | ?> |
42 | 42 | </div> |