| Summary: | Remove unused table-pager include | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Owen Leonard <oleonard> |
| Component: | Templates | Assignee: | Owen Leonard <oleonard> |
| Status: | CLOSED FIXED | QA Contact: | Jonathan Druart <jonathan.druart> |
| Severity: | normal | ||
| Priority: | P5 - low | CC: | jonathan.druart |
| 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: | ||
| Circulation function: | |||
| Bug Depends on: | 9443 | ||
| Bug Blocks: | |||
| Attachments: |
Bug 9925 - Remove unused table-pager include
Bug 9925 - Remove unused table-pager include Bug 9925 - Remove unused table-pager include |
||
|
Description
Owen Leonard
2013-03-25 17:00:12 UTC
Created attachment 17309 [details] [review] Bug 9925 - Remove unused table-pager include While the tablesorter plugin has not been completely eliminated from the staff client yet, there are no more instances where it requires the include file for showing paging controls. table-pager.inc can be safely removed. To test, apply the patch and confirm that the file is gone. A search of Koha templates for references to table-pager.inc should return no results. Created attachment 17384 [details] [review] Bug 9925 - Remove unused table-pager include While the tablesorter plugin has not been completely eliminated from the staff client yet, there are no more instances where it requires the include file for showing paging controls. table-pager.inc can be safely removed. To test, apply the patch and confirm that the file is gone. A search of Koha templates for references to table-pager.inc should return no results. Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> QA comment: This patch just remove an unused file. The following command returns no result: git grep table-pager | grep -v translator Marked as Passed QA. Created attachment 17607 [details] [review] Bug 9925 - Remove unused table-pager include While the tablesorter plugin has not been completely eliminated from the staff client yet, there are no more instances where it requires the include file for showing paging controls. table-pager.inc can be safely removed. To test, apply the patch and confirm that the file is gone. A search of Koha templates for references to table-pager.inc should return no results. Signed-off-by: Kyle M Hall <kyle@bywatersolutions.com> This patch has been pushed to master and 3.12.x. |