プロジェクト

全般

プロフィール

Backport（バックポート） #3631

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

誠二 天重 がほぼ10年前に追加. 9年以上前に更新.

ステータス:
New（新規）
優先度:
Normal（通常）
担当者:
-
対象バージョン:
OpenPNE 3.8.x
開始日:
2010-09-22
期日:
進捗率:

0%

説明

詳細

  • 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 表示 (781 Bytes) Hidenori Goto, 2010-09-22 18:15

関連するチケット

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

関係しているリビジョン

リビジョン 733605a9 (差分)
誠二 天重 がほぼ10年前に追加

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

リビジョン df464852 (差分)
誠二 天重 がほぼ10年前に追加

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

This reverts commit 733605a956e735fb1d31ff92fdb42caff52d5253.

リビジョン 4c20f3c9
Shinichi Urabe がほぼ10年前に追加

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)"

履歴

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

  • コピー元 Bug（バグ） #1606: opPluginInstallTask::execute()で、モデルクラスの存在チェックが不完全 を追加

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

  • 担当者 を削除 (誠二 天重)

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

  • ステータス を New（新規） から Pending Review（レビュー待ち） に変更
  • 進捗率 を 0 から 50 に変更

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

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

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

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

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

  • 担当者 を 誠二 天重 にセット

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

  • ステータス を Pending Testing（テスト待ち） から Rejected（差し戻し） に変更
  • 進捗率 を 70 から 50 に変更

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

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

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

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

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

  • ステータス を Rejected（差し戻し） から New（新規） に変更
  • 担当者 を削除 (誠二 天重)
  • 対象バージョン を OpenPNE 3.8.12 から OpenPNE 3.8.x に変更
  • 進捗率 を 50 から 0 に変更

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

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