Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Lots of warning due to PHP7.3 PCRE2 #8

Closed
shahariaazam opened this issue May 16, 2020 · 1 comment
Closed

Lots of warning due to PHP7.3 PCRE2 #8

shahariaazam opened this issue May 16, 2020 · 1 comment
Assignees

Comments

@shahariaazam
Copy link
Owner

The original DOM parser library used https://github.com/sunra/php-simple-html-dom-parser. It seems that sunra/php-simple-html-dom-parser library has been abandoned.

Due the recent changes in PHP7.3 (PCRE engine). It's showing lots of warnings. That repo already has a PR sunra/php-simple-html-dom-parser#63 . But kept un-marged. So planning to change this library in future.

PHP Warning:  preg_match(): Compilation failed: invalid range in character class at offset 4 in /var/www/DSE-Share-Market-Update/vendor/sunra/php-simple-html-dom-parser/Src/Sunra/PhpSimple/simplehtmldom_1_5/simple_html_dom.php on line 1378
PHP Stack trace:
PHP   1. {main}() /var/www/DSE-Share-Market-Update/test.php:0
PHP   2. ShahariaAzam\BDStockExchange\StockPrice->getCSEPricing() /var/www/DSE-Share-Market-Update/test.php:6
PHP   3. ShahariaAzam\BDStockExchange\StockPrice\CSE->getPricing() /var/www/DSE-Share-Market-Update/src/StockPrice.php:37
PHP   4. Sunra\PhpSimple\HtmlDomParser::str_get_html() /var/www/DSE-Share-Market-Update/src/StockPrice/CSE.php:28
PHP   5. call_user_func_array:{/var/www/DSE-Share-Market-Update/vendor/sunra/php-simple-html-dom-parser/Src/Sunra/PhpSimple/HtmlDomParser.php:21}() /var/www/DSE-Share-Market-Update/vendor/sunra/php-simple-html-dom-parser/Src/Sunra/PhpSimple/HtmlDomParser.php:21
PHP   6. simplehtmldom_1_5\str_get_html() /var/www/DSE-Share-Market-Update/vendor/sunra/php-simple-html-dom-parser/Src/Sunra/PhpSimple/HtmlDomParser.php:21
PHP   7. simplehtmldom_1_5\simple_html_dom->load() /var/www/DSE-Share-Market-Update/vendor/sunra/php-simple-html-dom-parser/Src/Sunra/PhpSimple/simplehtmldom_1_5/simple_html_dom.php:102
PHP   8. simplehtmldom_1_5\simple_html_dom->parse() /var/www/DSE-Share-Market-Update/vendor/sunra/php-simple-html-dom-parser/Src/Sunra/PhpSimple/simplehtmldom_1_5/simple_html_dom.php:1081
PHP   9. simplehtmldom_1_5\simple_html_dom->read_tag() /var/www/DSE-Share-Market-Update/vendor/sunra/php-simple-html-dom-parser/Src/Sunra/PhpSimple/simplehtmldom_1_5/simple_html_dom.php:1187
PHP  10. preg_match() /var/www/DSE-Share-Market-Update/vendor/sunra/php-simple-html-dom-parser/Src/Sunra/PhpSimple/simplehtmldom_1_5/simple_html_dom.php:1378
@shahariaazam
Copy link
Owner Author

This has been fixed on this commit.
14bba38

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

No branches or pull requests

1 participant