Should this be calling a transaction method?