操作
Backport(バックポート) #2554
完了opToolkit::getRandom() に static キーワードが付いていない
開始日:
2011-10-14
期日:
進捗率:
100%
予定工数:
説明
opToolkit クラスのメソッドには static を付けるべきだが、このメソッドだけ付いていない。
確認バージョン¶
OpenPNE 3.7.0-dev (master)
OpenPNE 3.6.1 (stable-3.6.x)
Maki Takahashi さんが約13年前に更新
- ステータス を New(新規) から Pending Review(レビュー待ち) に変更
- 進捗率 を 0 から 50 に変更
更新履歴 66ffe2e4d56a35e22776173a3c820717925310a8 で適用されました。
Minoru Takai さんがほぼ13年前に更新
- ステータス を Pending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
- 対象バージョン を OpenPNE 3.6.x から OpenPNE 3.6.1 に変更
- 進捗率 を 50 から 70 に変更
レビューしました。問題ありません。
この修正内容が妥当かどうかは元チケットでレビューしています。この修正は OpenPNE のバージョンに依存するようなものではないので、 stable-3.6.x ブランチに対してのレビューは、元チケットと同じ修正が stable-3.6.x ブランチに取り込まれているかどうかだけを確認すれば十分で、それを確認しました。
また、 OpenPNE-3.6.1 リリース時に取り込むチケットが決められているようで、そこにこのチケットは含まれていなかったようですが、 stable-3.6.x ブランチにこのコミットが行われており、動作テスト(元チケットに示している)も容易なので OpenPNE-3.6.1 で取り込んでしまいましょう。
Yuma Sakata さんがほぼ13年前に更新
- ステータス を Pending Testing(テスト待ち) から Fixed(完了) に変更
- 進捗率 を 70 から 100 に変更
スターテストは不要のため、クローズします。
Minoru Takai さんがほぼ13年前に更新
note-3 を読んでもテスターテストを不要と考えることが理解できませんが、 note-3 時点でも実装者テストを行なっており、 stable-3.6.x ブランチにおいて、この修正後に opToolkit::getRandom() メソッドをコールできることを確認していることを示しておきます。
操作