? ? ? Success

User tests: Successful: Unsuccessful:

avatar Bakual
Bakual
17 Aug 2020

Since we deprecated the strings in staging (#30110), this PR removes the strings in 4.0.

Summary of Changes

Removing unneeded strings

Testing Instructions

Nothing to test as the strings are not used anymore (after staging is merged up).

avatar Bakual Bakual - open - 17 Aug 2020
avatar Bakual Bakual - change - 17 Aug 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 17 Aug 2020
Category Administration Language & Strings
avatar Quy Quy - test_item - 17 Aug 2020 - Tested successfully
avatar Quy
Quy - comment - 17 Aug 2020

I have tested this item successfully on 694d77f


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

avatar ChristineWk ChristineWk - test_item - 17 Aug 2020 - Tested successfully
avatar ChristineWk
ChristineWk - comment - 17 Aug 2020

I have tested this item successfully on 694d77f


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

avatar Quy Quy - change - 17 Aug 2020
Status Pending Ready to Commit
avatar Quy
Quy - comment - 17 Aug 2020

RTC


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

avatar Bakual
Bakual - comment - 17 Aug 2020

Please wait with merge till the staging is merged up and PR #30110 is part of 4.0-dev. Otherwise it gets messy I think ?

avatar richard67
richard67 - comment - 14 Nov 2020

There were conflicts meanwhile. I've allowed myself to solve them.

But we still have to wait with merging. The changes from PR #30110 have meanwhile arrived in 3.10-dev, but they haven't been merged up into 4.0-dev yet.

avatar richard67 richard67 - change - 1 Dec 2020
Labels Added: ?
avatar richard67
richard67 - comment - 1 Dec 2020

I've allowed myself to solve conflicts. After recent merge from 3.10-dev up to 4.0-dev, only components/com_content/forms/article.xml is remaining using the removed strings.

So we still have to wait for that one before we can merge this one here.

avatar richard67
richard67 - comment - 1 Dec 2020

@Bakual Hmm, I just see that file components/com_content/forms/article.xml was not part of #30110 , and here only admin and API files are handled. That XML file contains these language stings, too, but they are not defined in the site language file ... am confused.

avatar Bakual
Bakual - comment - 1 Dec 2020

Looks certainly like I missed those for some reason.
That form file is used in the menuitem edit form in the backend. That's why the strings are missing in frontend.

avatar richard67
richard67 - comment - 2 Dec 2020

@Bakual Can't find that file in J3. Is it new in J4? And will you fix that, too?

avatar Bakual
Bakual - comment - 2 Dec 2020

@richard67 The file is there, I just created the PR to remove it.
I have no clue why I didn't see it back then, probably the same reason you're not seeing it today ?

avatar richard67
richard67 - comment - 2 Dec 2020

So this one has still to wait, this time for #31557 .

avatar Bakual
Bakual - comment - 2 Dec 2020

Looks like, yes. Sorry for that.

avatar wilsonge wilsonge - close - 18 Dec 2020
avatar wilsonge wilsonge - merge - 18 Dec 2020
avatar wilsonge wilsonge - change - 18 Dec 2020
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-12-18 03:01:19
Closed_By wilsonge
avatar wilsonge
wilsonge - comment - 18 Dec 2020

Can confirm nothing is using them after my latest set of merges

Add a Comment

Login with GitHub to post a comment