Summary: | Desks link is available in left side menu even if UseCirculationDesks is disabled | ||
---|---|---|---|
Product: | Koha | Reporter: | Caroline Cyr La Rose <caroline.cyr-la-rose> |
Component: | System Administration | Assignee: | Katrin Fischer <katrin.fischer> |
Status: | CLOSED FIXED | QA Contact: | Testopia <testopia> |
Severity: | trivial | ||
Priority: | P5 - low | CC: | aleisha, gmcharlt, lucas, nick, sally.healey |
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: |
20.11.00, 20.05.03
|
|
Circulation function: | |||
Attachments: |
Bug 25919: Only show Desks in admin navigation when UseCirculationDesks is used
Bug 25919: Only show Desks in admin navigation when UseCirculationDesks is used Bug 25919: Only show Desks in admin navigation when UseCirculationDesks is used |
Description
Caroline Cyr La Rose
2020-07-02 21:28:06 UTC
Actually, I just turned on the syspref and 'Desks' still does not appear on my main administration page... Maybe it's just my installation that's buggy. Created attachment 107057 [details] [review] Bug 25919: Only show Desks in admin navigation when UseCirculationDesks is used We were missing a check for the UseCirculationDesks in the template so the "Desks" link would only appear in the admin navigation when the pref feature is activated. To test: - Turn off UseCirculationDesks - Go to any administration page that shows the navigation on the left - Verify Desks shows - Apply patch - Reload the admin page - Desks should be gone - Turn on UseCirculationDesks - Verify the Desks reappear Created attachment 107433 [details] [review] Bug 25919: Only show Desks in admin navigation when UseCirculationDesks is used We were missing a check for the UseCirculationDesks in the template so the "Desks" link would only appear in the admin navigation when the pref feature is activated. To test: - Turn off UseCirculationDesks - Go to any administration page that shows the navigation on the left - Verify Desks shows - Apply patch - Reload the admin page - Desks should be gone - Turn on UseCirculationDesks - Verify the Desks reappear Signed-off-by: Sally <sally.healey@cheshiresharedservices.gov.uk> Created attachment 107937 [details] [review] Bug 25919: Only show Desks in admin navigation when UseCirculationDesks is used We were missing a check for the UseCirculationDesks in the template so the "Desks" link would only appear in the admin navigation when the pref feature is activated. To test: - Turn off UseCirculationDesks - Go to any administration page that shows the navigation on the left - Verify Desks shows - Apply patch - Reload the admin page - Desks should be gone - Turn on UseCirculationDesks - Verify the Desks reappear Signed-off-by: Sally <sally.healey@cheshiresharedservices.gov.uk> Signed-off-by: Nick Clemens <nick@bywatersolutions.com> Works as described. One question: even after this I can access the page and define desks if the syspref is disabled - should we block this too? Pushed to master for 20.11, thanks to everybody involved! backported to 20.05.x for 20.05.03 UseCirculationDesks not in 19.11.x, not backported For the sake of the release notes, I'm marking this as a bug rather than an enhancement, as it does not add/modify any functionality. Feel free to change it back if you are opposed to it. |