Thank you for your contribution.

1. You probably can chain your calls such as `$TemporaryInvoice->getArticles()->setUser($this->getCustomer())->setCurrency($this->getCurrency()` as currently the `getArticles()` has been invoked twice.
2. Two places of comment-out code - better to remove them to keep your codebase clean.
3. When users switch from one currency to another, it's better to conert the value automatically using third party currency converter API.
4. No unit tests found to cover the functionalities.


Your contribution has been evaluated according to [Utopian policies and guidelines](https://join.utopian.io/guidelines), as well as a predefined set of questions pertaining to the category.

To view those questions and the relevant answers related to your post, [click here](https://review.utopian.io/result/3/2-3-2-2-2-2-1-).

---- 
Need help? Chat with us on [Discord](https://discord.gg/uTyJkNm).

[[utopian-moderator]](https://join.utopian.io/)