BXC-4412 basic set command #84

krwong · 2024-02-26T18:17:29Z

PermissionsCommand: add set command
PermissionMappingOption: add --cdm-id option
PermissionsService: add setPermissions method, updateCsvRecords helper method, previousStatePermissions instance variable
add tests

src/main/java/edu/unc/lib/boxc/migration/cdm/services/PermissionsService.java

src/test/java/edu/unc/lib/boxc/migration/cdm/services/PermissionsServiceTest.java

… helper method, small fixes

src/main/java/edu/unc/lib/boxc/migration/cdm/services/PermissionsService.java

…etIds

src/main/java/edu/unc/lib/boxc/migration/cdm/services/PermissionsService.java

krwong added 2 commits February 26, 2024 13:10

add set command and tests, add id option

d668c2a

remove unused code

fa98632

bbpennel requested changes Feb 27, 2024

View reviewed changes

remove previousStatePermissions instance variable, add getPermissions…

62b13c3

… helper method, small fixes

bbpennel approved these changes Feb 27, 2024

View reviewed changes

fix query, use prepared stmt, add tests for setting grouped work

cb2bbd0

bbpennel reviewed Feb 28, 2024

View reviewed changes

src/main/java/edu/unc/lib/boxc/migration/cdm/services/PermissionsService.java Outdated Show resolved Hide resolved

move prepared statement to doesIdExistInIndex helper method, revert g…

2095e03

…etIds

bbpennel reviewed Feb 28, 2024

View reviewed changes

src/main/java/edu/unc/lib/boxc/migration/cdm/services/PermissionsService.java Outdated Show resolved Hide resolved

krwong added 2 commits February 28, 2024 15:26

add groupa:group11 in grouped_gilmer, fix tests

774987c

remove unused code

e047045

bbpennel approved these changes Feb 28, 2024

View reviewed changes

bbpennel merged commit 65aa414 into main Feb 29, 2024
2 checks passed

bbpennel deleted the BXC-4412-perms-set-cmd branch February 29, 2024 19:28

Provide feedback