Input for mutation type can only bind data in the last field

[thangchung]

I'm not sure if it is a bug or not, but when I created the input type with multiple fields like

input CreateProductInput {
  name: String!
  price: Float!
  imageUrl: String
  desc: String
}

with mutation type

type Mutation {
  createProduct(message: CreateProductInput!): Product!
}

schema {
  query: Query
  mutation: Mutation
}

on the server side, I have created CoolStoreResolvers class

public class CoolStoreResolvers : ResolverMap
{
    public CoolStoreResolvers(ICoolStoreResolverService resolverService)
    {
        
        this["Mutation"] = new FieldResolverMap
        {
            {"createProduct",  resolverService.CreateProductAsync}
        };
    }
}

and CoolStoreResolverService as

public class CoolStoreResolverService : ICoolStoreResolverService
{
    public async Task<IResolveResult> CreateProductAsync(ResolverContext context)
    {
        var input = context.GetArgument<CreateProductRequest>("message");
		
		// do something		

        return await Task.FromResult(As(product));
    }
}

I tested with graphiql with the test data like

mutation createProduct($message: CreateProductInput!) {
  createProduct(message: $message) {
    name
	price
	desc
  }
}

Input variables as below

{
  "message": {
    "name": "product 1",
    "price": 100,
    "desc": "this is product 1"
  }
}

In the code above, I always got only desc field binding data with the string this is product 1, and the rest of fields such as name, price and imageUrl were null.

I have seen the samples folder from this project only have one field for input type as

input InputMessage {
    content: String!
}

Is it because this library could not bind multiple field data for input type?

I used

<PackageReference Include="tanka.graphql" Version="0.3.1" />
<PackageReference Include="tanka.graphql.server" Version="0.3.1" />

[pekkah]

It should work. I'll check it out later.

[pekkah]

It seems to be working on at least on the executor side

tanka-graphql/tests/graphql.tests/ExecutorFacts.cs

Line 173 in 2f2119a

public async Task Mutation1()

.

I'll check if the SignalR is somehow messing up the serialization of the input variables.

[pekkah]

Found the problem and fixed it. Fix will be in latest beta as soon as the builds finish and I can merge the PR.

[pekkah]

Fixed in 0.3.2

[thangchung]

@pekkah Thank you very much. It works. I can create the inputs at https://github.com/vietnam-devs/coolstore-microservices/blob/master/src/graphql/coolstore.graphql#L10
I made a ref to vietnam-devs/coolstore-microservices#41 for remembering