v1.7.18 - Clarifies Rollup Logging Messages (#691)

* Fixes #690

Author: jamessimone

README.md

@@ -24,11 +24,11 @@ As well, don't miss [the Wiki](../../wiki), which includes even more info for co
 
 ## Deployment & Setup
 
-<a href="https://login.salesforce.com/packaging/installPackage.apexp?p0=04tKf000000sy3yIAA">
+<a href="https://login.salesforce.com/packaging/installPackage.apexp?p0=04tKf000000sy48IAA">
   <img alt="Deploy to Salesforce" src="./media/deploy-package-to-prod.png">
 </a>
 
-<a href="https://test.salesforce.com/packaging/installPackage.apexp?p0=04tKf000000sy3yIAA">
+<a href="https://test.salesforce.com/packaging/installPackage.apexp?p0=04tKf000000sy48IAA">
   <img alt="Deploy to Salesforce Sandbox" src="./media/deploy-package-to-sandbox.png">
 </a>
 <br/>

extra-tests/classes/RollupLoggerTests.cls

@@ -62,7 +62,7 @@ private class RollupLoggerTests {
     RollupPlugin.pluginMocks = new List<RollupPlugin__mdt>{ new RollupPlugin__mdt(DeveloperName = ControlUpdatingLogger.class.getName()) };
 
     RollupLogger.Instance.log('Should not be logged', System.LoggingLevel.DEBUG);
-    System.assertEquals('logging isn\'t enabled, no further output', localLogString);
+    System.assertEquals('logging isn\'t enabled, further log messages paused unless otherwise re-activated', localLogString);
 
     RollupLogger.Instance.updateRollupControl(new RollupControl__mdt(IsRollupLoggingEnabled__c = true));
     String expectedLogMessage = 'hi';

package.json

@@ -1,6 +1,6 @@
 {
   "name": "apex-rollup",
-  "version": "1.7.17",
+  "version": "1.7.18",
   "description": "Fast, configurable, elastically scaling custom rollup solution. Apex Invocable action, one-liner Apex trigger/CMDT-driven logic, and scheduled Apex-ready.",
   "repository": {
     "type": "git",

rollup-namespaced/README.md

@@ -18,12 +18,12 @@ For more info, see the base `README`.
 
 ## Deployment & Setup
 
-<a href="https://login.salesforce.com/packaging/installPackage.apexp?p0=04tKf000000sy43IAA">
+<a href="https://login.salesforce.com/packaging/installPackage.apexp?p0=04tKf000000sy4DIAQ">
   <img alt="Deploy to Salesforce"
        src="./media/deploy-package-to-prod.png">
 </a>
 
-<a href="https://test.salesforce.com/packaging/installPackage.apexp?p0=04tKf000000sy43IAA">
+<a href="https://test.salesforce.com/packaging/installPackage.apexp?p0=04tKf000000sy4DIAQ">
   <img alt="Deploy to Salesforce Sandbox"
        src="./media/deploy-package-to-sandbox.png">
 </a>

rollup-namespaced/sfdx-project.json

@@ -4,8 +4,8 @@
             "default": true,
             "package": "apex-rollup-namespaced",
             "path": "rollup-namespaced/source/rollup",
-            "versionName": "Fixes issue with inadvertent special handling for non-polymorphic type fields in parent where clauses",
-            "versionNumber": "1.2.14.0",
+            "versionName": "Updates some logging messages generated by RollupLogger to clarify when logging is disabled and then re-enabled",
+            "versionNumber": "1.2.15.0",
             "versionDescription": "Fast, configurable, elastically scaling custom rollup solution. Apex Invocable action, one-liner Apex trigger/CMDT-driven logic, and scheduled Apex-ready.",
             "releaseNotesUrl": "https://github.com/jamessimone/apex-rollup/releases/latest",
             "unpackagedMetadata": {
@@ -31,6 +31,7 @@
         "[email protected]": "04tKf000000sxy4IAA",
         "[email protected]": "04tKf000000sy3eIAA",
         "[email protected]": "04tKf000000sy3tIAA",
-        "[email protected]": "04tKf000000sy43IAA"
+        "[email protected]": "04tKf000000sy43IAA",
+        "[email protected]": "04tKf000000sy4DIAQ"
     }
 }

rollup/core/classes/RollupLogger.cls

@@ -2,7 +2,7 @@
 global without sharing virtual class RollupLogger implements ILogger {
   @TestVisible
   // this gets updated via the pipeline as the version number gets incremented
-  private static final String CURRENT_VERSION_NUMBER = 'v1.7.17';
+  private static final String CURRENT_VERSION_NUMBER = 'v1.7.18';
   private static final System.LoggingLevel FALLBACK_LOGGING_LEVEL = System.LoggingLevel.DEBUG;
   private static final RollupPlugin PLUGIN = new RollupPlugin();
 
@@ -180,7 +180,7 @@ global without sharing virtual class RollupLogger implements ILogger {
       if (this.control.IsRollupLoggingEnabled__c == false && this.hasDisabledMessageBeenLogged == false) {
         this.hasDisabledMessageBeenLogged = true;
         this.control.IsRollupLoggingEnabled__c = true;
-        this.log('logging isn\'t enabled, no further output', System.LoggingLevel.WARN);
+        this.log('logging isn\'t enabled, further log messages paused unless otherwise re-activated', System.LoggingLevel.INFO);
         this.control.IsRollupLoggingEnabled__c = false;
       } else if (this.control.IsRollupLoggingEnabled__c) {
         for (ILogger logger : this.loggers) {
@@ -198,8 +198,12 @@ global without sharing virtual class RollupLogger implements ILogger {
     }
 
     public CombinedLogger updateRollupControl(RollupControl__mdt control) {
+      String updateMessage = 'updating control record';
+      if (control.IsRollupLoggingEnabled__c == true && this.control.IsRollupLoggingEnabled__c != true) {
+        updateMessage = 'logging is now re-enabled, ' + updateMessage;
+      }
       this.control = control;
-      this.log('updating control record', control, System.LoggingLevel.INFO);
+      this.log(updateMessage, control, System.LoggingLevel.INFO);
       return this;
     }
   }

sfdx-project.json

@@ -5,8 +5,8 @@
             "package": "apex-rollup",
             "path": "rollup",
             "scopeProfiles": true,
-            "versionName": "Fixes issue with inadvertent special handling for non-polymorphic type fields in parent where clauses",
-            "versionNumber": "1.7.17.0",
+            "versionName": "Updates some logging messages generated by RollupLogger to clarify when logging is disabled and then re-enabled",
+            "versionNumber": "1.7.18.0",
             "versionDescription": "Fast, configurable, elastically scaling custom rollup solution. Apex Invocable action, one-liner Apex trigger/CMDT-driven logic, and scheduled Apex-ready.",
             "releaseNotesUrl": "https://github.com/jamessimone/apex-rollup/releases/latest",
             "unpackagedMetadata": {
@@ -111,6 +111,7 @@
         "[email protected]": "04tKf000000sxxzIAA",
         "[email protected]": "04tKf000000sy3ZIAQ",
         "[email protected]": "04tKf000000sy3oIAA",
-        "[email protected]": "04tKf000000sy3yIAA"
+        "[email protected]": "04tKf000000sy3yIAA",
+        "[email protected]": "04tKf000000sy48IAA"
     }
 }