? ? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
21 May 2018

Remove hard coded string and replace with translatable string

Add new string for the site name section - wrong string was being used

Ensure all section strings are consistent Capital case

Update en-US translation to be in sync with en-GB

avatar brianteeman brianteeman - open - 21 May 2018
avatar brianteeman brianteeman - change - 21 May 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 21 May 2018
Category Installation Language & Strings
avatar brianteeman brianteeman - change - 21 May 2018
Labels Added: ? ?
ada04f2 21 May 2018 avatar brianteeman oops
2932746 21 May 2018 avatar brianteeman oops
f0376ff 21 May 2018 avatar brianteeman grrh
80363df 21 May 2018 avatar brianteeman grrh
avatar wilsonge
wilsonge - comment - 22 May 2018

LGTM - are you planning on addressing the other two issues @Quy raised @brianteeman ? They're slightly less related to the main PR so I'm easy either way - just don't want to prematurely merge :)

avatar infograf768
infograf768 - comment - 22 May 2018

Not sure we should merge this now as we have quite a few code descrepancies vs lang strings that we should solve altogether if possible.

Examples:
<h3><?php echo JText::_('Do you want to install sample data?'); ?></h3> // We DO need a lang string for that one.
and
$setting->label = \JText::_('INSTL_MAGIC_QUOTES_RUNTIME'); // string missing but do we use it?

avatar brianteeman
brianteeman - comment - 23 May 2018

@infograf768 think I have caught all the missing strings now.

avatar infograf768
infograf768 - comment - 23 May 2018

INSTL_MAGIC_QUOTES_RUNTIME is missing when we look at code.
INSTL_MAGIC_QUOTES_RUNTIME="Magic Quotes Runtime"

avatar mbabker
mbabker - comment - 23 May 2018

In the context of 4.0 there really should be no references to Magic Quotes Runtime. PHP 5.4 removed it.

avatar brianteeman
brianteeman - comment - 23 May 2018

That is really beyond the scope of the intent of this PR but i have removed it

ee1648a 23 May 2018 avatar brianteeman magic
avatar mbabker
mbabker - comment - 23 May 2018

It only caught my eye because it was pointed out the string was missing.

avatar infograf768
infograf768 - comment - 24 May 2018

Please also take it off from UpdateModel.php so that we are done with it.

avatar brianteeman
brianteeman - comment - 24 May 2018

@infograf768 already done in a seperate pr

avatar wilsonge wilsonge - change - 30 May 2018
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-05-30 17:21:15
Closed_By wilsonge
avatar wilsonge wilsonge - close - 30 May 2018
avatar wilsonge wilsonge - merge - 30 May 2018
avatar wilsonge
wilsonge - comment - 30 May 2018

Thanks!

avatar brianteeman
brianteeman - comment - 30 May 2018

Thanks

Add a Comment

Login with GitHub to post a comment