LiquiFi

The `createVestPlan` function is the main entry point to creating a vesting plan. It makes implicit assumptions on the input parameters that are not enforced by the contract, which can lead to mistakes such as accidentally sending additional funds.

An event should always be emitted when a state change is performed in order to facilitate smart contract monitoring by other systems which want to integrate with the smart contract. This is not the case for the functions and the correspondingly modified state variables: 1. `CliffModule.setVestingController()`, after changing `vestingControllerAddress`. 2. `CliffModule.createPlan()`, after changing `cliffPlans[]`. 3. `CliffModule.release()`, after changing `cliffPlans[]`. 4. `LinearModule.setVestingController()`, after changing `vestingControllerAddress`. 5. `LinearModule.createPlan()`, after changing `linearPlans[]`. 6. `LinearModule.release()`, after changing `linearPlans[]`. 7. `StreamModule.setVestingController()`, after changing `vestingControllerAddress`. 8. `StreamModule.createPlan()`, after changing `streamPlans[]`. 9. `StreamModule.release()`, after changing `streamPlans[]`. 10. `EmployeeRegistry.addOwner()`, after changing `companyOwners`, and `companyId`. 11. `EmployeeRegistry.addEmployee()`, after changing `companyIdToEmployees[]` and `employeeToCompanyIds`. 12. `EmployeeRegistry.transferOwner()`, after changing `companyOwners`. 13. `EmployeeRegistry.archiveEmployee()`, after changing `companyIdToEmployees[]`. 14. `Escrow.deposit()`, after changing `escrowBalance[][]`. 15. `Escrow.withdraw()`, after changing `escrowBalance[][]`.