Add input default values.

claudetech · Mar 4, 2016 · 628d40c · 628d40c
1 parent 5c6ee9c
commit 628d40c
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 3 deletions.
diff --git a/lib/inputs/base.js b/lib/inputs/base.js
@@ -7,9 +7,7 @@ export default class BaseInput {
     riot.observable(this)
     assert(config.name, 'An input must have a name')
     this.config = config
-    if (config.value) {
-      this._setValue(config.value)
-    }
+    this._setValue(config.value || this.defaultValue, {silent: true})
     if (config.formName) {
       this.formName = config.formName
     }
@@ -66,6 +64,10 @@ export default class BaseInput {
     return this.config.type || this.constructor.type
   }
 
+  get defaultValue() {
+    return undefined
+  }
+
   // TODO: pre pack some validators to avoid having to pass a callback
   validate() {
     if (this.config.validate) {

diff --git a/tests/unit/base-input_test.js b/tests/unit/base-input_test.js
@@ -2,6 +2,9 @@ import {expect}    from 'chai'
 import {BaseInput} from 'riot-form'
 
 class DummyInput extends BaseInput {
+  get defaultValue() {
+    return 'dummy'
+  }
 }
 DummyInput.defaultTag = 'dummy-tag'
 
@@ -15,6 +18,11 @@ describe('BaseInput', () => {
       const input = new DummyInput({name: 'hello', value: 'foobar'})
       expect(input.value).to.eq('foobar')
     })
+
+    it('should set value to default value', () => {
+      const input = new DummyInput({name: 'hello'})
+      expect(input.value).to.eq('dummy')
+    })
   })
 
   describe('name', () => {