PHP 7 Compatibility

  • Resolved Myar

    (@myar)


    5 years, 6 months ago

    Hi,

    while checking plugins for newest php version compatibility, I found errors with your code. Some are are security issues as well:

    FILE: /var/www/html/public/lostingames/wp-content/plugins/wp-rss-aggregator/vendor/dhii/stringable-interface/src/StringableInterface.php
    ———————————————————————————————————————————————————————
    FOUND 1 ERROR AFFECTING 1 LINE
    ———————————————————————————————————————————————————————
    3 | 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: /var/www/html/public/lostingames/wp-content/plugins/wp-rss-aggregator/vendor/twig/twig/src/Lexer.php
    ———————————————————————————————————-
    FOUND 0 ERRORS AND 2 WARNINGS AFFECTING 2 LINES
    ———————————————————————————————————-
    171 | WARNING | INI directive ‘mbstring.func_overload’ is deprecated since PHP 7.2
    175 | WARNING | INI directive ‘mbstring.func_overload’ is deprecated since PHP 7.2
    ———————————————————————————————————-

    FILE: /var/www/html/public/lostingames/wp-content/plugins/wp-rss-aggregator/vendor/twig/twig/src/Profiler/Profile.php
    ——————————————————————————————————————————————————–
    FOUND 0 ERRORS AND 2 WARNINGS AFFECTING 2 LINES
    ——————————————————————————————————————————————————–
    174 | WARNING | Method name “Profile::__serialize” is discouraged; PHP has reserved all method names with a double underscore prefix for future use.
    182 | WARNING | Method name “Profile::__unserialize” is discouraged; PHP has reserved all method names with a double underscore prefix for future use.
    ——————————————————————————————————————————————————–

    FILE: /var/www/html/public/lostingames/wp-content/plugins/wp-rss-aggregator/vendor/twig/twig/src/Compiler.php
    ————————————————————————————————————-
    FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
    ————————————————————————————————————-
    223 | WARNING | INI directive ‘mbstring.func_overload’ is deprecated since PHP 7.2
    ————————————————————————————————————-

    FILE: /var/www/html/public/lostingames/wp-content/plugins/wp-rss-aggregator/includes/opml-importer.php
    —————————————————————————————————————————————————————-
    FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
    —————————————————————————————————————————————————————-
    48 | WARNING | Method name “WPRSS_OPML_Importer::__contruct” is discouraged; PHP has reserved all method names with a double underscore prefix for future use.
    —————————————————————————————————————————————————————-

    FILE: /var/www/html/public/lostingames/wp-content/plugins/wp-rss-aggregator/includes/Aventura/Wprss/Core/DataObject.php
    ——————————————————————————————————————————————————-
    FOUND 0 ERRORS AND 3 WARNINGS AFFECTING 3 LINES
    ——————————————————————————————————————————————————-
    387 | WARNING | Method name “DataObject::__toArray” is discouraged; PHP has reserved all method names with a double underscore prefix for future use.
    443 | WARNING | Method name “DataObject::__toXml” is discouraged; PHP has reserved all method names with a double underscore prefix for future use.
    488 | WARNING | Method name “DataObject::__toJson” is discouraged; PHP has reserved all method names with a double underscore prefix for future use.
    ——————————————————————————————————————————————————-

    FILE: /var/www/html/public/lostingames/wp-content/plugins/wp-rss-aggregator/includes/Aventura/Wprss/Core/Model/Regex/HtmlEncoder.php
    ————————————————————————————————————————————
    FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
    ————————————————————————————————————————————
    562 | WARNING | Function each() is deprecated since PHP 7.2; Use a foreach loop instead
    ————————————————————————————————————————————

    FILE: /var/www/html/public/lostingames/wp-content/plugins/wp-rss-aggregator/includes/feed-access.php
    —————————————————————————————————-
    FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
    —————————————————————————————————-
    411 | WARNING | INI directive ‘safe_mode’ is deprecated since PHP 5.3 and removed since PHP 5.4
    —————————————————————————————————-

    FILE: /var/www/html/public/lostingames/wp-content/plugins/wp-rss-aggregator/includes/libraries/php-markdown/markdown.php
    ————————————————————————————————————————
    FOUND 0 ERRORS AND 3 WARNINGS AFFECTING 3 LINES
    ————————————————————————————————————————
    242 | WARNING | Use of deprecated PHP4 style class constructor is not supported since PHP 7.
    1653 | WARNING | Function create_function() is deprecated since PHP 7.2; Use an anonymous function instead
    1694 | WARNING | Use of deprecated PHP4 style class constructor is not supported since PHP 7.
    ————————————————————————————————————————

    FILE: /var/www/html/public/lostingames/wp-content/plugins/wp-rss-aggregator/includes/system-info.php
    ————————————————————————————————————–
    FOUND 2 ERRORS AND 1 WARNING AFFECTING 3 LINES
    ————————————————————————————————————–
    81 | WARNING | INI directive ‘safe_mode’ is deprecated since PHP 5.3 and removed since PHP 5.4
    277 | ERROR | Extension ‘mysql_’ is deprecated since PHP 5.5 and removed since PHP 7.0; Use mysqli instead
    279 | ERROR | Extension ‘mysql_’ is deprecated since PHP 5.5 and removed since PHP 7.0; Use mysqli instead
    ————————————————————————————————————–

Viewing 1 replies (of 1 total)
  • Plugin Author Mark Zahra

    (@markzahra)

    Hi @myar, we can confirm that these warnings and errors are not harmful.

    Unfortunately, there’s nothing we can do for the first one – it’s a class that’s needed and it’s name has “string” in it so this may continue to appear but have no negative impact on your site.

    As for the rest, we can do more ourselves to prevent these warnings. Our developers have been informed and they will be working on them in future updates.

Viewing 1 replies (of 1 total)
  • The topic ‘PHP 7 Compatibility’ is closed to new replies.