@@ -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; |
@@ -17,7 +17,7 @@ |
||
| 17 | 17 | public function _on_watched_dba_delete(midcom_core_dbaobject $object) |
| 18 | 18 | { |
| 19 | 19 | $qb = org_openpsa_directmarketing_campaign_member_dba::new_query_builder(); |
| 20 | - if ( $object instanceof midcom_db_person |
|
| 20 | + if ($object instanceof midcom_db_person |
|
| 21 | 21 | || $object instanceof org_openpsa_contacts_person_dba) { |
| 22 | 22 | $qb->add_constraint('person', '=', $object->id); |
| 23 | 23 | } elseif ($object instanceof org_openpsa_directmarketing_campaign_dba) { |
@@ -140,7 +140,7 @@ |
||
| 140 | 140 | return; |
| 141 | 141 | } |
| 142 | 142 | |
| 143 | - if ( !is_writable($config['filename']) |
|
| 143 | + if (!is_writable($config['filename']) |
|
| 144 | 144 | && !is_writable(dirname($config['filename']))) { |
| 145 | 145 | debug_add("Error logging file {$config['filename']} is not writable", MIDCOM_LOG_WARN); |
| 146 | 146 | return; |