abecedne radenie relacii + moznost automatickeho prehravania videi

jose1711 commented 9 years ago

rozhodne ziaden state-of-art kod, ale mne tam tieto 2 moznosti velmi chybali..

kapitan-iglu commented 9 years ago

Zdar, so sortovanim relacii podla nazvu suhlasim, tato featura sa moze prebrat.

K automatickemu prehravaniu zvolenej kvality mam tieto vyhrady:

V zozname streamurllist sa nachadzaju vsetky URL 2x kvoli zle odsadenemu riadku streamurllist.append(stream_url).
Stream ponuka viac druhov kvalit ako len 3, standardne 240p, 360p, 480p, 720p, 1080p (len zriedkavo chyba 1080p). Napriek tomu, ze zuzit vyber kvality na nizku/strednu/vysoku moze byt pre BFU vhodnejsie, lepsie by bolo mat na vyber vsetky, tak ako ich ponuka stream na offic stranke.
Nepaci sa mi, ze ak si user v settingsoch vyberie konkretnu kvalitu, tak sa aj tak (zbytocne) vygeneruje directory so zoznamom vsetkych dostupnych videolinkov a az potom sa spusti xbmc.Player().play().
Kvoli predchadzajucemu bodu je v niektorych situaciach plugin nepouzitelny - po ukonceni prehravania (pri zvolenej kvalite) sa video opat spusti od zaciatku ako dosledok toho, ze kodi sa snazi opat vylistovat directory so zoznamom linkov (mode=10) sucastou coho je opat volanie xbmc.Player().play()...

Featura automatickeho prehrania videa so zvolenou kvalitou chybala aj mne. Pred cca dvoma tyzdnami som ju spravil a teraz testujem. Vecer commitnem nech mozu skusat ostatny a potom nech lud rozhodne, ktora verzia sa preberie do mastra..

jose1711 commented 9 years ago

ahoj kapitan,

ked to uz mas hotove, tak nie je co riesit - ja som to robil hlavne kvoli sebe (a trochu horucejsou a neskusenou ihlou). mozno by vsak bolo fajn pridat do repo aj TODO - nech sa predchadza situaciam, ze dvaja robia na tom istom.

dik, jose

kapitan-iglu commented 9 years ago

Mas pravdu, mal som si privlastnit issue #2 hned ako som na tom zacal robit.

Mozno by sa vsetky "svojvolne" upravy/opravy mohli riesit cez issues - kazdy si pred zacatim prac vlozi issue s popisom co ide robit a privlastni si ju. Tym by sa zabranilo dvojitej robote (aj ked by to vyzeralo trochu narcisticky :)

Kukni sa na/otestuj #4 ci vyhovuje tvojim predstavam.

jose1711 commented 9 years ago

za mna moze byt. +1 merge :-)

kapitan-iglu commented 9 years ago

A co s abecednym triedenim relacii? Spravis novy pull request len s touto featurov? Alebo to mam okopcit ku svojim upravam?

jose1711 commented 9 years ago

kludne to tam dohod, pripadne vylepsi. dik

kodi-czsk / plugin.video.dmd-czech.stream

abecedne radenie relacii + moznost automatickeho prehravania videi #3