Merge pull request #32 from voldyman/master

Initialize aggregations map in AllMatches search, fixes #31

AUTHORS

@@ -8,3 +8,4 @@
 # Please keep the list sorted.
 
 Marty Schoch <[email protected]>
+Akshay Shekher <[email protected]>

search.go

@@ -184,7 +184,8 @@ type AllMatches struct {
 func NewAllMatches(q Query) *AllMatches {
 	return &AllMatches{
 		BaseSearch: BaseSearch{
-			query: q,
+			query:        q,
+			aggregations: make(search.Aggregations),
 		},
 	}
 }

search_test.go

@@ -21,6 +21,7 @@ import (
 	"strconv"
 	"testing"
 
+	"github.com/blugelabs/bluge/search/aggregations"
 	"github.com/blugelabs/bluge/search/highlight"
 
 	"github.com/blugelabs/bluge/analysis/char"
@@ -1261,3 +1262,12 @@ func TestSearchHighlightingWithRegexpReplacement(t *testing.T) {
 		t.Fatal(err)
 	}
 }
+
+func TestAllMatchesWithAggregationIssue31(t *testing.T) {
+	query := NewMatchQuery("bluge").SetField("name")
+	request := NewAllMatches(query)
+
+	// This line would panic because aggregations map was not initialized internally
+	// should not panic with the fix
+	request.AddAggregation("score", aggregations.MaxStartingAt(search.DocumentScore(), 0))
+}