diff --git a/contracts/GitHubAPIReg.sol b/contracts/GitHubAPIReg.sol index 18f3191..161a4ac 100644 --- a/contracts/GitHubAPIReg.sol +++ b/contracts/GitHubAPIReg.sol @@ -22,7 +22,7 @@ contract GitHubAPIReg is Controlled, usingOraclize { } //owner management - function setAPICredentials(string _client_id, string _client_secret) only_owner { + function setAPICredentials(string _client_id, string _client_secret) onlyController { strings.slice [] memory cm = new strings.slice[](5); cm[0] = strings.toSlice("?client_id="); cm[1] = _client_id.toSlice(); @@ -31,7 +31,7 @@ contract GitHubAPIReg is Controlled, usingOraclize { cred = strings.toSlice("").join(cm); } - function clearAPICredentials() only_owner { + function clearAPICredentials() onlyController { cred = ""; } diff --git a/contracts/GitHubOracle.sol b/contracts/GitHubOracle.sol index e684f8c..0092317 100644 --- a/contracts/GitHubOracle.sol +++ b/contracts/GitHubOracle.sol @@ -41,7 +41,7 @@ contract GitHubOracle is Controlled, DGitI { mapping (string => string) pending; } - function __init_regs() only_owner { + function __init_regs() onlyController { if(address(userReg) == 0x0){ userReg = GHUserReg.create(); } @@ -50,7 +50,7 @@ contract GitHubOracle is Controlled, DGitI { } } - function __set_points_script(string _arg) only_owner { + function __set_points_script(string _arg) onlyController { if(address(gitHubPoints) == 0x0){ gitHubPoints = GHPoints.create(_arg); }else { @@ -58,10 +58,10 @@ contract GitHubOracle is Controlled, DGitI { } } - function __setController(address __newcontroller) only_owner { - userReg.setOwner(newContract); - repoReg.setOwner(newContract); - gitHubPoints.setOwner(newContract); + function __changeController(address _newController) onlyController { + userReg.changeController(_newController); + repoReg.changeController(newContract); + gitHubPoints.changeController(newContract); } function update(string _repository, string _token) payable { @@ -99,7 +99,7 @@ contract GitHubOracle is Controlled, DGitI { event NewPoints(uint repoId, uint userId, uint total, bool claimed); function __newPoints(uint _repoId, uint _userId, uint _points) - only_owner { + oraclized { GitRepositoryI repoaddr = GitRepositoryI(repositoryReg.getAddr(_repoId)); bool claimed = repoaddr.claim(userReg.getAddr(_userId), _points); if(!claimed){ //try to claim points diff --git a/contracts/GitHubPoints.sol b/contracts/GitHubPoints.sol index 9ba424d..1ae94d6 100644 --- a/contracts/GitHubPoints.sol +++ b/contracts/GitHubPoints.sol @@ -31,14 +31,14 @@ contract GitHubPoints is Controlled, usingOraclize{ bytes20 commitid; } - function start(string _repository, string _branch, string _cred) payable only_owner { + function start(string _repository, string _branch, string _cred) payable onlyController { if(bytes(_cred).length == 0) _cred = cred; bytes32 ocid = oraclize_query("nested", _query_start(_repository,_branch,_cred)); command[ocid] = Command.START; branches[ocid] = _branch; } - function update(string _repository, string _branch, string _lastCommit, string _cred) payable only_owner { + function update(string _repository, string _branch, string _lastCommit, string _cred) payable onlyController { if(bytes(_cred).length == 0) _cred = cred; bytes32 ocid = oraclize_query("nested", _query_update(_repository,_branch,_lastCommit,_cred)); command[ocid] = Command.UPDATE; @@ -47,7 +47,7 @@ contract GitHubPoints is Controlled, usingOraclize{ } function resume(string _repository, string _branch, string _lastCommit, string _limitCommit, string _cred) - payable only_owner { + payable onlyController { if(bytes(_cred).length == 0) _cred = cred; bytes32 ocid = oraclize_query("nested", _query_resume(_repository,_branch,_lastCommit,_limitCommit,_cred)); command[ocid] = Command.RESUME; @@ -56,7 +56,7 @@ contract GitHubPoints is Controlled, usingOraclize{ } function issue(string _repository, string _issue, string _cred) - payable only_owner { + payable onlyController { if(bytes(_cred).length == 0) _cred = cred; command[ocid] = Command.ISSUE; bytes32 ocid = oraclize_query("nested", _query_issue(_repository,_issue,_cred)); @@ -197,15 +197,15 @@ contract GitHubPoints is Controlled, usingOraclize{ oraclize_setProof(proofType_TLSNotary | proofStorage_IPFS); } - function setAPICredentials(string _client_id_comma_client_secret) only_owner { + function setAPICredentials(string _client_id_comma_client_secret) onlyController { cred = _client_id_comma_client_secret; } - function setScript(string _script) only_owner{ + function setScript(string _script) onlyController{ script = _script; } - function clearAPICredentials() only_owner { + function clearAPICredentials() onlyController { cred = ""; } diff --git a/contracts/GitHubRepositoryReg.sol b/contracts/GitHubRepositoryReg.sol index db1060a..eec2e75 100644 --- a/contracts/GitHubRepositoryReg.sol +++ b/contracts/GitHubRepositoryReg.sol @@ -69,8 +69,8 @@ contract GitHubRepositoryReg is NameRegistry, GitHubAPIReg { (default_branch,pos) = getNextString(v,pos); address repoAddr = repositories[projectId].addr; if(repoAddr == 0x0){ - GitRepositoryI repo = GitFactory.newGitRepository(projectId, full_name); - repo.setOwner(owner); + GitReposcitoryI repo = GitFactory.newGitRepository(projectId, full_name); + repo.changeController(controller); repoAddr = address(repo); indexes[sha3(repoAddr)] = projectId; indexes[sha3(full_name)] = projectId; diff --git a/contracts/GitRepository.sol b/contracts/GitRepository.sol index 2011173..5d5d8b1 100644 --- a/contracts/GitRepository.sol +++ b/contracts/GitRepository.sol @@ -52,7 +52,7 @@ contract GitRepository is GitRepositoryI { //oracle claim request function claim(address _user, uint _total) - only_owner returns (bool) { + onlyController returns (bool) { if(!token.lock() && _user != 0x0){ token.mint(_user, _total); return true; @@ -61,12 +61,12 @@ contract GitRepository is GitRepositoryI { } } - function setBounty(uint256 _issueId, bool _state, uint256 _closedAt) only_owner { + function setBounty(uint256 _issueId, bool _state, uint256 _closedAt) onlyController { if (_state) bountyBank.open(_issueId); else bountyBank.close(_issueId,_closedAt); } - function setBountyPoints(uint256 _issueId, address _claimer, uint256 _points) only_owner { + function setBountyPoints(uint256 _issueId, address _claimer, uint256 _points) onlyController { bountyBank.setClaimer(_issueId,_claimer,_points); } diff --git a/contracts/GitRepositoryToken.sol b/contracts/GitRepositoryToken.sol index b06b337..4621281 100644 --- a/contracts/GitRepositoryToken.sol +++ b/contracts/GitRepositoryToken.sol @@ -27,7 +27,7 @@ contract GitRepositoryToken is LockerToken { } function mint(address _who, uint256 _value) - only_owner when_locked(false) { + onlyController when_locked(false) { _mint(_who,_value); } diff --git a/contracts/LockerToken.sol b/contracts/LockerToken.sol index 7ea7e40..e5e3656 100644 --- a/contracts/LockerToken.sol +++ b/contracts/LockerToken.sol @@ -14,13 +14,13 @@ pragma solidity ^0.4.11; contract LockerToken is AbstractToken, Lockable, Controlled { Lockable public locker = this; - function unlinkLocker() only_owner { + function unlinkLocker() onlyController { locker = this; } - function linkLocker(Lockable _locker) only_owner { + function linkLocker(Lockable _locker) onlyController { locker = _locker; } - function setlock(bool _lock) only_owner { + function setlock(bool _lock) onlyController { setLock(_lock); }