Skip to content

Conversation

@kazunorimiura
Copy link
Contributor

概要

is_saved_the_key() メソッドから、 auto-draft 状態の投稿に対する早期リターン判定を削除しました。

変更理由

現在の実装では、投稿ステータスが auto-draft の場合、メタデータの存在に関わらず常に false を返していました。しかし、 auto-draft 状態でメタデータを保存するプラグインが存在します。たとえば、Polylangプラグインは、投稿の複製時に auto-draft 状態で本文やメタデータを引き継ぐ実装を行っています。

メタデータ存在確認のクエリを節約できるという理由以外に、auto-draft チェックを入れる理由が思い当たらなかったため、常に存在確認するように変更しました。

変更内容

  • auto-draft ステータスのチェックを削除

テスト方法

  1. WordPress 6.8.3をインストール
  2. Smart Custom Fields 5.0.4を有効化し、text、wysiwygタイプのフィールドを持つリピートフィールドを作成
  3. Polylang Pro 3.7.4を有効化し、日・英・仏・中の言語を設定
  4. 日本語版の投稿を作成
  5. 日本語版の投稿を複製するかたちで英・仏・中版の投稿を作成
  6. Smart Custom Fields経由で登録されたメタデータが複製され、正しくフォームにセットされていることを確認

@inc2734
Copy link
Owner

inc2734 commented Nov 19, 2025

理由はもう覚えていないのですが、ご指摘どおりな気がするのでマージします!

@inc2734 inc2734 merged commit afbede3 into inc2734:master Nov 19, 2025
inc2734 added a commit that referenced this pull request Nov 19, 2025
@kazunorimiura
Copy link
Contributor Author

ありがとうございます!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants