Affects Version/s: 5.1.2, 5.2.3
Fix Version/s: None
Branch Version/s:5.2.x, 5.1.x
Backported to Branch:Committed
Similar Issues:Show 5 results
LPS-22157 Allow mergeable properties to be overriden by hooks LPS-43957 ServiceBuilder illegal arrayable combination validation LPS-15598 JSON service expects String of arrays as a comma separated String LPS-10628 Allow finder column to select an array of a certain column LPS-38542 Expando Values cannot convert data into arrays if read in from JSON String
"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:
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.