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

Content view routes layout parameter should accept true-ish values

Details

    • Icon: Bug Bug
    • Resolution: Unresolved
    • Icon: Medium Medium
    • None
    • 2.5.9, 3.0.3
    • Platform > MVC
    • None
    • Candidates for next sprint

    Description

      Discovered when reviewing ezsystems/developer-documentation#1056.
      layout parameter for _ez_content_translation_view and _ez_content_view does not work as it was designed to do.

      The both pairs

      • (/view/content/{contentId}/full/true/{locationId}, /view/content/{contentId}/full/false/{locationId})
      • (/view/content/{contentId}/full/1/{locationId}, /view/content/{contentId}/full/0/{locationId})

      should behave the same, with the preference for literal boolean as being more close to the natural language.

      TBD if this can be solved with Symfony Routing config or just by ugly ifs (or maybe hack-ish json_encode).

      As a follow-up I'd go for deprecating in 3.x int values in favor of true/false and in 4.x enforcing it via Symfony Route parameter requirement.

      Attachments

        Activity

          People

            Unassigned Unassigned
            andrew.longosz@ibexa.co Andrew Longosz
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated: