I decided to format the image urls in the ItemModel. In the dao mulitple methods would need to do this, so I could have made it a helper method in the dao. But I ended up making it a part of the ItemModel, since I figured Item model was supposed to create the returned item and each image reference just needs an S3 bucket reference. I can move it later if needed, but for now that's where I thought it best fit.
I decided to format the image urls in the ItemModel. In the dao mulitple methods would need to do this, so I could have made it a helper method in the dao. But I ended up making it a part of the ItemModel, since I figured Item model was supposed to create the returned item and each image reference just needs an S3 bucket reference. I can move it later if needed, but for now that's where I thought it best fit.