@@ -77,11 +77,11 @@ discard block |
||
77 | 77 | |
78 | 78 | public function resolve_object_link(midcom_db_topic $topic, midcom_core_dbaobject $object) : ?string |
79 | 79 | { |
80 | - if ( $object instanceof org_openpsa_contacts_group_dba |
|
80 | + if ($object instanceof org_openpsa_contacts_group_dba |
|
81 | 81 | || $object instanceof midcom_db_group) { |
82 | 82 | return "group/{$object->guid}/"; |
83 | 83 | } |
84 | - if ( $object instanceof org_openpsa_contacts_person_dba |
|
84 | + if ($object instanceof org_openpsa_contacts_person_dba |
|
85 | 85 | || $object instanceof midcom_db_person) { |
86 | 86 | return "person/{$object->guid}/"; |
87 | 87 | } |
@@ -110,8 +110,8 @@ discard block |
||
110 | 110 | // We have a feed URL, but we should check if it is GeoRSS as well |
111 | 111 | $items = net_nemein_rss_fetch::raw_fetch($data['rss_url'])->get_items(); |
112 | 112 | |
113 | - if ( !empty($items) |
|
114 | - && ( $items[0]->get_latitude() |
|
113 | + if (!empty($items) |
|
114 | + && ($items[0]->get_latitude() |
|
115 | 115 | || $items[0]->get_longitude())) { |
116 | 116 | // This is a GeoRSS feed |
117 | 117 | $data['georss_url'] = $data['rss_url']; |
@@ -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> |
@@ -4,7 +4,7 @@ discard block |
||
4 | 4 | $count = count($data['navigation_items']); |
5 | 5 | |
6 | 6 | foreach ($data['navigation_items'] as $i => $item) { |
7 | - if ( isset($item[MIDCOM_NAV_SORTABLE]) |
|
7 | + if (isset($item[MIDCOM_NAV_SORTABLE]) |
|
8 | 8 | && !$item[MIDCOM_NAV_SORTABLE]) { |
9 | 9 | continue; |
10 | 10 | } |
@@ -19,7 +19,7 @@ discard block |
||
19 | 19 | } |
20 | 20 | |
21 | 21 | // Skip all components that return the default icon |
22 | - if ( isset($item[MIDCOM_NAV_COMPONENT]) |
|
22 | + if (isset($item[MIDCOM_NAV_COMPONENT]) |
|
23 | 23 | && ($tmp = midcom::get()->componentloader->get_component_icon($item[MIDCOM_NAV_COMPONENT], false))) { |
24 | 24 | $icon = '<i class="fa fa-' . $tmp . '"></i>'; |
25 | 25 | } elseif (!$item[MIDCOM_NAV_GUID]) { |
@@ -17,7 +17,7 @@ |
||
17 | 17 | } |
18 | 18 | |
19 | 19 | if (!$data['diff']) { |
20 | - echo "<dt>". $data['l10n']->get('no changes in content') ."</dt>\n"; |
|
20 | + echo "<dt>" . $data['l10n']->get('no changes in content') . "</dt>\n"; |
|
21 | 21 | } |
22 | 22 | ?> |
23 | 23 | </dl> |
24 | 24 | \ No newline at end of file |
@@ -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 |
@@ -34,7 +34,7 @@ |
||
34 | 34 | <div class="value"><?php echo $formatter->date($invoice->deliverydate); ?></div></div> |
35 | 35 | <?php } ?> |
36 | 36 | |
37 | - <div class="field"><div class="title"><?php echo $data['l10n_midcom']->get('description');?>: </div> |
|
37 | + <div class="field"><div class="title"><?php echo $data['l10n_midcom']->get('description'); ?>: </div> |
|
38 | 38 | <div class="description value">&(view['description']:h);</div></div> |
39 | 39 | |
40 | 40 | <?php |
@@ -46,7 +46,7 @@ |
||
46 | 46 | <?php |
47 | 47 | //index is needed for sorting |
48 | 48 | echo "'index_name',"; |
49 | - echo "'" . $data['l10n']->get('entry title') ."',"; |
|
49 | + echo "'" . $data['l10n']->get('entry title') . "',"; |
|
50 | 50 | echo "'" . $data['l10n']->get('entry text') . "',"; |
51 | 51 | echo "'" . $data['l10n']->get('entry created') . "',"; |
52 | 52 | echo "'index_creator', '" . $data['l10n_midcom']->get('creator') . "',"; |
@@ -85,7 +85,7 @@ discard block |
||
85 | 85 | { |
86 | 86 | ksort($constraints); |
87 | 87 | foreach ($constraints as $key => $data) { |
88 | - if ( !array_key_exists('value', $data) |
|
88 | + if (!array_key_exists('value', $data) |
|
89 | 89 | || empty($data['field']) |
90 | 90 | || empty($data['op'])) { |
91 | 91 | debug_add("Constraint #{$key} is not correctly defined, skipping", MIDCOM_LOG_WARN); |
@@ -146,7 +146,7 @@ discard block |
||
146 | 146 | { |
147 | 147 | $query = $this->request["term"]; |
148 | 148 | $wildcard_query = $query; |
149 | - if ( isset($this->request['auto_wildcards']) |
|
149 | + if (isset($this->request['auto_wildcards']) |
|
150 | 150 | && !str_contains($query, '%')) { |
151 | 151 | switch ($this->request['auto_wildcards']) { |
152 | 152 | case 'start': |
@@ -253,9 +253,7 @@ discard block |
||
253 | 253 | return $label; |
254 | 254 | } |
255 | 255 | } |
256 | - return midcom_helper_reflector::get($object)->get_object_label($object) ?: |
|
257 | - self::build_label($object, array_column($result_headers, 'name')) ?: |
|
258 | - get_class($object) . ' #' . $object->id; |
|
256 | + return midcom_helper_reflector::get($object)->get_object_label($object) ?: self::build_label($object, array_column($result_headers, 'name')) ?: get_class($object) . ' #' . $object->id; |
|
259 | 257 | } |
260 | 258 | |
261 | 259 | private static function build_label($object, array $fields) : string |
@@ -289,7 +287,7 @@ discard block |
||
289 | 287 | } |
290 | 288 | return self::sanitize_label($value); |
291 | 289 | } |
292 | - if ( $field == 'username' |
|
290 | + if ($field == 'username' |
|
293 | 291 | && $object instanceof midcom_db_person) { |
294 | 292 | $account = new midcom_core_account($object); |
295 | 293 | return self::sanitize_label($account->get_username()); |