Skip to content

New notice "Deprecated: Using ${var} in strings is deprecated, use {$var} instead in...JsonPath.php on line 77" #74

@das-peter

Description

@das-peter

Update the package today and now get the mentioned deprecated exception (using php 8.2.27):

Deprecated: Using ${var} in strings is deprecated, use {$var} instead in /var/www/drupal/vendor/galbar/jsonpath/src/Galbar/JsonPath/JsonPath.php on line 77

I think this is related to #72 particularly https://github.com/Galbar/JsonPath-PHP/pull/72/files#diff-20616e2324ff697f3485512697cce8ddf24d9f0c73e2b4a559d29d009e7aad3eR77

Where

$recursivePath = "$$recursivePath";

became

$recursivePath = "${$recursivePath}";

If I got that correct that was never a variable variable but is intended to be a "short" for '$' . $recursivePath.
The deprecation warning makes me concerned this might now be evaluated as variable variable.

We might want to change this to $recursivePath = '$' . $recursivePath; to avoid any potential for collision - given I'm right about my interpretation of the code.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions