プロジェクト

全般

プロフィール

Bug(バグ) #1797

#1432 で修正した $size に整数値以外の値を渡すことができる

Shinichi Urabe13年以上前に追加. 8年以上前に更新.

ステータス:
Fixed(完了)
優先度:
Normal(通常)
担当者:
対象バージョン:
開始日:
2010-11-17
期日:
進捗率:

100%

3.6 で発生するか:
Yes
3.8 で発生するか:
Unknown (未調査)

説明

現象

1. /pc_backend.php/monitoring/imageList/size/2.2 のように整数以外の値を入力する
2. ページャーの件数の表示、ページャーのリンク先のURLにその値が影響してしまう

原因

取得したサイズ値をチェックしていない

修正内容

ページャサイズを 20, 50, 100, 500 に制限するようにした
制限値以外のページャサイズにすると 20 で表示されるようにした


関連するチケット

関連している OpenPNE 3 - Bug(バグ) #1432: 管理画面の画像表示で表示数を変更した後のページャーの動作に問題 Fixed(完了) 2010-07-26
関連している OpenPNE 3 - Backport(バックポート) #2016: #1432 で修正した $size に整数値以外の値を渡すことができる Fixed(完了) 2010-11-17 2011-05-06
関連している OpenPNE 3 - Backport(バックポート) #2071: #1432 で修正した $size に整数値以外の値を渡すことができる Fixed(完了) 2010-11-17 2011-06-24
関連している OpenPNE 3 - Bug(バグ) #2242: 管理画面の「アップロード画像リスト」「アップロードファイルリスト」において、 size パラメータに無効な値を含めてリクエストした際、その後のページ遷移でもその値が引き継がれる New(新規) 2011-06-23

関係しているリビジョン

リビジョン 2f40254b (差分)
Shinichi Urabeほぼ13年前に追加

(fixes #1797) fixed valid allowed size.

リビジョン 5d8edae3 (差分)
Shinichi Urabeほぼ13年前に追加

(fixes #1797) fixed valid allowed size.

リビジョン 272e6e9d (差分)
Shinichi Urabeほぼ13年前に追加

(refs #1797) I change it to perform a check of the paging size in a method.

リビジョン a4544dc8 (差分)
Shinichi Urabeほぼ13年前に追加

(refs #1797) Because a variable name storing paging size is inappropriate, I change it.
I delete the variable that is not used.

履歴

#1 Shinichi Urabe13年以上前に更新

  • 対象バージョン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 Sakata12年以上前に更新

  • ステータスPending Testing(テスト待ち) から Fixed(完了) に変更
  • 進捗率70 から 100 に変更

テストOKです。

#12 kaoru n8年以上前に更新

  • 3.8 で発生するかUnknown (未調査) にセット

他の形式にエクスポート: Atom PDF