I added a new field on Opportunity table to keep a count of records, as well as track the updates made on that record. This is needed for an external system we use. Opportinity table already has around 5000 records now. I would like to update the new field that I added with some sequence order initially. Say each record to have a unique sequence number like 1,2,3 etc(can't use the Id field for unique). The field "Identifier__c" was added for this purpose with data type number and unique contraint. For that I created a temporary class and trigger. But it is throwing the error.
Review all error messages below to correct your data. Apex trigger OpportunitySAPIdentifierHandler caused an unexpected exception,
contact your administrator: OpportunitySAPIdentifierHandler: execution
of BeforeUpdate caused by: System.DmlException: Update failed. First
exception on row 0 with id 006E00000038FPSIA2; first error:
CANNOT_INSERT_UPDATE_ACTIVATE_ENTITY, OpportunitySAPIdentifierHandler:
execution of BeforeUpdate caused by: System.DmlException: Update
failed. First exception on row 0 with id 006E00000038FPTIA2; first
error: CANNOT_INSERT_UPDATE_ACTIVATE_ENTITY,
OpportunitySAPIdentifierHandler: execution of BeforeUpdate caused by:
System.DmlException: Update failed. First exception on row 0 with id
006E00000038FPSIA2; first error: SELF_REFERENCE_FROM_TRIGGER, Object
(id = 006E00000038FPS) is currently in trigger
OpportunitySAPIdentifierHandler, therefore it cannot recursively
update itself: [] Class.AutoNumber.OpportunityAutoNumber: line 76,
column 1 Trigger.OpportunitySAPIdentifierHandler: line 4, column 1: []
Class.AutoNumber.OpportunityAutoNumber: line 76, column 1
Trigger.OpportunitySAPIdentifierHandler: line 4, column 1: []:
Class.AutoNumber.OpportunityAutoNumber: line 76, column 1
Below is the trigger and class used.
trigger Opportuniwith sIdentifierHandler on Opportunity (before insert,before update) {
If(Trigger.isInsert || Trigger.isUpdate){
AutoNumber.OpportunityAutoNumber(Trigger.new);
}
}
The class is
public with sharing class AutoNumber {
public static void OpportunityAutoNumber (List<Opportunity> Opp) {
if(Trigger.isUpdate || Trigger.isInsert) {
for(Opportunity p: Opp) {
List<Opportunity> oppList = new List<Opportunity>();
oppList = [SELECT Id, createdDate, Identifier__c FROM Opportunity where Identifier__c = null and Id != :p.Id limit 100 FOR update];
for (Opportunity oppL: oppList) {
if(oppL.Identifier__c == null) {
List<Opportunity> OpportunityList = new List<Opportunity>([select Identifier__c from Opportunity where Identifier__c != null order by Identifier__c desc limit 1]);
if (!OpportunityList.isEmpty()) {
Decimal maxval = OpportunityList.get(0).Identifier__c;
oppL.Identifier__c = maxval + 1;
update oppL;
}
}
// integer currentCounter = Integer.valueOf(opp.Identifier__c);
// oppr.Identifier__c = currentCounter + 1;
// update oppr;
}
}
}
}
}
Best Answer
To prevent a recursive call, you should make sure your trigger only executes one time. Add a class with a static boolean variable. In the trigger, have a condition that checks the value of the boolean. Once the trigger executes, change the value to false.
****Utility class****