Autobind decorator

I want to use autobind decorator for components.

This https://github.com/andreypopp/autobind-decorator/blob/master/src/index.js work fine with React (method binding). But have issue with RHL. Decorator bind method to instance. When new RHL add new prototype to prototype chain nothing happened, because method already in instance and new method in proto not visible when getter called.

I use ugly hack and check in getter if (this.props) to determine is this – instance.

It is not an issue with RHL or decorator itself. I try to find nice solution.

[yangmillstheory]

I don’t think this is fully working with react-hot-loader@3.0.0-beta.2 yet.

Small test commit here.

In the demo above there’s 3 steps:

1. without @autobind on onClick, click the button. This doesn’t have the right context - this is null.
2. add @autobind to onClick - this is recognized by RHL and correct logs the right state in the console (this is bound to the Counter instance).
3. remove @autobind, this is still bound to the Counter instance (you would expect it not to be).

RHL doesn’t remove the binding when using fat arrows to declare the method either.

Hope that’s helpful, would really love to use this with RHL!

[slonoed]

For me this issue not actual now. I switch to arrow functions in class definition.

class Cmp extends Component {
  myBindedFunc = () => {
     this.callSmth();
  }
}

Can I close this issue?