Bug 10473 - Max length should be 2 digit for adding multiple copies in add items page
As per the discussion, a prompt on a hard coded soft-limit is
far more acceptable as a solution.
TEST PLAN
---------
0) Back up your DB. -- because a backup is always good!
1) Log in to staff client
2) Navigate to any biblio details
(e.g. cgi-bin/koha/catalogue/detail.pl?biblionumber=#####)
3) Click the 'Edit' dropdown button.
4) Click 'Edit items'.
5) Click 'Add multiple items'
6) Enter a crazy high number (e.g. 999)
7) Click 'Add'
-- Koha just adds it! YIKES!
8) Apply patch
9) Repeat steps 5-7
10) Click 'Cancel'
-- Koha does not add the items.
11) Repeat steps 5-7
12) Click 'Ok'
-- Koha does add the items.
13) run koha QA test tools
14) Restore your DB.
Signed-off-by: Nick Clemens <nick@quecheelibrary.org>
Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com>
Signed-off-by: Tomas Cohen Arazi <tomascohen@gmail.com>
This commit is contained in:
parent
b969aa3fa3
commit
70d1c40a50
|
|
@ -85,6 +85,10 @@ function CheckMultipleAdd(f) {
|
|||
alert(_("Please enter a number of items to create."));
|
||||
return false;
|
||||
}
|
||||
<!-- Add a soft-limit of 99 with a reminder about potential data entry error -->
|
||||
if (f>99) {
|
||||
return confirm(_("More than 99 copies will be added. Should they be added?"));
|
||||
}
|
||||
}
|
||||
function Dopop(link,i) {
|
||||
defaultvalue=document.forms[0].field_value[i].value;
|
||||
|
|
@ -250,7 +254,7 @@ $(document).ready(function() {
|
|||
<input type="submit" name="add_duplicate_submit" value="Add & duplicate" onclick="return Check(this.form)" />
|
||||
<input type="submit" name="add_multiple_copies" value="Add multiple items" onclick="javascript:this.nextSibling.style.visibility='visible';document.f.number_of_copies.focus(); return false;" /><span id="add_multiple_copies_span" style="visibility:hidden">
|
||||
<label for="number_of_copies">Number of items to add : </label>
|
||||
<input type="text" id="number_of_copies" name="number_of_copies" value="" size="2" />
|
||||
<input type="text" id="number_of_copies" name="number_of_copies" value="" placeholder="3"/>
|
||||
<input type="submit" id="add_multiple_copies_submit" name="add_multiple_copies_submit" value="Add" onclick="javascript:return Check(this.form) && CheckMultipleAdd(this.form.number_of_copies.value);" />
|
||||
</span>
|
||||
|
||||
|
|
|
|||
Loading…
Reference in New Issue