Attachment #72580 for bug #18606

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




    require Exporter;
    @ISA    = qw( Exporter );
    @EXPORT = qw(
      TransferCollection
    );
}

=head2 TransferCollection

 ( $success, $errorcode, $errormessage ) = TransferCollection( $colId, $colBranchcode );

Transfers a collection to another branch

 Input:
   $colId: id of the collection to be updated
   $colBranchcode: branch where collection is moving to

 Output:
   $success: 1 if all database operations were successful, 0 otherwise
   $errorCode: Code for reason of failure, good for translating errors in templates
   $errorMessage: English description of error

=cut

sub TransferCollection {
    my ( $colId, $colBranchcode ) = @_;

    ## Check for all necessary parameters
    if ( !$colId ) {
        return ( 0, 1, "NO_ID" );
    }
    if ( !$colBranchcode ) {
        return ( 0, 2, "NO_BRANCHCODE" );
    }

    my $dbh = C4::Context->dbh;

    my $sth;
    $sth = $dbh->prepare(
        "UPDATE collections
                        SET 
                        colBranchcode = ? 
                        WHERE colId = ?"
    );
    $sth->execute( $colBranchcode, $colId ) or return ( 0, 4, $sth->errstr() );

    $sth = $dbh->prepare(q{
        SELECT items.itemnumber, items.barcode FROM collections_tracking
        LEFT JOIN items ON collections_tracking.itemnumber = items.itemnumber
        LEFT JOIN issues ON items.itemnumber = issues.itemnumber
        WHERE issues.borrowernumber IS NULL
          AND collections_tracking.colId = ?
    });
    $sth->execute($colId) or return ( 0, 4, $sth->errstr );
    my @results;
    while ( my $item = $sth->fetchrow_hashref ) {
        my ($status) = CheckReserves( $item->{itemnumber} );
        my @transfers = C4::Circulation::GetTransfers( $item->{itemnumber} );
        C4::Circulation::transferbook( $colBranchcode, $item->{barcode}, my $ignore_reserves = 1 ) unless ( $status eq 'Waiting' || @transfers );
    }

    return 1;

}

=head2 isItemInThisCollection

  $inCollection = isItemInThisCollection( $itemnumber, $colId );




package Koha::RotatingCollection;

# Copyright Josef Moravec 2017
#
# This file is part of Koha.
#

use Carp;

use Koha::Database;
use Koha::DateUtils;
use Koha::Exceptions;
use Koha::Holds;
use Koha::Items;
use Koha::RotatingCollection::Trackings;


    return $collection_tracking->delete;
}

=head3 transfer

$collection->transfer( $library_object )

throws
    Koha::Exceptions::MissingParameter
    Koha::Exceptions::ObjectNotFound

=cut

sub transfer {
    my ( $self, $library ) = @_;

    Koha::Exceptions::MissingParameter->throw if not defined $library;

    Koha::Exceptions::ObjectNotFound->throw if ref($library) ne 'Koha::Library';

    $self->colBranchcode( $library->branchcode )->store;

    my $from = C4::Context->userenv->{'branch'} if C4::Context->userenv;

    my $items = $self->items;
    while ( my $item = $items->next ) {
        my $holds = Koha::Holds->search( {
            itemnumber => $item->itemnumber,
            found      => 'W',
        } );


        # If no user context is defined the default from library for transfer will be the 'holding' one
        $from = $item->holdingbranch if not defined $from;
        unless ($holds->count || $item->get_transfer) {
            my $transfer = Koha::Item::Transfer->new( {
                frombranch => $from,
                tobranch => $library->branchcode,
                itemnumber => $item->itemnumber,
                datesent => output_pref({ dt => dt_from_string, dateformat => 'iso', dateonly => 1 }),
                comments => $self->colTitle,
            } )->store;
            $item->holdingbranch( $library->branchcode )->store;
        }
    }
}

=head3 type

=cut




