Bug（バグ） #1797
完了
#1432 で修正した $size に整数値以外の値を渡すことができる
Shinichi Urabe さんがほぼ14年前に追加.
約9年前に更新.
説明
1. /pc_backend.php/monitoring/imageList/size/2.2 のように整数以外の値を入力する
2. ページャーの件数の表示、ページャーのリンク先のURLにその値が影響してしまう
取得したサイズ値をチェックしていない
修正内容¶
ページャサイズを 20, 50, 100, 500 に制限するようにした
制限値以外のページャサイズにすると 20 で表示されるようにした
- 対象バージョン を OpenPNE 3.7.0 にセット
- ステータス を New（新規） から Pending Review（レビュー待ち） に変更
- 進捗率 を 0 から 50 に変更
- 担当者 を Shinichi Urabe にセット
- ステータス を Pending Review（レビュー待ち） から Rejected（差し戻し） に変更
2点気になったのでフィードバックします。
1.
許可サイズのリストが存在するのであれば、
ページャを取得するメソッド側でチェックを行うべきではないでしょうか？
FileTable::getImageFilePager()
FileTable::getFilePager()
2.
apps/pc_backend/templates/_changePageSize.php で
$default 変数が使用されていないようなので、
不要であれば削除をお願いします。
また個人的に FileTable::getDefaultPagerSize() がリテラルで指定されていることが気になりました。
return $allowedPagerSizeList[0];
のように、許可リストから値を取得するほうが自然ではないでしょうか。
- ステータス を Rejected（差し戻し） から Pending Review（レビュー待ち） に変更
- ステータス を Pending Review（レビュー待ち） から Pending Testing（テスト待ち） に変更
- 進捗率 を 50 から 70 に変更
- ステータス を Pending Testing（テスト待ち） から Rejected（差し戻し） に変更
- 進捗率 を 70 から 50 に変更
- 3.6 で発生するか を Yes にセット
再度レビューしました。差し戻します。
本チケットの現象として報告されている、以下の点のうち「ページャーのリンク先のURL」についての修正がおこなわれていません。
2. ページャーの件数の表示、ページャーのリンク先のURLにその値が影響してしまう
- ステータス を Rejected（差し戻し） から Pending Testing（テスト待ち） に変更
- 進捗率 を 50 から 70 に変更
- ステータス を Pending Testing（テスト待ち） から Fixed（完了） に変更
- 進捗率 を 70 から 100 に変更
- 3.8 で発生するか を Unknown (未調査) にセット
他の形式にエクスポート: Atom
PDF
