Changes between Initial Version and Version 1 of Ticket #14163, comment 3


Ignore:
Timestamp:
Oct 28, 2011, 6:51:48 PM (11 years ago)
Author:
ben hockey
Comment:

Legend:

Unmodified
Added
Removed
Modified
  • Ticket #14163, comment 3

    initial v1  
    1 a possible fix is this:
     1a possible fix is this (UPDATED):
    22
    33{{{
     
    55Index: StatefulModel.js
    66===================================================================
    7 --- StatefulModel.js    (revision 26917)
     7--- StatefulModel.js    (revision 26911)
    88+++ StatefulModel.js    (working copy)
    9 @@ -371,9 +371,7 @@
     9@@ -370,10 +370,8 @@
     10                        //              the data structure.
    1011                        // tags:
    1112                        //              private
    12                         var data = (args && args.data) || this.data;
     13-                       var data = (args && args.data) || this.data;
    1314-                       if(data){
    1415-                               this._createModel(data);
    1516-                       }
     17+                       var data = (args && "data" in args) ? args.data : this.data;
    1618+                       this._createModel(data);
    1719                },
    1820
    1921                //////////////////////// PRIVATE METHODS ////////////////////////
     22@@ -385,7 +383,7 @@
     23                        //              The input for the model, as a plain JavaScript object.
     24                        // tags:
     25                        //              private
     26-                       if(lang.isObject(obj) && !(obj instanceof Date) && !(obj instanceof RegExp)){
     27+                       if(lang.isObject(obj) && !(obj instanceof Date) && !(obj instanceof RegExp) && obj !== null){
     28                                for(var x in obj){
     29                                        var newProp = new StatefulModel({ data : obj[x] });
     30                                        this.set(x, newProp);
    2031}}}
    2132