Refactor VariablePropertiesMixin.

Author: BirjuVachhani

lib/src/api/mixins.dart

@@ -4,7 +4,6 @@
 
 import 'package:json_annotation/json_annotation.dart';
 
-import 'models/condition.dart';
 import 'models/models.dart';
 import 'nodes/nodes.dart';
 
@@ -794,7 +793,7 @@ mixin ScalableMixin on BaseNode {
 
 /// A mixin that allows a node to store variables for its properties so that
 /// value of those properties can be retrieved from those variables if present.
-mixin VariablePropertiesMixin on BaseNode {
+mixin VariablePropertiesMixin {
   /// property_name -> variable_name
   @JsonKey(defaultValue: {})
   Map<String, String> variables = {};
@@ -1038,6 +1037,8 @@ mixin SerializableMixin {
   Map toJson();
 }
 
+/// A mixin that adds the ability to add conditions to a node.
 mixin ConditionsMixin {
+  /// List of conditions that are applied to this node.
   List<BaseCondition> conditions = [];
 }

lib/src/api/nodes/accordion_node.dart

@@ -20,7 +20,6 @@ class AccordionNode extends SceneNode
         ChildrenMixin,
         RowColumnMixin,
         CustomPropertiesMixin,
-        VariablePropertiesMixin,
         IsolatedMixin {
   @override
   final String type = 'accordion';
@@ -54,6 +53,8 @@ class AccordionNode extends SceneNode
     super.reactions,
     required List<String> children,
     this.isExpanded = true,
+    super.variables,
+    super.multipleVariables,
   }) {
     setChildrenMixin(children: children);
     setRowColumnMixin(

lib/src/api/nodes/accordion_node.g.dart

@@ -21,7 +21,7 @@ const double kAppBarDefaultHeight = 56;
 /// in Flutter for more details.
 @JsonSerializable()
 class AppBarNode extends SceneNode
-    with CustomPropertiesMixin, VariablePropertiesMixin, ParentReactionMixin {
+    with CustomPropertiesMixin, ParentReactionMixin {
   @override
   final String type = 'appBar';
 
@@ -48,6 +48,8 @@ class AppBarNode extends SceneNode
     super.edgePins,
     super.aspectRatioLock,
     super.positioningMode,
+    super.variables,
+    super.multipleVariables,
     required this.properties,
   });
 

lib/src/api/nodes/app_bar_node.dart

@@ -28,7 +28,8 @@ bool isTestLayout = false;
 /// If you wish to update the values of a node, it is highly recommended you
 /// use the [update] extension method or the [updateNode] method inside the
 /// [NodeProcessor] class.
-abstract class BaseNode with SerializableMixin, EquatableMixin {
+abstract class BaseNode
+    with SerializableMixin, EquatableMixin, VariablePropertiesMixin {
   /// [type] is a string representation for the type of this node. It is a
   /// unique key that this node class uses for static registration. It is
   /// usually the lowerCamelCase of the class' name. Overriding [type] is
@@ -383,6 +384,8 @@ abstract class BaseNode with SerializableMixin, EquatableMixin {
     this.parentID = '',
     this.widthFactor,
     this.heightFactor,
+    Map<String, String>? variables,
+    Map<String, List<String>>? multipleVariables,
   })  : _basicBoxLocal = basicBoxLocal,
         _rotationDegrees = rotationDegrees,
         globalRotationDegrees = rotationDegrees,
@@ -409,6 +412,8 @@ abstract class BaseNode with SerializableMixin, EquatableMixin {
 
     NodeProcessor._computeInnerBoxLocal(this);
     NodeProcessor._computeInnerBoxGlobal(this);
+    setVariablesMixin(
+        variables: variables, multipleVariables: multipleVariables);
   }
 
   /// The minimum internal size is the size that this node

lib/src/api/nodes/app_bar_node.g.dart

@@ -61,7 +61,7 @@ enum ButtonTypeEnum {
 /// more details.
 @JsonSerializable()
 class ButtonNode extends SceneNode
-    with CustomPropertiesMixin, VariablePropertiesMixin {
+    with CustomPropertiesMixin {
   @override
   final String type = 'button';
 
@@ -127,12 +127,9 @@ class ButtonNode extends SceneNode
     super.parentID,
     super.positioningMode,
     required this.properties,
-    Map<String, String>? variables,
-    Map<String, List<String>>? multipleVariables,
-  }) {
-    setVariablesMixin(
-        variables: variables, multipleVariables: multipleVariables);
-  }
+    super.variables,
+    super.multipleVariables,
+  });
 
   @override
   BoxConstraintsModel internalConstraints({

lib/src/api/nodes/auto_placeholder_node.g.dart

@@ -19,8 +19,7 @@ class CanvasNode extends ParentNode
         RowColumnMixin,
         IsolatedMixin,
         ScrollableMixin,
-        CustomPropertiesMixin,
-        VariablePropertiesMixin {
+        CustomPropertiesMixin {
   @override
   final String type = 'canvas';