@Transactional missing

[harkinj]

Hi,
I think the GameServiceImpl ::newAttemptForUser method should have the @transactional attribute added?
Thanks
John

[mechero]

Hi John,

That's an excellent finding. Yes, for a proper consistency between score and badges, it should. Otherwise, the user might end up with some extra score but without the associated badges. It's not critical, but it can be avoided with the annotation.

I take note of this for a second edition of the book, thank you!

[harkinj]

Thanks for the feedback, great book by the way :)