PHP 7

  • Resolved Harry Milatz

    (@harry-milatz)


    9 years, 7 months ago

    Hi,
    maybe for the next update:
    plugins/responsive-menu/src/app/Mappers/scss.inc.php

    ------------------------------------------------------------------------------------------------------------------------------------
FOUND 4 ERRORS AFFECTING 4 LINES
------------------------------------------------------------------------------------------------------------------------------------
 3387 | ERROR | 'string' is a reserved keyword introduced in PHP version 7.0 and cannot be invoked as a function (T_STRING)
 3585 | ERROR | Function name, class name, namespace name or constant name can not be reserved keyword 'string' (since version 7.0)
 3698 | ERROR | 'string' is a reserved keyword introduced in PHP version 7.0 and cannot be invoked as a function (T_STRING)
 3942 | ERROR | 'string' is a reserved keyword introduced in PHP version 7.0 and cannot be invoked as a function (T_STRING)
------------------------------------------------------------------------------------------------------------------------------------

    Best regards,
    Harry

Viewing 4 replies - 1 through 4 (of 4 total)
  • Plugin Contributor responsivemenu

    (@responsivemenu)

    9 years, 7 months ago

    Hey Harry,

    What tool did you use to find this as none of my tools have ever found these issues and I run PHP 7 on all my sites.

    I am very interested to know.

    All the best

    Plugin Contributor responsivemenu

    (@responsivemenu)

    9 years, 7 months ago

    Hi Harry,

    I have updated the function names in the beta version. Can you download it and check that the errors go away?

    https://downloads.wordpress.org/plugin/responsive-menu.beta.zip

    All the best

    Saša

    (@stodorovic)

    9 years, 7 months ago

    Plugin which we used for tests is https://wordpress.org/support/plugin/php-compatibility-checker

    I just tested latest release and it’s OK. Now, I see only warning:

    FILE: …/wp-content/plugins/responsive-menu/src/app/Controllers/Admin.php
    ——————————————————————————————————
    FOUND 0 ERRORS AND 1 WARNING AFFECTING 1 LINE
    ——————————————————————————————————
    1 | WARNING | File has mixed line endings; this may cause incorrect results
    It seems that line (or line before) $updated_options = $this->service->combineOptions($default_options, $imported_options); is ending with LF (Unix style), but rest of lines use CR LF (Windows)

    Basically previous release was also OK. It was issue related to wimg – https://github.com/wimg/PHPCompatibility/issues/115

    I use free version on PHP7 and it works without any issues.

    Plugin Contributor responsivemenu

    (@responsivemenu)

    9 years, 7 months ago

    Hey there,

    Thanks for letting me know and providing the update!

    All the best

Viewing 4 replies - 1 through 4 (of 4 total)

The topic ‘PHP 7’ is closed to new replies.

Tags