Merge pull request react-bootstrap#1022 from AlexKVal/factoriesDeprecation

Make deprecation warnings safe

Author: AlexKVal

src/templates/factory.index.js.template

@@ -1,8 +1,10 @@
+import warning from 'react/lib/warning';
+
 <% _.forEach(components, function (component) { %>
 import <%= component %> from './<%= component %>';
 <% }); %>
 
-console.warn('Support for factories will be removed in v0.25, for details see https://github.com/react-bootstrap/react-bootstrap/issues/825');
+warning(false, 'Support for factories will be removed in v0.25, for details see https://github.com/react-bootstrap/react-bootstrap/issues/825');
 
 export default {
 <% _.forEach(components, function (component) { %>

src/templates/factory.js.template

@@ -1,6 +1,7 @@
 import React from 'react';
+import warning from 'react/lib/warning';
 import <%= name %> from '../<%= name %>';
 
-console.warn('Support for factories will be removed in v0.25, for details see https://github.com/react-bootstrap/react-bootstrap/issues/825');
+warning(false, 'Support for factories will be removed in v0.25, for details see https://github.com/react-bootstrap/react-bootstrap/issues/825');
 
 export default React.createFactory(<%= name %>);

src/utils/deprecationWarning.js

@@ -1,14 +1,11 @@
-export default function deprecationWarning(oldname, newname, link) {
-  if (process.env.NODE_ENV !== 'production') {
-    if ((typeof console === 'undefined') || (typeof console.warn !== 'function')) {
-      return;
-    }
+import warning from 'react/lib/warning';
 
-    let message = `${oldname} is deprecated. Use ${newname} instead.`;
-    console.warn(message);
+export default function deprecationWarning(oldname, newname, link) {
+  let message = `${oldname} is deprecated. Use ${newname} instead.`;
 
-    if (link) {
-      console.warn(`You can read more about it at ${link}`);
-    }
+  if (link) {
+    message += `\nYou can read more about it at ${link}`;
   }
+
+  warning(false, message);
 }