Project

General

Profile

Backport(バックポート) #1471

error at the application cascading of doctrine.(Doctrineのカスケーディングデリート 時にエラーになる)

Added by Masato Nagasawa about 9 years ago. Updated almost 9 years ago.

Status:
Fixed(完了)
Priority:
Normal(通常)
Target version:
Start date:
2010-08-03
Due date:
% Done:

100%


Description

Overview (現象)

Doctrine側でカスケーディングデリートを有効にした場合、日記を削除しようとするとエラーになります。

Causes (原因)

lib/behavior/opApplicationLevelCascadingListener.class.php の 32行目でエラーになっており、
$record が空の配列であるにも関わらずdelete()を呼んでいてメソッドが解決できずエラーとなっています。

Way to fix (修正内容)

$relations が Doctrine_Collection でない場合に array($relations) としているのが原因で、
この場合、$relationsが空の配列であった場合に要素数が1となってしまい、foreach で空の $record を処理してしまいます。

 22       if (!($relations instanceof Doctrine_Collection))
 23       {
 24         $relations = array($relations);
 25       }

以下のようにして配列の要素数が0であった場合、continue するように対処するのが良いのではと思います。

 22       if (!($relations instanceof Doctrine_Collection))
 23       {
            if (!count($relations))
            {
              continue;
            }
 24         $relations = array($relations);
 25       }


Related issues

Related to OpenPNE 3 - Backport(バックポート) #1475: error at the application cascading of doctrine.(Doctrineのカスケーディングデリート 時にエラーになる) Fixed(完了) 2010-08-03

Associated revisions

Revision 65b85f0a (diff)
Added by Masato Nagasawa about 9 years ago

fixed cascading delete in the application level cascading.(fixes #1471, BP from #1475)

Revision 4e837c27 (diff)
Added by Masato Nagasawa about 9 years ago

fixed check instance of class. (fixes #1471, BP from #1475)

Revision 1dc41608 (diff)
Added by Shinichi Urabe about 9 years ago

Revert "fixed check instance of class. (fixes #1471, BP from #1475)"

This reverts commit 4e837c27bc1e233ce6e44cac5e06c186dab3ca32.

Revision d9f4be3d (diff)
Added by Shinichi Urabe about 9 years ago

Revert "fixed cascading delete in the application level cascading.(fixes #1471, BP from #1475)"

This reverts commit 65b85f0ad59531c22ec02e7b264391bfd98ac8c6.

Revision 29dc823b (diff)
Added by Masato Nagasawa almost 9 years ago

fixed cascading delete in the application level cascading.(fixes #1471, BP from #1475)

Revision 3321466e (diff)
Added by Masato Nagasawa almost 9 years ago

fixed check instance of class. (fixes #1471, BP from #1475)

History

#1 Updated by Masato Nagasawa about 9 years ago

  • Target version changed from OpenPNE 3.7.0 to OpenPNE 3.4.7

#2 Updated by Masato Nagasawa about 9 years ago

  • Tracker changed from Bug(バグ) to Backport(バックポート)

#3 Updated by Masato Nagasawa about 9 years ago

  • Subject changed from Doctrineのカスケーディングデリート 時にエラーになる to error at the application cascading of doctrine.(Doctrineのカスケーディングデリート 時にエラーになる)
  • Assignee set to Masato Nagasawa

#4 Updated by Masato Nagasawa about 9 years ago

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

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

#5 Updated by Masato Nagasawa about 9 years ago

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

#6 Updated by Masato Nagasawa about 9 years ago

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

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

#7 Updated by Shinichi Urabe about 9 years ago

  • Status changed from Pending Review(レビュー待ち) to Rejected(差し戻し)
  • Assignee changed from Masato Nagasawa to Shinichi Urabe

次のバージョンで対応します

#8 Updated by Shinichi Urabe about 9 years ago

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

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

#9 Updated by Shinichi Urabe about 9 years ago

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

#10 Updated by Shinichi Urabe about 9 years ago

  • Status changed from Pending Review(レビュー待ち) to Pending Fixing(修正待ち)
  • Assignee changed from Shinichi Urabe to Masato Nagasawa
  • Target version changed from OpenPNE 3.4.7 to OpenPNE 3.4.8

#11 Updated by Shinichi Urabe almost 9 years ago

  • Target version changed from OpenPNE 3.4.8 to OpenPNE 3.4.9

#12 Updated by Masato Nagasawa almost 9 years ago

  • Status changed from Pending Fixing(修正待ち) to Pending Review(レビュー待ち)

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

#13 Updated by Masato Nagasawa almost 9 years ago

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

#14 Updated by Mutsumi Imamura almost 9 years ago

動作確認しました。テストOKです。

#15 Updated by Rimpei Ogawa almost 9 years ago

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

#16 Updated by Mutsumi Imamura almost 9 years ago

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

Also available in: Atom PDF