here is my code public class AttachmentTriggerHandler { public void appendingCaseNumberToAttachment(list<Attachment> attaches){ list<Attachment> listOfAttachments = new list<Attachment>(); set<Id> parentIds = new set<Id>(); for(Attachment newAttachments:attaches){ system.debug('attaches====>'+newAttachments); if(((String)newAttachments.ParentID).startswith('500')){ parentIds.add(newAttachments.ParentID); System.debug('parentIds===='+parentIds); } Map<id,Case> casemap=new map<Id,Case>([select id,CaseNumber,(SELECT id,Name from Attachments) from case where id IN:parentIds]); for(Case cc:casemap.values()){ for(Attachment attach:cc.Attachments){ System.debug('Attachments========'+attach); attach.name=cc.CaseNumber+'-'+attach.Name; listOfAttachments.add(attach); //Attachments.Name = Attachments.Name+'-'+Attachments.(case.id).ParentId; } } } }}Please suggest me where i am doing mistake