Attachment 142850 Details for Bug 24616 – Bug 24616: Do not save form if user press cancel

[patch] Bug 24616: Do not save form if user press cancel

0001-Bug-24616-Do-not-save-form-if-user-press-cancel.patch (text/plain), 1.57 KB, created by Emmi Takkinen on 2022-11-01 10:01:28 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Emmi Takkinen

Created: 2022-11-01 10:01:28 UTC

Size: 1.57 KB

patch

obsolete

>From 47a7037dbefb23b28d00bb61fcc1de4c74d023c8 Mon Sep 17 00:00:00 2001
>From: Emmi Takkinen <emmi.takkinen@koha-suomi.fi>
>Date: Tue, 1 Nov 2022 09:21:57 +0200
>Subject: [PATCH] Bug 24616: Do not save form if user press cancel
>
>If one starts to copy a notice but then cancels
>this when pop-up is displayed, changes are still
>saved. This patch fixes this.
>
>To test:
>1. Copy notice to a library where it already exists
>2. Press save and then cancel when pop-up is
>displayed
>=> Form and changes are still saved
>3. Apply this patch.
>4. Repeat steps 1. and 2.
>=> Form is no longer saved
>
>Sponsored-by: Koha-Suomi Oy
>---
> koha-tmpl/intranet-tmpl/prog/js/letter.js | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
>diff --git a/koha-tmpl/intranet-tmpl/prog/js/letter.js b/koha-tmpl/intranet-tmpl/prog/js/letter.js
>index da044f5dd4..c61d167c70 100644
>--- a/koha-tmpl/intranet-tmpl/prog/js/letter.js
>+++ b/koha-tmpl/intranet-tmpl/prog/js/letter.js
>@@ -29,7 +29,7 @@ function checkCodes( new_lettercode, new_branchcode ){
>         }
>     });
> }
>-
>+//This has bug
> function confirmOverwrite( new_lettercode, new_branchcode ){
>     var letter_exists;
>     $.ajax({
>@@ -127,7 +127,7 @@ $(document).ready(function() {
>                     var new_lettercode = $("#code").val();
>                     var new_branchcode = $("#branch").val();
>                     var confirm = confirmOverwrite( new_lettercode, new_branchcode );
>-                    if( confirm && confirm === false){
>+                    if(!confirm){
>                         return false;
>                     }
>                 }
>-- 
>2.34.1
>

Actions: View | Diff | Splinter Review

Attachments on bug 24616: 138242 | 138503 | 142672 | 142850 | 142856 | 145238 | 146148 | 146149 | 146150