matt/spl-tools
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Improve error handling in page_renderer.php

Browse Source
This commit is contained in:
Matt Way
2024-08-11 21:51:43 +10:00
parent 4d9d633676
commit e9a7de83cd
1 changed files with 6 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
php-template/page_renderer.php
View File

@@ -4,9 +4,10 @@ class SPLPageRenderer {
public function renderCouncilPage($config, $candidates) { public function renderCouncilPage($config, $candidates) {
ob_start(); ob_start();
set_error_handler(function($errno, $errstr, $errfile, $errline) { $didError = false;
// Clear any output if an error occurred
ob_clean(); set_error_handler(function($errno, $errstr, $errfile, $errline) use(&$didError) {
$didError = true;
error_log("Error: $errstr in $errfile on line $errline"); error_log("Error: $errstr in $errfile on line $errline");
return true; // Prevent default error handling return true; // Prevent default error handling
}); });
@@ -17,8 +18,8 @@ class SPLPageRenderer {
$content = ob_get_clean(); $content = ob_get_clean();
// Explictly return null if we didn't generate any content // Explictly return null if we didn't generate any content or if there was an error
if (!empty($content)) { if (!empty($content) && !$didError) {
return $content; return $content;
} else { } else {
return null; return null;