Skip to content

Catch json_encode errors#14

Open
ulkoalex wants to merge 18 commits intovladkens:masterfrom
ulkoalex:master
Open

Catch json_encode errors#14
ulkoalex wants to merge 18 commits intovladkens:masterfrom
ulkoalex:master

Conversation

@ulkoalex
Copy link
Copy Markdown

@ulkoalex ulkoalex commented Jan 20, 2018

test.css.txt
Catch json_encode errors, like Malformed UTF-8 characters, possibly incorrectly encoded (e.g. broken encoding in content CSS rule, example attached)

Without checking if json_encode ran successfully further execution logic breaks e.g. $output will be false, and foreach will trigger E_WARNING level error

@ulkoalex ulkoalex force-pushed the master branch 2 times, most recently from c6e827b to 1dae65d Compare January 20, 2018 23:32
@ulkoalex ulkoalex mentioned this pull request Mar 6, 2023
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