not validate embedded, if attribute is not safe *

Author: serieznyi

AbstractEmbeddedBehavior.php

@@ -105,9 +105,11 @@ public function proxy($event)
      */
     public function validate()
     {
-        $this->storage->setScenario($this->owner->scenario);
-        if (!$this->storage->validate()) {
-            $this->owner->addError($this->attribute, \Yii::t('yii', 'Embedded document in {attribute} must be valid.'));
+        if ($this->owner->isAttributeSafe($this->fakeAttribute)) {
+            $this->storage->setScenario($this->owner->scenario);
+            if (!$this->storage->validate()) {
+                $this->owner->addError($this->attribute, \Yii::t('yii', 'Embedded document in {attribute} must be valid.'));
+            }
         }
     }
 

composer.json

@@ -16,7 +16,9 @@
     "minimum-stability": "dev",
     "require": {
         "yiisoft/yii2": "*",
-        "yiisoft/yii2-mongodb": "*"
+        "yiisoft/yii2-mongodb": "*",
+        "yiisoft/yii2-codeception": "*",
+        "codeception/codeception": "*"
     },
     "autoload": {
       "psr-4": {

tests/_bootstrap.php

@@ -9,9 +9,9 @@
 
 defined('YII_ENV') or define('YII_ENV', 'test');
 
-require_once(__DIR__ . '/../../../../vendor/autoload.php');
+require_once(__DIR__ . '/../vendor/autoload.php');
 
-require_once(__DIR__ . '/../../../../vendor/yiisoft/yii2/Yii.php');
+require_once(__DIR__ . '/../vendor/yiisoft/yii2/Yii.php');
 
 $config = require('_config.php');
 

tests/_config.php

@@ -14,7 +14,7 @@
         ],
         'mongodb' => [
             'class' => 'yii\mongodb\Connection',
-            'dsn' => 'mongodb://mongo:27017/mongodb_embedded_test',
+            'dsn' => 'mongodb://localhost:27018/mongodb_embedded_test',
         ],
     ],
 ];

tests/models/SlaveEmbeddedClass.php

@@ -22,8 +22,8 @@ class SlaveEmbeddedClass extends EmbeddedDocument
     public function rules()
     {
         return [
-            ['value', 'boolean', 'on' => 'requiredValue'],
             ['name', 'integer', 'on' => 'requiredName'],
+            ['value', 'boolean', 'on' => 'requiredValue'],
             [['name', 'value'], 'safe', 'on'=>'default']
         ];
     }