• Type: Improvement
• Resolution: Won't Fix
• Priority: Minor
• Fix Version/s: 3.8, 4.0.0
• Affects Version/s: None
• Component/s: None
• Labels:

  None

[CMS-4339] Add sort filters on filtered pages service

Collapse comment: Bérénice Maurel added a comment - 06/Nov/15 15:17

Bérénice Maurel added a comment - 06/Nov/15 15:17

Add a sort on sitemap is under-performance.
The default alphabetical order have already been implemented on this service.

Expand comment: Bérénice Maurel added a comment - 06/Nov/15 15:17

Bérénice Maurel added a comment - 06/Nov/15 15:17 Add a sort on sitemap is under-performance. The default alphabetical order have already been implemented on this service.

Collapse comment: Bérénice Maurel added a comment - 30/Oct/15 14:33

Bérénice Maurel added a comment - 30/Oct/15 14:33

Currently, default page filter service sort criteria is an alphabetical filter on title.
I think, we should let the user choice between : alphabetical or sitemap order.

It should be useful for users who used it as summary.

To do :

• Add a service parameter : alphabetical or sitemap
• Update org.ametys.web.repository.page.actions.SetPageFilterInRequestAttributesAction to call addSortCriteria on PageFilter object.
• Update pipelines which call SetPageFilterInRequestAttributesAction

Note : Filters already set the sort criteria.

Expand comment: Bérénice Maurel added a comment - 30/Oct/15 14:33

Bérénice Maurel added a comment - 30/Oct/15 14:33 Currently, default page filter service sort criteria is an alphabetical filter on title. I think, we should let the user choice between : alphabetical or sitemap order. It should be useful for users who used it as summary. To do : Add a service parameter : alphabetical or sitemap Update org.ametys.web.repository.page.actions.SetPageFilterInRequestAttributesAction to call addSortCriteria on PageFilter object. Update pipelines which call SetPageFilterInRequestAttributesAction Note : Filters already set the sort criteria.

Collapse comment: Raphaël Franchet added a comment - 22/Mar/13 12:15

Raphaël Franchet added a comment - 22/Mar/13 12:15

Or maybe we could at least add a hardcoded sort criteria to do an alphabetical sort on titles.

Expand comment: Raphaël Franchet added a comment - 22/Mar/13 12:15

Raphaël Franchet added a comment - 22/Mar/13 12:15 Or maybe we could at least add a hardcoded sort criteria to do an alphabetical sort on titles.

Collapse comment: Raphaël Franchet added a comment - 22/Mar/13 12:13

Raphaël Franchet added a comment - 22/Mar/13 12:13

The content digest can have many criteria, a sort is necessary.
For pages, you only have the title... I think this is more a rendering issue...

Expand comment: Raphaël Franchet added a comment - 22/Mar/13 12:13

Raphaël Franchet added a comment - 22/Mar/13 12:13 The content digest can have many criteria, a sort is necessary. For pages, you only have the title... I think this is more a rendering issue...