@@ -341,7 +341,7 @@ discard block |
||
| 341 | 341 | $tmpfile = null; |
| 342 | 342 | $imagesize = getimagesize($this->_filename); |
| 343 | 343 | |
| 344 | - if ( $imagesize[2] == IMAGETYPE_JPEG |
|
| 344 | + if ($imagesize[2] == IMAGETYPE_JPEG |
|
| 345 | 345 | && $this->_jpegtran_available()) { |
| 346 | 346 | /* jpegtran */ |
| 347 | 347 | $operations = [ |
@@ -409,7 +409,7 @@ discard block |
||
| 409 | 409 | $imagesize = getimagesize($this->_filename); |
| 410 | 410 | |
| 411 | 411 | // Try lossless jpegtran rotation if possible |
| 412 | - if ( $imagesize[2] == IMAGETYPE_JPEG |
|
| 412 | + if ($imagesize[2] == IMAGETYPE_JPEG |
|
| 413 | 413 | && ($rotate % 90 == 0) |
| 414 | 414 | && $this->_jpegtran_available()) { |
| 415 | 415 | $tmpfile = $this->_get_tempfile(); |
@@ -539,7 +539,7 @@ discard block |
||
| 539 | 539 | */ |
| 540 | 540 | public function fill($x = null, $y = null, $color = null, string $gravity = 'center') |
| 541 | 541 | { |
| 542 | - if ( empty($x) |
|
| 542 | + if (empty($x) |
|
| 543 | 543 | || empty($y) |
| 544 | 544 | || empty($color)) { |
| 545 | 545 | //This is a bit silly, but here for backwards compatibility... |
@@ -97,7 +97,7 @@ |
||
| 97 | 97 | $attachment->update(); |
| 98 | 98 | } else { |
| 99 | 99 | $attachment = $this->invoice->create_attachment($filename, $this->invoice->get_label(), "application/pdf"); |
| 100 | - if ( !$attachment |
|
| 100 | + if (!$attachment |
|
| 101 | 101 | || !$this->invoice->set_parameter("midcom.helper.datamanager2.type.blobs", "guids_pdf_file", $attachment->guid . ":" . $attachment->guid)) { |
| 102 | 102 | throw new midcom_error("Failed to create invoice attachment for pdf: " . midcom_connection::get_error_string()); |
| 103 | 103 | } |
@@ -131,7 +131,7 @@ discard block |
||
| 131 | 131 | self::$_localedb[$this->_library] = []; |
| 132 | 132 | } |
| 133 | 133 | |
| 134 | - $this->_stringdb =& self::$_localedb[$this->_library]; |
|
| 134 | + $this->_stringdb = & self::$_localedb[$this->_library]; |
|
| 135 | 135 | |
| 136 | 136 | $this->set_language(midcom::get()->i18n->get_current_language()); |
| 137 | 137 | } |
@@ -256,7 +256,7 @@ discard block |
||
| 256 | 256 | private function error(string $message, string $filename, int $line) : midcom_error |
| 257 | 257 | { |
| 258 | 258 | $line++; // Array is 0-indexed |
| 259 | - return new midcom_error('L10n DB SYNTAX ERROR: ' . $message . ' at ' . $filename . ' ' . $line); |
|
| 259 | + return new midcom_error('L10n DB SYNTAX ERROR: ' . $message . ' at ' . $filename . ' ' . $line); |
|
| 260 | 260 | } |
| 261 | 261 | |
| 262 | 262 | /** |
@@ -92,7 +92,7 @@ discard block |
||
| 92 | 92 | $tmp = $this->_object; |
| 93 | 93 | |
| 94 | 94 | $i = 0; |
| 95 | - while ( !empty($tmp->guid) |
|
| 95 | + while (!empty($tmp->guid) |
|
| 96 | 96 | && !is_a($tmp, midcom_db_topic::class) |
| 97 | 97 | && $i < 100) { |
| 98 | 98 | // Get the parent; wishing eventually to get a topic |
@@ -137,7 +137,7 @@ discard block |
||
| 137 | 137 | |
| 138 | 138 | $assignees = $this->load_assignees(); |
| 139 | 139 | $this->process_assignees($assignees, $schemadb); |
| 140 | - $assignee_field =& $schemadb->get('privileges')->get_field('add_assignee'); |
|
| 140 | + $assignee_field = & $schemadb->get('privileges')->get_field('add_assignee'); |
|
| 141 | 141 | |
| 142 | 142 | if (!$this->additional_assignee) { |
| 143 | 143 | // Populate additional assignee selector |
@@ -149,7 +149,7 @@ |
||
| 149 | 149 | $child_indent = $indent + 20; |
| 150 | 150 | echo "{$prefix}<tbody class=\"children\">\n"; |
| 151 | 151 | foreach ($child_types as $type => $children) { |
| 152 | - if ( count($children) < 10 |
|
| 152 | + if (count($children) < 10 |
|
| 153 | 153 | || isset($_GET['show_children'][$object->guid][$type])) { |
| 154 | 154 | foreach ($children as $child) { |
| 155 | 155 | $this->show_type($child, $child_indent, "{$prefix} ", false); |
@@ -28,7 +28,7 @@ |
||
| 28 | 28 | &(published:h); |
| 29 | 29 | <?php |
| 30 | 30 | if ($data['linked']) { |
| 31 | - echo $data['l10n']->get('to') ." <a href=\"{$data['node'][MIDCOM_NAV_ABSOLUTEURL]}\">{$data['node'][MIDCOM_NAV_NAME]}</a>\n"; |
|
| 31 | + echo $data['l10n']->get('to') . " <a href=\"{$data['node'][MIDCOM_NAV_ABSOLUTEURL]}\">{$data['node'][MIDCOM_NAV_NAME]}</a>\n"; |
|
| 32 | 32 | } |
| 33 | 33 | ?> |
| 34 | 34 | </p> |
@@ -50,7 +50,7 @@ discard block |
||
| 50 | 50 | */ |
| 51 | 51 | private function search_nodes(array $node, midcom_helper_nav $nap, string $prefix) |
| 52 | 52 | { |
| 53 | - if ( !array_key_exists($node[MIDCOM_NAV_COMPONENT], $this->_request_data['components']) |
|
| 53 | + if (!array_key_exists($node[MIDCOM_NAV_COMPONENT], $this->_request_data['components']) |
|
| 54 | 54 | && $node[MIDCOM_NAV_COMPONENT] != 'midcom.helper.search') { |
| 55 | 55 | $l10n = $this->_i18n->get_l10n($node[MIDCOM_NAV_COMPONENT]); |
| 56 | 56 | $this->_request_data['components'][$node[MIDCOM_NAV_COMPONENT]] = $l10n->get($node[MIDCOM_NAV_COMPONENT]); |
@@ -98,7 +98,7 @@ discard block |
||
| 98 | 98 | } |
| 99 | 99 | $this->prepare_formdata($_REQUEST['type']); |
| 100 | 100 | |
| 101 | - if ( count(explode(' ', $data['query'])) == 1 |
|
| 101 | + if (count(explode(' ', $data['query'])) == 1 |
|
| 102 | 102 | && !str_contains($data['query'], '*') |
| 103 | 103 | && $this->_config->get('single_term_auto_wildcard')) { |
| 104 | 104 | //If there is only one search term append * to the query if auto_wildcard is enabled |
@@ -33,7 +33,7 @@ discard block |
||
| 33 | 33 | public function _handler_create(Request $request, string $guid) |
| 34 | 34 | { |
| 35 | 35 | $current_object = midcom::get()->dbfactory->get_object_by_guid($guid); |
| 36 | - $entry= new org_openpsa_relatedto_journal_entry_dba(); |
|
| 36 | + $entry = new org_openpsa_relatedto_journal_entry_dba(); |
|
| 37 | 37 | $entry->linkGuid = $current_object->guid; |
| 38 | 38 | |
| 39 | 39 | midcom::get()->head->set_pagetitle($this->_l10n->get('add journal entry')); |
@@ -52,7 +52,7 @@ discard block |
||
| 52 | 52 | $workflow = $this->get_workflow('datamanager', ['controller' => $data['controller']]); |
| 53 | 53 | if ($entry->can_do('midgard:delete')) { |
| 54 | 54 | $delete = $this->get_workflow('delete', ['object' => $entry]); |
| 55 | - $url = $this->router->generate('journal_entry_delete', ['guid' => $guid ]); |
|
| 55 | + $url = $this->router->generate('journal_entry_delete', ['guid' => $guid]); |
|
| 56 | 56 | $workflow->add_dialog_button($delete, $url); |
| 57 | 57 | } |
| 58 | 58 | return $workflow->run($request); |
@@ -84,7 +84,7 @@ discard block |
||
| 84 | 84 | public function form_end(FormView $view, array $data) |
| 85 | 85 | { |
| 86 | 86 | $string = ''; |
| 87 | - if ( !isset($data['render_rest']) |
|
| 87 | + if (!isset($data['render_rest']) |
|
| 88 | 88 | || $data['render_rest']) { |
| 89 | 89 | $string .= $this->renderer->rest($view); |
| 90 | 90 | } |
@@ -223,7 +223,7 @@ discard block |
||
| 223 | 223 | $data['attr']['class'] = 'shorttext'; |
| 224 | 224 | } |
| 225 | 225 | |
| 226 | - if ( !empty($data['value']) |
|
| 226 | + if (!empty($data['value']) |
|
| 227 | 227 | || is_numeric($data['value'])) { |
| 228 | 228 | $data['attr']['value'] = $data['value']; |
| 229 | 229 | } |
@@ -265,7 +265,7 @@ discard block |
||
| 265 | 265 | { |
| 266 | 266 | if ($view->vars['readonly']) { |
| 267 | 267 | $string = $this->get_view_renderer()->autocomplete_widget($view, $data); |
| 268 | - return $string . $this->renderer->widget($view['selection']); |
|
| 268 | + return $string . $this->renderer->widget($view['selection']); |
|
| 269 | 269 | } |
| 270 | 270 | |
| 271 | 271 | autocomplete::add_head_elements($data['handler_options']['creation_mode_enabled'], $data['handler_options']['sortable']); |
@@ -275,7 +275,7 @@ discard block |
||
| 275 | 275 | $jsinit .= "midcom_helper_datamanager2_autocomplete.create_dm2_widget('{$element_id}_search_input', {$data['min_chars']});\n"; |
| 276 | 276 | |
| 277 | 277 | $string = '<fieldset ' . $this->renderer->block($view, 'widget_container_attributes') . '>'; |
| 278 | - $string .= $this->renderer->widget($view['selection']); |
|
| 278 | + $string .= $this->renderer->widget($view['selection']); |
|
| 279 | 279 | $string .= ' ' . $this->renderer->widget($view['search_input']); |
| 280 | 280 | $string .= '</fieldset>'; |
| 281 | 281 | return $string . $this->jsinit($jsinit); |
@@ -324,7 +324,7 @@ discard block |
||
| 324 | 324 | } |
| 325 | 325 | |
| 326 | 326 | $string = '<select '; |
| 327 | - if ( $data['required'] |
|
| 327 | + if ($data['required'] |
|
| 328 | 328 | && null === $data['placeholder'] |
| 329 | 329 | && $data['placeholder_in_choices'] === false |
| 330 | 330 | && $data['multiple'] === false) { |
@@ -338,7 +338,7 @@ discard block |
||
| 338 | 338 | $string .= $this->renderer->block($view, 'widget_attributes', $data) . '>'; |
| 339 | 339 | if (null !== $data['placeholder']) { |
| 340 | 340 | $string .= '<option value=""'; |
| 341 | - if ( $data['required'] |
|
| 341 | + if ($data['required'] |
|
| 342 | 342 | && empty($data['value']) |
| 343 | 343 | && "0" !== $data['value']) { |
| 344 | 344 | $string .= ' selected="selected"'; |
@@ -500,7 +500,7 @@ discard block |
||
| 500 | 500 | $string .= '<label class="midcom_datamanager_photo_label">' . $this->renderer->humanize('delete photo') . ' ' . $this->renderer->widget($data['form']['delete']) . '</label>'; |
| 501 | 501 | $string .= '<ul>'; |
| 502 | 502 | foreach ($objects as $info) { |
| 503 | - if ( $info['size_x'] |
|
| 503 | + if ($info['size_x'] |
|
| 504 | 504 | && $info['size_y']) { |
| 505 | 505 | $size = "{$info['size_x']}×{$info['size_y']}"; |
| 506 | 506 | } else { |
@@ -525,7 +525,7 @@ discard block |
||
| 525 | 525 | $string .= '</td></tr></table></div>'; |
| 526 | 526 | $string .= $this->renderer->row($data['form']['identifier']); |
| 527 | 527 | |
| 528 | - return $string . $this->jsinit('init_image_widget("' . $view->vars['id'] .'");'); |
|
| 528 | + return $string . $this->jsinit('init_image_widget("' . $view->vars['id'] . '");'); |
|
| 529 | 529 | } |
| 530 | 530 | |
| 531 | 531 | public function subform_widget(FormView $view, array $data) |