VKCOM / vk-api-schema

JSON Schema of VK API
MIT License
207 stars 63 forks source link

Unknown `groups.getMembers` responses schema #221

Open elias506 opened 2 years ago

elias506 commented 2 years ago

Have this responses for groups.getMembers method:

"responses": {
        "response": {
          "$ref": "responses.json#/definitions/groups_getMembers_response"
        },
        "fieldsResponse": {
          "$ref": "responses.json#/definitions/groups_getMembers_fields_response"
        },
        "filterResponse": {
          "$ref": "responses.json#/definitions/groups_getMembers_filter_response"
        }
      },

But, as far as I understand, fields like in groups_getMembers_fields_response will be returned by fields in request in array, there is no reason for separate response. And filters field just set filters for response, no reason for separate response

In doc https://dev.vk.com/method/groups.getMembers has completely different result description

FeeeeK commented 2 years ago

На самом деле тут все куда сложнее. Ответ сервера зависит от параметров. Мне лень искать все эти объекты, поэтому просто добавлю скриншоты уже сгенерированного по схеме кода.

FeeeeK commented 2 years ago

Просьба к тем, кто генерирует схему и не собирается менять этот метод: назовите респонсы с учетом имен этих параметров, как вы делали в некоторых методах. Пусть это и очень неочевидная хрень, но раз до меня уже дошло, что раз параметра из названия респонса нет в методе, то это должно быть значением какого-либо другого параметра, то так и оставьте, не придумывайте что-то более сложное и запутанное.