Attachment 31674 Details for Bug 12937 – Bug 12937 [Follow-up] Deleting suggestions should ask for a confirmation

[patch] Bug 12937 [Follow-up] Deleting suggestions should ask for a confirmation

Bug-12937-Follow-up-Deleting-suggestions-should-as.patch (text/plain), 2.34 KB, created by Owen Leonard on 2014-09-17 12:34:42 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Owen Leonard

Created: 2014-09-17 12:34:42 UTC

Size: 2.34 KB

patch

obsolete

>From 2c74c11d4e2a61d4e2b706b9fda3e34a94b2ce15 Mon Sep 17 00:00:00 2001
>From: Owen Leonard <oleonard@myacpl.org>
>Date: Wed, 17 Sep 2014 08:31:27 -0400
>Subject: [PATCH] Bug 12937 [Follow-up] Deleting suggestions should ask for a
> confirmation
>Content-Type: text/plain; charset="utf-8"
>
>This minor follow-up revises the language a bit: Adding missing question
>marks and using the more standard prompt "Please select at least one..."
>
>To test, follow the previous test plan and confirm that the prompts are
>improved:
>
>- go on the suggestion management page
>- don't select any suggestion and try to delete (check the checkbox and
>  submit the form). Verify that you get an alert.
>- select 1 suggestion and delete it, verify you get a confirmation
>  message
>- select 1+ suggestions and delete it, verify you get a confirmation
>  message
>---
> .../prog/en/modules/suggestion/suggestion.tt       |    6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/suggestion/suggestion.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/suggestion/suggestion.tt
>index 70d00ad..e3096c3 100644
>--- a/koha-tmpl/intranet-tmpl/prog/en/modules/suggestion/suggestion.tt
>+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/suggestion/suggestion.tt
>@@ -141,16 +141,16 @@ $(document).ready(function() {
>         if ( action_delete_selected ) {
>             var suggestions_to_delete = $(this).find("input[name='edit_field']:checked");
>             if ( suggestions_to_delete.length == 0 ) {
>-                alert(_("No suggestion to delete"));
>+                alert(_("Please select at least one suggestion to delete"));
>                 e.preventDefault();
>                 return false;
>             } else if ( suggestions_to_delete.length == 1 ) {
>-                if ( ! confirm(_("Are you sure you want to delete this suggestion")) ) {
>+                if ( ! confirm(_("Are you sure you want to delete this suggestion?")) ) {
>                     e.preventDefault();
>                     return false;
>                 }
>             } else if ( suggestions_to_delete.length > 1 ) {
>-                if ( ! confirm(_("Are you sure you want to delete these suggestions")) ) {
>+                if ( ! confirm(_("Are you sure you want to delete these suggestions?")) ) {
>                     e.preventDefault();
>                     return false;
>                 }
>-- 
>1.7.9.5

Actions: View | Diff | Splinter Review

Attachments on bug 12937: 31668 | 31673 | 31674 | 31684 | 31731 | 31732