12.0p ListGrid with showAllRecords true unnecessary fetch

[Blama]

Hi Isomorphic,

please see this online testcase (v12.0p_2019-08-11):

isc.ListGrid.create({
    ID:"dsListGrid", 
    width: "100%",
    height: "100%",
    autoFetchData: true,
    dataSource: "supplyItem",
    //drawAheadRatio: 5.0,
    //showRowNumbers: true,
    //initialCriteria: { fieldName: "SKU", operator: "startsWith", value: "1" }
    showAllRecords: true,
});

When you run the sample, you'll see two fetches, one for 0...75, another for 75...3959. I'd expect only one unbounded fetch here.

This is very minor for me, I only found it experimenting with it and drawAheadRatio.

Additionally, the showAllRecords-docs speak about "showAllRows", which is not the name of the property, but the name of a GridRenderer property. IMHO it should either say showAllRecords, or link to GridRenderer, if this is somehow related.

Best regards
Blama

[Isomorphic]

By its nature, showAllRows: true is incompatible with ListGrid.dataFetchMode: "paged". In the nightly builds for SC 12.0 and newer releases dated 2019-08-15 and beyond, a warning will be logged for such a configuration and the docs have been updated to reflect that it's not supported.

[Blama]

Hi Isomorphic,

ok, perfect. That makes sense. I also saw the docs change to ListGrid.showAllRecords, thanks. You do have a typo in "showAllRecordss" now, though.

Best regards
Blama