[jira] Created: (OFBIZ-1043) ContentPermissionServices.viewContentPermission needs to set checkId

Previous Topic Next Topic
 
classic Classic list List threaded Threaded
2 messages Options
Reply | Threaded
Open this post in threaded view
|

[jira] Created: (OFBIZ-1043) ContentPermissionServices.viewContentPermission needs to set checkId

Nicolas Malin (Jira)
ContentPermissionServices.viewContentPermission needs to set checkId
--------------------------------------------------------------------

                 Key: OFBIZ-1043
                 URL: https://issues.apache.org/jira/browse/OFBIZ-1043
             Project: OFBiz (The Open for Business Project)
          Issue Type: Bug
          Components: content
    Affects Versions: SVN trunk
            Reporter: Al Byers
            Assignee: Al Byers


Is there any reason why ContentPermission.viewContentPermission does not need the same code as updateContentPermission that sets "checkId"? Below is some code where I have added that code.

If this is a bug, I guess it should also be applied to the release branch?

-Al

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

Reply | Threaded
Open this post in threaded view
|

[jira] Updated: (OFBIZ-1043) ContentPermissionServices.viewContentPermission needs to set checkId

Nicolas Malin (Jira)

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

Al Byers updated OFBIZ-1043:
----------------------------

    Attachment: ContentPermissionServices.diff

Here is the proposed patch

> ContentPermissionServices.viewContentPermission needs to set checkId
> --------------------------------------------------------------------
>
>                 Key: OFBIZ-1043
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-1043
>             Project: OFBiz (The Open for Business Project)
>          Issue Type: Bug
>          Components: content
>    Affects Versions: SVN trunk
>            Reporter: Al Byers
>            Assignee: Al Byers
>         Attachments: ContentPermissionServices.diff
>
>
> Is there any reason why ContentPermission.viewContentPermission does not need the same code as updateContentPermission that sets "checkId"? Below is some code where I have added that code.
> If this is a bug, I guess it should also be applied to the release branch?
> -Al

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