Project

General

Profile

Bug(バグ) #966

"CAPTCHA" display in the member config and etc. in the environment of mobile. (CAPTCHAがモバイル環境で表示されている)

Added by Shogo Kawahara over 11 years ago. Updated over 11 years ago.

Status:
Fixed(完了)
Priority:
High(高め)
Target version:
Start date:
2010-04-19
Due date:
% Done:

100%

3.6 で発生するか:
Unknown (未調査)
3.8 で発生するか:
Unknown (未調査)

Description

Overview (現象)

"CAPTCHA" display in the member config and etc. in the environment of mobile.
(CAPTCHAがモバイル環境で表示されている)

Environment (環境)

  • OpenPNE3.4.x
  • OpenPNE3.5.x-dev

Causes (原因)

  • web/index.php loads configuration for PC first, execute pc_frontendConfiguration::configure() and it enables CAPTCHA (web/index.php は 先にPCの設定をロードするために pc_frontendConfiguration::configure() が実行され、CAPTCHAが有効になる。)
 sfConfig::set('op_is_use_captcha', true);
  • after doing it, opExecutionFilter::handleAction() loads from configuration database (その後、opExecutionFilter::handleAction() で、上記の設定が存在していた場合 データベースの設定がロードされる。)

Way to fix (修正内容)

Changed to load opMobileUserAgent and decide application directly (web/index.php 内で直接 opMobileUserAgent を読みこみ、使用するべきアプリケーションを決定するように修正。)


Related issues

Related to OpenPNE 3 - Backport(バックポート) #1044: "CAPTCHA" display in the member config and etc. in the environment of mobile. (CAPTCHAがモバイル環境で表示されている) Fixed(完了) 2010-05-11

Associated revisions

Revision bbe6d2d6 (diff)
Added by Kousuke Ebihara over 11 years ago

fixed web/index.php loads two different applications (fixes #966)

Revision 019fd58b (diff)
Added by Kousuke Ebihara over 11 years ago

changed ordering to add include_path (fixes #966)

History

#1 Updated by Kousuke Ebihara over 11 years ago

  • Status changed from New(新規) to Accepted(着手)
  • Assignee set to Kousuke Ebihara

#2 Updated by Kousuke Ebihara over 11 years ago

  • Status changed from Accepted(着手) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

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

#3 Updated by Shogo Kawahara over 11 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)

index.php にて


set_include_path(get_include_path().PATH_SEPARATOR.dirname(__FILE__).'/../lib/vendor/PEAR/');

のようにPEARを include_path に設定していますが、この場合PEARに実行環境が備わっていた場合 実行環境に備わっている PEAR.php を読み出した後に、opProjectConfiguration::setIncludePath() で OpenPNE に同梱された PEAR ライブラリを優先して読み込むようにするため、再び PEAR.php をロードし、エラーが発生します。

index.php での set_include_path を設定するときの、パスの順序を変更するべきだと思います。

#4 Updated by Kousuke Ebihara over 11 years ago

  • Status changed from Rejected(差し戻し) to Pending Review(レビュー待ち)

更新履歴 019fd58ba783af4f1ab4163764fa1cb8ccfa635d で適用されました。

#5 Updated by Kousuke Ebihara over 11 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Testing(テスト待ち)

#6 Updated by Kiwa Sakai over 11 years ago

  • Status changed from Pending Testing(テスト待ち) to Fixed(完了)
  • % Done changed from 50 to 100

完了にします

Also available in: Atom PDF