Uploaded image for project: 'OFBiz'
  1. OFBiz
  2. OFBIZ-5920

Incorrect use of if-has-permission tag

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: Release Branch 13.07, Trunk
    • Fix Version/s: 14.12.01, 12.04.06, 13.07.02
    • Component/s: ecommerce
    • Labels:
      None

      Description

      action value used in if-has-permission tag is incorrect in some places. Ideally action value should be start with "" but some places it used without ""

      ForumScreens.xml#ViewForumMessage

      <if-has-permission permission="CONTENTMGR" action="CREATE"/>
      

      Due to this if user has correct permission then also it return false and user will not be able to view the screen.

        Activity

        Hide
        deepak.dixit Deepak Dixit added a comment -

        Here is the patch for this issue, found some more places where if-has-permission tag used in different way, will create a new ticket to fix the if-has-permission tags and will used the consistent pattern.

        Show
        deepak.dixit Deepak Dixit added a comment - Here is the patch for this issue, found some more places where if-has-permission tag used in different way, will create a new ticket to fix the if-has-permission tags and will used the consistent pattern.
        Hide
        jacques.le.roux Jacques Le Roux added a comment -

        Thanks Deepak,

        Your patch is in
        trunk r1647266
        R13.07 r1647267
        R12.04 r1647268

        I wonder if this patch does not fix some other already reported issues, anyway it's good!

        Show
        jacques.le.roux Jacques Le Roux added a comment - Thanks Deepak, Your patch is in trunk r1647266 R13.07 r1647267 R12.04 r1647268 I wonder if this patch does not fix some other already reported issues, anyway it's good!

          People

          • Assignee:
            jacques.le.roux Jacques Le Roux
            Reporter:
            deepak.dixit Deepak Dixit
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development