Browse Source

Merge pull request #3845 from overleaf/ho-upload-file-mark-updated

mark project as updated when uploading/replacing files

GitOrigin-RevId: 948a3351ac59d4f08411dd83bb543444de0b9b52
Simon Detheridge 5 years ago
parent
commit
9e2db00d53

+ 3 - 0
services/web/app/src/Features/Project/ProjectEntityUpdateHandler.js

@@ -533,6 +533,7 @@ const ProjectEntityUpdateHandler = {
               if (error != null) {
               if (error != null) {
                 return callback(error)
                 return callback(error)
               }
               }
+              ProjectUpdateHandler.markAsUpdated(projectId, new Date(), userId)
               callback(null, fileRef, folderId)
               callback(null, fileRef, folderId)
             }
             }
           )
           )
@@ -631,6 +632,8 @@ const ProjectEntityUpdateHandler = {
               if (err != null) {
               if (err != null) {
                 return callback(err)
                 return callback(err)
               }
               }
+              ProjectUpdateHandler.markAsUpdated(projectId, new Date(), userId)
+
               DocumentUpdaterHandler.updateProjectStructure(
               DocumentUpdaterHandler.updateProjectStructure(
                 projectId,
                 projectId,
                 projectHistoryId,
                 projectHistoryId,

+ 14 - 0
services/web/test/unit/src/Project/ProjectEntityUpdateHandlerTests.js

@@ -651,6 +651,13 @@ describe('ProjectEntityUpdateHandler', function() {
           .should.equal(true)
           .should.equal(true)
       })
       })
 
 
+      it('should mark the project as updated', function() {
+        const args = this.ProjectUpdater.markAsUpdated.args[0]
+        args[0].should.equal(projectId)
+        args[1].should.exist
+        args[2].should.equal(userId)
+      })
+
       it('sends the change in project structure to the doc updater', function() {
       it('sends the change in project structure to the doc updater', function() {
         const newFiles = [
         const newFiles = [
           {
           {
@@ -769,6 +776,13 @@ describe('ProjectEntityUpdateHandler', function() {
         .should.equal(true)
         .should.equal(true)
     })
     })
 
 
+    it('should mark the project as updated', function() {
+      const args = this.ProjectUpdater.markAsUpdated.args[0]
+      args[0].should.equal(projectId)
+      args[1].should.exist
+      args[2].should.equal(userId)
+    })
+
     it('updates the project structure in the doc updater', function() {
     it('updates the project structure in the doc updater', function() {
       const oldFiles = [
       const oldFiles = [
         {
         {