Variablesを利用しないで引数を指定した場合に出荷日の登録ができない

[okazy]

Variablesを利用しないで引数を指定した場合に出荷日の登録ができない不具合があります。

文字列から DataTime へは DateTimeType::parseValue() で変換しています。

eccube-api4/GraphQL/Type/Definition/DateTimeType.php

Lines 59 to 67 in e2d2692

	public function parseValue($value)
	{
	$dateTime = DateTime::createFromFormat(DateTime::ATOM, $value);
	if ($dateTime) {
	return $dateTime;
	} else {
	throw new InvalidArgumentException('DateTime parse error, please specify in "Y-m-d\TH:i:sP".'.Utils::printSafe($value));
	}
	}

Mutationのリゾルバ内で引数を利用していますが、リゾルバ実行時点で文字列のまま引数が渡される場合があります。
リクエストの引数の指定の方法の違いでこちらの処理を通らないで文字列のまま場合があるようです。

eccube-api4/GraphQL/Mutation/UpdateShippedMutation.php

Line 130 in e2d2692

public function updateShipped($root, $args)

引数を直接指定する場合

mutation UpdateShipped {
  updateShipped (
    id: 1,
    shipping_date: "2020-05-18T12:57:08+09:00"
  ) {
    id
    shipping_date
  }
}

こちらの場合はリゾルバ実行時点で $args から文字列が取得できます。

引数をvariablesで指定する場合

mutation UpdateShipped (
  $shipping_date: DateTime
){
  updateShipped (
    id: 99,
    shipping_date: $shipping_date
  ) {
    id
    shipping_date
  }
}

{
  "shipping_date": "2020-05-18T12:57:08+09:00"
}

こちらの場合はリゾルバ実行時点で $args から DateTime が取得できます。

リゾルバで文字列か DateTime か調べることで回避できますが、他のクエリでも起こり得ることかと思いますので、原因の追究はしておきたいです。

[okazy]

GraphQLのドキュメントを読んでみましたが、Variablesを利用するかどうかで挙動の違いについての記載はありませんでした。
https://graphql.org/learn/queries/

いったん、文字列がきても DateTime がきても問題がないように修正しようと思います。

[okazy]

#80 で修正されました。