@@ -15,7 +15,7 @@ |
||
15 | 15 | |
16 | 16 | foreach ($properties as $propname => $val) { |
17 | 17 | $description = preg_replace('/ *\n */', "\n", $val['value']); |
18 | - if ( $val['link'] |
|
18 | + if ($val['link'] |
|
19 | 19 | && $linked_component = midcom::get()->dbclassloader->get_component_for_class($val['link_name'])) { |
20 | 20 | $proplink = $data['router']->generate('help', ['component' => $linked_component, 'help_id' => 'mgdschemas']); |
21 | 21 | $proplink = "<a href='{$proplink}#{$val['link_name']}' title='{$linked_component}/{$val['link_name']}::{$val['link_target']}'>{$val['link_name']}:{$val['link_target']}</a>"; |
@@ -110,7 +110,7 @@ |
||
110 | 110 | public function _on_updated() |
111 | 111 | { |
112 | 112 | // Sync the object's ACL properties into MidCOM ACL system |
113 | - if ( !$this->_skip_acl_refresh) { |
|
113 | + if (!$this->_skip_acl_refresh) { |
|
114 | 114 | if ($this->orgOpenpsaAccesstype && $this->orgOpenpsaOwnerWg) { |
115 | 115 | debug_add("Synchronizing task ACLs to MidCOM"); |
116 | 116 | $sync = new org_openpsa_core_acl_synchronizer(); |
@@ -51,7 +51,7 @@ |
||
51 | 51 | $this->article->require_do('midgard:update'); |
52 | 52 | |
53 | 53 | $schemadb = $data['schemadb']; |
54 | - if ( $this->_config->get('simple_name_handling') |
|
54 | + if ($this->_config->get('simple_name_handling') |
|
55 | 55 | && !midcom::get()->auth->can_user_do('midcom:urlname')) { |
56 | 56 | foreach ($schemadb->all() as $schema) { |
57 | 57 | $schema->get_field('name')['readonly'] = true; |
@@ -107,7 +107,7 @@ |
||
107 | 107 | |
108 | 108 | // sending per email enabled in billing data? |
109 | 109 | $billing_data = org_openpsa_invoices_billing_data_dba::get_by_object($this->invoice); |
110 | - if ( !$this->invoice->sent |
|
110 | + if (!$this->invoice->sent |
|
111 | 111 | && intval($billing_data->sendingoption) == 2) { |
112 | 112 | $buttons[] = $this->build_button('send_by_mail', 'paper-plane'); |
113 | 113 | } |
@@ -46,7 +46,7 @@ discard block |
||
46 | 46 | ] |
47 | 47 | ]); |
48 | 48 | |
49 | - $resolver->setNormalizer('constraints', function (Options $options, $value) { |
|
49 | + $resolver->setNormalizer('constraints', function(Options $options, $value) { |
|
50 | 50 | if ($options['required']) { |
51 | 51 | return [new constraint(['config' => $options['type_config']])]; |
52 | 52 | } |
@@ -78,7 +78,7 @@ discard block |
||
78 | 78 | |
79 | 79 | $builder->add('delete', CheckboxType::class, ['attr' => [ |
80 | 80 | "class" => "midcom_datamanager_photo_checkbox" |
81 | - ], "required" => false ]); |
|
81 | + ], "required" => false]); |
|
82 | 82 | $builder->add('identifier', HiddenType::class); |
83 | 83 | } |
84 | 84 |
@@ -59,7 +59,7 @@ |
||
59 | 59 | |
60 | 60 | public function __set($name, $value) |
61 | 61 | { |
62 | - if ( $name == 'homepage' |
|
62 | + if ($name == 'homepage' |
|
63 | 63 | && !empty($value) |
64 | 64 | && $value != $this->homepage) { |
65 | 65 | $this->_register_prober = true; |
@@ -319,7 +319,7 @@ discard block |
||
319 | 319 | $password_length = mb_strlen($password); |
320 | 320 | |
321 | 321 | if ($password_length < $this->_config->get('min_password_length')) { |
322 | - if ($show_ui_message){ |
|
322 | + if ($show_ui_message) { |
|
323 | 323 | midcom::get()->uimessages->add($this->_l10n->get('org.openpsa.user'), $this->_l10n->get('password too short'), 'error'); |
324 | 324 | } |
325 | 325 | return false; |
@@ -337,7 +337,7 @@ discard block |
||
337 | 337 | } |
338 | 338 | |
339 | 339 | if ($score < $this->_config->get('min_password_score')) { |
340 | - if ($show_ui_message){ |
|
340 | + if ($show_ui_message) { |
|
341 | 341 | midcom::get()->uimessages->add($this->_l10n->get('org.openpsa.user'), $this->_l10n->get('password weak'), 'error'); |
342 | 342 | } |
343 | 343 | return false; |
@@ -151,7 +151,7 @@ discard block |
||
151 | 151 | $account = $this->get_account(); |
152 | 152 | if (!empty($new_password)) { |
153 | 153 | //check if the new encrypted password was already used |
154 | - if ( !$this->check_password_reuse($new_password, true) |
|
154 | + if (!$this->check_password_reuse($new_password, true) |
|
155 | 155 | || !$this->check_password_strength($new_password, true)) { |
156 | 156 | $this->errstr = "password strength too low"; |
157 | 157 | return false; |
@@ -270,7 +270,7 @@ discard block |
||
270 | 270 | $password_length = mb_strlen($password); |
271 | 271 | |
272 | 272 | if ($password_length < $this->_config->get('min_password_length')) { |
273 | - if ($show_ui_message){ |
|
273 | + if ($show_ui_message) { |
|
274 | 274 | midcom::get()->uimessages->add($this->_l10n->get('org.openpsa.user'), $this->_l10n->get('password too short'), 'error'); |
275 | 275 | } |
276 | 276 | return false; |
@@ -288,7 +288,7 @@ discard block |
||
288 | 288 | } |
289 | 289 | |
290 | 290 | if ($score < $this->_config->get('min_password_score')) { |
291 | - if ($show_ui_message){ |
|
291 | + if ($show_ui_message) { |
|
292 | 292 | midcom::get()->uimessages->add($this->_l10n->get('org.openpsa.user'), $this->_l10n->get('password weak'), 'error'); |
293 | 293 | } |
294 | 294 | return false; |
@@ -432,7 +432,7 @@ discard block |
||
432 | 432 | |
433 | 433 | public static function get_person_by_formdata(array $data) : ?midcom_db_person |
434 | 434 | { |
435 | - if ( empty($data['username']) |
|
435 | + if (empty($data['username']) |
|
436 | 436 | || empty($data['password'])) { |
437 | 437 | return null; |
438 | 438 | } |
@@ -463,7 +463,7 @@ discard block |
||
463 | 463 | $max_attempts = midcom_baseclasses_components_configuration::get($component, 'config')->get('max_password_attempts'); |
464 | 464 | $timeframe = midcom_baseclasses_components_configuration::get($component, 'config')->get('password_block_timeframe_min'); |
465 | 465 | |
466 | - if ( $max_attempts == 0 |
|
466 | + if ($max_attempts == 0 |
|
467 | 467 | || $timeframe == 0) { |
468 | 468 | return $stat; |
469 | 469 | } |
@@ -485,7 +485,7 @@ discard block |
||
485 | 485 | * If the maximum number of attempts is reached and the oldest attempt |
486 | 486 | * on the stack is within our defined timeframe, we block the account |
487 | 487 | */ |
488 | - if ( count($attempts) >= $max_attempts |
|
488 | + if (count($attempts) >= $max_attempts |
|
489 | 489 | && $attempts[$max_attempts - 1] >= (time() - ($timeframe * 60))) { |
490 | 490 | $this->disable_account(); |
491 | 491 | $stat = false; |
@@ -409,7 +409,8 @@ |
||
409 | 409 | //Get the parent object |
410 | 410 | try { |
411 | 411 | $parent = midcom::get()->dbfactory->get_object_by_guid($this->objectguid); |
412 | - } catch (midcom_error $e) { |
|
412 | + } |
|
413 | + catch (midcom_error $e) { |
|
413 | 414 | $e->log(); |
414 | 415 | return; |
415 | 416 | } |
@@ -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; |
@@ -84,7 +84,7 @@ discard block |
||
84 | 84 | net_nemein_rss_manage::add_toolbar_buttons($this->_node_toolbar, $this->_topic->can_do('midgard:create')); |
85 | 85 | } |
86 | 86 | |
87 | - if ( $this->_topic->can_do('midgard:update') |
|
87 | + if ($this->_topic->can_do('midgard:update') |
|
88 | 88 | && $this->_topic->can_do('midcom:component_config')) { |
89 | 89 | $buttons[] = $workflow->get_button('config/', [ |
90 | 90 | MIDCOM_TOOLBAR_LABEL => $this->_l10n_midcom->get('component configuration'), |
@@ -114,7 +114,7 @@ discard block |
||
114 | 114 | $this->_request_data['categories'] = explode(',', $this->_config->get('categories')); |
115 | 115 | |
116 | 116 | foreach ($this->_request_data['schemadb']->all() as $schema) { |
117 | - if ( $schema->has_field('categories') |
|
117 | + if ($schema->has_field('categories') |
|
118 | 118 | && $schema->get_field('categories')['type'] == 'select') { |
119 | 119 | // TODO: Merge schema local and component config categories? |
120 | 120 | $options = array_combine($this->_request_data['categories'], $this->_request_data['categories']); |