【確認待ち】[ 投稿タイプマネージャー ] 投稿タイプの設定を保存した時に自動的にパーマリンク保存

drill-lancer commented 1 year ago

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

https://github.com/vektor-inc/vektor-wp-libraries/issues/124

どういう変更をしたか？

カスタム投稿タイプの設定を保存した時に自動的にパーマリンク保存するように変更
確認中に上記カスタム投稿タイプで広告アラートで発生したエラーを修正

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

ソースコードについて

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

デザイン・UI

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

プログラムの変更の場合

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

[ ] 書けそうなテストは書いたか？ ⇒テストが書けそうにないので省略

その他

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

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

カスタム投稿タイプ作成
パーマリンクを更新せずに1️⃣の投稿タイプで投稿を作成
2️⃣の記事が表示されるのを確認

確認URL

ローカル環境

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

カスタム投稿タイプ作成
パーマリンクを更新せずに1️⃣の投稿タイプで投稿を作成
2️⃣の記事が表示されるのを確認

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

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

レビュワー向け

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

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

kurudrive commented 1 year ago

@drill-lancer ありがとうございます。一番最初に発行した時にはまだパーマリンクが書き換わらないようなのですが、投稿タイプの設定を保存し終わったタイミングでパーマリンクを上書きってできませんかね（・ｗ・？

https://github.com/vektor-inc/vk-all-in-one-expansion-unit/assets/3272660/eea4fca3-98fb-42db-9eb4-a5c38e7a9915

あと、これがうまくいったら↓は不要なので削除したいです。

スクリーンショット 2023-10-08 19 46 26

kurudrive commented 1 year ago

※ |￣￣￣￣────￣￣￣￣） 10/10以降の対応で結構です

drill-lancer commented 1 year ago

@kurudrive パーマリンクを更新するのは flush_rewrite_rules が有効のようですがどうやら register_post_type が発動するタイミングでないと flush_rewrite_rules を発動する意味はなさそうです。ただそのタイミングで発動すると毎回発動することになるのでそこで新たな不具合の火種になりそうです。発動済みか否かのオプション値を設けて制御する方法を考えています。

https://elearn.jp/wpman/function/flush_rewrite_rules.html https://torounit.com/blog/2015/09/02/2077/

kurudrive commented 1 year ago

保存が完了した時の save option 関連のアクションフックってなかったっけ？あれば、この投稿タイプの保存だった場合に、保存完了時に走らせればいけるかなと思ってました。なんとなくそう思ってただけで調べてません...。

drill-lancer commented 1 year ago

@kurudrive 動画を見ていると

①最初にカスタム投稿タイプを保存したときには register_post_type が走っていないので以前のパーマリンクがそのまま保存される ②次に保存したときには register_post_type が走った後なのでパーマリンクが更新された

というように見受けられるので register_post_type が走った後でないと無理だと思います。

drill-lancer commented 1 year ago

@kurudrive カスタム投稿マネージャーの add_post_type 内の register_post_type が走ったときに一度も flush_rewrite_rules が走っていなければ走らせるようにしました。判定にはその投稿のカスタムフィールドを使用しています。

kurudrive commented 1 year ago

@drill-lancer 広告アラートの変更とコンフリクトしたっぽいので新しい方でコミットしてくださいーん。

drill-lancer commented 1 year ago

@kurudrive 競合解消しました。

kurudrive commented 1 year ago

@drill-lancer ばっちりです！ありがとうございます！

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