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


##########
client/idm/console/src/main/java/org/apache/syncope/client/console/status/AnyStatusDirectoryPanel.java:
##########
@@ -378,7 +378,7 @@ protected List<StatusBean> getStatusBeans(final long first, 
final long count) {
                                     actual instanceof GroupTO);
                             statusBean.setLinked(false);
                             return statusBean;
-                        }).collect(Collectors.toList()));
+                        }).toList());

Review Comment:
   Ok thanks.
   The checks are needed especially with the operations that are involving the 
changed classes, so for example searching for Users, Groups or Any Objects, 
merging Linked Accounts, perform some Reconciliation etc.
   
   Any way, let's go ahead and merge, just keep in mind that some 
immutable-related issues might arise and we know where they are from.



-- 
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