You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@ofbiz.apache.org by "Dennis Balkir (JIRA)" <ji...@apache.org> on 2019/04/15 16:07:00 UTC

[jira] [Commented] (OFBIZ-10925) Wrong permission check in ProductScreens#FindProduct

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

Dennis Balkir commented on OFBIZ-10925:
---------------------------------------

I just removed the permission service completely.
There was the service {{productGenericPermission}} which could have been used instead, but this one calls the simple-method {{checkProductRelatedPermission}}, which then check for the {{CATALOG}} permisson. Since this is already done in the {{CommonProductDecorator}} before, it would not make any sense to do this a sceond time.

> Wrong permission check in ProductScreens#FindProduct
> ----------------------------------------------------
>
>                 Key: OFBIZ-10925
>                 URL: https://issues.apache.org/jira/browse/OFBIZ-10925
>             Project: OFBiz
>          Issue Type: Bug
>          Components: product
>            Reporter: Dennis Balkir
>            Priority: Major
>         Attachments: OFBIZ-10925_ProductScreens.patch
>
>
> The permisson serviceĀ {{acctgAgreementPermissionCheck}} is used in the screen {{FindProduct}} in {{ProductScreens.xml}}. This doesn't really make sense, since there is no connection to the accounting application here.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)