? ? Success

User tests: Successful: Unsuccessful:

avatar infograf768
infograf768
17 May 2016

The language string for language installation is the same as used in general install.
COM_INSTALLER_INSTALL_SUCCESS="Installation of the %s was successful."
In the case of general install, the variable is of the type

COM_INSTALLER_TYPE_TYPE_COMPONENT="component"
COM_INSTALLER_TYPE_TYPE_FILE="file"
COM_INSTALLER_TYPE_TYPE_LANGUAGE="language"
COM_INSTALLER_TYPE_TYPE_LIBRARY="library"
COM_INSTALLER_TYPE_TYPE_MODULE="module"
COM_INSTALLER_TYPE_TYPE_PACKAGE="package"
COM_INSTALLER_TYPE_TYPE_PLUGIN="plugin"
COM_INSTALLER_TYPE_TYPE_TEMPLATE="template"

This does not fit with installing a language pack as we get

screen shot 2016-05-17 at 08 20 48

Ugly in any language...

This PR adds a new string for language installation:
COM_INSTALLER_INSTALL_LANGUAGE_SUCCESS="Installation of the <strong>%s</strong> language pack was successful."
Which gives:

screen shot 2016-05-17 at 09 33 12

To test go to administrator/index.php?option=com_installer&view=languages and install any language.

avatar infograf768 infograf768 - open - 17 May 2016
avatar infograf768 infograf768 - change - 17 May 2016
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 17 May 2016
Labels Added: ? ?
avatar infograf768
infograf768 - comment - 17 May 2016

@brianteeman
I can change pack to package
COM_INSTALLER_INSTALL_LANGUAGE_SUCCESS="Installation of the <strong>%s</strong> language package was successful."
if you wish.

avatar brianteeman
brianteeman - comment - 17 May 2016

I was wondering if we even need that word at all.
COM_INSTALLER_INSTALL_LANGUAGE_SUCCESS="Installation of the
%s language was successful."

On 17 May 2016 at 08:52, infograf768 notifications@github.com wrote:

@brianteeman https://github.com/brianteeman
I can change pack to package
COM_INSTALLER_INSTALL_LANGUAGE_SUCCESS="Installation of the
%s language package was successful."
if you wish.


You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
#10523 (comment)

Brian Teeman
Co-founder Joomla! and OpenSourceMatters Inc.
http://brian.teeman.net/

avatar brianteeman brianteeman - change - 17 May 2016
Category Installation Language & Strings
avatar brianteeman brianteeman - change - 17 May 2016
Labels
avatar infograf768
infograf768 - comment - 17 May 2016

No objection.

avatar sgagner
sgagner - comment - 17 May 2016

I think we can skip "pack" or "package" and just use "language"

avatar brianteeman
brianteeman - comment - 17 May 2016

Make it so and I will make it RTC- its a simple and obvious improvement

avatar imanickam
imanickam - comment - 17 May 2016

The text phrase "Installation of the %s language was successful." is fine.

avatar infograf768
infograf768 - comment - 17 May 2016

Please mark test as successful IN https://issues.joomla.org/tracker/joomla-cms/10523

avatar brianteeman brianteeman - test_item - 17 May 2016 - Tested successfully
avatar brianteeman
brianteeman - comment - 17 May 2016

I have tested this item :white_check_mark: successfully on 940a822


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

avatar brianteeman brianteeman - change - 17 May 2016
Status Pending Ready to Commit
Labels
avatar brianteeman
brianteeman - comment - 17 May 2016

RTC


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

avatar joomla-cms-bot joomla-cms-bot - change - 17 May 2016
Labels Added: ?
avatar brianteeman brianteeman - change - 17 May 2016
Milestone Added:
avatar sgagner
sgagner - comment - 17 May 2016

We could make even simpler like: "%s language is now installed."


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

avatar infograf768
infograf768 - comment - 17 May 2016

@sgagner
Stefan, if we were doing that (which indeed could be done), it would not be consequent with other success messages we have all over.
Let's see that, if necessary, for 4.0. :smiley:

avatar BurtNL BurtNL - test_item - 17 May 2016 - Tested successfully
avatar BurtNL
BurtNL - comment - 17 May 2016

I have tested this item :white_check_mark: successfully on 940a822


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

avatar MATsxm MATsxm - test_item - 17 May 2016 - Tested successfully
avatar MATsxm
MATsxm - comment - 17 May 2016

I have tested this item :white_check_mark: successfully on 940a822


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

avatar Kubik-Rubik
Kubik-Rubik - comment - 17 May 2016

Thank you @infograf768!

avatar Kubik-Rubik Kubik-Rubik - change - 17 May 2016
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2016-05-17 15:51:07
Closed_By Kubik-Rubik
avatar zero-24
zero-24 - comment - 17 May 2016

@joomla-cms-bot come here and fix the label :P

avatar joomla-cms-bot joomla-cms-bot - change - 17 May 2016
Labels Removed: ?

Add a Comment

Login with GitHub to post a comment