Merge pull request #180 from AtlasOfLivingAustralia/feature/issue176

Fixed checkWarnings issue after making SpeciesViewModel a DataModelIt…

Author: salomon-j

grails-app/assets/javascripts/forms.js

@@ -722,7 +722,8 @@ function orEmptyArray(v) {
             var constraints = {
                 val: warningRule
             };
-            return validate({val: self()}, constraints, {fullMessages: false});
+            var data = ko.utils.unwrapObservable(self); // The SpeciesViewModel is a DataModelItem but not a knockout observable
+            return validate({val: data}, constraints, {fullMessages: false});
         };
 
         self.evaluateBehaviour = function (type, defaultValue) {

src/test/js/spec/SpeciesViewModelSpec.js

@@ -0,0 +1,7 @@
+describe("SpeciesViewModel Spec", function () {
+    it("Can participate in the DataModelItem calls like checkWarnings", function () {
+
+        let speciesViewModel = new SpeciesViewModel({}, {searchBieUrl:'/species/searchBie'}, {});
+        expect(speciesViewModel.checkWarnings()).toBeUndefined();
+    });
+});