Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 5.1.2, 5.2.3
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None
    • Branch Version/s:
      5.2.x, 5.1.x
    • Backported to Branch:
      Committed

      Description

      "layout.static.portlets.all" is an example of a string array (ie comma delimited list of entries) that is customizable from a Hook.

      But hooks should play safely with each other by adding onto the string array instead of overwriting each other.

      Suppose you have test1-hook that has values "hello,world"

      Suppose you have test2-hook that has values "alpha,beta"

      Suppose the portal has a default value of "1_WAR_chatportlet"

      The final value used in PropsValues.LAYOUT_STATIC_PORTLETS_ALL should resolve to:

      1_WAR_chatportlet,hello,world,alpha,beta

      Before the fix, it would either be:

      "hello,world" or "alpha,beta" with no guarantee.

      The order is still not guaranteed, but the values are now guaranteed to be merged.

        Issue Links

          Activity

            People

            • Assignee:
              support-lep@liferay.com SE Support
              Reporter:
              brian.chan Brian Chan
              Participants of an Issue:
              Recent user:
              Randy Zhu (Inactive)
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:
                Days since last comment:
                7 years, 28 weeks, 2 days ago

                Development

                  Subcomponents