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

Issue#26 fix. #28

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Issue#26 fix. #28

wants to merge 1 commit into from

Conversation

anri-vin
Copy link

GetWallExecute now should work as expected. #26

GetWallExecute now should work as expected.
Copy link
Owner

@Dementiy Dementiy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

В чем смысл переименования max_count в count? Вроде этот метод без нареканий работал.

@anri-vin
Copy link
Author

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

@Dementiy
Copy link
Owner

Почему нигде не учитывается? 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).
А какая была ошибка?

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

Successfully merging this pull request may close these issues.

None yet

2 participants