From a5d609ff3cdfdc65e4ed88334596b336997b22a3 Mon Sep 17 00:00:00 2001 From: Jonathan Druart Date: Fri, 5 Aug 2011 10:13:20 +0200 Subject: [PATCH] Bug 6390: Implement 'see mine/my branches/all baskets'-syspref MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Code written by Edgar Fuß , Mathematisches Institut der Uniersität Bonn Signed-off-by: Katrin Fischer - rebased on current master, fixed a conflict in booksellers.pl - reworked permission checks to account for general acq permissions and superlibrarian permission - rephrased text of system preference (Thx to Jared Camins-Esakov) --- acqui/booksellers.pl | 28 ++++++++++++++----- admin/systempreferences.pl | 1 + .../en/modules/admin/preferences/acquisitions.pref | 7 +++++ 3 files changed, 28 insertions(+), 8 deletions(-) diff --git a/acqui/booksellers.pl b/acqui/booksellers.pl index fb6b2f1..2b16fe4 100755 --- a/acqui/booksellers.pl +++ b/acqui/booksellers.pl @@ -59,6 +59,7 @@ use CGI; use C4::Dates qw/format_date/; use C4::Bookseller qw/ GetBookSellerFromId GetBookSeller /; use C4::Members qw/GetMember/; +use C4::Context; my $query = CGI->new; my ( $template, $loggedinuser, $cookie ) = get_template_and_user( @@ -95,6 +96,11 @@ if ($loggedinuser) { $uid = GetMember( borrowernumber => $loggedinuser )->{userid}; } +my $userenv = C4::Context::userenv; +my $viewbaskets = C4::Context->preference('AcqViewBaskets'); + +my $userbranch = $userenv->{branch}; + #build result page my $loop_suppliers = []; @@ -102,19 +108,25 @@ for my $vendor (@suppliers) { my $baskets = get_vendors_baskets( $vendor->{id} ); my $loop_basket = []; + for my $basket ( @{$baskets} ) { - if (( $basket->{authorisedby} - && $basket->{authorisedby} eq $loggedinuser - ) - || haspermission( $uid, { acquisition => q{*} } ) - ) { + my $authorisedby = $basket->{authorisedby}; + my $basketbranch = GetMember( borrowernumber => $authorisedby )->{branchcode}; + + if ($userenv->{'flags'} & 1 || #user is superlibrarian + (haspermission( $uid, { acquisition => q{*} } ) && #user has acq permissions and + ($viewbaskets eq 'all' || #user is allowed to see all baskets + ($viewbaskets eq 'branch' && $authorisedby && $userbranch eq $basketbranch) || #basket belongs to user's branch + ($basket->{authorisedby} && $viewbaskets == 'user' && $authorisedby == $loggedinuser) #user created this basket + ) + ) + ) { for my $date_field (qw( creationdate closedate)) { if ( $basket->{$date_field} ) { - $basket->{$date_field} = - format_date( $basket->{$date_field} ); + $basket->{$date_field} = format_date( $basket->{$date_field} ); } } - push @{$loop_basket}, $basket; + push @{$loop_basket}, $basket; } } diff --git a/admin/systempreferences.pl b/admin/systempreferences.pl index 87ecac4..78d0768 100755 --- a/admin/systempreferences.pl +++ b/admin/systempreferences.pl @@ -76,6 +76,7 @@ my %tabsysprefs; $tabsysprefs{AcqCreateItem}="Acquisitions"; $tabsysprefs{OrderPdfFormat}="Acquisitions"; $tabsysprefs{CurrencyFormat}="Acquisitions"; + $tabsysprefs{AcqViewBaskets}="Acquisitions"; # Admin $tabsysprefs{singleBranchMode} = "Admin"; diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/acquisitions.pref b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/acquisitions.pref index 43a7ad5..70e660d 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/acquisitions.pref +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/preferences/acquisitions.pref @@ -16,6 +16,13 @@ Acquisitions: 1: always ask for confirmation. 2: do not ask for confirmation. - + - Show baskets + - pref: AcqViewBaskets + choices: + user: created by staff member. + branch: from staff member's branch. + all: in system, regardless of owner. + - - Display currencies using the following format - pref: CurrencyFormat choices: -- 1.7.4.1