Proposed correction: in biblio-view-menu.inc change CAN_user_parameters to CAN_user_tools_view_system_logs. Regards, Janusz --- ../koha-3.2.2/koha-tmpl/intranet-tmpl/prog/en/includes/biblio-view-menu.inc 2010-12-22 01:58:29.000000000 +0200 +++ /usr/share/koha/intranet/htdocs/intranet-tmpl/prog/en/includes/biblio-view-menu.inc 2010-12-28 11:22:51.000000000 +0200 @@ -29,7 +29,7 @@ <ul> <!-- TMPL_IF NAME="issuehistoryview" --><li class="active"><!-- TMPL_ELSE --><li><!-- /TMPL_IF --> <a href="/cgi-bin/koha/catalogue/issuehistory.pl?biblionumber=<!-- TMPL_IF NAME="object" --><!-- TMPL_VAR NAME="object" --><!-- TMPL_ELSE --><!-- TMPL_IF NAME="object" --><!-- TMPL_VAR NAME="object" --><!-- TMPL_ELSE --><!-- TMPL_VAR NAME="biblionumber" --><!-- /TMPL_IF --><!-- /TMPL_IF -->" >Checkout history</a></li> -<!-- 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&modules=CATALOGUING&action=MODIFY&object=<!-- TMPL_VAR NAME="biblionumber"-->">Modification log</a> </li><!-- /TMPL_IF --> +<!-- TMPL_IF NAME="CAN_user_tools_view_system_logs"--><!-- TMPL_IF NAME="logview" --><li class="active"><!-- TMPL_ELSE --><li><!-- /TMPL_IF --><a href="/cgi-bin/koha/tools/viewlog.pl?do_it=1&modules=CATALOGUING&action=MODIFY&object=<!-- TMPL_VAR NAME="biblionumber"-->">Modification log</a> </li><!-- /TMPL_IF --> </ul> </div>
Created attachment 2928 [details] [review] Suggested changes reformatted as a patch Changes tested fine, so I went ahead and signed off at the same time.
Pushed please test
This bug is mentioned in: Reimplement fix for bug 5557 - Link to logs for user with view_system_logs permission http://lists.koha-community.org/pipermail/koha-patches/2010-December/013481.html