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
"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.