search

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

wordpress plugin of powerful support
https://ja.wordpress.org/plugins/vk-all-in-one-expansion-unit/
GNU General Public License v2.0
7 stars 1 forks source link

【修正待ち】【カスタム投稿タイプマネージャー】カスタム投稿タイプを生成した瞬間にパーマリンクを修正するように変更 #714

Closed drill-lancer closed 3 years ago

drill-lancer commented 3 years ago

カスタム投稿タイプマネージャーでカスタム投稿タイプを生成した瞬間にパーマリンクを修正するように変更しました。 一応手動でパーマリンク更新を促すメッセージはそれをするのが王道という意味で残しておきました。

kurudrive commented 3 years ago

@drill-lancer これ post 保存する時のアクションフックの方がいいんじゃない？ カスタムフィールド増やさなくていいし。

drill-lancer commented 3 years ago

これ post 保存する時のアクションフックの方がいいんじゃない？ カスタムフィールド増やさなくていいし。

カスタム投稿を登録するより前のタイミングなのでそれだと意味がなくなるような気がしたのでこのようにしました。

kurudrive commented 3 years ago

@drill-lancer 元々 save_post だったのね。 これだと問題出る？ https://github.com/vektor-inc/vk-all-in-one-expansion-unit/pull/716/files

drill-lancer commented 3 years ago

@kurudrive 問題がでるのかどうかがわかりづらいです。 一見問題なさそうに見えますが・・・

kurudrive commented 3 years ago

@drill-lancer あぁ、いや

カスタム投稿を登録するより前のタイミングなのでそれだと意味がなくなるような気がした

というのが RICK さん方式の書き方になった理由だったから、 多分 カスタム投稿の登録より後で実行されるので、僕が確認する限りは問題なかったんだけど、 実際には何か発生したりしたからこの方式をとらなかったのかなーと思ったので確認ですー。

RICKさんが試して問題なければこれでいきます。

でも単純に元コードがあったからすぐできただけだし、RICKさんの作業は無駄じゃないよ！