@@ -9,11 +9,11 @@ |
||
| 9 | 9 | use midcom\datamanager\indexer\client; |
| 10 | 10 | use midcom\datamanager\datamanager; |
| 11 | 11 | |
| 12 | - /** |
|
| 13 | - * Indexer client class |
|
| 14 | - * |
|
| 15 | - * @package org.openpsa.invoices |
|
| 16 | - */ |
|
| 12 | + /** |
|
| 13 | + * Indexer client class |
|
| 14 | + * |
|
| 15 | + * @package org.openpsa.invoices |
|
| 16 | + */ |
|
| 17 | 17 | class org_openpsa_invoices_midcom_indexer extends client |
| 18 | 18 | { |
| 19 | 19 | public function prepare_document(midcom_services_indexer_document &$document, datamanager $dm) |
@@ -30,7 +30,8 @@ |
||
| 30 | 30 | $this->config = $config; |
| 31 | 31 | } |
| 32 | 32 | |
| 33 | - protected function cast($value) { |
|
| 33 | + protected function cast($value) |
|
| 34 | + { |
|
| 34 | 35 | if ($this->config['type'] == 'number' && !is_numeric($value)) { |
| 35 | 36 | $value = (float) $value; |
| 36 | 37 | } |
@@ -30,7 +30,8 @@ |
||
| 30 | 30 | foreach ($result as $wikipage) { |
| 31 | 31 | try { |
| 32 | 32 | $datamanager->set_storage($wikipage); |
| 33 | - } catch (midcom_error $e) { |
|
| 33 | + } |
|
| 34 | + catch (midcom_error $e) { |
|
| 34 | 35 | $e->log(MIDCOM_LOG_WARN); |
| 35 | 36 | continue; |
| 36 | 37 | } |
@@ -44,7 +44,7 @@ |
||
| 44 | 44 | |
| 45 | 45 | public function resolve_object_link(midcom_db_topic $topic, midcom_core_dbaobject $object) : ?string |
| 46 | 46 | { |
| 47 | - if ( $object instanceof midcom_db_article |
|
| 47 | + if ($object instanceof midcom_db_article |
|
| 48 | 48 | && $topic->id == $object->topic) { |
| 49 | 49 | if ($object->name == 'index') { |
| 50 | 50 | return ''; |
@@ -22,7 +22,7 @@ |
||
| 22 | 22 | |
| 23 | 23 | // Get the calendar root event |
| 24 | 24 | if ($root_event = org_openpsa_calendar_interface::find_root_event()) { |
| 25 | - $field =& $schemadb->get('default')->get_field('calendar'); |
|
| 25 | + $field = & $schemadb->get('default')->get_field('calendar'); |
|
| 26 | 26 | $field['type_config']['privilege_object'] = $root_event; |
| 27 | 27 | $field['type_config']['assignee'] = 'group:' . $group->guid; |
| 28 | 28 | } |
@@ -22,7 +22,7 @@ |
||
| 22 | 22 | |
| 23 | 23 | // Get the calendar root event |
| 24 | 24 | if ($root_event = org_openpsa_calendar_interface::find_root_event()) { |
| 25 | - $field =& $schemadb->get('default')->get_field('calendar'); |
|
| 25 | + $field = & $schemadb->get('default')->get_field('calendar'); |
|
| 26 | 26 | $field['type_config']['privilege_object'] = $root_event; |
| 27 | 27 | $field['type_config']['assignee'] = 'user:' . $person->guid; |
| 28 | 28 | } |
@@ -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 | } |
@@ -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 | } |
@@ -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 { |
@@ -7,7 +7,8 @@ |
||
| 7 | 7 | try { |
| 8 | 8 | $person = new midcom_db_person($metadata->locker); |
| 9 | 9 | $name = $person->name; |
| 10 | -} catch (midcom_error $e) { |
|
| 10 | +} |
|
| 11 | +catch (midcom_error $e) { |
|
| 11 | 12 | $name = $l10n_midcom->get('unknown user'); |
| 12 | 13 | $e->log(); |
| 13 | 14 | } |
@@ -12,7 +12,8 @@ |
||
| 12 | 12 | foreach ($data['wikilinks'] as $link) { |
| 13 | 13 | try { |
| 14 | 14 | $page = new net_nemein_wiki_wikipage($link->frompage); |
| 15 | - } catch (midcom_error $e) { |
|
| 15 | + } |
|
| 16 | + catch (midcom_error $e) { |
|
| 16 | 17 | $e->log(); |
| 17 | 18 | continue; |
| 18 | 19 | } |