? ? Pending

User tests: Successful: Unsuccessful:

avatar joomdonation
joomdonation
12 May 2021

Pull Request for Issue #33784.

Summary of Changes

The Multilingual Associations can only be used for multilingual website with Associations enabled in System - Language Filter plugin. This PRs modify code to only show Multilingual Associations menu item from Components menu backend if the conditions meet.

Testing Instructions

  1. Install Joomla 4.0-dev
  2. Access to Administrator area of your site, then access to Components menu item on the left sidebar
  3. Before patch, you see Multilingual Associations menu item
  4. Apply Patch:
  • You see the menu item not being displayed
  • Go to System -> Plugins, enable System - Language Filter and make sure Item Associations parameter set tot Yes. Then check it again, the menu item Multilingual Associations displayed.

Actual result BEFORE applying this Pull Request

Multilingual Associations menu item always displayed.

Expected result AFTER applying this Pull Request

Multilingual Associations menu item always displayed if conditions meet

avatar joomdonation joomdonation - open - 12 May 2021
avatar joomdonation joomdonation - change - 12 May 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 12 May 2021
Category Modules Administration
avatar sandramay0905 sandramay0905 - test_item - 12 May 2021 - Tested successfully
avatar sandramay0905
sandramay0905 - comment - 12 May 2021

I have tested this item successfully on 9b63c5f

Off topic this pull request, but shouldn't in the options the multilingual associations also be hidden? If yes, i can make a issue.


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

avatar joomdonation
joomdonation - comment - 12 May 2021

@sandramay0905 What options you are talking about? Could you show me a screenshot? Thanks

avatar sandramay0905
sandramay0905 - comment - 12 May 2021

The Options are shown even the multilingual associations are not visible by the pr:

image

avatar joomdonation
joomdonation - comment - 12 May 2021

Ah, for that, I guess we will have to disable the component completely. Maybe you can add comment to the original issue #33784 ? I guess I should re-open that issue, too

avatar infograf768
infograf768 - comment - 12 May 2021

Patch OK.
We could leave the Options imho as anyway I guess we should not introduce disabling a specific extension/component in com_config depending on the language filter setting.
The Options page would load ok and when closed, will anyway show the alert.
The only remaining hint to inform admin users that the site can be multilingual with associations, if desired.

Screen Shot 2021-05-12 at 09 20 39

avatar infograf768 infograf768 - test_item - 12 May 2021 - Tested successfully
avatar infograf768
infograf768 - comment - 12 May 2021

I have tested this item successfully on 9b63c5f


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

avatar infograf768 infograf768 - change - 12 May 2021
Status Pending Ready to Commit
avatar infograf768
infograf768 - comment - 12 May 2021

RTC


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

avatar richard67 richard67 - change - 12 May 2021
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-05-12 09:58:37
Closed_By richard67
Labels Added: ? ?
avatar richard67 richard67 - close - 12 May 2021
avatar richard67 richard67 - merge - 12 May 2021
avatar richard67
richard67 - comment - 12 May 2021

Thanks!

avatar brianteeman
brianteeman - comment - 12 May 2021

Maybe now we will be allowed to only show the multilingual sample data in the same situation?

avatar joomdonation
joomdonation - comment - 12 May 2021

@brianteeman I haven't tried that multilingual sample data, so I don't know. If the sample data install additional languages, enable the Language Filter plugin to make the site ready multilingual, then It should stay there. If not (just setup data like articles... only), then I think we can do the same check with how we are doing in this PR.

avatar richard67
richard67 - comment - 12 May 2021

@joomdonation The multilingual sample data doesn't install languages. But it does all the rest, enable the Language Filter plugin, create menus and home menu items ...

avatar infograf768
infograf768 - comment - 12 May 2021

Concerning the multilingual sample data: old discussion...

We have 2 informations:

  1. The description

Screen Shot 2021-05-12 at 12 16 58

  1. A warning when installing if only one language is present

Screen Shot 2021-05-12 at 12 19 34

imho, this is enough information and a good hint that the site can be multingual, info which is not provided anywhere else.

Add a Comment

Login with GitHub to post a comment