Infrastructure/ghost
0
Fork 0
mirror of https://github.com/TryGhost/Ghost.git synced 2025-03-11 02:12:21 -05:00
Code Issues Activity

Moved verified email check closer to freeze logic

Browse source 
refs https://github.com/TryGhost/Team/issues/912

- Previous logic was a bit misleading because it prevented from reading the real threshold configured with an instance once the verified flag was present in the config.
- The reshuffle made here allows to check the freeze logic based on the threshold and then process the returned result accordingly instead of having hidden logic behind "importThreshold" config value
This commit is contained in:
Naz 2021-07-27 15:44:08 +04:00 • committed by naz
parent 8bc4d00fe6
commit 086840873e
1 changed files with 3 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
core/server/services/members/service.js
View file

@ -54,10 +54,6 @@ function reconfigureMembersAPI() {
}
const getThreshold = () => {
if (_.get(config.get('hostSettings'), 'emailVerification.verified')) {
return null;
}
return _.get(config.get('hostSettings'), 'emailVerification.importThreshold');
};
@ -78,7 +74,9 @@ const processImport = async (options) => {
const freezeTriggered = result.meta.freeze;
delete result.meta.freeze;
if (freezeTriggered) {
const isVerifiedEmail = config.get('hostSettings:emailVerification:verified') === true;
if ((!isVerifiedEmail) && freezeTriggered) {
limits.init({
emails: {
disabled: true,