wordpressでタグ保存する「wp_set_object_terms」において、配列で一括させたい

Question

###目的
タグをフロントエンドで選択保存すべく、後述のようにwp_set_object_termsを使っています。

機能の実装はできたのですが、タグ数が多いので保存の関数を一括させたいと考えています。

###現状のコード
現状は１つず保存しています。

// 選択
<input type="checkbox" name="one" value="one">one 
<input type="checkbox" name="two" value="two">two
<input type="checkbox" name="three" value="three">three

// 保存（１つずつ）
if (isset($_POST) && $_POST['one']!='') {
  wp_set_object_terms( $topic_id, 'one', 'custom-tag', true );
}  
if (isset($_POST) && $_POST['two']!='') {
  wp_set_object_terms( $topic_id, 'two', 'custom-tag', true );
}  
if (isset($_POST) && $_POST['three']!='') {
  wp_set_object_terms( $topic_id, 'three', 'custom-tag', true );
} 

しかしタグはoneとtwoとthreeだけでなくたくさんあるので、上の保存を一括させたいです。

###目的のためのコード
そこで保存の部分を配列で一括させるべく、次のようなコードに挑戦してみました。

// 選択
<input type="checkbox" name="custom-tag[]" value="one">one 
<input type="checkbox" name="custom-tag[]" value="two">two
<input type="checkbox" name="custom-tag[]" value="three">three

// 保存（一括）
if (isset($_POST) && $_POST['custom-tag']!='') {
  wp_set_object_terms( $post_id, 'custom-tag[]', 'custom-tag', true );
}  

しかしこれではうまくいきませんでした。

たとえばoneとtwoにチェックして保存しても、custom-tag[]という文字がそのまま、custom-tagのタグとして保存されてしまいます。

言うまでもなく保存したいのはoneとtwoで、custom-tag[]という文字列ではありません…。

そのためにどこを書き直せばいいか、もしお分かりになる方がいらっしゃいましたらご教示頂けましたら幸いです。

宜しくお願い致します。

Answer 1

wp_set_object_terms( $post_id, 'custom-tag[]', 'custom-tag', true );

は、'custom-tag[]' という文字列を渡しているので、そのまま登録されてしまいます。

まとめて渡すのであれば、配列を渡す必要があります。

$_POST['custom-tag'] に入っているのが、custom-tag タームの ID かスラッグであれば、試していませんが、こんな感じでどでしょうか？

if (isset($_POST) && $_POST['custom-tag']!='') {
  wp_set_object_terms( $post_id, $_POST['custom-tag'], 'custom-tag', true );
}

関数リファレンス/wp_set_object_terms 参照

Comments

[CHERRY]

if 文も isset($_POST['custom-tag']) で簡潔になりそうだけど...

[EggBenedict]

できました！どうもありがとうございます！