Use existing util

Author: tobio

internal/elasticsearch/index/index/update.go

@@ -2,12 +2,12 @@ package index
 
 import (
 	"context"
-	"reflect"
 
 	"github.com/elastic/terraform-provider-elasticstack/internal/clients"
 	"github.com/elastic/terraform-provider-elasticstack/internal/clients/elasticsearch"
 	"github.com/elastic/terraform-provider-elasticstack/internal/diagutil"
 	"github.com/elastic/terraform-provider-elasticstack/internal/models"
+	"github.com/elastic/terraform-provider-elasticstack/internal/utils"
 	"github.com/hashicorp/terraform-plugin-framework-jsontypes/jsontypes"
 	"github.com/hashicorp/terraform-plugin-framework/diag"
 	"github.com/hashicorp/terraform-plugin-framework/resource"
@@ -115,7 +115,7 @@ func (r *Resource) updateSettings(ctx context.Context, client *clients.ApiClient
 		}
 	}
 
-	if !deepEqual(planDynamicSettings, stateDynamicSettings) {
+	if !utils.MapsEqual(planDynamicSettings, stateDynamicSettings) {
 		// Settings which are being removed must be explicitly set to null in the new settings
 		for setting := range stateDynamicSettings {
 			if _, ok := planDynamicSettings[setting]; !ok {
@@ -149,24 +149,3 @@ func (r *Resource) updateMappings(ctx context.Context, client *clients.ApiClient
 
 	return nil
 }
-
-// deepEqual compares two maps for deep equality, handling slices and other types
-// that are not comparable with maps.Equal
-func deepEqual(a, b map[string]interface{}) bool {
-	if len(a) != len(b) {
-		return false
-	}
-
-	for key, valueA := range a {
-		valueB, exists := b[key]
-		if !exists {
-			return false
-		}
-
-		if !reflect.DeepEqual(valueA, valueB) {
-			return false
-		}
-	}
-
-	return true
-}