CF7のreCAPCHAがフォーム以外のページで表示されないようにする処理を追加

kurudrive commented 11 months ago

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

Contact Form 7 で、ユーザー向けに送信するメールで送信先を [your-email] にしていると、警告が出るようになった。
Contact Form 7 のインテグレーションから reCAPCHA を設定すると警告は解除されるが、全ページに reCAPCHA が表示されてしまう
Invisible reCAPTCHA for WordPress は便利だが更新されてないのでそろそろお勧めするのはまずい

どういう変更をしたか？

Contact Form 7 のフォームを設置したページにのみ reCAPCHA が表示されるように変更

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

ソースコードについて

[x] 複数の意図の変更（機能の不具合修正 + 別の機能追加など）を含んでいないか？
[x] 関数名 / 変数名 / クラス名 / 保存値名はそれだけで内容が想像できるものになっているか？紛らわしい命名になっていないか？
[x] 関数名 / 変数名 / クラス名 / 保存値名は既存のコードの命名規則に沿ったものになっているか？

デザイン・UI

[ ] 初見のユーザーが予備知識無しで使っても使いやすいようになっているか？
[ ] 情報意味を考慮した意味グルーピング・余白になっているか？
[ ] アラートの表示など追加した場合は他の同様の表示と同じデザインになっているか？

-> UIなし

プログラムの変更の場合

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

[ ] 書けそうなテストは書いたか？ → テストサイトのURLに応じたreCAPCHAのキー登録などが必要なため目視確認のみ

その他

[x] readme.txt に変更内容は書いたか？
[x] Files changed (変更ファイル)の内容は目視でちゃんと確認したか？
[x] このチェック項目を機械的にチェックするのではなく本当にちゃんと確認をしたか？
[x] レビュワーが確認しないでリリースしてしまっても問題ないレベルまでちゃんと作りこみ・確認をしたか？

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

CF7でフォームを作成（デフォルトで可）
メールタブでメール (2) を有効にして、送信先メールアドレスが [your-email] になっている事を確認
フォームを保存 -> たぶん警告がでる
お問い合わせ > インテグレーション画面で reCAPCHA をセットアップ
CF7のフォームを固定ページなどに配置
該当の固定ページを見て reCAPCHA が表示される事を確認
ExUnit > 有効化設定で Contact Form 7 アセットファイル最適化を有効化
フォーム以外のページでは reCAPCHA が表示されない事を確認
CF7 を停止してエラーが発生しない事を確認

確認URL

ローカルでよろしくお願いいたします。

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

実装者に同じ
処理で問題点、改善点があれば指摘よろしくお願いいたします。

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

[x] このテンプレートのチェック項目をちゃんと確認してチェックしたか？

レビュワー向け

確認して変更が反映されていない場合の確認事項

プルしたか？
ビルドしたか？
ビルドしたディレクトリは正しいか（別の開発環境のディレクトリを見ていないか）？
npm install したか？
composer install したか？

osmdik commented 11 months ago

@kurudrive 確認しました。

フォーム以外のページでは reCAPCHA が表示されない事を確認

のところで、reCAPCHA が表示されないことは確認できましたがフォームのあるページでも表示されなくなりました。僕の環境のせいだったらすみません。

kurudrive commented 11 months ago

@osmdik テーマ何の環境です？

osmdik commented 11 months ago

@kurudrive TT3 と X-T9 で確認しました。

kurudrive commented 11 months ago

@osmdik あー...ブロックテーマだと挙動が違うっぽい...orz ありがとうございました。

osmdik commented 11 months ago

@kurudrive lightningでは正常に動作することが確認できました！

kurudrive commented 11 months ago

@osmdik 該当不具合修正しました！ありがとうございました！

vektor-inc / vk-all-in-one-expansion-unit