Update plugin Anime

[xulek]

zmiana od @po50on

#30

[rysson]

Dyskusja w toku.

[rysson]

To już poproszę na wzór oryginału, nieco bardziej czytelne czyli:

counter -= 1

[rysson]

Jako ciekawostkę, bo nie wymagam zmiany (było też tak sobie), podaję alternatywę:

    for counter in xrange(2019, 1982, -1):
        ...

Wtedy już by nie było odejmowania counter na końcu.

[rysson]

@po50on, Tutaj pytanie, poprzednio było od 1982 do 2019 (włącznie).
Teraz jest od 2019 do 1983 (włącznie). Czy 1982 ma teraz nie wchodzić w zakres?
Jeśli ma wejść to powinno być:

    while counter >= 1982:
        ...

albo w mojej propozycji (drugi parametr xrange() i range() czyli koniec zakresu zawsze nie wchodzi):

    for counter in xrange(2019, 1981, -1):
        ...

[po50on]

twoja propozycja mi sie podoba

[xulek]

można jeszcze 2019 zamienić na:
int(datetime.date.today().year) + 1
oczywiście wcześniej importując datetime

[rysson]

Jak to jest rok, to jasne. Nawet int nie jest potrzebny

import datetime.date
datetime.date.today().year + 1

[rysson]

@po50on, ależ nie ma sprawy, wszyscy się uczymy. Gorzej jak ktoś nie chce się rozwijać.

jeszcze szukam sposobu na wyswietlanie wersi jezykowej linkow

A gdzie na stronie jest taka informacja? Nie widzę jej przy liście seriali.

Masz na myśli listę odcinków? Tam już są flagi.

<tr>
  <td class="text-center"><strong>12</strong></td>
  <td class="episode-title">Świt ciemności.</td>
  <td style="padding: 0" class="text-center"><span class="sprites PL"></span></td>
  <td class="bg-success text-center filler">Nie</td>
  <td class="text-center"><a href="../odcinek/11eyes/12" class="btn btn-xs btn-success _visited">Zobacz</a></td>
</tr>

Jeśli tak, to może to wyglądać np. tak:

def ListowaniOdcinkow():
    url = params['url']
    iconimage = params.get('iconimage', '')

    r = client.request(url)
    result = client.parseDOM(r, 'table', attrs={'class': 'table table-bordered table-striped table-hover episodes'})
    for tr in client.parseDOM(result, 'tr'):
        link = client.parseDOM(tr, 'a', ret='href')[0]
        nazwa = client.parseDOM(tr, 'td', attrs={'class': 'episode-title'})[0]
        odcinek = client.parseDOM(tr, 'strong')[0]
        lang = client.parseDOM(tr, 'span', ret='class')[0].split()[-1]

        nazwa = '{ep:02d}. {title} ({lang})'.format(title=nazwa, ep=int(odcinek), lang=lang)
        addon.addDir(nazwa, urljoin(url, link), mode=5, icon=iconimage)

Na górze pliku jeszcze trzeba dodać (dla Python 2):

from urlparse import urljoin

Pisałem „w ciemno”, więc musisz to przetestować. Powinna być lista typu: 12. Świt ciemności. (PL).
nazwa = określa co się pojawi. Do formatowania używa mini języka.

EDIT:
Słowo wyjaśnienie dotyczące lang = .
Trik polega na pobraniu elementu span z niego class.
Jest jeden span to od razu po niego sięgamy [0].
Dzielimy klasę na słowa .split() i pobieramy ostatnie słowo [-1], czyli w tym przykładzie PL.

[rysson]

@xulek, jak już dodamy nowy sposób parsowania to może wyglądać tak:

def ListowaniOdcinkow():
    url = params['url']
    iconimage = params.get('iconimage', '')

    r = client.request(url)
    for a, name, ep, lang in \
            pdom.select(r, 'table.episodes tr {a, .episode-title, strong, span}'):
        lang = lang.attr('class').split()[-1]
        title = '{ep.text}. {title.text} ({lang})'.format(**locals())
        addon.addDir(title, urljoin(url, a.href), mode=5, icon=iconimage)

Albo, porównawczo, wyciągając od razu wartości węzłów:

def ListowaniOdcinkow():
    url = params['url']
    iconimage = params.get('iconimage', '')

    r = client.request(url)
    for (link,), (name,), (ep,), (lang,) in pdom.select(r,
            'table.episodes tr {a(href), .episode-title::text, strong::text, span(class)}'):
        lang = lang.split()[-1]
        title = '{ep}. {title} ({lang})'.format(**locals())
        addon.addDir(title, urljoin(url, link), mode=5, icon=iconimage)

[po50on]

niestety wywala blad
ERROR: EXCEPTION Thrown (PythonToCppException) : -->Python callback/script returned the following error<--
- NOTE: IGNORING THIS CAN LEAD TO MEMORY LEAKS!
Error Type: <type 'exceptions.IndexError'>
Error Contents: list index out of range
Traceback (most recent call last):
File "C:\Users\wald\AppData\Roaming\Kodi\addons\plugin.video.anime2\main.py", line 312, in
ListowaniOdcinkow()
File "C:\Users\wald\AppData\Roaming\Kodi\addons\plugin.video.anime2\main.py", line 148, in ListowaniOdcinkow
link = client.parseDOM(tr, 'a', ret='href')[0]
IndexError: list index out of range
-->End of Python script error report<--

[po50on]

bardziej mi chodziło o jezyk w linkach

[xulek]

U mnie wtyczka działa bez błędów, a jeśli chodzi o język źródła to zapomniałem go dodać w listowaniu odcinków. Jak znajdę chwilkę to wrzucę tu poprawkę :)

[rysson]

@po50on, zajrzę wieczorem i sprawdzę.
Co to znaczy „język w linkach”? Możesz podać przykład?

[xulek]

@rysson podejrzewam, że chodzi o oznaczenie czy odcinek z danego linku jest w języku polskim. Jest zrobiona zmienna lang, ale nie jest używana.

[rysson]

@xulek. Możliwe, ale wizualnie na liście za to odpowiada element <span class="sprites PL">, renderowany jako flaga przy odcinku, a @po50on napisał, że nie bardzo o to chodzi. Dlatego go pytam co konkretnie ma na myśli.

[rysson]

OKi, zaczynam łapać, język jest w dwóch miejscach. Na liście odcinków i po wybraniu odcinka jest lista linków w różnymi językami (i tam jest class="sprite PL lang").

Wcześniej nie zaszedłem tak daleko. Akurat to nie jest mój ulubiony gatunek, choć Pszczółkę Maję oglądałem na dobranockę :-) Była a niebo lepsza od Koralgola :-D

Co do niedziałania, to złapał nagłówek listy a tam nie ma danych. Trzeba by dodać tuż po pobraniu strony i znalezieniu table, a przed pętlą:

   result = client.parseDOM(result, 'tbody')

Może jeszcze spojrzę na język i wystawię zmiany.

[po50on]

dieki a pomoc przyokazji ucze sie pythona :)

[rysson]

@po50on, fajnie, ja też :-) Nawet książkę kupiłem „Zaawansowany Python (Fluent Python)”. Gdzieś od połowy (~350 str.) zaczęsło robić się ciekawie.

Poprzerabiałem nieco te funkcję, wokół których się kręciłeś. Zerknij na #34 i zobacz czy tak może być. Jak masz pytania to już tam. Przy okazji mógłbyś sprawdzić czy działa, bo ja tylko przejrzałem ze 3..4 pozycje.