@@ -86,7 +86,7 @@ |
||
86 | 86 | require $basedir . $file; |
87 | 87 | } |
88 | 88 | |
89 | - midcom::get()->auth->drop_sudo();# |
|
89 | + midcom::get()->auth->drop_sudo(); # |
|
90 | 90 | return 0; |
91 | 91 | } |
92 | 92 |
@@ -80,7 +80,8 @@ |
||
80 | 80 | |
81 | 81 | try { |
82 | 82 | require $basedir . $file; |
83 | - } catch (\midcom_error_forbidden $e) { |
|
83 | + } |
|
84 | + catch (\midcom_error_forbidden $e) { |
|
84 | 85 | $dialog = $this->getHelperSet()->get('question'); |
85 | 86 | $this->login($dialog, $input, $output); |
86 | 87 | require $basedir . $file; |
@@ -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; |
@@ -237,7 +237,8 @@ |
||
237 | 237 | { |
238 | 238 | try { |
239 | 239 | return $this->get_l10n($component)->get($stringid); |
240 | - } catch (midcom_error $e) { |
|
240 | + } |
|
241 | + catch (midcom_error $e) { |
|
241 | 242 | $e->log(MIDCOM_LOG_WARN); |
242 | 243 | return $stringid; |
243 | 244 | } |
@@ -99,7 +99,8 @@ |
||
99 | 99 | if ($request->request->has('midcom_helper_datamanager2_save')) { |
100 | 100 | try { |
101 | 101 | $rules = $this->_load_rules($request); |
102 | - } catch (midcom_error $e) { |
|
102 | + } |
|
103 | + catch (midcom_error $e) { |
|
103 | 104 | midcom::get()->uimessages->add('org.openpsa.directmarketing', $this->_l10n->get($e->getMessage()), 'error'); |
104 | 105 | return; |
105 | 106 | } |
@@ -36,7 +36,7 @@ discard block |
||
36 | 36 | if ($name_property === null) { |
37 | 37 | $name_property = midcom_helper_reflector::get_name_property($this->_object); |
38 | 38 | } |
39 | - if ( empty($name_property) |
|
39 | + if (empty($name_property) |
|
40 | 40 | || !midcom_helper_reflector::get($this->_object)->property_exists($name_property)) { |
41 | 41 | // Could not resolve valid property |
42 | 42 | return null; |
@@ -132,7 +132,7 @@ discard block |
||
132 | 132 | |
133 | 133 | $qb->add_constraint($child_name_property, '=', $name); |
134 | 134 | if ($qb->count()) { |
135 | - debug_add("Name clash in sibling class {$schema_type} for " . get_class($this->_object) . " #{$this->_object->id} (path '" . midcom_helper_reflector_tree::resolve_path($this->_object, '/') . "')" ); |
|
135 | + debug_add("Name clash in sibling class {$schema_type} for " . get_class($this->_object) . " #{$this->_object->id} (path '" . midcom_helper_reflector_tree::resolve_path($this->_object, '/') . "')"); |
|
136 | 136 | return false; |
137 | 137 | } |
138 | 138 | } |
@@ -209,7 +209,7 @@ discard block |
||
209 | 209 | } while (!$this->name_is_unique()); |
210 | 210 | |
211 | 211 | // Get a copy of the current, usable name |
212 | - $ret = (string)$this->_object->{$name_prop}; |
|
212 | + $ret = (string) $this->_object->{$name_prop}; |
|
213 | 213 | // Restore the original name |
214 | 214 | $this->_object->{$name_prop} = $original_name; |
215 | 215 | return $ret; |
@@ -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 | } |
@@ -161,7 +161,8 @@ discard block |
||
161 | 161 | $response = $this->handle($this->request); |
162 | 162 | $response->send(); |
163 | 163 | $this->terminate($this->request, $response); |
164 | - } catch (Error $e) { |
|
164 | + } |
|
165 | + catch (Error $e) { |
|
165 | 166 | $this->getHttpKernel()->terminateWithException($e); |
166 | 167 | } |
167 | 168 | } |
@@ -218,11 +219,13 @@ discard block |
||
218 | 219 | $this->skip_page_style = true; |
219 | 220 | try { |
220 | 221 | $response = $this->handle($request, HttpKernelInterface::SUB_REQUEST, false); |
221 | - } catch (midcom_error_notfound | midcom_error_forbidden $e) { |
|
222 | + } |
|
223 | + catch (midcom_error_notfound | midcom_error_forbidden $e) { |
|
222 | 224 | $e->log(); |
223 | 225 | midcom_core_context::leave(); |
224 | 226 | return; |
225 | - } finally { |
|
227 | + } |
|
228 | + finally { |
|
226 | 229 | $this->skip_page_style = $backup; |
227 | 230 | } |
228 | 231 |
@@ -45,7 +45,8 @@ |
||
45 | 45 | break; |
46 | 46 | } |
47 | 47 | } |
48 | - } catch (midcom_error $e) { |
|
48 | + } |
|
49 | + catch (midcom_error $e) { |
|
49 | 50 | } |
50 | 51 | |
51 | 52 | $path_parts = array_reverse($path_parts); |
@@ -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 | } |