Lennart Regebro wrote:

> Jean-Marc Orliaguet wrote:
>
>> +                    user_append = result[user].append
>>                      for dict in merged[user]:
>> -                        result[user].append(
>> +                        user_append(
>
>
> I'm not to fond of renaming methods unless there is a good reason,
> Otherwise it look fine.
>
Hi,

It's an optimization alias.


What bugs me though is that the getRolesInContext() seems to be wrong
(I'm doing a rollback on this one).

The original issue is that the CPSWorkflow guard
StackDefinitionGuard.check() ignores the roles associated with
'role:authenticated'

Julien hinted at the fact it could have something to do with
getRolesInContext() but apparently the problem is somewhere else.
maybe getGroupsInContext() ?

If the 'role:authenticated' group has the 'WorkspaceMember' role
getRolesInContext(object) now returns 'WorkspaceMember' in the list of
roles. It  allows the create transition, but local user roles get messed
up.

/JM


_______________________________________________
cps-devel mailing list
http://lists.nuxeo.com/mailman/listinfo/cps-devel

Reply via email to