?
avatar brianteeman
brianteeman
19 Feb 2019

There are two language files for the media manager.

language\en-GB\en-GB.com_media.ini
administrator\language\en-GB\en-GB.com_media.ini

I would expect them to have the same number of strings as there is no difference between the front end and the admin views (at least that I could see)

I would also expect that the values of the strings was the same in both but it is not
eg
COM_MEDIA_NAME="Image Name"
COM_MEDIA_NAME="Name"

avatar brianteeman brianteeman - open - 19 Feb 2019
avatar joomla-cms-bot joomla-cms-bot - change - 19 Feb 2019
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - labeled - 19 Feb 2019
avatar puneeth2001
puneeth2001 - comment - 19 Feb 2019

@brianteeman just clarifying that it is been called from administrator\language\en-GB\en-GB.com_media.ini i.e. 'COM_MEDIA_NAME="Name" ' this is the correct one I think

avatar mbabker
mbabker - comment - 19 Feb 2019

If the admin language file is the only one used then the frontend file should be removed. It's an excess and unused resource and it puts extra work on translators to maintain an unused resource (and they already have enough excess work in their workflows).

avatar brianteeman
brianteeman - comment - 19 Feb 2019

@puneeth2001 then please remove the incorrect file if it is not being used AND check that all the required strings are present. For example COM_MEDIA_INCREASE_GRID is not present in the admin version but it is needed

avatar puneeth2001
puneeth2001 - comment - 19 Feb 2019

@brianteeman surprisingly only 'COM_MEDIA_NAME' string is the only thing which is present in common in both the files(but with different string values) and remaining all strings are different i.e if one is present in a file it is not present in the other one. I'll put the screenshot of the code if necessary.

avatar brianteeman
brianteeman - comment - 19 Feb 2019

This string is still missing COM_MEDIA_INCREASE_GRID Check the code!!!

avatar puneeth2001
puneeth2001 - comment - 19 Feb 2019

ok I'll check it

avatar puneeth2001
puneeth2001 - comment - 19 Feb 2019

sorry!! yes, there are some strings along with this which must be added.

avatar puneeth2001
puneeth2001 - comment - 19 Feb 2019

@brianteeman there are a lot of strings which need to be added and I am not sure that they might be used from the other file somewhere I think it's better to leave it that way or any suggestions??

avatar brianteeman
brianteeman - comment - 19 Feb 2019

No it is not better to leave it broken

avatar puneeth2001
puneeth2001 - comment - 20 Feb 2019

ok I'll do it.

On Wed, 20 Feb 2019, 03:55 Brian Teeman <notifications@github.com wrote:

No it is not better to leave it broken


You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
#23943 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/ApyKVOWM6b7sJDgjPLBJTa3ZJxbovZBbks5vPHnAgaJpZM4bCyZM
.

avatar brianteeman brianteeman - change - 20 Feb 2019
Status New Closed
Closed_Date 0000-00-00 00:00:00 2019-02-20 15:25:16
Closed_By brianteeman
avatar brianteeman brianteeman - close - 20 Feb 2019
avatar brianteeman
brianteeman - comment - 20 Feb 2019

Closed see #23957

Add a Comment

Login with GitHub to post a comment