Project

General

Profile

Backport(バックポート) #2071

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

Added by Masato Nagasawa over 8 years ago. Updated over 8 years ago.

Status:
Fixed(完了)
Priority:
High(高め)
Target version:
Start date:
2010-11-17
Due date:
2011-06-24
% Done:

100%


Description

現象

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

原因

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

修正内容

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


Related issues

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

Associated revisions

Revision 527e7575 (diff)
Added by Shinichi Urabe over 8 years ago

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

Revision 10c8cbd3 (diff)
Added by Shinichi Urabe over 8 years ago

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

Revision b023ccf3 (diff)
Added by Shinichi Urabe over 8 years ago

(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.

Revision 6369ca27 (diff)
Added by Shinichi Urabe about 8 years ago

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

Revision 62d384e8 (diff)
Added by Shinichi Urabe about 8 years ago

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

Revision 12206cde (diff)
Added by Shinichi Urabe about 8 years ago

(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)

History

#1 Updated by Masato Nagasawa over 8 years ago

  • Target version changed from OpenPNE3.6beta9 to OpenPNE 3.6.0

#2 Updated by Masato Nagasawa over 8 years ago

  • Target version changed from OpenPNE 3.6.0 to OpenPNE3.6beta11

#3 Updated by Kousuke Ebihara over 8 years ago

  • Due date changed from 2011-05-06 to 2011-06-24

#4 Updated by Kousuke Ebihara over 8 years ago

  • Status changed from New(新規) to Accepted(着手)
  • Assignee changed from Shinichi Urabe to Kousuke Ebihara

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

#5 Updated by Kousuke Ebihara over 8 years ago

  • Status changed from Accepted(着手) to New(新規)
  • Assignee deleted (Kousuke Ebihara)

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

#6 Updated by Shinichi Urabe over 8 years ago

  • Status changed from New(新規) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

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

#7 Updated by Kousuke Ebihara over 8 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)
  • Assignee set to Kousuke Ebihara

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

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

#8 Updated by Kousuke Ebihara over 8 years ago

  • Status changed from Rejected(差し戻し) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

Kousuke Ebihara は書きました:

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

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

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

#9 Updated by Mutsumi Imamura over 8 years ago

  • Status changed from Pending Testing(テスト待ち) to Fixed(完了)
  • % Done changed from 70 to 100

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

Also available in: Atom PDF