? Pending

User tests: Successful: Unsuccessful:

avatar richard67
richard67
12 Nov 2018

Pull Request for Issue #23023 .

Summary of Changes

Remove unused module class (formerly called module class suffix) "_menu" from menu modules' parameters in new installation and sample data.

Testing Instructions

Code review.

Expected result

The module class "_menu" is not added to parameters of any menu modules on new installation and on any kind of sample data installation.

Actual result

The module class "_menu" is added to the parameters of the default "Main Menu" on new installation of J4, of diverse menus in testing sample data and of the language-specific main menus in multilanguage sample data, but it has no effect on styling.
In comparison to that, when creating a new menu, the module class "_menu" is not added to the new menu.
This inconistency and the obsolete module class added in the cases mentioned above might confuse administrators who create or maintain menu modules.

Documentation Changes Required

None.

avatar richard67 richard67 - open - 12 Nov 2018
avatar richard67 richard67 - change - 12 Nov 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 12 Nov 2018
Category SQL Installation Postgresql Front End Plugins
avatar richard67
richard67 - comment - 12 Nov 2018

@ciar4n Could you review or test this PR?


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

avatar ciar4n
ciar4n - comment - 12 Nov 2018

I have tested this item successfully on a8a484d


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

avatar ciar4n ciar4n - test_item - 12 Nov 2018 - Tested successfully
avatar Quy
Quy - comment - 12 Nov 2018

I have tested this item successfully on a8a484d


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

avatar Quy Quy - test_item - 12 Nov 2018 - Tested successfully
avatar Quy Quy - change - 12 Nov 2018
Status Pending Ready to Commit
avatar Quy
Quy - comment - 12 Nov 2018

RTC


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

avatar richard67
richard67 - comment - 12 Nov 2018

Thanks to all testers.

avatar wilsonge wilsonge - change - 14 Nov 2018
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-11-14 19:24:26
Closed_By wilsonge
Labels Added: ?
avatar wilsonge wilsonge - close - 14 Nov 2018
avatar wilsonge wilsonge - merge - 14 Nov 2018
avatar wilsonge
wilsonge - comment - 14 Nov 2018

Thanks!

avatar richard67
richard67 - comment - 14 Nov 2018

Thanks, too.

Add a Comment

Login with GitHub to post a comment