Better validation handling in EEmbeddedArrayBehavior

extra/EEmbeddedArraysBehavior.php

@@ -41,28 +41,34 @@ private function parseExistingArray()
 		if(is_array($this->getOwner()->{$this->arrayPropertyName}))
 		{
 			$arrayOfDocs = array();
-			foreach($this->getOwner()->{$this->arrayPropertyName} as $key=>$doc)
+			foreach($this->getOwner()->{$this->arrayPropertyName} as $doc)
 			{
-				$arrayOfDocs[$key] = new $this->arrayDocClassName;
-				$arrayOfDocs[$key]->setAttributes($doc, false);
+				$obj = new $this->arrayDocClassName;
+				$obj->setAttributes($doc, false);
+				$arrayOfDocs[] = $obj;
 			}
 			$this->getOwner()->{$this->arrayPropertyName} = $arrayOfDocs;
 		}
 	}
 
+	public function afterValidate($event)
+	{
+		parent::afterValidate($event);
+		foreach($this->getOwner()->{$this->arrayPropertyName} as $doc)
+		{
+			if(!$doc->validate())
+				$this->getOwner()->addErrors($doc->getErrors());
+		}
+	}
+
 	public function beforeSave($event)
 	{
 		if(is_array($this->getOwner()->{$this->arrayPropertyName}))
 		{
 			$arrayOfDocs = array();
-			foreach($this->getOwner()->{$this->arrayPropertyName} as $key=>$doc)
+			foreach($this->getOwner()->{$this->arrayPropertyName} as $doc)
 			{
-				if($this->getOwner()->{$this->arrayPropertyName}[$key]->validate())
-				{
-					$arrayOfDocs[$key] = $this->getOwner()->{$this->arrayPropertyName}[$key]->toArray();
-				}
-				else
-					return false;
+				$arrayOfDocs[] = $doc->toArray();
 			}
 			$this->getOwner()->{$this->arrayPropertyName} = $arrayOfDocs;
 			return true;