trigger tocOportunityAfterUpdate on Opportunity (after update) {
    List<Opportunity> allids = new List<Opportunity>();
    List<Opportunity> toUpdate = new List<Opportunity>();
    List<tfCommissionOpportunity__c> toComUpdate = new List<tfCommissionOpportunity__c>();
    try {

        /*This for loop was redesigned to minimize the number of DML calls(select)
        Collecting the Opportunity objects that require updating in a List<Opportunity> and then doing a single update at the end of the for loop, 
        passing it the List<Opportunity> object.        */

        for (Opportunity o : Trigger.new) {

             Opportunity existingOpportunity= Trigger.oldMap.get(o.Id);
             allids.add(existingOpportunity);
        }

        for (Integer i = 0; i < Trigger.new.size(); i++) {
            if (Trigger.new[i].Billing_Report_Date__c != null && (Trigger.old[i].Billing_Report_Date__c == null || Trigger.old[i].Billing_Report_Date__c != Trigger.new[i].Billing_Report_Date__c)) {
                
   /* The select statement was inside the for loop and that is executing more than 20 times (the limit on SOQL queries within Apex triggers). 
                To remedy that, used an IN clause to compare multiple IDs at once, rather than using = in the SOQL WHERE clause.
                */
  Opportunity[] oppList = [Select o.tocClonedFrom__c, o.RecordType.Name, o.Id, o.Billing_Report_Date__c, (Select Id, Billing_Report_Date__c From Opportunities__r) From Opportunity o where o.tocClonedFrom__c in :allids or o.Id in :allids];
    
                for (Opportunity op : oppList) {
                    Opportunity opportunity = Trigger.oldMap.get(op.Id);
                    String id = opportunity.id;
                    if(op.RecordType.Name == 'Payment' && op.Id != id ) 
                    {
          Opportunity updOpp = new Opportunity(Id=op.Id);
                        updOpp.Billing_Report_Date__c  = Trigger.new[i].Billing_Report_Date__c;

                        updOpp.tfomsBillingCompleteDate__c = Trigger.new[i].tfomsBillingCompleteDate__c;
                        updOpp.tocBillingEffectiveDate__c = Trigger.new[i].tocBillingEffectiveDate__c;
                        toUpdate.add(updOpp);
                    }
                    if((op.tocClonedFrom__c == id && op.RecordType.Name == 'Payment') || op.Id == id )
                    {
                        tfCommissionOpportunity__c[] commOppsList = op.Opportunities__r;
                            for(tfCommissionOpportunity__c commOpp : commOppsList){
                            tfCommissionOpportunity__c updCommOpp = new tfCommissionOpportunity__c(Id=commOpp.Id);
                            updCommOpp.Billing_Report_Date__c = Trigger.new[i].Billing_Report_Date__c;
                        }
                    }
                }
                //Doing the bulk updates after the for loop due to the limit on SOQL queries within Apex triggers. 
  update toUpdate;
                update toComUpdate;
            }
            
        }
    } catch (Exception e){
        
    }
}