Model Field: persist:false does not work

[badgerb1]

Hi guys,

If I set a field to persist:false it still gets persisted to the localstorage.

Thanks
Bob

[mitchellsimoens]

Thanks for the report. I have made a test case for you:

new Ext.data.Store({
    fields : [
        { name : 'text' },
        { name : 'foo', persist : false }
    ],

    data : [
        {
            text : 'Test',
            foo  : 'bar'
        }
    ],

    proxy : {
        type : 'localstorage',
        id   : 'test'
    }
}).sync();

[martinvidec]

this works for me in sencha touch 2.0.0:

Ext.define('My.override.of.ext.data.proxy.WebStorage', {
    override: 'Ext.data.proxy.WebStorage',
    
    setRecord: function(record, id){
        if (id) {
            record.setId(id);
        } else {
            id = record.getId();
        }


        var me = this,
            rawData = record.getData(),
            data    = {},
            Model   = me.getModel(),
            fields  = Model.getFields().items,
            length  = fields.length,
            i = 0,
            field, name, obj, key, dateFormat;


        for (; i < length; i++) {
            field = fields[i];
            name  = field.getName();
            
            if(field.getPersist() === false){
                continue;
            }
            
            if (typeof field.getEncode() == 'function') {
                data[name] = field.getEncode()(rawData[name], record);
            } else {
                if (field.getType().type == 'date' && Ext.isDate(rawData[name])) {
                    dateFormat = field.getDateFormat();
                    if (dateFormat) {
                        data[name] = Ext.Date.format(rawData[name], dateFormat);
                    } else {
                        data[name] = rawData[name].getTime();
                    }
                } else {
                    data[name] = rawData[name];
                }
            }
        }


        obj = me.getStorageObject();
        key = me.getRecordKey(id);


        //keep the cache up to date
        me.cache[id] = record;


        //iPad bug requires that we remove the item before setting it
        obj.removeItem(key);
        obj.setItem(key, Ext.encode(data));
    }
});

[farion]

Currently the id is also saved to the value. ExtJS does not save the id.

What behavior is the correct one?

[TommyMaintz]

This has been fixed for the next release. Thanks for the report and the fix!