? ? Pending

User tests: Successful: Unsuccessful:

avatar SharkyKZ
SharkyKZ
8 Jul 2019

Pull Request for Issue #25344 .

Summary of Changes

Loads global administrator language file when viewing backend view in frontend (i.e. in Editors-XTD modals).

Testing Instructions

Enable Debug Language in Global Configuration.
Edit an article in frontend.
In editor, use Contact, Field and Article buttons.

Expected result

No untranslated strings.

Actual result

Untranslated strings:

# JROOT\libraries\src\Language\Text.php

JAUTHOR_ASC="ASC"
JAUTHOR_DESC="DESC"
JCATEGORY_ASC="ASC"
JCATEGORY_DESC="DESC"
JDATE_ASC="ASC"
JDATE_DESC="DESC"
JFEATURED_ASC="ASC"
JFEATURED_DESC="DESC"
JGLOBAL_HITS_ASC="HITS ASC"
JGLOBAL_HITS_DESC="HITS DESC"
JGLOBAL_LIST_ALIAS="LIST ALIAS"
JGLOBAL_LIST_ALIAS_NOTE="LIST ALIAS NOTE"
JGLOBAL_SORT_BY="SORT BY"
JGLOBAL_TITLE_ASC="TITLE ASC"
JGLOBAL_TITLE_DESC="TITLE DESC"
JGRID_HEADING_ACCESS_ASC="HEADING ACCESS ASC"
JGRID_HEADING_ACCESS_DESC="HEADING ACCESS DESC"
JGRID_HEADING_ID_ASC="HEADING ID ASC"
JGRID_HEADING_ID_DESC="HEADING ID DESC"
JGRID_HEADING_LANGUAGE_ASC="HEADING LANGUAGE ASC"
JGRID_HEADING_LANGUAGE_DESC="HEADING LANGUAGE DESC"
JGRID_HEADING_ORDERING_ASC="HEADING ORDERING ASC"
JGRID_HEADING_ORDERING_DESC="HEADING ORDERING DESC"
JSTATUS_ASC="ASC"
JSTATUS_DESC="DESC"

Documentation Changes Required

No.

avatar SharkyKZ SharkyKZ - open - 8 Jul 2019
avatar SharkyKZ SharkyKZ - change - 8 Jul 2019
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 8 Jul 2019
Category Front End com_contact com_content com_fields
avatar SharkyKZ SharkyKZ - change - 8 Jul 2019
Labels Added: ?
avatar infograf768
infograf768 - comment - 8 Jul 2019

IMHO, this is not the solution.
We may have different values in site xx-XX.ini and admin xx-XX.ini for the same constant.

The only correct solution is to add all missing lang strings in frontend (I know it is a pain).

avatar infograf768
infograf768 - comment - 8 Jul 2019

Hmm
I tested. It looks that there is no impact doing it this way for these modals.

**Loaded** : JROOT/language/en-GB/en-GB.ini
**Loaded** : JROOT/administrator/language/en-GB/en-GB.ini

Only possible issue would be for TTs providing only frontend translation.
Any way to change the loading order?

avatar infograf768
infograf768 - comment - 8 Jul 2019

I have tested this item successfully on ff7c1f6


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

avatar infograf768 infograf768 - test_item - 8 Jul 2019 - Tested successfully
avatar SharkyKZ
SharkyKZ - comment - 8 Jul 2019

Any way to change the loading order?

I don't think so. Global language file is first loaded very early.

avatar infograf768
infograf768 - comment - 8 Jul 2019

Re-started drone

avatar Quy
Quy - comment - 8 Jul 2019

I have tested this item successfully on ff7c1f6


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

avatar Quy Quy - test_item - 8 Jul 2019 - Tested successfully
avatar Quy Quy - change - 8 Jul 2019
Status Pending Ready to Commit
avatar Quy
Quy - comment - 8 Jul 2019

RTC


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

avatar brianteeman
brianteeman - comment - 8 Jul 2019

IMHO, this is not the solution.
We may have different values in site xx-XX.ini and admin xx-XX.ini for the same constant.

The only correct solution is to add all missing lang strings in frontend (I know it is a pain).

agree

avatar infograf768
infograf768 - comment - 8 Jul 2019

@SharkyKZ
will make proper PR to include the strings in frontend. That would be for 3.9.10 as it is too late for 3.9.9
At the same time I will check why was added loading the backend main ini in #12321 . May need some help if the proper PR does not take care of it..

avatar infograf768
infograf768 - comment - 8 Jul 2019

We do not need to load admin main ini when the strings are added in frontend.
My patch will therefore delete these calls.

avatar infograf768
infograf768 - comment - 8 Jul 2019

Maintainers have decided to rather add the strings.
See and test #25475

Thanks. Closing this one.

avatar infograf768 infograf768 - change - 8 Jul 2019
Status Ready to Commit Closed
Closed_Date 0000-00-00 00:00:00 2019-07-08 16:32:15
Closed_By infograf768
Labels Added: ?
avatar infograf768 infograf768 - close - 8 Jul 2019

Add a Comment

Login with GitHub to post a comment