Project

General

Profile

Backport(バックポート) #2016

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

Added by Shinichi Urabe over 8 years ago. Updated over 8 years ago.

Status:
Fixed(完了)
Priority:
High(高め)
Target version:
Start date:
2010-11-17
Due date:
2011-05-06
% 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 687e4a10 (diff)
Added by Shinichi Urabe over 8 years ago

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

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

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

History

#1 Updated by Shinichi Urabe over 8 years ago

  • Priority changed from Normal(通常) to High(高め)

#2 Updated by Shinichi Urabe over 8 years ago

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

#3 Updated by Shinichi Urabe over 8 years ago

5d8edae366f46846342e08fa162e02a78bb08207 で修正しました。(コミットコメントのチケット番号修正忘れていました。。)

#4 Updated by Shinichi Urabe over 8 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

#5 Updated by Yuma Sakata over 8 years ago

テストOKです。

#6 Updated by Masato Nagasawa over 8 years ago

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

#1797 のチケットでフィードバックしています。

#7 Updated by Shinichi Urabe over 8 years ago

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

http://redmine.openpne.jp/issues/1797#note-6 と同様の修正をマージしました。

#8 Updated by Masato Nagasawa over 8 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
  • % Done changed from 50 to 70

コードチェックOKです。

#9 Updated by Yuma Sakata over 8 years ago

再テストOKです。

#10 Updated by Shinichi Urabe over 8 years ago

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

テスト、レビュー完了のためチケット閉じます

Also available in: Atom PDF