Summary: | Content of ExportRemoveFields is not picked to pre-fill field list | ||
---|---|---|---|
Product: | Koha | Reporter: | Jonathan Druart <jonathan.druart> |
Component: | Architecture, internals, and plumbing | Assignee: | Jonathan Druart <jonathan.druart> |
Status: | CLOSED FIXED | QA Contact: | Marcel de Rooy <m.de.rooy> |
Severity: | normal | ||
Priority: | P5 - low | CC: | julian.maurice, katrin.fischer, kyle, m.de.rooy, mtj |
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: | ||
Bug Depends on: | 13190 | ||
Bug Blocks: | 15498 | ||
Attachments: |
Bug 18136: Fix ExportRemoveFields pre-fill behaviour
Bug 18136: Fix ExportRemoveFields pre-fill behaviour |
Description
Jonathan Druart
2017-02-17 09:55:14 UTC
Created attachment 60384 [details] [review] Bug 18136: Fix ExportRemoveFields pre-fill behaviour Looks like it's caused by bug 13190. Variable export_remove_fields is not sent to the template, let's retrieve the syspref's value using the TT plugin. Test plan: Fill ExportRemoveFields with something (100a for instance) Go on a checkout list page At the bottom, the "do not export fields" input box should be pre-filled with the content of the syspref Created attachment 60397 [details] [review] Bug 18136: Fix ExportRemoveFields pre-fill behaviour Looks like it's caused by bug 13190. Variable export_remove_fields is not sent to the template, let's retrieve the syspref's value using the TT plugin. Test plan: Fill ExportRemoveFields with something (100a for instance) Go on a checkout list page At the bottom, the "do not export fields" input box should be pre-filled with the content of the syspref Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Passing QA now too. This should definitely not wait 4 months :) Kyle: Push it now ! Pushed to master for 17.05, thanks Jonathan! This patch has been pushed to 16.11.x and will be in 16.11.04. Pushed to 3.22.x for 3.22.17 Pushed to 16.05.x, for 16.05.10 release |