Upgrade to ExtJS 4.0.7 - Released 10/19/2011
[extjs.git] / docs / source / AbstractStore.html
index 00312c1..a9528f6 100644 (file)
@@ -1,29 +1,44 @@
-<!DOCTYPE html><html><head><title>Sencha Documentation Project</title><link rel="stylesheet" href="../reset.css" type="text/css"><link rel="stylesheet" href="../prettify.css" type="text/css"><link rel="stylesheet" href="../prettify_sa.css" type="text/css"><script type="text/javascript" src="../prettify.js"></script></head><body onload="prettyPrint()"><pre class="prettyprint"><pre><span id='Ext-data.AbstractStore'>/**
+<!DOCTYPE html>
+<html>
+<head>
+  <meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
+  <title>The source code</title>
+  <link href="../resources/prettify/prettify.css" type="text/css" rel="stylesheet" />
+  <script type="text/javascript" src="../resources/prettify/prettify.js"></script>
+  <style type="text/css">
+    .highlight { display: block; background-color: #ddd; }
+  </style>
+  <script type="text/javascript">
+    function highlight() {
+      document.getElementById(location.hash.replace(/#/, "")).className = "highlight";
+    }
+  </script>
+</head>
+<body onload="prettyPrint(); highlight();">
+  <pre class="prettyprint lang-js"><span id='Ext-data-AbstractStore'>/**
 </span> * @author Ed Spencer
- * @class Ext.data.AbstractStore
  *
- * &lt;p&gt;AbstractStore is a superclass of {@link Ext.data.Store} and {@link Ext.data.TreeStore}. It's never used directly,
- * but offers a set of methods used by both of those subclasses.&lt;/p&gt;
+ * AbstractStore is a superclass of {@link Ext.data.Store} and {@link Ext.data.TreeStore}. It's never used directly,
+ * but offers a set of methods used by both of those subclasses.
  * 
- * &lt;p&gt;We've left it here in the docs for reference purposes, but unless you need to make a whole new type of Store, what
+ * We've left it here in the docs for reference purposes, but unless you need to make a whole new type of Store, what
  * you're probably looking for is {@link Ext.data.Store}. If you're still interested, here's a brief description of what 
- * AbstractStore is and is not.&lt;/p&gt;
+ * AbstractStore is and is not.
  * 
- * &lt;p&gt;AbstractStore provides the basic configuration for anything that can be considered a Store. It expects to be 
+ * AbstractStore provides the basic configuration for anything that can be considered a Store. It expects to be 
  * given a {@link Ext.data.Model Model} that represents the type of data in the Store. It also expects to be given a 
- * {@link Ext.data.proxy.Proxy Proxy} that handles the loading of data into the Store.&lt;/p&gt;
+ * {@link Ext.data.proxy.Proxy Proxy} that handles the loading of data into the Store.
  * 
- * &lt;p&gt;AbstractStore provides a few helpful methods such as {@link #load} and {@link #sync}, which load and save data
+ * AbstractStore provides a few helpful methods such as {@link #load} and {@link #sync}, which load and save data
  * respectively, passing the requests through the configured {@link #proxy}. Both built-in Store subclasses add extra
  * behavior to each of these functions. Note also that each AbstractStore subclass has its own way of storing data - 
  * in {@link Ext.data.Store} the data is saved as a flat {@link Ext.util.MixedCollection MixedCollection}, whereas in
- * {@link Ext.data.TreeStore TreeStore} we use a {@link Ext.data.Tree} to maintain the data's hierarchy.&lt;/p&gt;
- * 
- * TODO: Update these docs to explain about the sortable and filterable mixins.
- * &lt;p&gt;Finally, AbstractStore provides an API for sorting and filtering data via its {@link #sorters} and {@link #filters}
- * {@link Ext.util.MixedCollection MixedCollections}. Although this functionality is provided by AbstractStore, there's a
- * good description of how to use it in the introduction of {@link Ext.data.Store}.
+ * {@link Ext.data.TreeStore TreeStore} we use a {@link Ext.data.Tree} to maintain the data's hierarchy.
  * 
+ * The store provides filtering and sorting support. This sorting/filtering can happen on the client side
+ * or can be completed on the server. This is controlled by the {@link Ext.data.Store#remoteSort remoteSort} and
+ * {@link Ext.data.Store#remoteFilter remoteFilter} config options. For more information see the {@link #sort} and
+ * {@link Ext.data.Store#filter filter} methods.
  */
 Ext.define('Ext.data.AbstractStore', {
     requires: ['Ext.util.MixedCollection', 'Ext.data.Operation', 'Ext.util.Filter'],
@@ -48,107 +63,113 @@ Ext.define('Ext.data.AbstractStore', {
     remoteSort  : false,
     remoteFilter: false,
 
-<span id='Ext-data.AbstractStore-cfg-proxy'>    /**
-</span>     * @cfg {String/Ext.data.proxy.Proxy/Object} proxy The Proxy to use for this Store. This can be either a string, a config
-     * object or a Proxy instance - see {@link #setProxy} for details.
+<span id='Ext-data-AbstractStore-cfg-proxy'>    /**
+</span>     * @cfg {String/Ext.data.proxy.Proxy/Object} proxy
+     * The Proxy to use for this Store. This can be either a string, a config object or a Proxy instance -
+     * see {@link #setProxy} for details.
      */
 
-<span id='Ext-data.AbstractStore-cfg-autoLoad'>    /**
-</span>     * @cfg {Boolean/Object} autoLoad If data is not specified, and if autoLoad is true or an Object, this store's load method
-     * is automatically called after creation. If the value of autoLoad is an Object, this Object will be passed to the store's
-     * load method. Defaults to false.
+<span id='Ext-data-AbstractStore-cfg-autoLoad'>    /**
+</span>     * @cfg {Boolean/Object} autoLoad
+     * If data is not specified, and if autoLoad is true or an Object, this store's load method is automatically called
+     * after creation. If the value of autoLoad is an Object, this Object will be passed to the store's load method.
+     * Defaults to false.
      */
     autoLoad: false,
 
-<span id='Ext-data.AbstractStore-cfg-autoSync'>    /**
-</span>     * @cfg {Boolean} autoSync True to automatically sync the Store with its Proxy after every edit to one of its Records.
-     * Defaults to false.
+<span id='Ext-data-AbstractStore-cfg-autoSync'>    /**
+</span>     * @cfg {Boolean} autoSync
+     * True to automatically sync the Store with its Proxy after every edit to one of its Records. Defaults to false.
      */
     autoSync: false,
 
-<span id='Ext-data.AbstractStore-property-batchUpdateMode'>    /**
-</span>     * Sets the updating behavior based on batch synchronization. 'operation' (the default) will update the Store's
+<span id='Ext-data-AbstractStore-property-batchUpdateMode'>    /**
+</span>     * @property {String} batchUpdateMode
+     * Sets the updating behavior based on batch synchronization. 'operation' (the default) will update the Store's
      * internal representation of the data after each operation of the batch has completed, 'complete' will wait until
      * the entire batch has been completed before updating the Store's data. 'complete' is a good choice for local
      * storage proxies, 'operation' is better for remote proxies, where there is a comparatively high latency.
-     * @property batchUpdateMode
-     * @type String
      */
     batchUpdateMode: 'operation',
 
-<span id='Ext-data.AbstractStore-property-filterOnLoad'>    /**
-</span>     * If true, any filters attached to this Store will be run after loading data, before the datachanged event is fired.
-     * Defaults to true, ignored if {@link #remoteFilter} is true
-     * @property filterOnLoad
-     * @type Boolean
+<span id='Ext-data-AbstractStore-property-filterOnLoad'>    /**
+</span>     * @property {Boolean} filterOnLoad
+     * If true, any filters attached to this Store will be run after loading data, before the datachanged event is fired.
+     * Defaults to true, ignored if {@link Ext.data.Store#remoteFilter remoteFilter} is true
      */
     filterOnLoad: true,
 
-<span id='Ext-data.AbstractStore-property-sortOnLoad'>    /**
-</span>     * If true, any sorters attached to this Store will be run after loading data, before the datachanged event is fired.
-     * Defaults to true, igored if {@link #remoteSort} is true
-     * @property sortOnLoad
-     * @type Boolean
+<span id='Ext-data-AbstractStore-property-sortOnLoad'>    /**
+</span>     * @property {Boolean} sortOnLoad
+     * If true, any sorters attached to this Store will be run after loading data, before the datachanged event is fired.
+     * Defaults to true, igored if {@link Ext.data.Store#remoteSort remoteSort} is true
      */
     sortOnLoad: true,
 
-<span id='Ext-data.AbstractStore-property-implicitModel'>    /**
-</span>     * True if a model was created implicitly for this Store. This happens if a fields array is passed to the Store's constructor
-     * instead of a model constructor or name.
-     * @property implicitModel
-     * @type Boolean
+<span id='Ext-data-AbstractStore-property-implicitModel'>    /**
+</span>     * @property {Boolean} implicitModel
+     * True if a model was created implicitly for this Store. This happens if a fields array is passed to the Store's
+     * constructor instead of a model constructor or name.
      * @private
      */
     implicitModel: false,
 
-<span id='Ext-data.AbstractStore-property-defaultProxyType'>    /**
-</span>     * The string type of the Proxy to create if none is specified. This defaults to creating a {@link Ext.data.proxy.Memory memory proxy}.
-     * @property defaultProxyType
-     * @type String
+<span id='Ext-data-AbstractStore-property-defaultProxyType'>    /**
+</span>     * @property {String} defaultProxyType
+     * The string type of the Proxy to create if none is specified. This defaults to creating a
+     * {@link Ext.data.proxy.Memory memory proxy}.
      */
     defaultProxyType: 'memory',
 
-<span id='Ext-data.AbstractStore-property-isDestroyed'>    /**
-</span>     * True if the Store has already been destroyed via {@link #destroyStore}. If this is true, the reference to Store should be deleted
+<span id='Ext-data-AbstractStore-property-isDestroyed'>    /**
+</span>     * @property {Boolean} isDestroyed
+     * True if the Store has already been destroyed. If this is true, the reference to Store should be deleted
      * as it will not function correctly any more.
-     * @property isDestroyed
-     * @type Boolean
      */
     isDestroyed: false,
 
     isStore: true,
 
-<span id='Ext-data.AbstractStore-cfg-storeId'>    /**
-</span>     * @cfg {String} storeId Optional unique identifier for this store. If present, this Store will be registered with 
-     * the {@link Ext.data.StoreManager}, making it easy to reuse elsewhere. Defaults to undefined.
+<span id='Ext-data-AbstractStore-cfg-storeId'>    /**
+</span>     * @cfg {String} storeId
+     * Unique identifier for this store. If present, this Store will be registered with the {@link Ext.data.StoreManager},
+     * making it easy to reuse elsewhere. Defaults to undefined.
      */
     
-<span id='Ext-data.AbstractStore-cfg-fields'>    /**
-</span>     * @cfg {Array} fields
+<span id='Ext-data-AbstractStore-cfg-fields'>    /**
+</span>     * @cfg {Object[]} fields
      * This may be used in place of specifying a {@link #model} configuration. The fields should be a 
      * set of {@link Ext.data.Field} configuration objects. The store will automatically create a {@link Ext.data.Model}
      * with these fields. In general this configuration option should be avoided, it exists for the purposes of
      * backwards compatibility. For anything more complicated, such as specifying a particular id property or
-     * assocations, a {@link Ext.data.Model} should be defined and specified for the {@link #model} config.
+     * assocations, a {@link Ext.data.Model} should be defined and specified for the {@link #model}
+     * config.
+     */
+
+<span id='Ext-data-AbstractStore-cfg-model'>    /**
+</span>     * @cfg {String} model
+     * Name of the {@link Ext.data.Model Model} associated with this store.
+     * The string is used as an argument for {@link Ext.ModelManager#getModel}.
      */
 
     sortRoot: 'data',
     
     //documented above
     constructor: function(config) {
-        var me = this;
+        var me = this,
+            filters;
         
         me.addEvents(
-<span id='Ext-data.AbstractStore-event-add'>            /**
+<span id='Ext-data-AbstractStore-event-add'>            /**
 </span>             * @event add
              * Fired when a Model instance has been added to this Store
              * @param {Ext.data.Store} store The store
-             * @param {Array} records The Model instances that were added
+             * @param {Ext.data.Model[]} records The Model instances that were added
              * @param {Number} index The index at which the instances were inserted
              */
             'add',
 
-<span id='Ext-data.AbstractStore-event-remove'>            /**
+<span id='Ext-data-AbstractStore-event-remove'>            /**
 </span>             * @event remove
              * Fired when a Model instance has been removed from this Store
              * @param {Ext.data.Store} store The Store object
@@ -157,52 +178,62 @@ Ext.define('Ext.data.AbstractStore', {
              */
             'remove',
             
-<span id='Ext-data.AbstractStore-event-update'>            /**
+<span id='Ext-data-AbstractStore-event-update'>            /**
 </span>             * @event update
-             * Fires when a Record has been updated
-             * @param {Store} this
+             * Fires when a Model instance has been updated
+             * @param {Ext.data.Store} this
              * @param {Ext.data.Model} record The Model instance that was updated
              * @param {String} operation The update operation being performed. Value may be one of:
-             * &lt;pre&gt;&lt;code&gt;
-               Ext.data.Model.EDIT
-               Ext.data.Model.REJECT
-               Ext.data.Model.COMMIT
-             * &lt;/code&gt;&lt;/pre&gt;
+             *
+             *     Ext.data.Model.EDIT
+             *     Ext.data.Model.REJECT
+             *     Ext.data.Model.COMMIT
              */
             'update',
 
-<span id='Ext-data.AbstractStore-event-datachanged'>            /**
+<span id='Ext-data-AbstractStore-event-datachanged'>            /**
 </span>             * @event datachanged
-             * Fires whenever the records in the Store have changed in some way - this could include adding or removing records,
-             * or updating the data in existing records
+             * Fires whenever the records in the Store have changed in some way - this could include adding or removing
+             * records, or updating the data in existing records
              * @param {Ext.data.Store} this The data store
              */
             'datachanged',
 
-<span id='Ext-data.AbstractStore-event-beforeload'>            /**
+<span id='Ext-data-AbstractStore-event-beforeload'>            /**
 </span>             * @event beforeload
-             * Event description
+             * Fires before a request is made for a new data object. If the beforeload handler returns false the load
+             * action will be canceled.
              * @param {Ext.data.Store} store This Store
-             * @param {Ext.data.Operation} operation The Ext.data.Operation object that will be passed to the Proxy to load the Store
+             * @param {Ext.data.Operation} operation The Ext.data.Operation object that will be passed to the Proxy to
+             * load the Store
              */
             'beforeload',
 
-<span id='Ext-data.AbstractStore-event-load'>            /**
+<span id='Ext-data-AbstractStore-event-load'>            /**
 </span>             * @event load
              * Fires whenever the store reads data from a remote data source.
              * @param {Ext.data.Store} this
-             * @param {Array} records An array of records
+             * @param {Ext.data.Model[]} records An array of records
              * @param {Boolean} successful True if the operation was successful.
              */
             'load',
+            
+<span id='Ext-data-AbstractStore-event-write'>            /**
+</span>             * @event write
+             * Fires whenever a successful write has been made via the configured {@link #proxy Proxy}
+             * @param {Ext.data.Store} store This Store
+             * @param {Ext.data.Operation} operation The {@link Ext.data.Operation Operation} object that was used in
+             * the write
+             */
+            'write',
 
-<span id='Ext-data.AbstractStore-event-beforesync'>            /**
+<span id='Ext-data-AbstractStore-event-beforesync'>            /**
 </span>             * @event beforesync
-             * Called before a call to {@link #sync} is executed. Return false from any listener to cancel the synv
+             * Fired before a call to {@link #sync} is executed. Return false from any listener to cancel the synv
              * @param {Object} options Hash of all records to be synchronized, broken down into create, update and destroy
              */
             'beforesync',
-<span id='Ext-data.AbstractStore-event-clear'>            /**
+<span id='Ext-data-AbstractStore-event-clear'>            /**
 </span>             * @event clear
              * Fired after the {@link #removeAll} method is called.
              * @param {Ext.data.Store} this
@@ -211,22 +242,21 @@ Ext.define('Ext.data.AbstractStore', {
         );
         
         Ext.apply(me, config);
+        // don't use *config* anymore from here on... use *me* instead...
 
-<span id='Ext-data.AbstractStore-property-removed'>        /**
+<span id='Ext-data-AbstractStore-property-removed'>        /**
 </span>         * Temporary cache in which removed model instances are kept until successfully synchronised with a Proxy,
          * at which point this is cleared.
          * @private
-         * @property removed
-         * @type Array
+         * @property {Ext.data.Model[]} removed
          */
         me.removed = [];
 
         me.mixins.observable.constructor.apply(me, arguments);
-        me.model = Ext.ModelManager.getModel(config.model || me.model);
+        me.model = Ext.ModelManager.getModel(me.model);
 
-<span id='Ext-data.AbstractStore-property-modelDefaults'>        /**
-</span>         * @property modelDefaults
-         * @type Object
+<span id='Ext-data-AbstractStore-property-modelDefaults'>        /**
+</span>         * @property {Object} modelDefaults
          * @private
          * A set of default values to be applied to every model instance added via {@link #insert} or created via {@link #create}.
          * This is used internally by associations to set foreign keys and other fields. See the Association classes source code
@@ -248,9 +278,17 @@ Ext.define('Ext.data.AbstractStore', {
 
             me.implicitModel = true;
         }
+        
+        // &lt;debug&gt;
+        if (!me.model) {
+            if (Ext.isDefined(Ext.global.console)) {
+                Ext.global.console.warn('Store defined with no model. You may have mistyped the model name.');
+            }
+        }
+        // &lt;/debug&gt;
 
         //ensures that the Proxy is instantiated correctly
-        me.setProxy(config.proxy || me.proxy || me.model.getProxy());
+        me.setProxy(me.proxy || me.model.getProxy());
 
         if (me.id &amp;&amp; !me.storeId) {
             me.storeId = me.id;
@@ -263,18 +301,18 @@ Ext.define('Ext.data.AbstractStore', {
         
         me.mixins.sortable.initSortable.call(me);        
         
-<span id='Ext-data.AbstractStore-property-filters'>        /**
-</span>         * The collection of {@link Ext.util.Filter Filters} currently applied to this Store
-         * @property filters
-         * @type Ext.util.MixedCollection
+<span id='Ext-data-AbstractStore-property-filters'>        /**
+</span>         * @property {Ext.util.MixedCollection} filters
+         * The collection of {@link Ext.util.Filter Filters} currently applied to this Store
          */
+        filters = me.decodeFilters(me.filters);
         me.filters = Ext.create('Ext.util.MixedCollection');
-        me.filters.addAll(me.decodeFilters(config.filters));
+        me.filters.addAll(filters);
     },
 
-<span id='Ext-data.AbstractStore-method-setProxy'>    /**
+<span id='Ext-data-AbstractStore-method-setProxy'>    /**
 </span>     * Sets the Store's Proxy by string, config object or Proxy instance
-     * @param {String|Object|Ext.data.proxy.Proxy} proxy The new Proxy, which can be either a type string, a configuration object
+     * @param {String/Object/Ext.data.proxy.Proxy} proxy The new Proxy, which can be either a type string, a configuration object
      * or an Ext.data.proxy.Proxy instance
      * @return {Ext.data.proxy.Proxy} The attached Proxy object
      */
@@ -301,7 +339,7 @@ Ext.define('Ext.data.AbstractStore', {
         return me.proxy;
     },
 
-<span id='Ext-data.AbstractStore-method-getProxy'>    /**
+<span id='Ext-data-AbstractStore-method-getProxy'>    /**
 </span>     * Returns the proxy currently attached to this proxy instance
      * @return {Ext.data.proxy.Proxy} The Proxy instance
      */
@@ -350,7 +388,7 @@ Ext.define('Ext.data.AbstractStore', {
         return me.proxy.update(operation, me.onProxyWrite, me);
     },
 
-<span id='Ext-data.AbstractStore-method-onProxyWrite'>    /**
+<span id='Ext-data-AbstractStore-method-onProxyWrite'>    /**
 </span>     * @private
      * Callback for any write Operation over the Proxy. Updates the Store's MixedCollection to reflect
      * the updates provided by the Proxy
@@ -398,7 +436,7 @@ Ext.define('Ext.data.AbstractStore', {
         return me.proxy.destroy(operation, me.onProxyWrite, me);
     },
 
-<span id='Ext-data.AbstractStore-method-onBatchOperationComplete'>    /**
+<span id='Ext-data-AbstractStore-method-onBatchOperationComplete'>    /**
 </span>     * @private
      * Attached as the 'operationcomplete' event listener to a proxy's Batch object. By default just calls through
      * to onProxyWrite.
@@ -407,7 +445,7 @@ Ext.define('Ext.data.AbstractStore', {
         return this.onProxyWrite(operation);
     },
 
-<span id='Ext-data.AbstractStore-method-onBatchComplete'>    /**
+<span id='Ext-data-AbstractStore-method-onBatchComplete'>    /**
 </span>     * @private
      * Attached as the 'complete' event listener to a proxy's Batch object. Iterates over the batch operations
      * and updates the Store's internal data MixedCollection.
@@ -437,7 +475,7 @@ Ext.define('Ext.data.AbstractStore', {
         // batch.retry();
     },
 
-<span id='Ext-data.AbstractStore-method-filterNew'>    /**
+<span id='Ext-data-AbstractStore-method-filterNew'>    /**
 </span>     * @private
      * Filter function for new records.
      */
@@ -446,24 +484,24 @@ Ext.define('Ext.data.AbstractStore', {
         return item.phantom === true &amp;&amp; item.isValid();
     },
 
-<span id='Ext-data.AbstractStore-method-getNewRecords'>    /**
+<span id='Ext-data-AbstractStore-method-getNewRecords'>    /**
 </span>     * Returns all Model instances that are either currently a phantom (e.g. have no id), or have an ID but have not
      * yet been saved on this Store (this happens when adding a non-phantom record from another Store into this one)
-     * @return {Array} The Model instances
+     * @return {Ext.data.Model[]} The Model instances
      */
     getNewRecords: function() {
         return [];
     },
 
-<span id='Ext-data.AbstractStore-method-getUpdatedRecords'>    /**
+<span id='Ext-data-AbstractStore-method-getUpdatedRecords'>    /**
 </span>     * Returns all Model instances that have been updated in the Store but not yet synchronized with the Proxy
-     * @return {Array} The updated Model instances
+     * @return {Ext.data.Model[]} The updated Model instances
      */
     getUpdatedRecords: function() {
         return [];
     },
 
-<span id='Ext-data.AbstractStore-method-filterUpdated'>    /**
+<span id='Ext-data-AbstractStore-method-filterUpdated'>    /**
 </span>     * @private
      * Filter function for updated records.
      */
@@ -472,7 +510,10 @@ Ext.define('Ext.data.AbstractStore', {
         return item.dirty === true &amp;&amp; item.phantom !== true &amp;&amp; item.isValid();
     },
 
-    //returns any records that have been removed from the store but not yet destroyed on the proxy
+<span id='Ext-data-AbstractStore-method-getRemovedRecords'>    /**
+</span>     * Returns any records that have been removed from the store but not yet destroyed on the proxy.
+     * @return {Ext.data.Model[]} The removed Model instances
+     */
     getRemovedRecords: function() {
         return this.removed;
     },
@@ -481,11 +522,11 @@ Ext.define('Ext.data.AbstractStore', {
 
     },
 
-<span id='Ext-data.AbstractStore-method-decodeFilters'>    /**
+<span id='Ext-data-AbstractStore-method-decodeFilters'>    /**
 </span>     * @private
      * Normalizes an array of filter objects, ensuring that they are all Ext.util.Filter instances
-     * @param {Array} filters The filters array
-     * @return {Array} Array of Ext.util.Filter objects
+     * @param {Object[]} filters The filters array
+     * @return {Ext.util.Filter[]} Array of Ext.util.Filter objects
      */
     decodeFilters: function(filters) {
         if (!Ext.isArray(filters)) {
@@ -539,7 +580,7 @@ Ext.define('Ext.data.AbstractStore', {
 
     },
     
-<span id='Ext-data.AbstractStore-method-sync'>    /**
+<span id='Ext-data-AbstractStore-method-sync'>    /**
 </span>     * Synchronizes the Store with its Proxy. This asks the Proxy to batch together any new, updated
      * and deleted records in the store, updating the Store's internal representation of the records
      * as each operation completes.
@@ -573,7 +614,7 @@ Ext.define('Ext.data.AbstractStore', {
     },
 
 
-<span id='Ext-data.AbstractStore-method-getBatchListeners'>    /**
+<span id='Ext-data-AbstractStore-method-getBatchListeners'>    /**
 </span>     * @private
      * Returns an object which is passed in as the listeners argument to proxy.batch inside this.sync.
      * This is broken out into a separate function to allow for customisation of the listeners
@@ -600,9 +641,9 @@ Ext.define('Ext.data.AbstractStore', {
         return this.sync.apply(this, arguments);
     },
 
-<span id='Ext-data.AbstractStore-method-load'>    /**
+<span id='Ext-data-AbstractStore-method-load'>    /**
 </span>     * Loads the Store using its configured {@link #proxy}.
-     * @param {Object} options Optional config object. This is passed into the {@link Ext.data.Operation Operation}
+     * @param {Object} options (optional) config object. This is passed into the {@link Ext.data.Operation Operation}
      * object that is created and then sent to the proxy's {@link Ext.data.proxy.Proxy#read} function
      */
     load: function(options) {
@@ -627,7 +668,7 @@ Ext.define('Ext.data.AbstractStore', {
         return me;
     },
 
-<span id='Ext-data.AbstractStore-method-afterEdit'>    /**
+<span id='Ext-data-AbstractStore-method-afterEdit'>    /**
 </span>     * @private
      * A model instance should call this method on the Store it has been {@link Ext.data.Model#join joined} to.
      * @param {Ext.data.Model} record The model instance that was edited
@@ -642,7 +683,7 @@ Ext.define('Ext.data.AbstractStore', {
         me.fireEvent('update', me, record, Ext.data.Model.EDIT);
     },
 
-<span id='Ext-data.AbstractStore-method-afterReject'>    /**
+<span id='Ext-data-AbstractStore-method-afterReject'>    /**
 </span>     * @private
      * A model instance should call this method on the Store it has been {@link Ext.data.Model#join joined} to..
      * @param {Ext.data.Model} record The model instance that was edited
@@ -651,7 +692,7 @@ Ext.define('Ext.data.AbstractStore', {
         this.fireEvent('update', this, record, Ext.data.Model.REJECT);
     },
 
-<span id='Ext-data.AbstractStore-method-afterCommit'>    /**
+<span id='Ext-data-AbstractStore-method-afterCommit'>    /**
 </span>     * @private
      * A model instance should call this method on the Store it has been {@link Ext.data.Model#join joined} to.
      * @param {Ext.data.Model} record The model instance that was edited
@@ -698,21 +739,24 @@ Ext.define('Ext.data.AbstractStore', {
 
     getById: Ext.emptyFn,
     
-<span id='Ext-data.AbstractStore-property-removeAll'>    /**
+<span id='Ext-data-AbstractStore-method-removeAll'>    /**
 </span>     * Removes all records from the store. This method does a &quot;fast remove&quot;,
      * individual remove events are not called. The {@link #clear} event is
      * fired upon completion.
+     * @method
      */
     removeAll: Ext.emptyFn,
     // individual substores should implement a &quot;fast&quot; remove
     // and fire a clear event afterwards
 
-<span id='Ext-data.AbstractStore-method-isLoading'>    /**
+<span id='Ext-data-AbstractStore-method-isLoading'>    /**
 </span>     * Returns true if the Store is currently performing a load operation
      * @return {Boolean} True if the Store is currently loading
      */
     isLoading: function() {
-        return this.loading;
+        return !!this.loading;
      }
 });
-</pre></pre></body></html>
\ No newline at end of file
+</pre>
+</body>
+</html>