andreychernykh / bug-reproduce-1 Goto Github PK

Do you want to request a feature or report a bug?
Bug

What is the current behavior?
Class properties, defined as ClassName.propertyName = .... are undefined

class User extends React.Component {
  render() {
    console.log(User.propTypes) // undefined
    console.log(User.myStaticProperty) // undefined
  }
}
User.propTypes = {};
User.myStaticProperty = '42';

If the current behavior is a bug, please provide the steps to reproduce and a minimal repository on GitHub that we can yarn install and yarn test.

What is the expected behavior?

class User extends React.Component {
  render() {
    console.log(User.propTypes) // {}
    console.log(User.myStaticProperty) // '42'
  }
}
User.propTypes = {};
User.myStaticProperty = '42';

Please provide your exact Metro configuration and mention your Metro, node, yarn/npm version and operating system.

// package.json:
    "@babel/cli": "^7.2.3",
    "@babel/core": "^7.3.4",
    "@babel/plugin-proposal-class-properties": "^7.3.4",
    "@babel/preset-env": "^7.3.4",
    "@babel/preset-react": "^7.0.0",
    "metro-babel7-plugin-react-transform": "^0.52.0",

// babel.config.js
module.exports = {
  presets: ['@babel/env', '@babel/react'],
  plugins: ['@babel/proposal-class-properties'],
  env: {
    development: {
      plugins: [
        [
          'module:metro-babel7-plugin-react-transform',
          {
            transforms: [{
              transform: 'react-transform-hmr',
              imports: ['react'],
              locals: ['module'],
            }],
          },
        ],
      ],
    },
  },
};

node: 10.15.1
yarn: 1.13.0

It looks like some changes were made in this method: https://github.com/facebook/metro/blob/08f41b7d3d937738492d17f4a92e135d4c1134fa/packages/metro-babel7-plugin-react-transform/src/index.js#L151
We believe, that might be a cause of this bug.