Adding array of geo gives "Input for predicate example.Loc of type scalar is uid"

[commesan]

• What version of Dgraph are you using?
  1.06

When using:

     type loc struct {
		Type   string    `json:"type,omitempty"`
		Coords []float64 `json:"coordinates,omitempty"`
	}

	type Tracker struct {
		Uid  string `json:"uid,omitempty"`
		Name string `json:"example.Name,omitempty"`
		Loc  []loc  `json:"example.Loc,omitempty"`
	}

	t := Tracker{
		Name: "Tom Baker",
		Loc: []loc{
			{
				Type:   "Point",
				Coords: []float64{1.1, 2},
			},
			{
				Type:   "Point",
				Coords: []float64{1.9, 5},
			},
		},
	}

And mapping:

        example.Name: string @index(exact) .
	example.Loc: [geo] @index(geo) .

I get Input for predicate example.Loc of type scalar is uid

When using geo as a list the SetJson parser seems to view the geojson types as nodes #instead of scalars.

[commesan]

Here's a runnable example.

package main

import (
	"context"
	"encoding/json"
	"fmt"
	"github.com/dgraph-io/dgo"
	"github.com/dgraph-io/dgo/protos/api"
	"google.golang.org/grpc"
	"log"
)

type loc struct {
	Type   string    `json:"type,omitempty"`
	Coords []float64 `json:"coordinates,omitempty"`
}

type Tracker struct {
	Uid  string `json:"uid,omitempty"`
	Name string `json:"example.Name,omitempty"`
	Loc  []loc  `json:"example.Loc,omitempty"`
}

func main() {
	conn, err := grpc.Dial("127.0.0.1:9080", grpc.WithInsecure())
	if err != nil {
		log.Fatal("While trying to dial gRPC")
	}
	defer conn.Close()

	dc := api.NewDgraphClient(conn)
	dg := dgo.NewDgraphClient(dc)

	op := &api.Operation{}
	op.Schema = `
	example.Name: string @index(exact) .
	example.Loc: [geo] @index(geo) .
`

	ctx := context.Background()
	err = dg.Alter(ctx, op)
	if err != nil {
		log.Fatal(err)
	}

	t := Tracker{
		Name: "Tom Baker",
		Loc: []loc{
			{
				Type:   "Point",
				Coords: []float64{1.1, 2},
			},
			{
				Type:   "Point",
				Coords: []float64{1.9, 5},
			},
		},
	}

	mu := &api.Mutation{
		CommitNow: true,
	}
	tb, err := json.Marshal(t)
	if err != nil {
		log.Fatal(err)
	}

	mu.SetJson = tb
	_, err = dg.NewTxn().Mutate(ctx, mu)
	if err != nil {
		log.Fatal(err)
	}

	// Assigned uids for nodes which were created would be returned in the resp.AssignedUids map.
	variables := map[string]string{"$name": "Tom Baker"}
	q := `query GetTom($name: string){
	        me(func: eq(example.Name, $name)) {
		        example.Name
		        example.Loc		
	        }
       }`

	resp, err := dg.NewTxn().QueryWithVars(ctx, q, variables)
	if err != nil {
		log.Fatal(err)
	}

	type Root struct {
		Me []Tracker `json:"me"`
	}

	var r Root
	err = json.Unmarshal(resp.Json, &r)
	if err != nil {
		log.Fatal(err)
	}

	fmt.Println(string(resp.Json))
}

This gives: pc error: code = Unknown desc = Input for predicate example.Loc of type scalar is uid

Adding the array of geopoints like this:

{
  set {
    _:tracker0 <example.Name> "Tom Baker" .
    _:tracker0 <example.Loc> "{'type':'Point', 'coordinates':[1.1, 2.0]}"^^<geo:geojson> .
    _:tracker0 <example.Loc> "{'type':'Point', 'coordinates':[1.1, 3.0]}"^^<geo:geojson> .
  }
}

does add them as a list of points.

[commesan]

I tracked the problem down.

Handling the []interface{} case in nquads_from_json.go doesn't check for geotypes and assumes all non basic types should be handled as uid.

I'll write a fix for this and submit a pull request if that's ok?