? Pending

User tests: Successful: Unsuccessful:

avatar izharaazmi
izharaazmi
31 Aug 2017

Pull Request for Issue #17729

Summary of Changes

Set mesage type to error

Testing Instructions

Do something that fails menu deletion. e.g - add return false with some dummy setError message from MenusModelMenu::delete method.

Expected result

Red error message

Actual result

Green error message

Documentation Changes Required

None

avatar joomla-cms-bot joomla-cms-bot - change - 31 Aug 2017
Category Administration com_menus
avatar izharaazmi izharaazmi - open - 31 Aug 2017
avatar izharaazmi izharaazmi - change - 31 Aug 2017
Status New Pending
avatar franz-wohlkoenig franz-wohlkoenig - test_item - 31 Aug 2017 - Tested successfully
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 31 Aug 2017

I have tested this item successfully on d759fd0


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/17796.

avatar brianteeman brianteeman - test_item - 31 Aug 2017 - Tested successfully
avatar brianteeman
brianteeman - comment - 31 Aug 2017

I have tested this item successfully on d759fd0


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/17796.

avatar franz-wohlkoenig franz-wohlkoenig - change - 31 Aug 2017
Status Pending Ready to Commit
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 31 Aug 2017

RTC after two successful tests.

avatar mbabker mbabker - change - 31 Aug 2017
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-08-31 11:51:58
Closed_By mbabker
Labels Added: ?
avatar mbabker mbabker - close - 31 Aug 2017
avatar mbabker mbabker - merge - 31 Aug 2017

Add a Comment

Login with GitHub to post a comment