@@ -19,8 +19,8 @@ |
||
| 19 | 19 | public function _on_initialize() |
| 20 | 20 | { |
| 21 | 21 | // Match / |
| 22 | - if ( $this->_config->get('redirection_type') === null |
|
| 23 | - || ( $this->_topic->can_do('net.nemein.redirector:noredirect') |
|
| 22 | + if ($this->_config->get('redirection_type') === null |
|
| 23 | + || ($this->_topic->can_do('net.nemein.redirector:noredirect') |
|
| 24 | 24 | && !$this->_config->get('admin_redirection'))) { |
| 25 | 25 | $this->_request_switch['redirect'] = [ |
| 26 | 26 | 'handler' => [net_nemein_redirector_handler_tinyurl::class, 'list'], |
@@ -158,7 +158,7 @@ discard block |
||
| 158 | 158 | $localpath = $path; |
| 159 | 159 | $matches['latest_parent'] = $folder_tree['/']; |
| 160 | 160 | $missing_levels = 0; |
| 161 | - while ( $localpath |
|
| 161 | + while ($localpath |
|
| 162 | 162 | && $localpath != '/') { |
| 163 | 163 | $localpath = dirname($localpath); |
| 164 | 164 | $missing_levels++; |
@@ -185,7 +185,7 @@ discard block |
||
| 185 | 185 | |
| 186 | 186 | $root_folder = $folder; |
| 187 | 187 | $max = 100; |
| 188 | - while ( $folder[MIDCOM_NAV_COMPONENT] == 'net.nemein.wiki' |
|
| 188 | + while ($folder[MIDCOM_NAV_COMPONENT] == 'net.nemein.wiki' |
|
| 189 | 189 | && (($parent = $nap->get_node_uplink($folder[MIDCOM_NAV_ID])) != -1) |
| 190 | 190 | && $max > 0) { |
| 191 | 191 | $root_folder = $folder; |
@@ -125,7 +125,7 @@ discard block |
||
| 125 | 125 | } |
| 126 | 126 | // TODO: Shared tasks need a separate check (different member object) |
| 127 | 127 | |
| 128 | - if ( !empty($this->busy_members) |
|
| 128 | + if (!empty($this->busy_members) |
|
| 129 | 129 | || !empty($this->busy_resources)) { |
| 130 | 130 | //Unresolved conflicts (note return value is for conflicts not lack of them) |
| 131 | 131 | midcom::get()->auth->drop_sudo(); |
@@ -188,7 +188,7 @@ discard block |
||
| 188 | 188 | } |
| 189 | 189 | |
| 190 | 190 | if (array_key_exists($member->event, $modified_events)) { |
| 191 | - $event =& $modified_events[$member->event]; |
|
| 191 | + $event = & $modified_events[$member->event]; |
|
| 192 | 192 | $set_as_modified = false; |
| 193 | 193 | } else { |
| 194 | 194 | try { |
@@ -202,7 +202,7 @@ discard block |
||
| 202 | 202 | } |
| 203 | 203 | debug_add("overlap found in event {$event->title} (#{$event->id})"); |
| 204 | 204 | |
| 205 | - if ( $event->tentative |
|
| 205 | + if ($event->tentative |
|
| 206 | 206 | && $rob_tentative) { |
| 207 | 207 | debug_add('event is tentative, robbing resources'); |
| 208 | 208 | $event->resources = array_diff_key($event->resources, $this->_event->resources); |
@@ -229,7 +229,7 @@ discard block |
||
| 229 | 229 | } |
| 230 | 230 | debug_add("overlap found in event {$event->title} (#{$event->id})"); |
| 231 | 231 | |
| 232 | - if ( $event->tentative |
|
| 232 | + if ($event->tentative |
|
| 233 | 233 | && $rob_tentative) { |
| 234 | 234 | debug_add('event is tentative, robbing participants'); |
| 235 | 235 | $event->participants = array_diff_key($event->participants, $this->_event->participants); |
@@ -102,7 +102,7 @@ |
||
| 102 | 102 | |
| 103 | 103 | // If user has preference for this message, we use that |
| 104 | 104 | $personal_preferences = $recipient->list_parameters('org.openpsa.notifications'); |
| 105 | - if ( !empty($personal_preferences) |
|
| 105 | + if (!empty($personal_preferences) |
|
| 106 | 106 | && array_key_exists("{$component}:{$action}", $personal_preferences)) { |
| 107 | 107 | return $personal_preferences[$action]; |
| 108 | 108 | } |
@@ -249,7 +249,7 @@ |
||
| 249 | 249 | $new_status = $this->_find_status($this->_status_map[$this->status_type], $status_types); |
| 250 | 250 | } |
| 251 | 251 | |
| 252 | - if ( $new_status !== null |
|
| 252 | + if ($new_status !== null |
|
| 253 | 253 | && $this->status != $new_status) { |
| 254 | 254 | $this->status = $new_status; |
| 255 | 255 | $update_required = true; |
@@ -127,7 +127,7 @@ discard block |
||
| 127 | 127 | foreach ($this->_datamanagers as $type => $datamanager) { |
| 128 | 128 | foreach ($datamanager->get_schema($this->schemas[$type])->get('fields') as $name => $config) { |
| 129 | 129 | $title = $config['title']; |
| 130 | - if ( $this->include_totals |
|
| 130 | + if ($this->include_totals |
|
| 131 | 131 | && $config['type'] == 'number') { |
| 132 | 132 | $this->_totals[$type . '-' . $name] = 0; |
| 133 | 133 | } |
@@ -194,18 +194,18 @@ discard block |
||
| 194 | 194 | $data = array_pad($data, $target_size, ''); |
| 195 | 195 | continue; |
| 196 | 196 | } |
| 197 | - $object =& $row[$type]; |
|
| 197 | + $object = & $row[$type]; |
|
| 198 | 198 | |
| 199 | 199 | $datamanager->set_storage($object, $this->schemas[$type]); |
| 200 | 200 | |
| 201 | - if ( $this->include_guid |
|
| 201 | + if ($this->include_guid |
|
| 202 | 202 | && $type == $first_type) { |
| 203 | 203 | $data[] = $object->guid; |
| 204 | 204 | } |
| 205 | 205 | |
| 206 | 206 | $csvdata = $datamanager->get_content_csv(); |
| 207 | 207 | foreach ($datamanager->get_schema()->get('fields') as $fieldname => $config) { |
| 208 | - if ( $this->include_totals |
|
| 208 | + if ($this->include_totals |
|
| 209 | 209 | && $config['type'] == 'number') { |
| 210 | 210 | $this->_totals[$type . '-' . $fieldname] += $csvdata[$fieldname]; |
| 211 | 211 | } |
@@ -233,7 +233,7 @@ discard block |
||
| 233 | 233 | if (empty($this->csv['charset'])) { |
| 234 | 234 | // Default to ISO-LATIN-15 (Latin-1 with EURO sign etc) |
| 235 | 235 | $this->csv['charset'] = 'ISO-8859-15'; |
| 236 | - if ( isset($_SERVER['HTTP_USER_AGENT']) |
|
| 236 | + if (isset($_SERVER['HTTP_USER_AGENT']) |
|
| 237 | 237 | && !preg_match('/Windows/i', $_SERVER['HTTP_USER_AGENT'])) { |
| 238 | 238 | // Except when not on windows, then default to UTF-8 |
| 239 | 239 | $this->csv['charset'] = 'UTF-8'; |
@@ -100,7 +100,7 @@ |
||
| 100 | 100 | foreach ($arguments as $i => $argument) { |
| 101 | 101 | if ($argument === '__request_data__') { |
| 102 | 102 | $context = $event->getRequest()->attributes->get('context'); |
| 103 | - $arguments[$i] =& $context->get_custom_key('request_data'); |
|
| 103 | + $arguments[$i] = & $context->get_custom_key('request_data'); |
|
| 104 | 104 | } |
| 105 | 105 | } |
| 106 | 106 | $event->setArguments($arguments); |
@@ -261,7 +261,7 @@ |
||
| 261 | 261 | if ($part === '') { |
| 262 | 262 | continue; |
| 263 | 263 | } |
| 264 | - if ( midcom::get()->config->get('theme') |
|
| 264 | + if (midcom::get()->config->get('theme') |
|
| 265 | 265 | && !$args_started |
| 266 | 266 | && self::check_page_exists($part)) { |
| 267 | 267 | $page_style .= '/' . $part; |
@@ -162,7 +162,7 @@ |
||
| 162 | 162 | if (!$this->_message_stack) { |
| 163 | 163 | return; |
| 164 | 164 | } |
| 165 | - if ( $show_simple |
|
| 165 | + if ($show_simple |
|
| 166 | 166 | || !midcom::get()->auth->can_user_do('midcom:ajax', null, static::class)) { |
| 167 | 167 | $this->show_simple(); |
| 168 | 168 | return; |