@@ -36,7 +36,8 @@ discard block |
||
36 | 36 | // Add person to campaign |
37 | 37 | try { |
38 | 38 | $campaign = new org_openpsa_directmarketing_campaign_dba($add_to); |
39 | - } catch (midcom_error $e) { |
|
39 | + } |
|
40 | + catch (midcom_error $e) { |
|
40 | 41 | // FIXME: More informative error message |
41 | 42 | $this->notify('Failed adding person %s to campaign %s', $add_to, 'error'); |
42 | 43 | return; |
@@ -86,7 +87,8 @@ discard block |
||
86 | 87 | try { |
87 | 88 | $campaigns[$membership->campaign] = new org_openpsa_directmarketing_campaign_dba($membership->campaign); |
88 | 89 | $campaign_membership_map[$membership->campaign] = $membership; |
89 | - } catch (midcom_error $e) { |
|
90 | + } |
|
91 | + catch (midcom_error $e) { |
|
90 | 92 | debug_add('Failed to load campaign ' . $membership->campaign . ', reason: ' . $e->getMessage()); |
91 | 93 | } |
92 | 94 | } |
@@ -48,7 +48,8 @@ discard block |
||
48 | 48 | // We're creating invoice for chosen customer |
49 | 49 | try { |
50 | 50 | $this->customer = new org_openpsa_contacts_group_dba($company); |
51 | - } catch (midcom_error $e) { |
|
51 | + } |
|
52 | + catch (midcom_error $e) { |
|
52 | 53 | $contact = new org_openpsa_contacts_person_dba($company); |
53 | 54 | $this->contact_id = $contact->id; |
54 | 55 | } |
@@ -155,7 +156,8 @@ discard block |
||
155 | 156 | if (!empty($this->invoice->customer)) { |
156 | 157 | try { |
157 | 158 | $this->customer = org_openpsa_contacts_group_dba::get_cached($this->invoice->customer); |
158 | - } catch (midcom_error $e) { |
|
159 | + } |
|
160 | + catch (midcom_error $e) { |
|
159 | 161 | $customer_field['hidden'] = true; |
160 | 162 | $e->log(); |
161 | 163 | } |
@@ -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 | } |
@@ -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); |
@@ -40,7 +40,8 @@ |
||
40 | 40 | { |
41 | 41 | try { |
42 | 42 | $this->db = connection::get_em()->getConnection()->getWrappedConnection(); |
43 | - } catch (\Exception $e) { |
|
43 | + } |
|
44 | + catch (\Exception $e) { |
|
44 | 45 | $this->db = $this->create_connection($input, $output); |
45 | 46 | } |
46 | 47 |
@@ -83,7 +83,8 @@ |
||
83 | 83 | $productgroup = new org_openpsa_products_product_group_dba($this->_product->productGroup); |
84 | 84 | $replacements['<PRODUCTGROUP_TITLE>'] = $productgroup->title; |
85 | 85 | $replacements['<PRODUCTGROUP_CODE>'] = $productgroup->code; |
86 | - } catch (midcom_error $e) { |
|
86 | + } |
|
87 | + catch (midcom_error $e) { |
|
87 | 88 | $replacements['<PRODUCTGROUP_TITLE>'] = ''; |
88 | 89 | $replacements['<PRODUCTGROUP_CODE>'] = ''; |
89 | 90 | } |
@@ -68,7 +68,8 @@ discard block |
||
68 | 68 | $field['type_config']['options'] = [0 => '', $customer->id => $customer->official]; |
69 | 69 | |
70 | 70 | $defaults['customer'] = $customer->id; |
71 | - } catch (midcom_error $e) { |
|
71 | + } |
|
72 | + catch (midcom_error $e) { |
|
72 | 73 | $customer = new org_openpsa_contacts_person_dba($guid); |
73 | 74 | $defaults['customerContact'] = $customer->id; |
74 | 75 | $field['type_config']['options'] = $this->list_groups(new org_openpsa_sales_salesproject_dba, [$customer->id => true]); |
@@ -142,7 +143,8 @@ discard block |
||
142 | 143 | try { |
143 | 144 | $company = org_openpsa_contacts_group_dba::get_cached($company_id); |
144 | 145 | $ret[$company->id] = $company->get_label(); |
145 | - } catch (midcom_error $e) { |
|
146 | + } |
|
147 | + catch (midcom_error $e) { |
|
146 | 148 | $e->log(); |
147 | 149 | } |
148 | 150 | } |
@@ -319,7 +319,7 @@ discard block |
||
319 | 319 | $password_length = mb_strlen($password); |
320 | 320 | |
321 | 321 | if ($password_length < $this->_config->get('min_password_length')) { |
322 | - if ($show_ui_message){ |
|
322 | + if ($show_ui_message) { |
|
323 | 323 | midcom::get()->uimessages->add($this->_l10n->get('org.openpsa.user'), $this->_l10n->get('password too short'), 'error'); |
324 | 324 | } |
325 | 325 | return false; |
@@ -337,7 +337,7 @@ discard block |
||
337 | 337 | } |
338 | 338 | |
339 | 339 | if ($score < $this->_config->get('min_password_score')) { |
340 | - if ($show_ui_message){ |
|
340 | + if ($show_ui_message) { |
|
341 | 341 | midcom::get()->uimessages->add($this->_l10n->get('org.openpsa.user'), $this->_l10n->get('password weak'), 'error'); |
342 | 342 | } |
343 | 343 | return false; |
@@ -131,7 +131,8 @@ discard block |
||
131 | 131 | try { |
132 | 132 | $this->_backend->index($documents); |
133 | 133 | return true; |
134 | - } catch (Exception $e) { |
|
134 | + } |
|
135 | + catch (Exception $e) { |
|
135 | 136 | if ($batch) { |
136 | 137 | throw $e; |
137 | 138 | } |
@@ -160,7 +161,8 @@ discard block |
||
160 | 161 | try { |
161 | 162 | $this->_backend->delete($RIs); |
162 | 163 | return true; |
163 | - } catch (Exception $e) { |
|
164 | + } |
|
165 | + catch (Exception $e) { |
|
164 | 166 | debug_add("Deleting error: " . $e->getMessage(), MIDCOM_LOG_ERROR); |
165 | 167 | return false; |
166 | 168 | } |
@@ -182,7 +184,8 @@ discard block |
||
182 | 184 | try { |
183 | 185 | $this->_backend->delete_all($constraint); |
184 | 186 | return true; |
185 | - } catch (Exception $e) { |
|
187 | + } |
|
188 | + catch (Exception $e) { |
|
186 | 189 | debug_add("Deleting error: " . $e->getMessage(), MIDCOM_LOG_ERROR); |
187 | 190 | return false; |
188 | 191 | } |
@@ -216,7 +219,8 @@ discard block |
||
216 | 219 | |
217 | 220 | try { |
218 | 221 | $result_raw = $this->_backend->query($query, $filter, $options); |
219 | - } catch (Exception $e) { |
|
222 | + } |
|
223 | + catch (Exception $e) { |
|
220 | 224 | debug_add("Query error: " . $e->getMessage(), MIDCOM_LOG_ERROR); |
221 | 225 | return $result; |
222 | 226 | } |
@@ -237,7 +241,8 @@ discard block |
||
237 | 241 | if (!empty($document->topic_guid)) { |
238 | 242 | try { |
239 | 243 | midcom_db_topic::get_cached($document->topic_guid); |
240 | - } catch (midcom_error $e) { |
|
244 | + } |
|
245 | + catch (midcom_error $e) { |
|
241 | 246 | // Skip document, the object is hidden. |
242 | 247 | debug_add("Skipping the generic document {$document->title}, its topic seems to be invisible, we cannot proceed."); |
243 | 248 | continue; |
@@ -250,7 +255,8 @@ discard block |
||
250 | 255 | // Strip language code from end of RI if it looks like "<GUID>_<LANG>" |
251 | 256 | try { |
252 | 257 | midcom::get()->dbfactory->get_object_by_guid(preg_replace('/^([0-9a-f]{32,80})_[a-z]{2}$/', '\\1', $document->RI)); |
253 | - } catch (midcom_error $e) { |
|
258 | + } |
|
259 | + catch (midcom_error $e) { |
|
254 | 260 | // Skip document, the object is hidden, deleted or otherwise unavailable. |
255 | 261 | //@todo Maybe nonexistent objects should be removed from index? |
256 | 262 | continue; |