@@ -99,7 +99,7 @@ |
||
| 99 | 99 | $qb = midcom::get()->dbfactory->new_query_builder($this->config['type_config']['mapping_class_name']); |
| 100 | 100 | $qb->add_constraint($this->config['type_config']['master_fieldname'], '=', $this->get_master_foreign_key()); |
| 101 | 101 | |
| 102 | - if ( $this->config['type_config']['sortable'] |
|
| 102 | + if ($this->config['type_config']['sortable'] |
|
| 103 | 103 | && preg_match('/^(ASC|DESC)/i', $this->config['type_config']['sortable_sort_order'], $regs)) { |
| 104 | 104 | $order = strtoupper($regs[1]); |
| 105 | 105 | $qb->add_order('metadata.score', $order); |
@@ -336,7 +336,7 @@ |
||
| 336 | 336 | public function update_cache() |
| 337 | 337 | { |
| 338 | 338 | // Check if the attachment can be read anonymously |
| 339 | - if ( midcom::get()->config->get('attachment_cache_enabled') |
|
| 339 | + if (midcom::get()->config->get('attachment_cache_enabled') |
|
| 340 | 340 | && !$this->can_do('midgard:read', 'EVERYONE')) { |
| 341 | 341 | // Not public file, ensure it is removed |
| 342 | 342 | $filename = $this->get_cache_path(); |
@@ -68,7 +68,7 @@ |
||
| 68 | 68 | } |
| 69 | 69 | |
| 70 | 70 | $config = $this->get_config_for_topic($topic); |
| 71 | - if ( $object->name == 'index' |
|
| 71 | + if ($object->name == 'index' |
|
| 72 | 72 | && !$config->get('autoindex')) { |
| 73 | 73 | return ''; |
| 74 | 74 | } |
@@ -42,7 +42,7 @@ discard block |
||
| 42 | 42 | |
| 43 | 43 | public function _on_creating() |
| 44 | 44 | { |
| 45 | - return ( $this->validate_tag($this->tag) |
|
| 45 | + return ($this->validate_tag($this->tag) |
|
| 46 | 46 | && $this->_check_duplicates() == 0); |
| 47 | 47 | } |
| 48 | 48 | |
@@ -66,7 +66,7 @@ discard block |
||
| 66 | 66 | midcom::get()->uimessages->add(midcom::get()->i18n->get_string('net.nemein.tag', 'net.nemein.tag'), sprintf(midcom::get()->i18n->get_string('tag "%s" is not valid. tags may not be numeric', 'net.nemein.tag'), $tag), 'info'); |
| 67 | 67 | return false; |
| 68 | 68 | } |
| 69 | - if ( strstr($tag, '"') |
|
| 69 | + if (strstr($tag, '"') |
|
| 70 | 70 | || strstr($tag, "'")) { |
| 71 | 71 | midcom::get()->uimessages->add(midcom::get()->i18n->get_string('net.nemein.tag', 'net.nemein.tag'), sprintf(midcom::get()->i18n->get_string('tag "%s" is not valid. tags may not contain quotes', 'net.nemein.tag'), $tag), 'info'); |
| 72 | 72 | return false; |
@@ -77,7 +77,7 @@ discard block |
||
| 77 | 77 | |
| 78 | 78 | public function _on_updating() |
| 79 | 79 | { |
| 80 | - return ( $this->validate_tag($this->tag) |
|
| 80 | + return ($this->validate_tag($this->tag) |
|
| 81 | 81 | && $this->_check_duplicates() == 0); |
| 82 | 82 | } |
| 83 | 83 | |
@@ -4,7 +4,7 @@ |
||
| 4 | 4 | <h1>Merge tags</h1> |
| 5 | 5 | |
| 6 | 6 | <?php |
| 7 | -if ( isset($_POST['from']) |
|
| 7 | +if (isset($_POST['from']) |
|
| 8 | 8 | && isset($_POST['to'])) { |
| 9 | 9 | if (net_nemein_tag_handler::merge_tags($_POST['from'], $_POST['to'])) { |
| 10 | 10 | echo "<p>Successfully merged tag \"{$_POST['from']}\" to \"{$_POST['to']}\"</p>\n"; |
@@ -50,7 +50,7 @@ discard block |
||
| 50 | 50 | return; |
| 51 | 51 | } |
| 52 | 52 | |
| 53 | - if ( $history['date'] < $from |
|
| 53 | + if ($history['date'] < $from |
|
| 54 | 54 | || $history['date'] > $to) { |
| 55 | 55 | // We can ignore revisions outside the timeframe |
| 56 | 56 | continue; |
@@ -93,7 +93,7 @@ discard block |
||
| 93 | 93 | $this->_seek_updated($from); |
| 94 | 94 | |
| 95 | 95 | $i = 0; |
| 96 | - while ( $this->_updated_pages < $this->_max_pages |
|
| 96 | + while ($this->_updated_pages < $this->_max_pages |
|
| 97 | 97 | && $i < 20) { |
| 98 | 98 | // Expand seek by another two weeks |
| 99 | 99 | $to = $from; |
@@ -29,7 +29,7 @@ |
||
| 29 | 29 | |
| 30 | 30 | public function _on_creating() |
| 31 | 31 | { |
| 32 | - if ( $this->title == '' |
|
| 32 | + if ($this->title == '' |
|
| 33 | 33 | || !$this->topic) { |
| 34 | 34 | // We must have wikiword and topic at this stage |
| 35 | 35 | return false; |
@@ -61,7 +61,7 @@ |
||
| 61 | 61 | public function get_link($key = 0, $rel = 'alternate') |
| 62 | 62 | { |
| 63 | 63 | $link = parent::get_link($key, $rel); |
| 64 | - if ( $rel !== 'alternate' |
|
| 64 | + if ($rel !== 'alternate' |
|
| 65 | 65 | || $key !== 0) { |
| 66 | 66 | return $link; |
| 67 | 67 | } |
@@ -109,7 +109,7 @@ |
||
| 109 | 109 | */ |
| 110 | 110 | public function members_to_fields() |
| 111 | 111 | { |
| 112 | - if ( empty($this->topic_guid) |
|
| 112 | + if (empty($this->topic_guid) |
|
| 113 | 113 | || empty($this->topic_url) |
| 114 | 114 | || empty($this->component)) { |
| 115 | 115 | //if one of those is missing, we override all three to ensure consistency |