Backport(バックポート) #2554
完了
opToolkit::getRandom() に static キーワードが付いていない
Maki Takahashi さんが約13年前に追加.
ほぼ13年前に更新.
説明
opToolkit クラスのメソッドには static を付けるべきだが、このメソッドだけ付いていない。
確認バージョン¶
OpenPNE 3.7.0-dev (master)
OpenPNE 3.6.1 (stable-3.6.x)
関連するチケット
1 (0件未完了 — 1件完了)
- ステータス を New(新規) から Pending Review(レビュー待ち) に変更
- 進捗率 を 0 から 50 に変更
- 対象バージョン を OpenPNE 3.6.1 から OpenPNE 3.6.x に変更
- ステータス を 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 で取り込んでしまいましょう。
- ステータス を Pending Testing(テスト待ち) から Fixed(完了) に変更
- 進捗率 を 70 から 100 に変更
note-3 を読んでもテスターテストを不要と考えることが理解できませんが、 note-3 時点でも実装者テストを行なっており、 stable-3.6.x ブランチにおいて、この修正後に opToolkit::getRandom() メソッドをコールできることを確認していることを示しておきます。
他の形式にエクスポート: Atom
PDF