require tokenfactory in mmt constructor, fixes in interfaces, bump solcv

config/contracts.json

@@ -17,6 +17,12 @@
     "contracts": {
       "ERC20Receiver": {
         "deploy": false
+      },
+      "MiniMeToken": {
+        "deploy": false
+      },
+      "MiniMeTokenFactory": {
+        "deploy": true
       }
     }
   }

contracts/token/ApproveAndCallFallBack.sol

@@ -1,5 +1,10 @@
 pragma solidity ^0.4.23;
 
-contract ApproveAndCallFallBack {
-    function receiveApproval(address from, uint256 _amount, address _token, bytes _data) public;
+interface ApproveAndCallFallBack {
+    function receiveApproval(
+        address from,
+        uint256 _amount,
+        address _token,
+        bytes _data
+    ) external;
 }

contracts/token/MiniMeToken.sol

@@ -116,6 +116,7 @@ contract MiniMeToken is MiniMeTokenInterface, Controlled {
     ) 
         public
     {
+        require(_tokenFactory != address(0)); //if not set, clone feature will not work properly
         tokenFactory = MiniMeTokenFactory(_tokenFactory);
         name = _tokenName;                                 // Set the name
         decimals = _decimalUnits;                          // Set the decimals
@@ -399,11 +400,11 @@ contract MiniMeToken is MiniMeTokenInterface, Controlled {
 
     /**
      * @notice Creates a new clone token with the initial distribution being
-     *  this token at `snapshotBlock`
+     *  this token at `_snapshotBlock`
      * @param _cloneTokenName Name of the clone token
      * @param _cloneDecimalUnits Number of decimals of the smallest unit
      * @param _cloneTokenSymbol Symbol of the clone token
-     * @param snapshotBlock Block when the distribution of the parent token is
+     * @param _snapshotBlock Block when the distribution of the parent token is
      *  copied to set the initial distribution of the new clone token;
      *  if the block is zero than the actual block, the current block is used
      * @param _transfersEnabled True if transfers are allowed in the clone

contracts/token/MiniMeTokenFactory.sol

@@ -1,4 +1,4 @@
-pragma solidity ^0.4.11;
+pragma solidity ^0.4.23;
 
 import "./MiniMeToken.sol";
 

contracts/token/MiniMeTokenInterface.sol

@@ -1,4 +1,4 @@
-pragma solidity ^0.4.17;
+pragma solidity ^0.4.23;
 
 import "./ERC20Token.sol";
 
@@ -19,7 +19,7 @@ contract MiniMeTokenInterface is ERC20Token {
         uint256 _amount,
         bytes _extraData
     ) 
-        public 
+        external 
         returns (bool success);
 
     /**    

contracts/token/TokenController.sol

@@ -1,14 +1,14 @@
-pragma solidity ^0.4.14;
+pragma solidity ^0.4.23;
 /**
  * @dev The token controller contract must implement these functions
  */
-contract TokenController {
+interface TokenController {
     /**
      * @notice Called when `_owner` sends ether to the MiniMe Token contract
      * @param _owner The address that sent the ether to create tokens
      * @return True if the ether is accepted, false if it throws
      */
-    function proxyPayment(address _owner) payable returns(bool);
+    function proxyPayment(address _owner) external payable returns(bool);
 
     /**
      * @notice Notifies the controller about a token transfer allowing the
@@ -18,7 +18,7 @@ contract TokenController {
      * @param _amount The amount of the transfer
      * @return False if the controller does not authorize the transfer
      */
-    function onTransfer(address _from, address _to, uint _amount) returns(bool);
+    function onTransfer(address _from, address _to, uint _amount) external returns(bool);
 
     /**
      * @notice Notifies the controller about an approval allowing the
@@ -28,6 +28,6 @@ contract TokenController {
      * @param _amount The amount in the `approve()` call
      * @return False if the controller does not authorize the approval
      */
-    function onApprove(address _owner, address _spender, uint _amount)
+    function onApprove(address _owner, address _spender, uint _amount) external
         returns(bool);
 }