Bug 9965

Summary: Improve integration of report plugins with existing reports
Product: Koha Reporter: Kyle M Hall <kyle.m.hall>
Component: ReportsAssignee: Kyle M Hall <kyle.m.hall>
Status: Failed QA --- QA Contact:
Severity: enhancement    
Priority: P5 - low CC: dcook, jonathan.druart, katrin.fischer, paola.rossi
Version: Main   
Hardware: All   
OS: All   
Change sponsored?: --- Patch complexity: Trivial patch
Documentation contact: Documentation submission:
Text to go in the release notes:
Version(s) released in:
Attachments: Bug 9965 - Improve integration of report plugins with existing reports
sign off of the patch

Description Kyle M Hall 2013-04-01 18:31:21 UTC
Right now we have a link to in the Reports module to access all report plugins. It would probably be better for the report plugins were just listed on the reports page directly.
Comment 1 Kyle M Hall 2013-04-01 18:33:40 UTC Comment hidden (obsolete)
Comment 2 Paola Rossi 2013-05-09 12:49:27 UTC
Created attachment 18039 [details] [review]
sign off of the patch
Comment 3 Jonathan Druart 2013-05-23 10:50:13 UTC
QA comment:

1/ There are 3 tabulation characters:
 FAIL	koha-tmpl/intranet-tmpl/prog/en/modules/reports/reports-home.tt
   FAIL	  forbidden patterns
		The patch introduces a forbidden pattern: tabulation character (73)
		The patch introduces a forbidden pattern: withespace character  (74)
		The patch introduces a forbidden pattern: tabulation character (15)

2/ The conditional test in reports-home.tt should be
      [% IF UseKohaPlugins && CAN_user_plugins_report && plugins %]
Otherwise a single h2 title is displayed.

3/ Why do you change "Report plugins" with "Other reports" while a "Others" already exists?

Marked as Failed QA.
Comment 4 David Cook 2022-08-22 02:58:49 UTC
Since we're already doing this for Tools, it would be nice to do it for Reports too.