? Failure

User tests: Successful: Unsuccessful:

avatar mbabker
mbabker
22 Sep 2016

Summary of Changes

  • Remove conditionals for returning JError::raise* response or throwing exceptions in favor of throwing exceptions for many library functions
  • Remove returning JError::raise* response in favor of throwing exceptions in other library functions
  • Remove silent discarding of errors when retrieving the menu, pathway, and router objects via the application classes
  • Have Redis cache throw exceptions as noted in code

Testing Instructions

Review

Documentation Changes Required

Error handling expectations are changed from handling legacy JError behaviors to catching exceptions in noted places.

avatar mbabker mbabker - open - 22 Sep 2016
avatar mbabker mbabker - change - 22 Sep 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 22 Sep 2016
Category Libraries
avatar joomla-cms-bot joomla-cms-bot - change - 22 Sep 2016
Labels Added: ?
avatar zero-24 zero-24 - change - 22 Sep 2016
Labels Added: ?
avatar wilsonge wilsonge - change - 23 Sep 2016
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-09-23 08:40:13
Closed_By wilsonge
avatar wilsonge wilsonge - close - 23 Sep 2016
avatar wilsonge wilsonge - merge - 23 Sep 2016

Add a Comment

Login with GitHub to post a comment