From bab9e88c1ae64c9a21b198df32f482322a5bf4dc Mon Sep 17 00:00:00 2001
From: Ricardo Guilherme Schmidt <3esmit@gmail.com>
Date: Thu, 22 Feb 2018 05:16:06 -0300
Subject: [PATCH] manager should call only by execute

---
 contracts/identity/Identity.sol | 19 +++++++++++--------
 1 file changed, 11 insertions(+), 8 deletions(-)

diff --git a/contracts/identity/Identity.sol b/contracts/identity/Identity.sol
index 0e463fd..2bf49bb 100644
--- a/contracts/identity/Identity.sol
+++ b/contracts/identity/Identity.sol
@@ -31,11 +31,8 @@ contract Identity is ERC725, ERC735 {
         _;
     }
 
-    modifier managerOrSelf {
-        require(
-            isKeyType(bytes32(msg.sender), MANAGEMENT_KEY) || 
-            msg.sender == address(this)
-        );
+    modifier selfOnly {
+        require(msg.sender == address(this));
         _;
     }
 
@@ -68,7 +65,7 @@ contract Identity is ERC725, ERC735 {
         uint256 _type
     )
         public
-        managerOrSelf
+        selfOnly
         returns (bool success)
     {
         _addKey(_key, _purpose, _type);
@@ -80,7 +77,7 @@ contract Identity is ERC725, ERC735 {
         uint256 _purpose
     )
         public
-        managerOrSelf
+        selfOnly
         returns (bool success)
     {
         _removeKey(_key, _purpose);
@@ -140,7 +137,13 @@ contract Identity is ERC725, ERC735 {
         
     }
 
-    function setMiminumApprovalsByKeyType(uint256 _type, uint8 _minimumApprovals) public managerOrSelf {
+    function setMiminumApprovalsByKeyType(
+        uint256 _type,
+        uint8 _minimumApprovals
+    ) 
+        public 
+        selfOnly
+    {
         minimumApprovalsByKeyType[_type] = _minimumApprovals;
     }