• Resolved mishash

    (@mishash)


    Hello,

    I am using the latest version of Geolocation IP Detection on my website, and in preparation to site migration to PHP 8.1, I tested plugin compatibility with PHP 8.1 using PHPCompatibility tool. There are multiple issues with the plugin code, please see the output below. Please let me know when do you plan to resolve these.

    Thanks,

    Mike

    FILE: …tested\geoip-detect\vendor\symfony\polyfill-intl-normalizer\Normalizer.php FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE

    93 | WARNING | INI directive 'mbstring.func_overload' is deprecated since PHP | | 7.2

    FILE: …-untested\geoip-detect\vendor\symfony\property-access\PropertyAccessor.php FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE

    125 | WARNING | Since PHP 7.0, functions inspecting arguments, like
    | | func_get_arg(), no longer report the original value as passed
    | | to a parameter, but will instead provide the current value.
    | | The parameter "$writeInfoExtractor" was used, and possibly | | changed (by reference), on line 124.

    FILE: …ent\plugins-untested\geoip-detect\vendor\symfony\string\AbstractString.php FOUND 1 ERROR AFFECTING 1 LINE

    12 | ERROR | 'string' is a reserved keyword as of PHP version 7.0 and should
    | | not be used to name a class, interface or trait or as part of a | | namespace (T_NAMESPACE)

    FILE: …gins-untested\geoip-detect\vendor\symfony\string\AbstractUnicodeString.php FOUND 1 ERROR AFFECTING 1 LINE

    12 | ERROR | 'string' is a reserved keyword as of PHP version 7.0 and should
    | | not be used to name a class, interface or trait or as part of a | | namespace (T_NAMESPACE)

    FILE: …content\plugins-untested\geoip-detect\vendor\symfony\string\ByteString.php FOUND 1 ERROR AFFECTING 1 LINE

    12 | ERROR | 'string' is a reserved keyword as of PHP version 7.0 and should
    | | not be used to name a class, interface or trait or as part of a | | namespace (T_NAMESPACE)

    FILE: …nt\plugins-untested\geoip-detect\vendor\symfony\string\CodePointString.php FOUND 1 ERROR AFFECTING 1 LINE

    12 | ERROR | 'string' is a reserved keyword as of PHP version 7.0 and should
    | | not be used to name a class, interface or trait or as part of a | | namespace (T_NAMESPACE)

    FILE: …tested\geoip-detect\vendor\symfony\string\Exception\ExceptionInterface.php FOUND 1 ERROR AFFECTING 1 LINE

    12 | ERROR | 'string' is a reserved keyword as of PHP version 7.0 and should
    | | not be used to name a class, interface or trait or as part of a | | namespace (T_NAMESPACE)

    FILE: …\geoip-detect\vendor\symfony\string\Exception\InvalidArgumentException.php FOUND 1 ERROR AFFECTING 1 LINE

    12 | ERROR | 'string' is a reserved keyword as of PHP version 7.0 and should
    | | not be used to name a class, interface or trait or as part of a | | namespace (T_NAMESPACE)

    FILE: …untested\geoip-detect\vendor\symfony\string\Exception\RuntimeException.php FOUND 1 ERROR AFFECTING 1 LINE

    12 | ERROR | 'string' is a reserved keyword as of PHP version 7.0 and should
    | | not be used to name a class, interface or trait or as part of a | | namespace (T_NAMESPACE) FILE: …untested\geoip-detect\vendor\symfony\string\Inflector\EnglishInflector.php FOUND 1 ERROR AFFECTING 1 LINE

    12 | ERROR | 'string' is a reserved keyword as of PHP version 7.0 and should
    | | not be used to name a class, interface or trait or as part of a | | namespace (T_NAMESPACE)

    FILE: …-untested\geoip-detect\vendor\symfony\string\Inflector\FrenchInflector.php FOUND 1 ERROR AFFECTING 1 LINE

    12 | ERROR | 'string' is a reserved keyword as of PHP version 7.0 and should
    | | not be used to name a class, interface or trait or as part of a | | namespace (T_NAMESPACE)

    FILE: …tested\geoip-detect\vendor\symfony\string\Inflector\InflectorInterface.php FOUND 1 ERROR AFFECTING 1 LINE

    12 | ERROR | 'string' is a reserved keyword as of PHP version 7.0 and should
    | | not be used to name a class, interface or trait or as part of a | | namespace (T_NAMESPACE)

    FILE: …content\plugins-untested\geoip-detect\vendor\symfony\string\LazyString.php FOUND 1 ERROR AFFECTING 1 LINE

    12 | ERROR | 'string' is a reserved keyword as of PHP version 7.0 and should
    | | not be used to name a class, interface or trait or as part of a | | namespace (T_NAMESPACE)

    FILE: …lugins-untested\geoip-detect\vendor\symfony\string\Resources\functions.php FOUND 1 ERROR AFFECTING 1 LINE

    12 | ERROR | 'string' is a reserved keyword as of PHP version 7.0 and should
    | | not be used to name a class, interface or trait or as part of a | | namespace (T_NAMESPACE)

    FILE: …ugins-untested\geoip-detect\vendor\symfony\string\Slugger\AsciiSlugger.php FOUND 1 ERROR AFFECTING 1 LINE

    12 | ERROR | 'string' is a reserved keyword as of PHP version 7.0 and should
    | | not be used to name a class, interface or trait or as part of a | | namespace (T_NAMESPACE)

    FILE: …s-untested\geoip-detect\vendor\symfony\string\Slugger\SluggerInterface.php FOUND 1 ERROR AFFECTING 1 LINE

    12 | ERROR | 'string' is a reserved keyword as of PHP version 7.0 and should
    | | not be used to name a class, interface or trait or as part of a | | namespace (T_NAMESPACE)

    FILE: …tent\plugins-untested\geoip-detect\vendor\symfony\string\UnicodeString.php FOUND 1 ERROR AFFECTING 1 LINE

    12 | ERROR | 'string' is a reserved keyword as of PHP version 7.0 and should
    | | not be used to name a class, interface or trait or as part of a | | namespace (T_NAMESPACE)

    Time: 5.13 secs; Memory: 66MB
Viewing 1 replies (of 1 total)
  • Plugin Author Benjamin Pick

    (@benjaminpick)

    The plugin is working with PHP 8.1 / 8.2 without problems. I cannot really resolve the issues for two reasons:

    • The plugins minimum PHP version is 7.2.5
    • When committing the plugin to the wordpress SVN, the PHP files need to be syntactically valid in PHP 5.6 !

    Both requirements should change at some point, but in the meantime, I cannot do anything.

Viewing 1 replies (of 1 total)
  • You must be logged in to reply to this topic.