tedious / JShrink

Javascript Minifier built in PHP
http://www.tedivm.com
BSD 3-Clause "New" or "Revised" License
751 stars 152 forks source link

"strpos()" including deprecated String #89

Closed Darkice78 closed 1 year ago

Darkice78 commented 5 years ago

When i turn my server to PHP7.3 i get the following error: Deprecated: strpos(): Non-string needles will be interpreted as strings in the future. Use an explicit chr() call to preserve the current behavior in .../minifier.php on line 184

webrectoversocreationbe commented 5 years ago

that's why i'm here... to check if a new release is done, or explain what i've done to resolve this error : if ((string)$this->b!='' && strpos('(-+[@', (string)$this->b) !== false) {

TimGS commented 4 years ago

+1. Running PHP 7.3 on Debian 10.

tedivm commented 1 year ago

Please reopen with an example code that will trigger this. That said I think the existing behavior with php8 should work without warning or error.