Ticket #1031 (closed issue: fixed)
Select in lazyloading mode should not use getItemIds()
| Reported by: | Artur Signell | Owned by: | Jani Laakso |
|---|---|---|---|
| Priority: | blocker | Milestone: | 4.0.4-rc4 |
| Component: | Server-side framework | Version: | 4.0.3 |
| Keywords: | Cc: | ||
| Known Issue description: | |||
| Hours estimate: | 1 | Deadline (dd.mm.yyyy): | |
| Known Issue version (since): | Known Issue title: | ||
| Hours done: | Depends to: | ||
| Affects documentation: | no | ||
| Known Issue workaround: | |||
| Affects release notes: | yes | Contract: | |
Description
Current paint method for lazyloading select uses getItemIds()
target.addAttribute("total",
(getItemIds() != null) ? getItemIds().size() : 0);
It should use container.size() instead of getItemIds().size(). A server-side lazy container does not want to return ALL itemids in the container but should be aware of how many items it contains:
target.addAttribute("total",
(items != null) ? items.size() : 0);
It is currently impossible to implement a server-side lazy loading select by extending select (paint method cannot be overriden since optionsStream and lazyLoadingPageLength are private).
Change History
Note: See
TracTickets for help on using
tickets.
