Results 1 to 2 of 2

Thread: [OPEN-1226] and code and documentation issues

    Thank you for reporting this bug. We will make it our priority to review this report.
  1. #1

    Default [OPEN-1226] and code and documentation issues

    The DataReader class uses a 'meta' object instead of the standard ExtJS convention of a config object. The configuration options are not fully documented; DataReader references idProperty in the extractData function.

    DataReader uses instanceof XmlReader/JsonReader in the extractData function that doesn't seem appropriate for an abstract parent class.

    The ExtJS documentation for DataReader does not mention the read or readResponse methods that are used by HttpProxy when the Store data is read from a remote URL. This led to quite a bit of frustration and source code spelunking when attempting to create my own DataReader implementation and realizing that the documented functions are never called when loading data remotely.

    This looks like a dark corner of the ExtJS code, so I wanted to shed some light on these issues while they were fresh in my mind. Thanks for the hard work producing a great library!

  2. #2
    Sencha User Condor's Avatar
    Join Date
    Mar 2007
    The Netherlands


    These are known issues.

    The fact that DataReader doesn't has a config object parameter in the constructor is for backward compatibility (can't be changed without breaking existing code).

    The documentation for DataReader is indeed a bit sparse, but since DataReader is an abstract class that you won't instantiate, I don't see this as a big problem.

    I completely agree about the instanceof. I would have made the property names configurable instead of hardcoding the descending classes.

    In Ext 4 the complete data package will be rewritten and merged with Sencha Touch (which has the advantage that you can use a single server implementation and client side data configuration for both desktop and mobile development).

Similar Threads

  1. Replies: 8
    Last Post: 20 Jun 2011, 6:59 AM
  2. [OPEN-1134] bug
    By XASD in forum Ext 3.x: Bugs
    Replies: 2
    Last Post: 19 Jul 2010, 2:01 AM
  3. Documentation for
    By Devany in forum Ext 1.x: Help & Discussion
    Replies: 2
    Last Post: 16 Apr 2007, 12:14 AM
  4. Documentation for
    By sfwalter in forum Ext 2.x: Help & Discussion
    Replies: 1
    Last Post: 3 Apr 2007, 9:23 AM
  5. any documentation available for the new ?
    By neongrau in forum Ext 2.x: Help & Discussion
    Replies: 0
    Last Post: 7 Mar 2007, 6:58 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