Row with null Id fix

beyond-the-cloud-dev · Oct 27, 2023 · 29232ac · 29232ac · vercel · Oct 27, 2023
1 parent 2bb5566
commit 29232ac
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 1 deletion.
diff --git a/force-app/main/default/classes/SOQL.cls b/force-app/main/default/classes/SOQL.cls
@@ -547,7 +547,11 @@ public virtual inherited sharing class SOQL implements Queryable {
     public Set<String> toValuesOf(SObjectField fieldToExtract) {
         // https://salesforce.stackexchange.com/questions/393308/get-a-list-of-one-column-from-a-soql-result
         builder.fields.clearAllFields(); // other fields not needed
-        return new Map<String, SObject>(with(fieldToExtract, 'Id').groupBy(fieldToExtract).toAggregated()).keySet();
+        try {
+            return new Map<String, SObject>(with(fieldToExtract, 'Id').groupBy(fieldToExtract).toAggregated()).keySet();
+        } catch (ListException e) { // Row with null Id at index: 0
+            return new Set<String>();
+        }
     }
 
     public Integer toInteger() {

diff --git a/force-app/main/default/classes/SOQL_Test.cls b/force-app/main/default/classes/SOQL_Test.cls
@@ -1775,6 +1775,18 @@ private class SOQL_Test {
         Assert.areEqual(2, accountNames.size());
     }
 
+    @IsTest
+    static void toValuesOfWhenNoValues() {
+        // Setup
+        insertAccounts();
+
+        // Test
+        Set<String> accountNames = SOQL.of(Account.SObjectType).toValuesOf(Account.Industry);
+
+        // Verify
+        Assert.areEqual(0, accountNames.size());
+    }
+
     @IsTest
     static void toValuesOfWithDefaultFields() {
         // Setup