TryGhost/Ghost
1
0
Fork 0
mirror of https://github.com/TryGhost/Ghost.git synced 2024-12-18 07:51:55 +03:00
Code Issues Projects Releases Wiki Activity

Fixed stripe migration to avoid complimentary prices as monthly/yearly price

Browse Source 
closes https://github.com/TryGhost/Team/issues/778

- cleans up the stripe migration to add default monthly/yearly prices for sites, which had a possibility of using complimentary (0 amount prices) in edge cases
- adds missing return in the same migration for an unlikely failure to parse stripe plans
This commit is contained in:
Rishabh 2021-07-16 13:01:55 +05:30 committed by Rishabh Garg
parent ceec3efe6a
commit 069accdbe8
1 changed files with 5 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
ghost/members-api/lib/migrations.js
View File

@ -262,6 +262,7 @@ module.exports = class StripeMigrations {
plans = JSON.parse(stripePlans.get('value'));
} catch (err) {
this._logging.warn('Skipping population of members_monthly_price_id, could not parse stripe_plans');
return;
}
const monthlyPlan = plans.find((plan) => {
@ -284,10 +285,8 @@ module.exports = class StripeMigrations {
if (!monthlyPrice) {
this._logging.info('Could not find active Monthly price from stripe_plans - searching by interval');
monthlyPrice = await this._StripePrice.findOne({
interval: 'month',
active: true
});
monthlyPrice = await this._StripePrice.where('amount', '>', 0)
.where({interval: 'month', active: true}).fetch();
}
if (!monthlyPrice) {
@ -357,10 +356,8 @@ module.exports = class StripeMigrations {
if (!yearlyPrice) {
this._logging.info('Could not find active yearly price from stripe_plans - searching by interval');
yearlyPrice = await this._StripePrice.findOne({
interval: 'year',
active: true
});
yearlyPrice = await this._StripePrice.where('amount', '>', 0)
.where({interval: 'year', active: true}).fetch();
}
if (!yearlyPrice) {