人気記事の数がゼロということはほぼほぼレアケースであるということ（動作確認などで管理者が最良となる出力結果を選ぶと思われるため）

拝承。

確かに必要ないと思います。削除しておきました。

対策されていることを確認しました。
本件をクローズとします。

カテゴリーを取得し、get_access_ranking_recordsに渡す必要はありますか？

確かに必要ないと思います。削除しておきました。

人気記事を選択した際、期間中の人気記事がゼロだった。
このとき、post__inはnullとなり、全記事が対象となってしまう。
集計結果がゼロのときは表示しない？

こちらは以下の理由から、このままの仕様でいかせていただこうと思います。

1. 人気記事の数がゼロということはほぼほぼレアケースであるということ（動作確認などで管理者が最良となる出力結果を選ぶと思われるため）
2. カルーセルに何も表示されない（カルーセルが非表示となる）よりは、何かしら記事が出て微細ながらも回流に貢献した方が良いと思うから

以下でカテゴリーを反映しており、人気記事 & カテゴリーと絞り込みとなっています。
カテゴリーを取得し、get_access_ranking_recordsに渡す必要はありますか？

https://github.com/xserver-inc/cocoon/blob/2118f99aa56354e8ea0b64bf3fa007021a51d47a/tmp/carousel.php#L25-L26

array(4) {
  =>
  string(9) "post_date"
  =>
  bool(true)
  =>
  string(2) "18"
  =>
  array(0) {
  }
}

説明文は言われるように「人気記事、カテゴリー、タグの順に優先され、いずれか一つが表示されます。」とすればいいのでは？

こちらも追加しておきました。

しかし既に設定されている場合、影響があるかも知れず。
このため、現状のままチェックボックスが安全かと思います。

ラジオボタンだとこれが怖いんですよね。
すでにCocoon設定を変更されている方が、戸惑いが少なく移行するためにも、説明文の変更で対応させていただきました。
https://github.com/xserver-inc/cocoon/commit/5f864c8f96914bb90bf041d411bc9d1dbb084df3

tmp/carousel.phpを以下のように変更する。

https://notepad.pw/share/fPtc3ijPdElC5wTd5AJY

●説明文

ラジオコントロールを追加して、どれかを選ばせるUIにするか。

それとも現在のUIはそのままで「人気記事 > カテゴリ > タグ」の優先度でそれぞれ個別に表示されるという旨の説明文を追加するか。

説明文は言われるように「人気記事、カテゴリー、タグの順に優先され、いずれか一つが表示されます。」とすればいいのでは？

また、ラジオボタンに変更した場合、説明がなくても動作が明確でスッキリします。
しかし既に設定されている場合、影響があるかも知れず。
このため、現状のままチェックボックスが安全かと思います。

カルーセルは意図した投稿IDをカード表示しているわけではないため、リースノートに仕様変更の注意点を記し、再設定してもいいかと思います。

仕様を見直し、以下の条件とした方が、一番分かりがいいかとと思いました。

• 人気記事
• カテゴリーごと
• タグごと

処理の方はそのようにできるけど、説明文のUIはどうしたらよいものかと思いました。

ラジオコントロールを追加して、どれかを選ばせるUIにするか。

それとも現在のUIはそのままで「人気記事 > カテゴリ > タグ」の優先度でそれぞれ個別に表示されるという旨の説明文を追加するか。

例えば、人気記事の説明文に「人気記事を選択すると、人気記事のみが優先して表示されます。」

カテゴリの説明文に「カテゴリを選択すると、カテゴリーに関連付けられた記事のみが表示されます。（※カテゴリー関連記事を表示させたい場合は、人気記事を選択しないでください）」

タグの説明文に「タグを選択すると、タグに関連付けられた記事のみが表示されます。（※タグ関連記事を表示させたい場合は、人気記事・カテゴリーを選択しないでください）」

みたいに書くとか。

ただいずれの方法もちょっと分かりづらいかもしれません。

●人気記事「本日」＋カテゴリー

上記ですが、人気記事でもなく、カテゴリーでもなく、全記事が対象となってしまうので、このケースはバグだと思います。

以下の人気記事の条件にカテゴリーが含めようとするので、無理があり。

https://github.com/xserver-inc/cocoon/blob/2a6e06f825b0948f075859ebf0dc814c6897d94a/tmp/carousel.php#L21-L30

仕様を見直し、以下の条件とした方が、一番分かりがいいかとと思いました。

• 人気記事
• カテゴリーごと
• タグごと