avatar infograf768
infograf768
5 Apr 2017

Pull Request for Issue #15094

Summary of Changes

As title says.
When creating a frontend menu module the default is to show the sub menus.

				<field
					name="showAllChildren"
					type="radio"
					label="MOD_MENU_FIELD_ALLCHILDREN_LABEL"
					description="MOD_MENU_FIELD_ALLCHILDREN_DESC"
					class="btn-group btn-group-yesno"
					default="1"
					>
					<option value="1">JYES</option>
					<option value="0">JNO</option>
				</field>

This normalises this setting in all sample data.
I see no unwanted consequences: when no submenus available, no change.

Testing Instructions

Patch an instance of Joomla and install with various sample data
This can also be tested using this pack
https://github.com/infograf768/joomla-cms/archive/sub_menus_modules.zip

avatar infograf768 infograf768 - open - 5 Apr 2017
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 5 Apr 2017

see its an complete install. Test now.

avatar infograf768
infograf768 - comment - 5 Apr 2017

It's indeed mostly code review, but you can check is none or any sample data installs ok, and then look at the menu modules to confirm that Sub Menu Items parameter is set to Yes.

avatar franz-wohlkoenig
franz-wohlkoenig - comment - 5 Apr 2017

tested successfully (didn't get Button on Issue Tracker to mark)

avatar franz-wohlkoenig franz-wohlkoenig - change - 5 Apr 2017
Category Installation SQL
avatar franz-wohlkoenig franz-wohlkoenig - change - 5 Apr 2017
Status New Confirmed
avatar infograf768 infograf768 - change - 5 Apr 2017
Status Confirmed Pending
avatar infograf768
infograf768 - comment - 5 Apr 2017

We need one more tester.

avatar Quy
Quy - comment - 5 Apr 2017

Tested successfully via code review.

Should this file be updated too?
\joomla-cms\tests\unit\stubs\database\jos_modules.csv


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

avatar franz-wohlkoenig franz-wohlkoenig - change - 5 Apr 2017
Status Pending Ready to Commit
avatar franz-wohlkoenig
franz-wohlkoenig - comment - 5 Apr 2017

RTC after two successful testes.

avatar infograf768 infograf768 - change - 5 Apr 2017
Milestone Added:
avatar rdeutz rdeutz - change - 6 Apr 2017
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-04-06 07:53:54
Closed_By rdeutz
avatar rdeutz rdeutz - close - 6 Apr 2017
avatar rdeutz rdeutz - merge - 6 Apr 2017
avatar joomla-cms-bot joomla-cms-bot - change - 6 Apr 2017
Category Installation SQL SQL Installation Postgresql MS SQL

Add a Comment

Login with GitHub to post a comment