? ? Success

User tests: Successful: Unsuccessful:

avatar andrepereiradasilva
andrepereiradasilva
25 Aug 2016

Summary of Changes

Add javascript message titles and ajax errors strings to installation tempalte.

You will notice that now all installation warning, error, notices and message have title like in the rest of joomla.
image

Testing Instructions

Code review.

Documentation Changes Required

None.

avatar andrepereiradasilva andrepereiradasilva - open - 25 Aug 2016
avatar andrepereiradasilva andrepereiradasilva - change - 25 Aug 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 25 Aug 2016
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - change - 25 Aug 2016
Category Installation
avatar andrepereiradasilva andrepereiradasilva - change - 25 Aug 2016
The description was changed
avatar andrepereiradasilva andrepereiradasilva - edited - 25 Aug 2016
avatar andrepereiradasilva andrepereiradasilva - change - 25 Aug 2016
The description was changed
avatar andrepereiradasilva andrepereiradasilva - edited - 25 Aug 2016
avatar andrepereiradasilva andrepereiradasilva - change - 25 Aug 2016
The description was changed
avatar andrepereiradasilva andrepereiradasilva - edited - 25 Aug 2016
avatar andrepereiradasilva
andrepereiradasilva - comment - 25 Aug 2016

@Bakual please check this one because at installation we don't have the languages installed yet, so (i think) al joomla core messages loaded will be in english. right?

If so, is it better to add those language strings to the installation en-GB file, i mean, instead of loading joomla core and lib language file?

avatar brianteeman
brianteeman - comment - 25 Aug 2016

You can have installation in all languages

avatar brianteeman
brianteeman - comment - 25 Aug 2016

For example you can do this
Before running the installation, open installation/localize.xml and set the following on line 3:

ar-AA

avatar andrepereiradasilva
andrepereiradasilva - comment - 25 Aug 2016

yes brian but i'm loading language files that are not part of the installationlanguage files, so the language is not yet installed in joomla.

avatar yvesh
yvesh - comment - 25 Aug 2016

@andrepereiradasilva Do you think it would be a better solution adding these language strings to the installation language files?

avatar andrepereiradasilva
andrepereiradasilva - comment - 25 Aug 2016

the way the installation is designed ... yes i think we need all those language files in the installation languages files. Because the language itself would only be installed later.

I will make that change

avatar joomla-cms-bot joomla-cms-bot - change - 25 Aug 2016
Labels Added: ?
avatar joomla-cms-bot joomla-cms-bot - change - 25 Aug 2016
Category Installation Installation Language & Strings
avatar andrepereiradasilva
andrepereiradasilva - comment - 25 Aug 2016

ok done

avatar brianteeman
brianteeman - comment - 26 Aug 2016

@andrepereiradasilva can you update the description to remove the bit about loading the core lang files


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

avatar brianteeman brianteeman - test_item - 26 Aug 2016 - Tested successfully
avatar brianteeman
brianteeman - comment - 26 Aug 2016

I have tested this item successfully on 3761d4e


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

avatar andrepereiradasilva andrepereiradasilva - change - 26 Aug 2016
The description was changed
avatar andrepereiradasilva
andrepereiradasilva - comment - 26 Aug 2016
avatar brianteeman
brianteeman - comment - 26 Aug 2016
avatar bhavikTailored
bhavikTailored - comment - 26 Aug 2016
avatar bhavikTailored bhavikTailored - test_item - 26 Aug 2016 - Tested successfully
avatar bhavikTailored
bhavikTailored - comment - 26 Aug 2016

I have tested this item successfully on 3761d4e


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

avatar zero-24 zero-24 - change - 26 Aug 2016
Status Pending Ready to Commit
avatar zero-24
zero-24 - comment - 26 Aug 2016

RTC.


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

avatar joomla-cms-bot joomla-cms-bot - change - 26 Aug 2016
Labels Added: ?
avatar rdeutz rdeutz - change - 26 Aug 2016
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-08-26 11:53:35
Closed_By rdeutz
avatar joomla-cms-bot joomla-cms-bot - change - 26 Aug 2016
Labels Removed: ?
avatar tomartailored tomartailored - test_item - 27 Aug 2016 - Tested successfully
avatar tomartailored
tomartailored - comment - 27 Aug 2016

I have tested this item successfully on 3761d4e

I have Tested successfully


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

avatar brianteeman
brianteeman - comment - 27 Aug 2016

@tomartailored This has already been merged - so either you are not testing with the current staging which you must always do or you didnt really test it but I am sure that you did didnt you


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

Add a Comment

Login with GitHub to post a comment