feat: Update the vlan_ids property in `routeros_interface_bridge_vl…

…an` to support multiple values

docs/resources/interface_bridge_vlan.md

@@ -4,7 +4,7 @@
 ## Example Usage
 ```terraform
 resource "routeros_interface_bridge_vlan" "bridge_vlan" {
-  vlan_ids = "50"
+  vlan_ids = ["50"]
   bridge   = "bridge"
   tagged = [
     "bridge",

examples/resources/routeros_interface_bridge_vlan/resource.tf

@@ -1,5 +1,5 @@
 resource "routeros_interface_bridge_vlan" "bridge_vlan" {
-  vlan_ids = "50"
+  vlan_ids = ["50"]
   bridge   = "bridge"
   tagged = [
     "bridge",

routeros/resource_interface_bridge_vlan.go

@@ -79,8 +79,11 @@ func ResourceInterfaceBridgeVlan() *schema.Resource {
 				"separated values. E.g. untagged=ether3,ether4",
 		},
 		"vlan_ids": {
-			Type:     schema.TypeString,
+			Type:     schema.TypeSet,
 			Required: true,
+			Elem: &schema.Schema{
+				Type: schema.TypeString,
+			},
 			Description: "The list of VLAN IDs for certain port configuration. This setting accepts VLAN ID range " +
 				"as well as comma separated values. E.g. vlan-ids=100-115,120,122,128-130.",
 		},
@@ -96,5 +99,13 @@ func ResourceInterfaceBridgeVlan() *schema.Resource {
 		},
 
 		Schema: resSchema,
+		SchemaVersion: 1,
+		StateUpgraders: []schema.StateUpgrader{
+			{
+				Type: ResourceInterfaceBridgeVlanV0().CoreConfigSchema().ImpliedType(),
+				Upgrade: stateMigrationScalarToList("vlan_ids"),
+				Version: 0,
+			},
+		},
 	}
 }

routeros/resource_interface_bridge_vlan_test.go

@@ -41,7 +41,7 @@ resource "routeros_interface_bridge_vlan" "test_vlan" {
 	bridge   = "bridge"
 	untagged = ["ether1"]
 	tagged	 = ["bridge"]
-	vlan_ids = 200
+	vlan_ids = [200]
   }
 
 `

routeros/resource_interface_bridge_vlan_v0.go

@@ -0,0 +1,100 @@
+package routeros
+
+import (
+	"github.com/hashicorp/terraform-plugin-sdk/v2/helper/schema"
+)
+
+/*
+[
+  {
+    ".id": "*1",
+    "bridge": "bridge",
+    "comment": "Management",
+    "current-tagged": "bridge,ether2,ether3",
+    "current-untagged": "",
+    "disabled": "false",
+    "dynamic": "false",
+    "tagged": "ether2,ether4,ether5,bridge,ether3",
+    "untagged": "",
+    "vlan-ids": "2"
+  },
+  {...}
+]
+*/
+
+// ResourceInterfaceBridgeVlan https://wiki.mikrotik.com/wiki/Manual:Interface/Bridge#Bridge_VLAN_Filtering
+func ResourceInterfaceBridgeVlanV0() *schema.Resource {
+	resSchema := map[string]*schema.Schema{
+		MetaResourcePath: PropResourcePath("/interface/bridge/vlan"),
+		MetaId:           PropId(Id),
+		MetaSkipFields:   PropSkipFields("debug_info"),
+
+		"bridge": {
+			Type:        schema.TypeString,
+			Required:    true,
+			Description: "The bridge interface which the respective VLAN entry is intended for.",
+		},
+		KeyComment: PropCommentRw,
+		"current_tagged": {
+			Type:     schema.TypeList,
+			Computed: true,
+			Elem: &schema.Schema{
+				Type: schema.TypeString,
+			},
+		},
+		"current_untagged": {
+			Type:     schema.TypeList,
+			Computed: true,
+			Elem: &schema.Schema{
+				Type: schema.TypeString,
+			},
+		},
+		KeyDisabled: PropDisabledRw,
+		KeyDynamic:  PropDynamicRo,
+		"mvrp_forbidden": {
+			Type:     schema.TypeList,
+			Optional: true,
+			Elem: &schema.Schema{
+				Type: schema.TypeString,
+			},
+			Description:      "Ports that ignore all MRP messages and remains Not Registered (MT), as well as disables applicant from declaring specific VLAN ID (available since RouterOS 7.15).",
+			DiffSuppressFunc: AlwaysPresentNotUserProvided,
+		},
+		"tagged": {
+			Type:     schema.TypeList,
+			Optional: true,
+			Elem: &schema.Schema{
+				Type: schema.TypeString,
+			},
+			Description: "Interface list with a VLAN tag adding action in egress. This setting accepts comma " +
+				"separated values. E.g. tagged=ether1,ether2.",
+		},
+		"untagged": {
+			Type:     schema.TypeList,
+			Optional: true,
+			Elem: &schema.Schema{
+				Type: schema.TypeString,
+			},
+			Description: "Interface list with a VLAN tag removing action in egress. This setting accepts comma " +
+				"separated values. E.g. untagged=ether3,ether4",
+		},
+		"vlan_ids": {
+			Type:     schema.TypeString,
+			Required: true,
+			Description: "The list of VLAN IDs for certain port configuration. This setting accepts VLAN ID range " +
+				"as well as comma separated values. E.g. vlan-ids=100-115,120,122,128-130.",
+		},
+	}
+
+	return &schema.Resource{
+		CreateContext: DefaultCreate(resSchema),
+		ReadContext:   DefaultRead(resSchema),
+		UpdateContext: DefaultUpdate(resSchema),
+		DeleteContext: DefaultDelete(resSchema),
+		Importer: &schema.ResourceImporter{
+			StateContext: schema.ImportStatePassthroughContext,
+		},
+
+		Schema: resSchema,
+	}
+}