Sorry, there's some spaghetti in this code, but it meets our need. And we took shortcuts in some places (ex: don't check if the products are still valid).

The primary thing we were trying to achieve was to NOT clone the revenue schedule (I don't think very many customers use this feature). There are other differences - for example this does not put the user into edit mode of the new cloned but not saved record; This clones the record, saves it to the DB, then goes to that record.

public class OpportunityClone_Class { public Opportunity OldRecord {get; set; } public Opportunity NewRecord {get; set; } public Id OpptyIdOld {get; set; } public Boolean CloneOLI {get; set; } public Boolean forceError {get; set; } // this is the constructor public OpportunityClone_Class (){ //default forceError = False; //get the passed parm OpptyIdOld = System.currentPageReference().getParameters().get('id') ; if ( System.currentPageReference().getParameters().get('cloneoli') == '1') { CloneOLI = True ; } else { CloneOLI = False ; } } // this will be invoked from VForce page // in this kind of VForce page / controller, the constructor can not do the work public PageReference doit(){ //SPECIFICALLY EXCLUDE THESE FIELDS: //this gets the original record... OldRecord = [ Select AccountId , Amount , CampaignId , CloseDate , CreatedById , CreatedDate , CurrencyIsoCode , Description , Fiscal , FiscalQuarter , FiscalYear , ForecastCategory , HasOpportunityLineItem , Id , IsClosed , IsDeleted , IsWon , LastActivityDate , LastModifiedById , LastModifiedDate , LeadSource , Name , NextStep , OwnerId , Pricebook2Id , RecordTypeId , StageName , SystemModstamp , Type , (Select Id, OpportunityId , Description, PriceBookEntryId, Quantity, TotalPrice from OpportunityLineItems) from Opportunity o where id = :OpptyIdOld ] ; try { // clones the original (resets id) Opportunity NewRecord = OldRecord.clone(false, false); //over ride some fields NewRecord.Name = '[CLONED] ' + OldRecord.Name; NewRecord.Amount = 0; //for good or bad, not resetting the pricebookid - would cause lots of other problems //NewRecord.Pricebook2Id = null; //Closed deals can be any date. Non-Closed deals can only be future. if (!NewRecord.IsClosed) { NewRecord.CloseDate = system.today(); } //insert cloned record insert NewRecord; //this will cause a failure, i.e. the catch if (forceError) { insert NewRecord; } system.debug ('NEW RECORD: ' + newRecord.id + ' ' + newRecord.Name); if (CloneOLI == True) { OpportunityLineItem[] products = oldRecord.OpportunityLineItems.DeepClone(false); for (OpportunityLineItem each : products) { each.OpportunityId = NewRecord.id; each.TotalPrice = 0; //each.Description = '[CLONED]' + each.description; } insert products; } PageReference home = new PageReference('/' + newRecord.id); home.setRedirect(true); return home; } catch (Exception e) { system.debug ('ERROR! ' + e); ApexPages.Message myMsg = new ApexPages.Message(ApexPages.Severity.ERROR, 'Error creating new record: ' + e); ApexPages.addMessage(myMsg); return null; } } }