Project

General

Profile

Backport(バックポート) #2513

ナインテーブルガジェットを3行x2列にしても3行目が表示されない

Added by pnetan   almost 8 years ago. Updated about 5 years ago.

Status:
New(新規)
Priority:
Normal(通常)
Assignee:
-
Target version:
Start date:
2011-10-18
Due date:
% Done:

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++): ?>

(取得したアイテム数)を(設定した行数)で割った値の小数部を切り上げた値を「表示する行数」としているのは明らかにおかしい。

修正方針

<?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
とか

Related issues

Related to OpenPNE 3 - Bug(バグ) #1676: ナインテーブルガジェットを3行x2列にしても3行目が表示されない Fixed(完了) 2010-10-14

History

#1 Updated by Yuya Watanabe almost 8 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チケットに変更して対応を行う必要があります.

#2 Updated by Rimpei Ogawa about 5 years ago

  • Target version set to OpenPNE 3.6.x

Also available in: Atom PDF