プロジェクト

全般

プロフィール

Bug(バグ) #1877

完了

functional test で特定のパラメータで checkElement() を呼び出すとエラーになる

Masato Nagasawa さんがほぼ14年前に追加. ほぼ9年前に更新.

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

100%

予定工数:
3.6 で発生するか:
Yes
3.8 で発生するか:
Unknown (未調査)

説明

Overview (現象)

opDiaryPlugin の functional test でエラーになっている。

plugins/opDiaryPlugin/test/functional/pc_frontend/diaryOpenTest.php

PHP Fatal error:  Call to undefined method DOMDocument::getAttribute() in lib/test/opDomCssSelector.class.php on line 158

Fatal error: Call to undefined method DOMDocument::getAttribute() in lib/test/opDomCssSelector.class.php on line 158

Causes (原因)

原因は分かりませんが、opDomCssSelector::getElementsForNode() で修正した内容に問題があるようです。

Way to fix (修正内容)

opDomCssSelector 及び opTesterResponse を削除して、opTestFunctional::__construct() で opTesterResponse を使用しないようにします。

opTestFunctional::checkCSRF() を実装する上で、
opDomCssSelector にバグがあることが分かり修正しましたが、
現在は独自の方法で実装したため opDomCssSelector::getElementsForNode() の修正は不要になっています。


関連するチケット 3 (0件未完了3件完了)

関連している OpenPNE 3 - Enhancement(機能追加・改善) #1741: XSS 脆弱性、 CSRF 脆弱性のテストを完備できるように体制を整えるFixed(完了)Kousuke Ebihara2010-10-262011-06-24

操作
コピー先 OpenPNE 3 - Backport(バックポート) #3636: functional test で特定のパラメータで checkElement() を呼び出すとエラーになるInvalid(無効)2011-01-19

操作
コピー先 OpenPNE 3 - Backport(バックポート) #3663: functional test で特定のパラメータで checkElement() を呼び出すとエラーになるInvalid(無効)2014-07-15

操作

Masato Nagasawa さんがほぼ14年前に更新

  • ステータスNew(新規) から Accepted(着手) に変更

Masato Nagasawa さんがほぼ14年前に更新

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

Kousuke Ebihara さんが13年以上前に更新

  • 対象バージョンOpenPNE3.6beta9 から OpenPNE3.6beta11 に変更

Masato Nagasawa さんが13年以上前に更新

  • 対象バージョンOpenPNE3.6beta11 から OpenPNE 3.6.0 に変更

Masato Nagasawa さんが13年以上前に更新

  • 対象バージョンOpenPNE 3.6.0 から OpenPNE3.6beta11 に変更

Kousuke Ebihara さんが13年以上前に更新

  • 対象バージョンOpenPNE3.6beta11 から OpenPNE3.6beta12 に変更

親チケットが「テスト待ち」「完了」でないものをすべて beta12 以降での対応とします。

Mutsumi Imamura さんが13年以上前に更新

  • 対象バージョンOpenPNE3.6beta12 から OpenPNE 3.7.0 に変更

Shouta Kashiwagi さんがほぼ13年前に更新

  • 対象バージョンOpenPNE 3.7.0 から 252 に変更

Shouta Kashiwagi さんが12年以上前に更新

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

Shinichi Urabe さんが10年以上前に更新

  • 対象バージョンOpenPNE 3.8.x から OpenPNE 3.8.12 に変更
  • 3.8 で発生するかUnknown (未調査) にセット

誠二 天重 さんが10年以上前に更新

誠二 天重 さんが10年以上前に更新

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

誠二 天重 さんが10年以上前に更新

  • 対象バージョンOpenPNE 3.8.x から OpenPNE 3.9.0-old に変更

Shinichi Urabe さんが10年以上前に更新

stable-3.6.x にしかはいっていない修正で、 master や stable-3.8.x には修正が入っていません
https://redmine.openpne.jp/projects/op3/repository/revisions/26e9c6e1862478b2d214df305a3f6546a76a6733 に対する修正ですが、この修正自体も stable-3.6.x にしか存在しません。

stable-3.6.x に対するバグチケットとして扱うべきもののように思いますが、master に修正がないという状態をどう扱うべきかが分かりません

isao sano さんが10年以上前に更新

誠二 天重 さんが10年以上前に更新

誠二 天重 さんが10年以上前に更新

  • 対象バージョンOpenPNE 3.9.0-old から OpenPNE 3.6.16 に変更

本チケットは、 #1741 と関連しており、3.6の開発用のバグチケットだったものと思われます。
master, stable-3.6.xおよびstable-3.8.xの現在のソースコードの状態は妥当です。
開発時のチケットが閉じられないまま放置されていたという状態のようですので、レビューおよびテストをこのチケットで実行し問題なければ、このチケットを閉じるということにします。

Shinichi Urabe さんが10年以上前に更新

  • ステータスPending Review(レビュー待ち) から Pending Testing(テスト待ち) に変更
  • 進捗率50 から 70 に変更

レビューOKです

isao sano さんが10年以上前に更新

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

問題ありません。
note-17の通り、既にstable-3.6.xに取り込まれていますのでマージ作業は必要ありません。

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