Skip to content

Фикс ошибки с парсингом аудиозаписей#312

Closed
vec715 wants to merge 1 commit intopython273:masterfrom
vec715:feature/fix-audio-parsing
Closed

Фикс ошибки с парсингом аудиозаписей#312
vec715 wants to merge 1 commit intopython273:masterfrom
vec715:feature/fix-audio-parsing

Conversation

@vec715
Copy link
Copy Markdown

@vec715 vec715 commented Nov 21, 2019

Related issue: #311

@bakatrouble
Copy link
Copy Markdown
Contributor

Мой код полагался, что тип id - int
Неплохо было бы не ломать совместимость

И почему берутся поля vk_id и id? Следует парсить либо content_id из JSON, либо атрибут тега data-id (и там, и там формат OWNERID_AUDIOID.
Опять же, мой код опирается на id старого вида (большее число для добавленных позже) чтобы отлавливать новые добавленные, а id в JSON мало того что какое-то совершенно постороннее число, так еще и не всегда присутствует.

@bakatrouble
Copy link
Copy Markdown
Contributor

Корректная логика в #315

@python273 python273 closed this Nov 25, 2019
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.

3 participants