@@ -138,7 +138,7 @@ |
||
| 138 | 138 | $prefix = $this->context->get_key(MIDCOM_CONTEXT_ANCHORPREFIX); |
| 139 | 139 | |
| 140 | 140 | // Check if we need to start up a plugin. |
| 141 | - if ( count($argv) > 1 |
|
| 141 | + if (count($argv) > 1 |
|
| 142 | 142 | && $config = plugin::get_config($argv[0], $argv[1])) { |
| 143 | 143 | $namespace = array_shift($argv); |
| 144 | 144 | $name = array_shift($argv); |
@@ -59,7 +59,8 @@ discard block |
||
| 59 | 59 | |
| 60 | 60 | try { |
| 61 | 61 | $result = $router->match($url); |
| 62 | - } catch (ResourceNotFoundException) { |
|
| 62 | + } |
|
| 63 | + catch (ResourceNotFoundException) { |
|
| 63 | 64 | throw new midcom_error_notfound('This URL method is unknown.'); |
| 64 | 65 | } |
| 65 | 66 | $this->request->attributes->add($result); |
@@ -118,11 +119,13 @@ discard block |
||
| 118 | 119 | |
| 119 | 120 | try { |
| 120 | 121 | $result = $router->match($url); |
| 121 | - } catch (ResourceNotFoundException) { |
|
| 122 | + } |
|
| 123 | + catch (ResourceNotFoundException) { |
|
| 122 | 124 | // No match |
| 123 | 125 | debug_add("Component {$viewer->_component} in {$viewer->_topic->name} declared unable to handle request.", MIDCOM_LOG_INFO); |
| 124 | 126 | throw new midcom_error_notfound("This page is not available on this server."); |
| 125 | - } catch (MethodNotAllowedException $e) { |
|
| 127 | + } |
|
| 128 | + catch (MethodNotAllowedException $e) { |
|
| 126 | 129 | debug_add("Component {$viewer->_component} in {$viewer->_topic->name} declared unable to handle request (method not allowed).", MIDCOM_LOG_INFO); |
| 127 | 130 | throw new midcom_error_forbidden($e->getMessage(), Response::HTTP_METHOD_NOT_ALLOWED); |
| 128 | 131 | } |
@@ -33,7 +33,7 @@ |
||
| 33 | 33 | public function _on_loaded() |
| 34 | 34 | { |
| 35 | 35 | // if we are missing the component, use the nullcomponent. |
| 36 | - if ( !$this->component |
|
| 36 | + if (!$this->component |
|
| 37 | 37 | || !midcom::get()->componentloader->is_installed($this->component)) { |
| 38 | 38 | debug_add("Topic {$this->id} has no component assigned to it, using 'midcom.core.nullcomponent'.", |
| 39 | 39 | MIDCOM_LOG_INFO); |
@@ -72,7 +72,8 @@ |
||
| 72 | 72 | $component = key($watch); |
| 73 | 73 | try { |
| 74 | 74 | $interface = $this->loader->get_interface_class($component); |
| 75 | - } catch (\midcom_error $e) { |
|
| 75 | + } |
|
| 76 | + catch (\midcom_error $e) { |
|
| 76 | 77 | debug_add("Failed to load the component {$component}: " . $e->getMessage(), MIDCOM_LOG_INFO); |
| 77 | 78 | continue; |
| 78 | 79 | } |
@@ -101,7 +101,7 @@ |
||
| 101 | 101 | $rawdata = base64_decode($request->cookies->get('midcom_services_i18n')); |
| 102 | 102 | $array = unserialize($rawdata); |
| 103 | 103 | |
| 104 | - if ( !array_key_exists('language', $array) |
|
| 104 | + if (!array_key_exists('language', $array) |
|
| 105 | 105 | || !array_key_exists('charset', $array)) { |
| 106 | 106 | debug_add("Rejecting cookie, it seems invalid."); |
| 107 | 107 | return false; |
@@ -155,7 +155,8 @@ discard block |
||
| 155 | 155 | { |
| 156 | 156 | try { |
| 157 | 157 | Locales::getName($lang); |
| 158 | - } catch (Exception $e) { |
|
| 158 | + } |
|
| 159 | + catch (Exception $e) { |
|
| 159 | 160 | debug_add("Language {$lang} not found:" . $e->getMessage(), MIDCOM_LOG_ERROR); |
| 160 | 161 | return false; |
| 161 | 162 | } |
@@ -236,7 +237,8 @@ discard block |
||
| 236 | 237 | { |
| 237 | 238 | try { |
| 238 | 239 | return $this->get_l10n($component)->get($stringid); |
| 239 | - } catch (midcom_error $e) { |
|
| 240 | + } |
|
| 241 | + catch (midcom_error $e) { |
|
| 240 | 242 | $e->log(MIDCOM_LOG_WARN); |
| 241 | 243 | return $stringid; |
| 242 | 244 | } |
@@ -97,7 +97,7 @@ |
||
| 97 | 97 | $loader->load($this->getProjectDir() . '/config/services.yml'); |
| 98 | 98 | } |
| 99 | 99 | if ($classes = midcom::get_registered_service_classes()) { |
| 100 | - $loader->load(function (ContainerBuilder $container) use ($classes) { |
|
| 100 | + $loader->load(function(ContainerBuilder $container) use ($classes) { |
|
| 101 | 101 | foreach ($classes as $id => $class) { |
| 102 | 102 | $container->findDefinition($id)->setClass($class); |
| 103 | 103 | } |
@@ -159,7 +159,8 @@ discard block |
||
| 159 | 159 | $response = $this->handle($request); |
| 160 | 160 | $response->send(); |
| 161 | 161 | $this->terminate($request, $response); |
| 162 | - } catch (Error $e) { |
|
| 162 | + } |
|
| 163 | + catch (Error $e) { |
|
| 163 | 164 | $this->getHttpKernel()->terminateWithException($e); |
| 164 | 165 | } |
| 165 | 166 | } |
@@ -210,9 +211,11 @@ discard block |
||
| 210 | 211 | try { |
| 211 | 212 | $response = $this->handle($request, HttpKernelInterface::SUB_REQUEST, false); |
| 212 | 213 | echo $response->getContent(); |
| 213 | - } catch (midcom_error_notfound | midcom_error_forbidden $e) { |
|
| 214 | + } |
|
| 215 | + catch (midcom_error_notfound | midcom_error_forbidden $e) { |
|
| 214 | 216 | $e->log(); |
| 215 | - } finally { |
|
| 217 | + } |
|
| 218 | + finally { |
|
| 216 | 219 | $this->skip_page_style = $backup; |
| 217 | 220 | midcom_core_context::leave(); |
| 218 | 221 | } |
@@ -152,7 +152,7 @@ |
||
| 152 | 152 | $preparsed = midcom_helper_formatter::compile($style); |
| 153 | 153 | |
| 154 | 154 | if (midcom_core_context::get()->has_custom_key('request_data')) { |
| 155 | - $data =& midcom_core_context::get()->get_custom_key('request_data'); |
|
| 155 | + $data = & midcom_core_context::get()->get_custom_key('request_data'); |
|
| 156 | 156 | } |
| 157 | 157 | |
| 158 | 158 | try { |
@@ -136,7 +136,8 @@ |
||
| 136 | 136 | |
| 137 | 137 | try { |
| 138 | 138 | eval('?>' . $preparsed); |
| 139 | - } catch (ParseError $e) { |
|
| 139 | + } |
|
| 140 | + catch (ParseError $e) { |
|
| 140 | 141 | throw new midcom_error("Failed to parse style element '{$path}': " . $e->getMessage() . ' in line ' . $e->getLine()); |
| 141 | 142 | } |
| 142 | 143 | } |
@@ -196,7 +196,7 @@ |
||
| 196 | 196 | |
| 197 | 197 | private function resolve_includes(string $content) : string |
| 198 | 198 | { |
| 199 | - return preg_replace_callback("/<\\(([a-zA-Z0-9 _-]+)\\)>/", function (array $matches) { |
|
| 199 | + return preg_replace_callback("/<\\(([a-zA-Z0-9 _-]+)\\)>/", function(array $matches) { |
|
| 200 | 200 | $element = $matches[1]; |
| 201 | 201 | |
| 202 | 202 | switch ($element) { |
@@ -98,7 +98,7 @@ |
||
| 98 | 98 | { |
| 99 | 99 | if (!isset($this->_modules[$name])) { |
| 100 | 100 | $this->_modules[$name] = $module; |
| 101 | - $this->$name =& $this->_modules[$name]; |
|
| 101 | + $this->$name = & $this->_modules[$name]; |
|
| 102 | 102 | } |
| 103 | 103 | } |
| 104 | 104 | |
@@ -218,8 +218,8 @@ discard block |
||
| 218 | 218 | { |
| 219 | 219 | $config = midcom_baseclasses_components_configuration::get('net.nehmer.comments', 'config'); |
| 220 | 220 | |
| 221 | - if ( $config->get('ratings_enable') |
|
| 222 | - && ( $config->get('ratings_cache_to_object') |
|
| 221 | + if ($config->get('ratings_enable') |
|
| 222 | + && ($config->get('ratings_cache_to_object') |
|
| 223 | 223 | || $config->get('comment_count_cache_to_object'))) { |
| 224 | 224 | // Handle ratings |
| 225 | 225 | $comments = self::list_by_objectguid($this->objectguid); |
@@ -268,7 +268,7 @@ discard block |
||
| 268 | 268 | |
| 269 | 269 | private function _send_notifications() |
| 270 | 270 | { |
| 271 | - if ( empty($this->title) |
|
| 271 | + if (empty($this->title) |
|
| 272 | 272 | && empty($this->content)) { |
| 273 | 273 | // No need to send notifications about empty rating entries |
| 274 | 274 | return; |