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

Screen broken on click ThirdPartyLogin, OAuth2GitHub, GitHubUser, OAuth2LinkedIn and LinkedInUser entities.

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: Trunk
    • Fix Version/s: Trunk
    • Component/s: passport
    • Labels:
      None

      Description

      PassportEntityLabels does not exist, but its reference is used in passport/entitydef/entitymodel.xml

      1. OFBiz9390.png
        25 kB
        Renuka Srishti
      2. OFBIZ-9390.patch
        3 kB
        Renuka Srishti

        Activity

        Hide
        Renuka_Srishti Renuka Srishti added a comment -

        Here is the patch for this issue.

        Show
        Renuka_Srishti Renuka Srishti added a comment - Here is the patch for this issue.
        Hide
        Renuka_Srishti Renuka Srishti added a comment -

        Here is the image to show the issue.

        Show
        Renuka_Srishti Renuka Srishti added a comment - Here is the image to show the issue.
        Hide
        jacques.le.roux Jacques Le Roux added a comment -

        Thanks Renuka,

        Should we not use PassportUiLabels.xml rather than simply removing PassportEntityLabels (or maybe add a PassportEntityLabels file)? Shi Jinghai ?

        Show
        jacques.le.roux Jacques Le Roux added a comment - Thanks Renuka, Should we not use PassportUiLabels.xml rather than simply removing PassportEntityLabels (or maybe add a PassportEntityLabels file)? Shi Jinghai ?
        Hide
        Renuka_Srishti Renuka Srishti added a comment -

        Thanks Jacques,
        I think simply removing PassportEntityLabels is fine, as I have gone through all these entities and there is no need of PassportUiLabels.xml.

        Show
        Renuka_Srishti Renuka Srishti added a comment - Thanks Jacques, I think simply removing PassportEntityLabels is fine, as I have gone through all these entities and there is no need of PassportUiLabels.xml.
        Hide
        shi.jinghai Shi Jinghai added a comment -

        Thanks Renuka!

        I agree with you, your patch is in rev. 1798029. Please check if it's right.

        Show
        shi.jinghai Shi Jinghai added a comment - Thanks Renuka! I agree with you, your patch is in rev. 1798029. Please check if it's right.
        Hide
        jacques.le.roux Jacques Le Roux added a comment -

        Thanks to both,

        We now need to close the issue. Renuka would you like the honor ?

        Show
        jacques.le.roux Jacques Le Roux added a comment - Thanks to both, We now need to close the issue. Renuka would you like the honor ?
        Hide
        Renuka_Srishti Renuka Srishti added a comment -

        Thanks Shi and Thanks Jacques for the honor.

        Show
        Renuka_Srishti Renuka Srishti added a comment - Thanks Shi and Thanks Jacques for the honor.

          People

          • Assignee:
            Renuka_Srishti Renuka Srishti
            Reporter:
            Renuka_Srishti Renuka Srishti
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development