Skip to content

Array and string offset access syntax with curly braces #1047

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 1 commit into from
Closed

Array and string offset access syntax with curly braces #1047

wants to merge 1 commit into from

Conversation

sreichel
Copy link
Contributor

@sreichel sreichel commented Jun 18, 2020

Fixed: "PHP Deprecated: Array and string offset access syntax with curly braces is deprecated" for php7.4

Ref #829, #859 (@kkrieger85)

Fixed: "PHP Deprecated:  Array and string offset access syntax with curly braces is deprecated" for php7.4
@sreichel sreichel added the PHP 7.x Related to PHP 7.x label Jun 18, 2020
@sreichel sreichel added this to the PHP 7.4 milestone Jun 18, 2020
Copy link
Contributor

@kkrieger85 kkrieger85 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@rvelhote
Copy link
Contributor

There are also a bunch of equivalent deprecation warnings for Array and string offset access syntax with curly braces is deprecated under lib/Zend however the static code analysis for 7.4 ignores this folder. Is this being done on purpose or it's an oversight?

find . -not \( -path ./.phpstorm.meta.php -prune \) -not \( -path ./lib/PEAR -prune \) -not \( -path ./lib/phpseclib -prune \) -not \( -path ./lib/Zend -prune \) -type f -name "*.php" -exec php -d error_reporting=32767 -l {} \; 2>&1 1> /dev/null | grep "^"

@sreichel
Copy link
Contributor Author

@rvelhote good catch. Libs should be included too.

@sreichel
Copy link
Contributor Author

Please merge #1058 instead. @rvelhote please add your comment there.

@sreichel sreichel closed this Jun 24, 2020
@sreichel sreichel deleted the hotfix/php74/curly-braces branch June 24, 2020 15:41
@sreichel sreichel removed this from the PHP 7.4 milestone Jun 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate PHP 7.x Related to PHP 7.x
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants