search

yllekkram / sysc4907-video-store-system

1 stars 3 forks source link

The service layer does not appear to provide a way to create a new order. #39

Closed yllekkram closed 11 years ago

yllekkram commented 11 years ago

OrderServiceImpl provides methods to add and remove rental and purchases from an Order, but there does not appear to be a way to create/save a new order.

The Order creation doesn't need to be fancy; it just needs to create an object with the bare minimum of data. After that, rentals and purchases can be added to it.

yllekkram commented 11 years ago

The confirmPayement method requires an orderID as a parameter. The presentation layer should not be responsible for generating IDs for the entities.

smartelli commented 11 years ago

my next commit addresses this issue