Details

    • Type: Regression Bug
    • Status: Closed
    • Priority: Critical
    • Resolution: Fixed
    • Affects Version/s: 6.1.1 CE GA2, 6.1.20 EE GA2
    • Component/s: WCM
    • Labels:
      None
    • Branch Version/s:
      6.1.x
    • Backported to Branch:
      Committed

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

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                michael.saechang Michael Saechang
                Reporter:
                hangman Roman
                Participants of an Issue:
                Recent user:
                Esther Sanz
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Days since last comment:
                  4 years, 41 weeks, 2 days ago