From 190e63a6bb995f0f17c4a13cd32014ceef08a369 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Gr=C3=A9goire=20Paris?= Date: Sun, 10 Jun 2012 23:59:07 +0300 Subject: [PATCH] Show the advice only when necessary. This is indeed a good advice, but people who already have implemented __toString() might get confused about it. --- lib/Doctrine/ORM/ORMInvalidArgumentException.php | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/lib/Doctrine/ORM/ORMInvalidArgumentException.php b/lib/Doctrine/ORM/ORMInvalidArgumentException.php index b90bf21bc..480f88b1f 100644 --- a/lib/Doctrine/ORM/ORMInvalidArgumentException.php +++ b/lib/Doctrine/ORM/ORMInvalidArgumentException.php @@ -61,9 +61,11 @@ class ORMInvalidArgumentException extends \InvalidArgumentException . " configured to cascade persist operations for entity: " . self::objToStr($entry) . "." . " To solve this issue: Either explicitly call EntityManager#persist()" . " on this unknown entity or configure cascade persist " - . " this association in the mapping for example @ManyToOne(..,cascade={\"persist\"}). " - . " If you cannot find out which entity causes the problem" - . " implement '" . $assoc['targetEntity'] . "#__toString()' to get a clue."); + . " this association in the mapping for example @ManyToOne(..,cascade={\"persist\"})." + . method_exists($entry, '__toString') ? + "": + " If you cannot find out which entity causes the problem" + ." implement '" . $assoc['targetEntity'] . "#__toString()' to get a clue."); } static public function detachedEntityFoundThroughRelationship(array $assoc, $entry)