Serialization #611

Merged
merged 1 commit into from Dec 20, 2013

Conversation

Projects
None yet
3 participants
@daffl
Contributor

daffl commented Dec 20, 2013

can.Model doesn't use serialize object while saving and using can.Model.Queue plugin

@justinbmeyer

This comment has been minimized.

Show comment
Hide comment
@justinbmeyer

justinbmeyer Dec 14, 2013

Contributor

Please provide an example of what you mean.

Contributor

justinbmeyer commented Dec 14, 2013

Please provide an example of what you mean.

@szmeku

This comment has been minimized.

Show comment
Hide comment
@szmeku

szmeku Dec 14, 2013

CustomModel = can.Model.extend({ 
  attributes: {
    variable: 'type'
  },
  serialize: {
    type: function(val){
      return val + ' some suffix';
    }
  }
},{})

c = new CustomModel({ variable: 'lkfjsal' });

c.save();

this doesn't save c with variable === 'lkfjsal some suffix' when i'll include can.Model.Queue sources

szmeku commented Dec 14, 2013

CustomModel = can.Model.extend({ 
  attributes: {
    variable: 'type'
  },
  serialize: {
    type: function(val){
      return val + ' some suffix';
    }
  }
},{})

c = new CustomModel({ variable: 'lkfjsal' });

c.save();

this doesn't save c with variable === 'lkfjsal some suffix' when i'll include can.Model.Queue sources

@szmeku

This comment has been minimized.

Show comment
Hide comment

szmeku commented Dec 20, 2013

thx

@daffl daffl reopened this Dec 20, 2013

daffl added a commit that referenced this pull request Dec 20, 2013

@daffl daffl merged commit 8978cee into master Dec 20, 2013

1 check passed

default The Travis CI build passed
Details

@daffl daffl deleted the queue-serialize-fix branch Dec 20, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment