Attributes to fields fix (#163)

* fix the newer version * add a test to make sure we pull the correct information * ensure the datatypes are correct in both redis > 2.2 Co-authored-by: Avital Fine <[email protected]> Co-authored-by: Chayim <[email protected]>
RediSearch · May 3, 2022 · 36a153e · 36a153e
1 parent 9a9429a
commit 36a153e
Show file tree

Hide file tree

Showing 2 changed files with 32 additions and 1 deletion.
diff --git a/redisearch/client.go b/redisearch/client.go
@@ -586,8 +586,14 @@ func (i *Client) Info() (*IndexInfo, error) {
 		switch key {
 		case "index_options":
 			indexOptions, _ = redis.Strings(res[ii+1], nil)
-		case "fields", "attributes":
+		case "fields":
 			schemaAttributes, _ = redis.Values(res[ii+1], nil)
+		case "attributes":
+			for _, attr := range res[ii+1].([]interface{}) {
+				l := len(attr.([]interface{}))
+				schemaAttributes = append(schemaAttributes, attr.([]interface{})[3:l])
+
+			}
 		}
 	}
 

diff --git a/redisearch/client_test.go b/redisearch/client_test.go
@@ -1218,3 +1218,28 @@ func TestClient_InfoSchemaFields(t *testing.T) {
 	}
 	assert.True(t, reflect.DeepEqual(expVectorField, info.Schema.Fields[1]))
 }
+
+func TestClient_InfoFieldsTest(t *testing.T) {
+	c := createClient("ft-info-fields-test")
+	flush(c)
+	schema := NewSchema(DefaultOptions).
+		AddField(NewTextFieldOptions("text", TextFieldOptions{Sortable: true, PhoneticMatcher: PhoneticDoubleMetaphoneEnglish})).
+		AddField(NewGeoField("geo")).
+		AddField(NewNumericField("numeric"))
+	// In this example we will only index keys started by product:
+	indexDefinition := NewIndexDefinition().AddPrefix("ft-info-fields-test:")
+	// Add the Index Definition
+	err := c.CreateIndexWithIndexDefinition(schema, indexDefinition)
+	assert.Nil(t, err)
+
+	info, err := c.Info()
+	assert.Nil(t, err)
+	// Check to make sure the fields that we get back match the fields that we created
+	assert.Equal(t,
+		[]Field(
+			[]Field{
+				Field{Name: "text", Type: 0, Sortable: false, Options: TextFieldOptions{Weight: 1, Sortable: true, NoStem: false, NoIndex: false, PhoneticMatcher: "", As: ""}},
+				Field{Name: "geo", Type: 2, Sortable: false, Options: interface{}(nil)},
+				Field{Name: "numeric", Type: 1, Sortable: false, Options: NumericFieldOptions{Sortable: false, NoIndex: false, As: ""}}}),
+		info.Schema.Fields)
+}