function readOnly(count){ }
Sign Up ›
Starting November 20, the site will be set to read-only. On December 4, 2023,
forum discussions will move to the Trailblazer Community.
Learn More

Browse by Topic
ShowAll Questionssorted byDate Posted
+ Start a Discussion
davidCe17icdavidCe17ic 

Trying to add condition to uncheck checkbox when query returns false

This is probably going to be something really simple but I cannot get this to work. I have a piece of code which is checking a checkbox on the case object when a possible duplicate is submitted to SFDC. It does everything it is supposed to however I want to add a condition so when I close out the possible duplicate cases, the query should evaluate to false and the checkbox should be unchecked. Any ideas?

 

Here is the code so far: 

 

trigger CaseDupeTrigger on Case (before insert, before update)
{
List<Case> c = new List<Case>();
if (Trigger.isInsert)
{
for (Case caseId : Trigger.new){

c = [select id,suppliedemail,isclosed from case where suppliedemail =: caseId.suppliedemail and isclosed=false];

if (c.size() > 0) {
for(Case caseIdOld : c)
caseIdOld.Potential_Duplicate_Case__c=true;
}
}
}
else if (Trigger.isUpdate)
{
for (Case caseId : Trigger.new)
{
Case oldcaseId = Trigger.oldMap.get(caseId.id);
if (caseId.suppliedemail != oldcaseId.suppliedemail)
{
c = [select id, suppliedemail, isclosed from case where suppliedemail =: caseId.suppliedemail and isclosed=false];

if (c.size() > 0)
{
for(Case caseIdOld : c)
caseIdOld.Potential_Duplicate_Case__c=true;
}

}
}
}

if(c.size() > 0)
update c;

}

asish1989asish1989
trigger CaseDupeTrigger on Case (before insert, before update) 
{
	List<Case> c = new List<Case>();
	if (Trigger.isInsert)
	{
		for (Case caseId : Trigger.new){

			c = [select id,suppliedemail,isclosed from case where suppliedemail =: caseId.suppliedemail and isclosed=false];

			if (c.size() > 0) {
				for(Case caseIdOld : c) 
					caseIdOld.Potential_Duplicate_Case__c=true;
			} 
		}
	}
	else if (Trigger.isUpdate)
	{
		for (Case caseId : Trigger.new)
		{
			
			Case oldcaseId = Trigger.oldMap.get(caseId.id);
			if (caseId.suppliedemail != oldcaseId.suppliedemail)
			{
				c = [select id, suppliedemail, isclosed from case where suppliedemail =: caseId.suppliedemail and isclosed=false];

				if (c.size() > 0) 
				{
					for(Case caseIdOld : c)
						caseIdOld.Potential_Duplicate_Case__c=true;
						
				}

			}
			
			If(caseId.Status == 'Closed'){
				for(Case caseIdOld : c)
						caseIdOld.Potential_Duplicate_Case__c=false;
			
			}
		}
	}

}

 
davidCe17icdavidCe17ic

Thanks for the suggestion Asish. 


While syntactically correct (no errors) the original trigger to check the checkbox when a dupe is found is not firing anymore. 

 

Any suggestions? 
asish1989asish1989

add System.debug(); and see trigger is fired or not, If fired and again test which block It is entering and which block It is not entering.
davidCe17icdavidCe17ic

So I got the trigger to work again, but adding the additional If statement doesn't work as desired.

 

When closing out the potential duplicate, the checkbox remains ticked on the non-duplicate.

 

If(caseId.Status == 'Closed'){ for(Case caseIdOld : c) caseIdOld.Potential_Duplicate_

Case__c=false; }