Set permissions doesn't tick the top level check box and doesn't always clear child check boxes. How to reproduce this bug: 1. Go to /cgi-bin/koha/members/member-flags.pl?member=<borroweid> / search for some patron and click set permissions from the more menu. 2. Choose all child checkboxes of some top level checkbox 3. Notice that the top level check box didn't got the tick automatically. 1. 1. Go to /cgi-bin/koha/members/member-flags.pl?member=<borroweid> / search for some patron and click set permissions from the more menu. 2. Choose all child checkboxes of some top level checkbox 3. Check the top level check box 4. Uncheck the top level check box 5. Notice that the child checkboxes didn't become unchecked.
Still valid.
The first half of this report is still valid. I can't reproduce the second half (child checkboxes not becoming unchecked), so I'm modifying the report title.
Created attachment 95702 [details] [review] Bug 14741: Selecting all child permissions doesn't select the top level check box This patch modifies the patron flags page so that manually selecting all the child permissions also checks the checkbox for the parent permission. To test, apply the patch, open a patron record, select More -> Set permissions. - Expand a set of permissions, for instance "Add, modify, and view patron information." - If any boxes are checked, uncheck them. - Check each of the "child" checkboxes ("Add, modify, and view..." and "View patron infos..." ). - When both checkboxes are checked, the "Add, modify, and view" parent permission should be automatically checked.
Created attachment 95751 [details] [review] Bug 14741: Selecting all child permissions doesn't select the top level check box This patch modifies the patron flags page so that manually selecting all the child permissions also checks the checkbox for the parent permission. To test, apply the patch, open a patron record, select More -> Set permissions. - Expand a set of permissions, for instance "Add, modify, and view patron information." - If any boxes are checked, uncheck them. - Check each of the "child" checkboxes ("Add, modify, and view..." and "View patron infos..." ). - When both checkboxes are checked, the "Add, modify, and view" parent permission should be automatically checked. Signed-off-by: Maryse Simard <maryse.simard@inlibro.com>
Created attachment 95791 [details] [review] Bug 14741: Selecting all child permissions doesn't select the top level check box This patch modifies the patron flags page so that manually selecting all the child permissions also checks the checkbox for the parent permission. To test, apply the patch, open a patron record, select More -> Set permissions. - Expand a set of permissions, for instance "Add, modify, and view patron information." - If any boxes are checked, uncheck them. - Check each of the "child" checkboxes ("Add, modify, and view..." and "View patron infos..." ). - When both checkboxes are checked, the "Add, modify, and view" parent permission should be automatically checked. Signed-off-by: Maryse Simard <maryse.simard@inlibro.com> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Thanks everyone, Pushed to master for 19.11.00