Uploaded image for project: 'eZ Publish / Platform'
  1. eZ Publish / Platform
  2. EZP-31950

[3.x] Drop missed dynamic settings injections from the Symfony 5 Container config

    XMLWordPrintable

    Details

    • Epic Name:
      [3.x] Dynamic settings injection cleanup
    • Sprint:
      [3.2] - Sprint 7

      Description

      We still have in our codebase (applies to EZP, EZEE, EC) references to dynamic settings for Semantic configuration and service parameters. Since container is now frozen in Symfony 5, they're not going to work either at all or at least as expected.

      Most of the work has been done when handling EZP-31220, but there are still leftovers in EC codebase and in EZP, EZEE cases we document as configurable, but don't have configured explicitly for clean setup.

      The solution involves either:

      • dynamic replacement on the fly in places where the setting was used
      • subscribe to onScopeChange Platform Event and update properties
      • in case of expressions containing multiple settings in one string, e.g. some path, use eZ\Publish\SPI\SiteAccess\ConfigProcessor to parse such string.
        See ezsystems/ezplatform-kernel#108 for the example solution.[1]

       

       [1] In the next major we need to figure out something else, because this solution is misleading, letting someone believe that dynamic settings injection is still working.

        Attachments

          Activity

            People

            Assignee:
            mikolaj.adamczyk@ez.no Mikołaj Adamczyk
            Reporter:
            andrzej.longosz@ez.no Andrzej Longosz
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated: