Fix/prepare 2.7.0 (#90)

* husky & deps

* added releaseTokens method & fixed tests

* mv to finance dir

* renamed access dir

* fixed naming & diamond presets & adjusted tests

* fix

* adjusted diamond onlyOwner & tests

* added renounce ownership

* added event & virtual

Author: dovgopoly

.husky/pre-commit

@@ -1,4 +1 @@
-#!/bin/sh
-. "$(dirname "$0")/_/husky.sh"
-
 npm run lint-fix && git add -u

contracts/access-control/MerkleWhitelisted.sol renamed to contracts/access/MerkleWhitelisted.sol

@@ -6,7 +6,7 @@ import {Initializable} from "@openzeppelin/contracts-upgradeable/proxy/utils/Ini
 
 import {SetHelper} from "../libs/arrays/SetHelper.sol";
 import {TypeCaster} from "../libs/utils/TypeCaster.sol";
-import {IMultiOwnable} from "../interfaces/access-control/IMultiOwnable.sol";
+import {IMultiOwnable} from "../interfaces/access/IMultiOwnable.sol";
 
 /**
  * @notice The MultiOwnable module

contracts/access-control/MultiOwnable.sol renamed to contracts/access/MultiOwnable.sol

@@ -3,7 +3,7 @@ pragma solidity ^0.8.4;
 
 import {Initializable} from "@openzeppelin/contracts-upgradeable/proxy/utils/Initializable.sol";
 
-import {IRBAC} from "../interfaces/access-control/IRBAC.sol";
+import {IRBAC} from "../interfaces/access/IRBAC.sol";
 
 import {TypeCaster} from "../libs/utils/TypeCaster.sol";
 import {SetHelper} from "../libs/arrays/SetHelper.sol";

contracts/access-control/PermanentOwnable.sol renamed to contracts/access/PermanentOwnable.sol

@@ -1,7 +1,7 @@
 // SPDX-License-Identifier: MIT
 pragma solidity ^0.8.4;
 
-import {IRBACGroupable} from "../../interfaces/access-control/extensions/IRBACGroupable.sol";
+import {IRBACGroupable} from "../../interfaces/access/extensions/IRBACGroupable.sol";
 
 import {StringSet} from "../../libs/data-structures/StringSet.sol";
 import {SetHelper} from "../../libs/arrays/SetHelper.sol";

contracts/access-control/RBAC.sol renamed to contracts/access/RBAC.sol

@@ -2,7 +2,7 @@
 pragma solidity ^0.8.4;
 
 import {AbstractPoolContractsRegistry} from "../AbstractPoolContractsRegistry.sol";
-import {MultiOwnable} from "../../../access-control/MultiOwnable.sol";
+import {MultiOwnable} from "../../../access/MultiOwnable.sol";
 
 /**
  * @notice The MultiOwnable preset of PoolContractsRegistry

contracts/access-control/extensions/RBACGroupable.sol renamed to contracts/access/extensions/RBACGroupable.sol

@@ -2,7 +2,7 @@
 pragma solidity ^0.8.4;
 
 import {AbstractContractsRegistry} from "../AbstractContractsRegistry.sol";
-import {MultiOwnable} from "../../access-control/MultiOwnable.sol";
+import {MultiOwnable} from "../../access/MultiOwnable.sol";
 
 /**
  * @notice The MultiOwnable preset of ContractsRegistry

contracts/contracts-registry/pools/presets/MultiOwnablePoolContractsRegistry.sol

@@ -1,8 +1,6 @@
 // SPDX-License-Identifier: MIT
 pragma solidity ^0.8.4;
 
-import {Strings} from "@openzeppelin/contracts/utils/Strings.sol";
-
 import {DiamondAccessControlStorage, IAccessControl} from "./DiamondAccessControlStorage.sol";
 
 /**
@@ -12,19 +10,6 @@ import {DiamondAccessControlStorage, IAccessControl} from "./DiamondAccessContro
  * by the Diamond Standard.
  */
 abstract contract DiamondAccessControl is DiamondAccessControlStorage {
-    /**
-     * @dev Modifier that checks that an account has a specific role. Reverts
-     * with a standardized message including the required role.
-     *
-     * The format of the revert reason is given by the following regular expression:
-     *
-     *  /^AccessControl: account (0x[0-9a-f]{40}) is missing role (0x[0-9a-f]{64})$/
-     */
-    modifier onlyRole(bytes32 role_) {
-        _checkRole(role_);
-        _;
-    }
-
     /**
      * @notice Sets `DEFAULT_ADMIN_ROLE` to `msg.sender`
      */
@@ -106,36 +91,4 @@ abstract contract DiamondAccessControl is DiamondAccessControlStorage {
 
         emit RoleRevoked(role_, account_, msg.sender);
     }
-
-    /**
-     * @dev Revert with a standard message if `_msgSender()` is missing `role`.
-     * Overriding this function changes the behavior of the {onlyRole} modifier.
-     *
-     * Format of the revert message is described in {_checkRole}.
-     */
-    function _checkRole(bytes32 role_) internal view virtual {
-        _checkRole(role_, msg.sender);
-    }
-
-    /**_
-     * @dev Revert with a standard message if `account` is missing `role`.
-     *
-     * The format of the revert reason is given by the following regular expression:
-     *
-     *  /^AccessControl: account (0x[0-9a-f]{40}) is missing role (0x[0-9a-f]{64})$/
-     */
-    function _checkRole(bytes32 role_, address account_) internal view virtual {
-        if (!hasRole(role_, account_)) {
-            revert(
-                string(
-                    abi.encodePacked(
-                        "AccessControl: account ",
-                        Strings.toHexString(account_),
-                        " is missing role ",
-                        Strings.toHexString(uint256(role_), 32)
-                    )
-                )
-            );
-        }
-    }
 }