Map.put error - getting unexpected token error

Question

This is the code I am using:

trigger ParentOppToChild on Opportunity (after update)
{

    List <Opportunity> oppQueryResults = new List <Opportunity>([Select ID, (Select ID, Dealership__c From OpportunityLineItems) From Opportunity]);

    Map<ID,OpportunityLineItem> accountTolineItems = New Map<ID,OpportunityLineItem>();
    for(Opportunity o : oppQueryResults)
    {
        if(o.StageName = 'Closed Won')
        {
            for(OpportunityLineItem oli : o.OpportunityLineItems)
            {

                accountToLineItems.put(oli.Dealership__c, new OpportunityLineItem{oli});

                if(accountToLineItems.containsKey(oli.Dealership__c))
                {
                    accountToLineItems.containsKey(oli.Dealership__c).add(oli);
                }
                else
                {
                    accountToLineItems.put(oli.Dealership__c,New OpportunityLineItem[]{oli});
                }
            }
        }

    }   
}

And this is the error that I am getting: unexpected token: '{' at line 21

Sorry if this is really obvious but I have changed the code around so many times because of this and I can't seem to find an answer elsewhere.

Answer 1

Instead use this on line 21 :

accountToLineItems.put(oli.Dealership__c,New list<OpportunityLineItem>{oli});

Also look this link : https://developer.salesforce.com/forums/ForumsMain?id=906F0000000913SIAQ

Answer 2

Not sure about your complete use case.. you shouldn't be querying all the opportunities and its line items in your trigger.. you should just limit the opportunity records that got updated.

but to the error, you should change

accountToLineItems.put(oli.Dealership__c, new OpportunityLineItem{oli});

to

accountToLineItems.put(oli.Dealership__c, new List<OpportunityLineItem>());

but you are doing this initialization inside the for loop which is not required since the next if.. else.. part takes care of the initialization as well..

and also you should change this line as well

accountToLineItems.containsKey(oli.Dealership__c).add(oli);

to

accountToLineItems.get(oli.Dealership__c).add(oli);

so, your overall code will look like this

trigger ParentOppToChild on Opportunity (after update)
{

    List <Opportunity> oppQueryResults = new List <Opportunity>([Select ID, (Select ID, Dealership__c From OpportunityLineItems) From Opportunity]);

    Map<ID,OpportunityLineItem> accountTolineItems = New Map<ID,OpportunityLineItem>();
    for(Opportunity o : oppQueryResults)
    {
        if(o.StageName = 'Closed Won')
        {
            for(OpportunityLineItem oli : o.OpportunityLineItems)
            {

                if(accountToLineItems.containsKey(oli.Dealership__c))
                {
                    accountToLineItems.get(oli.Dealership__c).add(oli);
                }
                else
                {
                    accountToLineItems.put(oli.Dealership__c, new List<OpportunityLineItem>());
                }
            }
        }

    }   
}