Bulk updated the convention used throughout the codebase for checking…

… Booleans from 'if(condition == true)' to just 'if(condition)'

config/experience-cloud/classes/MyProfilePageControllerTest.cls

@@ -19,7 +19,7 @@ public with sharing class MyProfilePageControllerTest {
             // System.Assert.areEqual(currentUser.Id, controller.getUser().Id, 'Did not successfully load the current user');
             // System.Assert.isTrue(controller.getIsEdit() == false, 'isEdit should default to false');
             controller.edit();
-            // System.Assert.isTrue(controller.getIsEdit() == true);
+            // System.Assert.isTrue(controller.getIsEdit());
             controller.cancel();
             // System.Assert.isTrue(controller.getIsEdit() == false);
 
@@ -41,7 +41,7 @@ public with sharing class MyProfilePageControllerTest {
                 // System.Assert.areEqual(existingPortalUser.Id, controller.getUser().Id, 'Did not successfully load the current user');
                 // System.Assert.isTrue(controller.getIsEdit() == false, 'isEdit should default to false');
                 controller.edit();
-                // System.Assert.isTrue(controller.getIsEdit() == true);
+                // System.Assert.isTrue(controller.getIsEdit());
 
                 controller.cancel();
                 // System.Assert.isTrue(controller.getIsEdit() == false);

docs/apex/Configuration/LoggerPlugin.md

@@ -43,7 +43,7 @@ The list of matching `LoggerPlugin_t` records
 
 #### `getPluginConfigurations()` → `List<LoggerPlugin_t>`
 
-Returns all enabled `LoggerPlugin_t` records, based on `IsEnabled__c == true`
+Returns all enabled `LoggerPlugin_t` records (where `IsEnabled__c` is set to `true`)
 
 ##### Return
 

docs/apex/Logger-Engine/FlowLogger.md

@@ -76,7 +76,7 @@ Boolean used to determine if logs are saved to Salesforce
 
 ###### `saveMethodName` → `String`
 
-String name of the instance of Logger.SaveMethod to use when 'Save Log' == true
+String name of the instance of Logger.SaveMethod to use when 'Save Log' is set to `true`
 
 ###### `scenario` → `String`
 

docs/apex/Logger-Engine/Logger.md

@@ -4738,7 +4738,7 @@ Boolean
 
 #### `newEntry(System.LoggingLevel loggingLevel, LogMessage logMessage, Boolean shouldSave)` → `LogEntryEventBuilder`
 
-Adds a new instance of LogEntryEventBuilder to Logger's buffer, if shouldSave == true
+Adds a new instance of LogEntryEventBuilder to Logger's buffer, when `shouldSave` is `true`
 
 ##### Parameters
 

docs/apex/Test-Utilities/LoggerMockDataCreator.md

@@ -76,9 +76,9 @@ Creates a mock instance of `Database.DeleteResult` - a mock is used instead of a
 
 ##### Parameters
 
-| Param       | Description                                                     |
-| ----------- | --------------------------------------------------------------- |
-| `isSuccess` | Indicates if the generated mock should have `isSuccess` == true |
+| Param       | Description                                             |
+| ----------- | ------------------------------------------------------- |
+| `isSuccess` | Indicates if the generated mock should have `isSuccess` |
 
 ##### Return
 
@@ -96,10 +96,10 @@ Creates a mock instance of `Database.DeleteResult` - a mock is used instead of a
 
 ##### Parameters
 
-| Param       | Description                                                     |
-| ----------- | --------------------------------------------------------------- |
-| `isSuccess` | Indicates if the generated mock should have `isSuccess` == true |
-| `recordId`  | The record ID to use within the mock result                     |
+| Param       | Description                                             |
+| ----------- | ------------------------------------------------------- |
+| `isSuccess` | Indicates if the generated mock should have `isSuccess` |
+| `recordId`  | The record ID to use within the mock result             |
 
 ##### Return
 
@@ -117,9 +117,9 @@ Creates a mock instance of `Database.LeadConvertResult` - a mock is used instead
 
 ##### Parameters
 
-| Param       | Description                                                     |
-| ----------- | --------------------------------------------------------------- |
-| `isSuccess` | Indicates if the generated mock should have `isSuccess` == true |
+| Param       | Description                                             |
+| ----------- | ------------------------------------------------------- |
+| `isSuccess` | Indicates if the generated mock should have `isSuccess` |
 
 ##### Return
 
@@ -137,10 +137,10 @@ Creates a mock instance of `Database.LeadConvertResult` - a mock is used instead
 
 ##### Parameters
 
-| Param       | Description                                                     |
-| ----------- | --------------------------------------------------------------- |
-| `isSuccess` | Indicates if the generated mock should have `isSuccess` == true |
-| `recordId`  | The record ID to use within the mock result                     |
+| Param       | Description                                             |
+| ----------- | ------------------------------------------------------- |
+| `isSuccess` | Indicates if the generated mock should have `isSuccess` |
+| `recordId`  | The record ID to use within the mock result             |
 
 ##### Return
 
@@ -158,9 +158,9 @@ Creates a mock instance of `Database.MergeResult` - a mock is used instead of an
 
 ##### Parameters
 
-| Param       | Description                                                     |
-| ----------- | --------------------------------------------------------------- |
-| `isSuccess` | Indicates if the generated mock should have `isSuccess` == true |
+| Param       | Description                                             |
+| ----------- | ------------------------------------------------------- |
+| `isSuccess` | Indicates if the generated mock should have `isSuccess` |
 
 ##### Return
 
@@ -178,10 +178,10 @@ Creates a mock instance of `Database.MergeResult` - a mock is used instead of an
 
 ##### Parameters
 
-| Param       | Description                                                     |
-| ----------- | --------------------------------------------------------------- |
-| `isSuccess` | Indicates if the generated mock should have `isSuccess` == true |
-| `recordId`  | The record ID to use within the mock result                     |
+| Param       | Description                                             |
+| ----------- | ------------------------------------------------------- |
+| `isSuccess` | Indicates if the generated mock should have `isSuccess` |
+| `recordId`  | The record ID to use within the mock result             |
 
 ##### Return
 
@@ -199,9 +199,9 @@ Creates a mock instance of `Database.SaveResult` - a mock is used instead of an
 
 ##### Parameters
 
-| Param       | Description                                                     |
-| ----------- | --------------------------------------------------------------- |
-| `isSuccess` | Indicates if the generated mock should have `isSuccess` == true |
+| Param       | Description                                             |
+| ----------- | ------------------------------------------------------- |
+| `isSuccess` | Indicates if the generated mock should have `isSuccess` |
 
 ##### Return
 
@@ -219,10 +219,10 @@ Creates a mock instance of `Database.SaveResult` - a mock is used instead of an
 
 ##### Parameters
 
-| Param       | Description                                                     |
-| ----------- | --------------------------------------------------------------- |
-| `isSuccess` | Indicates if the generated mock should have `isSuccess` == true |
-| `recordId`  | The record ID to use within the mock result                     |
+| Param       | Description                                             |
+| ----------- | ------------------------------------------------------- |
+| `isSuccess` | Indicates if the generated mock should have `isSuccess` |
+| `recordId`  | The record ID to use within the mock result             |
 
 ##### Return
 
@@ -240,9 +240,9 @@ Creates a mock instance of `Database.UndeleteResult` - a mock is used instead of
 
 ##### Parameters
 
-| Param       | Description                                                     |
-| ----------- | --------------------------------------------------------------- |
-| `isSuccess` | Indicates if the generated mock should have `isSuccess` == true |
+| Param       | Description                                             |
+| ----------- | ------------------------------------------------------- |
+| `isSuccess` | Indicates if the generated mock should have `isSuccess` |
 
 ##### Return
 
@@ -260,10 +260,10 @@ Creates a mock instance of `Database.UndeleteResult` - a mock is used instead of
 
 ##### Parameters
 
-| Param       | Description                                                     |
-| ----------- | --------------------------------------------------------------- |
-| `isSuccess` | Indicates if the generated mock should have `isSuccess` == true |
-| `recordId`  | The record ID to use within the mock result                     |
+| Param       | Description                                             |
+| ----------- | ------------------------------------------------------- |
+| `isSuccess` | Indicates if the generated mock should have `isSuccess` |
+| `recordId`  | The record ID to use within the mock result             |
 
 ##### Return
 
@@ -281,10 +281,10 @@ Creates a mock instance of `Database.UpsertResult` - a mock is used instead of a
 
 ##### Parameters
 
-| Param       | Description                                                     |
-| ----------- | --------------------------------------------------------------- |
-| `isSuccess` | Indicates if the generated mock should have `isSuccess` == true |
-| `isCreated` | Indicates if the generated mock should have `isCreated` == true |
+| Param       | Description                                             |
+| ----------- | ------------------------------------------------------- |
+| `isSuccess` | Indicates if the generated mock should have `isSuccess` |
+| `isCreated` | Indicates if the generated mock should have `isCreated` |
 
 ##### Return
 
@@ -302,11 +302,11 @@ Creates a mock instance of `Database.UpsertResult` - a mock is used instead of a
 
 ##### Parameters
 
-| Param       | Description                                                     |
-| ----------- | --------------------------------------------------------------- |
-| `isSuccess` | Indicates if the generated mock should have `isSuccess` == true |
-| `isCreated` | Indicates if the generated mock should have `isCreated` == true |
-| `recordId`  | The record ID to use within the mock result                     |
+| Param       | Description                                             |
+| ----------- | ------------------------------------------------------- |
+| `isSuccess` | Indicates if the generated mock should have `isSuccess` |
+| `isCreated` | Indicates if the generated mock should have `isCreated` |
+| `recordId`  | The record ID to use within the mock result             |
 
 ##### Return
 

docs/apex/Test-Utilities/LoggerTestConfigurator.md

@@ -148,11 +148,11 @@ Loads the mock `LogStatus_t` during test execution
 
 #### `setupMockSObjectHandlerConfigurations()` → `void`
 
-Creates mock instances of `LoggerSObjectHandler_t` for each `SObjectType` used by Nebula Logger, with each `LoggerSObjectHandler_t` automatically set to `IsEnabled__c == true`
+Creates mock instances of `LoggerSObjectHandler_t` for each `SObjectType` used by Nebula Logger, with `IsEnabled__c` set to `true` on each `LoggerSObjectHandler_t` record
 
 #### `setupMockSObjectHandlerConfigurations(Boolean isEnabled)` → `void`
 
-Creates mock instances of `LoggerSObjectHandler_t` for each `SObjectType` used by Nebula Logger, with each `LoggerSObjectHandler_t` enabled/disabled based on the provided boolean
+Creates mock instances of `LoggerSObjectHandler_t` for each `SObjectType` used by Nebula Logger, with `IsEnabled__c` set based on the provided boolean on each `LoggerSObjectHandler_t` record
 
 ##### Parameters
 

nebula-logger/core/main/configuration/classes/LoggerCache.cls

@@ -220,7 +220,7 @@ public without sharing class LoggerCache {
         public void put(String key, Object value) {
             this.transactionCache.put(key, value);
 
-            if (LoggerParameter.USE_PLATFORM_CACHE == true && this.cachePartitionDelegate.isAvailable() == true) {
+            if (LoggerParameter.USE_PLATFORM_CACHE && this.cachePartitionDelegate.isAvailable()) {
                 // Platform cache does not support storing null values, so a predefined value is used as a substitute
                 if (value == null) {
                     value = PLATFORM_CACHE_NULL_VALUE;
@@ -232,7 +232,7 @@ public without sharing class LoggerCache {
         public void remove(String key) {
             this.transactionCache.remove(key);
 
-            if (LoggerParameter.USE_PLATFORM_CACHE == true && this.cachePartitionDelegate.isAvailable() == true) {
+            if (LoggerParameter.USE_PLATFORM_CACHE && this.cachePartitionDelegate.isAvailable()) {
                 this.cachePartitionDelegate.remove(key);
             }
         }

nebula-logger/core/main/configuration/classes/LoggerParameter.cls

@@ -647,7 +647,7 @@ public class LoggerParameter {
     public static List<LoggerParameter__mdt> matchOnPrefix(String developerNamePrefix) {
         List<LoggerParameter__mdt> matchingParameters = new List<LoggerParameter__mdt>();
         for (String parameterDeveloperName : DEVELOPER_NAME_TO_RECORD.keySet()) {
-            if (parameterDeveloperName.startsWith(developerNamePrefix) == true) {
+            if (parameterDeveloperName.startsWith(developerNamePrefix)) {
                 matchingParameters.add(DEVELOPER_NAME_TO_RECORD.get(parameterDeveloperName));
             }
         }
@@ -656,13 +656,13 @@ public class LoggerParameter {
 
     private static Map<String, LoggerParameter__mdt> loadRecords() {
         Map<String, LoggerParameter__mdt> parameters = LoggerParameter__mdt.getAll().clone();
-        if (System.Test.isRunningTest() == true) {
+        if (System.Test.isRunningTest()) {
             // Keep a copy of any records that *should* be loaded during tests
             // Currently, only the record `SystemDebugMessageFormat` has a use case for this functionality,
             // but others can be easily added if other use cases are found
             Map<String, LoggerParameter__mdt> parametersToLoadDuringTests = new Map<String, LoggerParameter__mdt>();
             for (String testContextParameterName : PARAMETERS_TO_LOAD_DURING_TESTS) {
-                if (parameters.containsKey(testContextParameterName) == true) {
+                if (parameters.containsKey(testContextParameterName)) {
                     parametersToLoadDuringTests.put(testContextParameterName, parameters.get(testContextParameterName));
                 }
             }
@@ -674,7 +674,7 @@ public class LoggerParameter {
 
     @TestVisible
     private static void setMock(LoggerParameter__mdt parameter) {
-        if (String.isBlank(parameter.DeveloperName) == true) {
+        if (String.isBlank(parameter.DeveloperName)) {
             throw new System.IllegalArgumentException('DeveloperName is required on `LoggerParameter__mdt: \n' + JSON.serializePretty(parameter));
         }
 

nebula-logger/core/main/configuration/classes/LoggerPlugin.cls

@@ -30,7 +30,7 @@ public without sharing class LoggerPlugin {
     }
 
     /**
-     * @description Returns all enabled `LoggerPlugin__mdt` records, based on `IsEnabled__c == true`
+     * @description Returns all enabled `LoggerPlugin__mdt` records (where `IsEnabled__c` is set to `true`)
      * @return      The list of enabled `LoggerPlugin__mdt` records
      */
     public static List<LoggerPlugin__mdt> getPluginConfigurations() {
@@ -54,7 +54,7 @@ public without sharing class LoggerPlugin {
                     break;
                 }
             }
-            if (matchesFilterFields == true) {
+            if (matchesFilterFields) {
                 matchingPluginConfigurationSorters.add(
                     new PluginConfigurationSorter(pluginConfiguration).sortBy(sortByField).sortBy(Schema.LoggerPlugin__mdt.DeveloperName)
                 );
@@ -105,22 +105,22 @@ public without sharing class LoggerPlugin {
     private static Map<String, LoggerPlugin__mdt> loadEnabledRecords() {
         Map<String, LoggerPlugin__mdt> pluginDeveloperNameToConfiguration = new Map<String, LoggerPlugin__mdt>();
         for (LoggerPlugin__mdt pluginConfiguration : LoggerPlugin__mdt.getAll().values()) {
-            if (pluginConfiguration.IsEnabled__c == true) {
+            if (pluginConfiguration.IsEnabled__c) {
                 pluginDeveloperNameToConfiguration.put(pluginConfiguration.DeveloperName, pluginConfiguration);
             }
         }
-        if (System.Test.isRunningTest() == true) {
+        if (System.Test.isRunningTest()) {
             pluginDeveloperNameToConfiguration.clear();
         }
         return pluginDeveloperNameToConfiguration;
     }
 
     @TestVisible
     private static void setMock(LoggerPlugin__mdt pluginConfiguration) {
-        if (String.isBlank(pluginConfiguration.DeveloperName) == true) {
+        if (String.isBlank(pluginConfiguration.DeveloperName)) {
             throw new System.IllegalArgumentException('DeveloperName is required on mock LoggerPlugin__mdt: \n' + JSON.serializePretty(pluginConfiguration));
         }
-        if (pluginConfiguration.IsEnabled__c == true) {
+        if (pluginConfiguration.IsEnabled__c) {
             DEVELOPER_NAME_TO_RECORD.put(pluginConfiguration.DeveloperName, pluginConfiguration);
         }
     }
@@ -154,7 +154,7 @@ public without sharing class LoggerPlugin {
                 // Now, the actual comparisons
                 if (this.pluginConfiguration.get(field) == that.pluginConfiguration.get(field)) {
                     continue;
-                } else if (this.pluginConfiguration.get(field) == null && that.pluginConfiguration.get(field) != null || thisIsGreaterThanThat == true) {
+                } else if (this.pluginConfiguration.get(field) == null && that.pluginConfiguration.get(field) != null || thisIsGreaterThanThat) {
                     return 1;
                 } else {
                     return -1;

nebula-logger/core/main/configuration/classes/LoggerScenarioRule.cls

@@ -33,24 +33,24 @@ public without sharing class LoggerScenarioRule {
 
     @TestVisible
     private static void setMock(LoggerScenarioRule__mdt scenarioRule) {
-        if (String.isBlank(scenarioRule.Scenario__c) == true) {
+        if (String.isBlank(scenarioRule.Scenario__c)) {
             throw new System.IllegalArgumentException('Scenario__c is required on `LoggerScenarioRule__mdt: \n' + JSON.serializePretty(scenarioRule));
         }
 
-        if (isValid(scenarioRule) == true) {
+        if (isValid(scenarioRule)) {
             SCENARIO_NAME_TO_SCENARIO_RULE.put(scenarioRule.Scenario__c, scenarioRule);
         }
     }
 
     private static Map<String, LoggerScenarioRule__mdt> loadLogScenarioRules() {
         Map<String, LoggerScenarioRule__mdt> scenarioRules = new Map<String, LoggerScenarioRule__mdt>();
         for (LoggerScenarioRule__mdt scenarioRule : LoggerScenarioRule__mdt.getAll().values()) {
-            if (isValid(scenarioRule) == true) {
+            if (isValid(scenarioRule)) {
                 scenarioRules.put(scenarioRule.Scenario__c, scenarioRule);
             }
         }
 
-        if (System.Test.isRunningTest() == true) {
+        if (System.Test.isRunningTest()) {
             scenarioRules.clear();
         }
 
@@ -59,12 +59,12 @@ public without sharing class LoggerScenarioRule {
 
     private static Boolean isValid(LoggerScenarioRule__mdt scenarioRule) {
         Boolean isValid = false;
-        if (scenarioRule.IsEnabled__c == true) {
+        if (scenarioRule.IsEnabled__c) {
             Datetime currentTime = System.now();
             Boolean startTimeIsValid = scenarioRule.StartTime__c == null || scenarioRule.StartTime__c <= currentTime;
             Boolean endTimeIsValid = scenarioRule.EndTime__c == null || scenarioRule.EndTime__c >= currentTime;
 
-            isValid = startTimeIsValid == true && endTimeIsValid == true;
+            isValid = startTimeIsValid && endTimeIsValid;
         }
         return isValid;
     }

nebula-logger/core/main/log-management/classes/LogBatchPurgeController.cls

@@ -23,8 +23,7 @@ public with sharing class LogBatchPurgeController {
      */
     @AuraEnabled
     public static Boolean canUserRunLogBatchPurger() {
-        return Schema.Log__c.SObjectType.getDescribe().isDeletable() == true ||
-            System.FeatureManagement.checkPermission(CAN_EXECUTE_LOG_BATCH_PURGER_PERMISSION) == true;
+        return Schema.Log__c.SObjectType.getDescribe().isDeletable() || System.FeatureManagement.checkPermission(CAN_EXECUTE_LOG_BATCH_PURGER_PERMISSION);
     }
 
     /**