Passed
Pull Request — master (#204)
by
unknown
23:41
created
lib/org/openpsa/reports/viewer.php 1 patch
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -103,7 +103,8 @@
 block discarded – undo
103 103
             try {
104 104
                 midcom_db_topic::get_cached($node_guid);
105 105
                 $available_generators[$component] = midcom::get()->i18n->get_string($component, $component);
106
-            } catch (midcom_error $e) {
106
+            }
107
+            catch (midcom_error $e) {
107 108
                 debug_add("topic for component '{$component}' not found or accessible");
108 109
             }
109 110
         }
Please login to merge, or discard this patch.
lib/net/nemein/wiki/handler/create.php 1 patch
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -76,7 +76,8 @@
 block discarded – undo
76 76
             } else {
77 77
                 try {
78 78
                     $this->initialize_index_article($topic);
79
-                } catch (midcom_error $e) {
79
+                }
80
+                catch (midcom_error $e) {
80 81
                     // Could not create index
81 82
                     $topic->delete();
82 83
                     throw $e;
Please login to merge, or discard this patch.
lib/org/openpsa/sales/handler.php 1 patch
Braces   +4 added lines, -2 removed lines patch added patch discarded remove patch
@@ -61,7 +61,8 @@  discard block
 block discarded – undo
61 61
                     //check if related at_entry exists
62 62
                     try {
63 63
                         $notification_entry = new midcom_services_at_entry_dba($entry);
64
-                    } catch (midcom_error $e) {
64
+                    }
65
+                    catch (midcom_error $e) {
65 66
                         //relatedto links to a non-existing at_entry - so create a new one an link to it
66 67
                         $notification_entry = new midcom_services_at_entry_dba();
67 68
                         $notification_entry->create();
@@ -83,7 +84,8 @@  discard block
 block discarded – undo
83 84
                     $notification_entry = new midcom_services_at_entry_dba($mc_entry->get_subkey($key, 'fromGuid'));
84 85
                     //check if related at_entry exists & delete it
85 86
                     $notification_entry->delete();
86
-                } catch (midcom_error $e) {
87
+                }
88
+                catch (midcom_error $e) {
87 89
                     $e->log();
88 90
                 }
89 91
             }
Please login to merge, or discard this patch.
lib/org/openpsa/httplib/main.php 1 patch
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -121,7 +121,8 @@
 block discarded – undo
121 121
 
122 122
         try {
123 123
             $response = $client->send($request);
124
-        } catch (Exception $e) {
124
+        }
125
+        catch (Exception $e) {
125 126
             $this->error = $e->getMessage();
126 127
             debug_add("Got error '{$this->error}' from HTTP request", MIDCOM_LOG_INFO);
127 128
             return false;
Please login to merge, or discard this patch.
lib/org/openpsa/projects/handler/task/view.php 1 patch
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -115,7 +115,8 @@
 block discarded – undo
115 115
                         MIDCOM_TOOLBAR_GLYPHICON => 'money',
116 116
                     ];
117 117
                 }
118
-            } catch (midcom_error $e) {
118
+            }
119
+            catch (midcom_error $e) {
119 120
             }
120 121
         }
121 122
 
Please login to merge, or discard this patch.
lib/org/openpsa/expenses/handler/index.php 1 patch
Braces   +4 added lines, -2 removed lines patch added patch discarded remove patch
@@ -114,7 +114,8 @@  discard block
 block discarded – undo
114 114
         foreach ($hours as $guid => $row) {
115 115
             try {
116 116
                 $task = org_openpsa_projects_task_dba::get_cached($row['task']);
117
-            } catch (midcom_error $e) {
117
+            }
118
+            catch (midcom_error $e) {
118 119
                 // Task couldn't be loaded, probably because of ACL
119 120
                 continue;
120 121
             }
@@ -127,7 +128,8 @@  discard block
 block discarded – undo
127 128
                     $person_object = org_openpsa_contacts_person_dba::get_cached($row['person']);
128 129
                     $person_label = $this->_get_list_link($person_object->name, null, null, $row['person']);
129 130
                     $person_name = $person_object->name;
130
-                } catch (midcom_error $e) {
131
+                }
132
+                catch (midcom_error $e) {
131 133
                     $person_label = $this->_l10n->get('no person');
132 134
                     $person_name = '';
133 135
                 }
Please login to merge, or discard this patch.
lib/midgard/admin/asgard/handler/object/manage.php 1 patch
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -61,7 +61,8 @@
 block discarded – undo
61 61
     {
62 62
         try {
63 63
             $this->_object = midcom::get()->dbfactory->get_object_by_guid($guid);
64
-        } catch (midcom_error $e) {
64
+        }
65
+        catch (midcom_error $e) {
65 66
             if (midcom_connection::get_error() == MGD_ERR_OBJECT_DELETED) {
66 67
                 $relocate = $this->router->generate('object_deleted', ['guid' => $guid]);
67 68
                 midcom::get()->relocate($relocate);
Please login to merge, or discard this patch.
lib/org/openpsa/projects/handler/task/list.php 1 patch
Braces   +4 added lines, -2 removed lines patch added patch discarded remove patch
@@ -305,7 +305,8 @@  discard block
 block discarded – undo
305 305
             $url = $this->router->generate('project', ['guid' => $project->guid]);
306 306
             $ret['project'] = '<a href="' . $url . '">' . $project->title . '</a>';
307 307
             $ret['index_project'] = $project->title;
308
-        } catch (midcom_error $e) {
308
+        }
309
+        catch (midcom_error $e) {
309 310
             $e->log();
310 311
         }
311 312
 
@@ -315,7 +316,8 @@  discard block
 block discarded – undo
315 316
                 $customer_url = "{$this->_request_data['contacts_url']}group/{$customer->guid}/";
316 317
                 $ret['customer'] = "<a href='{$customer_url}' title='{$customer->official}'>{$customer->get_label()}</a>";
317 318
                 $ret['index_customer'] = $customer->name;
318
-            } catch (midcom_error $e) {
319
+            }
320
+            catch (midcom_error $e) {
319 321
                 $ret['customer'] = '';
320 322
                 $ret['index_customer'] = '';
321 323
             }
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
@@ -61,7 +61,8 @@  discard block
 block discarded – undo
61 61
         try {
62 62
             $deliverable = org_openpsa_sales_salesproject_deliverable_dba::get_cached($at_entry->arguments['deliverable']);
63 63
             $salesproject = org_openpsa_sales_salesproject_dba::get_cached($deliverable->salesproject);
64
-        } catch (midcom_error $e) {
64
+        }
65
+        catch (midcom_error $e) {
65 66
             $e->log();
66 67
             return $invoice;
67 68
         }
@@ -103,7 +104,8 @@  discard block
 block discarded – undo
103 104
                     $invoice[$fieldname] = '<a href="' . $sales_url . 'list/customer/' . $object->guid . '/">' . $invoice[$fieldname] . '</a>';
104 105
                 }
105 106
                 $invoice['index_' . $fieldname] = $object->get_label();
106
-            } catch (midcom_error $e) {
107
+            }
108
+            catch (midcom_error $e) {
107 109
                 $e->log();
108 110
             }
109 111
         }
Please login to merge, or discard this patch.