Skip to content

[PHP8.2] Using ${var} in strings is deprecated fix #2078

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
merged 1 commit into from
Nov 28, 2024
Merged

Conversation

akagane99
Copy link
Contributor

@akagane99 akagane99 commented Nov 26, 2024

概要

上記修正です。

単体テスト

https://github.com/opensource-workshop/connect-cms/actions/runs/12021772987/job/33512856001

php8.2で画面表示にエラーがない事を確認しました。
(※ Connect1.x系のため、実際にはphp8.2はメール処理が動かないです。)

レビュー完了希望日

急ぎません

関連Pull requests/Issues

概要に記載

参考

なし

DB変更の有無

無し

チェックリスト

@akagane99 akagane99 changed the title change: PHP8.2 Using ${var} in strings is deprecated fix [PHP8.2] Using ${var} in strings is deprecated fix Nov 26, 2024
@akagane99 akagane99 self-assigned this Nov 26, 2024
@akagane99 akagane99 added the enhancement 機能強化 label Nov 26, 2024
@akagane99 akagane99 marked this pull request as ready for review November 26, 2024 01:18
Copy link
Contributor

@masaton0216 masaton0216 left a comment

Choose a reason for hiding this comment

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

ご対応ありがとうございます。
挙げてくれた5ファイルは確認しました。
別途、下記1ファイルもHITしたのですが、これも対応しておいた方が良さげですかね?

file_put_contents($path, str_replace('MAIL_FROM_NAME="${APP_NAME}"', "MAIL_FROM_NAME=\"{$request->mail_from_name}\"", file_get_contents($path)));

@akagane99
Copy link
Contributor Author

@masaton0216

別途、下記1ファイルもHITしたのですが、これも対応しておいた方が良さげですかね?

こちら.envのMAIL_FROM_NAME置換処理で、このままで良いかとー。
.env.example
https://github.com/opensource-workshop/connect-cms/blob/master/.env.example#L49

参考:Laravel 11でも.envのMAIL_FROM_NAMEはこのままでした。
https://github.com/laravel/laravel/blob/11.x/.env.example#L58

@masaton0216
Copy link
Contributor

了解です。マージしますー!

@masaton0216 masaton0216 merged commit 40467ea into master Nov 28, 2024
6 of 10 checks passed
@masaton0216 masaton0216 deleted the issues/1785 branch November 28, 2024 08:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement 機能強化
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants