Bug 14203

Summary: koha-translate error string for non-existent lang removal
Product: Koha Reporter: Indranil Das Gupta <indradg>
Component: Command-line UtilitiesAssignee: Indranil Das Gupta <indradg>
Status: CLOSED FIXED QA Contact: Testopia <testopia>
Severity: trivial    
Priority: P5 - low CC: chris, kyle, mtj, tomascohen, wizzyrea
Version: master   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: ---
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 14203 - Message for non-existent lang removal
[SIGNED OFF] Bug 14203 - Message for non-existent lang removal
Bug 14203 - Message for non-existent lang removal
[PASSED QA] Bug 14203 - Message for non-existent lang removal

Description Indranil Das Gupta 2015-05-14 19:24:56 UTC
An attempt to remove a language that is not installed shows the message:

"Error: the selected language is not already installed."

I feel that it will be nicer to just have "Error: the selected language is not installed."
Comment 1 Indranil Das Gupta 2015-05-14 19:47:54 UTC Comment hidden (obsolete)
Comment 2 Nick Clemens 2015-05-14 20:57:06 UTC Comment hidden (obsolete)
Comment 3 Jonathan Druart 2015-05-22 14:52:00 UTC
Created attachment 39422 [details] [review]
Bug 14203 - Message for non-existent lang removal

A trivial string patch to update the error message displayed to
user if koha-translate is used to attempt removal of a language
that is not installed.

Test plan
=========

1/ attempt to remove a non-existent language by
   <installdir>/debian/scripts/koha-translate --remove <langcode>
2/ it should show "Error: the selected language is not already
   installed."
3/ apply patch
4/ repeat step 1; it should show "Error: the selected language is
   not installed."

Signed-off-by: Nick Clemens <nick@quecheelibrary.org>

Signed-off-by: Jonathan Druart <jonathan.druart@koha-community.org>
Comment 4 Kyle M Hall 2015-05-22 14:55:03 UTC Comment hidden (obsolete)
Comment 5 Kyle M Hall 2015-05-22 14:55:38 UTC
Comment on attachment 39423 [details] [review]
[PASSED QA] Bug 14203 - Message for non-existent lang removal

Obsoleting because Jonathan beat me to it!
Comment 6 Tomás Cohen Arazi 2015-05-26 14:07:07 UTC
Patch pushed to master.

Thanks Indranil!
Comment 7 Chris Cormack 2015-05-27 08:09:59 UTC
Pushed to 3.20.x will be in 3.20.1
Comment 8 Liz Rea 2015-06-22 22:36:31 UTC
Patch pushed to 3.18.x will be in 3.18.08
Comment 9 Mason James 2015-07-25 09:45:41 UTC
Pushed to 3.16.x, will be in 3.16.13