[Koha-patches] [PATCH 05/54] MT3138 : Adding permissions on viewlog

[paul.poulain at biblibre.com]

From: Paul Poulain <paul.poulain at biblibre.com>

    Adding a new permission view_borrowers_log which depends on circulate
    Adding them to circulation menu (added in another patch with something else)

(updatedatabase later)

(written by hdl)
---
 .../prog/en/includes/members-menu.inc              |    2 +-
 tools/viewlog.pl                                   |    9 ++++++++-
 2 files changed, 9 insertions(+), 2 deletions(-)

diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/members-menu.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/members-menu.inc
index 513d415..e7aba8a 100644
--- a/koha-tmpl/intranet-tmpl/prog/en/includes/members-menu.inc
+++ b/koha-tmpl/intranet-tmpl/prog/en/includes/members-menu.inc
@@ -6,7 +6,7 @@
 	<!-- TMPL_IF NAME="finesview" --><li class="active"><!-- TMPL_ELSE --><li><!-- /TMPL_IF --><a href="/cgi-bin/koha/members/boraccount.pl?borrowernumber=<!-- TMPL_VAR NAME="borrowernumber" -->">Fines</a></li>
 	<!-- /TMPL_IF -->
 	<!-- TMPL_IF NAME="intranetreadinghistory" --><!-- TMPL_IF NAME="readingrecordview" --><li class="active"><!-- TMPL_ELSE --><li><!-- /TMPL_IF --><a href="/cgi-bin/koha/members/readingrec.pl?borrowernumber=<!-- TMPL_VAR NAME="borrowernumber" -->">Circulation History</a></li><!-- /TMPL_IF -->
-	<!-- TMPL_IF NAME="CAN_user_parameters"--><!-- TMPL_IF NAME="logview" --><li class="active"><!-- TMPL_ELSE --><li><!-- /TMPL_IF --><a href="/cgi-bin/koha/tools/viewlog.pl?do_it=1&amp;modules=MEMBERS&amp;action=MODIFY&amp;object=<!-- TMPL_VAR NAME="borrowernumber" -->">Modification Log</a></li><!-- /TMPL_IF -->
+	<!-- TMPL_IF NAME="CAN_user_circulate_view_borrowers_logs"--><!-- TMPL_IF NAME="logview" --><li class="active"><!-- TMPL_ELSE --><li><!-- /TMPL_IF --><a href="/cgi-bin/koha/tools/viewlog.pl?do_it=1&amp;modules=MEMBERS&amp;action=MODIFY&amp;object=<!-- TMPL_VAR NAME="borrowernumber" -->">Modification Log</a></li><!-- /TMPL_IF -->
 <!-- TMPL_IF NAME="EnhancedMessagingPreferences" -->
           <!-- TMPL_IF NAME="messagingview" --><li class="active"><!-- TMPL_ELSE --><li><!-- /TMPL_IF --><a href="/cgi-bin/koha/members/messaging.pl?borrowernumber=<!-- TMPL_VAR NAME="borrowernumber" -->">Messaging</a></li>
         <!-- /TMPL_IF -->
diff --git a/tools/viewlog.pl b/tools/viewlog.pl
index 84c2474..88c83d8 100755
--- a/tools/viewlog.pl
+++ b/tools/viewlog.pl
@@ -57,13 +57,20 @@ my $basename = $input->param("basename");
 my $output   = $input->param("output") || "screen";
 my $src      = $input->param("src");    # this param allows us to be told where we were called from -fbcit
 
+my $permissionrequired;
+if ($src){
+    $permissionrequired={ circulate => 'view_borrowers_logs' };
+}
+else {
+    $permissionrequired={ tools => 'view_system_logs' };
+}
 my ( $template, $borrowernumber, $cookie ) = get_template_and_user(
     {
         template_name   => "tools/viewlog.tmpl",
         query           => $input,
         type            => "intranet",
         authnotrequired => 0,
-        flagsrequired   => { tools => 'view_system_logs' },
+        flagsrequired   => $permissionrequired,
         debug           => 1,
     }
 );
-- 
1.7.1