Project

General

Profile

Backport(バックポート) #3631

opPluginInstallTask::execute()で、モデルクラスの存在チェックが不完全

Added by 誠二 天重 about 5 years ago. Updated over 4 years ago.

Status:
New(新規)
Priority:
Normal(通常)
Assignee:
-
Target version:
Start date:
2010-09-22
Due date:
% Done:

0%


Description

詳細

  • lib/task/opPluginInstallTask.class.phpのisSnsConfigTableExists()メソッドで、sns_configテーブルの存在チェックを行っているが、モデルの作成が行われていない場合、SnsConfigTableクラスは存在しても、Doctrine::getTableでエラーが発生する(継承元のBaseSnsConfigTableクラスが存在しないため)
  • OpenPNEのインストール処理の実行前にdatabases.ymlを手作業で作成し、--redoオプションでインストーラーを実行した場合、opPluginInstallTask::execute()内のdatabases.yml存在チェックはtrueになり、その後のSnsConfigの処理が実行されてしまう

現象の再現箇所

  1. OpenPNEインストールパッケージの初期状態にて、openpne:install実行前にconfig/databases.ymlを手作業で作成する
  2. openpne:install --redoでインストールを実行する
  3. プラグインのインストール処理でエラーになる

再現するバージョン

OpenPNE 3.7.0-DEV (1cd3238f9fe94d0bf4cc8ba7e955e911c1573e62)

パッチ

添付参照

0001-fixed-opPluginInstakkTask.patch View (781 Bytes) Hidenori Goto, 2010-09-22 18:15


Related issues

Copied from OpenPNE 3 - Bug(バグ) #1606: opPluginInstallTask::execute()で、モデルクラスの存在チェックが不完全 Works for me(再現せず) 2010-09-22

Associated revisions

Revision 733605a9 (diff)
Added by 誠二 天重 about 5 years ago

fix imperfections of existence check of the model class in opPluginInstallTask::execute() (refs #3631, BP from #1606)

Revision df464852 (diff)
Added by 誠二 天重 about 5 years ago

Revert "fix imperfections of existence check of the model class in opPluginInstallTask::execute() (refs #3631, BP from #1606)"

This reverts commit 733605a956e735fb1d31ff92fdb42caff52d5253.

Revision 4c20f3c9
Added by Shinichi Urabe about 5 years ago

Merge pull request #154 from amashigeseiji/t-3631-revert

Revert "fix imperfections of existence check of the model class in opPluginInstallTask::execute() (refs #3631, BP from #1606)"

History

#1 Updated by 誠二 天重 about 5 years ago

  • Copied from Bug(バグ) #1606: opPluginInstallTask::execute()で、モデルクラスの存在チェックが不完全 added

#2 Updated by 誠二 天重 about 5 years ago

  • Assignee deleted (誠二 天重)

#3 Updated by 誠二 天重 about 5 years ago

  • Status changed from New(新規) to Pending Review(レビュー待ち)
  • % Done changed from 0 to 50

https://github.com/openpne/OpenPNE3/pull/147
にてプルリクエスト。

#4 Updated by Shinichi Urabe about 5 years ago

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

レビューOKです
stable ブランチに取り込みました

#5 Updated by 誠二 天重 about 5 years ago

  • Assignee set to 誠二 天重

#6 Updated by Shinichi Urabe about 5 years ago

  • Status changed from Pending Testing(テスト待ち) to Rejected(差し戻し)
  • % Done changed from 70 to 50

Bug チケットにあわせ差し戻します

一旦修正を取り消して、プルリクエストをしてください

#7 Updated by 誠二 天重 about 5 years ago

https://github.com/openpne/OpenPNE3/pull/154
にてリバートのリクエスト投げました。

#8 Updated by Shinichi Urabe about 5 years ago

  • Status changed from Rejected(差し戻し) to New(新規)
  • Assignee deleted (誠二 天重)
  • Target version changed from OpenPNE 3.8.12 to OpenPNE 3.8.x
  • % Done changed from 50 to 0

取消を確認し、問題無いので、取り込みました
一旦バージョンを 3.8.x に変更し、ステータスを新規に戻します

Also available in: Atom PDF