From a54a69159a76088f2eb248b925ca6286e0ac6b7a Mon Sep 17 00:00:00 2001 From: James Allen Date: Fri, 6 Jul 2018 09:07:55 +0100 Subject: [PATCH] Revert "Add in option to sync all users" - too dangerous! This reverts commit 218e042aba6784f32777f496e1706300845c3f44. --- services/web/scripts/sync_v1_subscriptions.js | 16 ++++------------ 1 file changed, 4 insertions(+), 12 deletions(-) diff --git a/services/web/scripts/sync_v1_subscriptions.js b/services/web/scripts/sync_v1_subscriptions.js index c043a5589c..2f56eb7f56 100644 --- a/services/web/scripts/sync_v1_subscriptions.js +++ b/services/web/scripts/sync_v1_subscriptions.js @@ -1,13 +1,3 @@ -let USER_QUERY = null -if (process.argv[2] === '--all') { - USER_QUERY = {} -} else if (process.argv[2] === '--v1-only') { - USER_QUERY = { 'overleaf.id': { $exists: true } } -} else { - console.error(`Usage:\n\n\tnode ${process.argv[1]} (--all|--v1-only)`) - process.exit(1) -} - const {db} = require('../app/js/infrastructure/mongojs') const FeaturesUpdater = require( '../app/js/Features/Subscription/FeaturesUpdater' @@ -19,7 +9,7 @@ const async = require('async') const logger = require('logger-sharelatex') logger.logger.level('error') -const areFeaturesEqual = function (featuresA, featuresB) { +const areFeaturesEqual = function(featuresA, featuresB) { for (const feature in featuresA) { if (featuresA[feature] !== featuresB[feature]) { return false @@ -31,7 +21,9 @@ const areFeaturesEqual = function (featuresA, featuresB) { var outOfSyncUserCount = 0 var userCount = null -db.users.find(USER_QUERY, { +db.users.find({ + 'overleaf.id': { $exists: true } +}, { overleaf: 1, features: 1 }, function (error, users) {