Bug 32535 - BorrowerUnwantedField syspref should not include borrowers.flags
Summary: BorrowerUnwantedField syspref should not include borrowers.flags
Status: RESOLVED FIXED
Alias: None
Product: Koha
Classification: Unclassified
Component: System Administration (show other bugs)
Version: master
Hardware: All All
: P5 - low normal (vote)
Assignee: Katrin Fischer
QA Contact: Marcel de Rooy
URL:
Keywords:
Depends on:
Blocks: 32544
  Show dependency treegraph
 
Reported: 2022-12-27 18:58 UTC by Andrew Fuerste-Henry
Modified: 2023-06-08 22:32 UTC (History)
3 users (show)

See Also:
Change sponsored?: ---
Patch complexity: Trivial patch
Documentation contact:
Documentation submission:
Text to go in the release notes:
Version(s) released in:
23.05.00,22.11.02


Attachments
Bug 32535: Exclude flags from BorrowerUnwantedField system preference (1.60 KB, patch)
2022-12-28 20:42 UTC, Katrin Fischer
Details | Diff | Splinter Review
Bug 32535: Exclude flags from BorrowerUnwantedField system preference (1.66 KB, patch)
2022-12-28 20:52 UTC, ByWater Sandboxes
Details | Diff | Splinter Review
Bug 32535: Exclude flags from BorrowerUnwantedField system preference (1.76 KB, patch)
2023-01-13 10:54 UTC, Marcel de Rooy
Details | Diff | Splinter Review

Note You need to log in before you can comment on or make changes to this bug.
Description Andrew Fuerste-Henry 2022-12-27 18:58:19 UTC
The syspref BorrowerUnwantedField includes a checkbox for "System permissions (flags)." Checking it has no apparent effect; one is still able to edit patron permissions with the box checked. It should be removed from the syspref options.
Comment 1 Katrin Fischer 2022-12-28 20:42:25 UTC
Created attachment 144871 [details] [review]
Bug 32535: Exclude flags from BorrowerUnwantedField system preference

The system preference BorrowerUnwantedField includes a checkbox
for "System permissions (flags)." Checking it has no apparent effect;
one is still able to edit patron permissions with the box checked.
This patch excludes the field from the selectable fields.

To test:
* Search for the BorrowerUnwantedField system prefernce
* Verify that you can check/uncheck System permissions (flags)
* Verify it doesn't have any effect, as permissions are not set
  from the patron edit form, but separately
* Apply patch
* Verify you no longer can check/uncheck the field in the
  system preference
Comment 2 ByWater Sandboxes 2022-12-28 20:52:52 UTC
Created attachment 144873 [details] [review]
Bug 32535: Exclude flags from BorrowerUnwantedField system preference

The system preference BorrowerUnwantedField includes a checkbox
for "System permissions (flags)." Checking it has no apparent effect;
one is still able to edit patron permissions with the box checked.
This patch excludes the field from the selectable fields.

To test:
* Search for the BorrowerUnwantedField system prefernce
* Verify that you can check/uncheck System permissions (flags)
* Verify it doesn't have any effect, as permissions are not set
  from the patron edit form, but separately
* Apply patch
* Verify you no longer can check/uncheck the field in the
  system preference

Signed-off-by: Andrew Fuerste-Henry <andrewfh@dubcolib.org>
Comment 3 Katrin Fischer 2022-12-28 20:54:26 UTC
Thx, Andrew!
Comment 4 Andrew Fuerste-Henry 2022-12-28 20:58:19 UTC
Don't thank me yet! In signing off on this one I thought to look at all the other Unwanted and Mandatory field sysprefs and they all include flags as well. Filed bug 32544.
Comment 5 Katrin Fischer 2022-12-28 20:59:53 UTC
(In reply to Andrew Fuerste-Henry from comment #4)
> Don't thank me yet! In signing off on this one I thought to look at all the
> other Unwanted and Mandatory field sysprefs and they all include flags as
> well. Filed bug 32544.

Maybe you should think about joining the QA team ;)
Comment 6 Marcel de Rooy 2023-01-13 10:54:19 UTC
Created attachment 145261 [details] [review]
Bug 32535: Exclude flags from BorrowerUnwantedField system preference

The system preference BorrowerUnwantedField includes a checkbox
for "System permissions (flags)." Checking it has no apparent effect;
one is still able to edit patron permissions with the box checked.
This patch excludes the field from the selectable fields.

To test:
* Search for the BorrowerUnwantedField system prefernce
* Verify that you can check/uncheck System permissions (flags)
* Verify it doesn't have any effect, as permissions are not set
  from the patron edit form, but separately
* Apply patch
* Verify you no longer can check/uncheck the field in the
  system preference

Signed-off-by: Andrew Fuerste-Henry <andrewfh@dubcolib.org>

Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl>
Comment 7 Martin Renvoize 2023-01-18 15:50:26 UTC
Nice work everyone!

Pushed to 23.05.x for the next release
Comment 8 Jacob O'Mara 2023-01-18 17:06:03 UTC
Nice work, thanks everyone!

Pushed to 22.11.x for the next release.