(-) Skipping song if unavailable

[choppylion]

If song is not available due to requests of the copyright holder it has following tag:
<div class="audio_item audio_item_disabled"></div> so we can't extract any info from it and audio.select('.ai_artist') will return empty list. After that we'll get IndexError: list index out of range.

So I think it'll be good to check if tag is not containing class named 'audio_item_disabled'.
But how can we tell user about this by API?

[python273]

IndexError: list index out of range.

Can you show traceback?

[choppylion]

Sure. But don't pay attention on line numbers, because I've changed code a little bit.

Traceback (most recent call last):
  File "C:\Program Files\JetBrains\PyCharm 2017.1.2\helpers\pydev\pydevd.py", line 1585, in <module>
    globals = debugger.run(setup['file'], None, None, is_module)
  File "C:\Program Files\JetBrains\PyCharm 2017.1.2\helpers\pydev\pydevd.py", line 1015, in run
    pydev_imports.execfile(file, globals, locals)  # execute the script
  File "C:\Program Files\JetBrains\PyCharm 2017.1.2\helpers\pydev\_pydev_imps\_pydev_execfile.py", line 18, in execfile
    exec(compile(contents+"\n", file, 'exec'), glob, loc)
  File "C:/Work/vktotg/vktolocal.py", line 188, in <module>
    main()
  File "C:/Work/vktotg/vktolocal.py", line 178, in main
    process(session)
  File "C:/Work/vktotg/vktolocal.py", line 133, in process
    track_list = get_track_list(vk_session)
  File "C:/Work/vktotg/vktolocal.py", line 119, in get_track_list
    chunk = vk_audio.get(owner_id=user_id, offset=offset)
  File "C:\Tools\Python35\lib\site-packages\vk_api\audio.py", line 52, in get
    return scrap_data(response.text)
  File "C:\Tools\Python35\lib\site-packages\vk_api\audio.py", line 146, in scrap_data
    artist = audio.select('.ai_artist')[0].text
IndexError: list index out of range

[choppylion]

I have a proposal how to get info of disabled track.
We can get track ID of previous available track, find this track on DOM of desktop version and get its following sibling, that'll have all this data excepting link to track. So we can at least tell user what tracks are unavailable to download.

After some research, I got that we cannot get desktop DOM with specified offset.

[python273]

let's make

if 'audio_item_disabled' in audio['class']:
    continue

ai_artist = audio.select('.ai_artist')
...

[python273]

you can delete else

[choppylion]

I don't want to make any more changes under else. I'm gonna just replace lines 132-133 by implemented code in future.

[python273]

else block is not needed, until it's not implemented

[choppylion]

Deal

[python273]

Can you squash commits?

[python273]

Looks like GitHub has ability to squash PR's commits. 👍