fixed bug where grouped data transform didn't work if group-by column

was numeric
WeaveTeam · Aug 6, 2015 · 7b52ad7 · 7b52ad7
1 parent 8ed841c
commit 7b52ad7
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/WeaveData/src/weave/data/Transforms/GroupedDataTransform.as b/WeaveData/src/weave/data/Transforms/GroupedDataTransform.as
@@ -167,8 +167,14 @@ package weave.data.Transforms
 				var stringLookup:Object = {};
 				var keyType:String = groupKeyType.value || groupByColumn.getMetadata(ColumnMetadata.DATA_TYPE);
 				for each (var key:IQualifiedKey in groupByColumn.keys)
-				{
-					var localName:String = groupByColumn.getValueFromKey(key, String);
+				{
+					var localName:String;
+					// if the foreign key column is numeric, avoid using the formatted strings as keys
+					if (groupByColumn.getMetadata(ColumnMetadata.DATA_TYPE) == DataType.NUMBER)
+						localName = groupByColumn.getValueFromKey(key, Number);
+					else
+						localName = groupByColumn.getValueFromKey(key, String);
+
 					if (!stringLookup[localName])
 					{
 						stringLookup[localName] = true;