From 234c1c482a7bfc74a389da8d0f550ec88185ebce Mon Sep 17 00:00:00 2001
From: Pam Selle <204372+pselle@users.noreply.github.com>
Date: Fri, 2 Aug 2019 16:15:18 -0400
Subject: [PATCH] Add a comment

---
 lang/blocktoattr/variables.go | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/lang/blocktoattr/variables.go b/lang/blocktoattr/variables.go
index f2e89861e..59ba47bf4 100644
--- a/lang/blocktoattr/variables.go
+++ b/lang/blocktoattr/variables.go
@@ -34,6 +34,8 @@ func walkVariables(node dynblock.WalkVariablesNode, body hcl.Body, schema *confi
 		if blockS, exists := schema.BlockTypes[child.BlockTypeName]; exists {
 			vars = append(vars, walkVariables(child.Node, child.Body(), &blockS.Block)...)
 		} else if attrS, exists := schema.Attributes[child.BlockTypeName]; exists && attrS.Type.IsCollectionType() && attrS.Type.ElementType().IsObjectType() {
+			// ☝️Check for collection type before element type, because if this is a mis-placed reference,
+			// a panic here will prevent other useful diags from being elevated to show the user what to fix
 			synthSchema := SchemaForCtyElementType(attrS.Type.ElementType())
 			vars = append(vars, walkVariables(child.Node, child.Body(), synthSchema)...)
 		}