Do not translate constant functions from <0.4.0 as payable

Author: axic

abi.js

@@ -19,8 +19,8 @@ function update (compilerVersion, abi) {
     }
 
     if (item.type !== 'event') {
-      // add 'payable' to everything
-      if (semver.lt(compilerVersion, '0.4.0')) {
+      // add 'payable' to everything, except constant functions
+      if (!item.constant && semver.lt(compilerVersion, '0.4.0')) {
         item.payable = true;
       }
 

test/abi.js

@@ -22,7 +22,7 @@ tape('ABI translator', function (t) {
   });
   t.test('0.3.6 (constant function)', function (st) {
     var input = [ { inputs: [], type: 'function', constant: true } ];
-    st.deepEqual(abi.update('0.3.6', input), [ { inputs: [], payable: true, constant: true, stateMutability: 'payable', type: 'function' }, { payable: true, stateMutability: 'payable', type: 'fallback' } ]);
+    st.deepEqual(abi.update('0.3.6', input), [ { inputs: [], constant: true, stateMutability: 'view', type: 'function' }, { payable: true, stateMutability: 'payable', type: 'fallback' } ]);
     st.end();
   });
   t.test('0.3.6 (event)', function (st) {