Ticket #1504 (closed バグ指摘: 修正済)

Opened 9 years ago

Last modified 9 years ago

PEAR::HTTP_Request の配置を変更したためモジュールが動かない

Reported by: kim Owned by: somebody
Priority: Milestone: EC-CUBE2.11.4
Component: フロント Version: 2.11.3
Keywords: Cc:
修正済み: no

Description (last modified by Seasoft) (diff)

開発コミュニティーで以下の報告。

 http://xoops.ec-cube.net/modules/newbb/viewtopic.php?topic_id=9083&forum=15&post_id=43807#forumpost43807

#1475 のPEAR::HTTP_Requestの配置変更により決済モジュールなどの中のRequireが失敗し、利用できない状態になっています。

各決済モジュールがバージョンアップしたとした場合には、過去のバージョンでモジュールが動かなくなるため、PEAR::HTTP_Requestを元に戻す方が良いと思われます。

→ その方法ですと、PEAR::HTTP_Request に依存する、一般的なライブラリの利用に支障をきたす懸念があります。回避案をフォーラムに投稿しました。 http://xoops.ec-cube.net/modules/newbb/viewtopic.php?viewmode=thread&topic_id=9083&forum=15&post_id=43810 (Seasoft 塚田)

Change History

comment:1 follow-up: ↓ 2 Changed 9 years ago by Seasoft

  • Description modified (diff)

懸念される問題点を追記。

comment:2 in reply to: ↑ 1 ; follow-ups: ↓ 3 ↓ 4 Changed 9 years ago by nanasess

Seasoft への返信

懸念される問題点を追記。

#1475 にて削除された addPostDataArray() を使用している決済モジュールがあるようなので, 元に戻す方が良いと思われます.

2.11.2 までに同梱していた Request.php に起因する不具合の具体的な事象などありましたらご教示いただければ幸いです.

comment:3 in reply to: ↑ 2 Changed 9 years ago by Seasoft

nanasess への返信

#1475 にて削除された addPostDataArray() を使用している決済モジュールがあるようなので, 元に戻す方が良いと思われます.

これは、そもそも不適切な実装だと思いますので、EC-CUBE 本体で行われた改訂と同様に、決済モジュール側で改善していただくのが筋かと思います。

とはいえ、該当するモジュールの数などにもよると思いますが。最悪、現状の最新ソースに addPostDataArray をつけるという妥協も、視野に入れる必要はあるかもしれませんね・・・

2.11.2 までに同梱していた Request.php に起因する不具合の具体的な事象などありましたらご教示いただければ幸いです.

詳細は失念しましたが、説明に該当するケースが実案件にてございました。また、EC-CUBE の保守を考えた場合にも、PEAR モジュールに手を加え続けるようなスパイラルは保守性も悪く避けたいところです。

comment:4 in reply to: ↑ 2 Changed 9 years ago by Seasoft

nanasess への返信

ちなみに、2.11.3 の修正リリース (2.11.3a ?) などについては、ブランチを切るなどして、速やかに対応しやすい方法での対応で十分と考えています。

理想的な方向はデベロッパー版で継続して模索し、決済モジュールでの addPostDataArray() の利用も、その間に移行を促せれば、解決も可能かなと考えています。

comment:5 Changed 9 years ago by kotani

  • Status changed from new to closed
  • Resolution set to 修正済
  • Milestone changed from EC-CUBE 2.11.5(仮) to EC-CUBE2.11.4

旧パスに、新パスのRequest.phpを読み込むだけのファイルを設置し、 旧パスを読み込んでいる各モジュールでも正しく動作するように対応完了。

comment:6 Changed 9 years ago by kajiwara

追記(対応チケット番号)
r21282 r21283 にて対応完了済み。

Note: See TracTickets for help on using tickets.