From 917ad44cf07bd6322057bb3f75175331a4e27c77 Mon Sep 17 00:00:00 2001
From: Radek Simko <radek.simko@gmail.com>
Date: Sun, 26 Jun 2016 16:58:43 +0100
Subject: [PATCH] helper/schema: Fix readSet implementation (DiffFieldReader)

---
 helper/schema/field_reader_diff.go | 56 ++++++++++++++++++++++++++++--
 helper/schema/set.go               |  4 +++
 2 files changed, 57 insertions(+), 3 deletions(-)

diff --git a/helper/schema/field_reader_diff.go b/helper/schema/field_reader_diff.go
index ca6ebbee8..c62331548 100644
--- a/helper/schema/field_reader_diff.go
+++ b/helper/schema/field_reader_diff.go
@@ -238,10 +238,60 @@ func (r *DiffFieldReader) readSet(
 	// Create the set that will be our result
 	set := schema.ZeroValue().(*Set)
 
+	// Check if we're supposed to remove it
+	v, ok := r.Diff.Attributes[prefix+"#"]
+	if ok && v.New == "0" {
+		// I'm not entirely sure what's the point of
+		// returning empty set w/ Exists: true
+		return FieldReadResult{
+			Value:  set,
+			Exists: true,
+		}, nil
+	}
+
+	// Compose list of all keys (diff + source)
+	var keys []string
+
+	// Add keys from diff
+	diffContainsField := false
+	for k, _ := range r.Diff.Attributes {
+		if strings.HasPrefix(k, address[0]+".") {
+			diffContainsField = true
+		}
+		keys = append(keys, k)
+	}
+	// Bail out if diff doesn't contain the given field at all
+	if !diffContainsField {
+		return FieldReadResult{
+			Value:  set,
+			Exists: false,
+		}, nil
+	}
+	// Add keys from source
+	sourceResult, err := r.Source.ReadField(address)
+	if err == nil && sourceResult.Exists {
+		sourceSet := sourceResult.Value.(*Set)
+		sourceMap := sourceSet.Map()
+
+		for k, _ := range sourceMap {
+			key := prefix + k
+			_, ok := r.Diff.Attributes[key]
+			if !ok {
+				keys = append(keys, key)
+			}
+		}
+	}
+
+	// Keep the order consistent for hashing functions
+	sort.Strings(keys)
+
 	// Go through the map and find all the set items
-	for k, d := range r.Diff.Attributes {
-		if d.NewRemoved {
-			// If the field is removed, we always ignore it
+	// We are not iterating over the diff directly as some indexes
+	// may be missing and we expect the whole set to be returned.
+	for _, k := range keys {
+		d, ok := r.Diff.Attributes[k]
+		if ok && d.NewRemoved {
+			// If the field is being removed, we ignore it
 			continue
 		}
 		if !strings.HasPrefix(k, prefix) {
diff --git a/helper/schema/set.go b/helper/schema/set.go
index de05f40ee..0d5bebd7c 100644
--- a/helper/schema/set.go
+++ b/helper/schema/set.go
@@ -98,6 +98,10 @@ func (s *Set) List() []interface{} {
 	return result
 }
 
+func (s *Set) Map() map[string]interface{} {
+	return s.m
+}
+
 // Difference performs a set difference of the two sets, returning
 // a new third set that has only the elements unique to this set.
 func (s *Set) Difference(other *Set) *Set {