@@ -277,7 +277,7 @@ discard block |
||
277 | 277 | { |
278 | 278 | $success = true; |
279 | 279 | foreach ($this->_request_data['config']->_global as $global_key => $global_value) { |
280 | - if ( isset($config[$global_key]) |
|
280 | + if (isset($config[$global_key]) |
|
281 | 281 | && $config[$global_key] != $global_value) { |
282 | 282 | continue; |
283 | 283 | // Skip the ones we will set next |
@@ -290,7 +290,7 @@ discard block |
||
290 | 290 | } |
291 | 291 | |
292 | 292 | foreach ($config as $key => $value) { |
293 | - if ( is_array($value) |
|
293 | + if (is_array($value) |
|
294 | 294 | || is_object($value)) { |
295 | 295 | /** |
296 | 296 | * See http://trac.midgard-project.org/ticket/1442 |
@@ -61,11 +61,11 @@ |
||
61 | 61 | $fields[$key] = $this->detect_schema($key, $value); |
62 | 62 | } |
63 | 63 | |
64 | - if ( !isset($this->config->_local[$key]) |
|
64 | + if (!isset($this->config->_local[$key]) |
|
65 | 65 | || $this->config->_local[$key] == $this->config->_global[$key]) { |
66 | 66 | // No local configuration setting, note to user that this is the global value |
67 | 67 | $fields[$key]['title'] = $schema->get_l10n()->get($fields[$key]['title']); |
68 | - $fields[$key]['title'] .= " <span class=\"global\">(" . midcom::get()->i18n->get_string('global value', 'midgard.admin.asgard') .")</span>"; |
|
68 | + $fields[$key]['title'] .= " <span class=\"global\">(" . midcom::get()->i18n->get_string('global value', 'midgard.admin.asgard') . ")</span>"; |
|
69 | 69 | } |
70 | 70 | } |
71 | 71 |
@@ -179,9 +179,9 @@ discard block |
||
179 | 179 | $files = []; |
180 | 180 | $pattern = $component_dir . '*.{' . $this->_i18n->get_current_language() . ',' . $this->_i18n->get_fallback_language() . '}.txt'; |
181 | 181 | |
182 | - foreach (glob($pattern, GLOB_NOSORT|GLOB_BRACE) as $path) { |
|
182 | + foreach (glob($pattern, GLOB_NOSORT | GLOB_BRACE) as $path) { |
|
183 | 183 | $entry = basename($path); |
184 | - if ( str_starts_with($entry, 'index') |
|
184 | + if (str_starts_with($entry, 'index') |
|
185 | 185 | || str_starts_with($entry, 'handler') |
186 | 186 | || str_starts_with($entry, 'urlmethod')) { |
187 | 187 | // Ignore dotfiles, handlers & index.lang.txt |
@@ -232,7 +232,7 @@ discard block |
||
232 | 232 | $data = []; |
233 | 233 | |
234 | 234 | $exec_path = midcom::get()->componentloader->path_to_snippetpath($component) . '/exec/'; |
235 | - if ( !is_dir($exec_path) |
|
235 | + if (!is_dir($exec_path) |
|
236 | 236 | || !is_readable($exec_path)) { |
237 | 237 | // Directory not accessible, skip loading it |
238 | 238 | return $data; |
@@ -141,7 +141,7 @@ discard block |
||
141 | 141 | } |
142 | 142 | |
143 | 143 | //Check up |
144 | - if ( !$this->up |
|
144 | + if (!$this->up |
|
145 | 145 | && $this->title != '__org_openpsa_calendar') { |
146 | 146 | $root_event = org_openpsa_calendar_interface::find_root_event(); |
147 | 147 | $this->up = $root_event->id; |
@@ -166,7 +166,7 @@ discard block |
||
166 | 166 | |
167 | 167 | private function _check_timerange() : bool |
168 | 168 | { |
169 | - if ( !$this->start |
|
169 | + if (!$this->start |
|
170 | 170 | || !$this->end) { |
171 | 171 | debug_add('Event must have start and end timestamps'); |
172 | 172 | midcom_connection::set_error(MGD_ERR_RANGE); |
@@ -49,7 +49,7 @@ discard block |
||
49 | 49 | throw new midcom_error_midgard($e, $guid); |
50 | 50 | } |
51 | 51 | $person_class = midcom::get()->config->get('person_class'); |
52 | - if ( get_class($tmp) == 'midgard_person' |
|
52 | + if (get_class($tmp) == 'midgard_person' |
|
53 | 53 | && $person_class != 'midgard_person') { |
54 | 54 | $tmp = new $person_class($guid); |
55 | 55 | } |
@@ -79,7 +79,7 @@ discard block |
||
79 | 79 | } |
80 | 80 | $object = new $classname($src); |
81 | 81 | $cache[$classname][$object->guid] = $object; |
82 | - $cache[$classname][$object->id] =& $cache[$classname][$object->guid]; |
|
82 | + $cache[$classname][$object->id] = & $cache[$classname][$object->guid]; |
|
83 | 83 | return $cache[$classname][$object->guid]; |
84 | 84 | } |
85 | 85 | |
@@ -127,7 +127,7 @@ discard block |
||
127 | 127 | return true; |
128 | 128 | } |
129 | 129 | |
130 | - if ( isset($object->__object) |
|
130 | + if (isset($object->__object) |
|
131 | 131 | && is_object($object->__object) |
132 | 132 | && $object->__object instanceof $class) { |
133 | 133 | // Decorator whose MgdSchema object matches |
@@ -150,7 +150,7 @@ discard block |
||
150 | 150 | return $this->get_parent_data_uncached($object); |
151 | 151 | }); |
152 | 152 | |
153 | - if ( empty($parent_guid) |
|
153 | + if (empty($parent_guid) |
|
154 | 154 | || $parent_guid === $object->guid) { |
155 | 155 | return null; |
156 | 156 | } |
@@ -91,7 +91,7 @@ |
||
91 | 91 | midcom::get()->head->set_pagetitle($this->_request_data['page_title']); |
92 | 92 | |
93 | 93 | // Activate correct leaf |
94 | - if ( $this->_config->get('show_navigation_pseudo_leaves') |
|
94 | + if ($this->_config->get('show_navigation_pseudo_leaves') |
|
95 | 95 | && in_array($this->category, $this->_request_data['categories'])) { |
96 | 96 | $this->set_active_leaf($this->_topic->id . '_CAT_' . $this->category); |
97 | 97 | } |
@@ -131,8 +131,8 @@ discard block |
||
131 | 131 | $year_data = []; |
132 | 132 | $first_post = $this->_compute_welcome_first_post(); |
133 | 133 | $this->_request_data['first_post'] = $first_post; |
134 | - $this->_request_data['total_count'] =& $total_count; |
|
135 | - $this->_request_data['year_data'] =& $year_data; |
|
134 | + $this->_request_data['total_count'] = & $total_count; |
|
135 | + $this->_request_data['year_data'] = & $year_data; |
|
136 | 136 | if (!$first_post) { |
137 | 137 | return; |
138 | 138 | } |
@@ -263,7 +263,7 @@ discard block |
||
263 | 263 | switch ($handler_id) { |
264 | 264 | case 'archive-year-category': |
265 | 265 | $category = trim(strip_tags($args[1])); |
266 | - if ( $data['datamanager']->get_schema('default')->has_field('categories') |
|
266 | + if ($data['datamanager']->get_schema('default')->has_field('categories') |
|
267 | 267 | && !$data['datamanager']->get_schema('default')->get_field('categories')['type_config']['allow_multiple']) { |
268 | 268 | $qb->add_constraint('extra1', '=', (string) $category); |
269 | 269 | } else { |
@@ -352,7 +352,7 @@ discard block |
||
352 | 352 | throw new midcom_error_notfound("The year '{$year}' is not a valid year identifier."); |
353 | 353 | } |
354 | 354 | |
355 | - if ( $month < 1 |
|
355 | + if ($month < 1 |
|
356 | 356 | || $month > 12) { |
357 | 357 | throw new midcom_error_notfound("The year {$month} is not a valid year identifier."); |
358 | 358 | } |
@@ -12,7 +12,7 @@ discard block |
||
12 | 12 | $published = sprintf($data['l10n']->get('posted on %s.'), "<abbr title=\"" . strftime('%Y-%m-%dT%H:%M:%S%z', $data['article']->metadata->published) . "\">" . $published . "</abbr>"); |
13 | 13 | |
14 | 14 | if (array_key_exists('base_ajax_comments_url', $data)) { |
15 | - $published .= ' <a href="#switch_comments" onClick="showAjaxComments(this, \''.$data['article']->guid.'\'); return false;">' |
|
15 | + $published .= ' <a href="#switch_comments" onClick="showAjaxComments(this, \'' . $data['article']->guid . '\'); return false;">' |
|
16 | 16 | . sprintf($data['l10n']->get('%s comments'), net_nehmer_comments_comment::count_by_objectguid($data['article']->guid)) |
17 | 17 | . "</a>."; |
18 | 18 | } elseif ($data['config']->get('comments_enable')) { |
@@ -28,7 +28,7 @@ discard block |
||
28 | 28 | &(published:h); |
29 | 29 | <?php |
30 | 30 | if ($data['linked']) { |
31 | - echo $data['l10n']->get('to') ." <a href=\"{$data['node'][MIDCOM_NAV_ABSOLUTEURL]}\">{$data['node'][MIDCOM_NAV_NAME]}</a>\n"; |
|
31 | + echo $data['l10n']->get('to') . " <a href=\"{$data['node'][MIDCOM_NAV_ABSOLUTEURL]}\">{$data['node'][MIDCOM_NAV_NAME]}</a>\n"; |
|
32 | 32 | } |
33 | 33 | ?> |
34 | 34 | </p> |
@@ -35,7 +35,7 @@ discard block |
||
35 | 35 | { |
36 | 36 | $this->_object = midcom::get()->dbfactory->get_object_by_guid($guid); |
37 | 37 | |
38 | - if ( !is_a($this->_object, midcom_db_topic::class) |
|
38 | + if (!is_a($this->_object, midcom_db_topic::class) |
|
39 | 39 | && !is_a($this->_object, midcom_db_article::class)) { |
40 | 40 | throw new midcom_error_notfound("Moving only topics and articles is supported."); |
41 | 41 | } |
@@ -76,7 +76,7 @@ discard block |
||
76 | 76 | $folder = midcom_core_context::get()->get_key(MIDCOM_CONTEXT_ROOTTOPIC); |
77 | 77 | } |
78 | 78 | |
79 | - if ( is_a($this->_object, midcom_db_topic::class) |
|
79 | + if (is_a($this->_object, midcom_db_topic::class) |
|
80 | 80 | && $folder->up == $this->_object->id) { |
81 | 81 | $tree_disabled = true; |
82 | 82 | } |
@@ -89,7 +89,7 @@ discard block |
||
89 | 89 | $selected = ' checked="checked"'; |
90 | 90 | } |
91 | 91 | |
92 | - if ( !is_a($this->_object, midcom_db_topic::class) |
|
92 | + if (!is_a($this->_object, midcom_db_topic::class) |
|
93 | 93 | && $folder->component !== $this->current_folder->component) { |
94 | 94 | // Non-topic objects may only be moved under folders of same component |
95 | 95 | $class = 'wrong_component'; |