Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Deprecated: Use of "self" in callables is deprecated in VkCssOptimize.php on line 325 を修正 #16

Merged
merged 12 commits into from
Sep 25, 2024
Merged

Deprecated: Use of "self" in callables is deprecated in VkCssOptimize.php on line 325 を修正 #16

merged 12 commits into from
Sep 25, 2024

Conversation

drill-lancer
Copy link
Member

@drill-lancer drill-lancer commented Jun 19, 2024 •
edited
Loading

チケットへのリンク / 変更の理由（元のissueがあればリンクを貼り付ければOK）

Deprecated: Use of "self" in callables is deprecated in VkCssOptimize.php on line 325

どういう変更をしたか？

Deprecated: Use of "self" in callables is deprecated in VkCssOptimize.php on line 325 を修正

実装者の確認事項

実装者はレビュワーに回す前に以下の事を確認してチェックをつけてください。

  • 複数の意図の変更 （ 機能の不具合修正 + 別の機能追加など ） を含んでいないか？
  • Files changed (変更ファイル)の内容は目視で確認したか？
  • readme.txt に変更内容は書いたか？
  • 本当にちゃんと確認をしたか？

プログラムの変更の場合

テストを書かないのは普通ではありません。書けるテストは極力書くようにしてください。

  • 書けそうなテストは書いたか？
    ⇒ Deprecated を修正しただけなので省略したが、PHP 7.4 ~ 8.1 で動作するかの確認を追加しました。

変更内容について何を確認したか、どういう方法で確認をしたかなど

  1. Tree Shaking を有効化
  2. Deprecated: Use of "self" in callables is deprecated in VkCssOptimize.php on line 325 が出るのを確認
  3. VK Blocks Pro の composer.json の vektor-inc/vk-css-optimize のバージョンを dev-fix/deprecated-self-325 に変更
  4. vendor フォルダと composer.lock を削除した状態で composer install
  5. Deprecated: Use of "self" in callables is deprecated in VkCssOptimize.php on line 325 が消えているを確認
  6. ページのソースを開いて Tree Shaking が効いているのを確認

確認URL

ローカル環境

レビュワーに回す前の確認事項

  • 実装者はこのテンプレートのチェック項目をちゃんと確認してチェックしたか？

レビュワー確認方法・確認内容など

  1. Tree Shaking を有効化
  2. Deprecated: Use of "self" in callables is deprecated in VkCssOptimize.php on line 325 が出るのを確認
  3. VK Blocks Pro の composer.json の vektor-inc/vk-css-optimize のバージョンを dev-fix/deprecated-self-325 に変更
  4. vendor フォルダと composer.lock を削除した状態で composer install
  5. Deprecated: Use of "self" in callables is deprecated in VkCssOptimize.php on line 325 が消えているを確認
  6. ページのソースを開いて Tree Shaking が効いているのを確認

レビュワー向け

レビュワーが確認して変更が反映されていない場合の確認事項

レビューしてみて意図した動作をしない場合は再度ビルドするなど以下の項目を確認してください。

  • プルしたか？
  • ビルドしたか？
  • ビルドしたディレクトリは正しいか（別の開発環境のディレクトリを見ていないか）？
  • npm install したか？
  • composer install したか？
  • キャッシュをクリアして確認したか？

@drill-lancer drill-lancer requested a review from kurudrive June 19, 2024 01:33
@drill-lancer drill-lancer self-assigned this Jun 19, 2024
@drill-lancer drill-lancer marked this pull request as ready for review June 19, 2024 01:47
@drill-lancer drill-lancer requested a review from mthaichi June 19, 2024 01:47
@goutetsuguma
Copy link

Deprecated: Use of "self" in callables is deprecated in VkCssOptimize.php on line 325 が出るのを確認

こちらを確認しようと思いましたが、上記のエラーがわたしのローカル環境では確認できませんでした、、、
わたしの手順の問題の可能性もありますが、どなたか他の方ご確認いただけますか

@drill-lancer
Copy link
Member Author

drill-lancer commented Aug 26, 2024 •
edited
Loading

@goutetsuguma
PHP が 8.0 以上でないとエラーは発生しません。

@goutetsuguma
Copy link

@drill-lancer
テスト環境は PHP バージョン 8.2.9 でした。
すみません、vk-css-optimizeについてわたしがそもそも理解不足でして、、、
vk blocks proを developにして確認しておりますが、そもそも確認の仕方が違うのかもしれません、、

@drill-lancer
Copy link
Member Author

@goutetsuguma
多分 wp-config.php のデバッグモードが発動していないとかでしょうか？

@goutetsuguma
Copy link

ありがとうございます、デバックはtrueになっていました。
もしかすると私の環境の問題・確認の仕方がそもそも間違っているなどありそうですので、他の方で確認できましたらよろしくお願いします🙇

@goutetsuguma
Copy link

ああ、、、すみませんよくよくみたら画面上にエラーメッセージを表示しないようにしていました、、、（define('WP_DEBUG_DISPLAY', false);）
確認します

@goutetsuguma
Copy link

goutetsuguma commented Sep 11, 2024 •
edited
Loading

レビュワー確認方法で確認して、エラーが消えることを確認しました。Tree Shaking を有効でインラインで出力されていることを確認できました。

2人目確認待ち（で良いのかな？）にします
vk-css-optimize のレビューの流れを、私があまりわかっておりません。何か違いましたら教えてください

@goutetsuguma goutetsuguma changed the title 【確認待ち】Deprecated: Use of "self" in callables is deprecated in VkCssOptimize.php on line 325 を修正 【2人目確認待ち】Deprecated: Use of "self" in callables is deprecated in VkCssOptimize.php on line 325 を修正 Sep 11, 2024
@kurudrive
Merge branch 'main' into fix/deprecated-self-325
9a6b080 
# Conflicts:
#	README.md
#	package-lock.json
#	package.json
kurudrive
kurudrive requested changes Sep 25, 2024
View reviewed changes
@kurudrive kurudrive changed the title 【2人目確認待ち】Deprecated: Use of "self" in callables is deprecated in VkCssOptimize.php on line 325 を修正 Deprecated: Use of "self" in callables is deprecated in VkCssOptimize.php on line 325 を修正 Sep 25, 2024
@kurudrive kurudrive merged commit 248c769 into main Sep 25, 2024
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kurudrive kurudrive kurudrive requested changes

@mthaichi mthaichi Awaiting requested review from mthaichi

@sysbird sysbird Awaiting requested review from sysbird

@goutetsuguma goutetsuguma Awaiting requested review from goutetsuguma

@mtdkei mtdkei Awaiting requested review from mtdkei

Assignees

@drill-lancer drill-lancer

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@drill-lancer @goutetsuguma @kurudrive