@@ -47,7 +47,7 @@ discard block |
||
47 | 47 | $result = $this->is_username_available($fields); |
48 | 48 | |
49 | 49 | $accounthelper = new org_openpsa_user_accounthelper(); |
50 | - if ($fields['password']['switch'] && !$accounthelper->check_password_strength($fields['password']['password'])){ |
|
50 | + if ($fields['password']['switch'] && !$accounthelper->check_password_strength($fields['password']['password'])) { |
|
51 | 51 | $result = ['password' => midcom::get()->i18n->get_string('password weak', 'org.openpsa.user')]; |
52 | 52 | } |
53 | 53 | |
@@ -155,10 +155,10 @@ discard block |
||
155 | 155 | $user = new midcom_db_person($fields["person"]); |
156 | 156 | |
157 | 157 | $accounthelper = new org_openpsa_user_accounthelper($user); |
158 | - if (!$accounthelper->check_password_reuse($fields['new_password'])){ |
|
158 | + if (!$accounthelper->check_password_reuse($fields['new_password'])) { |
|
159 | 159 | $result['password'] = midcom::get()->i18n->get_string('password was already used', 'org.openpsa.user'); |
160 | 160 | } |
161 | - if (!$accounthelper->check_password_strength($fields['new_password'])){ |
|
161 | + if (!$accounthelper->check_password_strength($fields['new_password'])) { |
|
162 | 162 | $result['password'] = midcom::get()->i18n->get_string('password weak', 'org.openpsa.user'); |
163 | 163 | } |
164 | 164 | if (!empty($result)) { |
@@ -47,7 +47,7 @@ discard block |
||
47 | 47 | $result = $this->is_username_available($fields); |
48 | 48 | |
49 | 49 | $accounthelper = new org_openpsa_user_accounthelper(); |
50 | - if ($fields['password']['switch'] && !$accounthelper->check_password_strength($fields['password']['password'])){ |
|
50 | + if ($fields['password']['switch'] && !$accounthelper->check_password_strength($fields['password']['password'])) { |
|
51 | 51 | $result = ['password' => midcom::get()->i18n->get_string('password weak', 'org.openpsa.user')]; |
52 | 52 | } |
53 | 53 | |
@@ -155,10 +155,10 @@ discard block |
||
155 | 155 | $user = new midcom_db_person($fields["person"]); |
156 | 156 | |
157 | 157 | $accounthelper = new org_openpsa_user_accounthelper($user); |
158 | - if (!$accounthelper->check_password_reuse($fields['new_password'])){ |
|
158 | + if (!$accounthelper->check_password_reuse($fields['new_password'])) { |
|
159 | 159 | $result['password'] = midcom::get()->i18n->get_string('password was already used', 'org.openpsa.user'); |
160 | 160 | } |
161 | - if (!$accounthelper->check_password_strength($fields['new_password'])){ |
|
161 | + if (!$accounthelper->check_password_strength($fields['new_password'])) { |
|
162 | 162 | $result['password'] = midcom::get()->i18n->get_string('password weak', 'org.openpsa.user'); |
163 | 163 | } |
164 | 164 | if (!empty($result)) { |
@@ -88,7 +88,7 @@ discard block |
||
88 | 88 | $filename = midcom_helper_misc::urlize($this->invoice->get_label()) . '.pdf'; |
89 | 89 | |
90 | 90 | // tmp filename |
91 | - $tmp_file = midcom::get()->config->get('midcom_tempdir') . "/". $filename; |
|
91 | + $tmp_file = midcom::get()->config->get('midcom_tempdir') . "/" . $filename; |
|
92 | 92 | |
93 | 93 | // render pdf to tmp filename |
94 | 94 | $pdf_builder->render($tmp_file); |
@@ -101,7 +101,7 @@ discard block |
||
101 | 101 | $attachment->update(); |
102 | 102 | } else { |
103 | 103 | $attachment = $this->invoice->create_attachment($filename, $this->invoice->get_label(), "application/pdf"); |
104 | - if ( !$attachment |
|
104 | + if (!$attachment |
|
105 | 105 | || !$this->invoice->set_parameter("midcom.helper.datamanager2.type.blobs", "guids_pdf_file", $attachment->guid . ":" . $attachment->guid)) { |
106 | 106 | throw new midcom_error("Failed to create invoice attachment for pdf: " . midcom_connection::get_error_string()); |
107 | 107 | } |
@@ -191,7 +191,7 @@ discard block |
||
191 | 191 | $object_label = $object->guid; |
192 | 192 | } |
193 | 193 | echo "{$prefix} <td class=\"checkbox\"><input type=\"checkbox\" name=\"undelete[]\"{$disabled} value=\"{$object->guid}\" id=\"guid_{$object->guid}\" /></td>\n"; |
194 | - echo "{$prefix} <td class=\"label\" style=\"padding-left: {$indent}px\"><a href=\"". $this->router->generate('object_deleted', ['guid' =>$object->guid]) . "\">{$icon} " . $object_label . "</a></td>\n"; |
|
194 | + echo "{$prefix} <td class=\"label\" style=\"padding-left: {$indent}px\"><a href=\"" . $this->router->generate('object_deleted', ['guid' =>$object->guid]) . "\">{$icon} " . $object_label . "</a></td>\n"; |
|
195 | 195 | echo "{$prefix} <td class=\"nowrap\">" . strftime('%x %X', strtotime($object->metadata->revised)) . "</td>\n"; |
196 | 196 | |
197 | 197 | if (!empty($revisor->guid)) { |
@@ -207,7 +207,7 @@ discard block |
||
207 | 207 | $child_indent = $indent + 20; |
208 | 208 | echo "{$prefix}<tbody class=\"children\">\n"; |
209 | 209 | foreach ($child_types as $type => $children) { |
210 | - if ( count($children) < 10 |
|
210 | + if (count($children) < 10 |
|
211 | 211 | || isset($_GET['show_children'][$object->guid][$type])) { |
212 | 212 | foreach ($children as $child) { |
213 | 213 | $this->show_type($child, $child_indent, "{$prefix} ", false); |
@@ -153,7 +153,7 @@ discard block |
||
153 | 153 | return false; |
154 | 154 | } |
155 | 155 | |
156 | - if ( $key === MIDCOM_CONTEXT_ROOTTOPIC |
|
156 | + if ($key === MIDCOM_CONTEXT_ROOTTOPIC |
|
157 | 157 | && $this->_data[$key] === null) { |
158 | 158 | $this->_data[$key] = self::initialize_root_topic(); |
159 | 159 | } |
@@ -277,7 +277,7 @@ discard block |
||
277 | 277 | */ |
278 | 278 | public function set_custom_key($key, &$value) |
279 | 279 | { |
280 | - $this->_data[MIDCOM_CONTEXT_CUSTOMDATA][$key] =& $value; |
|
280 | + $this->_data[MIDCOM_CONTEXT_CUSTOMDATA][$key] = & $value; |
|
281 | 281 | } |
282 | 282 | |
283 | 283 | public function show() |
@@ -165,7 +165,8 @@ |
||
165 | 165 | try { |
166 | 166 | self::$root_topic = midcom_db_topic::get_cached($guid); |
167 | 167 | return self::$root_topic; |
168 | - } catch (midcom_error $e) { |
|
168 | + } |
|
169 | + catch (midcom_error $e) { |
|
169 | 170 | if ($e instanceof midcom_error_forbidden) { |
170 | 171 | throw $e; |
171 | 172 | } |
@@ -39,7 +39,7 @@ |
||
39 | 39 | |
40 | 40 | $schema_name = $args[0]; |
41 | 41 | $schemadb = $data['schemadb']; |
42 | - if ( $this->_config->get('simple_name_handling') |
|
42 | + if ($this->_config->get('simple_name_handling') |
|
43 | 43 | && !midcom::get()->auth->can_user_do('midcom:urlname')) { |
44 | 44 | foreach ($schemadb->all() as $schema) { |
45 | 45 | $schema->get_field('name')['readonly'] = true; |
@@ -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 |
@@ -82,7 +82,7 @@ discard block |
||
82 | 82 | { |
83 | 83 | $entries = array_merge($this->get_status_entries(), $this->get_journal_entries()); |
84 | 84 | |
85 | - usort($entries, function ($a, $b) { |
|
85 | + usort($entries, function($a, $b) { |
|
86 | 86 | if ($a['timestamp'] == $b['timestamp']) { |
87 | 87 | return $b['order'] <=> $a['order']; |
88 | 88 | } |
@@ -116,8 +116,8 @@ discard block |
||
116 | 116 | 'order' => 3 |
117 | 117 | ]; |
118 | 118 | } |
119 | - if ( $this->invoice->due |
|
120 | - && ( ( $this->invoice->due < time() |
|
119 | + if ($this->invoice->due |
|
120 | + && (($this->invoice->due < time() |
|
121 | 121 | && $this->invoice->paid == 0) |
122 | 122 | || $this->invoice->due < $this->invoice->paid)) { |
123 | 123 | $entries[] = [ |