Bug（バグ） #1797
完了
#1432 で修正した $size に整数値以外の値を渡すことができる
100%
説明
Shinichi Urabe さんが13年以上前に更新
- ステータス を New（新規） から Pending Review（レビュー待ち） に変更
- 進捗率 を 0 から 50 に変更
更新履歴 2f40254be35af0ee922eb3b5d9c7b9d57d94d3f2 で適用されました。
Masato Nagasawa さんが13年以上前に更新
- ステータス を Pending Review（レビュー待ち） から Rejected（差し戻し） に変更
2点気になったのでフィードバックします。
1.
許可サイズのリストが存在するのであれば、
ページャを取得するメソッド側でチェックを行うべきではないでしょうか？
FileTable::getImageFilePager()
FileTable::getFilePager()
2.
apps/pc_backend/templates/_changePageSize.php で
$default 変数が使用されていないようなので、
不要であれば削除をお願いします。
また個人的に FileTable::getDefaultPagerSize() がリテラルで指定されていることが気になりました。
return $allowedPagerSizeList[0];
のように、許可リストから値を取得するほうが自然ではないでしょうか。
Shinichi Urabe さんが13年以上前に更新
- ステータス を Rejected（差し戻し） から Pending Review（レビュー待ち） に変更
272e6e9d61123881a6744e19111d3850c5e8bf90 でFileTable クラス内のページャメソッドでサイズチェックするように修正し、デフォルト値はメンバ変数の配列の値を指定するように変更しました。
a4544dc848ea068b89891160ce8759102d539086 で使われていない変数を削除、変数名が$fileSizeListだと意味合いが違うので、$pagerSizeListに変更しました。
Masato Nagasawa さんが13年以上前に更新
- ステータス を Pending Review（レビュー待ち） から Pending Testing（テスト待ち） に変更
- 進捗率 を 50 から 70 に変更
コードチェックOKです。
Kousuke Ebihara さんが13年以上前に更新
- ステータス を Pending Testing（テスト待ち） から Rejected（差し戻し） に変更
- 進捗率 を 70 から 50 に変更
- 3.6 で発生するか を Yes にセット
再度レビューしました。差し戻します。
本チケットの現象として報告されている、以下の点のうち「ページャーのリンク先のURL」についての修正がおこなわれていません。
2. ページャーの件数の表示、ページャーのリンク先のURLにその値が影響してしまう
Kousuke Ebihara さんが13年以上前に更新
http://redmine.openpne.jp/issues/1797#note-8 の指摘事項についての対応引き受けます
Kousuke Ebihara さんが13年以上前に更新
- ステータス を Rejected（差し戻し） から Pending Testing（テスト待ち） に変更
- 進捗率 を 50 から 70 に変更
既にバックポート済みのチケットがクローズされていることを考慮し、 http://redmine.openpne.jp/issues/1797#note-8 の対応については #2242 でおこなうことにします。
Yuma Sakata さんが約13年前に更新
- ステータス を Pending Testing（テスト待ち） から Fixed（完了） に変更
- 進捗率 を 70 から 100 に変更
テストOKです。
