You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@ofbiz.apache.org by "Jacques Le Roux (JIRA)" <ji...@apache.org> on 2019/05/15 09:49:00 UTC
[jira] [Commented] (OFBIZ-11019) Cleaning the bloated ‘Security’ interface
[ https://issues.apache.org/jira/browse/OFBIZ-11019?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16840246#comment-16840246 ]
Jacques Le Roux commented on OFBIZ-11019:
-----------------------------------------
Hi Mathieu,
I don't see a urgent need for that but indeed why not cleaning stuff. The patch applies on trunk HEAD and looks good to me (I did not test).
BTW, I agree with Michael about deprecation rules and "notations": https://markmail.org/message/c3pukn7dg4q5nzqz
> Cleaning the bloated ‘Security’ interface
> -----------------------------------------
>
> Key: OFBIZ-11019
> URL: https://issues.apache.org/jira/browse/OFBIZ-11019
> Project: OFBiz
> Issue Type: Improvement
> Components: framework
> Affects Versions: Trunk
> Reporter: Mathieu Lirzin
> Assignee: Mathieu Lirzin
> Priority: Minor
> Attachments: OFBIZ-11019_Relax-the-contract-of-the-Security-interfac.patch
>
>
> The ‘hasPermission(String, HttpSession)’ method declaration from the ‘Security’ interface is breaking the minimality principle of interfaces because it is easily expressible in term of ‘hasPermission(String, GenericEntity)’. As a consequence a static helper method should be implemented to achieve same convenience without polluting the ‘Security’ interface.
> Other methods in that interface are suffering from the same issue.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)