Uploaded image for project: 'Solr'
  1. Solr
  2. SOLR-8284

JSON Facet API sort:index causes NPE

        Details

        • Type: Bug
        • Status: Closed
        • Priority: 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

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            michael.sun 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 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
            Permalink
            jira-bot 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
            jira-bot 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
            Permalink
            jira-bot 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
            jira-bot 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
            Permalink
            yseeley@gmail.com Yonik Seeley added a comment -

            Committed. Thanks Michael!

            Show
            yseeley@gmail.com Yonik Seeley added a comment - Committed. Thanks Michael!

              People

              • Assignee:
                yseeley@gmail.com Yonik Seeley
                Reporter:
                yseeley@gmail.com Yonik Seeley
              • Votes:
                0 Vote for this issue
                Watchers:
                4 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Development