? ? Success

User tests: Successful: Unsuccessful:

avatar infograf768
infograf768
2 Mar 2019

Summary of Changes

This PR adds a Notice and displays the language concerned when the Content Language for that language has been deleted and remains in the site an orphan home page while the site language is published.
It also adds a Notice when a Content Language is in the trash.

It also reinstate all states of a content language to display the correct icons.
published = 1
publsihed = 0
published = -2 (i.e. trash) and for this one displays the trash icon

Testing Instructions

Install a multilingual site with all languages available to install
French fr-FR
German de-DE
Persian fa-IR

Then use the Cpanel Multilingual Sample Data module to create a basic.

First test:
Display the Content languages Manager
Trash AND delete the fa-IR Persian content language.

You should get this when displaying the multilingual status module

screen shot 2019-03-02 at 18 25 09

Then unpublish the French fr-FR Content Language.
And trash (do NOT delete) the German de-DE Content language.
I.e. Manager should look like this
screen shot 2019-03-02 at 18 28 10

You should get this when displaying the multilingual status module
screen shot 2019-03-02 at 18 44 18

Edit

Unpublish also the fr-FR Home page to get a new notice.
Deleted Content Language error now displays the notice as last.
see #24062 (comment)

Reminder

It is OK to have an Unpublished Content Language (as French is above) as it lets the site owner prepare a new Language for the multilingual site.

avatar infograf768 infograf768 - open - 2 Mar 2019
avatar infograf768 infograf768 - change - 2 Mar 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 2 Mar 2019
Category Administration com_languages Language & Strings
avatar infograf768 infograf768 - change - 2 Mar 2019
The description was changed
avatar infograf768 infograf768 - edited - 2 Mar 2019
avatar infograf768 infograf768 - change - 2 Mar 2019
Labels Added: ? ?
avatar infograf768
infograf768 - comment - 3 Mar 2019

@Quy
Modification done.
Added new Notice when both Content Language and Home page are unpublished.

We should now get in that case:

screen shot 2019-03-03 at 10 22 15

can be fully tested now.

avatar infograf768 infograf768 - change - 3 Mar 2019
The description was changed
avatar infograf768 infograf768 - edited - 3 Mar 2019
avatar Quy
Quy - comment - 4 Mar 2019

Added new Notice when both Content Language and Home page are unpublished.

The Published Default Home pages has all checks even though all pages are unpublished.

avatar infograf768
infograf768 - comment - 4 Mar 2019

Added new Notice when both Content Language and Home page are unpublished.

The Published Default Home pages has all checks even though all pages are unpublished.

I do not understand this comment.

avatar Quy
Quy - comment - 4 Mar 2019

I don’t get the same result in the last column as in your last screenshot.

avatar infograf768
infograf768 - comment - 4 Mar 2019

I don’t get the same result in your last screenshot.

Please explain with more details.
The exact settings you have for each language and the resulting screenshot.

avatar Quy
Quy - comment - 4 Mar 2019

I am no longer at my computer so it will have to wait until tomorrow. I followed all your testing instructions. Only the last screenshot that I did not get the same result. All pages are unpublished.

avatar infograf768
infograf768 - comment - 4 Mar 2019

I may have forgotten to ask testing by unpublishing the French Home page AND its content language, in order to get the Notice and the right icons.
The order of the Notices may also have changed. One has to test multiple cases.

avatar infograf768
infograf768 - comment - 4 Mar 2019

We may need though another Notice when site language is enabled, content language is published and home page is unpublished.

I.e. this case for French and German
screen shot 2019-03-04 at 08 41 19

On it.

avatar infograf768
infograf768 - comment - 4 Mar 2019

Added notice when both content language and home page missing. added trash icon when home page is in the trash.

We now get in these cases:

screen shot 2019-03-04 at 12 40 18

avatar infograf768
infograf768 - comment - 4 Mar 2019

Added warning icon when home page does not exist.
screen shot 2019-03-04 at 12 56 37

avatar infograf768
infograf768 - comment - 21 Mar 2019
avatar franz-wohlkoenig franz-wohlkoenig - change - 11 Apr 2019
Category Administration com_languages Language & Strings Administration com_languages
avatar joomla-cms-bot joomla-cms-bot - change - 30 May 2019
Category Administration com_languages Administration com_languages Language & Strings
avatar Quy
Quy - comment - 27 Nov 2019

Please fix conflicts.

avatar infograf768
infograf768 - comment - 28 Nov 2019

@Quy
This needs a lot of work to solve the conflicts and also update the queries.
I may have to create a new PR
Will try when I have time.

09581c6 30 Nov 2019 avatar infograf768 space
avatar infograf768
infograf768 - comment - 30 Nov 2019

I am afraid this branch is dead. Please test this new PR
#27184

Closing

avatar infograf768 infograf768 - change - 30 Nov 2019
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2019-11-30 07:48:41
Closed_By infograf768

Add a Comment

Login with GitHub to post a comment