Details

    • Branch Version/s:
      6.1.x
    • Backported to Branch:
      Committed
    • Similar Issues:
      Show 5 results 

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

        Activity

        Hide
        Michael Saechang added a comment -

        Committed on:
        Portal 6.2.x GIT ID: 42c228e50e5ce0c1575dde0bf33db7699c413c92.

        Show
        Michael Saechang added a comment - Committed on: Portal 6.2.x GIT ID: 42c228e50e5ce0c1575dde0bf33db7699c413c92.

          People

          • Assignee:
            Michael Saechang
            Reporter:
            Roman
            Recent user:
            Michael Saechang
            Participants of an Issue:
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:
              Days since last comment:
              2 years, 33 weeks ago

              Development

                Structure Helper Panel