@@ -25,7 +25,7 @@ |
||
| 25 | 25 | |
| 26 | 26 | public function _on_creating() : bool |
| 27 | 27 | { |
| 28 | - if ( !$this->referrer |
|
| 28 | + if (!$this->referrer |
|
| 29 | 29 | && !empty($_SERVER['HTTP_REFERER'])) { |
| 30 | 30 | $this->referrer = $_SERVER['HTTP_REFERER']; |
| 31 | 31 | } |
@@ -40,7 +40,7 @@ discard block |
||
| 40 | 40 | |
| 41 | 41 | public function _on_creating() : bool |
| 42 | 42 | { |
| 43 | - return ( $this->validate_tag($this->tag) |
|
| 43 | + return ($this->validate_tag($this->tag) |
|
| 44 | 44 | && $this->_check_duplicates() == 0); |
| 45 | 45 | } |
| 46 | 46 | |
@@ -69,7 +69,7 @@ discard block |
||
| 69 | 69 | |
| 70 | 70 | public function _on_updating() : bool |
| 71 | 71 | { |
| 72 | - return ( $this->validate_tag($this->tag) |
|
| 72 | + return ($this->validate_tag($this->tag) |
|
| 73 | 73 | && $this->_check_duplicates() == 0); |
| 74 | 74 | } |
| 75 | 75 | |
@@ -56,7 +56,7 @@ |
||
| 56 | 56 | $this->save_attachment_list(); |
| 57 | 57 | } |
| 58 | 58 | |
| 59 | - if ( array_intersect_key(array_flip(['description', 'title', 'score']), $this->value) |
|
| 59 | + if (array_intersect_key(array_flip(['description', 'title', 'score']), $this->value) |
|
| 60 | 60 | && $main = $this->get_main()) { |
| 61 | 61 | $needs_update = false; |
| 62 | 62 | if (array_key_exists('description', $this->value)) { |
@@ -163,8 +163,8 @@ discard block |
||
| 163 | 163 | $this->_uncached = $config->get('cache_module_content_uncached'); |
| 164 | 164 | $this->_headers_strategy = $this->get_strategy('cache_module_content_headers_strategy'); |
| 165 | 165 | $this->_headers_strategy_authenticated = $this->get_strategy('cache_module_content_headers_strategy_authenticated'); |
| 166 | - $this->_default_lifetime = (int)$config->get('cache_module_content_default_lifetime'); |
|
| 167 | - $this->_default_lifetime_authenticated = (int)$config->get('cache_module_content_default_lifetime_authenticated'); |
|
| 166 | + $this->_default_lifetime = (int) $config->get('cache_module_content_default_lifetime'); |
|
| 167 | + $this->_default_lifetime_authenticated = (int) $config->get('cache_module_content_default_lifetime_authenticated'); |
|
| 168 | 168 | |
| 169 | 169 | if ($this->_headers_strategy == 'no-cache') { |
| 170 | 170 | // we can't call no_cache() here, because it would try to call back to this class via the global getter |
@@ -531,7 +531,7 @@ discard block |
||
| 531 | 531 | */ |
| 532 | 532 | public function write_meta_cache(string $content_id, Request $request, Response $response) |
| 533 | 533 | { |
| 534 | - if ( $this->_uncached |
|
| 534 | + if ($this->_uncached |
|
| 535 | 535 | || $this->_no_cache) { |
| 536 | 536 | return; |
| 537 | 537 | } |
@@ -593,7 +593,7 @@ discard block |
||
| 593 | 593 | |
| 594 | 594 | public function store_dl_content(int $context, string $dl_cache_data, Request $request) |
| 595 | 595 | { |
| 596 | - if ( $this->_no_cache |
|
| 596 | + if ($this->_no_cache |
|
| 597 | 597 | || $this->_uncached) { |
| 598 | 598 | return; |
| 599 | 599 | } |
@@ -627,7 +627,7 @@ discard block |
||
| 627 | 627 | |
| 628 | 628 | /* TODO: Doublecheck the way this is handled, now we just don't send it |
| 629 | 629 | * if headers_strategy implies caching */ |
| 630 | - if ( !$response->headers->has('Content-Length') |
|
| 630 | + if (!$response->headers->has('Content-Length') |
|
| 631 | 631 | && !in_array($this->_headers_strategy, ['public', 'private'])) { |
| 632 | 632 | $response->headers->set("Content-Length", strlen($response->getContent())); |
| 633 | 633 | } |
@@ -142,7 +142,7 @@ |
||
| 142 | 142 | if ($counter <= $this->_offset) { |
| 143 | 143 | continue; |
| 144 | 144 | } |
| 145 | - if ( $this->_limit |
|
| 145 | + if ($this->_limit |
|
| 146 | 146 | && $counter > ($this->_offset + $this->_limit)) { |
| 147 | 147 | break; |
| 148 | 148 | } |
@@ -47,7 +47,7 @@ discard block |
||
| 47 | 47 | $reindex_topic_uri = midcom::get()->get_page_prefix() . 'midcom-exec-midcom/reindex_singlenode.php'; |
| 48 | 48 | |
| 49 | 49 | $options = ['timeout' => 300]; |
| 50 | -if ( !empty($_SERVER['PHP_AUTH_USER']) |
|
| 50 | +if (!empty($_SERVER['PHP_AUTH_USER']) |
|
| 51 | 51 | && !empty($_SERVER['PHP_AUTH_PW'])) { |
| 52 | 52 | $options['auth'] = ['user' => $_SERVER['PHP_AUTH_USER'], 'password' => $_SERVER['PHP_AUTH_PW']]; |
| 53 | 53 | } |
@@ -65,7 +65,7 @@ discard block |
||
| 65 | 65 | } else { |
| 66 | 66 | echo "OK.\n"; |
| 67 | 67 | } |
| 68 | - }, function (TransferException $e) use ($uri) { |
|
| 68 | + }, function(TransferException $e) use ($uri) { |
|
| 69 | 69 | echo "failure.\n Background processing failed, error: {$e->getMessage()}\n"; |
| 70 | 70 | echo "Url: " . $uri . "\n"; |
| 71 | 71 | })->wait(); |
@@ -79,7 +79,7 @@ |
||
| 79 | 79 | private static function _check_permissions(bool $deleted) : bool |
| 80 | 80 | { |
| 81 | 81 | // PONDER: Check for some generic user privilege instead ?? |
| 82 | - if ( $deleted |
|
| 82 | + if ($deleted |
|
| 83 | 83 | && !midcom::get()->auth->admin |
| 84 | 84 | && !midcom::get()->auth->is_component_sudo()) { |
| 85 | 85 | debug_add('Non-admins are not allowed to list deleted objects', MIDCOM_LOG_ERROR); |
@@ -25,7 +25,7 @@ discard block |
||
| 25 | 25 | $this->add_breadcrumb('', $person->get_label()); |
| 26 | 26 | |
| 27 | 27 | $auth = midcom::get()->auth; |
| 28 | - if ( $person->guid == midcom::get()->auth->user->guid |
|
| 28 | + if ($person->guid == midcom::get()->auth->user->guid |
|
| 29 | 29 | || $auth->can_user_do('org.openpsa.user:manage', null, org_openpsa_user_interface::class)) { |
| 30 | 30 | $buttons = []; |
| 31 | 31 | $workflow = $this->get_workflow('datamanager'); |
@@ -38,7 +38,7 @@ discard block |
||
| 38 | 38 | $delete_workflow = $this->get_workflow('delete', ['object' => $person]); |
| 39 | 39 | $buttons[] = $delete_workflow->get_button($this->router->generate('user_delete', ['guid' => $person->guid])); |
| 40 | 40 | } |
| 41 | - if ( $data['account']->get_username() |
|
| 41 | + if ($data['account']->get_username() |
|
| 42 | 42 | && $person->can_do('midgard:privileges')) { |
| 43 | 43 | $buttons[] = $workflow->get_button($this->router->generate('user_privileges', ['guid' => $person->guid]), [ |
| 44 | 44 | MIDCOM_TOOLBAR_LABEL => $this->_l10n->get("permissions"), |
@@ -52,7 +52,7 @@ discard block |
||
| 52 | 52 | } |
| 53 | 53 | } |
| 54 | 54 | |
| 55 | - if ( empty($link->toComponent) |
|
| 55 | + if (empty($link->toComponent) |
|
| 56 | 56 | && !empty($link->fromComponent)) { |
| 57 | 57 | debug_add("Setting property 'toComponent' to '{$component}'"); |
| 58 | 58 | $link->toComponent = $component; |
@@ -61,7 +61,7 @@ discard block |
||
| 61 | 61 | $link->fromComponent = $component; |
| 62 | 62 | } |
| 63 | 63 | |
| 64 | - if ( empty($link->toGuid) |
|
| 64 | + if (empty($link->toGuid) |
|
| 65 | 65 | && !empty($link->fromGuid)) { |
| 66 | 66 | $link->toClass = get_class($obj); |
| 67 | 67 | $link->toGuid = $obj->guid; |