[ERC7540Vault] Wrong parameters in Deposit event

[eyqs]

The current implementation of deposit in ERC7540Vault.sol is:

function deposit(uint256 assets, address receiver, address controller) public returns (uint256 shares) {
    _validateController(controller);
    shares = manager.deposit(address(this), assets, receiver, controller);
    emit Deposit(receiver, controller, assets, shares);
}

But the ERC-7540 spec states that:

When the Deposit event is emitted, the first parameter MUST be the controller, and the second parameter MUST be the receiver.

The correct implementation should match the ERC-4626 spec, where controller has exchanged assets for shares (on behalf of owner in requestDeposit) and transferred those shares to receiver:

emit Deposit(controller, receiver, assets, shares);