I added the Apex Class and Apex trigger below to update the field Number_of_Contacts at the Account level when a Contact is added or removed to a certain Account.
My idea is to display in Accounts reports, how many Contacts an Account has. I had to do this, because Salesforce doesn't provide a Roll-Up Summary, at the Account level, to count Contacts.
I also tried creating a Flow, but it only works when a Contact is created or deleted.
Here are Apex Class and Trigger I tried to use:
Class:
public without sharing class ContactTriggerHandler {
private Set<Id> getAccountIds(List<Contact> contacts) {
Set<Id> accountIds = new Set<Id>();
for(Contact c : contacts) {
if(c.AccountId != null) {
accountIds.add(c.AccountId);
}
}
return accountIds;
}
private Map<Id, Account> getAccountMap(Set<Id> accountIds) {
return new Map<Id, Account>([SELECT Id, Number_of_Contacts__c FROM Account WHERE Id in :accountIds]);
}
public void process(List<Contact> contacts, System.TriggerOperation operation) {
Set<Id> accountIds = getAccountIds(contacts);
if(accountIds.size() > 0) {
Map<Id, Account> accountMap = getAccountMap(accountIds);
for(Contact c : contacts) {
if(accountMap.containsKey(c.AccountId)) {
switch on operation{
when AFTER_INSERT {
accountMap.get(c.AccountId).Number_of_Contacts__c += 1;
}
when AFTER_DELETE {
accountMap.get(c.AccountId).Number_of_Contacts__c -= 1;
}
when AFTER_UNDELETE {
accountMap.get(c.AccountId).Number_of_Contacts__c += 1;
}
}
}
}
update accountMap.values();
}
}
}
Trigger
trigger ContactTrigger on Contact (after insert, after delete, after undelete) {
ContactTriggerHandler handler = new ContactTriggerHandler();
switch on Trigger.operationType {
when AFTER_INSERT {
handler.process(Trigger.new, Trigger.operationType);
}
when AFTER_DELETE {
handler.process(Trigger.old, Trigger.operationType);
}
when AFTER_UNDELETE {
handler.process(Trigger.new, Trigger.operationType);
}
}
}
However, how can I include a line of code that updates the Number_of_Contacs__c field when a Contact moves to a different Account (like, an "After_Update" trigger)?
Thank you,
I tried some guidance on how to add AFTER UPDATE triggers in Apex Code, but I didn't succeed.
Trigger context variables give you access to old and new values, state of the database before the user clicked save and what the user has changed. You're passing from trigger to class only trigger.new but to get the id of original account the contact was linked to and update it - something like this.
Set<Id> idsToCheck = new Set<Id>();
// During update, only if account id changes...
for(Contact c : trigger.new){
Contact old = trigger.oldMap.get(c.Id);
if(c.AccountId != old.AccountId){
idsToCheck.add(c.AccountId);
idsToCheck.add(old.AccountId); // mark both for further processing
}
}
idsToCheck.remove(null);
List<Account> accounts = [SELECT Id, (SELECT Id FROM Contacts) FROM Account WHERE Id IN :idsToCheck];
for(Account a : accounts){
a.Number_of_Contacts__c = a.Contacts.size();
}
update accounts;
Is there a way to apply FLS Create check - Schema.sObjectType.Account.fields.Name.isCreateable() to the following?
public static Account createAccount() {
return new Account(
Name = 'Test',
OwnerId = UserInfo.getUserId()
);
}
Wondering if there is a way to apply without re-writing to the following:
public static Account createAccount() {
Account a = new Account();
if (Schema.sObjectType.Account.fields.Name.isCreateable()) {
a.Name = 'Test';
}
if (Schema.sObjectType.Account.fields.OwnerId.isCreateable()) {
a.OwnerId = UserInfo.getUserId();
}
insert a;
}
You can create a generic method that can iterate on each field to check for the FLS.
if access available then retain
if access not available then remove the particular field from the object instance.
I Have created a Generic Method to truncate the non-writable fields as follow:
public static List<SObject> truncateNotWriteableFields(List<SObject> listSObject){
Set<String> readOnlyFields = new Set<String>();
List<SObject> listSObjectNew = new List<SObject>();
if(listSObject.size() < 1){
return listSObjectNew;
}
Schema.SObjectType sObjType = listSObject.getSObjectType();
for(SObjectField field : sObjType.getDescribe().fields.getMap().values()){
if(field.getDescribe().isAccessible() && !field.getDescribe().isUpdateable() && !field.getDescribe().isCreateable()){
readOnlyFields.add(String.valueOf(field));
}
}
readOnlyFields.remove('Id'); // avoid removal in update
for(SObject obj : listSObject){
Map<String, Object> objMap = (Map<String, Object>) JSON.deserializeUntyped( JSON.serialize( obj ) );
objMap.keySet().removeAll(readOnlyFields);
SObject objWithoutNotWritableFields = (SObject) JSON.deserialize( JSON.serialize( objMap ), SObject.class );
system.debug('objWithoutNotWritableFields=>'+objWithoutNotWritableFields);
listSObjectNew.add(objWithoutNotWritableFields);
}
return listSObjectNew;
}
i am writing a visualforce page in which user has to select list of Opportunity i implemented pagination when user click previous button then it will render list panel again and i changed the value of list in the call my visualforce page segment is
<apex:pageBlockTable value="{!numbers}" var="n" align="center">
<apex:column >
<apex:inputCheckbox value="{!n.checked}"/>
</apex:column>
<apex:column value="{!n.cat.Id}" />
<apex:column value="{!n.cat.Name}" />
<apex:facet name="footer">Showing Page # {!pageNumber} of {!totalPages}</apex:facet>
</apex:pageBlockTable>
I have an OpportunityWrapper class for maintaining checkItem:
public class OpportunityWrapper {
public Boolean checked { get; set; }
public Opportunity cat { get; set; }
public OpportunityWrapper(){
cat = new Opportunity();
checked = false;
}
public OpportunityWrapper(Opportunity c){
cat = c;
checked = false;
}
public OpportunityWrapper(Opportunity c, Boolean checked){
cat = c;
this.checked = checked ;
}
}
The segment of code for getting list of OpportunityWrapper in custom controller is
public List<OpportunityWrapper> getNumbers() {
opp = new List<OpportunityWrapper>();
if ( selectedPage != '0' )
counter = list_size*integer.valueOf(selectedPage)-list_size;
//we have to catch query exceptions in case the list is greater than 2000 rows
try {
for ( Opportunity o : [SELECT Id,Name from Opportunity order by name
limit :list_size offset :counter] ) {
if ( !oppId.contains(o.Id) )
opp.add(new OpportunityWrapper(o));
else
opp.add(new OpportunityWrapper(o,true));
}
} catch ( QueryException e ) {
ApexPages.addMessages(e);
return null;
}
return opp;
}
and when the previous button is pressed following method is called
public PageReference Previous() {
//user clicked previous button
for ( OpportunityWrapper o : opp ) {
if ( o.checked && oppId.contains(o.cat.Id) )
oppId.add(o.cat.Id);
}
selectedPage = '0';
counter -= list_size;
return null ;
}
and the following are public member of custom class
private integer counter = 0; //keeps track of the offset
private integer list_size = 5;
public integer total_size;
List<OpportunityWrapper> opp ;
public List<OpportunityWrapper> oppwrapper = new List<OpportunityWrapper>(); //list of Opportunity wrapper shown in the page
public Set<String> oppId = new Set<String>(); //set for maintaining which Id's are checked
My target is when I checked any opportunity wrapper and goto next or previous list and when return back that item should be in checked but i am getting set value always empty but i am storing list of select value in the set of Previous() method why its showing Set always empty??
you can find out 2 solutions (one implemented with JS and another one with apex controller) in the following topic:
How to find out which checkboxes have been selected on the next page in VisualForce?
I have a VF page form that has a text field. The value of this text field is compared to an existing opportunity custom field. If the value of this text field exists in an opportunity, I go ahead and create a new record.
Everything works out fine except when I create a Apex Test Class.
I need to create an opportunity and hard code the value for the custom field. Unfortuneatly, this field is not writeable.
Has anyone ran into this issue before?
This is the error that I'm getting:
Error: Compile Error: Field is not writeable: Opportunity.Deal_Registration_ID__c at line 9 column 103
Below is my Apex Class.
My test class is at the very bottom.
Any help is greatly appreciated.
Cheers!
Rommel
public with sharing class VaultVF {
public Vault__c vault {get; set;}
public List<String> errorMsgs {get; set;}
public String saveResult {get; set;}
public String dealReg {get; set;}
public String oppId {get; set;}
public String email {get; set;}
public Boolean scenario1{get; set;}
public Boolean scenario2{get; set;}
public Boolean scenario3{get; set;}
// Generates country dropdown from country settings
public List<SelectOption> getCountriesSelectList() {
List<SelectOption> options = new List<SelectOption>();
options.add(new SelectOption('', '-- Select One --'));
// Find all the countries in the custom setting
Map<String, Country__c> countries = Country__c.getAll();
// Sort them by name
List<String> countryNames = new List<String>();
countryNames.addAll(countries.keySet());
countryNames.sort();
// Create the Select Options.
for (String countryName : countryNames) {
Country__c country = countries.get(countryName);
options.add(new SelectOption(country.Two_Letter_Code__c, country.Name));
}
return options;
}
//Constructor, set defaults
public VaultVF(ApexPages.StandardController controller) {
vault = (vault__c)controller.getRecord();
//Populate list of random characters for CAPTCHA verification
characters = new List<String>{'a','b','c','d','e','f','g','h',
'i','j','k','m','n','p','q','r','s','t','u','v','w',
'x','y','z','1','2','3','4','5','6','7','8','9'
};
errorMsgs = new List<String>();
}
//Determine if page has error so errors message can be displayed
public boolean getHasErrors(){
if(ApexPages.hasMessages() == true){
if(errorMsgs == null){
errorMsgs = new List<String>();
}
//Loop through errors and add to a list
for(ApexPages.Message m : ApexPages.getMessages()){
if(m.getSummary().startsWith('Uh oh.')){
errorMsgs.add(String.valueOf(m.getSummary()));
}
}
}
return ApexPages.hasMessages();
}
public void save(){
errorMsgs = new List<String>();
//Make sure captcha was correct
if(validateCAPTCHA() == false){
errorMsgs.add('Verification code was incorrect.');
captchaInput = '';
}
//Make sure the Scenario is selected
if(scenario1 == true){
vault.Scenario__c = 'BIG-IP as a Firewall';
}else if(scenario2 == true){
vault.Scenario__c = 'BIG-IP providing firewall service in conjunction with other vendors';
}else if(scenario3 == true){
vault.Scenario__c = 'BIG-IP front ending Firewalls';
if(vault.Load_Balancer_is_a_Firewall__c == null){
errorMsgs.add('Please specify if the load balancer is a firewall.');
}
if(vault.Providing_DDos_Connection__c == null){
errorMsgs.add('Please specify if this is providing DDos connection level protection for firewalls or both.');
}
if(vault.Providing_DDos_protection_except_ASM__c == null){
errorMsgs.add('Please specify if we are providing DDos protection (except ASM).');
}
if(vault.No_Traffic_Traverse__c == null){
errorMsgs.add('Please specify if there is Traffic that does not traverse the Firewalls.');
}
if(vault.Firewall_Vendor_Name__c == null){
errorMsgs.add('Please specify a Firewall Vendor.');
}
} else {
errorMsgs.add('Please select one of three Scenarios that is similar to your setup');
}
//-----Need to make sure deal registration number on opportuntiy is valid.-----
//Set some sort of is valid deal reg flag to false. We will assume the deal reg Id entered is invalid until we determine it is valid
vault.isValidDealReg__c = false;
//Query Account Id, and Deal_Registration_Id__c from the opportunity object where the deal registration Id equals the value entered in the form
List<Opportunity> opps = [select Id, Deal_Registration_ID__c from Opportunity where Deal_Registration_ID__c = :vault.Deal_Registration__c];
//check to see if registration id on the opp match the entered value
if(opps.size() > 0 && opps[0].Deal_Registration_ID__c == vault.Deal_Registration__c){
//If they do match set the valid deal reg Id flat to true
vault.isValidDealReg__c = true;
vault.Related_Opp__c = opps[0].Id;
//If they don't match then query all contacts on the account related to the opportunity
}
//If is valid deal reg flag is false add a message to the errorMSgs list indicated the entered
//deal registration is not valid with details on who they should contact
if(errorMsgs.size()>0){
//stop here
} else if(vault.isValidDealReg__c == false){
errorMsgs.add('Deal Registration # was incorrect. Please contact your representative.');
} else {
try{
vault.Status__c = 'Application Received';
insert vault;
saveResult = 'success';
}catch(exception e){
ApexPages.addMessage(new ApexPages.Message(ApexPages.Severity.FATAL, 'Uh oh. Something didn\'t work quite right. Please contact partners#f5.com for assistance. \n\n' + e.getMessage()));
saveResult = 'fail';
}
}
}
//------------------------Code for CAPTCHA verification----------------------------
public String captchaInput {get; set;}
List<String> characters;
String char1;
String char3;
String char5;
//This methods simply returns a random number between 0 and the size of the character list
public Integer randomNumber(){
Integer random = Math.Round(Math.Random() * characters.Size());
if(random == characters.size()){
random--;
}
return random;
}
/*Here we have 6 get methods that return 6 random characters to the page.
For chars 1,3, and 5 (the black characters) we are saving the the values so
that we can compare them with the user's input */
public String getChar1(){
char1 = characters[randomNumber()];
return char1;
}
public String getChar2(){
return characters[randomNumber()];
}
public String getChar3(){
char3 = characters[randomNumber()];
return char3;
}
public String getChar4(){
return characters[randomNumber()];
}
public String getChar5(){
char5 = characters[randomNumber()];
return char5;
}
public String getChar6(){
return characters[randomNumber()];
}
public String correctAnswer(){
return char1 + char3 + char5;
}
public Boolean validateCAPTCHA(){
if(captchaInput.length() == 3 && captchaInput.subString(0,1) == char1 && captchaInput.subString(1,2) == char3 && captchaInput.subString(2,3) == char5){
return true;
}else{
return false;
}
}
}
Below is Test Class
#isTest
public class VaultVFTEST {
/*This tests the VaultVF class on the Vault__c object*/
static testMethod void VaultVFTEST() {
//Create opp, hardcode the Deal_Registration_Id__c field with 'x111111111' and insert opp
Opportunity opp = new Opportunity(Name='test opp', StageName='stage', Deal_Registration_Id__c='x111111111', Probability = 95, CloseDate=system.today());
insert opp;
//Create and insert Vault record
Vault__c vault = new Vault__c(
Deal_Registration__c = 'x111111111',
Scenario__c = 'BIG-IP front ending Firewalls',
isValidDealReg__c = true,
Status__c = 'Application Received',
Load_Balancer_is_a_Firewall__c = 'Yes',
Providing_DDos_Connection__c = 'Firewalls',
Providing_DDos_protection_except_ASM__c = 'Yes',
No_Traffic_Traverse__c = 'Yes',
Firewall_Vendor_Name__c = 'Firewall Vendor Company',
First_Name__c = 'Test First Name',
Last_Name__c = 'Test Last Name',
Company_Name__c = 'Test Company Name',
Phone__c = '(206) 946-3126',
Email__c = 'test#email.com',
Country__c = 'United States',
Additional_Info__c = 'Test Additional Info',
Related_Opp__c = opp.Id
);
//Now insert the vault record to invoke the VaultVF class
Test.startTest();
insert vault;
Test.stopTest();
//Assertion Testing - Check to make sure the Deal_Registration__c value
//matches the Deal_Registration_Id__c value of the related opportunity
for(Vault__c v : [select Id, Deal_Registration__c from Vault__c where Deal_Registration__c IN :vault]){
system.assertEquals(v.Deal_Registration__c, opp.Deal_Registration_Id__c);
}
}
}
What's the Opportunity.Deal_Registration_ID__c field type? If it's a formula field or autonumber it will really really be not writeable from Apex.
Generally speaking in such scenarios you'd have to insert the Opportunity (without this field set), then read it back from database after a successful insert. If it's autonumber - magic will happen on it's own (although I wouldn't say it's a best idea to have some kind of license numbers consecutive & easy to fake ;)), if it's a formula field - make sure you've set all fields that are used in calculation prior to the insert.
So something like this should do the trick:
Opportunity opp = new Opportunity(Name='test opp', StageName='stage', Probability = 95, CloseDate=system.today());
insert opp;
opp = [SELECT Id, Name, Deal_Registration_ID__c FROM Opportunity WHERE Id = :opp.Id];
// Optionally - make sure value was set
System.assertNotEquals(null, opp.Deal_Registration_ID__c);
System.assert(opp.Deal_Registration_ID__c.length() > 0); // if it's a String
Vault__c vault = new Vault__c(
Deal_Registration__c = opp.Deal_Registration_ID__c ,
Scenario__c = 'BIG-IP front ending Firewalls',
isValidDealReg__c = true,
Status__c = 'Application Received',
Load_Balancer_is_a_Firewall__c = 'Yes',
Providing_DDos_Connection__c = 'Firewalls',
Providing_DDos_protection_except_ASM__c = 'Yes',
No_Traffic_Traverse__c = 'Yes',
Firewall_Vendor_Name__c = 'Firewall Vendor Company',
First_Name__c = 'Test First Name',
Last_Name__c = 'Test Last Name',
Company_Name__c = 'Test Company Name',
Phone__c = '(206) 946-3126',
Email__c = 'test#email.com',
Country__c = 'United States',
Additional_Info__c = 'Test Additional Info',
Related_Opp__c = opp.Id
);
Trying to set a value for a roll up summary field in the test class to improve code coverage. How do I do it?
public class clsPreferredIASetExt {
List<PreferredIA__c> preferredias;
public static PreferredIA__c[] tobeClosed = new PreferredIA__c[0];
public static PreferredIA__c[] newPreIAs = new PreferredIA__c[0];
public static PreferredIA__c loopadd;
public static PreferredContact__c[] contactlists = new PreferredContact__c[0];
public static Account[] InvoicedAccounts = new Account[0];
public static PreferredIA__c[] monkey;
public clspreferrediaSetExt(ApexPages.StandardSetController controller) {
preferredias = (List<PreferredIA__c>) controller.getSelected();
}
public void getInitCloseInv() {
tobeclosed = [select id, Account__c, Account__r.id, Account__r.Name,
Account__r.AccountNumber, Specialist__c,
PreferredInvoice__c, Status__c
from PreferredIA__c where Status__c = 'Invoiced' limit 150];
list<string> testme = new list<string>{};
for(PreferredIA__c a:tobeclosed) {
testme.add(a.Account__r.id);
}
InvoicedAccounts = [select id, EligibleIAs__c, PreferredOverride__c,
Preferred_Territory__r.rep__c, LastSurveyDate__c,
InitialInspectionComplete__c, Program_level__c,
PreferredExempt__c, Account_Status__c,
Active_IAs__c, Last_Training__c
from Account where id IN :testme];
Contactlists = [select id, Account__c
from PreferredContact__c where Account__c IN :testme];
for(PreferredIA__c q:tobeclosed) {
q.Status__c = 'Closed';
}
for(Account z:invoicedaccounts) {
/****************************************************************
The following condition is where I am trying to set the z.EligibleIAs__c
which is a roll up count field of PreferredIA__c objects associated with
the account.
****************************************************************/
if(z.EligibleIAs__c == 0
&& z.Program_Level__c == 'Preferred'
&& !z.PreferredExempt__c
&& (z.Account_Status__c == 'Active'
|| z.Account_Status__c == 'Product Only')) {
loopadd = new PreferredIA__c();
system.debug(z.id);
system.debug(z.Account_Status__c);
loopadd.Account__c = z.id;
if(z.PreferredOverride__c != null) {
loopadd.Specialist__c = z.PreferredOverride__c;
}
else {
loopadd.Specialist__c= z.Preferred_territory__r.Rep__c;
}
for(PreferredContact__c q:contactlists) {
if(q.Account__c == z.id) {
loopadd.PreferredContact__c = q.id;
}
}
loopadd.CreatedDate__c = Date.Today();
if(z.Last_training__c != null) {
loopadd.DueDate__c = z.Last_Training__c.AddDays(365);
}
else {
loopadd.DueDate__c = Date.Today().AddDays(365);
}
loopadd.initial__c = false;
loopadd.Status__c = 'Unacknowledged';
newPreIAs.add(loopadd);
}
z.InitialInspectionComplete__c = true;
}
try {
update tobeclosed;
update invoicedaccounts;
insert newPreIAs;
}
catch(system.dmlexception q) {
system.debug(q);
system.debug(invoicedaccounts);
system.debug(newPreIAs);
}
}
public void ReceivePPW() {
monkey = [select id, Status__c from PreferredIA__c
where id in :preferredias and status__c = 'Training Completed'];
for (PreferredIA__c m:monkey) {
m.status__c = 'Awaiting Invoice';
}
update monkey;
}
}
I can't actually see where you're trying to write to the field — or did you remove it because it wasn't working?
That aside, the answer is that you can not write to a roll-up summary field. If you require a value in that field you should insert child records to your parent test records, with appropriate field values such that your summary field calculates a value.
Also, I can see that you're querying PerferredIA__c at the start, your test methods should never depend on data being in the system already, you should insert your records yourself in your test code. The reason for this is that if you try to deploy to an org which has no relevant data, your tests will fail and so, subsequently, will your deployment.
For situations like these, consider mock objects (or just variables simulating expected values), similar to inserting test values as Lacey suggests. This technique is required to achieve 100% coverage when doing callouts, for example, since they terminate tests at the moment of the call.