Podmienianie znaków we wtyczkach

[GoogleCodeExporter]

Znalazłem problem w logice, na przykładzie serwisu ekino.tv, ale być może 
występuje też w innych, a mianowicie chodzi o znaki specjalne, a głównie o 
'+'.

W pliku:
/home/xbian/.xbmc/addons/plugin.video.polishtv.live/hosts/ekinotv.py
w linii 591:
{{{title = title.replace("+", " ")}}}
jest podmieniany znak '+' na spację, aczkolwiek natrafiłem akurat na film, 
który miał ograniczenie wiekowe zapisane w sposób 'Tytuł [12+]', więc znak 
plus został tutaj zastąpiony (zakładam, że ze względów zabezpieczeń 
przeciw znakom specjalnym).

Wszystko niby dobrze, ale w funkcji getMovieURL przekazany jest klucz, który 
jest tytułem z podmienionym już '+', natomiast porównywany jest później z 
linkami na stronie, gdzie znak '+' już nie jest podmieniony, powoduje to 
oczywiście brak linku do materiału wideo i nie działanie wtyczki.

Jako szybką poprawkę można dopisać do linii 232 w funkcji getMovieURL:
{{{name = name.replace("+", " ")}}}
Zaraz przed porównaniem:
{{{if key in name:}}}

Aczkolwiek jako docelowe rozwiązanie dobrze by było wprowadzić jakiegoś 
może globalne rozwiązanie do zamieniania znaków specjalnych.

NOTE: Numery linii mogą się nieco różnić, ponieważ mam dodanych parę 
linii z dodatkowym logowaniem itd.

Original issue reported on code.google.com by grzegorz...@syndrom-it.pl on 15 Dec 2012 at 6:46

[GoogleCodeExporter]

@grzegorz - masz prawa do "commit" w projekcie, wiec jesli chce zrobic poprawke 
to wal smialo.

Original comment by jatrn69 on 15 Dec 2012 at 8:13

[GoogleCodeExporter]

@grzegorz - ja to kiedyś pisałem. urllib.unquote_plus załatwi sprawę, 
także to można wywalić.

Original comment by ples...@gmail.com on 15 Dec 2012 at 8:55

[GoogleCodeExporter]

@jatr69:
Wiem, ale z racji iż na co dzień nie piszę tego projektu to nie chciałbym 
wcinać się w czyiś kod, samemu nie chciałbym, aby ktoś od niedzieli coś 
mi wrzucał do kodu.

Drugi powód, dla którego tego nie wrzuciłem przedstawił plesken, nie 
korzystam z najnowszej wersji i zakładałem, że już ktoś mógł się za to 
zabrać.

Original comment by grzegorz...@syndrom-it.pl on 16 Dec 2012 at 4:35

[GoogleCodeExporter]

zamykam issue

Original comment by jatrn69 on 10 Mar 2013 at 1:16

• Changed state: Done