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

[Advanced search] Fix TagQuery & TagsSystemProperty

    • Icon: Improvement Improvement
    • Resolution: Fixed
    • Icon: Major Major
    • 4.2.0
    • 4.2.0
    • Plugin : Search
    • None
    • 4.2 RC1

      In advanced search, if i select the criterion 'tags' (which is multiple) and 2 values (let's call them A and B for instance):

      • If I select the operator 'equal', then the results are all contents which contain the tag A or the tag B
        First question: I think it is right, but do you agree with this behavior ?
        The generated Solr Query is : tags:A OR tags:B
      • If I select the operator 'not-equal', then the results are all contents which does not contain either A, neither B
        Second question: Do you agree with this behavior ?
        The generated Solr Query is : *:* -tags:A OR *:* -tags:B

      As you can see, there are missing parenthesis in the generated query. So if the behavior is good, it is by accident (in the TagQuery the logical operator is OR)

      Thus, I propose the following PR with a fix in the TagQuery AND a fix in the TagsSystemProperty to keep the existing behavior.

      Please can we discuss those points and the PR ?

          [CMS-9673] [Advanced search] Fix TagQuery & TagsSystemProperty

          Issue there:
          The operator "different of" is not working as intended. It actually display results as if the search was "equal to".

          User agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/72.0.3626.121 Safari/537.36

          Thibaut Bergé (Inactive) added a comment - Issue there: The operator "different of" is not working as intended. It actually display results as if the search was "equal to". User agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/72.0.3626.121 Safari/537.36

            cedric Cédric Damioli
            sprieul Simon Prieul (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated:
              Resolved: