PhpOffice\PhpWord\Element\AbstractContainer error suppression

[daum]

In AbstractContainer.php on line 101 right now there is the following:

@list($count, $fontStyle, $paragraphStyle) = $args; // Suppress error

The issue is if a user has listeners for warnings/errors that they will still get the error which then can cause their scripts to send extra alert emails. Switching this to the following would have the same affect, just never produce the warnings to the listeners:

if (isset($args[0])) {
    $count = $args[0];
}
else {
    $count = null;
}
// Same for rest of variables

I'll submit a PR with the full fix. While this does have a few more lines of code, it will allow error monitoring to continue without false positives.

---

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

[gmta]

The issue is if a user has listeners for warnings/errors that they will still get the error which then can cause their scripts to send extra alert emails.

Yeah, if you use set_error_handler() then you should also read the remark about it bypassing error_reporting() completely. This is not PHPWord at fault with error suppression, this is your custom error handling ignoring the error level.

Fix your code:

if (!(error_reporting() & $errno)) {
    return;
}

[github-actions]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.
If this is still an issue for you, please try to help by debugging it further and sharing your results.
Thank you for your contributions.