Failure

User tests: Successful: Unsuccessful:

avatar infograf768
infograf768
20 Aug 2016

This PR completes 1f48268
By implementing the new modal_menu field in various places in core.

Menu item Alias

screen shot 2016-08-20 at 17 15 31

Menu item Login form: login redirect and logout redirect

screen shot 2016-08-20 at 17 17 16

Menu Item Logout

screen shot 2016-08-20 at 17 18 41

Module mod_login (frontend) for login and logout redirect

screen shot 2016-08-20 at 17 21 21

Module mod_menu (frontend) Base Item

screen shot 2016-08-20 at 17 23 05

Remarks:

  1. The code adapts itself to the <option value="">STRING_CONSTANT</option> in the field.
    i.e. when JDEFAULT or JCURRENT is used.
                <field
                    name="login"
                    type="modal_menu"
                    label="MOD_LOGIN_FIELD_LOGIN_REDIRECTURL_LABEL"
                    description="MOD_LOGIN_FIELD_LOGIN_REDIRECTURL_DESC"
                    disable="separator,alias,heading,url"
                    edit="true"
                    clear="true"
                    >
                    <option value="">JDEFAULT</option>
                </field>

Here when JCURRENT is used (See screenshots above when it is JDEFAULT):

screen shot 2016-08-20 at 17 33 25

  1. For multilanguage Associations, it keeps COM_MENUS_SELECT_A_MENUITEM

screen shot 2016-08-20 at 17 28 52

  1. When selecting a menu item through the modal, we now display the forced language concerned in the title of the modal when we make associations:

screen shot 2016-08-20 at 17 31 02

TODO:

  1. Add also the language name to the modal title when using forced language in other core components.
  2. Add a new editors-xtd letting enter menu item links in content via new button in the editor (Already done here, will post soon)
avatar infograf768 infograf768 - open - 20 Aug 2016
avatar infograf768
infograf768 - comment - 20 Aug 2016

@brianteeman
Can you please set milestone to 3.7.0

avatar brianteeman
brianteeman - comment - 20 Aug 2016

What is the purpose have having separate buttons for select/edit

avatar infograf768
infograf768 - comment - 20 Aug 2016

As we already do in core:
Select lets select an item in a modal list
Edit lets edit the item selected in another modal once the original item has been saved.

We already do that for articles, etc.

avatar infograf768
infograf768 - comment - 20 Aug 2016

BTW, Travis failure looks unrelated to this PR:
PHP Fatal error: Call to a member function children() on null in /home/travis/build/joomla/joomla-cms/libraries/joomla/form/fields/text.php on line 187

avatar brianteeman
brianteeman - comment - 20 Aug 2016

Ah sorry. Got it now.
Select = select item
Edit = edit the selected item

I was thinking that
Edit = edit the selection ie make a new selection

avatar brianteeman brianteeman - change - 21 Aug 2016
Category Administration
avatar brianteeman brianteeman - change - 21 Aug 2016
Status New Pending
avatar brianteeman brianteeman - change - 21 Aug 2016
Milestone Added:
avatar jreys jreys - test_item - 22 Aug 2016 - Tested successfully
avatar jreys
jreys - comment - 22 Aug 2016

I have tested this item successfully on 841245f

Also tested if ACL was working, it was :D


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

avatar infograf768
infograf768 - comment - 22 Aug 2016

Last modification for modal title, similar to
#11718
and
#11717

avatar alikon alikon - test_item - 23 Aug 2016 - Tested successfully
avatar alikon
alikon - comment - 23 Aug 2016

I have tested this item successfully on c93de30


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

avatar infograf768 infograf768 - change - 23 Aug 2016
Status Pending Ready to Commit
avatar infograf768 infograf768 - alter_testresult - 23 Aug 2016 - jreys: Tested successfully
avatar infograf768
infograf768 - comment - 23 Aug 2016

RTC

@wilsonge we have issues with Travis which look unrelated to this PR as well as to #11721


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

avatar infograf768
infograf768 - comment - 23 Aug 2016

hmm rtc set on issues does not get it here

avatar zero-24
zero-24 - comment - 23 Aug 2016

@brianteeman can you help here?

avatar joomla-cms-bot joomla-cms-bot - change - 23 Aug 2016
Category Administration Administration Components Front End Modules
avatar brianteeman brianteeman - change - 23 Aug 2016
Labels Added: ?
avatar brianteeman
brianteeman - comment - 23 Aug 2016

Manually set to RTC on github

avatar infograf768
infograf768 - comment - 23 Aug 2016

tks

avatar wilsonge wilsonge - change - 28 Aug 2016
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-08-28 20:32:12
Closed_By wilsonge
avatar wilsonge wilsonge - close - 28 Aug 2016
avatar wilsonge wilsonge - merge - 28 Aug 2016
avatar joomla-cms-bot joomla-cms-bot - close - 28 Aug 2016
avatar joomla-cms-bot joomla-cms-bot - change - 28 Aug 2016
Labels Removed: ?

Add a Comment

Login with GitHub to post a comment