lubosdz / parser-orsr Goto Github PK

Fixed in PR: #1

Zdravim. V prvom rade pekne dakujem, velmi uzitocna klasa 👍 . Len by som pre zaujimavost spomenul moj pripad pouzitia:
Na serveri nemam k dispozicii tidy php extension a instalacia stoji namahu viacero ludi :D
Tak som dal (namiesto efektu tidy-cka) sancu kombinacii :
libxml_use_internal_errors(true); ... $xml = new \DOMDocument('1.0', 'utf-8'); $xml->preserveWhiteSpace = false;

Samozrejme, tidy-ckove casti kodu som zakomentoval. Riadok libxml_use_internal_errors(true); sluzi na to, aby volanie $xml->loadHTML($html); nepadlo do chyby na nevalidnom html. Riadkom $xml->preserveWhiteSpace = false; hovorim DOMDocument-u, aby ignoroval whitespace-y medzi tagmi (inak by ich bral ako sucast dom).

Vysledok vyzera pouzitelne. Pouzivam vsak len metodu getDetailByICO(), takze neviem, mozno to neriesi globalne vsetky funkcie klasy...

riadok 531

$xml = new \DOMDocument('1.0', 'utf-8');
$xml->loadHTML($html);

$xpath = new \DOMXpath($xml);

V tejto casti nastane chyba, kedze sa php pokusa nacitat string, ktory je kodovany nespravne. Tato chyba nastava o riadok vyssie, kedy preg_replace pokazi kodovanie stringu. Deje sa to len na niektorych PHP verziach, ktore boli skompilovane inym sposobom. Spravne v tychto pripadoch treba pouzit flag u.

$html = preg_replace('/\s+/', ' ', $html);
$html = preg_replace('/\s+/u', ' ', $html);

Vytvoril som uz pull request, prosim o aplikovanie. #3

https://maxivak.com/working-with-regular-expressions-preg_-and-utf-8-strings-in-php/
https://stackoverflow.com/questions/35887031/why-does-w-match-non-english-characters-in-mac-os-x-php-environment

Moze nastat situacia ze pre jedno ICO je viac vysledkov

https://www.orsr.sk/hladaj_ico.asp?ICO=45281025

SRO medzitym zmenila nazov aj adresu. Parser spracuje udaje z prveho vysledku, ale spravny je (predpokladam ze vzdy) prave ten druhy. V nespravnom vysledku je takato informacia

Spis odstúpený na iný registrový súd z dôvodu miestnej nepríslušnosti

Deprecated trim with null value may be possible issue in future.

https://stackoverflow.com/questions/71707325/migration-to-php-8-1-how-to-fix-deprecated-passing-null-to-parameter-error-r

ak to dobre chapem getDetailByICO by ma vratit detail rovnako ako getDetailByPartialLink ale vracia list s 1 polozkou ako findByObchodneMeno