Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Blocker Blocker
    • Resolution: Fixed
    • Affects Version/s: 1.10.0
    • Fix Version/s: 1.10.1
    • Component/s: Platform > MVC
    • Labels:
      None

      Description

      It may happen that theme paths are not loaded by the design engine.

      This is due to the fact that TwigThemePass compiler pass checking the existence of twig.loader.filesystem service. Problem is that very service being private, it can be inlined by Symfony at compile time (if it's referenced only once), making it unavailable from the container at all.

      The issue occurred with Symfony v2.8.25 (probably occurs with Symfony 3.x as well). Themes for templates won't be parsed at all since the compiler pass won't execute (hence the blocker priority).

      The fix consists on checking ezdesign.twig_theme_loader instead, which decorates twig.loader.filesystem.

      See https://github.com/ezsystems/ezplatform-design-engine/pull/10 (already merged). A version 1.0.1 will be very much appreciated

        Issue Links

          Activity

          Show
          Jérôme Vieilledent added a comment - PR: https://github.com/ezsystems/ezplatform-design-engine/pull/10
          Show
          André Rømcke added a comment - Merged: https://github.com/ezsystems/ezplatform-design-engine/commit/ac1c5547d8a9b2f1a6a2c9108c98e480e7d24644 Tag: https://github.com/ezsystems/ezplatform-design-engine/releases/tag/v1.0.1
          Hide
          Rui Silva (Inactive) added a comment -

          Can you please provide detailed steps on how to reproduce this?
          QA does not have enough information to test this, given the current information.

          Show
          Rui Silva (Inactive) added a comment - Can you please provide detailed steps on how to reproduce this? QA does not have enough information to test this, given the current information.
          Hide
          Rui Silva (Inactive) added a comment -

          As apparently not enough information exists on description or documentation about this scope that allows QA to test this, sanity tests were executed, and no issues have been found around this.
          Approved by QA for 1.10.

          Show
          Rui Silva (Inactive) added a comment - As apparently not enough information exists on description or documentation about this scope that allows QA to test this, sanity tests were executed, and no issues have been found around this. Approved by QA for 1.10.

            People

            • Assignee:
              Unassigned
              Reporter:
              Jérôme Vieilledent
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: