fixbug, see: #333

kaola-fed · Mar 27, 2018 · f04e257 · f04e257
1 parent f2bd687
commit f04e257
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 8 deletions.
diff --git a/src/js/components/form/KLForm/KLFormItem/index.js b/src/js/components/form/KLForm/KLFormItem/index.js
@@ -36,14 +36,13 @@ const KLFormItem = Validation.extend({
   },
   init() {
     const parentValidator = this._parentValidator;
-    this.$watch('this.controls.length', (newValue, oldValue) => {
-      /* 处理kl-form-item下面kl-select数量变化的情况,当从没有变为有时,需要赋值 */
-      if (oldValue === undefined) {
-        return;
-      }
+    this.$watch('this.controls.length', () => {
       if (parentValidator && parentValidator.initSelectorSource) {
         parentValidator.initSelectorSource();
       }
+      if (parentValidator && parentValidator.initFormItem) {
+        parentValidator.initFormItem();
+      }
     });
 
     this.$watch('required', function () {

diff --git a/src/js/components/form/KLForm/index.js b/src/js/components/form/KLForm/index.js
@@ -58,6 +58,8 @@ const KLForm = Validation.extend({
       this.initSelectorSource();
       this.initFormItem();
     });
+
+    this.__reqSourceOnce = _.debounce(this.__reqSource.bind(this), 200, false);
   },
   initFormItem() {
     const controls = this.controls;
@@ -85,8 +87,7 @@ const KLForm = Validation.extend({
     if (!this.data.service || !this.selectors.length) {
       return;
     }
-
-    this.__reqSource();
+    this.__reqSourceOnce();
   },
   __getSourceKeys() {
     return this.selectors.map($formitem => $formitem.data.sourceKey);

diff --git a/src/js/util/validationMixin.js b/src/js/util/validationMixin.js
@@ -22,7 +22,7 @@ module.exports = function (Component) {
         this.$on('destroy', function () {
           const index = $outer.controls.indexOf(this);
           if (index !== -1) {
-            $outer.controls = $outer.controls.splice(index, 1);
+            $outer.controls.splice(index, 1);
           }
         });
       }