? ? Success

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
15 Aug 2016

Pull Request for Issue #10798

Summary of Changes

Added few strings but mainly it was correcting errors to make sure the correct string is bing used

avatar joomla-cms-bot joomla-cms-bot - change - 15 Aug 2016
Category Tags Administration Components Language & Strings Templates (admin) Libraries
avatar brianteeman brianteeman - open - 15 Aug 2016
avatar brianteeman brianteeman - change - 15 Aug 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 15 Aug 2016
Labels Added: ? ?
avatar infograf768
infograf768 - comment - 19 Aug 2016

Suggestion:
Change
COM_CONTACT_SESSION_INVALID="Invalid session cookie. Please check that you have cookies enabled in your web browser."
to a JLIB_ENVIRONMENT_SESSION_INVALID string as It could be used in the future in other extensions than contacts.

It would be placed just after
JLIB_ENVIRONMENT_SESSION_EXPIRED="Your session has expired. Please log in again."

avatar brianteeman
brianteeman - comment - 19 Aug 2016

COM_CONTACT_SESSION_INVALID
This is inside a function "Method to submit the contact form and send an email" so I dont think it is correct to make that change

avatar infograf768
infograf768 - comment - 20 Aug 2016

Many JLIB_ strings are used in specific extension methods. But nevermind

avatar wilsonge
wilsonge - comment - 20 Aug 2016

JLIB_ strings belong in the library translation files but can be used by extensions
extension strings belong in the extension language file and should only be used by that extension.

avatar brianteeman
brianteeman - comment - 26 Aug 2016

I have made the requested changed for
JLIB_STEMMER_INVALID_STEMMER
JLIB_CMS_WARNING_PROVIDE_VALID_NAME

Still not sure I understand the comments about COM_CONTACT_SESSION_INVALID should i move it to the JLIB_ or not?

avatar infograf768
infograf768 - comment - 26 Aug 2016

I suggested you change that constant to a JLIB_ one and create the string in lib.joomla.ini

avatar brianteeman
brianteeman - comment - 26 Aug 2016

OK thats done - please test and then we can finish with this one

avatar brianteeman
brianteeman - comment - 29 Aug 2016

@wilsonge @infograf768 all the changes you wanted have been done - please test so we can get it merged and give translators as much time as possible

avatar wilsonge wilsonge - change - 6 Sep 2016
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-09-06 14:19:55
Closed_By wilsonge
avatar brianteeman
brianteeman - comment - 6 Sep 2016

Thanks

Add a Comment

Login with GitHub to post a comment