Backport(バックポート) #2513
openナインテーブルガジェットを3行x2列にしても3行目が表示されない
0%
Description
概要¶
フレンドリストやコミュニティリストは、3x3の9面表示を行うパーツ apps/pc_frontend/templates/_partsNineTable.php を用いて表示している。
管理画面の「デザイン設定」、「ホーム画面ガジェット設定」では、フレンドリストなどの9面表示を行うガジェットに対して、「表示する行数・列数」を設定できる。
表示できるアイテム数が十分ある場合に、
- 6行1列と設定していると、1行x1列の表示となる
- 1行6列と設定していると、6行x6列の表示となり、2行目以降は空となる
- アイテム数が十分であっても、DBからアイテムを取得する際に設定値の「行数x列数」個しか取得しないようにアクションファイルが記述されている(_partsNineTable を呼んでいる側が行っている実装であり、実装によってはこの限りではない)
- 3行2列と設定していると、2行x2列の表示となる(報告のケース)
原因¶
apps/pc_frontend/templates/_partsNineTable.php の行数部分のループは http://trac.openpne.jp/changeset/9630 で書かれているが、実際に表示する行数 $row を算出する式が誤っている。
11:<?php $row = ceil(count($options->list) / $options->row) ?> 12-<?php for ($i = $j = 1; $row >= $i; $i++): ?>
(取得したアイテム数)を(設定した行数)で割った値の小数部を切り上げた値を「表示する行数」としているのは明らかにおかしい。
- http://trac.openpne.jp/changeset/9630
- http://trac.openpne.jp/changeset/10286#file13
- http://trac.openpne.jp/changeset/10609
修正方針¶
<?php $row = ceil(count($options->list) / $options->row) ?>
を
<?php $row = min($options->row, ceil(count($options->list) / $options->col)) ?>
のように変更する。 ceil(x) は、実数 x に対して、x の次に大きい整数値を(float型で)返す関数である。具体例と併せてこの修正の意味を示しておく。
- 2行5列で表示させたい場合:
- アイテムが10件取得できれば、min(2行, ceil(10件 / 5列)) = 2 となり、表示は2行となる
- アイテムが4件しかなければ、min(2行, ceil(4件 / 5列)) = min(2, ceil(0.8)) = min(2, 1) = 1 となり、表示は1行となる(1行5列)
- 3行3列で表示させたい場合:
- アイテムが9件取得できれば、min(3行, ceil(9件 / 3列)) = 3 となり、表示は3行となる
- アイテムが5件しかなければ、min(3行, ceil(5件 / 3列)) = min(3, ceil(1.666)) = min(3, 2) = 2 となり、表示は2行となる(2行3列)
- アイテムが1件しかなければ、min(3行, ceil(1件 / 3列)) = min(3, ceil(0.333)) = min(3, 1) = 1 となり、表示は1行となる(1行3列)
『表示行数 = ceil(アイテム数, 設定列数)』という式でもよさそうだが、概要で示したとおり「アイテム数が十分であっても、DBからアイテムを取得する際に設定値の「行数x列数」個しか取得しないようにアクションファイルが記述されている」のは実装側の判断であって、partsNineTable を使用する箇所で取得するアイテム数が「行数x列数」以下になる保証はないため、『表示行数 = min(設定行数, ceil(アイテム数, 設定列数))』と記述しておくことにする。
仕様について¶
取得できたアイテム数、設定された行数、設定された列数、このそれぞれについて、実際に表示される行数列数が何になるべきかについては、2系および修正前の3系から以下のように判断します。
- 3行3列の場合:
- アイテム1個:1行3列
- アイテム1個〜3個:1行3列
- アイテム4個:2行3列
- アイテムk個(k<=9):ceil(k/3)行3列
- r行c列の場合:
- アイテム1個:1行c列
- アイテムk個(k<=c):1行c列
- アイテムk個(c<k<=r*c):ceil(k/c)行c列
ここで示したいことは端的に言うと、アイテム数に応じて「空の行は作らない」が「空の列は作りうる」ということである。
補足¶
$row に ceil() の返り値が代入されることがあり、その場合 float 型になってしまうがこのチケットでは特に気にしないことにする。 $row は int 型を想定するといった修正や、このファイルの可読性を向上させることは現時点のソースコードに対しては可能だと思われる。そうした修正については対応するのであれば別チケットで行う。
報告¶
http://sns.openpne.jp/communityTopic/6408 より転記
3.6beta5 【ガジェット】ナインテーブルを2x3にしたら3個目が表示されない。 画像を見てもらうとわかると思いますが、横2縦3にしたときに3番目のデータが表示されない。 ↓余談 それとコミュニティのトップのコミュニティメンバー(ナインテーブル)もガジェットで編集できるようにしてほしいです。
もともと記載されていた内容¶
openpne3.6「 _partsNineTable.php」の rowとcolの関係を正しく表示できていない
http://sns.openpne.jp/diary/26976 より転記
トピック¶
_partsNineTable.php の11行目 <?php $row = ceil(count($options->list) / $options->row) ?> は <?php $row = ceil(count($options->list)) ?> じゃないとrowとcolの関係を正しく表示しない。 low=1 col=6 とか
Updated by Yuya Watanabe about 13 years ago
- Tracker changed from Bug(バグ) to Backport(バックポート)
- Subject changed from openpne3.6「 _partsNineTable.php」の rowとcolの関係を正しく表示できていない to ナインテーブルガジェットを3行x2列にしても3行目が表示されない
- Description updated (diff)
#1676 「ナインテーブルガジェットを3行x2列にしても3行目が表示されない」が 3.6 へバックポートされていないことが原因です.本チケットをBPチケットに変更して対応を行う必要があります.