'undefined' className

[chrisvasz]

// message.js
import './message.css';
function Message(props) {
  return <div className={props.className} styleName="message">message</div>;
}
render(<Message />);

// output from Babel with babel-plugin-react-css-modules
import './message.css';
function Message(props) {
  return <div className={props.className + ' message___3Hgmr'}>message</div>;
}
render(<Message />);

This outputs a div with className="undefined message___3Hgmr" because props.className is undefined.

A more robust output might look something like this:

// better output
import './message.css';
function Message(props) {
  return <div className={(props.className ? props.className + ' ' : '') + 'message___3Hgmr'}>message</div>;
}
render(<Message />);

[gajus]

I see.

I am not convinced that the overhead (ah additional IF conditional for each variable used in class declaration) is worth the fix. Of course, if the stylesheet includes "undefined" class, this will produce unexpected results.

This could be something thats enabled only in NODE_ENV=development.

[CurtisHumphrey]

PR: #62