Passed
Push — master ( f35cc9...e819cb )
by Andreas
18:06
created
lib/org/openpsa/invoices/handler/invoice/crud.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -141,8 +141,8 @@
 block discarded – undo
141 141
         $schema->get_field('paid')['hidden'] = empty($this->invoice->paid);
142 142
 
143 143
         $contact = $this->invoice->customerContact ?: $this->contact_id;
144
-        $customer_field =& $schema->get_field('customer');
145
-        $contact_field =& $schema->get_field('customerContact');
144
+        $customer_field = & $schema->get_field('customer');
145
+        $contact_field = & $schema->get_field('customerContact');
146 146
 
147 147
         if (!empty($contact)) {
148 148
             $customer_field['type_config']['options'] = $this->get_customers_for_contact($contact);
Please login to merge, or discard this patch.
lib/org/openpsa/reports/handler/projects/report.php 1 patch
Spacing   +4 added lines, -4 removed lines patch added patch discarded remove patch
@@ -47,12 +47,12 @@  discard block
 block discarded – undo
47 47
         $qb_hr = org_openpsa_expenses_hour_report_dba::new_query_builder();
48 48
         $qb_hr->add_constraint('date', '<=', (int) $query_data['end']);
49 49
         $qb_hr->add_constraint('date', '>=', (int) $query_data['start']);
50
-        if (   array_key_exists('invoiceable_filter', $query_data)
50
+        if (array_key_exists('invoiceable_filter', $query_data)
51 51
             && $query_data['invoiceable_filter'] != -1) {
52 52
             $qb_hr->add_constraint('invoiceable', '=', (bool) $query_data['invoiceable_filter']);
53 53
         }
54 54
 
55
-        if (   array_key_exists('invoiced_filter', $query_data)
55
+        if (array_key_exists('invoiced_filter', $query_data)
56 56
             && $query_data['invoiced_filter'] != -1) {
57 57
             if ((int) $query_data['invoiced_filter']) {
58 58
                 debug_add('invoiced_filter parsed as ONLY, adding constraint');
@@ -71,7 +71,7 @@  discard block
 block discarded – undo
71 71
             $tasks = $this->_expand_task($query_data['task']);
72 72
             $qb_hr->add_constraint('task', 'IN', $tasks);
73 73
         }
74
-        if (   array_key_exists('hour_type_filter', $query_data)
74
+        if (array_key_exists('hour_type_filter', $query_data)
75 75
             && $query_data['hour_type_filter'] != 'builtin:all') {
76 76
             $qb_hr->add_constraint('reportType', '=', $query_data['hour_type_filter']);
77 77
         }
@@ -140,7 +140,7 @@  discard block
 block discarded – undo
140 140
 
141 141
     private function add_to_group(array $new_row, string $matching, string $sort, string $title)
142 142
     {
143
-        $rows =& $this->_request_data['report']['rows'];
143
+        $rows = & $this->_request_data['report']['rows'];
144 144
         if (array_key_exists($matching, $rows)) {
145 145
             $rows[$matching]['rows'][] = $new_row;
146 146
             $rows[$matching]['total_hours'] += $new_row['hour']->hours;
Please login to merge, or discard this patch.
lib/org/openpsa/relatedto/handler/journal/entry.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -33,7 +33,7 @@  discard block
 block discarded – undo
33 33
     public function _handler_create(Request $request, string $guid)
34 34
     {
35 35
         $current_object = midcom::get()->dbfactory->get_object_by_guid($guid);
36
-        $entry= new org_openpsa_relatedto_journal_entry_dba();
36
+        $entry = new org_openpsa_relatedto_journal_entry_dba();
37 37
         $entry->linkGuid = $current_object->guid;
38 38
 
39 39
         midcom::get()->head->set_pagetitle($this->_l10n->get('add journal entry'));
@@ -52,7 +52,7 @@  discard block
 block discarded – undo
52 52
         $workflow = $this->get_workflow('datamanager', ['controller' => $data['controller']]);
53 53
         if ($entry->can_do('midgard:delete')) {
54 54
             $delete = $this->get_workflow('delete', ['object' => $entry]);
55
-            $url = $this->router->generate('journal_entry_delete', ['guid' => $guid ]);
55
+            $url = $this->router->generate('journal_entry_delete', ['guid' => $guid]);
56 56
             $workflow->add_dialog_button($delete, $url);
57 57
         }
58 58
         return $workflow->run($request);
Please login to merge, or discard this patch.
lib/org/openpsa/user/handler/person/view.php 1 patch
Spacing   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -24,7 +24,7 @@  discard block
 block discarded – undo
24 24
         $this->add_breadcrumb('', $person->get_label());
25 25
 
26 26
         $auth = midcom::get()->auth;
27
-        if (   $person->id == midcom_connection::get_user()
27
+        if ($person->id == midcom_connection::get_user()
28 28
             || $auth->can_user_do('org.openpsa.user:manage', null, org_openpsa_user_interface::class)) {
29 29
             $buttons = [];
30 30
             $workflow = $this->get_workflow('datamanager');
@@ -37,7 +37,7 @@  discard block
 block discarded – undo
37 37
                 $delete_workflow = $this->get_workflow('delete', ['object' => $person]);
38 38
                 $buttons[] = $delete_workflow->get_button($this->router->generate('user_delete', ['guid' => $person->guid]));
39 39
             }
40
-            if (   midcom_connection::is_user($person)
40
+            if (midcom_connection::is_user($person)
41 41
                 && $person->can_do('midgard:privileges')) {
42 42
                 $buttons[] = $workflow->get_button($this->router->generate('user_privileges', ['guid' => $person->guid]), [
43 43
                     MIDCOM_TOOLBAR_LABEL => $this->_l10n->get("permissions"),
Please login to merge, or discard this patch.