It seems it does not work when importing modules

[romainneutron]

If I use this syntax, it works perfectly:

import React from 'react';

export default class MyComp extends React.Component {
    render() { 
        //...
    }
}

MyComp.propTypes = {
    name: React.PropTypes.object
}

however, it does not work like this:

import React, { Component } from 'react';

export default class MyComp extends Component {
    render() { 
        //...
    }
}

MyComp.propTypes = {
    name: React.PropTypes.object
}

[romainneutron]

For the record, I'm using React 15, and I've spotted the issue down to

if (superClass.matchesPattern('React.Component') || superClass.matchesPattern('Component')) {
  path.remove();
}

In AssignmentExpression : superClass.matchesPattern('Component') returns false because of the second line of the matchesPattern function that return false at line 2:

if (!this.isMemberExpression()) return false;

Changing superClass.matchesPattern('Component') to superClass.node.name === 'Component' solves my issue but I guess it's a wrong fix :)

[oliviertassinari]

@romainneutron Thanks for this feedback.
Can you confirm that this pattern is working?

import React, { Component } from 'react';

export default class MyComp extends Component {
    static propTypes = {
        name: React.PropTypes.object
    };

    render() { 
        //...
    }
}

That sounds like the right direction.
We should refactor the AssignmentExpression to use some part of the ClassProperty logic.

[romainneutron]

I confirm @oliviertassinari , it works better using your snippet

[oliviertassinari]

@romainneutron I have just merged a fix.

[romainneutron]

Thanks for your fast response