@@ -356,7 +356,8 @@ |
||
| 356 | 356 | // The agreement allows invoicing only approved hours, therefore don't mark unapproved |
| 357 | 357 | $qb->add_constraint('metadata.isapproved', '=', true); |
| 358 | 358 | } |
| 359 | - } catch (midcom_error $e) { |
|
| 359 | + } |
|
| 360 | + catch (midcom_error $e) { |
|
| 360 | 361 | $e->log(); |
| 361 | 362 | } |
| 362 | 363 | |
@@ -171,7 +171,8 @@ |
||
| 171 | 171 | //create reflector with linked object to get the right label |
| 172 | 172 | try { |
| 173 | 173 | $linked_object = midcom::get()->dbfactory->get_object_by_guid($entry->linkGuid); |
| 174 | - } catch (midcom_error $e) { |
|
| 174 | + } |
|
| 175 | + catch (midcom_error $e) { |
|
| 175 | 176 | unset($data['entries'][$i]); |
| 176 | 177 | $e->log(); |
| 177 | 178 | continue; |
@@ -92,7 +92,8 @@ |
||
| 92 | 92 | $customer = new org_openpsa_contacts_group_dba($customer_id); |
| 93 | 93 | $customer_label = $customer->official; |
| 94 | 94 | $customer = $customer_id; |
| 95 | - } catch (midcom_error $e) { |
|
| 95 | + } |
|
| 96 | + catch (midcom_error $e) { |
|
| 96 | 97 | } |
| 97 | 98 | } |
| 98 | 99 | } |
@@ -148,7 +148,8 @@ |
||
| 148 | 148 | try { |
| 149 | 149 | $this->$function(); |
| 150 | 150 | $this->_response->success = true; |
| 151 | - } catch (midcom_error $e) { |
|
| 151 | + } |
|
| 152 | + catch (midcom_error $e) { |
|
| 152 | 153 | $this->_response->success = false; |
| 153 | 154 | $this->_response->error = $e->getMessage(); |
| 154 | 155 | } |
@@ -52,7 +52,8 @@ |
||
| 52 | 52 | // TODO: Should we sudo here to ensure getting correct prefs regardless of ACLs? |
| 53 | 53 | try { |
| 54 | 54 | $recipient = midcom_db_person::get_cached($recipient); |
| 55 | - } catch (midcom_error $e) { |
|
| 55 | + } |
|
| 56 | + catch (midcom_error $e) { |
|
| 56 | 57 | return false; |
| 57 | 58 | } |
| 58 | 59 | |
@@ -126,7 +126,8 @@ |
||
| 126 | 126 | $salesproject = org_openpsa_sales_salesproject_dba::get_cached($this->_deliverable->salesproject); |
| 127 | 127 | try { |
| 128 | 128 | $owner = midcom_db_person::get_cached($salesproject->owner); |
| 129 | - } catch (midcom_error $e) { |
|
| 129 | + } |
|
| 130 | + catch (midcom_error $e) { |
|
| 130 | 131 | $e->log(); |
| 131 | 132 | return; |
| 132 | 133 | } |
@@ -57,7 +57,8 @@ discard block |
||
| 57 | 57 | try { |
| 58 | 58 | $deliverable = org_openpsa_sales_salesproject_deliverable_dba::get_cached($at_entry->arguments['deliverable']); |
| 59 | 59 | $salesproject = org_openpsa_sales_salesproject_dba::get_cached($deliverable->salesproject); |
| 60 | - } catch (midcom_error $e) { |
|
| 60 | + } |
|
| 61 | + catch (midcom_error $e) { |
|
| 61 | 62 | $e->log(); |
| 62 | 63 | return $invoice; |
| 63 | 64 | } |
@@ -95,7 +96,8 @@ discard block |
||
| 95 | 96 | $object = $classname::get_cached($id); |
| 96 | 97 | $invoice[$fieldname] = $object->render_link(); |
| 97 | 98 | $invoice['index_' . $fieldname] = $object->get_label(); |
| 98 | - } catch (midcom_error $e) { |
|
| 99 | + } |
|
| 100 | + catch (midcom_error $e) { |
|
| 99 | 101 | $e->log(); |
| 100 | 102 | } |
| 101 | 103 | } |
@@ -134,7 +134,8 @@ discard block |
||
| 134 | 134 | |
| 135 | 135 | try { |
| 136 | 136 | $deliverable = new org_openpsa_sales_salesproject_deliverable_dba($args['deliverable']); |
| 137 | - } catch (midcom_error $e) { |
|
| 137 | + } |
|
| 138 | + catch (midcom_error $e) { |
|
| 138 | 139 | $handler->print_error("Deliverable {$args['deliverable']} not found: " . midcom_connection::get_error_string()); |
| 139 | 140 | return false; |
| 140 | 141 | } |
@@ -158,7 +159,8 @@ discard block |
||
| 158 | 159 | } |
| 159 | 160 | try { |
| 160 | 161 | $deliverable = new org_openpsa_sales_salesproject_deliverable_dba($args['deliverable']); |
| 161 | - } catch (midcom_error $e) { |
|
| 162 | + } |
|
| 163 | + catch (midcom_error $e) { |
|
| 162 | 164 | $handler->print_error('no deliverable with passed GUID: ' . $args['deliverable'] . ', aborting'); |
| 163 | 165 | return false; |
| 164 | 166 | } |
@@ -166,7 +168,8 @@ discard block |
||
| 166 | 168 | //get the owner of the salesproject the deliverable belongs to |
| 167 | 169 | try { |
| 168 | 170 | $project = new org_openpsa_sales_salesproject_dba($deliverable->salesproject); |
| 169 | - } catch (midcom_error $e) { |
|
| 171 | + } |
|
| 172 | + catch (midcom_error $e) { |
|
| 170 | 173 | $handler->print_error('Failed to load salesproject: ' . $e->getMessage()); |
| 171 | 174 | return false; |
| 172 | 175 | } |
@@ -51,7 +51,8 @@ |
||
| 51 | 51 | } |
| 52 | 52 | try { |
| 53 | 53 | $this->_request_data['product'] = org_openpsa_products_product_dba::get_cached($this->_deliverable->product); |
| 54 | - } catch (midcom_error $e) { |
|
| 54 | + } |
|
| 55 | + catch (midcom_error $e) { |
|
| 55 | 56 | $this->_request_data['product'] = false; |
| 56 | 57 | } |
| 57 | 58 | } |