Passed
Pull Request — master (#193)
by
unknown
20:53
created
lib/org/openpsa/projects/workflow.php 1 patch
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -356,7 +356,8 @@
 block discarded – undo
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
 
Please login to merge, or discard this patch.
lib/org/openpsa/relatedto/handler/journal/list.php 1 patch
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -171,7 +171,8 @@
 block discarded – undo
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;
Please login to merge, or discard this patch.
lib/org/openpsa/mypage/handler/workingon.php 1 patch
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -92,7 +92,8 @@
 block discarded – undo
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
             }
Please login to merge, or discard this patch.
lib/org/openpsa/slideshow/handler/edit.php 1 patch
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -148,7 +148,8 @@
 block discarded – undo
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
         }
Please login to merge, or discard this patch.
lib/org/openpsa/notifications/main.php 1 patch
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -52,7 +52,8 @@
 block discarded – undo
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
 
Please login to merge, or discard this patch.
lib/org/openpsa/invoices/scheduler.php 1 patch
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -126,7 +126,8 @@
 block discarded – undo
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
         }
Please login to merge, or discard this patch.
lib/org/openpsa/invoices/handler/scheduled.php 1 patch
Braces   +4 added lines, -2 removed lines patch added patch discarded remove patch
@@ -57,7 +57,8 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
         }
Please login to merge, or discard this patch.
lib/org/openpsa/sales/midcom/interfaces.php 1 patch
Braces   +6 added lines, -3 removed lines patch added patch discarded remove patch
@@ -134,7 +134,8 @@  discard block
 block discarded – undo
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
 block discarded – undo
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
 block discarded – undo
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
         }
Please login to merge, or discard this patch.
lib/org/openpsa/sales/handler/deliverable/view.php 1 patch
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -51,7 +51,8 @@
 block discarded – undo
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
     }
Please login to merge, or discard this patch.