Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix EZP-25960: Double layout using legacy /layout/set module and LegacyKernelController #59

Merged
merged 1 commit into from
Jul 18, 2016

Commits on Jul 5, 2016

  1. Fix EZP-25960: Double layout using legacy /layout/set module and Lega…

    …cyKernelController
    
    > https://jira.ez.no/browse/EZP-25960
    
    When using `/layout/set` legacy module through `LegacyKernelController`
    having a module layout configured, result from legacy kernel is rendered
    using this module layout.
    However, when using `/layout/set`, you already define a layout in legacy
    stack (via `layout.ini`. Problem is that the result from `/layout/set` will
    be embedded in the configured module layout, which can give very
    unexpected results.
    Best example is that when you want to render XML, JSON or anything not
    HTML, you'll have the result embedded in some HTML layout.)
    
    This patch adds `_ezpublishLegacyLayoutSet` route which corresponds to
    `/layout/set` module, and ensures that if this route is used through
    legacy kernel, custom layout won't be used.
    This fixes the usual behabior of layout module.
    lolautruche committed Jul 5, 2016
    Configuration menu
    Copy the full SHA
    7caeb7c View commit details
    Browse the repository at this point in the history