@@ -524,29 +524,20 @@ public AbstractUser removeRole(AbstractUser user, String roleString) {
524524
525525 @ Override
526526 public void removeRoleOfDeletedPetriNet (PetriNet petriNet ) {
527- String defaultRealm = collectionNameProvider .getDefaultRealmCollection ();
528- Pageable pageable = PageRequest .of (0 , paginationProperties .getBackendPageSize ());
529- Page <AbstractUser > users ;
530- do {
531- users = findAllByProcessRoles (petriNet .getRoles ().values ().stream ().map (ProcessRole ::get_id ).collect (Collectors .toSet ()), defaultRealm , pageable );
532- users .forEach (u -> {
533- petriNet .getRoles ().forEach ((k , role ) -> removeRole (u , role .get_id ()));
534- });
535- pageable = pageable .next ();
536- } while (users .hasNext ());
527+ removeRoleOfDeletedPetriNet (new HashSet <>(petriNet .getRoles ().values ()));
537528 }
538529
539530 @ Override
540531 public void removeRoleOfDeletedPetriNet (Set <ProcessRole > petriNetRoles ) {
541- String defaultRealm = collectionNameProvider .getDefaultRealmCollection ();
532+ String defaultRealmCollection = collectionNameProvider .getDefaultRealmCollection ();
542533 Pageable pageable = PageRequest .of (0 , paginationProperties .getBackendPageSize ());
534+ Collection <ProcessResourceId > roleIds = petriNetRoles .stream ().map (ProcessRole ::get_id ).collect (Collectors .toSet ());
543535 Page <AbstractUser > users ;
544536 do {
545- users = findAllByProcessRoles (petriNetRoles .stream ().map (ProcessRole ::get_id ).collect (Collectors .toSet ()), defaultRealm , pageable );
546- users .forEach (u -> {
547- petriNetRoles .forEach (role -> removeRole (u , role .get_id ()));
548- });
549- }while (users .hasNext ());
537+ users = searchUsersByRoleIds (roleIds , defaultRealmCollection , pageable );
538+ users .getContent ().forEach (u -> removeRoles (u , petriNetRoles ));
539+ pageable = pageable .next ();
540+ } while (users .hasNext ());
550541 }
551542
552543 @ Override
0 commit comments