@@ -88,7 +88,8 @@ |
||
| 88 | 88 | $style = midcom_db_style::get_cached($style_id); |
| 89 | 89 | $styleeditor_url = midcom_connection::get_url('self') . "__mfa/asgard/object/view/{$style->guid}/"; |
| 90 | 90 | $enabled = true; |
| 91 | - } catch (midcom_error $e) { |
|
| 91 | + } |
|
| 92 | + catch (midcom_error $e) { |
|
| 92 | 93 | $e->log(); |
| 93 | 94 | } |
| 94 | 95 | } |
@@ -37,7 +37,7 @@ discard block |
||
| 37 | 37 | } |
| 38 | 38 | $buttons = []; |
| 39 | 39 | $workflow = new midcom\workflow\datamanager; |
| 40 | - if ( $this->topic->can_do('midgard:update') |
|
| 40 | + if ($this->topic->can_do('midgard:update') |
|
| 41 | 41 | && $this->topic->can_do('midcom.admin.folder:topic_management')) { |
| 42 | 42 | $buttons[] = $workflow->get_button("__ais/folder/edit/", [ |
| 43 | 43 | MIDCOM_TOOLBAR_LABEL => midcom::get()->i18n->get_string('edit folder', 'midcom.admin.folder'), |
@@ -49,7 +49,7 @@ discard block |
||
| 49 | 49 | ]); |
| 50 | 50 | } |
| 51 | 51 | |
| 52 | - if ( $urltopic->can_do('midgard:update') |
|
| 52 | + if ($urltopic->can_do('midgard:update') |
|
| 53 | 53 | && $urltopic->can_do('midcom.admin.folder:topic_management')) { |
| 54 | 54 | // Allow to move other than root folder |
| 55 | 55 | if ($urltopic->guid !== midcom::get()->config->get('midcom_root_topic_guid')) { |
@@ -61,7 +61,7 @@ discard block |
||
| 61 | 61 | } |
| 62 | 62 | } |
| 63 | 63 | |
| 64 | - if ( $this->topic->can_do('midgard:update') |
|
| 64 | + if ($this->topic->can_do('midgard:update') |
|
| 65 | 65 | && $this->topic->can_do('midcom.admin.folder:topic_management')) { |
| 66 | 66 | $buttons[] = [ |
| 67 | 67 | MIDCOM_TOOLBAR_URL => "__ais/folder/order/", |
@@ -78,7 +78,7 @@ discard block |
||
| 78 | 78 | ]; |
| 79 | 79 | } |
| 80 | 80 | $buttons = array_merge($buttons, $this->get_approval_controls($this->topic, false)); |
| 81 | - if ( $this->topic->can_do('midcom.admin.folder:template_management') |
|
| 81 | + if ($this->topic->can_do('midcom.admin.folder:template_management') |
|
| 82 | 82 | && midcom::get()->auth->can_user_do('midgard.admin.asgard:manage_objects', null, 'midgard_admin_asgard_plugin')) { |
| 83 | 83 | $enabled = false; |
| 84 | 84 | $styleeditor_url = ''; |
@@ -103,7 +103,7 @@ discard block |
||
| 103 | 103 | ]; |
| 104 | 104 | } |
| 105 | 105 | |
| 106 | - if ( $this->topic->can_do('midgard:create') |
|
| 106 | + if ($this->topic->can_do('midgard:create') |
|
| 107 | 107 | && $this->topic->can_do('midcom.admin.folder:topic_management')) { |
| 108 | 108 | $buttons[] = $workflow->get_button("__ais/folder/create/", [ |
| 109 | 109 | MIDCOM_TOOLBAR_LABEL => midcom::get()->i18n->get_string('create subfolder', 'midcom.admin.folder'), |
@@ -111,7 +111,7 @@ discard block |
||
| 111 | 111 | MIDCOM_TOOLBAR_ACCESSKEY => 'f', |
| 112 | 112 | ]); |
| 113 | 113 | } |
| 114 | - if ( $urltopic->guid !== midcom::get()->config->get('midcom_root_topic_guid') |
|
| 114 | + if ($urltopic->guid !== midcom::get()->config->get('midcom_root_topic_guid') |
|
| 115 | 115 | && $urltopic->can_do('midgard:delete') |
| 116 | 116 | && $urltopic->can_do('midcom.admin.folder:topic_management')) { |
| 117 | 117 | $workflow = new midcom\workflow\delete(['object' => $urltopic, 'recursive' => true]); |
@@ -53,7 +53,7 @@ discard block |
||
| 53 | 53 | { |
| 54 | 54 | $node = $nap->get_node($node_id); |
| 55 | 55 | |
| 56 | - if ( !array_key_exists($node[MIDCOM_NAV_COMPONENT], $this->_request_data['components']) |
|
| 56 | + if (!array_key_exists($node[MIDCOM_NAV_COMPONENT], $this->_request_data['components']) |
|
| 57 | 57 | && $node[MIDCOM_NAV_COMPONENT] != 'midcom.helper.search') { |
| 58 | 58 | $l10n = $this->_i18n->get_l10n($node[MIDCOM_NAV_COMPONENT]); |
| 59 | 59 | $this->_request_data['components'][$node[MIDCOM_NAV_COMPONENT]] = $l10n->get($node[MIDCOM_NAV_COMPONENT]); |
@@ -112,7 +112,7 @@ discard block |
||
| 112 | 112 | $this->prepare_query_data(); |
| 113 | 113 | $this->prepare_formdata($_REQUEST['type']); |
| 114 | 114 | |
| 115 | - if ( count(explode(' ', $data['query'])) == 1 |
|
| 115 | + if (count(explode(' ', $data['query'])) == 1 |
|
| 116 | 116 | && strpos($data['query'], '*') === false |
| 117 | 117 | && $this->_config->get('single_term_auto_wilcard')) { |
| 118 | 118 | //If there is only one search term append * to the query if auto_wildcard is enabled |
@@ -192,12 +192,12 @@ discard block |
||
| 192 | 192 | $this->_request_data['last_document_number'] = $last_document_id + 1; |
| 193 | 193 | $this->_request_data['shown_documents'] = $last_document_id - $first_document_id + 1; |
| 194 | 194 | $this->_request_data['results_per_page'] = $results_per_page; |
| 195 | - $this->_request_data['all_results'] =& $result; |
|
| 195 | + $this->_request_data['all_results'] = & $result; |
|
| 196 | 196 | $this->_request_data['result'] = array_slice($result, $first_document_id, $results_per_page); |
| 197 | 197 | |
| 198 | 198 | // Register GUIDs for cache engine |
| 199 | 199 | foreach ($this->_request_data['result'] as $doc) { |
| 200 | - if ( !isset($doc->source) |
|
| 200 | + if (!isset($doc->source) |
|
| 201 | 201 | || !mgd_is_guid($doc->source)) { |
| 202 | 202 | // Non-Midgard results don't need to go through cache registration |
| 203 | 203 | continue; |
@@ -143,7 +143,8 @@ |
||
| 143 | 143 | |
| 144 | 144 | try { |
| 145 | 145 | $data = file_get_contents($filename); |
| 146 | - } catch (Exception $e) { |
|
| 146 | + } |
|
| 147 | + catch (Exception $e) { |
|
| 147 | 148 | return false; |
| 148 | 149 | } |
| 149 | 150 | |
@@ -100,12 +100,12 @@ |
||
| 100 | 100 | { |
| 101 | 101 | $this->_master = $master; |
| 102 | 102 | |
| 103 | - $this->_request_data =& $master->_request_data; |
|
| 103 | + $this->_request_data = & $master->_request_data; |
|
| 104 | 104 | $this->_topic = $master->_topic; |
| 105 | 105 | |
| 106 | 106 | // Load component specific stuff, special treatment if the handler has |
| 107 | 107 | // a component different than the master handler set. |
| 108 | - if ( $this->_component |
|
| 108 | + if ($this->_component |
|
| 109 | 109 | && $this->_component != $master->_component) { |
| 110 | 110 | $this->_config->store_from_object($this->_topic, $this->_component, true); |
| 111 | 111 | } else { |
@@ -4,7 +4,8 @@ |
||
| 4 | 4 | try { |
| 5 | 5 | $person = new midcom_db_person($metadata->locker); |
| 6 | 6 | $name = $person->name; |
| 7 | -} catch (midcom_error $e) { |
|
| 7 | +} |
|
| 8 | +catch (midcom_error $e) { |
|
| 8 | 9 | $name = $this->data['handler']->_l10n_midcom->get('unknown user'); |
| 9 | 10 | $e->log(); |
| 10 | 11 | } |
@@ -16,7 +16,8 @@ |
||
| 16 | 16 | foreach ($atts as $att) { |
| 17 | 17 | try { |
| 18 | 18 | $att->file_to_cache(); |
| 19 | - } catch (midcom_error $e) { |
|
| 19 | + } |
|
| 20 | + catch (midcom_error $e) { |
|
| 20 | 21 | echo 'Error: ' . $e->getMessage() . "\n"; |
| 21 | 22 | } |
| 22 | 23 | } |
@@ -55,7 +55,7 @@ |
||
| 55 | 55 | |
| 56 | 56 | $http_client = new org_openpsa_httplib(); |
| 57 | 57 | $http_client->set_param('timeout', 300); |
| 58 | -if ( !empty($_SERVER['PHP_AUTH_USER']) |
|
| 58 | +if (!empty($_SERVER['PHP_AUTH_USER']) |
|
| 59 | 59 | && !empty($_SERVER['PHP_AUTH_PW'])) { |
| 60 | 60 | $http_client->basicauth['user'] = $_SERVER['PHP_AUTH_USER']; |
| 61 | 61 | $http_client->basicauth['password'] = $_SERVER['PHP_AUTH_PW']; |
@@ -86,7 +86,7 @@ |
||
| 86 | 86 | return; |
| 87 | 87 | } |
| 88 | 88 | $initialized = true; |
| 89 | - if ( !midcom::get()->auth->user |
|
| 89 | + if (!midcom::get()->auth->user |
|
| 90 | 90 | || !midcom::get()->config->get('toolbars_enable_centralized') |
| 91 | 91 | || !midcom::get()->auth->can_user_do('midcom:centralized_toolbar', null, $this)) { |
| 92 | 92 | return; |
@@ -153,7 +153,8 @@ |
||
| 153 | 153 | $job['component'] = $component; |
| 154 | 154 | $this->_jobs[] = $job; |
| 155 | 155 | } |
| 156 | - } catch (midcom_error $e) { |
|
| 156 | + } |
|
| 157 | + catch (midcom_error $e) { |
|
| 157 | 158 | $e->log(MIDCOM_LOG_ERROR); |
| 158 | 159 | debug_print_r('Got this job declaration:', $job); |
| 159 | 160 | echo "ERROR: Failed to register a job for {$component}: " . $e->getMessage() . "\n"; |