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

Fix links in animezone.pl. Add lang to title.#34
xulek merged 1 commit into
masterfrom
anime1

Conversation

@rysson
Copy link
Copy Markdown
Collaborator

@rysson rysson commented Nov 22, 2018

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
Copy link
Copy Markdown
Collaborator Author

rysson commented Nov 23, 2018

@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
Copy link
Copy Markdown
Collaborator Author

rysson commented Nov 23, 2018

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
Copy link
Copy Markdown
Collaborator

xulek commented Nov 23, 2018

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

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

@rysson
Copy link
Copy Markdown
Collaborator Author

rysson commented Nov 23, 2018

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
Copy link
Copy Markdown
Contributor

po50on commented Nov 29, 2018

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
Copy link
Copy Markdown
Collaborator Author

rysson commented Nov 30, 2018

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
Copy link
Copy Markdown
Contributor

po50on commented Nov 30, 2018

ręcznie

@xulek xulek merged commit 63ad96b into master Dec 6, 2018
@po50on
Copy link
Copy Markdown
Contributor

po50on commented Dec 7, 2018

dzieki dziala poprawnie
czy mozna dodac zapamietywanie ogladanych ?

@po50on
Copy link
Copy Markdown
Contributor

po50on commented Dec 11, 2018

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

@xulek
Copy link
Copy Markdown
Collaborator

xulek commented Dec 13, 2018

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