Summary: | Use template wrapper for breadcrumbs: Plugins | ||
---|---|---|---|
Product: | Koha | Reporter: | Owen Leonard <oleonard> |
Component: | Templates | Assignee: | Owen Leonard <oleonard> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | minor | ||
Priority: | P5 - low | CC: | andrew.auld, testopia |
Version: | Main | ||
Hardware: | All | ||
OS: | All | ||
Change sponsored?: | --- | Patch complexity: | Small patch |
Documentation contact: | Documentation submission: | ||
Text to go in the release notes: | Version(s) released in: |
23.05.00,22.11.05
|
|
Circulation function: | |||
Bug Depends on: | 32507 | ||
Bug Blocks: | |||
Attachments: |
Bug 33429: Use template wrapper for breadcrumbs: Plugins
Bug 33429: Use template wrapper for breadcrumbs: Plugins Bug 33429: Use template wrapper for breadcrumbs: Plugins |
Description
Owen Leonard
2023-04-06 10:40:24 UTC
Created attachment 149230 [details] [review] Bug 33429: Use template wrapper for breadcrumbs: Plugins This patch updates plugin management templates so that they use the new WRAPPER for displaying breadcrumbs. To test, apply the patch and test each page and its variations. Breadcrumbs should look correct, and each link should be correct. - In koha-conf.xml, look for "enable_plugins" - If this is set to 0, plugins are disabled: <enable_plugins>0</enable_plugins> - Navigate directly to /cgi-bin/koha/plugins/plugins-home.pl, which should say "Plugins disabled." Check the breadcrumbs on this page. - Enable plugins by changing the settimg in koha-conf.xml to 1 (a restart of services is required): <enable_plugins>1</enable_plugins> - Go to Administration -> Manage plugins - Test this page and the "Upload plugin" page. I notice that on the plugins-home page (page title Plugins > Tools > Koha), the breadcrumb is: home > Tools > Plugins But on the plugins-upload page (page title Upload plugin > Plugins > Tools > Koha), the breadcrumb is: home > Plugins > Upload plugins Is this what you intended? I wasn't able to test the 'plugins-disabled' page. There are some existing inconsistencies in the plugins breadcrumbs which stem from the conflict between Koha's division of "tool plugin" from "other plugin." My patch doesn't attempt to resolve this, it just replaces the markup around what is there. Created attachment 149288 [details] [review] Bug 33429: Use template wrapper for breadcrumbs: Plugins This patch updates plugin management templates so that they use the new WRAPPER for displaying breadcrumbs. To test, apply the patch and test each page and its variations. Breadcrumbs should look correct, and each link should be correct. - In koha-conf.xml, look for "enable_plugins" - If this is set to 0, plugins are disabled: <enable_plugins>0</enable_plugins> - Navigate directly to /cgi-bin/koha/plugins/plugins-home.pl, which should say "Plugins disabled." Check the breadcrumbs on this page. - Enable plugins by changing the settimg in koha-conf.xml to 1 (a restart of services is required): <enable_plugins>1</enable_plugins> - Go to Administration -> Manage plugins - Test this page and the "Upload plugin" page. Signed-off-by: David Nind <david@davidnind.com> Created attachment 149301 [details] [review] Bug 33429: Use template wrapper for breadcrumbs: Plugins This patch updates plugin management templates so that they use the new WRAPPER for displaying breadcrumbs. To test, apply the patch and test each page and its variations. Breadcrumbs should look correct, and each link should be correct. - In koha-conf.xml, look for "enable_plugins" - If this is set to 0, plugins are disabled: <enable_plugins>0</enable_plugins> - Navigate directly to /cgi-bin/koha/plugins/plugins-home.pl, which should say "Plugins disabled." Check the breadcrumbs on this page. - Enable plugins by changing the settimg in koha-conf.xml to 1 (a restart of services is required): <enable_plugins>1</enable_plugins> - Go to Administration -> Manage plugins - Test this page and the "Upload plugin" page. Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Katrin Fischer <katrin.fischer.83@web.de> Pushed to master for 23.05. Nice work everyone, thanks! Nice work, thanks everyone! Pushed to 22.11.x for the next release. |