[% INCLUDE 'header.inc' %]
[% INCLUDE 'cat-search.inc' %]

<div id="breadcrumbs"><a href="/cgi-bin/koha/mainpage.pl">Home</a> &rsaquo; <a href="/cgi-bin/koha/tools/tools-home.pl">Tools</a> &rsaquo; <a href="/cgi-bin/koha/rotating_collections/rotatingCollections.pl">Rotating collections</a> &rsaquo; <a href="/cgi-bin/koha/rotating_collections/addItems.pl?colId=[% collection.colId %]">Collection <i>[% collection.colTitle %]</i></a> &rsaquo; Transfer collection</div>

<div id="doc3" class="yui-t2">
    <div id="bd">
        <div id="yui-main">
            <div class="yui-b">

                <h1>Transfer collection <i>[% collection.colTitle %]</i></h1>

                [% IF ( transferSuccess ) %]
                    <div class="dialog message">

                [% ELSE %]
                    <div>
                        <form action="transferCollection.pl" method="post">
                            <input type="hidden" name="colId" value="[% collection.colId %]" />
                            <fieldset class="rows">
                                <ol>
                                    <li>

Lines 21-28 use Modern::Perl; Link Here

(-)a/rotating_collections/transferCollection.pl (-20 / +10 lines)
21	use C4::Output;	21	use C4::Output;
22	use C4::Auth;	22	use C4::Auth;
23	use C4::Context;	23	use C4::Context;
24	use C4::RotatingCollections;
25		24
		25	use Koha::Libraries;
26	use Koha::RotatingCollections;	26	use Koha::RotatingCollections;
27		27
28	use CGI qw ( -utf8 );	28	use CGI qw ( -utf8 );
Lines 32-37 my $query = new CGI; Link Here
32	my $colId = $query->param('colId');	32	my $colId = $query->param('colId');
33	my $toBranch = $query->param('toBranch');	33	my $toBranch = $query->param('toBranch');
34		34
		35	my $collection = Koha::RotatingCollections->find( $colId );
		36
