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

Update regex patterns #91

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update regex patterns #91

wants to merge 1 commit into from

Conversation

bmooreitul
Copy link

Fix legacy pattern matching to be compatible with PHP 7.4+. These changes still work with older versions of php (like 5.2+). PHP 7.4 requires escaping dashes (-).
The only changes are from \w- to \w-
This modernizes while maintaining legacy support.

Fix legacy pattern matching to be compatible with PHP 7.4+. 
These changes still work with older versions of php (like 5.2+).
PHP 7.4 requires escaping dashes (\-).
The only changes are from \w- to \w\-
This modernizes while maintaining legacy support.
@bmooreitul
Copy link
Author

Update regex patterns to work with PHP 7.4-8+
Older PHP versions like 5.2 still work

@bmooreitul bmooreitul closed this Mar 28, 2023
@bmooreitul bmooreitul reopened this Mar 28, 2023
@TwinDigitalNL
Copy link

Would be great if this can be merged.

@bmooreitul
Copy link
Author

Would be great if this can be merged.

@TwinDigitalNL didnt realize anyone else was running into the same issue. I ended up just creating a new packae based on this one since it doesnt look like its maintained anymore. https://bitbucket.org/bmooreitul/php-simple-dom-parser/src/master/

You can use my version until if/when my suggestion ever gets merged.

composer require itul/php-simple-html-dom-parser

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.

2 participants