Bugzilla – Attachment 17831 Details for
Bug 7228
can't order more than 1 when importing all from staged file
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
[PASSED QA] Bug 7228: can't add items in an order with Modify
PASSED-QA-Bug-7228-cant-add-items-in-an-order-with.patch (text/plain), 3.36 KB, created by
Katrin Fischer
on 2013-04-30 05:42:39 UTC
(
hide
)
Description:
[PASSED QA] Bug 7228: can't add items in an order with Modify
Filename:
MIME Type:
Creator:
Katrin Fischer
Created:
2013-04-30 05:42:39 UTC
Size:
3.36 KB
patch
obsolete
>From 7527580b561c1944c4e77a93806ef31b3401da53 Mon Sep 17 00:00:00 2001 >From: Christophe Croullebois <christophe.croullebois@biblibre.com> >Date: Thu, 18 Apr 2013 12:55:18 +0200 >Subject: [PATCH] [PASSED QA] Bug 7228: can't add items in an order with > Modify > >We have a message if we want to add items and we can't add, substract only. >It's ok if we choose to create items on ordering, in this case koha can't add items, just substract >and in this case we have to delete manually the items(s) in the catalog. >But if via the syspref AcqCreateItem we choose to create items when receiving this limitation is not usefull >The patch just checks if the syspref AcqCreateItem is on 'ordering' >if not the message is not shown and we can add items > >Signed-off-by: Mathieu Saby <mathieu.saby@univ-rennes2.fr> >Here is the test I made for signing off : >after applying the patch >- syspref AcqCreateItem : create items on RECEIVING >- in a basket, create an order (quantity = 1) >- save the order >- reopen the order >- change the quantity (2 instead of 1) >- save the order >=> changing quantity was not possible before the patch > >- syspref AcqCreateItem : create items on CATALOGING >- in a basket, create an order (quantity = 1) >- save the order >- reopen the order >- change the quantity (2 instead of 1) >- save the order >=> changing quantity was not possible before the patch > >- syspref AcqCreateItem : create items on ORDERING >- in a basket, create an order (click on "add" to add an item => quantity = 1) >- save the order >- reopen the order >- try to change the quantity (2 instead of 1), without clicking on "add" to create a new item => you cannot (alert message) >=> the behavior is the same as before the patch > >Signed-off-by: Katrin Fischer <Katrin.Fischer.83@web.de> >Copied test plan from bug report. >Template only change deactivating the Javascript that blocks >you from changing the quantity when AcqCreateItem is set to >something else than 'ordering'. >Passes all tests and QA script. >--- > acqui/neworderempty.pl | 1 + > .../prog/en/modules/acqui/neworderempty.tt | 2 ++ > 2 files changed, 3 insertions(+) > >diff --git a/acqui/neworderempty.pl b/acqui/neworderempty.pl >index b1c9afa..33b061d 100755 >--- a/acqui/neworderempty.pl >+++ b/acqui/neworderempty.pl >@@ -414,6 +414,7 @@ $template->param( > barcode_subfield => $barcode_subfield, > import_batch_id => $import_batch_id, > subscriptionid => $subscriptionid, >+ acqcreate => C4::Context->preference("AcqCreateItem") eq "ordering" ? 1 : "", > (uc(C4::Context->preference("marcflavour"))) => 1 > ); > >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/neworderempty.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/neworderempty.tt >index 549bf62..cc554ed 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/neworderempty.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/acqui/neworderempty.tt >@@ -129,6 +129,7 @@ $(document).ready(function() > > //We apply the fonction only for modify option > [% IF ( quantityrec ) %] >+ [% IF ( acqcreate ) %] > $('#quantity').blur(function() > { > // if user decreases the quantity >@@ -148,6 +149,7 @@ $(document).ready(function() > } > }); > [% END %] >+ [% END %] > > //keep a copy of all budgets before removing the inactives > disabledBudgetsCopy = $('#budget_id').html(); >-- >1.7.9.5
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 7228
:
6314
|
17521
|
17522
|
17678
|
17831
|
17832
|
18129
|
18130