From 93ce4ba4d9da990e956d803787d0071475e2cc00 Mon Sep 17 00:00:00 2001
From: Andreas <me@grml.in>
Date: Fri, 3 Apr 2015 01:14:25 +0200
Subject: [PATCH] overwriting props will create console warnings

> Warning: Don't set .props.className of the React component <FlatButton />. Instead, specify the correct value when initially creating the > element. The element was created by LoginDialog.

elements should be cloned instead. React.cloneElement was added with React 0.13.
http://facebook.github.io/react/blog/2015/03/03/react-v0.13-rc2.html#react.cloneelement

Fixed a missing var statement, too.
---
 src/js/dialog-window.jsx | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/js/dialog-window.jsx b/src/js/dialog-window.jsx
index 13d5333c072be4..9f500b4d7258d9 100644
--- a/src/js/dialog-window.jsx
+++ b/src/js/dialog-window.jsx
@@ -94,9 +94,9 @@ var DialogWindow = React.createClass({
 
   _addClassName: function(reactObject, className) {
     var originalClassName = reactObject.props.className;
+    var newClassname = originalClassName ? originalClassName + ' ' + className : className;
 
-    reactObject.props.className = originalClassName ?
-      originalClassName + ' ' + className : className;
+    return React.cloneElement(reactObject, { className: newClassname });
   },
 
   _getAction: function(actionJSON, key) {
@@ -116,14 +116,14 @@ var DialogWindow = React.createClass({
 
     if (actions.length) {
       for (var i = 0; i < actions.length; i++) {
-        currentAction = actions[i];
+        var currentAction = actions[i];
 
         //if the current action isn't a react object, create one
         if (!React.isValidElement(currentAction)) {
           currentAction = this._getAction(currentAction, i);
         }
 
-        this._addClassName(currentAction, 'mui-dialog-window-action');
+        currentAction = this._addClassName(currentAction, 'mui-dialog-window-action');
         actionObjects.push(currentAction);
       };