Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

different response by KIND_OBJECT and KIND_LIST #56

Open
LiFeAiR opened this issue Aug 8, 2017 · 2 comments
Open

different response by KIND_OBJECT and KIND_LIST #56

LiFeAiR opened this issue Aug 8, 2017 · 2 comments

Comments

@LiFeAiR
Copy link
Contributor

LiFeAiR commented Aug 8, 2017

Если запросить тип данных и не указать ни одно свойство из объекта выводится ошибка
пример:
запрос:

{"query":"query{landing(id: 53237) {thematic} }","variables":null}

ответ:

{
    "data": {
        "landing": null
    },
    "errors": [
        {
            "message": "You have to specify fields for \"thematic\"",
            "locations": [
                {
                    "line": 1,
                    "column": 27
                }
            ]
        }
    ]
}

НО если так же не заполнить запрос для массива, то все нормально и ошибок нет)
пример:
запрос:

{"query":"query{landing(id: 53237){tariffs}}","variables":null}

ответ:

{"data":{"landing":{"tariffs":[[]]}}}

Хочется какого-то единообразия..

@viniychuk
Copy link
Member

Привет @LiFeAiR . Если ты запрашиваешь объект — ты обязан указать какие именно поля этого объекта ты хочешь получить; Если это массив scalar'ов — то всё должно работать и так.
Совсем другое дело если это массив объектов — тогда будет ошибка. Дай пожалуйста больше информации — структуру своей схемы, чтобы можно было понять есть ли там проблема.

@LiFeAiR
Copy link
Contributor Author

LiFeAiR commented Aug 9, 2017

LandingType
....
 public function build($config)
$config->addFields(
            [
                'id'                     => new NonNullType(new IdType()),
                'tariffs'                => new ListType(new TariffType()),
...

TariffType
...

 public function build($config)
    {
        $config->addFields(
            [
                'id'        => new NonNullType(new IdType()),
                'landingId' => new StringType(),
...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants