portals-jetspeed-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ate Douma (JIRA)" <jetspeed-...@portals.apache.org>
Subject [jira] Updated: (JS2-1153) Problem with GroupManager
Date Wed, 07 Apr 2010 13:13:33 GMT

     [ https://issues.apache.org/jira/browse/JS2-1153?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Ate Douma updated JS2-1153:
---------------------------

    Fix Version/s:     (was: 2.2.2)
                   2.2.1

Hi Francisco, 

I just looked at the mentioned methods in GroupManagerImpl and those indeed are wrong.
Your comments are spot on and indeed your suggested fix is appropriate.
I will commit these shortly, as well as fixing the exact same bug in RoleManagerImpl.
Thanks for catching this and reporting the issue!

> Problem with GroupManager
> -------------------------
>
>                 Key: JS2-1153
>                 URL: https://issues.apache.org/jira/browse/JS2-1153
>             Project: Jetspeed 2
>          Issue Type: Bug
>          Components: Components Core
>    Affects Versions: 2.2.0
>            Reporter: Francisco Javier Rosado
>            Assignee: Ate Douma
>             Fix For: 2.2.1
>
>
> When I make an association between two groups, the methods getGroupsAssociatedTo and
getGroupsAssociatedFrom returns the same result. for example:
> this.groupManager.addGroupToGroup(AA, BB, JetspeedPrincipalAssociationType.IS_PART_OF);
> This returns two records:
> List <Group> lista = this.groupManager.getGroupsAssociatedTo(AA, JetspeedPrincipalAssociationType.IS_PART_OF
);
> List <Group> lista = this.groupManager.getGroupsAssociatedFrom(AA, JetspeedPrincipalAssociationType.IS_PART_OF
);
> This returno no records:
> List <Group> lista = this.groupManager.getGroupsAssociatedTo(BB, JetspeedPrincipalAssociationType.IS_PART_OF
);
> List <Group> lista = this.groupManager.getGroupsAssociatedFrom(BB, JetspeedPrincipalAssociationType.IS_PART_OF
);
> I think the problem is in class [GroupManagerImpl], who extends [BaseJetspeedPrincipalManager].
GroupManagerImpl has "getGroupsAssociatedFrom" and "getGroupsAssociatedTo" methods, but both
call the same method of [BaseJetspeedPrincipalManager]:
> /* (non-Javadoc)
>      * @see org.apache.jetspeed.security.GroupManager#getGroupsAssociatedFrom(org.apache.jetspeed.security.Group,
java.lang.String)
>      */
>     @SuppressWarnings("unchecked")
>     public List<Group> getGroupsAssociatedFrom(Group from, String associationName)
>     {
>         return (List<Group>)getAssociatedFrom(from.getName(), from.getType(), associationName);
>     }
>     
>     /* (non-Javadoc)
>      * @see org.apache.jetspeed.security.GroupManager#getGroupsAssociatedTo(org.apache.jetspeed.security.Group,
java.lang.String)
>      */
>     @SuppressWarnings("unchecked")
>     public List<Group> getGroupsAssociatedTo(Group to, String associationName)
>     {
>         return (List<Group>)getAssociatedFrom(to.getName(), to.getType(), associationName);
>     }
> when "getGroupsAssociatedTo" method must call "BaseJetspeedPrincipalManager.getAssociatedTo",
nor "BaseJetspeedPrincipalManager.getAssociatedFrom".
> Regards
> Francisco Rosado

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: jetspeed-dev-unsubscribe@portals.apache.org
For additional commands, e-mail: jetspeed-dev-help@portals.apache.org


Mime
View raw message