Bug 7996: Correct wrong permissions for modification log
authorNick Clemens <nick@bywatersolutions.com>
Fri, 17 Aug 2018 11:07:41 +0000 (11:07 +0000)
committerNick Clemens <nick@bywatersolutions.com>
Wed, 22 Aug 2018 11:47:10 +0000 (11:47 +0000)
To test:
 1 - Create a staff patron with catalogue,borrowers,and view_system_logs
     permissions
 2 - Log in using that patron
 3 - Go to tools
 4 - Click 'Log viewer'
 5 - You get a blank page
 6 - View borrower circ screen, note you do not see 'Modification log'
 7 - Edit a borrower, note you do not see 'Modification log'
 8 - Apply patch
 9 - 'Log Viewer' should now load correctly
10 - You should see 'Modification log' tab during circ
11 - You should see 'Modification log' tab when editing patron

Signed-off-by: Owen Leonard <oleonard@myacpl.org>
Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de>
Signed-off-by: Nick Clemens <nick@bywatersolutions.com>
koha-tmpl/intranet-tmpl/prog/en/includes/circ-menu.inc
koha-tmpl/intranet-tmpl/prog/en/includes/members-menu.inc
koha-tmpl/intranet-tmpl/prog/en/modules/tools/viewlog.tt

index 03cc9a7..a69d099 100644 (file)
@@ -97,7 +97,7 @@
             [% IF ( holdshistoryview ) %]<li class="active">[% ELSE %]<li>[% END %]<a href="/cgi-bin/koha/members/holdshistory.pl?borrowernumber=[% patron.borrowernumber | html %]">Holds history</a></li>
         [% END %]
     [% END %]
-    [% IF ( CAN_user_parameters ) %]
+    [% IF ( CAN_user_tools_view_system_logs ) %]
         [% IF ( logview ) %]<li class="active">[% ELSE %]<li>[% END %]<a href="/cgi-bin/koha/tools/viewlog.pl?do_it=1&amp;modules=MEMBERS&amp;modules=circulation&amp;object=[% patron.borrowernumber | html %]&amp;src=circ">Modification log</a></li>
     [% END %]
     [% IF CAN_user_borrowers_edit_borrowers %]
index d7d08b5..190bd4f 100644 (file)
@@ -9,7 +9,7 @@
     [% IF ( intranetreadinghistory ) %]
         [% IF ( readingrecordview ) %]<li class="active">[% ELSE %]<li>[% END %]<a href="/cgi-bin/koha/members/readingrec.pl?borrowernumber=[% borrowernumber | html %]">Circulation history</a></li>
     [% END %]
-    [% IF ( CAN_user_parameters ) %]
+    [% IF ( CAN_user_tools_view_system_logs ) %]
         [% IF ( logview ) %]<li class="active">[% ELSE %]<li>[% END %]<a href="/cgi-bin/koha/tools/viewlog.pl?do_it=1&amp;modules=MEMBERS&amp;action=MODIFY&amp;object=[% borrowernumber | html %]">Modification log</a></li>
     [% END %]
        [% IF ( sentnotices ) %]<li class="active">[% ELSE %]<li>[% END %]<a href="/cgi-bin/koha/members/notices.pl?borrowernumber=[% borrowernumber | html %]">Notices</a></li>
index 2656d96..d1df207 100644 (file)
@@ -81,7 +81,7 @@
        <div id="bd">
                <div id="yui-main">
                        <div class="yui-b">
-                [% IF ( CAN_user_reports ) %]
+                [% IF ( CAN_user_tools_view_system_logs ) %]
                                <h1>Browse system logs</h1>
                                <form method="post" action="/cgi-bin/koha/tools/viewlog.pl">
                                        [% IF ( do_it ) %]