Bug 40827: Add context to plugin wrapper for report plugins
This patch updates the plugin wrapper to include the reports menu and have the breadcrumbs list reports instead of admin or tools when method="report" To Test: 1. Install the Kitchen Sink Plugin 2. Navigate to Reports->Report plugins->Run Report. 3. Observe the admin menu shows 4. Apply the patch 5. Repeat step 2 6. Observe the report menu shows and the breadcrumbs are for the report module. 7. Check that the tools and configuration show the appropriate tool/admin context. Signed-off-by: David Cook <dcook@prosentient.com.au> Signed-off-by: Lucas Gass <lucas@bywatersolutions.com>
This commit is contained in:
parent
88ffb14749
commit
96bb1ec97d
|
|
@ -15,6 +15,8 @@
|
|||
›
|
||||
[% IF ( method == 'tool' ) %]
|
||||
[% t("Tools") | html %]
|
||||
[% ELSIF (method == 'report' ) %]
|
||||
[% t("Reports") | html %]
|
||||
[% ELSE %]
|
||||
[% t("Administration") | html %]
|
||||
[% END %]
|
||||
|
|
@ -34,6 +36,8 @@
|
|||
[% WRAPPER breadcrumb_item %]
|
||||
[% IF ( method == 'tool' ) %]
|
||||
<a href="/cgi-bin/koha/tools/tools-home.pl">Tools</a>
|
||||
[% ELSIF (method == 'report' ) %]
|
||||
<a href="/cgi-bin/koha/reports/reports-home.pl">Reports</a>
|
||||
[% ELSE %]
|
||||
<a href="/cgi-bin/koha/admin/admin-home.pl">Administration</a>
|
||||
[% END %]
|
||||
|
|
@ -58,6 +62,8 @@
|
|||
<aside>
|
||||
[% IF ( method == 'tool' ) %]
|
||||
[% INCLUDE 'tools-menu.inc' %]
|
||||
[% ELSIF (method == 'report' ) %]
|
||||
[% INCLUDE 'reports-menu.inc' %]
|
||||
[% ELSE %]
|
||||
[% INCLUDE 'admin-menu.inc' %]
|
||||
[% END %]
|
||||
|
|
|
|||
Loading…
Reference in New Issue