search

jorgejcabrera / rent-app-service

0 stars 0 forks source link

no valida ningun estado, creo que no se puede prestar dos veces la misma cosa no? #3

Closed maurociancio closed 2 years ago

maurociancio commented 2 years ago

https://github.com/jorgejcabrera/rent-app-service/blob/cb8302b1891bcd4b23c6381ca89f201da98f504a/src/main/groovy/com/fiuba/rent_app/domain/item/Item.groovy#L77

jorgejcabrera commented 2 years ago

Hola Mauro, correcto. El servicio se encarga de realizar ese chequeo. https://github.com/jorgejcabrera/rent-app-service/blob/master/src/main/groovy/com/fiuba/rent_app/domain/order/service/OrderServiceAdapter.groovy#L21

maurociancio commented 2 years ago

Debe estar realizado en las clases del dominio.

jorgejcabrera commented 2 years ago

Hola Mauro, ahora el item sabe si puede ser rentado o no. Era esto lo que me mencionabas?

maurociancio commented 2 years ago

sip

maurociancio commented 2 years ago 
void rent() {
    this.status = RENTED
}

ese codigo sigue en la entidad. la entidad deberia validar que no pueda prestarse dos veces

maurociancio commented 2 years ago

no dije nada

jorgejcabrera commented 2 years ago

Cierro el issue, porque ahora el item sabe a traves de las ordenes si puede ser rentado o no: si existe una orden abierta, no se podra generar una nueva, es decir el item no podra ser rentado nuevamente hasta que no se devuelva.