Solr
  1. Solr
  2. SOLR-8284

JSON Facet API sort:index causes NPE

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 5.4, 6.0
    • Component/s: Facet Module
    • Labels:
      None

      Description

      sort:index was meant to be a shortcut for sort:"index asc", but this currently causes a NPE.

        Activity

        Hide
        Michael Sun added a comment -

        it's probably because facet.sortVariable is not set if there is no direction set. The patch is uploaded. There is a test added in patch to catch this failure.

        Show
        Michael Sun added a comment - it's probably because facet.sortVariable is not set if there is no direction set. The patch is uploaded. There is a test added in patch to catch this failure.
        Hide
        ASF subversion and git services added a comment -

        Commit 1714641 from Yonik Seeley in branch 'dev/trunk'
        [ https://svn.apache.org/r1714641 ]

        SOLR-8284: fix NPE short form sort:index and sort:count

        Show
        ASF subversion and git services added a comment - Commit 1714641 from Yonik Seeley in branch 'dev/trunk' [ https://svn.apache.org/r1714641 ] SOLR-8284 : fix NPE short form sort:index and sort:count
        Hide
        ASF subversion and git services added a comment -

        Commit 1714642 from Yonik Seeley in branch 'dev/branches/branch_5x'
        [ https://svn.apache.org/r1714642 ]

        SOLR-8284: fix NPE short form sort:index and sort:count

        Show
        ASF subversion and git services added a comment - Commit 1714642 from Yonik Seeley in branch 'dev/branches/branch_5x' [ https://svn.apache.org/r1714642 ] SOLR-8284 : fix NPE short form sort:index and sort:count
        Hide
        Yonik Seeley added a comment -

        Committed. Thanks Michael!

        Show
        Yonik Seeley added a comment - Committed. Thanks Michael!

          People

          • Assignee:
            Yonik Seeley
            Reporter:
            Yonik Seeley
          • Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development