#16 Blocks now only save if they've been modified

neo/fieldtypes/NeoFieldType.php

@@ -233,6 +233,7 @@ public function prepValueFromPost($data)
 			else
 			{
 				$block = $oldBlocksById[$blockId];
+				$block->modified = (isset($blockData['modified']) ? (bool) $blockData['modified'] : true);
 			}
 
 			$block->setOwner($this->element);

neo/models/Neo_BlockModel.php

@@ -8,6 +8,16 @@
  */
 class Neo_BlockModel extends BaseElementModel
 {
+	// Public properties
+
+	/**
+	 * Used to indicate whether this block needs to be saved to the database.
+	 *
+	 * @var bool
+	 */
+	public $modified = true;
+
+
 	// Protected properties
 
 	protected $elementType = Neo_ElementType::NeoBlock;

neo/services/NeoService.php

@@ -702,7 +702,7 @@ public function validateBlock(Neo_BlockModel $block)
 	 */
 	public function saveBlock(Neo_BlockModel $block, $validate = true)
 	{
-		if(!$validate || $this->validateBlock($block))
+		if($block->modified && (!$validate || $this->validateBlock($block)))
 		{
 			$blockRecord = $this->_getBlockRecord($block);
 			$isNewBlock = $blockRecord->isNewRecord();

src/input/Block.js

@@ -20,7 +20,8 @@ const _defaults = {
 	level: 0,
 	buttons: null,
 	enabled: true,
-	collapsed: false
+	collapsed: false,
+	modified: true
 }
 
 export default Garnish.Base.extend({
@@ -30,6 +31,8 @@ export default Garnish.Base.extend({
 	_initialised: false,
 	_expanded: true,
 	_enabled: true,
+	_modified: true,
+	_initialPostData: null,
 
 	init(settings = {})
 	{
@@ -39,6 +42,7 @@ export default Garnish.Base.extend({
 		this._blockType = settings.blockType
 		this._id = settings.id
 		this._buttons = settings.buttons
+		this._modified = settings.modified
 
 		NS.enter(this._templateNs)
 
@@ -47,7 +51,8 @@ export default Garnish.Base.extend({
 			id: this._id,
 			enabled: !!settings.enabled,
 			collapsed: !!settings.collapsed,
-			level: settings.level
+			level: settings.level,
+			modified: settings.modified
 		}))
 
 		NS.leave()
@@ -68,6 +73,7 @@ export default Garnish.Base.extend({
 		this.$enabledInput = $neo.filter('[data-neo-b="input.enabled"]')
 		this.$collapsedInput = $neo.filter('[data-neo-b="input.collapsed"]')
 		this.$levelInput = $neo.filter('[data-neo-b="input.level"]')
+		this.$modifiedInput = $neo.filter('[data-neo-b="input.modified"]')
 		this.$status = $neo.filter('[data-neo-b="status"]')
 
 		if(this._buttons)
@@ -95,6 +101,12 @@ export default Garnish.Base.extend({
 
 		this.addListener(this.$togglerButton, 'dblclick', '@doubleClickTitle')
 		this.addListener(this.$tabButton, 'click', '@setTab')
+
+		if(!this.isNew())
+		{
+			this._initialPostData = Garnish.getPostData(this.$contentContainer)
+			this._detectChangeInterval = setInterval(() => this._detectChange(), 500)
+		}
 	},
 
 	initUi()
@@ -382,6 +394,18 @@ export default Garnish.Base.extend({
 		}
 	},
 
+	_detectChange()
+	{
+		const postData = Garnish.getPostData(this.$contentContainer)
+		const modified = !Craft.compare(postData, this._initialPostData)
+
+		if(modified !== this._modified)
+		{
+			this.$modifiedInput.val(modified ? 1 : 0)
+			this._modified = modified
+		}
+	},
+
 	'@settingSelect'(e)
 	{
 		const $option = $(e.option)

src/input/templates/block.twig

@@ -1,6 +1,7 @@
 {% set hasTabs = (type.getTabs() is not empty) %}
 {% set isParent = type.isParent() %}
 <div class="ni_block">
+	<input type="hidden" name="{{ 'modified'|ns }}" value="{{ modified ? 1 : 0 }}" data-neo-b="input.modified">
 	<input type="hidden" name="{{ 'type'|ns }}" value="{{ type.getHandle() }}">
 	<input type="hidden" name="{{ 'enabled'|ns }}" value="{{ enabled ? 1 : 0 }}" data-neo-b="input.enabled">
 	<input type="hidden" name="{{ 'collapsed'|ns }}" value="{{ collapsed ? 1 : 0 }}" data-neo-b="input.collapsed">