ConcentratedLiquidityPoolManager Allowance Check Issue

[theKata]

referenced : ConcentratedLiquidityPoolManager.sol

Is there any reason why it is possible to call only msg.sender?

require(msg.sender == ownerOf(tokenId), "NOT_ID_OWNER"); 

If this happens, it will be difficult to create a 3rd party contract for the position NFT in the future. How about changing it to _isApprovedOrOwner as shown below?

require(_isApprovedOrOwner(msg.sender, tokenId), "NOT_ALLOWED");;

RELATED CODES

trident/contracts/pool/concentrated/ConcentratedLiquidityPoolManager.sol

Lines 92 to 94 in 8b183b1

	require(address(position.pool) == address(pool), "POOL_MIS_MATCH");
	require(position.lower == lower && position.upper == upper, "RANGE_MIS_MATCH");
	require(ownerOf(positionId) == msg.sender, "NOT_ID_OWNER");

trident/contracts/pool/concentrated/ConcentratedLiquidityPoolManager.sol

Lines 128 to 136 in 8b183b1

	function burn(
	uint256 tokenId,
	uint128 amount,
	address recipient,
	bool unwrapBento,
	uint256 minimumOut0,
	uint256 minimumOut1
	) external returns (uint256 token0Amount, uint256 token1Amount) {
	require(msg.sender == ownerOf(tokenId), "NOT_ID_OWNER");

trident/contracts/pool/concentrated/ConcentratedLiquidityPoolManager.sol

Lines 167 to 172 in 8b183b1

	function collect(
	uint256 tokenId,
	address recipient,
	bool unwrapBento
	) public returns (uint256 token0amount, uint256 token1amount) {
	require(msg.sender == ownerOf(tokenId), "NOT_ID_OWNER");

[theKata]

ConcentratedLiquidityPoolManager has not been deployed on the testnet yet, is there any reason?

[theKata]

It would be better to extend the parameter so that mint can separately specify the receiver