[#16912] - Regression, exception handling broken in J3.7.3 RC2
- Closed
- 30 Jun 2017
- Medium
- Build: staging
- # 16912
Steps to reproduce the issue
Changes made by that PR #13257 , do not check the exact type of the "callable"
it just blindly assumed that it is a variable method name
It breaks exception handling in J3.7.3 RC2 (changes in redirect plugin)
and also other changes may cause failures in the future if a other callable cases are used
Throw an exception in backend,
e.g. a PHP syntax error
and see that redirect plugin fails to call the previous exception handler
Expected result
Exceptions can be handled in backend
Actual result
Exception handling does not work in backend
System information (as much as possible)
J3.7.3 RC2
Additional comments
This is a release blocker, we no longer having exception handling usable in backend
| Labels |
Added:
?
|
||
| Title |
|
||||||
@rdeutz @frankmayer can you take a look here?
On it.
| Status | New | ⇒ | Information Required |
| Category | ⇒ | Administration |
| Status | Information Required | ⇒ | Closed |
| Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2017-06-30 04:28:52 |
| Closed_By | ⇒ | franz-wohlkoenig |
| Closed_By | franz-wohlkoenig | ⇒ | joomla-cms-bot |
Set to "closed" on behalf of @franz-wohlkoenig by The JTracker Application at issues.joomla.org/joomla-cms/16912
closed as havin PR #16913
This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/16912.
Printing the previous exception handler, used here:
https://github.com/joomla/joomla-cms/blob/staging/plugins/system/redirect/redirect.php#L112
print_r(self::$previousExceptionHandler);i get
Array ( [0] => JErrorPage [1] => render )so it can not be called like:
self::$previousExceptionHandler($error);