@@ -98,7 +98,8 @@ |
||
| 98 | 98 | if ($this->old_task) { |
| 99 | 99 | self::update_cache(new org_openpsa_projects_task_dba($this->old_task)); |
| 100 | 100 | } |
| 101 | - } catch (midcom_error $e) { |
|
| 101 | + } |
|
| 102 | + catch (midcom_error $e) { |
|
| 102 | 103 | $e->log(); |
| 103 | 104 | } |
| 104 | 105 | |
@@ -48,7 +48,8 @@ |
||
| 48 | 48 | foreach (array_unique($tasks_to_update) as $id) { |
| 49 | 49 | try { |
| 50 | 50 | org_openpsa_expenses_hour_report_dba::update_cache(new org_openpsa_projects_task_dba($id)); |
| 51 | - } catch (midcom_error $e) { |
|
| 51 | + } |
|
| 52 | + catch (midcom_error $e) { |
|
| 52 | 53 | $e->log(); |
| 53 | 54 | } |
| 54 | 55 | } |
@@ -31,7 +31,8 @@ discard block |
||
| 31 | 31 | try { |
| 32 | 32 | $agreement = new org_openpsa_sales_salesproject_deliverable_dba($agreement); |
| 33 | 33 | $agreement->update_units(); |
| 34 | - } catch (midcom_error $e) { |
|
| 34 | + } |
|
| 35 | + catch (midcom_error $e) { |
|
| 35 | 36 | $e->log(); |
| 36 | 37 | } |
| 37 | 38 | } |
@@ -49,7 +50,8 @@ discard block |
||
| 49 | 50 | |
| 50 | 51 | try { |
| 51 | 52 | $deliverable = new org_openpsa_sales_salesproject_deliverable_dba($args['deliverable']); |
| 52 | - } catch (midcom_error $e) { |
|
| 53 | + } |
|
| 54 | + catch (midcom_error $e) { |
|
| 53 | 55 | $handler->print_error("Deliverable {$args['deliverable']} not found: " . midcom_connection::get_error_string()); |
| 54 | 56 | return false; |
| 55 | 57 | } |
@@ -69,7 +71,8 @@ discard block |
||
| 69 | 71 | } |
| 70 | 72 | try { |
| 71 | 73 | $deliverable = new org_openpsa_sales_salesproject_deliverable_dba($args['deliverable']); |
| 72 | - } catch (midcom_error $e) { |
|
| 74 | + } |
|
| 75 | + catch (midcom_error $e) { |
|
| 73 | 76 | $handler->print_error('no deliverable with passed GUID: ' . $args['deliverable'] . ', aborting'); |
| 74 | 77 | return false; |
| 75 | 78 | } |
@@ -77,7 +80,8 @@ discard block |
||
| 77 | 80 | //get the owner of the salesproject the deliverable belongs to |
| 78 | 81 | try { |
| 79 | 82 | $project = new org_openpsa_sales_salesproject_dba($deliverable->salesproject); |
| 80 | - } catch (midcom_error $e) { |
|
| 83 | + } |
|
| 84 | + catch (midcom_error $e) { |
|
| 81 | 85 | $handler->print_error('Failed to load salesproject: ' . $e->getMessage()); |
| 82 | 86 | return false; |
| 83 | 87 | } |
@@ -8,7 +8,8 @@ |
||
| 8 | 8 | if ($customer->orgOpenpsaObtype < org_openpsa_contacts_group_dba::ORGANIZATION) { |
| 9 | 9 | $customer = false; |
| 10 | 10 | } |
| 11 | -} catch (midcom_error $e) { |
|
| 11 | +} |
|
| 12 | +catch (midcom_error $e) { |
|
| 12 | 13 | $customer = false; |
| 13 | 14 | } |
| 14 | 15 | |
@@ -96,7 +96,8 @@ discard block |
||
| 96 | 96 | $customer = org_openpsa_contacts_group_dba::get_cached($invoice->customer); |
| 97 | 97 | $entry['customer'] = "<a href=\"{$this->router->generate('list_customer_all', ['guid' => $customer->guid])}\">" . $customer->get_label() . "</a>"; |
| 98 | 98 | $entry['index_customer'] = $customer->get_label(); |
| 99 | - } catch (midcom_error $e) { |
|
| 99 | + } |
|
| 100 | + catch (midcom_error $e) { |
|
| 100 | 101 | $entry['customer'] = ''; |
| 101 | 102 | $entry['index_customer'] = ''; |
| 102 | 103 | } |
@@ -110,7 +111,8 @@ discard block |
||
| 110 | 111 | $entry['contact'] = "<a href=\"{$this->router->generate('list_customer_all', ['guid' => $contact->guid])}\">" . $contact->get_label() . "</a>"; |
| 111 | 112 | $entry['contact'] = $contact_widget->show_inline(); |
| 112 | 113 | $entry['index_contact'] = $contact->get_label(); |
| 113 | - } catch (midcom_error $e) { |
|
| 114 | + } |
|
| 115 | + catch (midcom_error $e) { |
|
| 114 | 116 | $entry['contact'] = ''; |
| 115 | 117 | $entry['index_contact'] = ''; |
| 116 | 118 | } |
@@ -257,7 +259,8 @@ discard block |
||
| 257 | 259 | { |
| 258 | 260 | try { |
| 259 | 261 | $this->_customer = new org_openpsa_contacts_group_dba($guid); |
| 260 | - } catch (midcom_error $e) { |
|
| 262 | + } |
|
| 263 | + catch (midcom_error $e) { |
|
| 261 | 264 | $this->_customer = new org_openpsa_contacts_person_dba($guid); |
| 262 | 265 | } |
| 263 | 266 | $data['customer'] = $this->_customer; |
@@ -31,7 +31,8 @@ discard block |
||
| 31 | 31 | if ($data['contacts_url']) { |
| 32 | 32 | $row['customer'] = "<a href=\"{$data['contacts_url']}group/{$customer->guid}/\">{$label}</a>"; |
| 33 | 33 | } |
| 34 | - } catch (midcom_error $e) { |
|
| 34 | + } |
|
| 35 | + catch (midcom_error $e) { |
|
| 35 | 36 | $e->log(); |
| 36 | 37 | } |
| 37 | 38 | } |
@@ -44,7 +45,8 @@ discard block |
||
| 44 | 45 | $contact_widget = org_openpsa_widgets_contact::get($salesproject->customerContact); |
| 45 | 46 | $row['index_customerContact'] = $customer->get_label(); |
| 46 | 47 | $row['customerContact'] = $contact_widget->show_inline(); |
| 47 | - } catch (midcom_error $e) { |
|
| 48 | + } |
|
| 49 | + catch (midcom_error $e) { |
|
| 48 | 50 | $e->log(); |
| 49 | 51 | } |
| 50 | 52 | } |
@@ -56,7 +58,8 @@ discard block |
||
| 56 | 58 | $owner_widget = org_openpsa_widgets_contact::get($salesproject->owner); |
| 57 | 59 | $row['index_owner'] = $owner->rname; |
| 58 | 60 | $row['owner'] = $owner_widget->show_inline(); |
| 59 | - } catch (midcom_error $e) { |
|
| 61 | + } |
|
| 62 | + catch (midcom_error $e) { |
|
| 60 | 63 | $row['index_owner'] = ''; |
| 61 | 64 | $row['owner'] = ''; |
| 62 | 65 | } |
@@ -102,7 +102,8 @@ discard block |
||
| 102 | 102 | foreach ($hours as $guid => $row) { |
| 103 | 103 | try { |
| 104 | 104 | $task = org_openpsa_projects_task_dba::get_cached($row['task']); |
| 105 | - } catch (midcom_error $e) { |
|
| 105 | + } |
|
| 106 | + catch (midcom_error $e) { |
|
| 106 | 107 | // Task couldn't be loaded, probably because of ACL |
| 107 | 108 | continue; |
| 108 | 109 | } |
@@ -115,7 +116,8 @@ discard block |
||
| 115 | 116 | $person_object = midcom_db_person::get_cached($row['person']); |
| 116 | 117 | $person_label = $this->_get_list_link($person_object->name, null, null, $row['person']); |
| 117 | 118 | $person_name = $person_object->name; |
| 118 | - } catch (midcom_error $e) { |
|
| 119 | + } |
|
| 120 | + catch (midcom_error $e) { |
|
| 119 | 121 | $person_label = $this->_l10n->get('no person'); |
| 120 | 122 | $person_name = ''; |
| 121 | 123 | } |
@@ -237,7 +237,8 @@ |
||
| 237 | 237 | { |
| 238 | 238 | try { |
| 239 | 239 | return $this->get_l10n($component)->get($stringid); |
| 240 | - } catch (midcom_error $e) { |
|
| 240 | + } |
|
| 241 | + catch (midcom_error $e) { |
|
| 241 | 242 | $e->log(MIDCOM_LOG_WARN); |
| 242 | 243 | return $stringid; |
| 243 | 244 | } |
@@ -69,7 +69,8 @@ discard block |
||
| 69 | 69 | |
| 70 | 70 | try { |
| 71 | 71 | $result = $router->match($url); |
| 72 | - } catch (ResourceNotFoundException $e) { |
|
| 72 | + } |
|
| 73 | + catch (ResourceNotFoundException $e) { |
|
| 73 | 74 | throw new midcom_error_notfound('This URL method is unknown.'); |
| 74 | 75 | } |
| 75 | 76 | $this->request->attributes->add($result); |
@@ -151,11 +152,13 @@ discard block |
||
| 151 | 152 | |
| 152 | 153 | try { |
| 153 | 154 | $result = $router->match($url); |
| 154 | - } catch (ResourceNotFoundException $e) { |
|
| 155 | + } |
|
| 156 | + catch (ResourceNotFoundException $e) { |
|
| 155 | 157 | // No match |
| 156 | 158 | debug_add("Component {$viewer->_component} in {$viewer->_topic->name} declared unable to handle request.", MIDCOM_LOG_INFO); |
| 157 | 159 | throw new midcom_error_notfound("This page is not available on this server."); |
| 158 | - } catch (MethodNotAllowedException $e) { |
|
| 160 | + } |
|
| 161 | + catch (MethodNotAllowedException $e) { |
|
| 159 | 162 | debug_add("Component {$viewer->_component} in {$viewer->_topic->name} declared unable to handle request (method not allowed).", MIDCOM_LOG_INFO); |
| 160 | 163 | throw new midcom_error_forbidden($e->getMessage(), Response::HTTP_METHOD_NOT_ALLOWED); |
| 161 | 164 | } |