@@ -409,7 +409,8 @@ |
||
| 409 | 409 | foreach ($this->_articles as $article_counter => $article) { |
| 410 | 410 | try { |
| 411 | 411 | $data['datamanager']->set_storage($article); |
| 412 | - } catch (midcom_error $e) { |
|
| 412 | + } |
|
| 413 | + catch (midcom_error $e) { |
|
| 413 | 414 | $e->log(); |
| 414 | 415 | continue; |
| 415 | 416 | } |
@@ -134,8 +134,8 @@ discard block |
||
| 134 | 134 | $year_data = []; |
| 135 | 135 | $first_post = $this->_compute_welcome_first_post(); |
| 136 | 136 | $this->_request_data['first_post'] = $first_post; |
| 137 | - $this->_request_data['total_count'] =& $total_count; |
|
| 138 | - $this->_request_data['year_data'] =& $year_data; |
|
| 137 | + $this->_request_data['total_count'] = & $total_count; |
|
| 138 | + $this->_request_data['year_data'] = & $year_data; |
|
| 139 | 139 | if (!$first_post) { |
| 140 | 140 | return; |
| 141 | 141 | } |
@@ -271,7 +271,7 @@ discard block |
||
| 271 | 271 | switch ($handler_id) { |
| 272 | 272 | case 'archive-year-category': |
| 273 | 273 | $data['category'] = trim(strip_tags($args[1])); |
| 274 | - if ( $data['datamanager']->get_schema('default')->has_field('categories') |
|
| 274 | + if ($data['datamanager']->get_schema('default')->has_field('categories') |
|
| 275 | 275 | && !$data['datamanager']->get_schema('default')->get_field('categories')['type_config']['allow_multiple']) { |
| 276 | 276 | $qb->add_constraint('extra1', '=', (string) $data['category']); |
| 277 | 277 | } else { |
@@ -360,7 +360,7 @@ discard block |
||
| 360 | 360 | throw new midcom_error_notfound("The year '{$year}' is not a valid year identifier."); |
| 361 | 361 | } |
| 362 | 362 | |
| 363 | - if ( $month < 1 |
|
| 363 | + if ($month < 1 |
|
| 364 | 364 | || $month > 12) { |
| 365 | 365 | throw new midcom_error_notfound("The year {$month} is not a valid year identifier."); |
| 366 | 366 | } |
@@ -17,7 +17,7 @@ |
||
| 17 | 17 | |
| 18 | 18 | public function form_widget_simple(FormView $view, array $data) |
| 19 | 19 | { |
| 20 | - if ( !empty($data['value']) |
|
| 20 | + if (!empty($data['value']) |
|
| 21 | 21 | || is_numeric($data['value'])) { |
| 22 | 22 | return $data['value']; |
| 23 | 23 | } |
@@ -44,7 +44,7 @@ |
||
| 44 | 44 | case 'save': |
| 45 | 45 | $this->_reset_password($data['controller']->get_form_values()); |
| 46 | 46 | $data['processing_msg'] = $this->_l10n->get('password reset, mail sent.'); |
| 47 | - $data['processing_msg_raw']= 'password reset, mail sent.'; |
|
| 47 | + $data['processing_msg_raw'] = 'password reset, mail sent.'; |
|
| 48 | 48 | $this->_success = true; |
| 49 | 49 | break; |
| 50 | 50 | |
@@ -30,7 +30,7 @@ |
||
| 30 | 30 | private function load_datamanager() |
| 31 | 31 | { |
| 32 | 32 | $dm = datamanager::from_schemadb($this->_config->get('schemadb_group')); |
| 33 | - $persons =& $dm->get_schema('default')->get_field('persons'); |
|
| 33 | + $persons = & $dm->get_schema('default')->get_field('persons'); |
|
| 34 | 34 | $persons['hidden'] = true; |
| 35 | 35 | return $dm->set_storage($this->_group); |
| 36 | 36 | } |
@@ -53,7 +53,7 @@ |
||
| 53 | 53 | |
| 54 | 54 | foreach ($schemadb->all() as $schema) { |
| 55 | 55 | if ($schema->has_field('name')) { |
| 56 | - $field =& $schema->get_field('name'); |
|
| 56 | + $field = & $schema->get_field('name'); |
|
| 57 | 57 | $field['required'] = ($this->_handler_id === 'edit'); |
| 58 | 58 | } |
| 59 | 59 | } |
@@ -46,7 +46,7 @@ discard block |
||
| 46 | 46 | ]); |
| 47 | 47 | } |
| 48 | 48 | |
| 49 | - if ( $this->_article->topic === $this->_topic->id |
|
| 49 | + if ($this->_article->topic === $this->_topic->id |
|
| 50 | 50 | && $this->_article->can_do('midgard:delete')) { |
| 51 | 51 | $delete = $this->get_workflow('delete', ['object' => $this->_article]); |
| 52 | 52 | $buttons[] = $delete->get_button("delete/{$this->_article->guid}/"); |
@@ -114,7 +114,7 @@ discard block |
||
| 114 | 114 | $this->_datamanager->set_storage($this->_article); |
| 115 | 115 | |
| 116 | 116 | $arg = $this->_article->name ?: $this->_article->guid; |
| 117 | - if ( $arg != 'index' |
|
| 117 | + if ($arg != 'index' |
|
| 118 | 118 | && $this->_config->get('hide_navigation')) { |
| 119 | 119 | $this->add_breadcrumb("{$arg}/", $this->_article->title); |
| 120 | 120 | } |
@@ -124,13 +124,13 @@ discard block |
||
| 124 | 124 | midcom::get()->metadata->set_request_metadata($this->_article->metadata->revised, $this->_article->guid); |
| 125 | 125 | $this->bind_view_to_object($this->_article, $this->_datamanager->get_schema()->get_name()); |
| 126 | 126 | |
| 127 | - if ( $this->_config->get('indexinnav') |
|
| 127 | + if ($this->_config->get('indexinnav') |
|
| 128 | 128 | || $this->_config->get('autoindex') |
| 129 | 129 | || $this->_article->name != 'index') { |
| 130 | 130 | $this->set_active_leaf($this->_article->id); |
| 131 | 131 | } |
| 132 | 132 | |
| 133 | - if ( $this->_config->get('folder_in_title') |
|
| 133 | + if ($this->_config->get('folder_in_title') |
|
| 134 | 134 | && $this->_topic->extra != $this->_article->title) { |
| 135 | 135 | midcom::get()->head->set_pagetitle("{$this->_topic->extra}: {$this->_article->title}"); |
| 136 | 136 | } else { |
@@ -28,10 +28,10 @@ |
||
| 28 | 28 | */ |
| 29 | 29 | private function load_controller() |
| 30 | 30 | { |
| 31 | - if ( $this->_config->get('simple_name_handling') |
|
| 31 | + if ($this->_config->get('simple_name_handling') |
|
| 32 | 32 | && !midcom::get()->auth->admin) { |
| 33 | 33 | foreach ($this->_request_data['schemadb']->all() as $schema) { |
| 34 | - $field =& $schema->get_field('name'); |
|
| 34 | + $field = & $schema->get_field('name'); |
|
| 35 | 35 | $field['readonly'] = true; |
| 36 | 36 | } |
| 37 | 37 | } |
@@ -32,7 +32,7 @@ |
||
| 32 | 32 | private function load_controller($schemaname, array $defaults) |
| 33 | 33 | { |
| 34 | 34 | if ($this->_config->get('simple_name_handling')) { |
| 35 | - $field =& $this->_request_data['schemadb']->get($schemaname)->get_field('name'); |
|
| 35 | + $field = & $this->_request_data['schemadb']->get($schemaname)->get_field('name'); |
|
| 36 | 36 | $field['hidden'] = true; |
| 37 | 37 | } |
| 38 | 38 | $dm = new datamanager($this->_request_data['schemadb']); |
@@ -53,7 +53,7 @@ |
||
| 53 | 53 | |
| 54 | 54 | public function _load_data($handler_id, array &$args, array &$data) |
| 55 | 55 | { |
| 56 | - if ( empty($_POST) |
|
| 56 | + if (empty($_POST) |
|
| 57 | 57 | && $data['session']->exists('POST_data')) { |
| 58 | 58 | $_POST = $data['session']->get('POST_data'); |
| 59 | 59 | $data['session']->remove('POST_data'); |