[SalesForce] Collection size 33,046 exceeds maximum size of 1,000 error in vf page

I have an issue here, where i need to populate geography__c list on vf page. I am getting the below error . I can't use Limit 1000 as i need to display all the records in vf page. Is there any workaround for this issue.

Collection size 33,046 exceeds maximum size of 1,000

Vf Page –

<apex:actionregion >
                    <apex:pageBlockSectionItem >
                        <apex:outputLabel value="{!$ObjectType.RFP__c.Fields.Country__c.Label}" style="width:87%; float:right;height:0px;font-weight:bold;font-family:Arial;font-size:11px;color:#2F4F4F;"/>

                        <apex:outputPanel id="countries" layout="block"  styleClass="requiredInput" style="width:82%; float:right;">
                            <apex:outputPanel id="countr" layout="block"  styleClass="requiredBlock"/>
                            <apex:selectList value="{!discountScheduleID}" size="1" styleClass=" chzn-select" >
                                <apex:actionSupport event="onchange" action="{!FindAllName}" rerender="geographies" oncomplete="renderChosen()" />
                                <apex:selectOptions value="{!dept}" />
                            </apex:selectList> 
                        </apex:outputPanel>

                    </apex:pageBlockSectionItem>

                    <apex:pageBlockSectionItem >
                        <apex:outputLabel value="{!$ObjectType.RFP__c.Fields.Type__c.Label}" style="width:87%; float:right;height:5px;font-weight:bold;font-family:Arial;font-size:11px;color:#2F4F4F;"/>

                        <apex:outputPanel id="type" layout="block"  styleClass="requiredInput" style="width:82%; float:right;">
                            <apex:outputPanel id="type1" layout="block"  styleClass="requiredBlock"/>
                            <apex:selectList value="{!TypeID}" size="1" styleClass=" chzn-select" >

                                <apex:selectOptions value="{!Type}" />
                            </apex:selectList> 
                        </apex:outputPanel>

                    </apex:pageBlockSectionItem>


                    <apex:pageBlockSectionItem >
                        <apex:outputLabel value="{!$ObjectType.RFP__c.Fields.Geography__c.Label}"  style="width:87%; float:right;height:8px;font-weight:bold;font-family:Arial;font-size:11px;color:#2F4F4F;"/>
                        <apex:outputPanel id="geographies" layout="block"  styleClass="requiredInput" style="width:82%; float:right;">
                            <apex:outputPanel id="geos" layout="block"  styleClass="requiredBlock"/>

                            <apex:selectList value="{!selectedGeographyIds }" multiselect="true" id="selectedGeographies" styleClass="fullWidth chzn-select" >

                                <apex:selectOptions value="{!AllName}" id="movieTextBox"></apex:selectOptions>

                            </apex:selectList>

                            <apex:outputPanel layout="block" styleClass="errorMsg" >
                                <apex:outputText value="{!errorMap['selectedGeographies']}" 
                                                 escape="false" rendered="{!errorMap['selectedGeographies'] != ''}" />
                            </apex:outputPanel>
                        </apex:outputPanel>
                    </apex:pageBlockSectionItem>
                </apex:actionregion>

Apex Class –

public void FindAllName()
    {
        system.debug('@@@@@@ Entered');
        AllName= new List<SelectOption>();
        AllName= getName();
    }  

    public List<ID> holdData(){

        List<ID> rfpGeographiesToInsert = new List<ID>();
        for (ID selectedGeographyId : selectedGeographyIds) {
            rfpGeographiesToInsert.add(selectedGeographyId);

            system.debug('#############' +  rfpGeographiesToInsert);    
        }
        return rfpGeographiesToInsert;
    } 


    public List<SelectOption> getName()
    {


        Set<String> sProj = new Set<String>();
        List<Geography__c > tohold = new List<Geography__c >();

        List<ID> lst = new List<ID>(); 
        lst = holdData();
        system.debug('@@@@@@' + lst);
        if(discountScheduleID != null )
        {
            system.debug('1111111');   
            if(lst.size()==0)
            {
                system.debug('2222222');  
                for(Geography__c Proj  :[select Id, Canonical_Name__c, Type__c from Geography__c where status__c = 'active' AND Country__c =: discountScheduleID AND Type__c =: TypeID order by Canonical_Name__c] )
                {
                    system.debug('3333333'); 
                    options.add(new SelectOption(Proj.ID,Proj.Canonical_Name__c));
                    //strGeographix += Proj.Canonical_Name__c + ', ';
                    //errorMap.put('selectedGeographies',Proj.Canonical_Name__c);

                }
            }
            else
            {options.clear();

             for(Geography__c Proj  :[select Id, Canonical_Name__c, Type__c from Geography__c where status__c = 'active' AND Country__c =: discountScheduleID   and Id not in:lst order by Canonical_Name__c] )
             {
                 system.debug('4444444');
                 options.add(new SelectOption(Proj.ID,Proj.Canonical_Name__c));
                 //strGeographix += Proj.Canonical_Name__c + ', ';
                 // errorMap.put('selectedGeographies',Proj.Canonical_Name__c);

             }

             for(Geography__c Proj1  :[select Id, Canonical_Name__c, Type__c from Geography__c where status__c = 'active'  AND Id in:lst order by Canonical_Name__c])
             {
                 options.add(new SelectOption(Proj1.ID,Proj1.Canonical_Name__c));

             }

            }
        }        
        return options;

    }
}

