Backport(バックポート) #2554
opToolkit::getRandom() に static キーワードが付いていない
Start date:
2011-10-14
Due date:
% Done:
100%
Description
opToolkit クラスのメソッドには static を付けるべきだが、このメソッドだけ付いていない。
確認バージョン¶
OpenPNE 3.7.0-dev (master)
OpenPNE 3.6.1 (stable-3.6.x)
Related issues
Associated revisions
History
#1
Updated by Maki Takahashi over 11 years ago
- Status changed from New(新規) to Pending Review(レビュー待ち)
- % Done changed from 0 to 50
更新履歴 66ffe2e4d56a35e22776173a3c820717925310a8 で適用されました。
#2
Updated by Maki Takahashi about 11 years ago
- Target version changed from OpenPNE 3.6.1 to OpenPNE 3.6.x
#3
Updated by Minoru Takai about 11 years ago
- Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)
- Target version changed from OpenPNE 3.6.x to OpenPNE 3.6.1
- % Done changed from 50 to 70
レビューしました。問題ありません。
この修正内容が妥当かどうかは元チケットでレビューしています。この修正は OpenPNE のバージョンに依存するようなものではないので、 stable-3.6.x ブランチに対してのレビューは、元チケットと同じ修正が stable-3.6.x ブランチに取り込まれているかどうかだけを確認すれば十分で、それを確認しました。
また、 OpenPNE-3.6.1 リリース時に取り込むチケットが決められているようで、そこにこのチケットは含まれていなかったようですが、 stable-3.6.x ブランチにこのコミットが行われており、動作テスト(元チケットに示している)も容易なので OpenPNE-3.6.1 で取り込んでしまいましょう。
#4
Updated by Yuma Sakata about 11 years ago
- Status changed from Pending Testing(テスト待ち) to Fixed(完了)
- % Done changed from 70 to 100
スターテストは不要のため、クローズします。
#5
Updated by Minoru Takai about 11 years ago
note-3 を読んでもテスターテストを不要と考えることが理解できませんが、 note-3 時点でも実装者テストを行なっており、 stable-3.6.x ブランチにおいて、この修正後に opToolkit::getRandom() メソッドをコールできることを確認していることを示しておきます。