@@ -26,7 +26,8 @@ |
||
| 26 | 26 | private Throwable $error; |
| 27 | 27 | |
| 28 | 28 | public function __construct(private array $error_actions, private midcom_helper_style $style) |
| 29 | - {} |
|
| 29 | + { |
|
| 30 | +} |
|
| 30 | 31 | |
| 31 | 32 | public static function getSubscribedEvents() |
| 32 | 33 | { |
@@ -52,7 +52,8 @@ |
||
| 52 | 52 | try { |
| 53 | 53 | \midgard_object_class::get_object_by_guid($param->parentguid); |
| 54 | 54 | $seen[$param->parentguid] = true; |
| 55 | - } catch (\Throwable) { |
|
| 55 | + } |
|
| 56 | + catch (\Throwable) { |
|
| 56 | 57 | $seen[$param->parentguid] = false; |
| 57 | 58 | } |
| 58 | 59 | } |
@@ -62,7 +62,8 @@ |
||
| 62 | 62 | try { |
| 63 | 63 | \midgard_object_class::get_object_by_guid($priv->objectguid); |
| 64 | 64 | $seen_parents[$priv->objectguid] = true; |
| 65 | - } catch (\Throwable) { |
|
| 65 | + } |
|
| 66 | + catch (\Throwable) { |
|
| 66 | 67 | $seen_parents[$priv->objectguid] = false; |
| 67 | 68 | } |
| 68 | 69 | } |
@@ -104,7 +104,8 @@ |
||
| 104 | 104 | $qb->add_constraint("location", "=", $location); |
| 105 | 105 | try { |
| 106 | 106 | $attachments = $qb->execute(); |
| 107 | - } catch (\Exception) { |
|
| 107 | + } |
|
| 108 | + catch (\Exception) { |
|
| 108 | 109 | return null; |
| 109 | 110 | } |
| 110 | 111 | if (empty($attachments)) { |
@@ -159,7 +159,8 @@ discard block |
||
| 159 | 159 | $response = $this->handle($request); |
| 160 | 160 | $response->send(); |
| 161 | 161 | $this->terminate($request, $response); |
| 162 | - } catch (Error $e) { |
|
| 162 | + } |
|
| 163 | + catch (Error $e) { |
|
| 163 | 164 | $this->getHttpKernel()->terminateWithException($e); |
| 164 | 165 | } |
| 165 | 166 | } |
@@ -210,9 +211,11 @@ discard block |
||
| 210 | 211 | try { |
| 211 | 212 | $response = $this->handle($request, HttpKernelInterface::SUB_REQUEST, false); |
| 212 | 213 | echo $response->getContent(); |
| 213 | - } catch (midcom_error_notfound | midcom_error_forbidden $e) { |
|
| 214 | + } |
|
| 215 | + catch (midcom_error_notfound | midcom_error_forbidden $e) { |
|
| 214 | 216 | $e->log(); |
| 215 | - } finally { |
|
| 217 | + } |
|
| 218 | + finally { |
|
| 216 | 219 | $this->skip_page_style = $backup; |
| 217 | 220 | midcom_core_context::leave(); |
| 218 | 221 | } |
@@ -65,7 +65,8 @@ |
||
| 65 | 65 | $backend = $this->load_backend($object); |
| 66 | 66 | try { |
| 67 | 67 | $backend->update(midcom::get()->auth->user->id ?? 'NOBODY', $object->get_rcs_message()); |
| 68 | - } catch (midcom_error $e) { |
|
| 68 | + } |
|
| 69 | + catch (midcom_error $e) { |
|
| 69 | 70 | debug_add('RCS: Could not save file!'); |
| 70 | 71 | $e->log(); |
| 71 | 72 | } |
@@ -267,7 +267,8 @@ |
||
| 267 | 267 | if (!empty($this->$property)) { |
| 268 | 268 | try { |
| 269 | 269 | return $class::get_cached($this->$property); |
| 270 | - } catch (midcom_error $e) { |
|
| 270 | + } |
|
| 271 | + catch (midcom_error $e) { |
|
| 271 | 272 | $e->log(); |
| 272 | 273 | } |
| 273 | 274 | } |
@@ -132,7 +132,8 @@ discard block |
||
| 132 | 132 | try { |
| 133 | 133 | $pdf_helper->render_and_attach(); |
| 134 | 134 | return $this->reply(true, $this->_l10n->get('pdf created')); |
| 135 | - } catch (midcom_error $e) { |
|
| 135 | + } |
|
| 136 | + catch (midcom_error $e) { |
|
| 136 | 137 | return $this->reply(false, $this->_l10n->get('pdf creation failed') . ': ' . $e->getMessage()); |
| 137 | 138 | } |
| 138 | 139 | } |
@@ -143,7 +144,8 @@ discard block |
||
| 143 | 144 | try { |
| 144 | 145 | $pdf_helper->render_and_attach('reminder'); |
| 145 | 146 | return $this->reply(true, $this->_l10n->get('payment warning pdf created')); |
| 146 | - } catch (midcom_error $e) { |
|
| 147 | + } |
|
| 148 | + catch (midcom_error $e) { |
|
| 147 | 149 | return $this->reply(false, $this->_l10n->get('payment warning pdf creation failed') . ': ' . $e->getMessage()); |
| 148 | 150 | } |
| 149 | 151 | } |