? Success

User tests: Successful: Unsuccessful:

avatar zero-24
zero-24
14 Jun 2014

Tracker

http://joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemEdit&tracker_item_id=33865&start=0

Open Point

Here:
https://github.com/zero-24/joomla-cms/blob/patch-9/libraries/joomla/client/helper.php#L222-229

we use

if (class_exists('JError'))
{
$return = JError::raiseWarning('SOME_ERROR_CODE', JText::_('JLIB_CLIENT_ERROR_HELPER_SETCREDENTIALSFROMREQUEST_FAILED'));
}
else
{
throw new InvalidArgumentException('Invalid user credentials');
}

can this replaced by

throw new InvalidArgumentException(JText::_('JLIB_CLIENT_ERROR_HELPER_SETCREDENTIALSFROMREQUEST_FAILED'));

Or should we replace the JERROR Statment with

JFactory::getApplication()->enqueueMessage();
avatar zero-24 zero-24 - open - 14 Jun 2014
avatar zero-24 zero-24 - change - 14 Jun 2014
Title
Use $app->enqueueMessage instead of JError and SOME_ERROR_CODE
[#33865] Use $app->enqueueMessage instead of JError and SOME_ERROR_CODE
avatar zero-24 zero-24 - close - 14 Jul 2014
avatar zero-24 zero-24 - change - 14 Jul 2014
Status New Closed
Closed_Date 0000-00-00 00:00:00 2014-07-14 12:08:46
avatar zero-24 zero-24 - close - 14 Jul 2014
avatar zero-24 zero-24 - head_ref_deleted - 14 Jul 2014

Add a Comment

Login with GitHub to post a comment