@@ -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 | } |
@@ -116,7 +116,8 @@ discard block |
||
| 116 | 116 | //TODO: check for duplicates ? |
| 117 | 117 | try { |
| 118 | 118 | $result = ['other_obj' => midcom::get()->dbfactory->get_object_by_guid($link[$other . 'Guid'])]; |
| 119 | - } catch (midcom_error $e) { |
|
| 119 | + } |
|
| 120 | + catch (midcom_error $e) { |
|
| 120 | 121 | continue; |
| 121 | 122 | } |
| 122 | 123 | $result['link'] = [ |
@@ -421,7 +422,8 @@ discard block |
||
| 421 | 422 | if (!($this->_object instanceof org_openpsa_relatedto_dba)) { |
| 422 | 423 | $response->status = "method requires guid of a link object as an argument"; |
| 423 | 424 | } |
| 424 | - } catch (midcom_error $e) { |
|
| 425 | + } |
|
| 426 | + catch (midcom_error $e) { |
|
| 425 | 427 | $response->status = "error: " . $e->getMessage(); |
| 426 | 428 | } |
| 427 | 429 | |
@@ -444,7 +446,8 @@ discard block |
||
| 444 | 446 | $relation = new org_openpsa_relatedto_dba($guid); |
| 445 | 447 | $response->result = $relation->delete(); |
| 446 | 448 | $response->status = 'Last message: ' . midcom_connection::get_error_string(); |
| 447 | - } catch (midcom_error $e) { |
|
| 449 | + } |
|
| 450 | + catch (midcom_error $e) { |
|
| 448 | 451 | $response->result = false; |
| 449 | 452 | $response->status = "Object '{$guid}' could not be loaded, error:" . $e->getMessage(); |
| 450 | 453 | } |
@@ -199,7 +199,7 @@ |
||
| 199 | 199 | private function _render_line(array $link, midcom_core_dbaobject &$other_obj) |
| 200 | 200 | { |
| 201 | 201 | $this->_request_data['link'] = $link; |
| 202 | - $this->_request_data['other_obj'] =& $other_obj; |
|
| 202 | + $this->_request_data['other_obj'] = & $other_obj; |
|
| 203 | 203 | $this->_request_data['icon'] = midcom_helper_reflector::get_object_icon($other_obj); |
| 204 | 204 | |
| 205 | 205 | if (get_class($other_obj) != $link['class']) { |
@@ -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 " <td>" . strftime('%x %X Z', $revision['date']) . "</td>\n"; |
| 37 | 37 | echo " <td>"; |
| 38 | 38 | |
| 39 | - if ( $revision['user'] |
|
| 39 | + if ($revision['user'] |
|
| 40 | 40 | && $user = midcom::get()->auth->get_user($revision['user'])) { |
| 41 | 41 | echo $user->get_storage()->name; |
| 42 | 42 | } elseif ($revision['ip']) { |
@@ -6,6 +6,6 @@ |
||
| 6 | 6 | &(view["content"]:h); |
| 7 | 7 | <?php |
| 8 | 8 | } else { |
| 9 | - echo "<p class=\"stub\">" . $data['l10n']->get('this page is stub')."</p>"; |
|
| 9 | + echo "<p class=\"stub\">" . $data['l10n']->get('this page is stub') . "</p>"; |
|
| 10 | 10 | } |
| 11 | 11 | ?> |
| 12 | 12 | \ No newline at end of file |