@@ -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']); |
@@ -61,7 +61,7 @@ |
||
61 | 61 | public static function get_by_object(org_openpsa_invoices_interfaces_customer $object) : self |
62 | 62 | { |
63 | 63 | if ( !($bd = self::get_billing_data(org_openpsa_contacts_group_dba::class, $object->customer)) |
64 | - // check if the customerContact is set and has invoice_data |
|
64 | + // check if the customerContact is set and has invoice_data |
|
65 | 65 | && !($bd = self::get_billing_data(org_openpsa_contacts_person_dba::class, $object->customerContact))) { |
66 | 66 | $bd = new self(); |
67 | 67 | $due = midcom_baseclasses_components_configuration::get('org.openpsa.invoices', 'config')->get('default_due_days'); |
@@ -60,7 +60,7 @@ |
||
60 | 60 | */ |
61 | 61 | public static function get_by_object(org_openpsa_invoices_interfaces_customer $object) : self |
62 | 62 | { |
63 | - if ( !($bd = self::get_billing_data(org_openpsa_contacts_group_dba::class, $object->customer)) |
|
63 | + if (!($bd = self::get_billing_data(org_openpsa_contacts_group_dba::class, $object->customer)) |
|
64 | 64 | // check if the customerContact is set and has invoice_data |
65 | 65 | && !($bd = self::get_billing_data(org_openpsa_contacts_person_dba::class, $object->customerContact))) { |
66 | 66 | $bd = new self(); |
@@ -96,7 +96,8 @@ discard block |
||
96 | 96 | // if the property useContactAddress is set |
97 | 97 | $billing_data[0]->set_address(); |
98 | 98 | return $billing_data[0]; |
99 | - } catch (midcom_error $e) { |
|
99 | + } |
|
100 | + catch (midcom_error $e) { |
|
100 | 101 | $e->log(); |
101 | 102 | return null; |
102 | 103 | } |
@@ -141,10 +142,12 @@ discard block |
||
141 | 142 | { |
142 | 143 | try { |
143 | 144 | return new org_openpsa_contacts_person_dba($this->linkGuid); |
144 | - } catch (midcom_error $e) { |
|
145 | + } |
|
146 | + catch (midcom_error $e) { |
|
145 | 147 | try { |
146 | 148 | return new org_openpsa_contacts_group_dba($this->linkGuid); |
147 | - } catch (midcom_error $e) { |
|
149 | + } |
|
150 | + catch (midcom_error $e) { |
|
148 | 151 | debug_add("Failed to load contact with GUID: " . $this->linkGuid . " - last error:" . $e->getMessage(), MIDCOM_LOG_ERROR); |
149 | 152 | return false; |
150 | 153 | } |
@@ -26,7 +26,7 @@ |
||
26 | 26 | |
27 | 27 | private function load_controller() : controller |
28 | 28 | { |
29 | - if ( $this->_config->get('simple_name_handling') |
|
29 | + if ($this->_config->get('simple_name_handling') |
|
30 | 30 | && !midcom::get()->auth->admin) { |
31 | 31 | foreach ($this->_request_data['schemadb']->all() as $schema) { |
32 | 32 | $schema->get_field('name')['readonly'] = true; |
@@ -314,7 +314,7 @@ |
||
314 | 314 | |
315 | 315 | private function add_head_elements(bool $centralized = false) : bool |
316 | 316 | { |
317 | - if ( !midcom::get()->auth->user |
|
317 | + if (!midcom::get()->auth->user |
|
318 | 318 | || !midcom::get()->config->get('toolbars_enable_centralized') |
319 | 319 | || !midcom::get()->auth->can_user_do('midcom:centralized_toolbar', null, __CLASS__)) { |
320 | 320 | return false; |
@@ -109,20 +109,20 @@ discard block |
||
109 | 109 | } |
110 | 110 | |
111 | 111 | // Approval attributes |
112 | - if ( midcom::get()->config->get('metadata_approval') |
|
112 | + if (midcom::get()->config->get('metadata_approval') |
|
113 | 113 | && !$object->metadata->is_approved()) { |
114 | 114 | $css_classes[] = 'unapproved'; |
115 | 115 | } |
116 | 116 | |
117 | 117 | // Hiding and scheduling attributes |
118 | - if ( ( !midcom::get()->config->get('show_hidden_objects') |
|
118 | + if ((!midcom::get()->config->get('show_hidden_objects') |
|
119 | 119 | || midcom::get()->config->get('metadata_scheduling')) |
120 | 120 | && !$object->metadata->is_visible()) { |
121 | 121 | $css_classes[] = 'hidden'; |
122 | 122 | } |
123 | 123 | |
124 | 124 | // Folder's class |
125 | - if ( $object instanceof midcom_db_topic |
|
125 | + if ($object instanceof midcom_db_topic |
|
126 | 126 | && $page_class = $object->get_parameter('midcom.services.metadata', 'page_class')) { |
127 | 127 | $css_classes[] = $page_class; |
128 | 128 | } |
@@ -193,7 +193,7 @@ discard block |
||
193 | 193 | private function _add_opengraph_metadata(midcom_helper_metadata $view_metadata) |
194 | 194 | { |
195 | 195 | $opengraph_type = $view_metadata->object->get_parameter('midcom.helper.metadata', 'opengraph_type'); |
196 | - if ( $opengraph_type |
|
196 | + if ($opengraph_type |
|
197 | 197 | && $opengraph_type != 'none') { |
198 | 198 | $request_metadata = $this->get_request_metadata(); |
199 | 199 |
@@ -111,7 +111,7 @@ |
||
111 | 111 | */ |
112 | 112 | public function _handler_moderate_ajax(Request $request, string $status, array &$data) |
113 | 113 | { |
114 | - if ( !$request->request->has('action') |
|
114 | + if (!$request->request->has('action') |
|
115 | 115 | || !$request->request->has('guid')) { |
116 | 116 | throw new midcom_error_notfound('Incomplete POST data'); |
117 | 117 | } |
@@ -86,7 +86,7 @@ |
||
86 | 86 | */ |
87 | 87 | public function _handler_add(Request $request, string $guid, array &$data) |
88 | 88 | { |
89 | - if ( !$request->request->has('product') |
|
89 | + if (!$request->request->has('product') |
|
90 | 90 | && !$request->request->has('org_openpsa_sales')) { |
91 | 91 | throw new midcom_error('No product specified, aborting.'); |
92 | 92 | } |
@@ -21,7 +21,7 @@ |
||
21 | 21 | */ |
22 | 22 | public function _handler_approval(Request $request, string $handler_id) |
23 | 23 | { |
24 | - if ( !$request->request->has('guid') |
|
24 | + if (!$request->request->has('guid') |
|
25 | 25 | || !$request->request->has('return_to')) { |
26 | 26 | throw new midcom_error('Cannot process approval request, request is incomplete.'); |
27 | 27 | } |
@@ -279,7 +279,7 @@ discard block |
||
279 | 279 | } |
280 | 280 | |
281 | 281 | $this->_leaves[$id] = $leaf; |
282 | - $this->_loaded_leaves[$node->id][$id] =& $this->_leaves[$id]; |
|
282 | + $this->_loaded_leaves[$node->id][$id] = & $this->_leaves[$id]; |
|
283 | 283 | } |
284 | 284 | } |
285 | 285 | |
@@ -337,7 +337,7 @@ discard block |
||
337 | 337 | continue; |
338 | 338 | } |
339 | 339 | |
340 | - if ( !$show_noentry |
|
340 | + if (!$show_noentry |
|
341 | 341 | && self::$_nodes[$id]->noentry) { |
342 | 342 | // Hide "noentry" items |
343 | 343 | continue; |