User tests: Successful: Unsuccessful:
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
Status | New | ⇒ | Pending |
Category | ⇒ | Installation Language & Strings |
Labels |
Added:
?
?
|
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 :)
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?
@infograf768 think I have caught all the missing strings now.
INSTL_MAGIC_QUOTES_RUNTIME is missing when we look at code.
INSTL_MAGIC_QUOTES_RUNTIME="Magic Quotes Runtime"
In the context of 4.0 there really should be no references to Magic Quotes Runtime. PHP 5.4 removed it.
That is really beyond the scope of the intent of this PR but i have removed it
It only caught my eye because it was pointed out the string was missing.
Please also take it off from UpdateModel.php so that we are done with it.
@infograf768 already done in a seperate pr
Status | Pending | ⇒ | Fixed in Code Base |
Closed_Date | 0000-00-00 00:00:00 | ⇒ | 2018-05-30 17:21:15 |
Closed_By | ⇒ | wilsonge |
Thanks!
Thanks
Please change
customise
https://github.com/joomla/joomla-cms/pull/20515/files#diff-08b8aee1d43899c68aa8a784f98becb5R85