github/overleaf
1
0
Fork 0
mirror of https://github.com/overleaf/overleaf.git synced 2024-11-21 20:47:08 -05:00
Code Issues Projects Releases Packages Wiki Activity

Merge pull request #26 from sharelatex/ja-wrap-track-changes-in-try-catch

Browse source 
Wrap RangeTracker in try catch
This commit is contained in:
James Allen 2017-03-06 19:25:16 +00:00 committed by GitHub
commit 16ebd155ec
1 changed files with 4 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
services/document-updater/app/coffee/RangesManager.coffee
View file

@ -13,7 +13,10 @@ module.exports = RangesManager =
if !!update.meta.tc if !!update.meta.tc
rangesTracker.setIdSeed(update.meta.tc) rangesTracker.setIdSeed(update.meta.tc)
for op in update.op for op in update.op
try
rangesTracker.applyOp(op, { user_id: update.meta?.user_id }) rangesTracker.applyOp(op, { user_id: update.meta?.user_id })
catch error
return callback(error)
if rangesTracker.changes?.length > RangesManager.MAX_CHANGES or rangesTracker.comments?.length > RangesManager.MAX_COMMENTS if rangesTracker.changes?.length > RangesManager.MAX_CHANGES or rangesTracker.comments?.length > RangesManager.MAX_COMMENTS
return callback new Error("too many comments or tracked changes") return callback new Error("too many comments or tracked changes")