@@ -36,7 +36,7 @@ |
||
36 | 36 | } |
37 | 37 | $config['name'] = $name; |
38 | 38 | $field = $this->prepare_field($config); |
39 | - if ( isset($config['default']) |
|
39 | + if (isset($config['default']) |
|
40 | 40 | && (!$this->object->id || $field instanceof transientnode)) { |
41 | 41 | $field->set_value($config['default']); |
42 | 42 | } |
@@ -55,7 +55,7 @@ |
||
55 | 55 | |
56 | 56 | private function set_context(midcom_core_context $context) |
57 | 57 | { |
58 | - if ( $context->get_key(MIDCOM_CONTEXT_PERMALINKGUID) === null |
|
58 | + if ($context->get_key(MIDCOM_CONTEXT_PERMALINKGUID) === null |
|
59 | 59 | || $context->get_key(MIDCOM_CONTEXT_PAGETITLE) == '') { |
60 | 60 | // Retrieve Metadata |
61 | 61 | $nav = new midcom_helper_nav(); |
@@ -12,10 +12,10 @@ |
||
12 | 12 | { |
13 | 13 | public function validate($value, Constraint $constraint) |
14 | 14 | { |
15 | - if ( empty($value) |
|
15 | + if (empty($value) |
|
16 | 16 | || !empty($value['delete']) |
17 | - || ( empty($value['file']) |
|
18 | - && ( (empty($constraint->config['do_not_save_archival']) && empty($value['archival'])) |
|
17 | + || (empty($value['file']) |
|
18 | + && ((empty($constraint->config['do_not_save_archival']) && empty($value['archival'])) |
|
19 | 19 | || (!empty($constraint->config['do_not_save_archival']) && empty($value['main']))))) { |
20 | 20 | $this->context->buildViolation('This value should not be blank.') |
21 | 21 | ->addViolation(); |
@@ -45,7 +45,7 @@ |
||
45 | 45 | if ($query->has('start')) { |
46 | 46 | $defaults['start'] = strtotime(substr($query->get('start'), 0, 19)) + 1; |
47 | 47 | if ($query->has('end')) { |
48 | - $defaults['end']= strtotime(substr($query->get('end'), 0, 19)); |
|
48 | + $defaults['end'] = strtotime(substr($query->get('end'), 0, 19)); |
|
49 | 49 | } else { |
50 | 50 | $defaults['end'] = $defaults['start'] + 3600; |
51 | 51 | } |
@@ -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); |
@@ -69,7 +69,8 @@ discard block |
||
69 | 69 | |
70 | 70 | try { |
71 | 71 | $result = $router->match($url); |
72 | - } catch (ResourceNotFoundException $e) { |
|
72 | + } |
|
73 | + catch (ResourceNotFoundException $e) { |
|
73 | 74 | throw new midcom_error_notfound('This URL method is unknown.'); |
74 | 75 | } |
75 | 76 | $this->request->attributes->add($result); |
@@ -151,11 +152,13 @@ discard block |
||
151 | 152 | |
152 | 153 | try { |
153 | 154 | $result = $router->match($url); |
154 | - } catch (ResourceNotFoundException $e) { |
|
155 | + } |
|
156 | + catch (ResourceNotFoundException $e) { |
|
155 | 157 | // No match |
156 | 158 | debug_add("Component {$viewer->_component} in {$viewer->_topic->name} declared unable to handle request.", MIDCOM_LOG_INFO); |
157 | 159 | throw new midcom_error_notfound("This page is not available on this server."); |
158 | - } catch (MethodNotAllowedException $e) { |
|
160 | + } |
|
161 | + catch (MethodNotAllowedException $e) { |
|
159 | 162 | debug_add("Component {$viewer->_component} in {$viewer->_topic->name} declared unable to handle request (method not allowed).", MIDCOM_LOG_INFO); |
160 | 163 | throw new midcom_error_forbidden($e->getMessage(), Response::HTTP_METHOD_NOT_ALLOWED); |
161 | 164 | } |
@@ -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; |
@@ -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 | } |