Merge pull request #20 from MarshallOfSound/master

Fixed bug where uploading an attachment would discard your changes
orionhealth · Mar 10, 2015 · ebe0149 · ebe0149
2 parents 50c6573 + e674840
commit ebe0149
Showing 1 changed file with 19 additions and 0 deletions.
diff --git a/src/main/webapp/modules/feature-test.js b/src/main/webapp/modules/feature-test.js
@@ -228,6 +228,25 @@ YUI.add('feature-test', function (Y) {
 		},
 		_addClickListeners: function () {
 			Y.all('.upload-attachment').each(Y.bind(function (item) {
+                item.ancestor().on('click', Y.bind(function(e) {
+                    var editInProgress = function(model) {
+                            var i = 0;
+                            for (var property in model.changed) {
+                                if (model.changed.hasOwnProperty(property)) {
+                                    i++;
+                                }
+                            }
+                            return i>0;
+                        },
+                        model = this.get('model');
+
+                    if (editInProgress(model)) {
+                        alert('Please save or discard your changes before uploading attachments');
+                        e.preventDefault();
+                        e.stopPropagation();
+                        return false;
+                    }
+                }, this));
 				item.on('change', Y.bind(function () {
 					var model = this.get('model'),
 						formgroup = item.ancestor(".image-form"),