[ 
https://issues.apache.org/jira/browse/SYNCOPE-1243?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16280372#comment-16280372
 ] 

Francesco Chicchiriccò commented on SYNCOPE-1243:
-------------------------------------------------

{quote}
a) int countUDynMembers(Group group);
Is this really necessary? The implementation is just "return 
findUDynMembers(group).size();".
{quote}

Definitely, but implementation should use SQL's {{COUNT(*)}} on it, which is 
(a) lighter and (b) safer (sure to avoid pagination)

{quote}
b) int countADynMembers(Group group);
I'll have to change this to "public int countADynMembers(final Group group, 
final String key)" or else "public Map<String, Integer> countADynMembers(final 
Group group)" - WDYT?
{quote}

Latter sounds better, effectively.

> Add information to GroupTO about user and AnyObject membership counts
> ---------------------------------------------------------------------
>
>                 Key: SYNCOPE-1243
>                 URL: https://issues.apache.org/jira/browse/SYNCOPE-1243
>             Project: Syncope
>          Issue Type: Improvement
>            Reporter: Colm O hEigeartaigh
>            Assignee: Colm O hEigeartaigh
>            Priority: Minor
>             Fix For: 2.0.7, 2.1.0
>
>         Attachments: SYNCOPE-1243.patch
>
>
> If we want to find out how many users or AnyObjects are members of a given 
> group, it's not possible to figure out via the group itself. Instead we need 
> to run a fiql on the users or Any Objects.
> This task is to add the following fields to GroupTO so that we can get this 
> information more easily:
>   - int staticUserMembershipCount
>   - int dynamicUserMembershipCount
>   - Map<String, Integer> staticAnyObjectMembershipCount
>   - Map<String, Integer> dynamicAnyObjectMembershipCount



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

Reply via email to