Editor: reset shared editors when moving between cells

Editor.js

@@ -293,6 +293,12 @@ define([
 
 			cellElement.innerHTML = '';
 			domClass.add(cellElement, 'dgrid-cell-editing');
+
+			// If a shared editor is being moved to a different cell reset it to clear validation state
+			if (cmp.domNode && cmp.domNode.parentNode !== cellElement) {
+				cmp.reset && cmp.reset();
+			}
+
 			cellElement.appendChild(cmp.domNode || cmp);
 
 			if (isWidget && !column.editOn) {

test/intern/functional/Editor.html

@@ -19,16 +19,23 @@ <h2>Grid with editors</h2>
 
 		<script>
 			var grid,
+				data = [
+					{ id: 0, name: '1', description: 'one' },
+					{ id: 1, name: '2', description: 'two' },
+					{ id: 2, name: '3', description: 'three' }
+				],
 				datachangeStack = [],
 				ready = false,
-				setEditorToTextBox;
+				setEditorToTextBox,
+				setEditorToValidationTextBox;
 
 			require([
 				'dojo/_base/declare',
 				'dgrid/Grid',
 				'dgrid/Editor',
-				'dijit/form/TextBox'
-			], function (declare, Grid, Editor, TextBox) {
+				'dijit/form/TextBox',
+				'dijit/form/ValidationTextBox'
+			], function (declare, Grid, Editor, TextBox, ValidationTextBox) {
 				function createColumnConfig(editorType) {
 					return {
 						id: 'ID',
@@ -48,11 +55,7 @@ <h2>Grid with editors</h2>
 					columns: createColumnConfig('text')
 				}, 'grid');
 
-				grid.renderArray([
-					{ id: 0, name: '1', description: 'one' },
-					{ id: 1, name: '2', description: 'two' },
-					{ id: 2, name: '3', description: 'three' }
-				]);
+				grid.renderArray(data);
 
 				// Push values received by the "dgrid-datachange" event into a global stack.
 				// The functional test can read from this stack to verify the values it is
@@ -65,6 +68,15 @@ <h2>Grid with editors</h2>
 					grid.set('columns', createColumnConfig(TextBox));
 				};
 
+				setEditorToValidationTextBox = function () {
+					var columns = createColumnConfig(ValidationTextBox);
+
+					columns.description.editorArgs = {
+						required: true
+					};
+					grid.set('columns', columns);
+				};
+
 				ready = true;
 			});
 		</script>

test/intern/functional/Editor.js

@@ -7,7 +7,7 @@ define([
 	'require'
 ], function (test, assert, util, pollUntil, keys, require) {
 	// Number of visible rows in the grid.
-	// Check the data loaded in test file (editor.html) and rows visible
+	// Check the data loaded in test file (Editor.html) and rows visible
 	// when the page is loaded to ensure this is correct.
 	var GRID_ROW_COUNT = 3;
 	var rowSelectorPrefix = '#grid-row-';
@@ -346,5 +346,34 @@ define([
 
 		test.test('autoSave: true', createAutosaveTest());
 		test.test('autoSave: true - TextBox', createAutosaveTest(setTextBox));
+
+		test.test('shared editor reset', function() {
+			return this.remote
+				.get(require.toUrl('./Editor.html'))
+				.then(pollUntil(function () {
+					return window.ready;
+				}, null, 5000))
+				.execute(function () {
+					/* global setEditorToValidationTextBox, data */
+					setEditorToValidationTextBox();
+					data[1].description = '';
+					data[2].description = '';
+					grid.refresh();
+					grid.renderArray(data);
+				})
+				.findByCssSelector('#grid-row-1 .field-description')
+					.click()
+					.end()
+				.findByCssSelector('#grid-row-2 .field-description')
+					.click()
+					.findByCssSelector('.dijitInputInner')
+						.getAttribute('aria-invalid')
+						.then(function (isInvalid) {
+							assert.notStrictEqual(isInvalid, 'true',
+								'Cell editor validation state should not carry over into newly active cell editor');
+							})
+						.end()
+					.end();
+		});
 	});
 });