Results 1 to 2 of 2

Thread: [CLOSED] BeanModelReader has wrong generic type

    Thank you for reporting this bug. We will make it our priority to review this report.
  1. #1
    Ext GWT Premium Member
    Join Date
    Sep 2010
    Posts
    2
    Vote Rating
    0
      0  

    Default [CLOSED] BeanModelReader has wrong generic type

    BeanModelReader has the following signature :
    Code:
    BeanModelReader implements DataReader<ListLoadResult<ModelData>>
    however it will only ever contain BeanModel instances (or an empty list).

    It should have the signature : BeanModelReader implements DataReader<ListLoadResult<BeanModel>>

    This means the following code is needed by the client :

    ListLoadResult<BeanModel> dataListLoadResult = ((ListLoadResult<BeanModel>)(ListLoadResult)reader.read(null, result));

    which is pretty messy.
    Last edited by andrew_l; 20 Sep 2010 at 8:49 PM. Reason: extra info

  2. #2
    Sencha Premium Member
    Join Date
    Sep 2007
    Posts
    13,976
    Vote Rating
    131
      0  

    Default

    The generics on the dataloading api are overall wrong. Also we cannot change this in any minor version update.

Similar Threads

  1. [FIXED] 2.0.4 - NumberField documentation wrong - not generic
    By bjwaski in forum Ext GWT: Bugs (2.x)
    Replies: 1
    Last Post: 18 Nov 2009, 8:00 AM
  2. [CLOSED] [2.0m] BaseTreeLoader nit work with BeanModelReader
    By Webflash in forum Ext GWT: Bugs (2.x)
    Replies: 1
    Last Post: 12 May 2009, 2:28 AM
  3. Replies: 5
    Last Post: 2 Apr 2009, 12:16 AM
  4. BeanModelReader::read() assumes all beans are of same concrete type
    By NitrousDigital in forum Ext GWT: Bugs (1.x)
    Replies: 0
    Last Post: 23 Feb 2009, 9:41 AM
  5. [CLOSED] LoadListener class should be generic
    By Cputerace in forum Ext GWT: Bugs (1.x)
    Replies: 1
    Last Post: 17 Feb 2009, 10:49 AM

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •