Upgraded sf cli to v2.13.9

Hopefully this will resolve some pipeline issues... maybe
jongpie · Oct 24, 2023 · 639359d · 639359d
1 parent 7a983ff
commit 639359d
Show file tree

Hide file tree

Showing 4 changed files with 12,660 additions and 12,203 deletions.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -300,7 +300,6 @@ jobs:
             - name: 'Temporarily delete problematic metadata'
               run: |
                   rm -rf ./config/experience-cloud/
-                  rm -rf ./nebula-logger/unsorted/main/default/profiles/
 
             - name: 'Create Beta Managed Package Version'
               run: npm run package:version:create:managed

diff --git a/nebula-logger/core/main/logger-engine/classes/LogEntryEventBuilder.cls b/nebula-logger/core/main/logger-engine/classes/LogEntryEventBuilder.cls
@@ -953,10 +953,12 @@ global with sharing class LogEntryEventBuilder {
             return null;
         }
 
-        // Record field-stripping isn't done by reference, so the map's values have to be reassigned
-        Map<String, SObject> strippedRecordIdToRecord = new Map<String, SObject>(); //changed key type from Id to String
+        // In Apex, a Map<Id, SObject> with a null ID value will break during serialization,
+        // so convert it to a Map<String, SObject> first
+        Map<String, SObject> strippedRecordIdToRecord = new Map<String, SObject>();
         for (Id recordId : recordIdToRecord.keySet()) {
             SObject record = recordIdToRecord.get(recordId);
+            // Record field-stripping isn't done by reference, so the record has to be reassigned
             record = isRecordFieldStrippingEnabled == false ? record : stripInaccessible(record);
             String recordIdString = recordId == null ? '' : recordId;
             strippedRecordIdToRecord.put(recordIdString, record);