FormItem doesn't detect control input inside

Environment

• antd version: 2.7.4
• OS and its version: OS X 10.12.3
• Browser and its version: Chrome 56

What did you do? Please provide steps to re-produce your problem.

We wrap our form fields into their own component which contain their validation logic. Example

UsernameInput.js

class UsernameInput extends Component {
  render() {
    return this.props.form.getFieldDecorator(this.props.id, {
      rules: [...]
    })(InputComponent);
  }
}

And we want to use them in our forms like this

// imports here

class CustomForm {
  render() {
    return (<Form>
      <Form.Item>
        <UsernameInput form={this.props.form} />
      </Form.Item>
    </Form>);
  }
}

export default Form.create()(CustomForm);

This doesn’t work because FormItem.getControls doesn’t detect the component as a control. Just sees the wrapper as UsernameInput.

What do you expected?

FormItem to detect UsernameInput and work as any input control. Show error messages, feedback, helps, etc.

What happen?

FormItem doesn’t find the control inside.

[gaastonsr]

We solved this by calling the component as a function

CustomForm.js

// imports here

class CustomForm {
  render() {
    return (<Form>
      <Form.Item>
        {UsernameInput({ form: this.props.form })}
      </Form.Item>
    </Form>);
  }
}

export default Form.create()(CustomForm);

But if you’re open to ideas, I think a possible solution would be to define a context in the FormItem and the Input can register themselves when it’s present. Example

FormItem.js

class FormItem extends Component {
  static childContextTypes = {
    registerControl: PropTypes.func
  };

  getChildContext() {
    return { registerControl: this.registerControl }; 
  }

  registerControl(control) {
    this.controls.push(control);
  }
}

Input.js

class Input extends Component {
  static contextTypes = {
    registerControl: PropTypes.func
  };

  componentDidMount() {
    if (this.context.registerControl) {
      this.registerControl(this);
    }
  }
}

[tomasfrancisco]

@gaastonsr Your solution help me a lot!

Is this is a still valid issue? The context suggestion sounds good to me.