プロジェクト

全般

プロフィール

Backport(バックポート) #2554

opToolkit::getRandom() に static キーワードが付いていない

Maki Takahashi12年以上前に追加. 12年以上前に更新.

ステータス:
Fixed(完了)
優先度:
Normal(通常)
担当者:
対象バージョン:
開始日:
2011-10-14
期日:
進捗率:

100%


説明

opToolkit クラスのメソッドには static を付けるべきだが、このメソッドだけ付いていない。

確認バージョン

OpenPNE 3.7.0-dev (master)
OpenPNE 3.6.1 (stable-3.6.x)


関連するチケット

関連している OpenPNE 3 - Bug(バグ) #2491: opToolkit::getRandom() に static キーワードが付いていない Fixed(完了) 2011-10-14

関係しているリビジョン

リビジョン 66ffe2e4 (差分)
Maki Takahashi12年以上前に追加

(fixes #2554, BP from #2491) added static keyword to opToolkit::getRandom()

履歴

#1 Maki Takahashi12年以上前に更新

  • ステータスNew(新規) から Pending Review(レビュー待ち) に変更
  • 進捗率0 から 50 に変更

更新履歴 66ffe2e4d56a35e22776173a3c820717925310a8 で適用されました。

#2 Maki Takahashi12年以上前に更新

  • 対象バージョンOpenPNE 3.6.1 から OpenPNE 3.6.x に変更

#3 Minoru Takai12年以上前に更新

  • ステータス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 で取り込んでしまいましょう。

#4 Yuma Sakata12年以上前に更新

  • ステータスPending Testing(テスト待ち) から Fixed(完了) に変更
  • 進捗率70 から 100 に変更

スターテストは不要のため、クローズします。

#5 Minoru Takai12年以上前に更新

note-3 を読んでもテスターテストを不要と考えることが理解できませんが、 note-3 時点でも実装者テストを行なっており、 stable-3.6.x ブランチにおいて、この修正後に opToolkit::getRandom() メソッドをコールできることを確認していることを示しておきます。

他の形式にエクスポート: Atom PDF