プロジェクト

全般

プロフィール

Backport(バックポート) #2071

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

Masato Nagasawaほぼ13年前に追加. ほぼ13年前に更新.

ステータス:
Fixed(完了)
優先度:
High(高め)
担当者:
対象バージョン:
開始日:
2010-11-17
期日:
2011-06-24
進捗率:

100%


説明

現象

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

原因

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

修正内容

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


関連するチケット

関連している OpenPNE 3 - Bug(バグ) #1797: #1432 で修正した $size に整数値以外の値を渡すことができる Fixed(完了) 2010-11-17

関係しているリビジョン

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

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

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

(fixes #2071, BP from #1797) fixed valid allowed size.

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

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

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

(fixes #2071, BP from #1797) fixed valid allowed size.
(cherry picked from commit 10c8cbd3b3f7a55dbbc2e3cc6a916e967dfa5b3e)

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

(refs #2071, BP from #1797) I change it to perform a check of the paging size in a method.
(cherry picked from commit 527e757513b267c5c9dec56c523b9f12222ad30f)

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

(refs #2071, BP from #1797) Because a variable name storing paging size is inappropriate, I change it. I delete the variable that is not used.
(cherry picked from commit b023ccf3af099963c80f13262f6dae2a0552cef2)

履歴

#1 Masato Nagasawaほぼ13年前に更新

  • 対象バージョンOpenPNE3.6beta9 から OpenPNE 3.6.0 に変更

#2 Masato Nagasawaほぼ13年前に更新

  • 対象バージョンOpenPNE 3.6.0 から OpenPNE3.6beta11 に変更

#3 Kousuke Ebiharaほぼ13年前に更新

  • 期日2011-05-06 から 2011-06-24 に変更

#4 Kousuke Ebiharaほぼ13年前に更新

  • ステータスNew(新規) から Accepted(着手) に変更
  • 担当者Shinichi Urabe から Kousuke Ebihara に変更

バックポート引き受けます

#5 Kousuke Ebiharaほぼ13年前に更新

  • ステータスAccepted(着手) から New(新規) に変更
  • 担当者 を削除 (Kousuke Ebihara)

元チケットを差し戻しました。バックポートはその状況を見てからおこないます。

#6 Shinichi Urabeほぼ13年前に更新

  • ステータスNew(新規) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

更新履歴 10c8cbd3b3f7a55dbbc2e3cc6a916e967dfa5b3e で適用されました。

#7 Kousuke Ebiharaほぼ13年前に更新

  • ステータスPending Review(レビュー待ち) から Rejected(差し戻し) に変更
  • 担当者Kousuke Ebihara にセット

すいません、 http://redmine.openpne.jp/issues/2071#note-5 のとおり元チケットを差し戻したにもかかわらずマージしてしまいました。

とりあえず元チケットと同じ「差し戻し」状態にしておきます。追加の対応をするか、特に対応せずに再び「レビュー待ち」「テスト待ち」に直すかどうかは元チケットの状態に従います。

#8 Kousuke Ebiharaほぼ13年前に更新

  • ステータスRejected(差し戻し) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

Kousuke Ebihara は書きました:

すいません、 http://redmine.openpne.jp/issues/2071#note-5 のとおり元チケットを差し戻したにもかかわらずマージしてしまいました。

とりあえず元チケットと同じ「差し戻し」状態にしておきます。追加の対応をするか、特に対応せずに再び「レビュー待ち」「テスト待ち」に直すかどうかは元チケットの状態に従います。

元チケットの指摘事項については #2242 で対応することにしました。このチケットのゴールは元チケットの(漏れのある)修正を取り込む、というところまでとします。

#9 Mutsumi Imamuraほぼ13年前に更新

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

動作確認しました。
テストOKです。

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