Passed
Push — master ( dc3f28...9cb9a1 )
by Andreas
16:51
created
lib/midcom/application.php 1 patch
Braces   +6 added lines, -3 removed lines patch added patch discarded remove patch
@@ -161,7 +161,8 @@  discard block
 block discarded – undo
161 161
             $response = $this->handle($request);
162 162
             $response->send();
163 163
             $this->terminate($request, $response);
164
-        } catch (Error $e) {
164
+        }
165
+        catch (Error $e) {
165 166
             $this->getHttpKernel()->terminateWithException($e);
166 167
         }
167 168
     }
@@ -218,11 +219,13 @@  discard block
 block discarded – undo
218 219
         $this->skip_page_style = true;
219 220
         try {
220 221
             $response = $this->handle($request, HttpKernelInterface::SUB_REQUEST, false);
221
-        } catch (midcom_error_notfound | midcom_error_forbidden $e) {
222
+        }
223
+        catch (midcom_error_notfound | midcom_error_forbidden $e) {
222 224
             $e->log();
223 225
             midcom_core_context::leave();
224 226
             return;
225
-        } finally {
227
+        }
228
+        finally {
226 229
             $this->skip_page_style = $backup;
227 230
         }
228 231
 
Please login to merge, or discard this patch.
src/midcom/console/command/cacheinvalidate.php 1 patch
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -44,7 +44,8 @@
 block discarded – undo
44 44
     {
45 45
         try {
46 46
             $this->cache->invalidate_all();
47
-        } catch (\Throwable $e) {
47
+        }
48
+        catch (\Throwable $e) {
48 49
             $output->writeln($e->getMessage());
49 50
         }
50 51
 
Please login to merge, or discard this patch.
lib/midgard/admin/asgard/handler/object/rcs.php 1 patch
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -27,7 +27,8 @@
 block discarded – undo
27 27
     {
28 28
         try {
29 29
             return parent::load_object($guid);
30
-        } catch (midcom_error_midgard $e) {
30
+        }
31
+        catch (midcom_error_midgard $e) {
31 32
             $mgd_exception = $e->getPrevious();
32 33
             if (   $mgd_exception
33 34
                 && $mgd_exception->getCode() == mgd_exception::OBJECT_DELETED) {
Please login to merge, or discard this patch.
lib/midcom/services/cache/module/nap.php 1 patch
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -107,7 +107,8 @@
 block discarded – undo
107 107
                 && $nodeobject = $nav->get_node($node->id)) {
108 108
                 return $nav->get_leaf($nodeobject[MIDCOM_NAV_ID] . '-' . $object->id);
109 109
             }
110
-        } catch (midcom_error $e) {
110
+        }
111
+        catch (midcom_error $e) {
111 112
             $e->log();
112 113
         }
113 114
         return null;
Please login to merge, or discard this patch.
lib/midcom/baseclasses/components/handler/rest.php 1 patch
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -181,7 +181,8 @@
 block discarded – undo
181 181
             if ($this->_response === null) {
182 182
                 $this->_stop('Could not handle request, unknown method', Response::HTTP_METHOD_NOT_ALLOWED);
183 183
             }
184
-        } catch (midcom_error $e) {
184
+        }
185
+        catch (midcom_error $e) {
185 186
             $this->_responseStatus = $e->getCode();
186 187
             return $this->_send_response($e->getMessage());
187 188
         }
Please login to merge, or discard this patch.
lib/org/openpsa/invoices/handler/invoice/action.php 1 patch
Braces   +4 added lines, -2 removed lines patch added patch discarded remove patch
@@ -125,7 +125,8 @@  discard block
 block discarded – undo
125 125
         try {
126 126
             $pdf_helper->render_and_attach();
127 127
             return $this->reply(true, $this->_l10n->get('pdf created'));
128
-        } catch (midcom_error $e) {
128
+        }
129
+        catch (midcom_error $e) {
129 130
             return $this->reply(false, $this->_l10n->get('pdf creation failed') . ': ' . $e->getMessage());
130 131
         }
131 132
     }
@@ -136,7 +137,8 @@  discard block
 block discarded – undo
136 137
         try {
137 138
             $pdf_helper->render_and_attach('reminder');
138 139
             return $this->reply(true, $this->_l10n->get('reminder pdf created'));
139
-        } catch (midcom_error $e) {
140
+        }
141
+        catch (midcom_error $e) {
140 142
             return $this->reply(false, $this->_l10n->get('reminder pdf creation failed') . ': ' . $e->getMessage());
141 143
         }
142 144
     }    
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
@@ -58,7 +58,8 @@
 block discarded – undo
58 58
         // TODO: Should we sudo here to ensure getting correct prefs regardless of ACLs?
59 59
         try {
60 60
             $recipient = midcom_db_person::get_cached($recipient);
61
-        } catch (midcom_error) {
61
+        }
62
+        catch (midcom_error) {
62 63
             return false;
63 64
         }
64 65
 
Please login to merge, or discard this patch.
lib/org/openpsa/documents/elFinderVolumeOpenpsa.php 1 patch
Braces   +10 added lines, -5 removed lines patch added patch discarded remove patch
@@ -83,7 +83,8 @@  discard block
 block discarded – undo
83 83
 
84 84
         $this->clearcache();
85 85
 
86
-        if ($file && $file['name'] === $name) { // file exists and check filename for item ID based filesystem
86
+        if ($file && $file['name'] === $name) {
87
+// file exists and check filename for item ID based filesystem
87 88
             if ($this->uploadOverwrite) {
88 89
                 if (!$file['write']) {
89 90
                     return $this->setError(elFinder::ERROR_PERM_DENIED);
@@ -139,11 +140,13 @@  discard block
 block discarded – undo
139 140
     {
140 141
         try {
141 142
             return org_openpsa_documents_document_dba::get_cached($path);
142
-        } catch (midcom_error $e) {
143
+        }
144
+        catch (midcom_error $e) {
143 145
             $e->log();
144 146
             try {
145 147
                 return org_openpsa_documents_directory::get_cached($path);
146
-            } catch (midcom_error $e) {
148
+            }
149
+            catch (midcom_error $e) {
147 150
                 $e->log();
148 151
             }
149 152
         }
@@ -275,7 +278,8 @@  discard block
 block discarded – undo
275 278
         $object = midcom::get()->dbfactory->get_object_by_guid($path);
276 279
         try {
277 280
             $parentdir = org_openpsa_documents_directory::get_cached($parent);
278
-        } catch (midcom_error $e) {
281
+        }
282
+        catch (midcom_error $e) {
279 283
             $e->log();
280 284
             return false;
281 285
         }
@@ -567,7 +571,8 @@  discard block
 block discarded – undo
567 571
         try {
568 572
             $doc = new org_openpsa_documents_document_dba($path);
569 573
             return $doc->delete();
570
-        } catch (midcom_error) {
574
+        }
575
+        catch (midcom_error) {
571 576
             return false;
572 577
         }
573 578
     }
Please login to merge, or discard this patch.
lib/org/openpsa/mypage/style/weekreview-day-item-task-status.php 1 patch
Braces   +2 added lines, -1 removed lines patch added patch discarded remove patch
@@ -33,7 +33,8 @@
 block discarded – undo
33 33
                 $task_label = "<a href=\"{$data['projects_url']}task/{$task->guid}/\">{$task_label}</a>";
34 34
             }
35 35
             echo $task_label;
36
-        } catch (midcom_error) {}
36
+        }
37
+        catch (midcom_error) {}
37 38
         ?>
38 39
     </td>
39 40
     <td class="multivalue">
Please login to merge, or discard this patch.