Skip to content

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

Merged
masaton0216 merged 1 commit into
masterfrom
issues/1785
Nov 28, 2024
Merged

[PHP8.2] Using ${var} in strings is deprecated fix#2078
masaton0216 merged 1 commit into
masterfrom
issues/1785

Conversation

@akagane99
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Contributor

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

@masaton0216 masaton0216 merged commit 40467ea into master Nov 28, 2024
@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