From 5ed9ace2cfc1e454a53125a71ef6e3a081055d2e Mon Sep 17 00:00:00 2001 From: satyam Date: Tue, 12 Feb 2019 15:02:22 +0530 Subject: [PATCH] remove timestamps from the events --- contracts/ModuleRegistry.sol | 8 ++--- contracts/Pausable.sol | 8 ++--- contracts/SecurityTokenRegistry.sol | 8 ++--- contracts/libraries/TokenLib.sol | 12 ++++---- .../modules/Checkpoint/DividendCheckpoint.sol | 16 +++++----- .../Experimental/Burn/TrackedRedemption.sol | 4 +-- .../Mixed/ScheduledCheckpoint.sol | 8 ++--- .../GeneralPermissionManager.sol | 8 ++--- .../PercentageTransferManager.sol | 4 +-- contracts/oracles/MakerDAOOracle.sol | 12 ++++---- contracts/oracles/PolyOracle.sol | 4 +-- contracts/oracles/StableOracle.sol | 8 ++--- contracts/tokens/SecurityToken.sol | 29 +++++++++---------- 13 files changed, 64 insertions(+), 65 deletions(-) diff --git a/contracts/ModuleRegistry.sol b/contracts/ModuleRegistry.sol index d90527993..974d5310d 100644 --- a/contracts/ModuleRegistry.sol +++ b/contracts/ModuleRegistry.sol @@ -40,9 +40,9 @@ contract ModuleRegistry is IModuleRegistry, EternalStorage { ////////// // Emit when network becomes paused - event Pause(uint256 _timestammp); + event Pause(address account); // Emit when network becomes unpaused - event Unpause(uint256 _timestamp); + event Unpause(address account); // Emit when Module is used by the SecurityToken event ModuleUsed(address indexed _moduleFactory, address indexed _securityToken); // Emit when the Module Factory gets registered on the ModuleRegistry contract @@ -359,7 +359,7 @@ contract ModuleRegistry is IModuleRegistry, EternalStorage { function pause() external whenNotPaused onlyOwner { set(Encoder.getKey("paused"), true); /*solium-disable-next-line security/no-block-members*/ - emit Pause(now); + emit Pause(msg.sender); } /** @@ -368,7 +368,7 @@ contract ModuleRegistry is IModuleRegistry, EternalStorage { function unpause() external whenPaused onlyOwner { set(Encoder.getKey("paused"), false); /*solium-disable-next-line security/no-block-members*/ - emit Unpause(now); + emit Unpause(msg.sender); } /** diff --git a/contracts/Pausable.sol b/contracts/Pausable.sol index 996275e5c..184987439 100644 --- a/contracts/Pausable.sol +++ b/contracts/Pausable.sol @@ -4,8 +4,8 @@ pragma solidity ^0.5.0; * @title Utility contract to allow pausing and unpausing of certain functions */ contract Pausable { - event Pause(uint256 _timestammp); - event Unpause(uint256 _timestamp); + event Pause(address account); + event Unpause(address account); bool public paused = false; @@ -31,7 +31,7 @@ contract Pausable { function _pause() internal whenNotPaused { paused = true; /*solium-disable-next-line security/no-block-members*/ - emit Pause(now); + emit Pause(msg.sender); } /** @@ -40,7 +40,7 @@ contract Pausable { function _unpause() internal whenPaused { paused = false; /*solium-disable-next-line security/no-block-members*/ - emit Unpause(now); + emit Unpause(msg.sender); } } diff --git a/contracts/SecurityTokenRegistry.sol b/contracts/SecurityTokenRegistry.sol index bfffbc535..88c9d31d5 100644 --- a/contracts/SecurityTokenRegistry.sol +++ b/contracts/SecurityTokenRegistry.sol @@ -79,9 +79,9 @@ contract SecurityTokenRegistry is EternalStorage, Proxy { string constant POLY_ORACLE = "StablePolyUsdOracle"; // Emit when network becomes paused - event Pause(uint256 _timestammp); + event Pause(address account); // Emit when network becomes unpaused - event Unpause(uint256 _timestamp); + event Unpause(address account); // Emit when the ticker is removed from the registry event TickerRemoved(string _ticker, uint256 _removedAt, address _removedBy); // Emit when the token ticker expiry is changed @@ -605,7 +605,7 @@ contract SecurityTokenRegistry is EternalStorage, Proxy { function pause() external whenNotPaused onlyOwner { set(PAUSED, true); /*solium-disable-next-line security/no-block-members*/ - emit Pause(now); + emit Pause(msg.sender); } /** @@ -614,7 +614,7 @@ contract SecurityTokenRegistry is EternalStorage, Proxy { function unpause() external whenPaused onlyOwner { set(PAUSED, false); /*solium-disable-next-line security/no-block-members*/ - emit Unpause(now); + emit Unpause(msg.sender); } /** diff --git a/contracts/libraries/TokenLib.sol b/contracts/libraries/TokenLib.sol index 18567243f..239c57941 100644 --- a/contracts/libraries/TokenLib.sol +++ b/contracts/libraries/TokenLib.sol @@ -34,11 +34,11 @@ library TokenLib { } // Emit when Module is archived from the SecurityToken - event ModuleArchived(uint8[] _types, address _module, uint256 _timestamp); + event ModuleArchived(uint8[] _types, address _module); // Emit when Module is unarchived from the SecurityToken - event ModuleUnarchived(uint8[] _types, address _module, uint256 _timestamp); + event ModuleUnarchived(uint8[] _types, address _module); // Emit when Module get removed from the securityToken - event ModuleRemoved(uint8[] _types, address _module, uint256 _timestamp); + event ModuleRemoved(uint8[] _types, address _module); // Emit when the budget allocated to a module is changed event ModuleBudgetChanged(uint8[] _moduleTypes, address _module, uint256 _oldBudget, uint256 _budget); @@ -51,7 +51,7 @@ library TokenLib { require(!_moduleData.isArchived, "Module archived"); require(_moduleData.module != address(0), "Module missing"); /*solium-disable-next-line security/no-block-members*/ - emit ModuleArchived(_moduleData.moduleTypes, _module, now); + emit ModuleArchived(_moduleData.moduleTypes, _module); _moduleData.isArchived = true; } @@ -63,7 +63,7 @@ library TokenLib { function unarchiveModule(ModuleData storage _moduleData, address _module) public { require(_moduleData.isArchived, "Module unarchived"); /*solium-disable-next-line security/no-block-members*/ - emit ModuleUnarchived(_moduleData.moduleTypes, _module, now); + emit ModuleUnarchived(_moduleData.moduleTypes, _module); _moduleData.isArchived = false; } @@ -82,7 +82,7 @@ library TokenLib { require(_modulesToData[_module].isArchived, "Not archived"); require(_modulesToData[_module].module != address(0), "Module missing"); /*solium-disable-next-line security/no-block-members*/ - emit ModuleRemoved(_modulesToData[_module].moduleTypes, _module, now); + emit ModuleRemoved(_modulesToData[_module].moduleTypes, _module); // Remove from module type list uint8[] memory moduleTypes = _modulesToData[_module].moduleTypes; for (uint256 i = 0; i < moduleTypes.length; i++) { diff --git a/contracts/modules/Checkpoint/DividendCheckpoint.sol b/contracts/modules/Checkpoint/DividendCheckpoint.sol index 3bfc358d8..e4e0d7dfb 100644 --- a/contracts/modules/Checkpoint/DividendCheckpoint.sol +++ b/contracts/modules/Checkpoint/DividendCheckpoint.sol @@ -20,10 +20,10 @@ import "openzeppelin-solidity/contracts/math/Math.sol"; contract DividendCheckpoint is DividendCheckpointStorage, ICheckpoint, Module { using SafeMath for uint256; - event SetDefaultExcludedAddresses(address[] _excluded, uint256 _timestamp); - event SetWithholding(address[] _investors, uint256[] _withholding, uint256 _timestamp); - event SetWithholdingFixed(address[] _investors, uint256 _withholding, uint256 _timestamp); - event SetWallet(address indexed _oldWallet, address indexed _newWallet, uint256 _timestamp); + event SetDefaultExcludedAddresses(address[] _excluded); + event SetWithholding(address[] _investors, uint256[] _withholding); + event SetWithholdingFixed(address[] _investors, uint256 _withholding); + event SetWallet(address indexed _oldWallet, address indexed _newWallet); modifier validDividendIndex(uint256 _dividendIndex) { require(_dividendIndex < dividends.length, "Invalid dividend"); @@ -63,7 +63,7 @@ contract DividendCheckpoint is DividendCheckpointStorage, ICheckpoint, Module { function _setWallet(address payable _wallet) internal { require(_wallet != address(0)); - emit SetWallet(wallet, _wallet, now); + emit SetWallet(wallet, _wallet); wallet = _wallet; } @@ -97,7 +97,7 @@ contract DividendCheckpoint is DividendCheckpointStorage, ICheckpoint, Module { } excluded = _excluded; /*solium-disable-next-line security/no-block-members*/ - emit SetDefaultExcludedAddresses(excluded, now); + emit SetDefaultExcludedAddresses(excluded); } /** @@ -108,7 +108,7 @@ contract DividendCheckpoint is DividendCheckpointStorage, ICheckpoint, Module { function setWithholding(address[] memory _investors, uint256[] memory _withholding) public withPerm(MANAGE) { require(_investors.length == _withholding.length, "Mismatched input lengths"); /*solium-disable-next-line security/no-block-members*/ - emit SetWithholding(_investors, _withholding, now); + emit SetWithholding(_investors, _withholding); for (uint256 i = 0; i < _investors.length; i++) { require(_withholding[i] <= 10 ** 18, "Incorrect withholding tax"); withholdingTax[_investors[i]] = _withholding[i]; @@ -123,7 +123,7 @@ contract DividendCheckpoint is DividendCheckpointStorage, ICheckpoint, Module { function setWithholdingFixed(address[] memory _investors, uint256 _withholding) public withPerm(MANAGE) { require(_withholding <= 10 ** 18, "Incorrect withholding tax"); /*solium-disable-next-line security/no-block-members*/ - emit SetWithholdingFixed(_investors, _withholding, now); + emit SetWithholdingFixed(_investors, _withholding); for (uint256 i = 0; i < _investors.length; i++) { withholdingTax[_investors[i]] = _withholding; } diff --git a/contracts/modules/Experimental/Burn/TrackedRedemption.sol b/contracts/modules/Experimental/Burn/TrackedRedemption.sol index eda7ca14c..e6af32d43 100644 --- a/contracts/modules/Experimental/Burn/TrackedRedemption.sol +++ b/contracts/modules/Experimental/Burn/TrackedRedemption.sol @@ -12,7 +12,7 @@ contract TrackedRedemption is IBurn, Module { mapping(address => uint256) redeemedTokens; - event Redeemed(address _investor, uint256 _value, uint256 _timestamp); + event Redeemed(address _investor, uint256 _value); /** * @notice Constructor @@ -37,7 +37,7 @@ contract TrackedRedemption is IBurn, Module { ISecurityToken(securityToken).burnFromWithData(msg.sender, _value, ""); redeemedTokens[msg.sender] = redeemedTokens[msg.sender].add(_value); /*solium-disable-next-line security/no-block-members*/ - emit Redeemed(msg.sender, _value, now); + emit Redeemed(msg.sender, _value); } /** diff --git a/contracts/modules/Experimental/Mixed/ScheduledCheckpoint.sol b/contracts/modules/Experimental/Mixed/ScheduledCheckpoint.sol index 9cc14c730..f0d1cbc9c 100644 --- a/contracts/modules/Experimental/Mixed/ScheduledCheckpoint.sol +++ b/contracts/modules/Experimental/Mixed/ScheduledCheckpoint.sol @@ -25,8 +25,8 @@ contract ScheduledCheckpoint is ICheckpoint, TransferManager { mapping(bytes32 => Schedule) public schedules; - event AddSchedule(bytes32 _name, uint256 _startTime, uint256 _interval, uint256 _timestamp); - event RemoveSchedule(bytes32 _name, uint256 _timestamp); + event AddSchedule(bytes32 _name, uint256 _startTime, uint256 _interval); + event RemoveSchedule(bytes32 _name); /** * @notice Constructor @@ -58,7 +58,7 @@ contract ScheduledCheckpoint is ICheckpoint, TransferManager { schedules[_name].interval = _interval; schedules[_name].index = names.length; names.push(_name); - emit AddSchedule(_name, _startTime, _interval, now); + emit AddSchedule(_name, _startTime, _interval); } /** @@ -74,7 +74,7 @@ contract ScheduledCheckpoint is ICheckpoint, TransferManager { schedules[names[index]].index = index; } delete schedules[_name]; - emit RemoveSchedule(_name, now); + emit RemoveSchedule(_name); } /** diff --git a/contracts/modules/PermissionManager/GeneralPermissionManager.sol b/contracts/modules/PermissionManager/GeneralPermissionManager.sol index 6ce855897..39b09050b 100644 --- a/contracts/modules/PermissionManager/GeneralPermissionManager.sol +++ b/contracts/modules/PermissionManager/GeneralPermissionManager.sol @@ -10,9 +10,9 @@ import "./GeneralPermissionManagerStorage.sol"; contract GeneralPermissionManager is GeneralPermissionManagerStorage, IPermissionManager, Module { /// Event emitted after any permission get changed for the delegate - event ChangePermission(address indexed _delegate, address _module, bytes32 _perm, bool _valid, uint256 _timestamp); + event ChangePermission(address indexed _delegate, address _module, bytes32 _perm, bool _valid); /// Used to notify when delegate is added in permission manager contract - event AddDelegate(address indexed _delegate, bytes32 _details, uint256 _timestamp); + event AddDelegate(address indexed _delegate, bytes32 _details); /// @notice constructor constructor(address _securityToken, address _polyToken) public Module(_securityToken, _polyToken) { @@ -52,7 +52,7 @@ contract GeneralPermissionManager is GeneralPermissionManagerStorage, IPermissio delegateDetails[_delegate] = _details; allDelegates.push(_delegate); /*solium-disable-next-line security/no-block-members*/ - emit AddDelegate(_delegate, _details, now); + emit AddDelegate(_delegate, _details); } /** @@ -207,7 +207,7 @@ contract GeneralPermissionManager is GeneralPermissionManagerStorage, IPermissio function _changePermission(address _delegate, address _module, bytes32 _perm, bool _valid) internal { perms[_module][_delegate][_perm] = _valid; /*solium-disable-next-line security/no-block-members*/ - emit ChangePermission(_delegate, _module, _perm, _valid, now); + emit ChangePermission(_delegate, _module, _perm, _valid); } /** diff --git a/contracts/modules/TransferManager/PercentageTransferManager.sol b/contracts/modules/TransferManager/PercentageTransferManager.sol index cdd0cc073..dbcfc5906 100644 --- a/contracts/modules/TransferManager/PercentageTransferManager.sol +++ b/contracts/modules/TransferManager/PercentageTransferManager.sol @@ -19,7 +19,7 @@ contract PercentageTransferManager is PercentageTransferManagerStorage, Transfer event ModifyHolderPercentage(uint256 _oldHolderPercentage, uint256 _newHolderPercentage); event ModifyWhitelist(address _investor, uint256 _dateAdded, address _addedBy, bool _valid); - event SetAllowPrimaryIssuance(bool _allowPrimaryIssuance, uint256 _timestamp); + event SetAllowPrimaryIssuance(bool _allowPrimaryIssuance); /** * @notice Constructor @@ -117,7 +117,7 @@ contract PercentageTransferManager is PercentageTransferManagerStorage, Transfer require(_allowPrimaryIssuance != allowPrimaryIssuance, "Must change setting"); allowPrimaryIssuance = _allowPrimaryIssuance; /*solium-disable-next-line security/no-block-members*/ - emit SetAllowPrimaryIssuance(_allowPrimaryIssuance, now); + emit SetAllowPrimaryIssuance(_allowPrimaryIssuance); } /** diff --git a/contracts/oracles/MakerDAOOracle.sol b/contracts/oracles/MakerDAOOracle.sol index 5a390cf72..f28c9e965 100644 --- a/contracts/oracles/MakerDAOOracle.sol +++ b/contracts/oracles/MakerDAOOracle.sol @@ -13,9 +13,9 @@ contract MakerDAOOracle is IOracle, Ownable { uint256 public manualPrice; /*solium-disable-next-line security/no-block-members*/ - event ChangeMedianizer(address _newMedianizer, address _oldMedianizer, uint256 _now); - event SetManualPrice(uint256 _oldPrice, uint256 _newPrice, uint256 _time); - event SetManualOverride(bool _override, uint256 _time); + event ChangeMedianizer(address _newMedianizer, address _oldMedianizer); + event SetManualPrice(uint256 _oldPrice, uint256 _newPrice); + event SetManualOverride(bool _override); /** * @notice Creates a new Maker based oracle @@ -36,7 +36,7 @@ contract MakerDAOOracle is IOracle, Ownable { function changeMedianier(address _medianizer) public onlyOwner { require(_medianizer != address(0), "0x not allowed"); /*solium-disable-next-line security/no-block-members*/ - emit ChangeMedianizer(_medianizer, medianizer, now); + emit ChangeMedianizer(_medianizer, medianizer); medianizer = _medianizer; } @@ -80,7 +80,7 @@ contract MakerDAOOracle is IOracle, Ownable { */ function setManualPrice(uint256 _price) public onlyOwner { /*solium-disable-next-line security/no-block-members*/ - emit SetManualPrice(manualPrice, _price, now); + emit SetManualPrice(manualPrice, _price); manualPrice = _price; } @@ -91,7 +91,7 @@ contract MakerDAOOracle is IOracle, Ownable { function setManualOverride(bool _override) public onlyOwner { manualOverride = _override; /*solium-disable-next-line security/no-block-members*/ - emit SetManualOverride(_override, now); + emit SetManualOverride(_override); } } diff --git a/contracts/oracles/PolyOracle.sol b/contracts/oracles/PolyOracle.sol index 5c7e4c856..a029a11ce 100644 --- a/contracts/oracles/PolyOracle.sol +++ b/contracts/oracles/PolyOracle.sol @@ -29,7 +29,7 @@ contract PolyOracle is usingOraclize, IOracle, Ownable { event PriceUpdated(uint256 _price, uint256 _oldPrice, bytes32 _queryId, uint256 _time); event NewOraclizeQuery(uint256 _time, bytes32 _queryId, string _query); - event AdminSet(address _admin, bool _valid, uint256 _time); + event AdminSet(address _admin, bool _valid); event StalePriceUpdate(bytes32 _queryId, uint256 _time, string _result); modifier isAdminOrOwner() { @@ -230,7 +230,7 @@ contract PolyOracle is usingOraclize, IOracle, Ownable { function setAdmin(address _admin, bool _valid) public onlyOwner { admin[_admin] = _valid; /*solium-disable-next-line security/no-block-members*/ - emit AdminSet(_admin, _valid, now); + emit AdminSet(_admin, _valid); } /** diff --git a/contracts/oracles/StableOracle.sol b/contracts/oracles/StableOracle.sol index aeb38b4af..3dabf26d2 100644 --- a/contracts/oracles/StableOracle.sol +++ b/contracts/oracles/StableOracle.sol @@ -17,8 +17,8 @@ contract StableOracle is IOracle, Ownable { /*solium-disable-next-line security/no-block-members*/ event ChangeOracle(address _oldOracle, address _newOracle); event ChangeEvictPercentage(uint256 _oldEvictPercentage, uint256 _newEvictPercentage); - event SetManualPrice(uint256 _oldPrice, uint256 _newPrice, uint256 _time); - event SetManualOverride(bool _override, uint256 _time); + event SetManualPrice(uint256 _oldPrice, uint256 _newPrice); + event SetManualOverride(bool _override); /** * @notice Creates a new stable oracle based on existing oracle @@ -96,7 +96,7 @@ contract StableOracle is IOracle, Ownable { */ function setManualPrice(uint256 _price) public onlyOwner { /*solium-disable-next-line security/no-block-members*/ - emit SetManualPrice(manualPrice, _price, now); + emit SetManualPrice(manualPrice, _price); manualPrice = _price; } @@ -107,7 +107,7 @@ contract StableOracle is IOracle, Ownable { function setManualOverride(bool _override) public onlyOwner { manualOverride = _override; /*solium-disable-next-line security/no-block-members*/ - emit SetManualOverride(_override, now); + emit SetManualOverride(_override); } } diff --git a/contracts/tokens/SecurityToken.sol b/contracts/tokens/SecurityToken.sol index 1351be50d..4da34cbdb 100644 --- a/contracts/tokens/SecurityToken.sol +++ b/contracts/tokens/SecurityToken.sol @@ -95,8 +95,7 @@ contract SecurityToken is ERC20, ERC20Detailed, ReentrancyGuard, RegistryUpdater address _module, uint256 _moduleCost, uint256 _budget, - bytes32 _label, - uint256 _timestamp + bytes32 _label ); // Emit when the token details get updated @@ -104,19 +103,19 @@ contract SecurityToken is ERC20, ERC20Detailed, ReentrancyGuard, RegistryUpdater // Emit when the granularity get changed event GranularityChanged(uint256 _oldGranularity, uint256 _newGranularity); // Emit when Module get archived from the securityToken - event ModuleArchived(uint8[] _types, address _module, uint256 _timestamp); + event ModuleArchived(uint8[] _types, address _module); // Emit when Module get unarchived from the securityToken - event ModuleUnarchived(uint8[] _types, address _module, uint256 _timestamp); + event ModuleUnarchived(uint8[] _types, address _module); // Emit when Module get removed from the securityToken - event ModuleRemoved(uint8[] _types, address _module, uint256 _timestamp); + event ModuleRemoved(uint8[] _types, address _module); // Emit when the budget allocated to a module is changed event ModuleBudgetChanged(uint8[] _moduleTypes, address _module, uint256 _oldBudget, uint256 _budget); // Emit when transfers are frozen or unfrozen - event FreezeTransfers(bool _status, uint256 _timestamp); + event FreezeTransfers(bool _status); // Emit when new checkpoint created - event CheckpointCreated(uint256 indexed _checkpointId, uint256 _timestamp); + event CheckpointCreated(uint256 indexed _checkpointId); // Emit when is permanently frozen by the issuer - event FreezeMinting(uint256 _timestamp); + event FreezeMinting(); // Events to log minting and burning event Minted(address indexed _to, uint256 _value); event Burnt(address indexed _from, uint256 _value); @@ -132,7 +131,7 @@ contract SecurityToken is ERC20, ERC20Detailed, ReentrancyGuard, RegistryUpdater bytes _data ); event ForceBurn(address indexed _controller, address indexed _from, uint256 _value, bool _verifyTransfer, bytes _data); - event DisableController(uint256 _timestamp); + event DisableController(); function _isModule(address _module, uint8 _type) internal view returns(bool) { require(modulesToData[_module].module == _module, "Wrong address"); @@ -266,7 +265,7 @@ contract SecurityToken is ERC20, ERC20Detailed, ReentrancyGuard, RegistryUpdater names[moduleName].push(module); //Emit log event /*solium-disable-next-line security/no-block-members*/ - emit ModuleAdded(moduleTypes, moduleName, _moduleFactory, module, moduleCost, _budget, _label, now); + emit ModuleAdded(moduleTypes, moduleName, _moduleFactory, module, moduleCost, _budget, _label); } /** @@ -459,7 +458,7 @@ contract SecurityToken is ERC20, ERC20Detailed, ReentrancyGuard, RegistryUpdater require(!transfersFrozen, "Already frozen"); transfersFrozen = true; /*solium-disable-next-line security/no-block-members*/ - emit FreezeTransfers(true, now); + emit FreezeTransfers(true); } /** @@ -469,7 +468,7 @@ contract SecurityToken is ERC20, ERC20Detailed, ReentrancyGuard, RegistryUpdater require(transfersFrozen, "Not frozen"); transfersFrozen = false; /*solium-disable-next-line security/no-block-members*/ - emit FreezeTransfers(false, now); + emit FreezeTransfers(false); } /** @@ -623,7 +622,7 @@ contract SecurityToken is ERC20, ERC20Detailed, ReentrancyGuard, RegistryUpdater function freezeMinting() external isMintingAllowed isEnabled("freezeMintingAllowed") onlyOwner { mintingFrozen = true; /*solium-disable-next-line security/no-block-members*/ - emit FreezeMinting(now); + emit FreezeMinting(); } /** @@ -749,7 +748,7 @@ contract SecurityToken is ERC20, ERC20Detailed, ReentrancyGuard, RegistryUpdater checkpointTimes.push(now); /*solium-disable-next-line security/no-block-members*/ checkpointTotalSupply[currentCheckpointId] = totalSupply(); - emit CheckpointCreated(currentCheckpointId, now); + emit CheckpointCreated(currentCheckpointId); return currentCheckpointId; } @@ -800,7 +799,7 @@ contract SecurityToken is ERC20, ERC20Detailed, ReentrancyGuard, RegistryUpdater controllerDisabled = true; delete controller; /*solium-disable-next-line security/no-block-members*/ - emit DisableController(now); + emit DisableController(); } /**