[SalesForce] How to use getQueryLocator(sObject[])

I'm getting an error 'Argument must be an inline query' on the return line. Does that mean I need to make a query of my sObject in the parameter of the getQueryLocator() as opposed to passing in a variable for a List of opportunities?

public Database.QueryLocator start(Database.BatchableContext BC) {
    return Database.getQueryLocator(woList);
}

where List<Opportunity> woList = new List<Opportunity>(trigger.new);

I would use my query there, but I'm writing a batchable class for a before Insert trigger. I can't think of anyway to write the query to identify the opportunities in trigger.new as they don't have Ids assigned yet.

Within context, I am trying to lookup a value passed in on an Opportunity (fs_wmCompanyId__c). I will then need to look to the account table to see if any account has that same value stored in a field (fs_wmCompanyId__c) on that table; if there is I will need the account ID. I will then loop back to the inserted Opportunity and set the account Id in the rs_client__c field.

I have it on a before insert trigger to ensure the rs_client__c field is set prior to saving the record; this was to make sure no validation rules prevent the update after save and that no integrations with other systems sync the data without the rs_client__c field. I figured the trigger should call a batchable class in case several hundred records were inserted at once. This is my first piece of code, so feel free to rip it apart and critique.

Best Answer

You actually need to provide a query string to the QueryLocator. If you want to use a collection instead, try using Iterable.

global Iterable start(Database.BatchableContext info){ 
    return woList; 
}

http://www.salesforce.com/us/developer/docs/apexcode/Content/apex_batch_interface.htm

Related Solutions

[SalesForce] How to create schedule batch apex which returns list of object records

I rewrote your batch class to the way I write batch Apex - I hope this helps.

global class ScheduleContact implements Database.Batchable<SObject> {

global Database.queryLocator start(Database.BatchableContext ctx) {
    DateTime theDateTime = System.now().addDays(-60);
    String query = 'Select FirstName, LastName, AccountId, Contract_End_Date__c';
    query += ' FROM Contact WHERE Payment_Type__c = \'Paid In Full\'';
    query += ' AND Contract_End_Date__c >= :theDateTime';

    return Database.getQueryLocator(query);

}
global void execute(Database.BatchableContext BC, List<sObject> scope) {
    List<Contact> contactList = (List<Contact>)scope;

    //for creating new opportunity.
    List<Opportunity> newOpps = new List<Opportunity>();

    for (Contact c : contactList) {
        Opportunity opp = new Opportunity();
        opp.Name = c.FirstName + ' ' + c.LastName + ' Opportunity';
        opp.CloseDate = c.Contract_End_Date__c;
        opp.Type = 'Renewal';
        opp.StageName = 'Met';
        opp.AccountId = c.AccountId;
        newOpps.add(opp);
    }

    insert newOpps;
}
global void finish(Database.BatchableContext BC) {

}
}

I am using batchable, making a dynamic Apex query string, passing a list to the execute, casting it to a list and then working through that list to make new Opportunities.

(Edit: Thanks to @dphil for noticing my unused variable - removed it.)

[SalesForce] Trigger to copy child object picklist values to parent object separated by comma

The code you have shared will not at all compile.

use close brace here for (Opportunity childObj : Trigger.new)
use childObj.AccountId instead of childObj.Account to access AccountId
When you use relationship query be sure to use Opportunities instead of Opportunity.
You should put Opportunity's Opp_Location__c into a List. Then use String.join() to add those with comma delimited.
And finally update accounts.

Trigger

trigger UpdateAmount on Opportunity (after insert, after update) { 
  Map<ID, Account > parentOpps = new Map<ID, Account>(); 
  Set<Id> listIds = new Set<Id>();

  for (Opportunity childObj : Trigger.new) {
    listIds.add(childObj.AccountId);
  }

  parentOpps = new Map<Id, Account>([SELECT id, Location_Type__c ,
                                        (SELECT ID, Opp_Location__c  FROM Opportunities) 
                                    FROM Account WHERE ID IN :listIds]);

  List<Account> lstAccount = new List<Account>();

  for(Account acct:parentOpps.values())
  {  
      List<String> strType = new List<String>();
      for(Opportunity oppty:acct.Opportunities)
      {
          strType.add(oppty.Opp_Location__c);
      }
      acct.Location_Type__c = String.join(strType,',');

  }

  update parentOpps.values();
}

Related Topic