Просмотр исходного кода

Treat paths properly in project content update methods

James Allen 12 лет назад
Родитель
Сommit
6c7c09075c

+ 2 - 2
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?

+ 2 - 2
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", ->