? Success

User tests: Successful: Unsuccessful:

avatar itbra
itbra
13 Mar 2014

Fixing issue described in bug #33465, related to JModelAdmin::delete() not reporting something when deletion failed due to empty row data.

avatar itbra itbra - open - 13 Mar 2014
avatar itbra itbra - change - 13 Mar 2014
The description was changed
Title
fix JTable::delete()
[#33465] fix JModelAdmin::delete()
Description <p>The model's delete-function evalutes whether the table has loaded a row or not. If an error occurs while the table loads a row and populates its state, an exception will be thrown, whereas when no error occurs while trying to load and the result is empty, the table simply returns false. The model assumes that when table-&gt;load() is not true, the table can serve an error, but this is not the case. So the user remains unexplained why deletion fails. The fix is to check whether there is an error available from the table and set it within the model or to set a generic error message telling the user that <em>The database row is empty.</em>.</p> <p>Fixing issue described in <a href="http://joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemEdit&amp;tracker_id=8103&amp;tracker_item_id=33465">bug #33465</a>, related to JModelAdmin::delete() not reporting something when deletion failed due to empty row data.</p>
Labels Added: ? ?
avatar brianteeman
brianteeman - comment - 8 Aug 2014

Closed as per the comment on the tracker

avatar brianteeman brianteeman - change - 8 Aug 2014
Status New Closed
Closed_Date 0000-00-00 00:00:00 2014-08-08 14:35:15
avatar brianteeman brianteeman - close - 8 Aug 2014

Add a Comment

Login with GitHub to post a comment