Skip to content

Fix links in animezone.pl. Add lang to title. - #34

Merged
xulek merged 1 commit into
masterfrom
anime1
Dec 6, 2018
Merged

xulek merged 1 commit into
masterfrom
anime1

Conversation

@rysson

@rysson rysson commented Nov 22, 2018

Copy link
Copy Markdown
Collaborator

Alternatywna względem #30 wersja drobnych poprawek w anime.

@po50on, zerknij czy działa, proszę.

@rysson
rysson requested a review from xulek November 22, 2018 17:12
@rysson rysson mentioned this pull request Nov 22, 2018
for tr in client.parseDOM(result, 'tr'):
r = re.search(r'(?is)[^<]*(?:<tr[^]>*>[^<]*)?<td[^>]*>(?P<name>[^<]*)<.*?'
r'<span class="(?:sprites +)?(?P<lang>[^"]*?)(?: +lang)?">.*?'
r'\bdata-\w+="(?P<data>[^"]*)"', tr)

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sam się skomentuję. ParseDOM jest na tyle toporna, że łatwiej mi było użyć własnego, drobnego, trzylinijkowego wyrażenia regularnego :-)
Z pdom.select, byłoby to dużo prostsze, ale to jeszcze moment.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

'drobnego' haha :D śmieszek :) Fajnie jakby Twoja metoda była już gotowa, to można by zacząć edytować wtyczki z jej wykorzystaniem. Dodatkowo chyba dobrą opcją byłoby zrobienie cache w sqllite dla każdej wtyczki aby to wszystko przyspieszyć do granic możliwości, ale to już temat na inną rozmowę :)

@rysson

rysson commented Nov 23, 2018

Copy link
Copy Markdown
Collaborator Author

@po50on, sprawdzałeś może czy u Ciebie działa?
Jak masz jakieś pytania to zachęcam do ich zadawania. Mogę wyjaśnić co i dlaczego zrobiłem.

@rysson

rysson commented Nov 23, 2018

Copy link
Copy Markdown
Collaborator Author

Dodatkowo chyba dobrą opcją byłoby zrobienie cache w sqllite dla każdej wtyczki aby to wszystko przyspieszyć do granic możliwości, ale to już temat na inną rozmowę :)

Tak, tak, tak! To jest bardzo dobry pomysł.

A co do parsowania, to jeszcze chwila. Muszę opanować nieco rozszerzone budowanie ZIP-ów, aby łatwo zależności zewnętrzne paczkować. Potem można wrzucić te wynalazki, ale nie ma co przerabiać wszystkiego jeszcze.

Może byśmy infrastrukturę wymyślili lepszą. Jakiś system klas dodatków. M.in. z cache, o którym wspomniałeś. Mażą mi się dużo udogodnienia, tylko jeszcze nie wiem jak to ująć. Może dekoratory, może generatory, może...
Znów wraca pytanie, jak możemy to ugadać? A jakby tak się skrzyknąć na np. Hangouts? Czy innym wynalazku (uprzedzam, u mnie Skype nie bardzo działa).

@xulek

xulek commented Nov 23, 2018

Copy link
Copy Markdown
Collaborator

A jakby tak się skrzyknąć na np. Hangouts?

Jak dla mnie może być :) Chociaż jeszcze z tego nie korzystałem :P

@rysson

rysson commented Nov 23, 2018

Copy link
Copy Markdown
Collaborator Author

Jak masz okazję korzystać z gmaila to tam jest guzik. Powinno działać przez przeglądarkę choć o ile się nie zmieniło, to trzeba było jakiś ogólny plugin zainstalować (do AV).

Teraz jeszcze tylko wspólny termin na naszego kółka :-)

@po50on

po50on commented Nov 29, 2018 •

Copy link
Copy Markdown
Contributor

witam no niestety wyskakuje mi

- NOTE: IGNORING THIS CAN LEAD TO MEMORY LEAKS!
  Error Type: <type 'exceptions.IndentationError'>
  Error Contents: ('unexpected indent', ('C:\\Users\\wald\\AppData\\Roaming\\Kodi\\addons\\script.module.ptw\\lib\\ptw\\__init__.py', 2, 1, ' PY2 = _sys.version_info < (3, 0)\n'))
  Traceback (most recent call last):
    File "C:\Users\wald\AppData\Roaming\Kodi\addons\plugin.video.anime2\main.py", line 16, in <module>
      from ptw.debug import log_exception
  IndentationError: ('unexpected indent', ('C:\\Users\\wald\\AppData\\Roaming\\Kodi\\addons\\script.module.ptw\\lib\\ptw\\__init__.py', 2, 1, ' PY2 = _sys.version_info < (3, 0)\n'))
  -->End of Python script error report<--

@rysson

rysson commented Nov 30, 2018

Copy link
Copy Markdown
Collaborator Author

Eee, a Ty zrobiłeś git checkout anime1 czy ręcznie kopiowałeś zmiany?
Bo widzisz, wygląda jakbyś skopiował ze strony i zaznaczyła Ci się dodatkowa spacja na początku linii.
Użyj gita.

@po50on

po50on commented Nov 30, 2018

Copy link
Copy Markdown
Contributor

ręcznie

@xulek
xulek merged commit 63ad96b into master Dec 6, 2018
@po50on

po50on commented Dec 7, 2018

Copy link
Copy Markdown
Contributor

dzieki dziala poprawnie
czy mozna dodac zapamietywanie ogladanych ?

@po50on

po50on commented Dec 11, 2018

Copy link
Copy Markdown
Contributor

witam jak mozna testowac moduly skryptow bo w pyscript mi nie wychodzi dla kodi

@xulek

xulek commented Dec 13, 2018

Copy link
Copy Markdown
Collaborator

witam jak mozna testowac moduly skryptow bo w pyscript mi nie wychodzi dla kodi

@po50on pewnie chodzi ci o debugowanie. Użyj pydevd na liclipse albo pycharm(potrzeba wersji pro) albo wtyczki webpdb

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants