implementing feedback, setting originalModel as optional

Author: madfalt

src/directives/formly-field.js

@@ -20,7 +20,7 @@ function formlyField($http, $q, $compile, $templateCache, $interpolate, formlyCo
     scope: {
       options: '=',
       model: '=',
-      originalModel: '=',
+      originalModel: '=?',
       formId: '@', // TODO remove formId in a breaking release
       index: '=?',
       fields: '=?',
@@ -85,6 +85,7 @@ function formlyField($http, $q, $compile, $templateCache, $interpolate, formlyCo
     function simplifyLife(options) {
       // add a few empty objects (if they don't already exist) so you don't have to undefined check everywhere
       formlyUtil.reverseDeepMerge(options, {
+        originalModel: options.model,
         extras: {},
         data: {},
         templateOptions: {},

src/directives/formly-field.test.js

@@ -1484,6 +1484,18 @@ describe('formly-field', function() {
       expect(field2.model).to.eq(scope.model.child);
       expect(field2.originalModel).not.to.eq(scope.model.child);
       expect(field2.originalModel).to.eq(scope.model);
+    });
+
+    it('should take field model as default for original model, if original value attributes has not been set', () => {
+      scope.fields = [
+        getNewField({key: 'foo', model: {foo: 'bar'}})
+      ];
+
+      compileAndDigest('<div formly-field class="formly-field" options="fields[0]" model="model"></div>');
+      $timeout.flush();
+
+      expect(field.model).to.eq(scope.fields[0].model);
+      expect(field.originalModel).to.eql(scope.fields[0].model);
 
     });
 

src/providers/formlyApiCheck.js

@@ -98,6 +98,7 @@ const fieldOptionsApiShape = {
   ).optional,
   key: apiCheck.oneOfType([apiCheck.string, apiCheck.number]).optional,
   model: modelChecker.optional,
+  originalModel: modelChecker.optional,
   className: apiCheck.string.optional,
   id: apiCheck.string.optional,
   name: apiCheck.string.optional,