@@ -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 | } |
@@ -261,7 +261,7 @@ discard block |
||
| 261 | 261 | switch ($handler_id) { |
| 262 | 262 | case 'archive-year-category': |
| 263 | 263 | $category = trim(strip_tags($args[1])); |
| 264 | - if ( $data['datamanager']->get_schema('default')->has_field('categories') |
|
| 264 | + if ($data['datamanager']->get_schema('default')->has_field('categories') |
|
| 265 | 265 | && !$data['datamanager']->get_schema('default')->get_field('categories')['type_config']['allow_multiple']) { |
| 266 | 266 | $qb->add_constraint('extra1', '=', $category); |
| 267 | 267 | } else { |
@@ -345,7 +345,7 @@ discard block |
||
| 345 | 345 | throw new midcom_error_notfound("The year '{$year}' is not valid."); |
| 346 | 346 | } |
| 347 | 347 | |
| 348 | - if ( $month < 1 |
|
| 348 | + if ($month < 1 |
|
| 349 | 349 | || $month > 12) { |
| 350 | 350 | throw new midcom_error_notfound("The month {$month} is not valid."); |
| 351 | 351 | } |
@@ -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 | } |
@@ -22,7 +22,7 @@ |
||
| 22 | 22 | </td> |
| 23 | 23 | <td class="revisor"> |
| 24 | 24 | <?php |
| 25 | - if ( $history['user'] |
|
| 25 | + if ($history['user'] |
|
| 26 | 26 | && $user = midcom::get()->auth->get_user($history['user'])) { |
| 27 | 27 | $person_label = org_openpsa_widgets_contact::get($user->guid)->show_inline(); |
| 28 | 28 | echo " {$person_label}\n"; |
@@ -29,7 +29,7 @@ |
||
| 29 | 29 | $formatter = $data['l10n']->get_formatter(); |
| 30 | 30 | foreach ($data['items'] as $item) { |
| 31 | 31 | echo "<tr>\n"; |
| 32 | - $date = (int)$item->get_date('U'); |
|
| 32 | + $date = (int) $item->get_date('U'); |
|
| 33 | 33 | if ($date == 0) { |
| 34 | 34 | echo " <td>" . $data['l10n']->get('n/a') . "</td>\n"; |
| 35 | 35 | } else { |
@@ -16,8 +16,8 @@ |
||
| 16 | 16 | $view['author'], |
| 17 | 17 | $formatter->datetime($created)); |
| 18 | 18 | |
| 19 | -if ( midcom::get()->auth->admin |
|
| 20 | - || ( midcom::get()->auth->user |
|
| 19 | +if (midcom::get()->auth->admin |
|
| 20 | + || (midcom::get()->auth->user |
|
| 21 | 21 | && $comment->can_do('midgard:delete'))) { |
| 22 | 22 | $creator = $comment->metadata->creator; |
| 23 | 23 | $created = $comment->metadata->created; |