Skip to content

Change split to preg_split#163

Closed
fagundes wants to merge 1 commit intospipu:masterfrom
fagundes:hotfix-remove-split
Closed

Change split to preg_split#163
fagundes wants to merge 1 commit intospipu:masterfrom
fagundes:hotfix-remove-split

Conversation

@fagundes
Copy link
Copy Markdown
Contributor

Fix #162 issue.

@saidmoya12
Copy link
Copy Markdown

Is no correct the Regexp Expression.

Additionally preg_slipt use more process.

@fagundes
Copy link
Copy Markdown
Contributor Author

I couldn't find the error.

About use more process, first split is deprecated since PHP 5.3 and it was removed in PHP 7.0. Second the difference of process between split and preg_split in this case is irrelevant.

spipu added a commit that referenced this pull request May 17, 2017
@spipu
Copy link
Copy Markdown
Owner

spipu commented May 17, 2017

fixed with the explode method

@spipu spipu closed this May 17, 2017
@fagundes fagundes deleted the hotfix-remove-split branch May 17, 2017 15:01
icebird93 pushed a commit to EduBase/html2pdf that referenced this pull request Jun 18, 2018
aaryarajojuH added a commit to aaryarajojuH/spipu that referenced this pull request Feb 2, 2022
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