From 0a16c350d150f4a0cdcc10ca7d356ef5242a03d6 Mon Sep 17 00:00:00 2001
From: Nicolas Gotchac <ngotchac@gmail.com>
Date: Wed, 10 May 2017 15:24:24 +0200
Subject: [PATCH] Can't deploy without compiling Contract (#5593)

---
 js/src/views/WriteContract/writeContract.js   |  3 ++-
 .../views/WriteContract/writeContractStore.js | 20 ++++++++++++++-----
 2 files changed, 17 insertions(+), 6 deletions(-)

diff --git a/js/src/views/WriteContract/writeContract.js b/js/src/views/WriteContract/writeContract.js
index df5ade348..634804c9c 100644
--- a/js/src/views/WriteContract/writeContract.js
+++ b/js/src/views/WriteContract/writeContract.js
@@ -345,12 +345,13 @@ class WriteContract extends Component {
             }
             onClick={ this.store.handleCompile }
             primary={ false }
-            disabled={ compiling }
+            disabled={ compiling || this.store.isPristine }
           />
           {
             contract
               ? (
                 <Button
+                  disabled={ compiling || !this.store.isPristine }
                   icon={ <SendIcon /> }
                   label={
                     <FormattedMessage
diff --git a/js/src/views/WriteContract/writeContractStore.js b/js/src/views/WriteContract/writeContractStore.js
index 289f2d2ba..202296447 100644
--- a/js/src/views/WriteContract/writeContractStore.js
+++ b/js/src/views/WriteContract/writeContractStore.js
@@ -15,7 +15,7 @@
 // along with Parity.  If not, see <http://www.gnu.org/licenses/>.
 
 import { debounce } from 'lodash';
-import { action, observable, transaction } from 'mobx';
+import { action, computed, observable, transaction } from 'mobx';
 import React from 'react';
 import { FormattedMessage } from 'react-intl';
 import store from 'store';
@@ -140,7 +140,7 @@ export default class WriteContractStore {
     this.worker = worker;
 
     return Promise.all([
-      this.fetchSolidityVersions(),
+      this.fetchSolidityVersions().then(() => this.handleCompile()),
       this.reloadContracts(undefined, undefined, false)
     ]);
   }
@@ -316,6 +316,18 @@ export default class WriteContractStore {
     });
   }
 
+  @computed get isPristine () {
+    return this.getHash() === this.lastCompilation.hash;
+  }
+
+  getHash () {
+    const build = this.builds[this.selectedBuild];
+    const version = build.longVersion;
+    const sourcecode = this.sourcecode.replace(/\s+/g, ' ');
+
+    return sha3(JSON.stringify({ version, sourcecode, optimize: this.optimize }));
+  }
+
   @action handleCompile = () => {
     transaction(() => {
       this.compiled = false;
@@ -324,9 +336,7 @@ export default class WriteContractStore {
     });
 
     const build = this.builds[this.selectedBuild];
-    const version = build.longVersion;
-    const sourcecode = this.sourcecode.replace(/\s+/g, ' ');
-    const hash = sha3(JSON.stringify({ version, sourcecode, optimize: this.optimize }));
+    const hash = this.getHash();
 
     let promise = Promise.resolve(null);