| Summary: | Sysprefs not reset by regressions.t | ||
|---|---|---|---|
| Product: | Koha | Reporter: | Jonathan Druart <jonathan.druart> |
| Component: | Test Suite | Assignee: | Jonathan Druart <jonathan.druart> |
| Status: | CLOSED FIXED | QA Contact: | Martin Renvoize (ashimema) <martin.renvoize> |
| Severity: | major | ||
| Priority: | P5 - low | CC: | fridolin.somers, martin.renvoize, nick |
| Version: | unspecified | ||
| Hardware: | All | ||
| OS: | All | ||
| GIT URL: | Initiative type: | --- | |
| Sponsorship status: | --- | Comma delimited list of Sponsors: | |
| Crowdfunding goal: | 0 | Patch complexity: | Trivial patch |
| Documentation contact: | Documentation submission: | ||
| Text to go in the release notes: | Version(s) released in: | ||
| Circulation function: | |||
| Bug Depends on: | 21775 | ||
| Bug Blocks: | |||
| Attachments: |
Bug 21956: (bug 21775 follow-up) Use set_preference to reset the value
Bug 21956: (bug 21775 follow-up) Use set_preference to reset the value |
||
Created attachment 82889 [details] [review] Bug 21956: (bug 21775 follow-up) Use set_preference to reset the value Test plan: Set AudioAlerts to 0 Prove t/db_dependent/selenium/regressions.t => Without this patch the value of the pref is not reset to 0 Skipping SO Created attachment 83344 [details] [review] Bug 21956: (bug 21775 follow-up) Use set_preference to reset the value Test plan: Set AudioAlerts to 0 Prove t/db_dependent/selenium/regressions.t => Without this patch the value of the pref is not reset to 0 Signed-off-by: Martin Renvoize <martin.renvoize@ptfs-europe.com> Simple fix, no typos found.. passes test plan.. Passing QA Awesome work all! Pushed ot master for 19.05 Pushed to 18.11.x for 18.11.01 |
Typo in regressions.t: C4::Context->preference('SearchEngine', $SearchEngine_value); C4::Context->preference('AudioAlerts', $AudioAlerts_value); must be set_preference