On cron run I have in my logs:

Deprecated function: strpos(): Non-string needles will be interpreted as strings in the future. Use an explicit chr() call to preserve the current behavior in JShrink\Minifier->loop() (regel 228 van /...../sites/all/modules/contrib/advagg/advagg_js_compress/jshrink.inc).

I am running php 7.3.

CommentFileSizeAuthor
#3 3115525-3.patch634 bytesjoelpittet

Comments

Anonymous’s picture

Anonymous (not verified) commented

cmseasy created an issue. See original summary.

thalles’s picture

thalles
Primary language Portuguese, Brazil
Location Teófilo Otoni - MG
commented

You can test this solution: #3024524: Deprecated function strpos after upgrade to PHP 7.3 and report the result here?

joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented
Version: 7.x-2.33 » 7.x-2.x-dev
Status: Active » Needs review
StatusFileSize
new 3115525-3.patch634 bytes

@cmseasy does this help your logs? patch as @thalles suggested.

joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented
Title: Deprecated function: strpos() » Deprecated strpos() on second argument as a non-string
thalles’s picture

thalles
Primary language Portuguese, Brazil
Location Teófilo Otoni - MG
commented

Related:
https://www.php.net/manual/pt_BR/migration73.deprecated.php

  • thalles committed c0cc2c5 on 7.x-2.x authored by joelpittet 
    Issue #3115525 by joelpittet: Deprecated strpos() on second argument as...
thalles’s picture

thalles
Primary language Portuguese, Brazil
Location Teófilo Otoni - MG
commented
Status: Needs review » Fixed

Thanks @all!

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.