AndrewShmig / Vkontakte-iOS-SDK-LV

[iOS] Powerful & flexible Vkontakte iOS SDK Latest Version
79 stars 23 forks source link

Версия API Вконтакте #80

Closed ghost closed 10 years ago

ghost commented 10 years ago

Для многих методов отличается результат в зависимости от версии API, было бы не плохо иметь возможность указывать эту версию.

AndrewShmig commented 10 years ago

@alaija спасибо, действительно хорошее замечание. попробуем в ближайшее время реализовать.

AndrewShmig commented 10 years ago

@alaija вы знаете, я немного стормозил. Вы можете передавать параметр версии АПИ методу, как собственной любой другой параметр метода.

ghost commented 10 years ago

Мне кажется было бы логично сделать это глобально при создании RequestManager'a ибо сегодня апи работает так, завтра поменяют и без этого параметра начнет работать по другому.

AndrewShmig commented 10 years ago

@alaija в таком случае СДК будет слишком зависима от наименования параметров и прочих условий каких-то. Ведь, если я "впишу" в RequestManager еще и выбор версии, то достаточно им изменить наименование параметра, как приложение либо будет падать, либо некорректно работать... в итоге я получу сообщения, что что-то отвалилось и тд и тп, а это излишняя работа по перепроверке.

Всегда есть возможность просто форкнуть и добавить нужный функционал :)

Я запомню вашу просьбу, если еще кто обратится с подобной, тогда подумаем.

ghost commented 10 years ago

На счет форкнуть без проблем, я то уже все что мне надо сделал, просто думал помочь интересному проекту, пока контакт свое сломаное в очередной раз апи чинит...

AndrewShmig commented 10 years ago

@alaija пул реквест и я приму) пока же мне всё таки не особо нравится это самое ограничение по версиям на каждый VKRequestManager. В случае чего мне необходимо будет создавать новый мэнеджер, где было бы проще указать параметр "v" и всё.

ghost commented 10 years ago

Не, мой фикс на пулреквест не катит, там тупо костыль. Если проблем о которых вы говорите, не выявлю, то может допилю и отправлю Вам.