Details
-
Type:
Regression Bug
-
Status:
Closed
-
Priority:
Critical
-
Resolution: Fixed
-
Affects Version/s: 6.1.1 CE GA2, 6.1.20 EE GA2
-
Fix Version/s: 6.1.X EE, 6.2.0 CE M2
-
Component/s: WCM
-
Labels:
-
Branch Version/s:6.1.x
-
Backported to Branch:Committed
-
Similar Issues:
Description
Hi all.
It seems like a regression issue with portal-ext.properties loading order/overriding:
original code from com.germinus.easyconf.AggregatedProperties class:
stars from 218 line of addIncludedPropertiesSources method:
String[] fileNames = tempConf
.getStringArray(Conventions.INCLUDE_PROPERTY);
for (int i = fileNames.length - 1; i >= 0; i--) { - reversed order
new COPY-PASTED from easyconf Liferay's com.liferay.portal.configuration.easyconf.ClassLoaderAggregateProperties code:
217 line of _addIncludedPropertiesSources method:
for (String fileName : fileNames) { - usual order - HERE is the bug - usual order
It's for loading but for printing LF still uses reversed order:
com.liferay.portal.configuration.ConfigurationImpl
421 line on printSources method:
for (int i = sources.size() - 1; i >= 0; i--) { - same reversed order 
properties overriding doesn't work because of for (String fileName : fileNames).
Committed on:
Portal 6.2.x GIT ID: 42c228e50e5ce0c1575dde0bf33db7699c413c92.