search

kodi-czsk / plugin.video.sosac.ph

GNU General Public License v2.0
17 stars 31 forks source link

Dabing filtrovani #157

Closed Saljack closed 6 years ago

Saljack commented 6 years ago

Cus, obnovil jsem praci na PR #131 tj. na filtrovani filmu podle dabingu resp. jazyku. Vsimnul jsem si toho v jinych pluginech a je posledni moznost, ktera me napada. V kazdem listu, kde jsou filmy (pismeno, posledni pridane, oblibene atp.) je na prvni pozici polozka Filter, ktera obsahuje jazyky. Tj. vyresila se duplikace v menu a vypada to celkem dobre a do budoucna by tam mohlo byt i dalsi filtrovani nejen podle jazyku.

Saljack commented 6 years ago

Co se tyka te jedne chyby v codeclimate, tak jsem se to pokusil zrefaktorovat a neprijde mi, ze by to nejak pomohlo citelnosti. V tomhle pripade mi prijde mit lepsi vice return statementu, nez to jeste vice refaktorovat.

lzoubek commented 6 years ago

@Saljack po mergi predchazejich PR je tam maly konflikt, muzes ho prosim opravit?

Saljack commented 6 years ago

@lzoubek opraveno muzes mergovat

Saljack commented 6 years ago

Jeste jsem si vsimnul, ze jsem udelal preklep pro #154 misto IMDB je tam ted IMBD. Opravil jsem to v tomhle PR.

lzoubek commented 6 years ago

ok, prosim jeste o opravu klimatickych chyb

Saljack commented 6 years ago

Opraveno aspon jsem se dozvedel co je cognitive complexity

xDereCx commented 6 years ago

je mozne ze tento posledny update ovplyvnil funkciu "add all to library" pre movies? https://pastebin.com/iZqBFwPp