@@ -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); |
@@ -202,9 +202,9 @@ |
||
| 202 | 202 | $toc .= "\n<ol class=\"midcom_helper_toc_formatter level_{$current_list_level}\">\n"; |
| 203 | 203 | foreach ($headings[4] as $key => $heading) { |
| 204 | 204 | $anchor = 'heading-' . md5($heading); |
| 205 | - $tag_level =& $headings[3][$key]; |
|
| 206 | - $heading_code =& $headings[0][$key]; |
|
| 207 | - $heading_tag =& $headings[2][$key]; |
|
| 205 | + $tag_level = & $headings[3][$key]; |
|
| 206 | + $heading_code = & $headings[0][$key]; |
|
| 207 | + $heading_tag = & $headings[2][$key]; |
|
| 208 | 208 | $heading_new_code = "<a id='{$anchor}'></a>{$heading_code}"; |
| 209 | 209 | $content = str_replace($heading_code, $heading_new_code, $content); |
| 210 | 210 | if ($current_tag_level === false) { |
@@ -114,7 +114,7 @@ |
||
| 114 | 114 | { |
| 115 | 115 | $nap = new midcom_helper_nav(); |
| 116 | 116 | $node = $nap->get_node($nap->get_current_node()); |
| 117 | - $compose_url = $node[MIDCOM_NAV_RELATIVEURL] . 'message/compose/' . $data['message']->guid .'/'; |
|
| 117 | + $compose_url = $node[MIDCOM_NAV_RELATIVEURL] . 'message/compose/' . $data['message']->guid . '/'; |
|
| 118 | 118 | $this->batch_url_base_full = $node[MIDCOM_NAV_RELATIVEURL] . 'message/send_bg/' . $data['message']->guid . '/'; |
| 119 | 119 | debug_add("compose_url: {$compose_url}"); |
| 120 | 120 | debug_add("batch_url base: {$this->batch_url_base_full}"); |
@@ -223,10 +223,10 @@ |
||
| 223 | 223 | $new_type_reflector = midcom_helper_reflector::get($new_type); |
| 224 | 224 | $link_properties = $new_type_reflector->get_link_properties(); |
| 225 | 225 | foreach ($link_properties as $property => $link) { |
| 226 | - if ( ($link['class'] && midcom_helper_reflector::is_same_class($link['class'], $this->_object->__mgdschema_class_name__)) |
|
| 226 | + if (($link['class'] && midcom_helper_reflector::is_same_class($link['class'], $this->_object->__mgdschema_class_name__)) |
|
| 227 | 227 | || $link['type'] == MGD_TYPE_GUID) { |
| 228 | 228 | $defaults[$property] = $this->_object->{$link['target']}; |
| 229 | - } elseif ( $property == $parent_property |
|
| 229 | + } elseif ($property == $parent_property |
|
| 230 | 230 | && midcom_helper_reflector::is_same_class($new_type, $this->_object->__mgdschema_class_name__)) { |
| 231 | 231 | $defaults[$property] = $this->_object->$parent_property; |
| 232 | 232 | } |
@@ -59,7 +59,7 @@ |
||
| 59 | 59 | |
| 60 | 60 | public function __set($name, $value) |
| 61 | 61 | { |
| 62 | - if ( $name == 'homepage' |
|
| 62 | + if ($name == 'homepage' |
|
| 63 | 63 | && !empty($value) |
| 64 | 64 | && $value != $this->homepage) { |
| 65 | 65 | $this->_register_prober = true; |
@@ -22,7 +22,7 @@ discard block |
||
| 22 | 22 | |
| 23 | 23 | public function __construct(array &$data) |
| 24 | 24 | { |
| 25 | - $this->_data =& $data; |
|
| 25 | + $this->_data = & $data; |
|
| 26 | 26 | midcom::get()->head->enable_jquery_ui(['accordion']); |
| 27 | 27 | } |
| 28 | 28 | |
@@ -32,14 +32,14 @@ discard block |
||
| 32 | 32 | return; |
| 33 | 33 | } |
| 34 | 34 | |
| 35 | - if ( midcom::get()->dbfactory->is_a($this->_data['object'], 'midgard_style') |
|
| 36 | - && ( $this->_data['handler_id'] !== 'object_create' |
|
| 35 | + if (midcom::get()->dbfactory->is_a($this->_data['object'], 'midgard_style') |
|
| 36 | + && ($this->_data['handler_id'] !== 'object_create' |
|
| 37 | 37 | || $this->_data['current_type'] == 'midgard_element')) { |
| 38 | 38 | // Suggest element names to create under a style |
| 39 | 39 | $this->_data['help_style_elementnames'] = $this->_get_style_elements_and_nodes($this->_data['object']->id); |
| 40 | 40 | midcom_show_style('midgard_admin_asgard_stylehelper_elementnames'); |
| 41 | 41 | |
| 42 | - } elseif ( midcom::get()->dbfactory->is_a($this->_data['object'], 'midgard_element') |
|
| 42 | + } elseif (midcom::get()->dbfactory->is_a($this->_data['object'], 'midgard_element') |
|
| 43 | 43 | && $this->_get_help_element()) { |
| 44 | 44 | midcom_show_style('midgard_admin_asgard_stylehelper_element'); |
| 45 | 45 | } |
@@ -47,7 +47,7 @@ discard block |
||
| 47 | 47 | |
| 48 | 48 | private function _get_help_element() : bool |
| 49 | 49 | { |
| 50 | - if ( empty($this->_data['object']->name) |
|
| 50 | + if (empty($this->_data['object']->name) |
|
| 51 | 51 | || empty($this->_data['object']->style)) { |
| 52 | 52 | // We cannot help with empty elements |
| 53 | 53 | return false; |