scope of this

[philfreo]

Seems a little strange to me that in the getter functions this refers to the model's attributes but in setters this refers to the model itself. Does there really need to be this inconsistency? It makes calling a method on the model hard in the getters.

[asciidisco]

Good catch.
I´ve came across this issue myself in the past week.
Next version of Mutators will break the current behaviour and changes the scope of the getter
functions to the model itself.

Thx, for reporting.

[asciidisco]

Btw.: Will be available at the end of the week or so, I´ll have to do some paid work first ;)

[philfreo]

for what it's worth, i wrote this to use instead of your library right now

https://gist.github.com/3056724

[asciidisco]

Had a quick look,
there are a couple of good ideas.
Is it okay if i 'steal' some of the lines (mentioning their origin) for a future version of mutators?

[philfreo]

of course, that's why i shared :)

[asciidisco]

Thank you.
I´ll leave the ticket open, so I won´t forget about it.

[mohitmayank]

hi, is there any update on this.

[philfreo]

@mohitmayank if this plugin's not updated, I suggest checking out that gist -- I've been using it successfully now for a while. probably will make my own plugin out of it at some point if there's not another good one

[philfreo]

although looks like it may have been fixed here in 57d5f00

[mohitmayank]

@philfreo thanks for your reply. i had a look at your gist earlier as well. but felt lazy at the time. wishful thinking :). anyways i am using your code now. i have also added a toJSON method to it.
Let me know if you roll a plugin with this. I'll help.
https://gist.github.com/mohitmayank/5038191

[rubiii]

@philfreo isn't this fixed with the changeset you linked to?
using v0.3.1, the scope for both setters and getters is the model.

[asciidisco]

Changes are in, preparing the next release atm.