vkR icon indicating copy to clipboard operation
vkR copied to clipboard

Issue#26 fix.

Open anri-vin opened this issue 8 years ago • 2 comments

GetWallExecute now should work as expected. #26

anri-vin avatar Oct 22 '17 19:10 anri-vin

В том то и проблема, что он не работает, если > 1 записи запрашивать. Я язык R первый раз в глаза вижу, но как я понял - передаваемый count вообще нигде не учитывается и по аналогии с остальными методами внес правки. В заголовке он count, а затем сразу превращается в max_count который никак не инициализирован. После переименования все заработало.

anri-vin avatar Oct 27 '17 12:10 anri-vin

Почему нигде не учитывается? count это количество записей, которые мы хотим получить. Если count = 0, то должны получить все записи. max_count это актуальное число записей, которые должны получить. Инициализация max_count https://github.com/Dementiy/vkR/blob/master/R/wall.R#L136:

max_count <- ifelse((response$count - offset) > count & count != 0, count, response$count - offset)

Предварительно происходит запрос к ВК. В ответе указывается общее число записей на стене response$count. Если это значение минус смещение больше чем число записей, которые мы хотим получить, то получаем столько записей сколько попросили (count), в противном случае получаем все доступные (response$count - offset). А какая была ошибка?

Dementiy avatar Oct 27 '17 16:10 Dementiy