Proxy SQL not persisted fields bug

[alanthales]

REQUIRED INFORMATION


Senha Touch version tested:

• 2.1.0
• 2.2.0rc

Browser versions tested against:

• Chrome 25

Description:

• SQL proxy create not persisted fields on database.

Steps to reproduce the problem:

• Create a model configured to proxy SQL and with a field seted to persist: false.
• Create a dataview to show this model.

The result that was expected:

• Don't create "No persisted fields" in database.

The result that occurs instead:

• "No persisted" fields being created on database.

Test Case:

• Not provided.

HELPFUL INFORMATION


Possible fix:

Ext.define('Appname.util.proxySQL', {
	override: 'Ext.data.proxy.SQL',
	
    getSchemaString: function() {
        var me = this,
            schema = [],
            model = me.getModel(),
            idProperty = model.getIdProperty(),
            fields = model.getFields().items,
            uniqueIdStrategy = me.getUniqueIdStrategy(),
            ln = fields.length,
            i, field, type, name;


        for (i = 0; i < ln; i++) {
            field = fields[i];
            type = field.getType().type;
            name = field.getName();
            
            if (!field.getPersist()) {
            	continue;
            }


            if (name === idProperty) {
                if (uniqueIdStrategy) {
                    type = me.convertToSqlType(type);
                    schema.unshift(idProperty + ' ' + type);
                } else {
                    schema.unshift(idProperty + ' INTEGER PRIMARY KEY AUTOINCREMENT');
                }
            } else {
                type = me.convertToSqlType(type);
                schema.push(name + ' ' + type);
            }
        }


        return schema.join(', ');
    }
});

[mitchellsimoens]

Can I get a simple test case to reproduce?

[alanthales]

Can I get a simple test case to reproduce?

Ext.define('MyModel', {
    extend: 'Ext.data.Model',

    config: {
        fields : [
        { name: 'code', type: 'int' },
        { name: 'value', type: 'float', persist: false }
    ]
    }
});

Ext.Viewport.add({
    xtype: 'dataview',
    itemTpl: 'code',
    store: {
    model: 'MyModel',
    proxy: 'sql',
    autoLoad: true
    }
});