search

alternativshik / vk_search

Listen music from Vk.com in Amarok
10 stars 3 forks source link

Extensive refactor of the script + attempt to fix status reporting. #6

Open intelfx opened 11 years ago

intelfx commented 11 years ago

Items are now organized into 4 levels (owner - artist - fake album entry - track), which made it possible to have correct "artist" and "title" data for tracks. This makes importing tracks more easy and allows lyrics fetcher scripts to do their job. Unfortunately, manual editing of tags after import is still required, since genre is set to track's owner and album is set to artist, but this is unavoidable (first - due to Amarok API limitations, second - due to Amarok scriptable service manager bug which makes it impossible to have empty album names).

I've also attempted to fix status reporting (URL structure changed in the meantime), but the corresponding methods somewhy ceased to be invoked (Amarok 2.8.0). Don't know why.

P. S.: I didn't bump the version.

intelfx commented 10 years ago

ping. do you still maintain this?

alternativshik commented 10 years ago

Временно нет возможности протестировать все и выложить на кде-аппс.

alternativshik commented 10 years ago

Дошли руки. Спасибо за проделанную работу.

  1. метод parseAudioUrl не сработает - например ответ от ВК: https://psv4.vk.me/c6108/u22040572/audios/dc6985beca3c.mp3?extra=rY7g6nBnSEwq5mkWEUI7hn1Z0xigRwxFwUaM8jJaUtT2I5WGMZAUGlVQctRHqtKNaPOAdbVlaKRRzwIZMkM1zVSPtI0

dc6985beca3c.mp3 - не попадет под регуляку - установка статуса ломается окончательно.

  1. Не согласен с дополнительным уровнем вложенности как фейковый альбом - и без него все отлично скроблится и подтягивается и вики и тексты песен + теряется фича с аватарками юзеров, так как по ним проще найти интересующего, импорт музыки... а много ли вообще качает народа? Не думаю. В крайнем случае, для этого есть easytag, который исправит все за минуту.

Исходя из всего вышеперечисленного - возьму некоторые наработки по рефакторингу + добавлю ввод капчи, когда ее просит ВК и выложу обновление в ближайшем времени. Весь пулл-реквест, пока, к сожалению, не возьму, т.к. не совпадает с изначальной идеологией плагина.