Add computed def as function

[JSteunou]

It might address #29 and be nice to have as it will allow to build the computed definition upon model context / options and facilitate inheritance

[alexbeletsky]

@JSteunou the code and tests looks good! but I'm trying to understand the practical meaning of this. can you give me example, where it's useful?

[JSteunou]

First of all is inheritance. Like Model#defaults or most of Marionette hash attributes being able to be defined those as function allow this

computed: function() {
    var parentComputed = ParentModel.prototype.computed.apply(this, arguments);
    parentComputed.anotherFields = { ... };
    return parentComputed;
}

Second, it allows immutable definition.

Third, it could allow to give parameters to computed, or at least read model properties to return a dynamic computed hash context aware.

[alexbeletsky]

@JSteunou I would ask you to update README file with the information about that feature. Once it's there and since it doesn't brake existing behaviour, I see no reason to do not merge it :) Good job.

[JSteunou]

README updated

[JSteunou]

Thank you, that was a fast PR :)

[JSteunou]

Do you think a new release could follow? I would love to use this new feature :D

[alexbeletsky]

Will release it soon!
On Aug 28, 2015 4:13 PM, "Jérôme Steunou" [email protected] wrote:

Do you think a new release could follow? I would love to use this new
feature :D

—
Reply to this email directly or view it on GitHub
#31 (comment)
.

[alexbeletsky]

› npm publish
+ [email protected]

[JSteunou]

Really nice, thank you!

[pebcakerror]

can you also tag the repo with 0.0.12 so that bower can use this feature