added subtitle support

[aykut]

Selam,

Subtitle destegi icin bir gelistirme yaptim.

[ghost]

Merhaba,

En kisa surede yaptiginiz degisikligi inceleyerek size geri donecegim fakat biz bilerek ve isteyerek bu destegi koymamistik plugin'e. Hatta yaptikdan sonra silmistik. Onu pesinen soyleyeyim.

Iyi gunler.

[cenkalti]

Bir de bu putio.py dosyasinin orijinali surada: https://github.com/cenkalti/putio.py

Eger cok diverge olmadiysa bunu git subrepository olarak eklememiz mumkun mu?

[ghost]

@aykut Eklediginiz subtitle ozelligini su commit icerisinde eklemistik: https://github.com/putdotio/plugin.video.putiov2/commit/2209c72afba063bf65a7f92d423b6043a1689277#diff-5cad3ec4d2e5fb17c7f35b53a1a20bdeR103

Orada biraz daha karmasik bir mantik var, subtitle'in set edildikten sonra gosterilebilmesi icin playerin "playing" statusunda olmasi gerekiyordu ve bunu yakalamak icin su an hatirlayamadigim bir plugin'in kodunda gordugum hileyi yaptim. While dongusu icinde bir sure uyuyarak playing durumunu kontrol ediyor ve ona gore set ediyor. Kotu bir cozum ama kullaniliyor baska yerlerde de.

Bunu yaptikdan sonra da altyazi desteginin XBMC icindeki yonetimini begenmemis ve subtitle isini tamamen XBMC'nin kendi pluginine devretmeye karar vererek ozelligi geri cekmistik @hafifuyku ile beraber. Yaptiginiz degisiklik icin tesekkur ederiz.

@cenk XBMC deposunda guncelleme isteyecek bir sey degil sanirim, guncelleyebiliriz fakat yeni surum cikmak istedigimiz zaman XBMC deposonu maintain eden gelistiriciler review yaparken bu duruma ne der bilmiyorum, arastirmak lazim. Her turlu convention'a takiyorlardi en son. :)

[cenkalti]

@purak ben baktim simdi ama bu konuda bir kural bulamadim sitelerinde. Sana birakiyorum; yeni surum gonderecegin zaman denersin istersen.

[ghost]

@cenk, denerim tabii ki. Sag olasin.

[aykut]

@purak raspbmc uzerindeki denemelerimde bahsettiginiz sorunla karsilastim, ayni hileyi yapmak zorunda kaldim.

Simdilik ben kendime sakliyorum o zaman bunu ve pr'i kapatiyorum :).

Iyi aksamlar.

[ghost]

Tesekkurler.