Description
Jonathan Druart
2023-09-20 12:31:11 UTC
Created attachment 155931 [details] [review] Bug 34844: Add manage_item_editor_templates to userpermissions.sql Test Plan: 1) perl /kohadevbox/misc4dev/run_tests.pl --run-db-upgrade-only; prove t/Koha/Auth/Permissions.t 2) Note failure 3) Apply this patch 4) perl /kohadevbox/misc4dev/run_tests.pl --run-db-upgrade-only; prove t/Koha/Auth/Permissions.t 5) Tests pass! Thanks, Kyle! Created attachment 155933 [details] [review] Bug 34844: Add manage_item_editor_templates to userpermissions.sql Test Plan: 1) perl /kohadevbox/misc4dev/run_tests.pl --run-db-upgrade-only; prove t/Koha/Auth/Permissions.t 2) Note failure 3) Apply this patch 4) perl /kohadevbox/misc4dev/run_tests.pl --run-db-upgrade-only; prove t/Koha/Auth/Permissions.t 5) Tests pass! Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> We are missing an atomic update. Created attachment 155994 [details] [review] Bug 34844: Add db update file Created attachment 155995 [details] [review] Bug 34844: Add db update file Created attachment 155996 [details] [review] Bug 34844: Add db update file WARN installer/data/mysql/atomicupdate/bug_34844.pl WARN tidiness The file is less tidy than before (bad/messy lines before: 0, now: 5) * Proving /usr/share/koha/t/Koha/Auth/Permissions.t KO! # Failed test 'Expected permissions generated for superlibrarian' # at /usr/share/koha/t/Koha/Auth/Permissions.t line 286. # Structures begin differing at: # $got->{CAN_user_editcatalogue_manage_item_editor_templates} = Does not exist # $expected->{CAN_user_editcatalogue_manage_item_editor_templates} = '1' # Looks like you failed 1 test of 1. Created attachment 156034 [details] [review] Bug 34844: Add manage_item_editor_templates to userpermissions.sql Test Plan: 1) perl /kohadevbox/misc4dev/run_tests.pl --run-db-upgrade-only; prove t/Koha/Auth/Permissions.t 2) Note failure 3) Apply this patch 4) perl /kohadevbox/misc4dev/run_tests.pl --run-db-upgrade-only; prove t/Koha/Auth/Permissions.t 5) Tests pass! Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Created attachment 156035 [details] [review] Bug 34844: Add db update file JD amended patch: tidy Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Created attachment 156045 [details] [review] Bug 34844: Add manage_item_editor_templates to userpermissions.sql Test Plan: 1) perl /kohadevbox/misc4dev/run_tests.pl --run-db-upgrade-only; prove t/Koha/Auth/Permissions.t 2) Note failure 3) Apply this patch 4) perl /kohadevbox/misc4dev/run_tests.pl --run-db-upgrade-only; prove t/Koha/Auth/Permissions.t 5) Tests pass! Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Created attachment 156046 [details] [review] Bug 34844: Add db update file JD amended patch: tidy Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Pushed to master for 23.11. Nice work everyone, thanks! Small typo at end of atomic update : +++ b/installer/data/mysql/atomicupdate/bug_34844.pl ... + + say $out "Added new permission 'manage_item_editor_templates'"; + }, + } Should be : +}; Pushed to 23.05.x for 23.05.04 Nice work everyone! Pushed to oldstable for 22.11.x |