Revert "Initialise full project history for old projects when project opened (#4687)" (#4740)

This reverts commit ac263dca8cf0d80186fee916a76e5572ec5649d4.

GitOrigin-RevId: cc82e9c7d51966a7be0e8bef06ae4afe3f73170c

services/web/app/src/Features/History/HistoryManager.js

@@ -8,14 +8,12 @@ module.exports = {
   initializeProject: callbackify(initializeProject),
   flushProject: callbackify(flushProject),
   resyncProject: callbackify(resyncProject),
-  forceResyncProject: callbackify(forceResyncProject),
   deleteProject: callbackify(deleteProject),
   injectUserDetails: callbackify(injectUserDetails),
   promises: {
     initializeProject,
     flushProject,
     resyncProject,
-    forceResyncProject,
     deleteProject,
     injectUserDetails,
   },
@@ -67,18 +65,6 @@ async function resyncProject(projectId) {
   }
 }
 
-async function forceResyncProject(projectId) {
-  try {
-    await request.post({
-      url: `${settings.apis.project_history.url}/project/${projectId}/resync?force=true`,
-    })
-  } catch (err) {
-    throw OError.tag(err, 'failed to force resync project history', {
-      projectId,
-    })
-  }
-}
-
 async function deleteProject(projectId, historyId) {
   try {
     const tasks = [

services/web/app/src/Features/Project/ProjectController.js

@@ -9,7 +9,6 @@ const { ObjectId } = require('mongodb')
 const ProjectDeleter = require('./ProjectDeleter')
 const ProjectDuplicator = require('./ProjectDuplicator')
 const ProjectCreationHandler = require('./ProjectCreationHandler')
-const ProjectHistoryHandler = require('./ProjectHistoryHandler')
 const EditorController = require('../Editor/EditorController')
 const ProjectHelper = require('./ProjectHelper')
 const metrics = require('@overleaf/metrics')
@@ -681,24 +680,6 @@ const ProjectController = {
         activate(cb) {
           InactiveProjectManager.reactivateProjectIfRequired(projectId, cb)
         },
-        ensureHistoryExists(cb) {
-          // enable full project history in background for older projects
-          if (!Settings.apis.project_history || !Features.hasFeature('saas')) {
-            return cb()
-          }
-          ProjectHistoryHandler.ensureHistoryExistsForProject(
-            projectId,
-            err => {
-              if (err) {
-                logger.error(
-                  { err, projectId },
-                  'error ensuring history exists for project'
-                )
-              }
-              cb()
-            }
-          )
-        },
         markAsOpened(cb) {
           // don't need to wait for this to complete
           ProjectUpdateHandler.markAsOpened(projectId, () => {})

services/web/app/src/Features/Project/ProjectHistoryHandler.js

@@ -19,7 +19,6 @@ const logger = require('logger-sharelatex')
 const settings = require('@overleaf/settings')
 const HistoryManager = require('../History/HistoryManager')
 const ProjectEntityUpdateHandler = require('./ProjectEntityUpdateHandler')
-const DocumentUpdaterHandler = require('../DocumentUpdater/DocumentUpdaterHandler')
 const { promisifyAll } = require('../../util/promises')
 
 const ProjectHistoryHandler = {
@@ -135,44 +134,31 @@ const ProjectHistoryHandler = {
         if (history_id != null) {
           return callback()
         } // history already exists, success
-        return HistoryManager.flushProject(project_id, function (err) {
+        return HistoryManager.initializeProject(function (err, history) {
           if (err != null) {
             return callback(err)
           }
-          return HistoryManager.initializeProject(function (err, history) {
+          if (!(history != null ? history.overleaf_id : undefined)) {
-            if (err != null) {
+            return callback(new Error('failed to initialize history id'))
-              return callback(err)
+          }
-            }
+          return ProjectHistoryHandler.setHistoryId(
-            if (!history || !history.overleaf_id) {
+            project_id,
-              return callback(new Error('failed to initialize history id'))
+            history.overleaf_id,
-            }
+            function (err) {
-            return ProjectHistoryHandler.setHistoryId(
+              if (err != null) {
-              project_id,
+                return callback(err)
-              history.overleaf_id,
-              function (err) {
-                if (err != null) {
-                  return callback(err)
-                }
-                return DocumentUpdaterHandler.flushProjectToMongoAndDelete(
-                  project_id,
-                  function (err) {
-                    if (err != null) {
-                      return callback(err)
-                    }
-                    return HistoryManager.forceResyncProject(
-                      project_id,
-                      function (err) {
-                        if (err != null) {
-                          return callback(err)
-                        }
-                        return HistoryManager.flushProject(project_id, callback)
-                      }
-                    )
-                  }
-                )
               }
-            )
+              return ProjectEntityUpdateHandler.resyncProjectHistory(
-          })
+                project_id,
+                function (err) {
+                  if (err != null) {
+                    return callback(err)
+                  }
+                  return HistoryManager.flushProject(project_id, callback)
+                }
+              )
+            }
+          )
         })
       }
     )

services/web/test/acceptance/src/mocks/MockProjectHistoryApi.js

@@ -128,10 +128,6 @@ class MockProjectHistoryApi extends AbstractMockApi {
     this.app.post('/project/:projectId/flush', (req, res) => {
       res.sendStatus(200)
     })
-
-    this.app.post('/project/:projectId/resync', (req, res) => {
-      res.sendStatus(204)
-    })
   }
 }
 

services/web/test/unit/src/Project/ProjectControllerTests.js

@@ -54,9 +54,6 @@ describe('ProjectController', function () {
         .stub()
         .callsArgWith(2, null, { _id: this.project_id }),
     }
-    this.ProjectHistoryHandler = {
-      ensureHistoryExistsForProject: sinon.stub().callsArg(1),
-    }
     this.SubscriptionLocator = { getUsersSubscription: sinon.stub() }
     this.LimitationsManager = { hasPaidSubscription: sinon.stub() }
     this.TagsHandler = { getAllTags: sinon.stub() }
@@ -143,7 +140,6 @@ describe('ProjectController', function () {
         './ProjectDeleter': this.ProjectDeleter,
         './ProjectDuplicator': this.ProjectDuplicator,
         './ProjectCreationHandler': this.ProjectCreationHandler,
-        './ProjectHistoryHandler': this.ProjectHistoryHandler,
         '../Editor/EditorController': this.EditorController,
         '../User/UserController': this.UserController,
         './ProjectHelper': this.ProjectHelper,
@@ -1028,18 +1024,6 @@ describe('ProjectController', function () {
       this.ProjectController.loadEditor(this.req, this.res)
     })
 
-    it('should ensureHistoryExistsForProject if saas and project_history enabled', function (done) {
-      this.Features.hasFeature.withArgs('saas').returns(true)
-      this.settings.apis.project_history = 'enabled'
-      this.res.render = (pageName, opts) => {
-        this.ProjectHistoryHandler.ensureHistoryExistsForProject
-          .calledWith(this.project_id)
-          .should.equal(true)
-        done()
-      }
-      this.ProjectController.loadEditor(this.req, this.res)
-    })
-
     it('should mark project as opened', function (done) {
       this.res.render = (pageName, opts) => {
         this.ProjectUpdateHandler.markAsOpened

services/web/test/unit/src/Project/ProjectHistoryHandlerTests.js

@@ -51,7 +51,6 @@ describe('ProjectHistoryHandler', function () {
         './ProjectDetailsHandler': (this.ProjectDetailsHandler = {}),
         '../History/HistoryManager': (this.HistoryManager = {}),
         './ProjectEntityUpdateHandler': (this.ProjectEntityUpdateHandler = {}),
-        '../DocumentUpdater/DocumentUpdaterHandler': (this.DocumentUpdaterHandler = {}),
       },
     }))
   })
@@ -63,10 +62,9 @@ describe('ProjectHistoryHandler', function () {
         .stub()
         .callsArgWith(0, null, { overleaf_id: this.newHistoryId })
       this.HistoryManager.flushProject = sinon.stub().callsArg(1)
-      this.HistoryManager.forceResyncProject = sinon.stub().callsArg(1)
+      return (this.ProjectEntityUpdateHandler.resyncProjectHistory = sinon
-      this.DocumentUpdaterHandler.flushProjectToMongoAndDelete = sinon
         .stub()
-        .callsArg(1)
+        .callsArg(1))
     })
 
     describe('when the history does not already exist', function () {
@@ -103,21 +101,14 @@ describe('ProjectHistoryHandler', function () {
           .should.equal(true)
       })
 
-      it('should trigger a hard resync of the project history', function () {
+      it('should resync the project history', function () {
-        return this.HistoryManager.forceResyncProject
+        return this.ProjectEntityUpdateHandler.resyncProjectHistory
           .calledWith(project_id)
           .should.equal(true)
       })
 
-      it('should flush the project history (twice)', function () {
+      it('should flush the project history', function () {
-        this.HistoryManager.flushProject.calledTwice.should.equal(true)
         return this.HistoryManager.flushProject
-          .alwaysCalledWith(project_id)
-          .should.equal(true)
-      })
-
-      it('should tell docupdater to flush and delete', function () {
-        return this.DocumentUpdaterHandler.flushProjectToMongoAndDelete
           .calledWith(project_id)
           .should.equal(true)
       })
@@ -155,8 +146,10 @@ describe('ProjectHistoryHandler', function () {
         return this.ProjectModel.updateOne.called.should.equal(false)
       })
 
-      it('should not trigger a hard resync of the project history', function () {
+      it('should not resync the project history', function () {
-        return this.HistoryManager.forceResyncProject.called.should.equal(false)
+        return this.ProjectEntityUpdateHandler.resyncProjectHistory.called.should.equal(
+          false
+        )
       })
 
       it('should not flush the project history', function () {