@@ -9,7 +9,7 @@ |
||
9 | 9 | continue; |
10 | 10 | } |
11 | 11 | |
12 | - if ( !$object->can_do('midgard:update') |
|
12 | + if (!$object->can_do('midgard:update') |
|
13 | 13 | || !$object->can_do('midgard:attachments')) { |
14 | 14 | echo sprintf(midcom::get()->i18n->get_string('not recreating object %s %s, reason %s', 'midcom'), $type, $object->guid, 'Insufficient privileges') . "\n"; |
15 | 15 | continue; |
@@ -30,7 +30,7 @@ discard block |
||
30 | 30 | <?php |
31 | 31 | foreach ($type_choices as $value => $label) { |
32 | 32 | $selected = ''; |
33 | - if ( isset($data['type_filter']) |
|
33 | + if (isset($data['type_filter']) |
|
34 | 34 | && $data['type_filter'] == $value) { |
35 | 35 | $selected = ' selected="selected"'; |
36 | 36 | } |
@@ -45,7 +45,7 @@ discard block |
||
45 | 45 | <?php |
46 | 46 | foreach ($revised_after_choices as $value => $label) { |
47 | 47 | $selected = ''; |
48 | - if ( isset($data['revised_after']) |
|
48 | + if (isset($data['revised_after']) |
|
49 | 49 | && $data['revised_after'] == date('Y-m-d', $value)) { |
50 | 50 | $selected = ' selected="selected"'; |
51 | 51 | } |
@@ -18,7 +18,7 @@ |
||
18 | 18 | public function initialize(midcom_baseclasses_components_viewer $master, Router $router) |
19 | 19 | { |
20 | 20 | $this->router = $router; |
21 | - $this->_request_data =& $master->_request_data; |
|
21 | + $this->_request_data = & $master->_request_data; |
|
22 | 22 | $this->_topic = $master->_topic; |
23 | 23 | $this->_request_data['l10n'] = $this->_l10n; |
24 | 24 |
@@ -138,7 +138,8 @@ |
||
138 | 138 | try { |
139 | 139 | $parent = new org_openpsa_projects_project($task->project); |
140 | 140 | $options['success_url'] = $this->router->generate('project', ['guid' => $parent->guid]); |
141 | - } catch (midcom_error $e) { |
|
141 | + } |
|
142 | + catch (midcom_error $e) { |
|
142 | 143 | $e->log(); |
143 | 144 | } |
144 | 145 |
@@ -89,7 +89,8 @@ |
||
89 | 89 | } elseif ((int) $group > 0) { |
90 | 90 | try { |
91 | 91 | $this->parent = new org_openpsa_products_product_group_dba((int) $group); |
92 | - } catch (midcom_error $e) { |
|
92 | + } |
|
93 | + catch (midcom_error $e) { |
|
93 | 94 | $e->log(); |
94 | 95 | } |
95 | 96 | } |
@@ -33,7 +33,7 @@ |
||
33 | 33 | midcom::get()->auth->require_valid_user(); |
34 | 34 | midcom::get()->skip_page_style = true; |
35 | 35 | |
36 | - $this->add_stylesheet(MIDCOM_STATIC_URL ."/midcom.helper.imagepopup/styling.css", 'screen'); |
|
36 | + $this->add_stylesheet(MIDCOM_STATIC_URL . "/midcom.helper.imagepopup/styling.css", 'screen'); |
|
37 | 37 | |
38 | 38 | $data['filetype'] = $filetype; |
39 | 39 | $data['object'] = null; |
@@ -100,7 +100,8 @@ |
||
100 | 100 | try { |
101 | 101 | $task = org_openpsa_projects_task_dba::get_cached($hour_report->task); |
102 | 102 | $options['success_url'] = $this->router->generate('list_hours_task', ['guid' => $task->guid]); |
103 | - } catch (midcom_error $e) { |
|
103 | + } |
|
104 | + catch (midcom_error $e) { |
|
104 | 105 | $e->log(); |
105 | 106 | } |
106 | 107 | return $this->get_workflow('delete', $options)->run($request); |
@@ -133,7 +133,7 @@ |
||
133 | 133 | return new midcom_response_relocate($relocate); |
134 | 134 | } |
135 | 135 | |
136 | - private function parse_input(ParameterBag $input, string $action) : int|bool |
|
136 | + private function parse_input(ParameterBag $input, string $action) : int | bool |
|
137 | 137 | { |
138 | 138 | if (!in_array($action, ['invoiceable', 'invoice', 'task'])) { |
139 | 139 | throw new midcom_error('passed action ' . $action . ' is unknown'); |
@@ -70,13 +70,13 @@ discard block |
||
70 | 70 | } |
71 | 71 | } elseif (!$this->form->isSubmitted()) { |
72 | 72 | $this->form->handleRequest($request); |
73 | - if ( $this->form->isSubmitted() |
|
73 | + if ($this->form->isSubmitted() |
|
74 | 74 | && $button = $this->form->getClickedButton()) { |
75 | 75 | $operation = $button->getConfig()->getOption('operation'); |
76 | 76 | } |
77 | 77 | } |
78 | 78 | |
79 | - if ( in_array($operation, [self::SAVE, self::NEXT]) |
|
79 | + if (in_array($operation, [self::SAVE, self::NEXT]) |
|
80 | 80 | && !$this->form->isValid()) { |
81 | 81 | $operation = self::EDIT; |
82 | 82 | } |
@@ -159,7 +159,7 @@ discard block |
||
159 | 159 | if ($storage->get_last_operation() === self::SAVE) { |
160 | 160 | // Get GUIDs and such for new-created dependent objects |
161 | 161 | $this->dm->set_storage($storage->get_value(), $this->dm->get_schema()->get_name()); |
162 | - } elseif ( !$this->form->isValid() |
|
162 | + } elseif (!$this->form->isValid() |
|
163 | 163 | && $storage->move_uploaded_files() > 0) { |
164 | 164 | $form = $this->dm->get_form($this->form->getName(), true); |
165 | 165 | $this->copy_errors($this->form, $form); |
@@ -18,7 +18,7 @@ discard block |
||
18 | 18 | $item->title = $data['article']->title; |
19 | 19 | $arg = $data['article']->name ?: $data['article']->guid; |
20 | 20 | |
21 | -if ( $data['config']->get('link_to_external_url') |
|
21 | +if ($data['config']->get('link_to_external_url') |
|
22 | 22 | && !empty($data['article']->url)) { |
23 | 23 | $item->link = $data['article']->url; |
24 | 24 | } else { |
@@ -37,7 +37,7 @@ discard block |
||
37 | 37 | $item->description .= '<div class="abstract">' . $view['abstract'] . '</div>'; |
38 | 38 | } |
39 | 39 | |
40 | -if ( array_key_exists('image', $view) |
|
40 | +if (array_key_exists('image', $view) |
|
41 | 41 | && $data['config']->get('rss_use_image')) { |
42 | 42 | $item->description .= "\n<div class=\"image\">" . $view['image'] . '</div>'; |
43 | 43 | } |