@@ -109,7 +109,7 @@ discard block |
||
| 109 | 109 | if (file_exists(MIDCOM_STATIC_ROOT . $jqgrid_path . 'i18n/grid.locale-' . $language . '.js')) { |
| 110 | 110 | $lang = $language; |
| 111 | 111 | } |
| 112 | - $head->add_jsfile(MIDCOM_STATIC_URL . $jqgrid_path . 'i18n/grid.locale-'. $lang . '.js'); |
|
| 112 | + $head->add_jsfile(MIDCOM_STATIC_URL . $jqgrid_path . 'i18n/grid.locale-' . $lang . '.js'); |
|
| 113 | 113 | $head->add_jsfile(MIDCOM_STATIC_URL . $jqgrid_path . 'jquery.jqgrid.min.js'); |
| 114 | 114 | |
| 115 | 115 | $head->add_jsfile(MIDCOM_STATIC_URL . '/midcom.grid/jqGrid.custom.js'); |
@@ -155,7 +155,7 @@ discard block |
||
| 155 | 155 | public function set_option(string $key, $value, bool $autoquote_string = true) : self |
| 156 | 156 | { |
| 157 | 157 | $this->_raw_options[$key] = $value; |
| 158 | - if ( $autoquote_string |
|
| 158 | + if ($autoquote_string |
|
| 159 | 159 | && is_string($value)) { |
| 160 | 160 | $value = '"' . str_replace('"', '\\"', $value) . '"'; |
| 161 | 161 | } elseif ($value === true) { |
@@ -185,7 +185,7 @@ discard block |
||
| 185 | 185 | public function set_select_column(string $name, string $label, string $options, array $selectdata) : self |
| 186 | 186 | { |
| 187 | 187 | $selectstring = implode(';', array_map( |
| 188 | - function ($key, $value) { |
|
| 188 | + function($key, $value) { |
|
| 189 | 189 | return $key . ':' . $value; |
| 190 | 190 | }, |
| 191 | 191 | array_keys($selectdata), |
@@ -231,7 +231,7 @@ discard block |
||
| 231 | 231 | */ |
| 232 | 232 | public function remove_column(string $name) |
| 233 | 233 | { |
| 234 | - if ( empty($name) |
|
| 234 | + if (empty($name) |
|
| 235 | 235 | || !array_key_exists($name, $this->_columns)) { |
| 236 | 236 | throw new midcom_error('Invalid column name ' . $name); |
| 237 | 237 | } |
@@ -122,8 +122,8 @@ |
||
| 122 | 122 | return 'undefined'; |
| 123 | 123 | } |
| 124 | 124 | |
| 125 | - if ( !str_starts_with($url, '/') |
|
| 126 | - && ! preg_match('|^https?://|', $url)) { |
|
| 125 | + if (!str_starts_with($url, '/') |
|
| 126 | + && !preg_match('|^https?://|', $url)) { |
|
| 127 | 127 | $url = midcom_core_context::get()->get_key(MIDCOM_CONTEXT_ANCHORPREFIX) . $url; |
| 128 | 128 | } |
| 129 | 129 | return '"' . $url . '"'; |
@@ -64,7 +64,7 @@ |
||
| 64 | 64 | * if nothing matched so far and the subtype is alphanumeric, uppercase it on the theory |
| 65 | 65 | * that it's probably a file extension |
| 66 | 66 | */ |
| 67 | - if ( $st_orig == $subtype |
|
| 67 | + if ($st_orig == $subtype |
|
| 68 | 68 | && preg_match('/^[a-z0-9]+$/', $subtype)) { |
| 69 | 69 | $subtype = strtoupper($subtype); |
| 70 | 70 | } |
@@ -84,7 +84,7 @@ discard block |
||
| 84 | 84 | return; |
| 85 | 85 | } |
| 86 | 86 | $parent = $this->get_parent(); |
| 87 | - if ( $parent |
|
| 87 | + if ($parent |
|
| 88 | 88 | && $parent->component == 'org.openpsa.documents') { |
| 89 | 89 | midcom::get()->auth->request_sudo('org.openpsa.documents'); |
| 90 | 90 | |
@@ -154,7 +154,7 @@ discard block |
||
| 154 | 154 | $backup_attachment = $backup->create_attachment($original_attachment->name, $original_attachment->title, $original_attachment->mimetype); |
| 155 | 155 | |
| 156 | 156 | $original_handle = $original_attachment->open('r'); |
| 157 | - if ( !$backup_attachment |
|
| 157 | + if (!$backup_attachment |
|
| 158 | 158 | || !$original_handle) { |
| 159 | 159 | // Failed to copy the attachment, abort |
| 160 | 160 | return $backup->delete(); |
@@ -32,7 +32,7 @@ |
||
| 32 | 32 | echo " <td><a href='" . $data['router']->generate('preview', ['guid' => $guid, 'revision' => $number]) . "'>{$revision['version']}</a></td>\n"; |
| 33 | 33 | echo " <td>" . $formatter->datetime($revision['date']) . "</td>\n"; |
| 34 | 34 | |
| 35 | - if ( $revision['user'] |
|
| 35 | + if ($revision['user'] |
|
| 36 | 36 | && $user = midcom::get()->auth->get_user($revision['user'])) { |
| 37 | 37 | $person_label = $user->get_storage()->name; |
| 38 | 38 | echo " <td>{$person_label}</td>\n"; |
@@ -17,7 +17,7 @@ |
||
| 17 | 17 | |
| 18 | 18 | public function form_widget_simple(FormView $view, array $data) |
| 19 | 19 | { |
| 20 | - if ( !empty($data['value']) |
|
| 20 | + if (!empty($data['value']) |
|
| 21 | 21 | || is_numeric($data['value'])) { |
| 22 | 22 | return $data['value']; |
| 23 | 23 | } |
@@ -143,7 +143,7 @@ |
||
| 143 | 143 | |
| 144 | 144 | public function form_widget_simple(FormView $view, array $data) |
| 145 | 145 | { |
| 146 | - if ( !empty($data['value']) |
|
| 146 | + if (!empty($data['value']) |
|
| 147 | 147 | || is_numeric($data['value'])) { |
| 148 | 148 | return $this->escape($data['value']); |
| 149 | 149 | } |
@@ -64,7 +64,7 @@ discard block |
||
| 64 | 64 | |
| 65 | 65 | protected function add_string_field(string $key, string $type) |
| 66 | 66 | { |
| 67 | - if ( $key == 'component' |
|
| 67 | + if ($key == 'component' |
|
| 68 | 68 | && $type == midcom_db_topic::class) { |
| 69 | 69 | $this->_add_component_dropdown($key); |
| 70 | 70 | return; |
@@ -134,12 +134,12 @@ discard block |
||
| 134 | 134 | parent::add_longtext_field($key); |
| 135 | 135 | |
| 136 | 136 | // Check the user preference and configuration |
| 137 | - if ( in_array($key, ['content', 'description']) |
|
| 137 | + if (in_array($key, ['content', 'description']) |
|
| 138 | 138 | && midgard_admin_asgard_plugin::get_preference('tinymce_enabled')) { |
| 139 | 139 | $this->schema['fields'][$key]['widget'] = 'tinymce'; |
| 140 | 140 | } |
| 141 | 141 | |
| 142 | - if ( in_array($key, ['value', 'code']) |
|
| 142 | + if (in_array($key, ['value', 'code']) |
|
| 143 | 143 | && midgard_admin_asgard_plugin::get_preference('codemirror_enabled')) { |
| 144 | 144 | $this->schema['fields'][$key]['widget'] = 'codemirror'; |
| 145 | 145 | } |
@@ -257,7 +257,7 @@ discard block |
||
| 257 | 257 | if ($score1 > $score2) { |
| 258 | 258 | return 1; |
| 259 | 259 | } |
| 260 | - if ( $score1 < 3 |
|
| 260 | + if ($score1 < 3 |
|
| 261 | 261 | || $score1 > 6) { |
| 262 | 262 | return strnatcmp($first, $second); |
| 263 | 263 | } |
@@ -277,7 +277,7 @@ discard block |
||
| 277 | 277 | { |
| 278 | 278 | $success = true; |
| 279 | 279 | foreach ($this->_request_data['config']->_global as $global_key => $global_value) { |
| 280 | - if ( isset($config[$global_key]) |
|
| 280 | + if (isset($config[$global_key]) |
|
| 281 | 281 | && $config[$global_key] != $global_value) { |
| 282 | 282 | continue; |
| 283 | 283 | // Skip the ones we will set next |
@@ -290,7 +290,7 @@ discard block |
||
| 290 | 290 | } |
| 291 | 291 | |
| 292 | 292 | foreach ($config as $key => $value) { |
| 293 | - if ( is_array($value) |
|
| 293 | + if (is_array($value) |
|
| 294 | 294 | || is_object($value)) { |
| 295 | 295 | /** |
| 296 | 296 | * See http://trac.midgard-project.org/ticket/1442 |