Bug(バグ) #1797
#1432 で修正した $size に整数値以外の値を渡すことができる
100%
説明
関連するチケット
関係しているリビジョン
(fixes #1797) fixed valid allowed size.
(fixes #1797) fixed valid allowed size.
(refs #1797) I change it to perform a check of the paging size in a method.
(refs #1797) Because a variable name storing paging size is inappropriate, I change it.
I delete the variable that is not used.
履歴
#1 Shinichi Urabe が13年以上前に更新
- 対象バージョン を OpenPNE 3.7.0 にセット
#2 Shinichi Urabe がほぼ13年前に更新
- ステータス を New(新規) から Pending Review(レビュー待ち) に変更
- 進捗率 を 0 から 50 に変更
更新履歴 2f40254be35af0ee922eb3b5d9c7b9d57d94d3f2 で適用されました。
#3 Shinichi Urabe がほぼ13年前に更新
- 担当者 を Shinichi Urabe にセット
#4 Shinichi Urabe がほぼ13年前に更新
更新履歴 5d8edae366f46846342e08fa162e02a78bb08207 で適用されました。
#5 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];
のように、許可リストから値を取得するほうが自然ではないでしょうか。
#6 Shinichi Urabe がほぼ13年前に更新
- ステータス を Rejected(差し戻し) から Pending Review(レビュー待ち) に変更
272e6e9d61123881a6744e19111d3850c5e8bf90 でFileTable クラス内のページャメソッドでサイズチェックするように修正し、デフォルト値はメンバ変数の配列の値を指定するように変更しました。
a4544dc848ea068b89891160ce8759102d539086 で使われていない変数を削除、変数名が$fileSizeListだと意味合いが違うので、$pagerSizeListに変更しました。
#7 Masato Nagasawa がほぼ13年前に更新
- ステータス を Pending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
- 進捗率 を 50 から 70 に変更
コードチェックOKです。
#8 Kousuke Ebihara がほぼ13年前に更新
- ステータス を Pending Testing(テスト待ち) から Rejected(差し戻し) に変更
- 進捗率 を 70 から 50 に変更
- 3.6 で発生するか を Yes にセット
再度レビューしました。差し戻します。
本チケットの現象として報告されている、以下の点のうち「ページャーのリンク先のURL」についての修正がおこなわれていません。
2. ページャーの件数の表示、ページャーのリンク先のURLにその値が影響してしまう
#9 Kousuke Ebihara がほぼ13年前に更新
http://redmine.openpne.jp/issues/1797#note-8 の指摘事項についての対応引き受けます
#10 Kousuke Ebihara がほぼ13年前に更新
- ステータス を Rejected(差し戻し) から Pending Testing(テスト待ち) に変更
- 進捗率 を 50 から 70 に変更
既にバックポート済みのチケットがクローズされていることを考慮し、 http://redmine.openpne.jp/issues/1797#note-8 の対応については #2242 でおこなうことにします。
#11 Yuma Sakata が12年以上前に更新
- ステータス を Pending Testing(テスト待ち) から Fixed(完了) に変更
- 進捗率 を 70 から 100 に変更
テストOKです。