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

Christopher Blasnik updated ROL-1852:
-------------------------------------

    Attachment: Weblog-member-management.patch

Patch f. ROL-1852



> Weblog member management / role assignment not working
> ------------------------------------------------------
>
>                 Key: ROL-1852
>                 URL: https://issues.apache.org/jira/browse/ROL-1852
>             Project: Roller
>          Issue Type: Bug
>          Components: JPA Backend, User Management
>    Affects Versions: 5.0
>         Environment: jboss-4.2.3, mysql, ubuntu 9.10
>            Reporter: Christopher Blasnik
>            Assignee: Roller Unassigned
>             Fix For: 5.0
>
>         Attachments: Weblog-member-management.patch
>
>
> Reassigning a weblog member's role (Members.java / Members.jsp) does nothing 
> on a "save" operation.
> Reason #1: String sval = getParameter("perm-" + 
> perms.getUser().getUserName()) is queried in stead of " ...getUserId()."
> .) Fixing this caused another error with the JPA  revoke / grant permissions 
> methods: openJPA threw an exception saying that the instance was already 
> deleted and could therefore not be updated.
> The possible identified cause was that the revokeWeblogPermissions method 
> did'nt commit the transaction, therefore the object id was still there. Also, 
> only releasing the transaction made it possible to store the grants in the DB.
> .) Next in line: grantWeblogPermissions IMO used the wrong method: 
> "addAction" instead of "setActions" --> this caused the permissions to be 
> added instead of being replaced (i.e. leading to entries like "-1, admin, 
> post" in roller_permission
> .) I added hashCode / equals methods to ObjectPermission and WeblogPermission 
> (you can scrap those if you don't need them)
> .) WeblogPermission.orm.xml: corrected typos: "true" --> "TRUE"; apparently 
> the queries are case-sensitive (at least on my system) which caused a lot of 
> the permission queries / method not to work at all.
> --> please refer to the added patch file

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

Reply via email to