プロジェクト

全般

プロフィール

Bug(バグ) #1877

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

Masato Nagasawa約13年前に追加. 約8年前に更新.

ステータス:
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() の修正は不要になっています。


関連するチケット

関連している OpenPNE 3 - Enhancement(機能追加・改善) #1741: XSS 脆弱性、 CSRF 脆弱性のテストを完備できるように体制を整える Fixed(完了) 2010-10-26 2011-06-24
コピー先 OpenPNE 3 - Backport(バックポート) #3636: functional test で特定のパラメータで checkElement() を呼び出すとエラーになる Invalid(無効) 2011-01-19
コピー先 OpenPNE 3 - Backport(バックポート) #3663: functional test で特定のパラメータで checkElement() を呼び出すとエラーになる Invalid(無効) 2014-07-15

関係しているリビジョン

リビジョン f203472c (差分)
Masato Nagasawa約13年前に追加

deleted files(opDomCssSelector, opTesterResponse) (refs #1877)

履歴

#1 Masato Nagasawa約13年前に更新

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

#2 Masato Nagasawa約13年前に更新

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

#3 Kousuke Ebiharaほぼ13年前に更新

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

#4 Masato Nagasawaほぼ13年前に更新

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

#5 Masato Nagasawaほぼ13年前に更新

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

#6 Kousuke Ebiharaほぼ13年前に更新

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

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

#7 Mutsumi Imamura12年以上前に更新

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

#8 Shouta Kashiwagi約12年前に更新

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

#9 Shouta Kashiwagiほぼ12年前に更新

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

#10 Shinichi Urabeほぼ10年前に更新

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

#11 誠二 天重ほぼ10年前に更新

#12 誠二 天重ほぼ10年前に更新

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

#13 誠二 天重ほぼ10年前に更新

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

#14 Shinichi Urabe9年以上前に更新

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 に修正がないという状態をどう扱うべきかが分かりません

#15 isao sano9年以上前に更新

#16 誠二 天重9年以上前に更新

#17 誠二 天重9年以上前に更新

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

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

#18 Shinichi Urabe9年以上前に更新

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

レビューOKです

#19 isao sano9年以上前に更新

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

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

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