? Success

User tests: Successful: Unsuccessful:

avatar Kubik-Rubik
Kubik-Rubik
10 Jun 2014

...items

JoomlaCode item: http://joomlacode.org/gf/project/joomla/tracker/?action=TrackerItemEdit&tracker_item_id=33781&start=50 (by Ruth Cheesley)

The static function menuitems is only used for the batch process. There is no need to exclude the URL type because this type can also have children in the menu. With the exclusion the batch process usage for this constellation is impossible.

Votes

# of Users Experiencing Issue
1/1
Average Importance Score
4.00

avatar Kubik-Rubik Kubik-Rubik - open - 10 Jun 2014
avatar Kubik-Rubik Kubik-Rubik - change - 10 Jun 2014
Title
Unable to select external URLs using the batch feature in menus to move ...
[#33781] Unable to select external URLs using the batch feature in menus to move ...
avatar nicksavov nicksavov - change - 21 Aug 2014
Labels Removed: ?
avatar brianteeman brianteeman - change - 21 Aug 2014
Status New Pending
avatar brianteeman
brianteeman - comment - 17 Oct 2014

How to reproduce the problem and/or test the patch::

Create a menu structure with the top level an external URL, sub-menu items can be anything. Have at least two child levels: - External URL Top Level -- 1st Child Level Article -- 1st Child Level Article --- 2nd Child Level Article --- 2nd Child Level Article Select the two '2nd Child Level Articles' and use the 'batch' feature to move them to be a child of the top level item. The top level item is not available to be selected in the menu dropdown. Set the top-level item to be a text separator (for example) and it is possible to select it using the method above.

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

avatar mtmeyrick mtmeyrick - test_item - 17 Oct 2014 - Tested successfully
avatar brianteeman
brianteeman - comment - 2 Jan 2015

@test Issue confirmed
Patch resolves it_


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

avatar brianteeman brianteeman - test_item - 2 Jan 2015 - Tested successfully
avatar brianteeman brianteeman - change - 2 Jan 2015
Status Pending Ready to Commit
avatar brianteeman
brianteeman - comment - 2 Jan 2015

Setting RTC


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

avatar brianteeman brianteeman - change - 2 Jan 2015
Labels Added: ?
avatar Kubik-Rubik
Kubik-Rubik - comment - 2 Jan 2015

Thank you for testing the PR, I will merge it now!

My first merge, whoohoo! :+1:

avatar Kubik-Rubik Kubik-Rubik - close - 2 Jan 2015
avatar Kubik-Rubik Kubik-Rubik - reference | e652ae5 - 2 Jan 15
avatar Kubik-Rubik Kubik-Rubik - merge - 2 Jan 2015
avatar Kubik-Rubik Kubik-Rubik - close - 2 Jan 2015
avatar Kubik-Rubik Kubik-Rubik - change - 2 Jan 2015
Status Ready to Commit Closed
Closed_Date 0000-00-00 00:00:00 2015-01-02 21:21:31
avatar Kubik-Rubik Kubik-Rubik - head_ref_deleted - 2 Jan 2015
avatar Kubik-Rubik Kubik-Rubik - change - 3 Jan 2015
Milestone Added:
avatar brianteeman
brianteeman - comment - 3 Jan 2015

@Kubik-Rubik I thought it was considered good practice not to merge your own PR :)

Add a Comment

Login with GitHub to post a comment