Відео

Thanks for making amazing content, keep it up bro :) Try this approach, Map<Id, String> phoneToIdMap = new Map<Id, String>(); for (Contact c : newContacts) { if (c.Phone != null && c.Phone != oldContacts.get(c.Id).Phone && c.AccountId != null) { phoneToIdMap.put(c.AccountId, c.Phone); } } if (phoneToIdMap.size() > 0) { List<Contact> conList = [SELECT Id, AccountId, Phone FROM Contact WHERE AccountId IN :phoneToIdMap.keySet()]; List<SObject> updateList = new List<SObject>(); Map<Id, Account> accMap = new Map<Id, Account>(); //we take map because if we directly insert acc in above list it gives duplicate Ids found error! for (Contact c : conList) { c.Phone = phoneToIdMap.get(c.AccountId); updateList.add(c); accMap.put( c.AccountId, new Account( Id = c.AccountId, Phone = phoneToIdMap.get(c.AccountId) ) ); } if (accMap.size() > 0) { updateList.addAll(accMap.values()); } if (updateList.size() > 0) { try { update updateList; } catch (Exception e) { System.debug('Exception occured in updating list --> '+e.getMessage()); } } }

Hi can anyone tell me why I'm not getting any changes in account object even tho I have done all the steps just like video .! 😮

Hi SFDC ninja: I have a doubt, Instead of adding to Map. Directly in line 25 we can null check accountId and create instance of Account there itself and update account. Works right ? Great work by the way, Your classes are very useful 🙏

I think we can also use account list and fetching related contacts and updating phone through map , its works for me actually in a simple way

I used a diff approach , class: public with sharing class phoneAccConsHandler { //whenever a contact phone gets updated, it's parent account phone should get updated, also all of the account's other contacts phone also should get updated with the same phone number. public static void afterUpdate(List<contact> newCons,Map<id,contact> oldConMap){ set<id> accIds=new Set<id>(); Map<id,string> accidPhoneMap=new Map<id,string>(); for(Contact con:newCons){ if(con.accountId!=null && con.Phone!=null && oldConMap==null){ accIds.add(con.accountId); accidPhoneMap.put(con.accountId,con.phone); }else if(con.accountId!=null && con.Phone!=null && oldConMap.get(con.id).phone!=con.phone || oldConMap.get(con.id).accountId!=con.accountId){ accIds.add(con.accountId); accIds.add(oldConMap.get(con.id).accountId); accidPhoneMap.put(con.accountId,con.phone); accidPhoneMap.put(oldConMap.get(con.id).accountId,con.phone); } } //update account Phone List<account> Accounts=new List<account>(); for(Account acc:[select id,phone from account where id in:accIds]){ acc.phone=accidPhoneMap.get(acc.id); Accounts.add(acc); } If(Accounts.size()!=0){ update Accounts; } //list all contact's of that account: List<contact> allConsToBeUpdated=new List<contact>(); List<contact> allCons=[Select id,accountId,Phone from Contact where accountId in:accIds]; for(Contact con:allCons){ con.phone=accidPhoneMap.get(con.accountId); allConsToBeUpdated.add(con); } if(allConsToBeUpdated.size()!=0){ update allConsToBeUpdated; } } }

pls upload the codes

@SFDC NINJA ,I saw all Ur Trigger Videos .Champion effect loved it. In this scenario lets say Account A has contacts c1,c2,c3 .where C1 phone updated to 999 and C2 phone to 888 .so Account phone and all c1,c2,c3 phone now 888. But if u update multiple contacts with apex and show the difference in video i think it adds more value and understandability for Biggers.

Thank you for sharing the solution, this is the trigger which I got in Accenture interview😊

Please can you write and show with apex handler

Why did this not go into recusrion 🤔

We need not to use a for loop in the Trigger.

good information

thank you bro for this awesome work and i have a scenario .if if are handling bulk delete an account does not have any opp but others have in this scenario else of Aggregate function will not execute .i have tried for this scenario plz have a look and guide me further trigger OppAmountSumOnAccount on Opportunity(after insert,after undelete,after delete,after update){ set<id> accountIds = new set<id>(); if(!trigger.new.isEmpty()){ if(trigger.isAfter && (trigger.isInsert || trigger.IsUndelete)){ for(opportunity opp : trigger.new){ if(opp.accountid != null){ accountIds.add(opp.accountId); } } } if(trigger.isAfter && trigger.isUpdate){ for(opportunity opp : trigger.new){ if(opp.accountid != null){ accountIds.add(opp.accountId); } if(trigger.oldMap.get(opp.id).accountId != null && (opp.accountId != trigger.oldMap.get(opp.id).accountId || opp.amount != trigger.oldMap.get(opp.id).amount)){ accountIds.add(trigger.oldMap.get(opp.id).accountId); } } } } if(!trigger.old.isEmpty()){ if(trigger.isAfter && trigger.isDelete){ for(opportunity opp : trigger.old){ if(opp.accountid != null){ accountIds.add(opp.accountId); } } } } map<id,account> accMap = new map<id,account>(); if(!accountIds.isEmpty()){ list<AggregateResult> agrList = [select accountId ids,sum(amount) sumAmnt from opportunity where accountId IN : accountIds group by AccountId]; if(!agrList.isEmpty()){ for(AggregateResult agr: agrList){ account acc = new account(); acc.id = (id)agr.get('ids'); acc.totalOppAmount__c = (Decimal)agr.get('sumAmnt'); accMap.put(acc.id,acc); if(accountids.contains(acc.id)){ accountids.remove(acc.id); } } } //here we are handling account which doesnot have any associated opportunity after the action for(id accid:accountids){ account acc1 = new Account(); acc1.id =accid; acc1.totalOppAmount__c = 0; accMap.put(acc1.id,acc1); } } if(!accMap.isEmpty()){ update accMap.values(); } }

Hi, I guess there is an issue in this snippet, In the second for loop you were iterating the trigger.new again but the accMap was created for only the records where the related account Id is not null and description should be updated. So, if a contact record is updated without any change on the description, we will face an issue in line 26 as KeyNotExists I guess. However we can overcome this by adding a containsKey() in line 26. Thanks!

want more iterview videos like this for different topics please

public static void updtAccDescriptionWithCon(List<Contact> newList , Map<Id,Contact> oldMap) { List<Account> AccLsts = new List<Account>(); if(!newList.isEmpty()){ for(Contact con : newList){ //if Update event if(oldMap !=null && con.AccountId !=null && (oldMap.get(con.Id).Description != con.Description)){ AccLsts.add(new Account(Id=con.AccountId ,Description =con.Description)); } } } if(!AccLsts.isEmpty()){ update AccLsts; } } //Please correct me if i was wrong.

Can you take this one in your next video. write a trigger to get the product count of opportunity and update count field on account

Only for Insert Operation----> public with sharing class OpportunityAmountTriggerHelper { public static void getOppAmount(List<Opportunity> lstNewOpp) { set<id> accIds=new set<id>(); if(!lstNewOpp.isEmpty()) { for(Opportunity opp:lstNewOpp) { if(opp.AccountId != null && opp.Amount != null) { accIds.add(opp.AccountId); } } } if(!accIds.isEmpty()) { List<Account> accToUpdate=new List<Account>(); map<id, Opportunity> mapOpp=new map<id, Opportunity>(); List<Opportunity> getAcc=[select Accountid, Amount from Opportunity where Accountid in: accIds ORDER BY Amount desc limit 1 OFFSET 1]; if(!getAcc.isEmpty()) { for(Opportunity opt:getAcc) { mapOpp.put(opt.Accountid, opt); } } for(Id ids:accIds) { if(mapOpp.containskey(ids)) { account accnt=new account(); accnt.id=ids; accnt.AnnualRevenue=mapOpp.get(ids).amount; accToUpdate.add(accnt); } } if(!accToUpdate.isEmpty()) { update accToUpdate; } } } }

Thank you

Thankyou Sir !!!!

Please can you make vedio on custom validation in LWC in detailed way

hi

Please can you make vedio on custom validation with full clarity in lwc component like howmany way are availble in lwc for custom validation

Nicely explained

Something I would be doing happily with validation rule/record trigger flow.

What is reason we can't call future from another future method??

Sir i am facing issue in a trigger "trigger will fire when we will update opportunity to closed won and it will create order and order item as that are in opportunity line item " when i am using try and catch block it is working fine but when i am not using i am not getting expected result please help trigger CreateOrderOnOpportunityWon on Opportunity (after update) { List<Order> ordersToInsert = new List<Order>(); List<OrderItem> orderItemsToInsert = new List<OrderItem>(); Set<Id> opportunityIds = new Set<Id>(); for (Opportunity opp : Trigger.new) { if (opp.IsWon && Trigger.oldMap.get(opp.Id).StageName != 'Closed Won') { System.debug('Opportunity moved to Closed Won: ' + opp.Id); opportunityIds.add(opp.Id); } } System.debug('Opportunity Ids moved to Closed Won: ' + opportunityIds); if (!opportunityIds.isEmpty()) { try { List<OpportunityLineItem> opportunityLineItems = [SELECT OpportunityId, Product2Id, Quantity, UnitPrice, PricebookEntryId FROM OpportunityLineItem WHERE OpportunityId IN :opportunityIds]; System.debug('Queried OpportunityLineItems: ' + opportunityLineItems); List<Opportunity> opportunities = [SELECT Id, AccountId, ContractId, Pricebook2Id FROM Opportunity WHERE Id IN :opportunityIds]; System.debug('Queried Opportunities: ' + opportunities); Map<Id, Order> oppOrderMap = new Map<Id, Order>(); for (Opportunity opp : opportunities) { Order newOrder = new Order(); newOrder.AccountId = opp.AccountId; newOrder.ContractId = opp.ContractId; newOrder.EffectiveDate = Date.today(); newOrder.Status = 'Draft'; newOrder.Pricebook2Id = opp.Pricebook2Id; ordersToInsert.add(newOrder); oppOrderMap.put(opp.Id, newOrder); System.debug('Prepared Order to insert: ' + newOrder); } if (!ordersToInsert.isEmpty()) { try { insert ordersToInsert; System.debug('Orders inserted successfully: ' + ordersToInsert); for (OpportunityLineItem oli : opportunityLineItems) { Order newOrder = oppOrderMap.get(oli.OpportunityId); if (newOrder != null) { OrderItem newOrderItem = new OrderItem(); newOrderItem.Product2Id = oli.Product2Id; newOrderItem.Quantity = oli.Quantity; newOrderItem.UnitPrice = oli.UnitPrice; newOrderItem.PricebookEntryId = oli.PricebookEntryId; // Add PricebookEntryId newOrderItem.OrderId = newOrder.Id; orderItemsToInsert.add(newOrderItem); System.debug('Prepared OrderItem to insert: ' + newOrderItem); } else { System.debug('No matching order found for OpportunityLineItem: ' + oli); } } if (!orderItemsToInsert.isEmpty()) { try { insert orderItemsToInsert; System.debug('OrderItems inserted successfully: ' + orderItemsToInsert); } catch (DmlException dme) { System.debug('Error inserting OrderItems: ' + dme.getMessage()); System.debug('DML Error Details: ' + dme.getDmlMessage(0)); System.debug('DML Error Status Code: ' + dme.getDmlStatusCode(0)); System.debug('DML Error Fields: ' + dme.getDmlFields(0)); } } else { System.debug('No OrderItems to insert'); } } catch (DmlException dme) { System.debug('Error inserting Orders: ' + dme.getMessage()); System.debug('DML Error Details: ' + dme.getDmlMessage(0)); System.debug('DML Error Status Code: ' + dme.getDmlStatusCode(0)); System.debug('DML Error Fields: ' + dme.getDmlFields(0)); } } else { System.debug('No Orders to insert'); } } catch (Exception e) { System.debug('Error in trigger execution: ' + e.getMessage()); System.debug('Exception Stack Trace: ' + e.getStackTraceString()); } } else { System.debug('No Opportunities moved to Closed Won'); } }

This is not working for me, getting recurssive DML exception. Can you help me

why am I getting error whenever I insert a new record with phone number?

I have used the sort method which will also help in bulk process , here is my code (done for only after insert) : public with sharing class accSecondOppHolder { //scenario: update account description field with the second highest related amount opportunity details. public static void afterOperation(List<opportunity> newOpps,map<id,opportunity> oldOppMap){ set<id> accId=new Set<id>(); for(Opportunity opp:newOpps){ if(opp.accountid!=null){ accId.add(opp.accountId); } } List<opportunity> allOpps=[Select id,stageName,accountId, amount from opportunity where accountId in:accId]; Map<id,List<decimal>> accidOppMap=new Map<id,List<decimal>>(); for(Opportunity opp:allOpps){ If(opp.accountId!=null && opp.amount!=null){ if(!accidOppMap.containsKey(opp.accountId)){ accidOppMap.put(opp.accountId,new List<decimal>{opp.amount}); }else{ accidOppMap.get(opp.accountId).add(opp.amount); } } } List<account> allAcc=new List<account>(); for(Id ids:accId){ Account acc=new Account(); if(accidOppMap.containsKey(ids) && accidOppMap.get(ids).size()!=0){ acc.id=ids; List<decimal> amountList= accidOppMap.get(ids); amountList.sort(); acc.Description='Second higest opportunity amount is: '+ amountList[amountList.size()-2]; allAcc.add(acc); }else{ acc.id=ids; acc.Description='No related opportunity exist'; allAcc.add(acc); } } if(allAcc.size()!=0){ update allAcc; } } }

Thank you for your video. I have a question , it is mentioned that we can process max 200 records at a time in execute method. Then how we are calculating how many times the execute method will be executed to process the 1236 records by specifying batch size as 400. Anyone please clear this doubt.

at what level was this question asked for?

Trigger - trigger Accdesc on Contact (after insert, after update, after delete, after undelete) { if (trigger.isAfter && trigger.isInsert) { UpdateAccDec.DMLConInsertion(trigger.new); } if (trigger.isAfter && trigger.isUpdate) { UpdateAccDec.DMLConUpdate(trigger.new, trigger.oldMap); System.debug('Checkpoint -1'); } if (trigger.isAfter && trigger.isDelete) { UpdateAccDec.DMLDelete(trigger.old); } if (trigger.isAfter && trigger.isUndelete) { UpdateAccDec.DMLUndelete(trigger.new); } } Handler Class - public class UpdateAccDec { public static void DMLConInsertion(List<Contact> conlist) { Set<Id> accIds = new Set<Id>(); List<Account> accList = new List<Account>(); System.debug('Checkpoint-2'); for (Contact c : conlist) { if (c.AccountId != null) { accIds.add(c.AccountId); } } System.debug('Accountids' + accIds); accList = [SELECT Id, Name, Description, (SELECT Id, Name, CreatedDate FROM Contacts) FROM Account WHERE Id IN :accIds]; for (Account a : accList) { String accDes = ''; for (Contact c : a.Contacts) { accDes += c.Name + ' ' + c.CreatedDate.format() + ' '; } if (accDes.endsWith(', ')) { accDes = accDes.removeEnd(', '); } a.Description = accDes; } if (!accList.isEmpty()) { update accList; } System.debug('Accounts' + accList); } public static void DMLConUpdate(List<Contact> conlist, Map<Id, Contact> oldMap) { Set<Id> oldAccIds = new Set<Id>(); Set<Id> newAccIds = new Set<Id>(); List<Contact> oldConList = new List<Contact>(); List<Contact> newConList = new List<Contact>(); System.debug('Checkpoint -2'); for (Contact c : conlist) { System.debug('Contact ids' + c.Id); Contact oldCon = oldMap.get(c.Id); System.debug('Oldcontactid' + oldCon.Id); if (c.AccountId != oldCon.AccountId) { oldAccIds.add(oldCon.AccountId); newAccIds.add(c.AccountId); } else if (c.AccountId != null) { newAccIds.add(c.AccountId); System.debug('New Accountid' + c.AccountId); } } System.debug('Old Accountids' + oldAccIds); System.debug('New Accountids' + newAccIds); oldConList = [SELECT Id, Name, CreatedDate, AccountId FROM Contact WHERE AccountId IN :oldAccIds]; newConList = [SELECT Id, Name, CreatedDate, AccountId FROM Contact WHERE AccountId IN :newAccIds]; System.debug('Old contact list' + oldConList); System.debug('New contact list' + newConList); if (!oldConList.isEmpty()) { DMLConInsertion(oldConList); } if (!newConList.isEmpty()) { DMLConInsertion(newConList); } } public static void DMLDelete(List<Contact> conlist) { if (!conlist.isEmpty()) { DMLConInsertion(conlist); // reuse the code } } public static void DMLUndelete(List<Contact> conlist) { if (!conlist.isEmpty()) { DMLConInsertion(conlist); // reuse the code } } }

can you please provide a pdf of all your trigger scenario

I think this code will not perform well in Bulikified manner. Revised version :: // EY case - esc field checked means , task will be created on primary cont of specific acc // after insert, after update public class CaseHandler{ public static void taskCreation(List<Case> newCases, Map<Id, Case> oldMapCases){ Case oldCase= new Case(); Set<Id> accIds = new Set<Id>(); Map<Id,Contact> contMap = new Map<Id, Contact>(); List<Task> tasksToUpdated = new List<Task>(); for(Case cs : newCases){ if(oldMapCases != null){ oldCase = oldMapCases.get(cs.Id) if(cs.Escalation && cs.Escalation != oldCase.Escalation && cs.AccountId != null){ accIds.add(cs.AccountId); } } else{ accIds.add(cs.AccountId); } } List<Contact> contactList = [SELECT Id, PrimaryContact FROM Contact WHERE PrimaryContact = true && AccountId IN: accIds ORDER BY CreatedDate DESC]; for(Contact con : contactList){ if(!contMap.containsKey(con.Id)){ Task tk = new Task(); tk.WhoId = con.Id; tk.Subject = 'Follow up task'; tasksToUpdated.add(tk); } } if(!tasksToUpdated.isEmpty()){ insert tasksToUpdated; } } }

your voice not clear and your explanation very hurry please explain slowly

If we use order by and offset can we decrease lines of code??

when changed the parent on contacts, and if the parent has more than 2 contacts it should throw error while chainging parent also give me code sir,

Trigger PreventMoreThanTwoContacs on Contact (before insert) { Set<Id> accountIds = new Set<Id>(); for (Contact contact : Trigger.new) { if (contact.AccountId != null) { accountIds.add(contact.AccountId); } } List<Account> accountsWithContacts = [ SELECT Id, (SELECT Id FROM Contacts) FROM Account WHERE Id IN :accountIds ]; for (Contact contact : Trigger.new) { if (contact.AccountId != null) { for (Account account : accountsWithContacts) { if (account.Id == contact.AccountId && account.Contacts.size() >= 2) { contact.addError('This Account already has two Contacts. You cannot add more.'); } } } } }

Great.. it will work for single account but its wont work for bulk records because it will store second highest amount for account getting in list. We may need to use map to store accountid and second highest amount.

Thank you very much sir

There was an issue in above trigger that I have updated the parent account of an opportunity in this scenario updated parents account total sum is updated properly but the old parent accounts total sum is showing incorrect.

I am waiting for that please try as soon as possible

Hello ninja, I send you Two trigger scenarios but now as well it will not solved

WRONG Solution, Code will not work for bulkified scenario, it's a good practise to bulkfy triggers.

which company asked this?

public class Expected { public static void display(List<Contact> conlist) { Set<Id> accids = new Set<Id>(); Map<Id, Contact> contactMap = new Map<Id, Contact>(); List<Contact> modifiedconlist = new List<Contact>(); List<Contact> latestconlist = new List<Contact>(); // Collect Account IDs from the newly inserted contacts for (Contact c : conlist) { if (c.AccountId != null) { accids.add(c.AccountId); } } // Query all existing contacts under the collected Account IDs whre new contact marked as true if (!accids.isEmpty()) { List<Contact> updateconlist = [SELECT Id, Name, New_Contact__c, AccountId FROM Contact WHERE AccountId IN :accids AND New_Contact__c = true]; System.debug('Retrive list of contacts' + updateconlist); // Disable the checkbox for all existing contacts for (Contact c : updateconlist) { c.New_Contact__c = false; modifiedconlist.add(c); contactMap.put(c.Id, c); } } System.debug('Checkpoint-2'); // Enable the checkbox for the newly inserted contacts for (Contact c : conlist) { if (c.AccountId != null) { // Create a new instance to avoid modifying read-only records Contact newContact = new Contact(Id = c.Id, New_Contact__c = true); latestconlist.add(newContact); } } System.debug('Checkpoint-3'); // Update the modified contacts if (!modifiedconlist.isEmpty()) { update modifiedconlist; } if (!latestconlist.isEmpty()) { update latestconlist; } System.debug('Retrive list of new contacts' + latestconlist); System.debug('Checkpoint-4'); } }

Thank you sir. Awesome series.

Thanks for sharing the knowledge. It's very great explanation. It could have more meaningful if the newly created contact is populated with the account which is same as case account.

Seems like a problem in the code... Line 24 is not bulkified. And also in line 56, there is an issue. You created a new skill in line 49 and how can u expect the same skill to be present in the map used in line 56. but I would same your channel has some good content. keep up the good work.