Bug 11490: Split MaxItemsForBatch into 2 prefs to clarify things
Prior to this patch, a single pref (MaxItemsForBatch) was used to restrict the use of the batch item modification and deletion. On the batch modification tool, the pref is used to limit the number of items a librarian is allowed to modify in a batch, but on the batch deletion tool the pref is used to limit the number of items to display. To clarify things, this patch split the pref into 2 new prefs: MaxItemsToDisplayForBatchDel and MaxItemsToProcessForBatchMod. Later we could add a MaxItemsToProcessForBatchDel if needed. Note that this patch should not introduce any changes in the current behaviors. Test plan: 0/ Do not execute the update DB entry 1/ Set a value != 1000 for MaxItemsForBatch 2/ Execute the update DB entry and confirm that the 2 new prefs are initially set with the value of MaxItemsForBatch and that MaxItemsForBatch has been deleted 3/ Set different values for these 2 prefs (2 and 3 for instance to ease the next steps) 4/ Try to delete less than MaxItemsToDisplayForBatchDel items => You must see the items details 5/ Try to delete more than MaxItemsToDisplayForBatchDel items => You must not see the items details but are allowed to delete them 6/ Try to modify less than MaxItemsToProcessForBatchMod items => You must see the items details 7/ Try to modify more than MaxItemsToProcessForBatchMod items => You must be restricted and not allowed to process Signed-off-by: Laurence Rault <laurence.rault@biblibre.com> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com>
This commit is contained in:
parent
20f4235048
commit
6b2c55ecdd
2 changed files with 8 additions and 4 deletions
|
@ -1,3 +1,4 @@
|
|||
[% USE Koha %]
|
||||
[% INCLUDE 'doc-head-open.inc' %]
|
||||
<title>Koha › Tools › Batch item modification</title>
|
||||
[% INCLUDE 'doc-head-close.inc' %]
|
||||
|
@ -163,7 +164,7 @@ $(document).ready(function(){
|
|||
[% IF ( show ) %]
|
||||
|
||||
[% IF ( too_many_items ) %]
|
||||
<p>Too many items ([% too_many_items %]): not displaying each one individually.</p>
|
||||
<p>Too many items ([% too_many_items %]): You are not allowed to edit more than [% Koha.Preference('MaxItemsToProcessForBatchMod') %] items in a batch.</p>
|
||||
[% FOREACH itemnumber IN itemnumbers_array %]
|
||||
<input type="hidden" name="itemnumber" value="[% itemnumber %]" />
|
||||
[% END %]
|
||||
|
|
|
@ -115,7 +115,7 @@ if ($op eq "action") {
|
|||
# Once the job is done
|
||||
if ($completedJobID) {
|
||||
# If we have a reasonable amount of items, we display them
|
||||
if (scalar(@itemnumbers) <= ( C4::Context->preference("MaxItemsForBatch") // 1000 ) ) {
|
||||
if (scalar(@itemnumbers) <= ( C4::Context->preference("MaxItemsToDisplayForBatchDel") // 1000 ) ) {
|
||||
$items_display_hashref=BuildItemsData(@itemnumbers);
|
||||
} else {
|
||||
# Else, we only display the barcode
|
||||
|
@ -270,8 +270,11 @@ if ($op eq "show"){
|
|||
|
||||
# Flag to tell the template there are valid results, hidden or not
|
||||
if(scalar(@itemnumbers) > 0){ $template->param("itemresults" => 1); }
|
||||
# Only display the items if there are no more than pref MaxItemsForBatch
|
||||
if (scalar(@itemnumbers) <= ( C4::Context->preference("MaxItemsForBatch") // 1000 ) ) {
|
||||
# Only display the items if there are no more than pref MaxItemsToProcessForBatchMod or MaxItemsToDisplayForBatchDel
|
||||
my $max_items = $del
|
||||
? C4::Context->preference("MaxItemsToDisplayForBatchDel")
|
||||
: C4::Context->preference("MaxItemsToProcessForBatchMod");
|
||||
if (scalar(@itemnumbers) <= ( $max_items // 1000 ) ) {
|
||||
$items_display_hashref=BuildItemsData(@itemnumbers);
|
||||
} else {
|
||||
$template->param("too_many_items" => scalar(@itemnumbers));
|
||||
|
|
Loading…
Reference in a new issue