ilgrosso commented on code in PR #1127:
URL: https://github.com/apache/syncope/pull/1127#discussion_r2184858405


##########
core/persistence-neo4j/src/main/java/org/apache/syncope/core/persistence/neo4j/dao/repo/GroupRepoExt.java:
##########
@@ -50,6 +51,8 @@ public interface GroupRepoExt extends AnyRepoExt<Group> {
 
     List<UMembership> findUMemberships(Group group);

Review Comment:
   Remove this method (see above).



##########
core/persistence-jpa/src/main/java/org/apache/syncope/core/persistence/jpa/dao/repo/GroupRepoExt.java:
##########
@@ -50,6 +51,8 @@ public interface GroupRepoExt extends AnyRepoExt<Group> {
 
     List<UMembership> findUMemberships(Group group);
 
+    List<UMembership> findUMemberships(Group group, Pageable pageable);

Review Comment:
   Please remove the existing  method
   
   ```java
   List<UMembership> findUMemberships(Group group);
   ```
   
   and replace all existing invocations
   
   ```findUMembership(group)```
   
   with
   
   ```findUMembership(group, Pageable.unpaged())```
   
   Naturally, you will need to adjust the new code to handle the case when 
`Pageable.unpaged()` is passed as an argument.



##########
core/persistence-api/src/main/java/org/apache/syncope/core/persistence/api/dao/GroupDAO.java:
##########
@@ -60,6 +61,8 @@ public interface GroupDAO extends AnyDAO<Group> {
 
     List<UMembership> findUMemberships(Group group);

Review Comment:
   Remove this method (see below).



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscr...@syncope.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to