Merge pull request react-bootstrap#614 from react-bootstrap/unrevert-536

Revert "Revert "adds ability to require components already wrapped in fa...

docs/src/GettingStartedPage.js

@@ -65,6 +65,16 @@ const Page = React.createClass({
   </script>
                     `}</code></pre>
                   </div>
+
+                  <h3>Without JSX</h3>
+                  <p>If you do not use JSX and just call components as functions, you must explicitly <a href="https://facebook.github.io/react/blog/2014/10/14/introducing-react-elements.html#deprecated-auto-generated-factories">create a factory before calling it</a>. React-bootstrap provides factories for you in <code>lib/factories</code>:</p>
+                  <div className="highlight">
+                    <pre><code className="js">{`
+  var Alert = require('react-bootstrap/lib/factories').Alert;
+  // or
+  var Alert = require('react-bootstrap/lib/factories/Alert');
+                    `}</code></pre>
+                  </div>
                 </div>
                 <div className="bs-docs-section">
                   <h2 id="browser-support" className="page-header">Browser support</h2>

package.json

@@ -10,7 +10,7 @@
   "scripts": {
     "build": "node run-babel tools/build.js",
     "test-watch": "karma start",
-    "test": "karma start --single-run && npm run lint && npm run build",
+    "test": "npm run lint && npm run build && karma start --single-run",
     "lint": "eslint src test docs ie8 tools webpack karma.conf.js webpack.config.js webpack.docs.js",
     "docs-build": "node run-babel tools/build.js --docs-only",
     "docs": "node run-babel docs/server.js",

src/templates/factory.index.js.template

@@ -0,0 +1,9 @@
+<% _.forEach(components, function (component) { %>
+import <%= component %> from './<%= component %>';
+<% }); %>
+
+export default {
+<% _.forEach(components, function (component) { %>
+  <%= component %>,
+<% }); %>
+}

src/templates/factory.js.template

@@ -0,0 +1,4 @@
+import React from 'react';
+import <%= name %> from '../<%= name %>';
+
+export default React.createFactory(<%= name %>);

test/FactoriesSpec.js

@@ -0,0 +1,20 @@
+import React from 'react';
+import components from '../tools/public-components';
+
+let props = {
+  Glyphicon: {glyph: 'star'},
+  Modal: {onRequestHide: function() {}},
+  ModalTrigger: {modal: React.DOM.div(null)},
+  OverlayTrigger: {overlay: React.DOM.div(null)}
+};
+
+function createTest(component) {
+  let factory = require(`../lib/factories/${component}`);
+  describe('factories', function () {
+    it(`Should have a ${component} factory`, function () {
+      assert.ok(React.isValidElement(factory(props[component])));
+    });
+  });
+}
+
+components.map(component => createTest(component));

tools/amd/build.js

@@ -3,6 +3,7 @@ import path from 'path';
 import fsp from 'fs-promise';
 import { copy } from '../fs-utils';
 import { exec } from '../exec';
+import generateFactories from '../generateFactories';
 import { repoRoot, srcRoot, bowerRoot } from '../constants';
 
 const packagePath = path.join(repoRoot, 'package.json');
@@ -12,6 +13,10 @@ const bowerJson = path.join(bowerRoot, 'bower.json');
 const readme = path.join(__dirname, 'README.md');
 const license = path.join(repoRoot, 'LICENSE');
 
+const babelOptions = '--modules amd --optional es7.objectRestSpread';
+
+const factoriesDestination = path.join(bowerRoot, 'factories');
+
 function bowerConfig() {
   return Promise.all([
     fsp.readFile(packagePath)
@@ -27,10 +32,11 @@ export default function BuildBower() {
   console.log('Building: '.cyan + 'bower module'.green);
 
   return exec(`rimraf ${bowerRoot}`)
-    .then(() => fsp.mkdir(bowerRoot))
+    .then(() => fsp.mkdirs(factoriesDestination))
     .then(() => Promise.all([
       bowerConfig(),
-      exec(`babel --modules amd --optional es7.objectRestSpread ${srcRoot} --out-dir ${path.join(bowerRoot, 'lib')}`),
+      generateFactories(babelOptions, factoriesDestination),
+      exec(`babel ${babelOptions} ${srcRoot} --out-dir ${bowerRoot}`),
       copy(readme, bowerRoot),
       copy(license, bowerRoot)
     ]))

tools/generateFactories.js

@@ -0,0 +1,31 @@
+import _ from 'lodash';
+import path from 'path';
+import fsp from 'fs-promise';
+import { exec } from './exec';
+import { srcRoot } from './constants';
+import components from './public-components';
+
+const templatePath = path.join(srcRoot, 'templates');
+const factoryTemplatePath = path.join(templatePath, 'factory.js.template');
+const indexTemplatePath = path.join(templatePath, 'factory.index.js.template');
+
+export default function generateFactories(babelOptions, destination) {
+
+  let generateCompiledFile = function (file, content) {
+    let outpath = path.join(destination, `${file}.js`);
+    return exec(`babel ${babelOptions} --out-file ${outpath} <<EOF\n ${content}`);
+  };
+
+  return Promise.all([
+    fsp.readFile(factoryTemplatePath)
+      .then(template => {
+        Promise.all(components.map(name => {
+          generateCompiledFile(name, _.template(template)({name: name}));
+        }));
+      }),
+    fsp.readFile(indexTemplatePath)
+      .then(template => _.template(template)({components: components}))
+      .then(content => generateCompiledFile('index', content))
+  ]);
+
+}

tools/lib/build.js

@@ -1,11 +1,21 @@
 import 'colors';
 import { exec } from '../exec';
+import path from 'path';
+import fsp from 'fs-promise';
 import { srcRoot, libRoot } from '../constants';
+import generateFactories from '../generateFactories';
+
+const factoryDestination = path.join(libRoot, 'factories');
+const babelOptions = '--optional es7.objectRestSpread';
 
 export default function BuildCommonJs() {
   console.log('Building: '.cyan + 'npm module'.green);
 
   return exec(`rimraf ${libRoot}`)
-    .then(() => exec(`babel --optional es7.objectRestSpread ${srcRoot} --out-dir ${libRoot}`))
+    .then(() => fsp.mkdirs(factoryDestination))
+    .then(() => Promise.all([
+      generateFactories(babelOptions, factoryDestination),
+      exec(`babel ${babelOptions} ${srcRoot} --out-dir ${libRoot}`)
+    ]))
     .then(() => console.log('Built: '.cyan + 'npm module'.green));
 }

tools/public-components.js

@@ -0,0 +1,11 @@
+import React from 'react';
+import index from '../src/index';
+
+let components = [];
+Object.keys(index).forEach(function (item) {
+  if (index[item] instanceof React.Component.constructor) {
+    components.push(item);
+  }
+});
+
+export default components;