From 6c7c09075c62f51640eb9b37cf6e537565c82948 Mon Sep 17 00:00:00 2001 From: James Allen Date: Fri, 3 Oct 2014 11:33:18 +0100 Subject: [PATCH] Treat paths properly in project content update methods --- .../coffee/Features/ThirdPartyDataStore/TpdsController.coffee | 4 ++-- .../coffee/ThirdPartyDataStore/TpdsControllerTests.coffee | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/services/web/app/coffee/Features/ThirdPartyDataStore/TpdsController.coffee b/services/web/app/coffee/Features/ThirdPartyDataStore/TpdsController.coffee index eb5fd43e3c..15e2179053 100644 --- a/services/web/app/coffee/Features/ThirdPartyDataStore/TpdsController.coffee +++ b/services/web/app/coffee/Features/ThirdPartyDataStore/TpdsController.coffee @@ -35,7 +35,7 @@ module.exports = updateProjectContents: (req, res, next = (error) ->) -> {project_id} = req.params - path = req.params[0] + path = "/" + req.params[0] # UpdateMerger expects leading slash logger.log project_id: project_id, path: path, "received project contents update" UpdateMerger.mergeUpdate project_id, path, req, (error) -> return next(error) if error? @@ -44,7 +44,7 @@ module.exports = deleteProjectContents: (req, res, next = (error) ->) -> {project_id} = req.params - path = req.params[0] + path = "/" + req.params[0] # UpdateMerger expects leading slash logger.log project_id: project_id, path: path, "received project contents delete request" UpdateMerger.deleteUpdate project_id, path, (error) -> return next(error) if error? diff --git a/services/web/test/UnitTests/coffee/ThirdPartyDataStore/TpdsControllerTests.coffee b/services/web/test/UnitTests/coffee/ThirdPartyDataStore/TpdsControllerTests.coffee index 0f7e5a9f71..12bc57df8d 100644 --- a/services/web/test/UnitTests/coffee/ThirdPartyDataStore/TpdsControllerTests.coffee +++ b/services/web/test/UnitTests/coffee/ThirdPartyDataStore/TpdsControllerTests.coffee @@ -86,7 +86,7 @@ describe 'TpdsController', -> it "should merge the update", -> @UpdateMerger.mergeUpdate - .calledWith(@project_id, @path, @req) + .calledWith(@project_id, "/" + @path, @req) .should.equal true it "should return a success", -> @@ -111,7 +111,7 @@ describe 'TpdsController', -> it "should delete the file", -> @UpdateMerger.deleteUpdate - .calledWith(@project_id, @path) + .calledWith(@project_id, "/" + @path) .should.equal true it "should return a success", ->