Attachment #108725 for bug #19889

View | Details | Raw Unified | Return to bug 19889
Collapse All | Expand All

Lines 409-416 sub MapItemsToHoldRequests { Link Here

(-)a/C4/HoldsQueue.pm (-2 / +2 lines)
409	next	409	next
410	if ( !$item->{holdallowed} )	410	if ( !$item->{holdallowed} )
411	\|\| ( $item->{holdallowed} == 1	411	\|\| ( $item->{holdallowed} == 1
412	&& $item->{homebranch} ne $request->{borrowerbranch}	412	&& $item->{homebranch} ne $request->{borrowerbranch} )
413	\|\| $item->{_object}->exclude_from_local_holds_priority );	413	\|\| $item->{_object}->exclude_from_local_holds_priority;
414		414
415	next if $request->{itemnumber} && $request->{itemnumber} != $item->{itemnumber};	415	next if $request->{itemnumber} && $request->{itemnumber} != $item->{itemnumber};
416		416

Lines 51-57 my ($template, $loggedinuser, $cookie) = get_template_and_user( Link Here

(-)a/catalogue/moredetail.pl (-1 / +1 lines)
51	);	51	);
52		52
53	$template->param(	53	$template->param(
54	updated_exclude_from_local_holds_priority => $query->param('updated_exclude_from_local_holds_priority')	54	updated_exclude_from_local_holds_priority => scalar($query->param('updated_exclude_from_local_holds_priority'))
55	);	55	);
56		56
57	if($query->cookie("holdfor")){	57	if($query->cookie("holdfor")){

Lines 56-62 for ($damaged,$itemlost,$withdrawn) { Link Here

(-)a/catalogue/updateitem.pl (-3 / +3 lines)
56	}	56	}
57	}	57	}
58		58
59	my $alerts = q{};	59	my $messages = q{};
60		60
61	# modify MARC item if input differs from items table.	61	# modify MARC item if input differs from items table.
62	if ( $op eq "set_non_public_note" ) {	62	if ( $op eq "set_non_public_note" ) {
Lines 76-82 elsif ( $op eq "set_public_note" ) { # i.e., itemnotes parameter passed from for Link Here
76	$item->withdrawn($withdrawn);	76	$item->withdrawn($withdrawn);
77	} elsif ( $op eq "set_exclude_priority" && $exclude_from_local_holds_priority ne $item_data_hashref->{'exclude_from_local_holds_priority'}) {	77	} elsif ( $op eq "set_exclude_priority" && $exclude_from_local_holds_priority ne $item_data_hashref->{'exclude_from_local_holds_priority'}) {
78	$item->exclude_from_local_holds_priority($exclude_from_local_holds_priority);	78	$item->exclude_from_local_holds_priority($exclude_from_local_holds_priority);
79	$alerts = "updated_exclude_from_local_holds_priority=$exclude_from_local_holds_priority&";	79	$messages = "updated_exclude_from_local_holds_priority=$exclude_from_local_holds_priority&";
80	} elsif ( $op eq "set_damaged" && $damaged ne $item_data_hashref->{'damaged'}) {	80	} elsif ( $op eq "set_damaged" && $damaged ne $item_data_hashref->{'damaged'}) {
81	$item->damaged($damaged);	81	$item->damaged($damaged);
82	} else {	82	} else {
Lines 89-92 $item->store; Link Here
89		89
90	LostItem($itemnumber, 'moredetail') if $op eq "set_lost";	90	LostItem($itemnumber, 'moredetail') if $op eq "set_lost";
91		91
92	print $cgi->redirect("moredetail.pl?" . $alerts . "biblionumber=$biblionumber&itemnumber=$itemnumber#item$itemnumber");	92	print $cgi->redirect("moredetail.pl?" . $messages . "biblionumber=$biblionumber&itemnumber=$itemnumber#item$itemnumber");

Lines 436-441 Link Here

(-)a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/categories.tt (+8 lines)
436	<th scope="col">Check previous checkout?</th>	436	<th scope="col">Check previous checkout?</th>
437	[% END %]	437	[% END %]
438	<th scope="col">Default privacy</th>	438	<th scope="col">Default privacy</th>
		439	<th scope="col">Exclude from local holds priority</th>
439	<th scope="col">Actions</th>	440	<th scope="col">Actions</th>
440	</tr>	441	</tr>
441	</thead>	442	</thead>
Lines 549-554 Link Here
549	Forever	550	Forever
550	[% END %]	551	[% END %]
551	</td>	552	</td>
		553	<td>
		554	[% IF category.exclude_from_local_holds_priority %]
		555	Yes
		556	[% ELSE %]
		557	No
		558	[% END %]
		559	</td>
552	<td class="actions">	560	<td class="actions">
553	<a class="btn btn-default btn-xs" href="/cgi-bin/koha/admin/categories.pl?op=add_form&categorycode=[% category.categorycode \|uri %]"><i class="fa fa-pencil"></i> Edit</a>	561	<a class="btn btn-default btn-xs" href="/cgi-bin/koha/admin/categories.pl?op=add_form&categorycode=[% category.categorycode \|uri %]"><i class="fa fa-pencil"></i> Edit</a>
554	<a class="btn btn-default btn-xs" href="/cgi-bin/koha/admin/categories.pl?op=delete_confirm&categorycode=[% category.categorycode \|uri %]"><i class="fa fa-trash"></i> Delete</a>	562	<a class="btn btn-default btn-xs" href="/cgi-bin/koha/admin/categories.pl?op=delete_confirm&categorycode=[% category.categorycode \|uri %]"><i class="fa fa-trash"></i> Delete</a>

Lines 61-67 Link Here

(-)a/koha-tmpl/intranet-tmpl/prog/en/modules/catalogue/moredetail.tt (-1 / +5 lines)
61	[% IF updated_exclude_from_local_holds_priority.defined %]	61	[% IF updated_exclude_from_local_holds_priority.defined %]
62	<div class="dialog message">	62	<div class="dialog message">
63	<i>Exclude from local holds priority</i> updated to	63	<i>Exclude from local holds priority</i> updated to
64	[% IF updated_exclude_from_local_holds_priority %]<i>Yes</i>[% ELSE %]<i>No</i>[% END %]	64	[% IF updated_exclude_from_local_holds_priority %]
		65	<i>Yes</i>
		66	[% ELSE %]
		67	<i>No</i>
		68	[% END %]
65	</div>	69	</div>
66	[% END %]	70	[% END %]
67		71

Lines 341-347 $(document).ready(function(){ Link Here

(-)a/koha-tmpl/intranet-tmpl/prog/en/modules/tools/batchMod-edit.tt (-1 / +1 lines)
341	</ol>	341	</ol>
342	</fieldset>	342	</fieldset>
343	<fieldset class="rows">	343	<fieldset class="rows">
344	<legend>Priority</legend>	344	<legend>Other attributes</legend>
345	<ol>	345	<ol>
346	<li>	346	<li>
347	<div class="subfield_line">	347	<div class="subfield_line">




	my $job = C4::BackgroundJob->fetch($sessionID, $completedJobID);

	# Calling the template
        add_saved_job_results_to_template($template, $completedJobID, $items_display_hashref);

    } else {
    # While the job is getting done

	    }
        }

        my $yesno = Koha::AuthorisedValues->search({category => 'YES_NO'});
        my $ynhash = {};

        while(my $yn = $yesno->next) {
            $ynhash->{'av'.$yn->authorised_value} = $yn->lib;
        }

        try {
            my $schema = Koha::Database->new->schema;
            $schema->txn_do(
                sub {
                    # For each item
                    my $i = 1;
                    my $extra_headers = {};
                    foreach my $itemnumber (@itemnumbers) {
                        $job->progress($i) if $runinbackground;
                        my $item = Koha::Items->find($itemnumber);

                        }
                        else {
                            my $modified_holds_priority = 0;
                            if (defined $exclude_from_local_holds_priority) {
                                if($item->exclude_from_local_holds_priority != $exclude_from_local_holds_priority) {
                                $item->exclude_from_local_holds_priority($exclude_from_local_holds_priority)->store;
                                $modified_holds_priority = 1;
                            }
                                $extra_headers->{exclude_from_local_holds_priority} = {name => 'Exclude from local holds priority', items => {}} unless defined $extra_headers->{exclude_from_local_holds_priority};
                                $extra_headers->{exclude_from_local_holds_priority}->{items}->{$item->itemnumber} = $ynhash->{'av'.$item->exclude_from_local_holds_priority};
                            }
                            my $modified = 0;
                            if ( $values_to_modify || $values_to_blank ) {

                                    {
                                        modified_items  => $modified_items,
                                        modified_fields => $modified_fields,
                                        extra_headers => $extra_headers,
                                    }
                                );
                            }

sub add_saved_job_results_to_template {
    my $template = shift;
    my $completedJobID = shift;
    my $items_display_hashref= shift;
    my $job = C4::BackgroundJob->fetch($sessionID, $completedJobID);
    my $results = $job->results();
    add_results_to_template($template, $results);

    my $fields = $job->get("modified_fields");
    my $items = $job->get("modified_items");
    my $extra_headers = $job->get("extra_headers");

    foreach my $header (keys %{$extra_headers}) {
        push @{$items_display_hashref->{item_header_loop}}, {header_value => $extra_headers->{$header}->{name}};
        foreach my $row (@{$items_display_hashref->{item_loop}}) {
            push @{$row->{item_value}}, {field => $extra_headers->{$header}->{items}->{$row->{itemnumber}}};
        }
    }

    $template->param(
        modified_items => $items,
        modified_fields => $fields,
- 

Return to bug 19889

Lines 154-160 if ($op eq "action") { Link Here

(-)a/tools/batchMod.pl (-3 / +25 lines)
154	my $job = C4::BackgroundJob->fetch($sessionID, $completedJobID);	154	my $job = C4::BackgroundJob->fetch($sessionID, $completedJobID);
155		155
156	# Calling the template	156	# Calling the template
157	add_saved_job_results_to_template($template, $completedJobID);	157	add_saved_job_results_to_template($template, $completedJobID, $items_display_hashref);
158		158
159	} else {	159	} else {
160	# While the job is getting done	160	# While the job is getting done
Lines 186-197 if ($op eq "action") { Link Here
186	}	186	}
187	}	187	}
188		188
		189	my $yesno = Koha::AuthorisedValues->search({category => 'YES_NO'});
		190	my $ynhash = {};
		191
		192	while(my $yn = $yesno->next) {
		193	$ynhash->{'av'.$yn->authorised_value} = $yn->lib;
		194	}
		195
189	try {	196	try {
190	my $schema = Koha::Database->new->schema;	197	my $schema = Koha::Database->new->schema;
191	$schema->txn_do(	198	$schema->txn_do(
192	sub {	199	sub {
193	# For each item	200	# For each item
194	my $i = 1;	201	my $i = 1;
		202	my $extra_headers = {};
195	foreach my $itemnumber (@itemnumbers) {	203	foreach my $itemnumber (@itemnumbers) {
196	$job->progress($i) if $runinbackground;	204	$job->progress($i) if $runinbackground;
197	my $item = Koha::Items->find($itemnumber);	205	my $item = Koha::Items->find($itemnumber);
Lines 233-241 if ($op eq "action") { Link Here
233	}	241	}
234	else {	242	else {
235	my $modified_holds_priority = 0;	243	my $modified_holds_priority = 0;
236	if (defined $exclude_from_local_holds_priority && $item->exclude_from_local_holds_priority != $exclude_from_local_holds_priority) {	244	if (defined $exclude_from_local_holds_priority) {
		245	if($item->exclude_from_local_holds_priority != $exclude_from_local_holds_priority) {
237	$item->exclude_from_local_holds_priority($exclude_from_local_holds_priority)->store;	246	$item->exclude_from_local_holds_priority($exclude_from_local_holds_priority)->store;
238	$modified_holds_priority = 1;	247	$modified_holds_priority = 1;
		248	}
		249	$extra_headers->{exclude_from_local_holds_priority} = {name => 'Exclude from local holds priority', items => {}} unless defined $extra_headers->{exclude_from_local_holds_priority};
		250	$extra_headers->{exclude_from_local_holds_priority}->{items}->{$item->itemnumber} = $ynhash->{'av'.$item->exclude_from_local_holds_priority};
239	}	251	}
240	my $modified = 0;	252	my $modified = 0;
241	if ( $values_to_modify \|\| $values_to_blank ) {	253	if ( $values_to_modify \|\| $values_to_blank ) {
Lines 305-310 if ($op eq "action") { Link Here
305	{	317	{
306	modified_items => $modified_items,	318	modified_items => $modified_items,
307	modified_fields => $modified_fields,	319	modified_fields => $modified_fields,
		320	extra_headers => $extra_headers,
308	}	321	}
309	);	322	);
310	}	323	}
Lines 772-783 sub add_results_to_template { Link Here
772	sub add_saved_job_results_to_template {	785	sub add_saved_job_results_to_template {
773	my $template = shift;	786	my $template = shift;
774	my $completedJobID = shift;	787	my $completedJobID = shift;
		788	my $items_display_hashref= shift;
775	my $job = C4::BackgroundJob->fetch($sessionID, $completedJobID);	789	my $job = C4::BackgroundJob->fetch($sessionID, $completedJobID);
776	my $results = $job->results();	790	my $results = $job->results();
777	add_results_to_template($template, $results);	791	add_results_to_template($template, $results);
778		792
779	my $fields = $job->get("modified_fields");	793	my $fields = $job->get("modified_fields");
780	my $items = $job->get("modified_items");	794	my $items = $job->get("modified_items");
		795	my $extra_headers = $job->get("extra_headers");
		796
		797	foreach my $header (keys %{$extra_headers}) {
		798	push @{$items_display_hashref->{item_header_loop}}, {header_value => $extra_headers->{$header}->{name}};
		799	foreach my $row (@{$items_display_hashref->{item_loop}}) {
		800	push @{$row->{item_value}}, {field => $extra_headers->{$header}->{items}->{$row->{itemnumber}}};
		801	}
		802	}
		803
781	$template->param(	804	$template->param(
782	modified_items => $items,	805	modified_items => $items,
783	modified_fields => $fields,	806	modified_fields => $fields,
784	-