Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Remove deprecated interpolation in functions.php #506

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

Merged

Conversation

@adambalint-srg
Copy link
Contributor

@adambalint-srg adambalint-srg commented Nov 16, 2022

${var} interpolation will be deprecated in php8.2. With this fix it is replaced with simple interpolation inside double quotes.

Link to RFC: https://wiki.php.net/rfc/deprecate_dollar_brace_string_interpolation

@oscarotero oscarotero merged commit d0ea95c into php-embed:master Nov 16, 2022
Copy link
Collaborator

Ok, thanks.
The tests are failing due a different issue.

adambalint-srg reacted with thumbs up emoji

Copy link

imliam commented Dec 12, 2022

@oscarotero thanks for merging this fix. Any chance of releasing a new tag with this fix now that PHP 8.2 is released? 🙏

Copy link
Collaborator

@imliam Yes, sorry. New version 4.4.7 released

imliam reacted with thumbs up emoji

LeSuisse pushed a commit to Enalean/tuleap that referenced this pull request Jan 9, 2023
This is part of request #30344: Run Tuleap with PHP 8.2
* Changelog: https://github.com/oscarotero/Embed/blob/v4.4.7/CHANGELOG.md
* Issue: php-embed/Embed#506
Change-Id: I6bdd650580327c7d65444d8a3e74982f9ec41126
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /