Uploaded image for project: 'CMS'
  1. CMS
  2. CMS-8877

Types of content in search filters should be listed by name and not by identifier

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Won't Fix
    • Icon: Major Major
    • 4.1.0
    • 4.1.0
    • None
    • None
    • IE
    • 4.1 M6

      How to reproduce it :

      • open the search tool
      • launch a research
      • in the filters, expand the list of type of contents
        > to select only article we have to select org.ametys.web.default.Content.article.

      Espected behavior :
      > to select only article we should select "Article"

        1. JIRA-list-type-content.png
          81 kB
          Laure Lopez

          [CMS-8877] Types of content in search filters should be listed by name and not by identifier

          Laurence Aumeunier added a comment -

          Unecessary fix since CMS-8908

          Laurence Aumeunier added a comment - Unecessary fix since CMS-8908

          Simon Prieul (Inactive) added a comment -

          SearchJsonReader:336 does I18nizableText label = searchCriterion.getFacetLabel(value, locale);
          org.ametys.cms.search.ui.model.impl.AbstractSearchUICriterion.getFacetLabel(String, Locale) is (in our case) based on the enumerator.
          But since commit https://code.ametys.org/projects/AMETYS/repos/cms/commits/d667a959937ba1f750ad89dacd3ac7bf914ca241, ContentTypeSystemProperty#getEnumeratorDefinition returns now null (from superclass).

          What do you think ?

          Simon Prieul (Inactive) added a comment - SearchJsonReader:336 does I18nizableText label = searchCriterion.getFacetLabel(value, locale); org.ametys.cms.search.ui.model.impl.AbstractSearchUICriterion.getFacetLabel(String, Locale) is (in our case) based on the enumerator. But since commit https://code.ametys.org/projects/AMETYS/repos/cms/commits/d667a959937ba1f750ad89dacd3ac7bf914ca241 , ContentTypeSystemProperty#getEnumeratorDefinition returns now null (from superclass). What do you think ?

            laurence Laurence Aumeunier
            llopez Laure Lopez
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: