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

Improve eZ Demo menu to order by menu location sorting preference

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: High High
    • Resolution: Duplicate
    • Affects Version/s: 5.3.2, 5.4.0-beta1
    • Fix Version/s: QA tracked issues
    • Component/s: DemoBundle
    • Labels:

      Description

      Add two folders "1" and "2" to the root level. Change the Ordering to "Name" "Descending"
      you'll notice the ordered presented in eZ Demo menu is still "1" "2"

        Issue Links

          Activity

          Hide
          Douglas Hammond added a comment - - edited

          The old code sorted by Priority and so does the new. I don't think the top menu in ez Demo ever respected the ordering(by priority, name, date) in the admin but has always been by priority.

          Old
          'sortClauses' => array( new SortClause\LocationPriority( Query::SORT_ASC ) )

          New
          'sortClauses' => array( new SortClause\Location\Priority( LocationQuery::SORT_ASC ) )

          Perhaps we need a new sortclause ContentOrder or in getTopMenuContent we will have to get the ordering from the $rootLocationId and create a sortclause based on that.

          Show
          Douglas Hammond added a comment - - edited The old code sorted by Priority and so does the new. I don't think the top menu in ez Demo ever respected the ordering(by priority, name, date) in the admin but has always been by priority. Old 'sortClauses' => array( new SortClause\LocationPriority( Query::SORT_ASC ) ) New 'sortClauses' => array( new SortClause\Location\Priority( LocationQuery::SORT_ASC ) ) Perhaps we need a new sortclause ContentOrder or in getTopMenuContent we will have to get the ordering from the $rootLocationId and create a sortclause based on that.
          Hide
          André Rømcke added a comment -

          Closing this as this is intended to be solved when we switch to KnpMenu for menu generation.

          Show
          André Rømcke added a comment - Closing this as this is intended to be solved when we switch to KnpMenu for menu generation.

            People

            • Assignee:
              Unassigned
              Reporter:
              Pedro Resende (Inactive)
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: