afterupdate error

error afterupdate
I got below error
"update failed Cannot_insert_update_activate_entity samplelotprocess: execution of afterupdate caused by ssytem dmlexception :update failed"

i noticed error shows in line no 82 at the time of update any idea why please advise me

trigger SampleLotProcess on Sample_Lot__c (after insert, after update) {
    class SampleLotProcessHandler {
        Sample_Lot__c[] oldRows;
        Sample_Lot__c[] newRows;
        Map<Id, Sample_Lot__c> oldMap;
        Map<Id, Sample_Lot__c> newMap;

    Map<String, RecordType> rtMap;


        public void initialize(Sample_Lot__c[] oldRows, Sample_Lot__c[] newRows, Map<Id, Sample_Lot__c> oldMap, Map<Id, Sample_Lot__c> newMap) {
            this.oldRows = oldRows;
            this.newRows = newRows;
            this.oldMap = oldMap;
            this.newMap = newMap;

        }

        public void onAfterInsert() {
            createReceipts();
        }

        public void onAfterUpdate() {
            createReceipts();
        }

        public void createReceipts() {
            Sample_Lot__c[] filteredRows = new Sample_Lot__c[0];
            Set<Id> transferIds = new Set<Id>();
            for (Sample_Lot__c item : newRows) {
                if (item.Status__c != 'Confirmed') continue;
                if (oldMap != null && item.Status__c == oldMap.get(item.Id).Status__c) continue;

                filteredRows.add(item);
                if (item.Original_Transfer_Transaction__c != null) {
                  transferIds.add(item.Original_Transfer_Transaction__c);
                }
            }

            Sample_Transaction__c[] transfers = [select id, Status__c, Transferred_From__c  from Sample_Transaction__c where id in :transferIds];
            Map<Id, Sample_Transaction__c> transferMap;
            if (transfers.isEmpty()) {
                transferMap = new Map<Id, Sample_Transaction__c>();
            } else {
                transferMap = new Map<Id, Sample_Transaction__c>(transfers);
            }

            Sample_Transaction__c[] forIns = new Sample_Transaction__c[0];

            for (Sample_Lot__c item : filteredRows) {
                if (item.Status__c != 'Confirmed') continue;
                if (oldMap != null && item.Status__c == oldMap.get(item.Id).Status__c) continue;

                Sample_Transaction__c newReceipt = new Sample_Transaction__c();
                newReceipt.Transfer_To__c = item.OwnerId;
                newReceipt.Comments__c = item.Comments__c;
                newReceipt.Quantity__c = item.Confirmed_Quantity__c;
                newReceipt.OwnerId = item.OwnerId;
                newReceipt.Transfer_To__c = item.OwnerId;

                if (transferMap.containsKey(item.Original_Transfer_Transaction__c)) {
                    newReceipt.Transferred_From__c = transferMap.get(item.Original_Transfer_Transaction__c).Transferred_From__c;
                    transferMap.get(item.Original_Transfer_Transaction__c).Status__c = 'Confirmed';
                }

                if (getRtMap().containsKey('Reckitt_Receipt')) {
                    newReceipt.RecordTypeId = getRtMap().get('Reckitt_Receipt').Id;
                }

                newReceipt.Product__c = item.Product_ID__c;
                newReceipt.Sample_Lot_ID__c = item.Id;
                newReceipt.Transaction_Date__c = Date.today();

                forIns.add(newReceipt);
            }

            if (!forIns.isEmpty()) {
                insert forIns;
            }

            if (!transferMap.isEmpty()) {
                update transferMap.values();
            }
        }

        public Map<String, RecordType> getRtMap() {
            if (rtMap == null) {
                rtMap = new Map<String, RecordType>();

                RecordType[] rtList = [select id, DeveloperName from RecordType where SObjectType = 'Sample_Transaction__c'];

                for (RecordType item : rtList) {
                    rtMap.put(item.DeveloperName, item);
                }
            }

            return rtMap;
        }
    }

    SampleLotProcessHandler handler = new SampleLotProcessHandler();
    handler.initialize(trigger.old, trigger.new, trigger.oldMap, trigger.newMap);

    if (trigger.isAfter && trigger.isInsert) {
        handler.onAfterInsert();
    } else if (trigger.isAfter && trigger.isUpdate) {
        handler.onAfterUpdate();
    }
}

March 13, 2017
·
Like
0
·
Follow
1

Best Answer chosen by Somasundaram S 1

LBK
It needs a good understanding of your business case to suggest a perfect solution.

However, you need perfect your transferMap preparation logic to work with your Lookup filter output.

You may want to put some debug statements in the code which reads and writes to transferMap map object to see where exactly the road block is.

Hope this helps.

March 14, 2017
·
Like
2
·
Dislike
0

LBK
Can you please post your code using "Add a Code Sample" option?

And, please post your Trigger and Helper classes in different code blocks, so that it will be easy for us to debug.

March 13, 2017
·
Like
0
·
Dislike
0

Somasundaram S 1
Please see below the details
when user login we load the below VF and
<apex:page controller="MySamplesController"> <apex:includeScript value="https://code.jquery.com/jquery-2.1.1.min.js"/> <apex:sectionHeader title="My Samples" subtitle="Home"/> <style> .columnA { width: 30%; font-weight: bold; } .columnB { width: 70%; } </style> <script> $j = jQuery.noConflict(); var rowid; function jsConfirmReceipt(aRowId) { rowid = aRowId; var pp = $j("#pSource div"); showPopup('Confirm Receipt', pp.html(), 400, 300); if (receipts) { item = $j.grep(receipts, function(e){ return e.Id == rowid; }); if (item && item.length != 0) { popupbox = $j("#" + window.box.id); //popupbox.find('[data-id="from"]').text(item[0].Original_Transfer_Transaction__r.Owner.Name); popupbox.find('[data-id="expirationDate"]').text(item[0].Expiration_Date__c); popupbox.find('[data-id="lotName"]').text(item[0].Name); popupbox.find('[data-id="shipmentDate"]').text(item[0].Shipped_Date__c); popupbox.find('[data-id="product"]').text(item[0].Product_ID__r.Name); popupbox.find('[data-id="quantity"]').text(item[0].Shipped_Quantity__c); } console.log(item); } } SimpleDialog.prototype.setHeight = function(h) { var b = this.getContentElement(); b.style.height = h + "px" }; function showPopup(stitle, sbody, pWidth, pHeight){ if (!pWidth) pWidth = 500; if (!pHeight) pHeight = 300; if (window.box) { box = window.box; } else { box = new SimpleDialog("Test"+Dialogs.getNextId(), false); box.allowKeyboardEsc = true; box.createDialog(); window.box = box; } box.setTitle(stitle); box.setContentInnerHTML(sbody); box.setupDefaultButtons(); //box.setHeight(pHeight); box.setWidth(pWidth); box.show(); } function saveDialog() { popupbox = $j("#" + window.box.id); q = popupbox.find('[data-id$="itemquantity"]'); c = popupbox.find('[data-id$="itemcomment"]'); console.log(rowid, q.val(), c.val()); confirmReceipt(rowid, q.val(), c.val()); window.box.hide(); } function hideDialog(){ window.box.hide(); } </script> <apex:form id="theForm"> <apex:actionFunction action="{!confirmReceipt}" name="confirmReceipt" status="statusFilter" rerender="receiptBlock,theTables"> <apex:param name="recid" value=""></apex:param> <apex:param name="quantity" value=""></apex:param> <apex:param name="comment" value=""></apex:param> </apex:actionFunction> <apex:pageBlock title="My Samples" mode="edit"> <apex:pageBlockSection title="Sample Transactions" columns="1" collapsible="false"> <p> Use the sample transaction links below to record sample movements, adjustments, or inventories. </p> <apex:outputLink title="View Transactions" value="{!URLFOR('/a0i?fcf=00B240000051O0j')}">My Transactions</apex:outputLink> <apex:outputLink title="View Orders" value="{!URLFOR('/a0i?fcf=00B240000051O0i')}">My Orders</apex:outputLink> <apex:outputLink title="New Transfer" value="{!URLFOR($Action.Sample_Transaction__c.Edit, $ObjectType.Sample_Transaction__c, [RecordType=rtTransfer])}">New Transfer</apex:outputLink> <apex:outputLink title="New Return" value="{!URLFOR($Action.Sample_Transaction__c.Edit, $ObjectType.Sample_Transaction__c, [RecordType=rtReturn])}">New Return</apex:outputLink> <apex:outputLink title="Sample Inventory Report" value="{!URLFOR('/00O24000000fWDV')}">Sample Inventory Report</apex:outputLink> </apex:pageBlockSection> <apex:pageBlockSection title="Sample Storage Location" columns="1" collapsible="false"> <apex:outputPanel rendered="{!IF(ISBLANK($User.Sample_Storage_Location__c), TRUE, FALSE)}"> No Sample Storage Location information is available. </apex:outputPanel> <apex:outputPanel rendered="{!IF(ISBLANK($User.Sample_Storage_Location__c), FALSE, TRUE)}"> {!$User.Sample_Storage_Location__c} </apex:outputPanel> </apex:pageBlockSection> <apex:pageBlockSection title="Pending Sample Receipts" columns="1" collapsible="false" id="receiptBlock"> <apex:outputPanel rendered="{!pendingLots.size != 0}"> <div> The following samples have been transferred to you. Please click the link to confirm the quantity received. </div> <script> var receipts = JSON.parse('{!JSENCODE(jsonReceipts)}'); console.log(receipts); </script> <apex:pageBlockTable value="{!pendingLots}" var="item"> <apex:column headerValue="Action"> <apex:outputLink value="#" onclick="jsConfirmReceipt('{!item.Id}');">Confirm</apex:outputLink> </apex:column> <apex:column headerValue="{!$ObjectType.Product__c.fields.Name.label}" value="{!item.Product_ID__r.Name}"/> <apex:column headerValue="{!$ObjectType.Sample_Lot__c.fields.Name.label}" value="{!item.Name}"/> <apex:column headerValue="{!$ObjectType.Sample_Lot__c.fields.Expiration_Date__c.label}" value="{!item.Expiration_Date__c}"/> <apex:column headerValue="{!$ObjectType.Sample_Lot__c.fields.Shipped_Quantity__c.label}" value="{!item.Shipped_Quantity__c}"/> <apex:column headerValue="{!$ObjectType.Sample_Lot__c.fields.Shipped_Date__c.label}" value="{!item.Shipped_Date__c}"/> </apex:pageBlockTable> </apex:outputPanel> <apex:outputPanel rendered="{!pendingLots.size == 0}"> No Pending Receipt </apex:outputPanel> </apex:pageBlockSection> <apex:pageBlockSection title="Sample Lots" columns="1" collapsible="false">  <p> <b>Active Sample Lots</b> </p> <apex:outputPanel id="theTables"> <apex:outputPanel > <apex:pageBlockTable value="{!sampleLots}" var="item"> <apex:column headerValue="{!$ObjectType.Product__c.fields.Name.label}" value="{!item.Product_ID__r.Name}"/> <apex:column headerValue="{!$ObjectType.Sample_Lot__c.fields.Name.label}" value="{!item.Name}"/> <apex:column headerValue="{!$ObjectType.Sample_Lot__c.fields.Expiration_Date__c.label}" value="{!item.Expiration_Date__c}"/> <apex:column headerValue="{!$ObjectType.Sample_Lot__c.fields.Active__c.label}"> <apex:inputCheckbox value="{!item.Active__c}"> <apex:actionSupport event="onclick" action="{!saveLot}" status="statusFilter" rerender="theTables"/> </apex:inputCheckbox> </apex:column> </apex:pageBlockTable> </apex:outputPanel> <p> <b>Inactive Sample Lots</b> </p> <apex:outputPanel > <apex:pageBlockTable value="{!inactiveSampleLots}" var="item"> <apex:column headerValue="{!$ObjectType.Product__c.fields.Name.label}" value="{!item.Product_ID__r.Name}"/> <apex:column headerValue="{!$ObjectType.Sample_Lot__c.fields.Name.label}" value="{!item.Name}"/> <apex:column headerValue="{!$ObjectType.Sample_Lot__c.fields.Expiration_Date__c.label}" value="{!item.Expiration_Date__c}"/> <apex:column headerValue="{!$ObjectType.Sample_Lot__c.fields.Active__c.label}"> <apex:inputCheckbox value="{!item.Active__c}"> <apex:actionSupport event="onclick" action="{!saveLot}" status="statusFilter" rerender="theTables"/> </apex:inputCheckbox> </apex:column> </apex:pageBlockTable> </apex:outputPanel> </apex:outputPanel> </apex:pageBlockSection> <apex:actionStatus id="statusFilter"> <apex:facet name="start"> <div style="position: fixed; top: 0; left: 0; right: 0; bottom: 0; opacity: 0.25; z-index: 1000; background-color: black;">   </div> <div style="position: fixed; left: 0; top: 0; bottom: 0; right: 0; z-index: 1001; margin: 15% 50%"> <div style="display: inline-block; padding: 2px; background-color: #fff; width: 125px;"> <img src="/img/loading.gif" style="float: left; margin: 8px;" /> <span style="display: inline-block; padding: 10px 0px;">Processing...</span> </div> </div> </apex:facet> </apex:actionStatus> </apex:pageBlock> <div id="pSource" style="display: none;"> <div> <apex:panelGrid columns="2" columnClasses="columnA,columnB" style="width:100%">  <apex:outputLabel value="{!$ObjectType.Product__c.fields.Name.label}"></apex:outputLabel> <span data-id="product"> </span> <apex:outputLabel value="{!$ObjectType.Sample_Lot__c.fields.Name.label}"></apex:outputLabel> <span data-id="lotName"> </span> <apex:outputLabel value="{!$ObjectType.Sample_Lot__c.fields.Expiration_Date__c.label}"></apex:outputLabel> <span data-id="expirationDate"> </span> <apex:outputLabel value="{!$ObjectType.Sample_Lot__c.fields.Shipped_Quantity__c.label}"></apex:outputLabel> <span data-id="quantity"> </span> <apex:outputLabel value="{!$ObjectType.Sample_Lot__c.fields.Shipped_Date__c.label}"></apex:outputLabel> <span data-id="shipmentDate"> </span> <apex:outputLabel value="{!$ObjectType.Sample_Lot__c.fields.Confirmed_Quantity__c.label}"></apex:outputLabel> <apex:inputField html-data-id="itemquantity" value="{!receipt.Confirmed_Quantity__c}" /> <apex:outputLabel value="{!$ObjectType.Sample_Lot__c.fields.Comments__c.label}"></apex:outputLabel> <apex:inputField html-data-id="itemcomment" value="{!receipt.Comments__c}" style="width:100%"/> </apex:panelGrid> <span style="display: inline-block; padding: 10px 31%;"> <apex:commandButton value="Confirm" action="{!confirmReceipt}" onclick="saveDialog();"/> <apex:commandButton value="Cancel" onclick="hideDialog();"/> </span> </div> </div> </apex:form> </apex:page>

user get the below screen and when user click the confirm button the popup screen open and then he click confirm button in popup then

calling below trigger in an object

trigger SampleLotProcess on Sample_Lot__c (after insert, after update) { class SampleLotProcessHandler { Sample_Lot__c[] oldRows; Sample_Lot__c[] newRows; Map<Id, Sample_Lot__c> oldMap; Map<Id, Sample_Lot__c> newMap; Map<String, RecordType> rtMap; public void initialize(Sample_Lot__c[] oldRows, Sample_Lot__c[] newRows, Map<Id, Sample_Lot__c> oldMap, Map<Id, Sample_Lot__c> newMap) { this.oldRows = oldRows; this.newRows = newRows; this.oldMap = oldMap; this.newMap = newMap; } public void onAfterInsert() { createReceipts(); } public void onAfterUpdate() { createReceipts(); } public void createReceipts() { Sample_Lot__c[] filteredRows = new Sample_Lot__c[0]; Set<Id> transferIds = new Set<Id>(); for (Sample_Lot__c item : newRows) { if (item.Status__c != 'Confirmed') continue; if (oldMap != null && item.Status__c == oldMap.get(item.Id).Status__c) continue; filteredRows.add(item); if (item.Original_Transfer_Transaction__c != null) { transferIds.add(item.Original_Transfer_Transaction__c); } } Sample_Transaction__c[] transfers = [select id, Status__c, Transferred_From__c from Sample_Transaction__c where id in :transferIds]; Map<Id, Sample_Transaction__c> transferMap; if (transfers.isEmpty()) { transferMap = new Map<Id, Sample_Transaction__c>(); } else { transferMap = new Map<Id, Sample_Transaction__c>(transfers); } Sample_Transaction__c[] forIns = new Sample_Transaction__c[0]; for (Sample_Lot__c item : filteredRows) { if (item.Status__c != 'Confirmed') continue; if (oldMap != null && item.Status__c == oldMap.get(item.Id).Status__c) continue; Sample_Transaction__c newReceipt = new Sample_Transaction__c(); newReceipt.Transfer_To__c = item.OwnerId; newReceipt.Comments__c = item.Comments__c; newReceipt.Quantity__c = item.Confirmed_Quantity__c; newReceipt.OwnerId = item.OwnerId; newReceipt.Transfer_To__c = item.OwnerId; if (transferMap.containsKey(item.Original_Transfer_Transaction__c)) { newReceipt.Transferred_From__c = transferMap.get(item.Original_Transfer_Transaction__c).Transferred_From__c; transferMap.get(item.Original_Transfer_Transaction__c).Status__c = 'Confirmed'; } if (getRtMap().containsKey('Reckitt_Receipt')) { newReceipt.RecordTypeId = getRtMap().get('Reckitt_Receipt').Id; } newReceipt.Product__c = item.Product_ID__c; newReceipt.Sample_Lot_ID__c = item.Id; newReceipt.Transaction_Date__c = Date.today(); forIns.add(newReceipt); } if (!forIns.isEmpty()) { insert forIns; } if (!transferMap.isEmpty()) { update transferMap.values(); } } public Map<String, RecordType> getRtMap() { if (rtMap == null) { rtMap = new Map<String, RecordType>(); RecordType[] rtList = [select id, DeveloperName from RecordType where SObjectType = 'Sample_Transaction__c']; for (RecordType item : rtList) { rtMap.put(item.DeveloperName, item); } } return rtMap; } } SampleLotProcessHandler handler = new SampleLotProcessHandler(); handler.initialize(trigger.old, trigger.new, trigger.oldMap, trigger.newMap); if (trigger.isAfter && trigger.isInsert) { handler.onAfterInsert(); } else if (trigger.isAfter && trigger.isUpdate) { handler.onAfterUpdate(); } }
and got error like below any idea why

please advise me

thanks
soma

March 14, 2017
·
Like
0
·
Dislike
0

LBK
Hi Soma,

From the error screen shot, I believe that this error is caused by either a Validation Rule or a Lookup Filter set on the Sample_Lot__c object.

Can you check if there is a validation rule that shows a message like "Only confirmed Sample Lots are available"?

March 14, 2017
·
Like
2
·
Dislike
0

Somasundaram S 1
wow LBK you are great .
yes one of the field uses below lookup , once i disbale filter it works fine but what could be the best solution without disbale the lookup filter
((Sample Transaction: Record TypeequalsReckitt Transfer, Reckitt Return) and (Sample Lot: StatusequalsConfirmed) and (Sample Lot: Owner IDequalsCurrent User: User ID)) or (Sample Transaction: Record Typenot equal toReckitt Return, Reckitt Transfer)

March 14, 2017
·
Like
0
·
Dislike
0

Browse by Topic

Welcome to Support!

Show

sorted by

afterupdate error

All Answers

You need to sign in to do that.