7 Commits

Author	SHA1	Message	Date
Fabien O'Carroll	66143dbb7c	Updated options parameter to be optional ... no-issue Since updating the product repository to force transactions, the options parameter was used in every call, meaning it wasn't optional any more, which broke usage. This updates the parameter to have a default so that existing usage still works.	2021-09-06 12:47:19 +02:00
Fabien O'Carroll	3b94ba7dce	Fixed update method not using transaction for reads ... no-issue Since we run our product repository methods in transactions now we must ensure that all database interations in the method use the transaction. This adds the missing options to the reading of existing prices so that they happen inside of the transaction.	2021-09-01 19:10:12 +02:00
Fabien O'Carroll	82506e1599	Passed transaction to all model methods ... refs https://github.com/TryGhost/Team/issues/982 These calls to the edit method were missing the transaction option from the parent which meant that they ran outside of the transaction and would cause the method to timeout.	2021-08-26 20:04:06 +02:00
Fabien O'Carroll	dd4d6aeae5	Wrapped product repo methods in transactions ... refs https://github.com/TryGhost/Team/issues/982 This ensures that we will not commit any rows to the database if something is to go wrong with a Stripe API request.	2021-08-26 20:03:16 +02:00
Fabien O'Carroll	6693d470d0	Removed superfluous benefits relation fetch ... refs https://github.com/TryGhost/Team/issues/919 As we pass the `benefits` to the Product model on creation, we do not need to manually fetch them again. In fact doing so causes a strange SQL error, where we attempt to run `SELECT undefined.*`.	2021-07-20 12:36:41 +01:00
Fabien O'Carroll	3e1084905e	Removed usage of raw Error class ... refs https://github.com/TryGhost/Team/issues/879	2021-07-14 14:17:38 +01:00
Fabien O'Carroll	d51fdc3f4a	Moved code out of index.js in directories ... refs https://github.com/TryGhost/Team/issues/879	2021-07-14 14:17:38 +01:00