Trigger is not getting Fired on After Insert

Question

I have a Custom Object as TUC object .When i create a record in TUC obj , it should automatically create a record in Event and Task.But the trigger written is not getting fired .Any help is very much appreciated.

Here is the Code :

trigger CreateEvent on TUC__c(after insert){
   List<Event> eventList = new List<Event>();
   for(TUC__c tucObj : Trigger.new){
      Event event = new Event();
      event.whatid  = tucObj.Account_Related__c;  
      event.Subject = tucObj.subject__c;
      event.OwnerId = tucObj.CreatedById;
      event.StartDateTime = tucObj.Start_Date__c;
      event.EndDateTime=tucObj.End_Date__c;
      eventList.add(event);
   }
   if(eventList.size()>0 && eventlist!==null){
      insert eventList;
   }
}

Answer 1

As other's have pointed out, you need to check for NULL before checking size b/c you'll get an NPE when you check size (so the if condition would be: if (eventList != null && eventList.size() > 0). Salesforce does short-circuiting on boolean evaluation so it exists immediately if eventList is null.

Having said that, you don't actually need to check for null or empty - just do the insert and if the list is empty, nothing happens, and it will never be null b/c you initialized it in the 2nd line.

Answer 2

Your final if condition is the problem. AFAIK actually you no need this check point when you are performing a DML since salesforce is handling it for us. Unfortunately I can't find where I read this (found it from this forum). But you can surround the insert statement with a try catch block as a good practice. So just remove your condition and retry the code as below.

trigger CreateEvent on TUC__c(after insert){
   List<Event> eventList = new List<Event>();
   for(TUC__c tucObj : Trigger.new){
      Event event = new Event();
      event.whatid  = tucObj.Account_Related__c;  
      event.Subject = tucObj.subject__c;
      event.OwnerId = tucObj.CreatedById;
      event.StartDateTime = tucObj.Start_Date__c;
      event.EndDateTime=tucObj.End_Date__c;
      eventList.add(event);
   }
   try{
      insert eventList;
   }
   catch(DmlException e){
       //your exception handling here
    }
}

Answer 3

When you are Checking list is empty or not use isEmpty() method instead of using null.....

Retry the code as below.....

trigger CreateEvent on TUC__c(after insert){
   List<Event> eventList = new List<Event>();
   for(TUC__c tucObj : Trigger.new){
      Event event = new Event();
      event.whatid  = tucObj.Account_Related__c;  
      event.Subject = tucObj.subject__c;
      event.OwnerId = tucObj.CreatedById;
      event.StartDateTime = tucObj.Start_Date__c;
      event.EndDateTime=tucObj.End_Date__c;
      eventList.add(event);
   }
   if(eventList.size()>0 && !eventlist.isEmpty()){
      insert eventList;
   }
}

Answer 4

I think this trigger will solve the problem for you.

 trigger CreateEvent on TUC__c(after insert){
  List<Event> eventList = new List<Event>();
  for(TUC__c tucObj : Trigger.new){
   Event event = new Event();
   event.whatid  = tucObj.Account_Related__c;  
   event.Subject = tucObj.subject__c;
   event.OwnerId = tucObj.CreatedById;
   event.StartDateTime = tucObj.Start_Date__c;
   event.EndDateTime=tucObj.End_Date__c;
   eventList.add(event);
  }
   if(!eventList.isEmpty()){
    insert eventList;
   }
 }

Answer 5

Please try below code.

      trigger CreateEvent on TUC__c(after insert){
      List<Event> eventList = new List<Event>();
      for(TUC__c tucObj : Trigger.new){
      Event event = new Event();
      event.whatid  = tucObj.Account_Related__c;  
      event.Subject = tucObj.subject__c;
      event.OwnerId = tucObj.CreatedById;
      event.StartDateTime = tucObj.Start_Date__c;
      event.EndDateTime=tucObj.End_Date__c;
      eventList.add(event);
      }
      if(eventList.size()>0){
      insert eventList;
       }
       }