@@ -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; |