?
avatar Kostelano
Kostelano
4 May 2020

Steps to reproduce the issue

Log in the control panel and press Edit Account, then click Save. You can see the message about saving to the profile in an unusual way :).

Screenshot_4

Among other things, the file with the original constant is loaded, and then en-GB.com_messages.ini.

JLIB_APPLICATION_SAVE_SUCCESS="Item saved."
JLIB_APPLICATION_SAVE_SUCCESS="Message sent."

Language Files Loaded
**Loaded** : JROOT\administrator\language\en-GB\en-GB.lib_joomla.ini
...
**Loaded** : JROOT\administrator\language\en-GB\en-GB.com_messages.ini

Need your own constant for "message sent" in the private message component.

System information (as much as possible)

Joomla 3.9.18
PHP 7.3.2

avatar Kostelano Kostelano - open - 4 May 2020
avatar joomla-cms-bot joomla-cms-bot - change - 4 May 2020
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 4 May 2020
avatar SharkyKZ
SharkyKZ - comment - 5 May 2020

@infograf768 can we remove JLIB_APPLICATION_SAVE_SUCCESS from com_messages.ini at this point or will this have to wait for 4.0?

avatar brianteeman
brianteeman - comment - 5 May 2020

@SharkyKZ surely it is needed for com_messages

I am guessing that the problem is that this language file should not be being loaded here

avatar SharkyKZ
SharkyKZ - comment - 5 May 2020

We can add COM_MESSAGES_SAVE_SUCCESS and there will be no need for JLIB_APPLICATION_SAVE_SUCCESS in com_messages.ini. But if we can't remove it now, we'll also have to add COM_ADMIN_SAVE_SUCCESS to com_admin.ini for now.

avatar SharkyKZ
SharkyKZ - comment - 5 May 2020

That the file is loading is fine (it's done by Actionlogs plugin) but it really shouldn't be overriding main library strings.

avatar infograf768
infograf768 - comment - 5 May 2020

JLIB strings in specific component strings were designed to override the main lib.joomla.ini strings on purpose.
This situation is indeed created by the new Actionlogs plugin.
At first sight, it does not look like creating a specific string in com_messages.ini and deleting the JLIB one would create any issue.
Evidently that would mean adding a save method in the user controller.

avatar SharkyKZ
SharkyKZ - comment - 5 May 2020

No changes needed in controller, it supports string prefixes.

avatar infograf768
infograf768 - comment - 5 May 2020

ah.. right.
Therefore no problem to add the string COM_MESSAGES_SAVE_SUCCESS and kill the JLIB one

avatar SharkyKZ SharkyKZ - change - 5 May 2020
Status New Confirmed
avatar SharkyKZ SharkyKZ - change - 5 May 2020
Status Confirmed Closed
Closed_Date 0000-00-00 00:00:00 2020-05-05 10:17:35
Closed_By SharkyKZ
avatar joomla-cms-bot joomla-cms-bot - change - 5 May 2020
Closed_By SharkyKZ joomla-cms-bot
avatar joomla-cms-bot joomla-cms-bot - close - 5 May 2020
avatar joomla-cms-bot
joomla-cms-bot - comment - 5 May 2020

Set to "closed" on behalf of @SharkyKZ by The JTracker Application at issues.joomla.org/joomla-cms/28934

avatar SharkyKZ
SharkyKZ - comment - 5 May 2020

Please test PR #28939.


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

Add a Comment

Login with GitHub to post a comment