35	my ( $template, $loggedinuser, $cookie ) = get_template_and_user(	37	my ( $template, $loggedinuser, $cookie ) = get_template_and_user(
36	{	38	{
37	template_name => "rotating_collections/transferCollection.tt",	39	template_name => "rotating_collections/transferCollection.tt",
Lines 44-74 my ( $template, $loggedinuser, $cookie ) = get_template_and_user( Link Here
44	);	46	);
45		47
46	## Transfer collection	48	## Transfer collection
47	my ( $success, $errorCode, $errorMessage );
48	if ($toBranch) {	49	if ($toBranch) {
49	( $success, $errorCode, $errorMessage ) =	50	my $branch = Koha::Libraries->find( $toBranch );
50	TransferCollection( $colId, $toBranch );	51
		52	my $transferred = eval ( $collection->transfer( $branch ) );
51		53
52	if ($success) {	54	if ( $@ and not $transferred ) {
		55	$template->param( transferFailure => 1 );
		56	} else {
53	$template->param( transferSuccess => 1 );	57	$template->param( transferSuccess => 1 );
54	}	58	}
55	else {
56	$template->param(
57	transferFailure => 1,
58	errorCode => $errorCode,
59	errorMessage => $errorMessage
60	);
61	}
62	}	59	}
63		60
64	## Get data about collection
65	my $collection = Koha::RotatingCollections->find($colId);
66
67	$template->param(	61	$template->param(
68	colId => $collection->colId,	62	collection => $collection,
69	colTitle => $collection->colTitle,
70	colDesc => $collection->colDesc,
71	colBranchcode => $collection->colBranchcode,
72	);	63	);
73		64
74	output_html_with_http_headers $query, $cookie, $template->output;	65	output_html_with_http_headers $query, $cookie, $template->output;
75	-

Return to bug 18606

Lines 48-115 BEGIN { Link Here

(-)a/C4/RotatingCollections.pm (-59 lines)
48	require Exporter;	48	require Exporter;
49	@ISA = qw( Exporter );	49	@ISA = qw( Exporter );
50	@EXPORT = qw(	50	@EXPORT = qw(
51	TransferCollection
52	);	51	);
53	}	52	}
54		53
55	=head2 TransferCollection
56
57	( $success, $errorcode, $errormessage ) = TransferCollection( $colId, $colBranchcode );
58
59	Transfers a collection to another branch
60
61	Input:
62	$colId: id of the collection to be updated
63	$colBranchcode: branch where collection is moving to
64
65	Output:
66	$success: 1 if all database operations were successful, 0 otherwise
67	$errorCode: Code for reason of failure, good for translating errors in templates
68	$errorMessage: English description of error
69
70	=cut
71
72	sub TransferCollection {
73	my ( $colId, $colBranchcode ) = @_;
74
75	## Check for all necessary parameters
76	if ( !$colId ) {
77	return ( 0, 1, "NO_ID" );
78	}
79	if ( !$colBranchcode ) {
80	return ( 0, 2, "NO_BRANCHCODE" );
81	}
82
83	my $dbh = C4::Context->dbh;
84
85	my $sth;
86	$sth = $dbh->prepare(
87	"UPDATE collections
88	SET
89	colBranchcode = ?
90	WHERE colId = ?"
91	);
92	$sth->execute( $colBranchcode, $colId ) or return ( 0, 4, $sth->errstr() );
93
94	$sth = $dbh->prepare(q{
95	SELECT items.itemnumber, items.barcode FROM collections_tracking
96	LEFT JOIN items ON collections_tracking.itemnumber = items.itemnumber
97	LEFT JOIN issues ON items.itemnumber = issues.itemnumber
98	WHERE issues.borrowernumber IS NULL
99	AND collections_tracking.colId = ?
100	});
101	$sth->execute($colId) or return ( 0, 4, $sth->errstr );
102	my @results;
103	while ( my $item = $sth->fetchrow_hashref ) {
104	my ($status) = CheckReserves( $item->{itemnumber} );
105	my @transfers = C4::Circulation::GetTransfers( $item->{itemnumber} );
106	C4::Circulation::transferbook( $colBranchcode, $item->{barcode}, my $ignore_reserves = 1 ) unless ( $status eq 'Waiting' \|\| @transfers );
107	}
108
109	return 1;
110
111	}
112
113	=head2 isItemInThisCollection	54	=head2 isItemInThisCollection
114		55
115	$inCollection = isItemInThisCollection( $itemnumber, $colId );	56	$inCollection = isItemInThisCollection( $itemnumber, $colId );

Lines 1-6 Link Here

(-)a/Koha/RotatingCollection.pm (-1 / +47 lines)
1	package Koha::RotatingCollection;	1	package Koha::RotatingCollection;
2		2
3	# Copyright Josef Moravec 2016	3	# Copyright Josef Moravec 2017
4	#	4	#
5	# This file is part of Koha.	5	# This file is part of Koha.
6	#	6	#
Lines 22-28 use Modern::Perl; Link Here
22	use Carp;	22	use Carp;
23		23
24	use Koha::Database;	24	use Koha::Database;
		25	use Koha::DateUtils;
25	use Koha::Exceptions;	26	use Koha::Exceptions;
		27	use Koha::Holds;
26	use Koha::Items;	28	use Koha::Items;
27	use Koha::RotatingCollection::Trackings;	29	use Koha::RotatingCollection::Trackings;
28		30
Lines 115-120 sub remove_item { Link Here
115	return $collection_tracking->delete;	117	return $collection_tracking->delete;
116	}	118	}
117		119
		120	=head3 transfer
		121
		122	$collection->transfer( $library_object )
		123
		124	throws
		125	Koha::Exceptions::MissingParameter
		126	Koha::Exceptions::ObjectNotFound
		127
		128	=cut
		129
		130	sub transfer {
		131	my ( $self, $library ) = @_;
		132
		133	Koha::Exceptions::MissingParameter->throw if not defined $library;
		134
		135	Koha::Exceptions::ObjectNotFound->throw if ref($library) ne 'Koha::Library';
		136
		137	$self->colBranchcode( $library->branchcode )->store;
		138
		139	my $from = C4::Context->userenv->{'branch'} if C4::Context->userenv;
		140
		141	my $items = $self->items;
		142	while ( my $item = $items->next ) {
		143	my $holds = Koha::Holds->search( {
		144	itemnumber => $item->itemnumber,
		145	found => 'W',
		146	} );
		147
		148
		149	# If no user context is defined the default from library for transfer will be the 'holding' one
		150	$from = $item->holdingbranch if not defined $from;
		151	unless ($holds->count \|\| $item->get_transfer) {
		152	my $transfer = Koha::Item::Transfer->new( {
		153	frombranch => $from,
		154	tobranch => $library->branchcode,
		155	itemnumber => $item->itemnumber,
		156	datesent => output_pref({ dt => dt_from_string, dateformat => 'iso', dateonly => 1 }),
		157	comments => $self->colTitle,
		158	} )->store;
		159	$item->holdingbranch( $library->branchcode )->store;
		160	}
		161	}
		162	}
		163
118	=head3 type	164	=head3 type
119		165
120	=cut	166	=cut

Lines 8-21 Link Here

(-)a/koha-tmpl/intranet-tmpl/prog/en/modules/rotating_collections/transferCollection.tt (-3 / +3 lines)
8	[% INCLUDE 'header.inc' %]	8	[% INCLUDE 'header.inc' %]
9	[% INCLUDE 'cat-search.inc' %]	9	[% INCLUDE 'cat-search.inc' %]
10		10
11	<div id="breadcrumbs"><a href="/cgi-bin/koha/mainpage.pl">Home</a> &rsaquo; <a href="/cgi-bin/koha/tools/tools-home.pl">Tools</a> &rsaquo; <a href="/cgi-bin/koha/rotating_collections/rotatingCollections.pl">Rotating collections</a> &rsaquo; <a href="/cgi-bin/koha/rotating_collections/addItems.pl?colId=[% colId %]">Collection <i>[% colTitle %]</i></a> &rsaquo; Transfer collection</div>	11	<div id="breadcrumbs"><a href="/cgi-bin/koha/mainpage.pl">Home</a> &rsaquo; <a href="/cgi-bin/koha/tools/tools-home.pl">Tools</a> &rsaquo; <a href="/cgi-bin/koha/rotating_collections/rotatingCollections.pl">Rotating collections</a> &rsaquo; <a href="/cgi-bin/koha/rotating_collections/addItems.pl?colId=[% collection.colId %]">Collection <i>[% collection.colTitle %]</i></a> &rsaquo; Transfer collection</div>
12		12
13	<div id="doc3" class="yui-t2">	13	<div id="doc3" class="yui-t2">
14	<div id="bd">	14	<div id="bd">
15	<div id="yui-main">	15	<div id="yui-main">
16	<div class="yui-b">	16	<div class="yui-b">
17		17
18	<h1>Transfer collection <i>[% colTitle %]</i></h1>	18	<h1>Transfer collection <i>[% collection.colTitle %]</i></h1>
19		19
20	[% IF ( transferSuccess ) %]	20	[% IF ( transferSuccess ) %]
21	<div class="dialog message">	21	<div class="dialog message">
Lines 33-39 Link Here
33	[% ELSE %]	33	[% ELSE %]
34	<div>	34	<div>
35	<form action="transferCollection.pl" method="post">	35	<form action="transferCollection.pl" method="post">
36	<input type="hidden" name="colId" value="[% colId %]" />	36	<input type="hidden" name="colId" value="[% collection.colId %]" />
37	<fieldset class="rows">	37	<fieldset class="rows">
38	<ol>	38	<ol>
39	<li>	39	<li>