Merge pull request #3 from azurerm/fix-coalesce-type

fix: coalesce return was not working using display name
azurerm · Jul 18, 2023 · 403c78d · 403c78d
2 parents d26a795 + 5fbd87a
commit 403c78d
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 6 deletions.
diff --git a/examples/main.tf b/examples/main.tf
@@ -1,5 +1,5 @@
 module "azure_location" {
-  source  = "../"
+  source   = "../"
   location = "West Europe"
 }
 
@@ -15,7 +15,7 @@ output "location" {
 # }
 
 module "azure_fake_location" {
-  source  = "../"
+  source   = "../"
   location = "Test Europe"
 }
 

diff --git a/main.tf b/main.tf
@@ -21,8 +21,10 @@ locals {
     }
   }
 
-  location = coalesce(
-    lookup(local.locations_name, var.location, null),
-    lookup(local.locations_display_name, var.location, "none")
-  )
+  lookup_name         = lookup(local.locations_name, var.location, null)
+  lookup_display_name = lookup(local.locations_display_name, var.location, null)
+  location = try(coalesce(
+    local.lookup_name,
+    local.lookup_display_name
+  ), "none")
 }