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

Merge pull request #440 from sharelatex/afc-fix-read-announcements-count

Fix read announcements count
James Allen 8 лет назад
Родитель
Сommit
0c89f488bd

+ 6 - 6
services/web/app/coffee/Features/Analytics/AnalyticsManager.coffee

@@ -17,17 +17,17 @@ makeFaultTolerantRequest = (userId, options, callback) ->
 	if settings.overleaf?
 		options.qs = Object.assign({}, options.qs, { fromV2: 1 })
 
-	makeRequest(options, callback)
+	makeRequest options, (err) ->
+		if err?
+			logger.err { err: err }, 'Request to analytics failed'
+
+	callback() # Do not wait for all the attempts
 
 makeRequest = (opts, callback)->
 	if settings.apis?.analytics?.url?
 		urlPath = opts.url
 		opts.url = "#{settings.apis.analytics.url}#{urlPath}"
-		request opts, (err) ->
-			if err?
-				logger.err { err: err }, 'Request to analytics failed'
-
-		callback() # Do not wait for all the attempts
+		request opts, callback
 	else
 		callback(new Errors.ServiceNotConfiguredError('Analytics service not configured'))