@@ -49,7 +49,7 @@ |
||
49 | 49 | |
50 | 50 | private function get_units($units = false) : float |
51 | 51 | { |
52 | - if ( $this->deliverable->invoiceByActualUnits |
|
52 | + if ($this->deliverable->invoiceByActualUnits |
|
53 | 53 | || $this->deliverable->plannedUnits == 0) { |
54 | 54 | // In most cases we calculate the price based on the actual units entered |
55 | 55 | return $units ?: $this->deliverable->units; |
@@ -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 | } |
@@ -27,7 +27,7 @@ discard block |
||
27 | 27 | $prefix = $nap->get_node($this->_topic->id)[MIDCOM_NAV_ABSOLUTEURL]; |
28 | 28 | |
29 | 29 | $tree = new org_openpsa_widgets_tree(org_openpsa_contacts_group_dba::class, 'owner'); |
30 | - $tree->link_callback = function ($guid) use ($prefix) { |
|
30 | + $tree->link_callback = function($guid) use ($prefix) { |
|
31 | 31 | return $prefix . 'group/' . $guid . '/'; |
32 | 32 | }; |
33 | 33 | $tree->constraints[] = ['orgOpenpsaObtype', '<', org_openpsa_contacts_group_dba::ORGANIZATION]; |
@@ -43,7 +43,7 @@ discard block |
||
43 | 43 | { |
44 | 44 | $my_company_guid = $this->_config->get('owner_organization'); |
45 | 45 | |
46 | - if ( empty($my_company_guid) |
|
46 | + if (empty($my_company_guid) |
|
47 | 47 | || !mgd_is_guid($my_company_guid)) { |
48 | 48 | if (midcom::get()->auth->admin) { |
49 | 49 | midcom::get()->uimessages->add( |
@@ -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 | } |