From 6f1a980726dffc12a9c0f27a6a863a2b9c20dac4 Mon Sep 17 00:00:00 2001
From: codes <djfoolishcodes@gmail.com>
Date: Wed, 4 Dec 2024 11:44:06 +0800
Subject: [PATCH] enhance:processNodeWidgets 1.Combo widget cannot cycle when
 clicking arrow 2.Add dataProcess so that when the widget value is changed,
 you can add conditional restrictions to the widget value 3.When modifying the
 widget value, when the widget value is bound to a property, the widget value
 and property should be modified only once, but calling setPropery directly
 causes the widget value to be modified twice.

---
 src/litegraph.js | 30 +++++++++++++++++++++---------
 1 file changed, 21 insertions(+), 9 deletions(-)

diff --git a/src/litegraph.js b/src/litegraph.js
index 359f6e1d0..c2699f4e3 100755
--- a/src/litegraph.js
+++ b/src/litegraph.js
@@ -10181,12 +10181,12 @@ LGraphNode.prototype.executeAction = function(action)
 								index = values_list.indexOf( String( w.value ) ) + delta;
 							else
 								index = values_list.indexOf( w.value ) + delta;
-							if (index >= values_list.length) {
-								index = values_list.length - 1;
-							}
-							if (index < 0) {
-								index = 0;
-							}
+                            if(index >= values_list.length) {
+                                index = 0;
+                            }
+                            if(index < 0) {
+                                index = values_list.length - 1;
+                            }
 							if( values.constructor === Array )
 								w.value = values[index];
 							else
@@ -10276,9 +10276,21 @@ LGraphNode.prototype.executeAction = function(action)
             if(widget.type == "number"){
                 value = Number(value);
             }
-            widget.value = value;
-            if ( widget.options && widget.options.property && node.properties[widget.options.property] !== undefined ) {
-                node.setProperty( widget.options.property, value );
+             widget.value = widget?.options?.dataProcess ? widget.options.dataProcess(value) : value;
+            if (widget.options && widget.options.property && node.properties[widget.options.property] !== undefined) {
+                const propertyName = widget.options.property;
+                if (!node.properties) {
+                    node.properties = {};
+                }
+                if (value !== node.properties[propertyName]) {
+                    var prev_value = node.properties[propertyName];
+                    node.properties[propertyName] = value;
+                    if(node.onPropertyChanged) {
+                        if(node.onPropertyChanged(propertyName, value, prev_value) === false)
+                            node.properties[propertyName] = prev_value;
+                    }
+                }
+
             }
             if (widget.callback) {
                 widget.callback(widget.value, that, node, pos, event);