Regards

Best Answer

You are trying to display 33,046 values in a picklist/drop down list I think. I don't think that is a very wise idea even from a usability point of view.

Either try to break the picklist up by geographies, or use a search like feature to fetch the required values.

Revision

public with sharing class Checkbox_Class {
    // The list of assets we're viewing
    public AssetsWrapper[] assets { get; set; }
    // The total count of assets selected
    public Integer selectedAssets { get; set; }

    // Initialize our data here
    public Checkbox_Class() {
        assets = new AssetWrapper[0];
        selectedAssets = null;
        refreshAssetList();
    }

    // Get the currently selected assets and update counter
    Asset[] getSelectedAssetList() {
        Asset[] results = new Asset[0];
        for(AssetWrapper item: assets) {
            if(item.selected) {
                results.add(item.acc);
            }
        }
        selectedAssets = results.isEmpty()? null: results.size();
        return results;
    }

    // Assumption: We'll be returning a page reference somewhere later.
    public PageReference getSelected() {
        getSelectedAssetList();
        return null;
    }

    // Query the database for a list of assets meeting the criteria
    public void refreshAssetList() {
        Map<Id, Asset> selectedAssets = new Map<Id, Asset>(getSelectedAssetList());
        assets.clear();
        for(Asset record: [SELECT   Id, SerialNumber, Name, Retired__c, Returned__c, Quantity, RMAed__c
                           FROM     Asset
                           WHERE    Retired__c = false AND Returned__c = false
                           ORDER BY CreatedDate DESC    
                           LIMIT    1000]) {
            assets.add(new AssetsWrapper(
                selectedAssets.containsKey(record.Id)?selectedAssets.get(record.Id):record,
                selectedAssets.containsKey(record.Id)
            ));
        }
    }

    // Wrapper class
    public class AssetsWrapper {
        public Asset acc { get; set; }
        public Boolean selected { get; set; }
        public AssetsWrapper(Asset record, Boolean isChecked) {
            acc = record;
            selected = isChecked;
        }
    }
}

Helpful Tips

Getters and setters may be called more than once per page transaction, and in particular, the getAssets function originally posted kept piling on records to the master list 100 at a time (duplicates). This is why the error occurred at exactly 1100 records. Instead, initialize lists only once, and refer to them as necessary.

The refresh function provided here could be linked to a button to refresh the list from scratch (e.g. in the event new assets were added), while still retaining the old checkboxes, if any. This is generally useful especially if the users need updated information.

In general, avoid returning a pagereference if the only possible value is null. In longer functions, this helps visually identify the difference between an action function that will redirect to a new page versus one that will never redirect to a new page. I left that function as is under the assumption that "Create Quote" would probably go to a new page at some point.

Don't use member variables when local variables and parameters will suffice. Each member variable consumes memory, which in turn uses view state (unless it is transient, which is a useful, but often confusing language feature). You don't want your view state getting any larger than necessary, because it slows down loading times and can eventually run afoul of the governor limits.

You can make the page "Read Only" if you want to increase your limits from 1000 items in a list to 10000. This only works if you plan on not creating any data on this page.

You could have used a StandardSetController to emulate all of this functionality, with a maximum query of 2000 records deep, and offer pagination (first, next, last, previous pages), and checkbox capabilities.

Try to avoid calling a variable "x", unless it's in a local loop somewhere that won't be used outside that area. A member variable of "x" is always a bad idea, because it's not apparent what "x" is. Is it the X in an X/Y coordinate? It is a location on the screen? Is it just a simple counter? Does it contain the answer to life, the universe, and everything? Obviously, in this code, it was easy to figure out, but a 10000 line class would be a different story, or even worse, a class that is later subclassed and that subclass uses X to communicate to functions in the main class.

[SalesForce] String length exceeds maximum: 6000000

Unfortunately you're SOL when it comes to that sort of limit, your only route is going to be moving the operation off Salesforce.

One option would be to create a Heroku app with a single endpoint that takes the attachment Id. The endpoint can then retrieve the attachment and then upload it to the 3rd party server.

Best Answer

Related Solutions

[SalesForce] Collection size 1,100 exceeds maximum size of 1,000. Limited the query to 100 items but still getting the error

Revision

Helpful Tips

[SalesForce] String length exceeds maximum: 6000000

Related Topic