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

Fix permissions at specialpurpose/myportal/widget/PortalAdmScreens.xml

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: Trunk
    • Fix Version/s: 14.12.01, 13.07.03, 16.11.01
    • Component/s: myportal
    • Labels:
      None

      Description

      The permission check at myportal/control/FindPortalPage always fails due to missing permission PORTALPAGE_MAINT that has no demo data.

      If PORTALPAGE_MAINT should remain the permission to be checked then demo data needs to be created for it.

      PORTALPAGE_ADMIN is the only PORTALPAGE_ type permission that exists if PORTALPAGE_MAINT needs to be replaced. However the menu to access the page in the first place uses permission MYPORTALBASE_ADMIN for controlling the display of the link.

      Whatever permission is chosen needs to be consistent with the menu link.

      1. OFBIZ-6400.patch
        0.5 kB
        Aditi Patidar

        Activity

        Hide
        ofbizzer Christian Carlow added a comment -

        If PORTALPAGE_ADMIN remains and is used for permission checking then it needs to be added to the SUPER security group.

        Show
        ofbizzer Christian Carlow added a comment - If PORTALPAGE_ADMIN remains and is used for permission checking then it needs to be added to the SUPER security group.
        Hide
        aditi17 Aditi Patidar added a comment - - edited

        PORTALPAGE_MAINT permission check does not exist, it's using PORTALPAGE_ADMIN permission. Attached patch for assigning PORTALPAGE_ADMIN permission to the SUPER security group.

        Show
        aditi17 Aditi Patidar added a comment - - edited PORTALPAGE_MAINT permission check does not exist, it's using PORTALPAGE_ADMIN permission. Attached patch for assigning PORTALPAGE_ADMIN permission to the SUPER security group.
        Hide
        deepak.dixit Deepak Dixit added a comment -

        Thanks Christian Carlow for reporting the issue and Aditi for providing the patch.
        Slightly modified patch has been committed at

        Trunk at r#1724946
        15.12 at r#1724947
        14.12 at r#1724948

        Show
        deepak.dixit Deepak Dixit added a comment - Thanks Christian Carlow for reporting the issue and Aditi for providing the patch. Slightly modified patch has been committed at Trunk at r#1724946 15.12 at r#1724947 14.12 at r#1724948
        Hide
        deepak.dixit Deepak Dixit added a comment -

        Also added some missing permission for SUPER user group as well.

        Show
        deepak.dixit Deepak Dixit added a comment - Also added some missing permission for SUPER user group as well.
        Hide
        jacques.le.roux Jacques Le Roux added a comment -

        Backported in R13.07 at r1725023

        Show
        jacques.le.roux Jacques Le Roux added a comment - Backported in R13.07 at r1725023

          People

          • Assignee:
            deepak.dixit Deepak Dixit
            Reporter:
            ofbizzer Christian Carlow
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development