Merge pull request #205 from overleaf/revert-200-bg-decaff-cleanup

Revert "decaff cleanup ResourceStateManager"

services/clsi/app/js/ResourceStateManager.js

@@ -1,10 +1,27 @@
+/* eslint-disable
+    handle-callback-err,
+    no-unused-vars,
+*/
+// TODO: This file was created by bulk-decaffeinate.
+// Fix any style issues and re-enable lint.
+/*
+ * decaffeinate suggestions:
+ * DS101: Remove unnecessary use of Array.from
+ * DS102: Remove unnecessary code created because of implicit returns
+ * DS103: Rewrite code to no longer use __guard__
+ * DS201: Simplify complex destructure assignments
+ * DS207: Consider shorter variations of null checks
+ * Full docs: https://github.com/decaffeinate/decaffeinate/blob/master/docs/suggestions.md
+ */
+let ResourceStateManager
 const Path = require('path')
 const fs = require('fs')
 const logger = require('logger-sharelatex')
+const settings = require('settings-sharelatex')
 const Errors = require('./Errors')
 const SafeReader = require('./SafeReader')
 
-module.exports = {
+module.exports = ResourceStateManager = {
   // The sync state is an identifier which must match for an
   // incremental update to be allowed.
   //
@@ -23,12 +40,15 @@ module.exports = {
   SYNC_STATE_MAX_SIZE: 128 * 1024,
 
   saveProjectState(state, resources, basePath, callback) {
+    if (callback == null) {
+      callback = function (error) {}
+    }
     const stateFile = Path.join(basePath, this.SYNC_STATE_FILE)
     if (state == null) {
       // remove the file if no state passed in
       logger.log({ state, basePath }, 'clearing sync state')
-      fs.unlink(stateFile, function (err) {
+      return fs.unlink(stateFile, function (err) {
-        if (err && err.code !== 'ENOENT') {
+        if (err != null && err.code !== 'ENOENT') {
           return callback(err)
         } else {
           return callback()
@@ -36,24 +56,29 @@ module.exports = {
       })
     } else {
       logger.log({ state, basePath }, 'writing sync state')
-      const resourceList = resources.map((resource) => resource.path)
+      const resourceList = Array.from(resources).map(
-      fs.writeFile(
+        (resource) => resource.path
+      )
+      return fs.writeFile(
         stateFile,
-        [...resourceList, `stateHash:${state}`].join('\n'),
+        [...Array.from(resourceList), `stateHash:${state}`].join('\n'),
         callback
       )
     }
   },
 
   checkProjectStateMatches(state, basePath, callback) {
+    if (callback == null) {
+      callback = function (error, resources) {}
+    }
     const stateFile = Path.join(basePath, this.SYNC_STATE_FILE)
     const size = this.SYNC_STATE_MAX_SIZE
-    SafeReader.readFile(stateFile, size, 'utf8', function (
+    return SafeReader.readFile(stateFile, size, 'utf8', function (
       err,
       result,
       bytesRead
     ) {
-      if (err) {
+      if (err != null) {
         return callback(err)
       }
       if (bytesRead === size) {
@@ -62,7 +87,10 @@ module.exports = {
           'project state file truncated'
         )
       }
-      const array = result.toString().split('\n')
+      const array =
+        __guard__(result != null ? result.toString() : undefined, (x) =>
+          x.split('\n')
+        ) || []
       const adjustedLength = Math.max(array.length, 1)
       const resourceList = array.slice(0, adjustedLength - 1)
       const oldState = array[adjustedLength - 1]
@@ -76,27 +104,36 @@ module.exports = {
           new Errors.FilesOutOfSyncError('invalid state for incremental update')
         )
       } else {
-        const resources = resourceList.map((path) => ({ path }))
+        const resources = Array.from(resourceList).map((path) => ({ path }))
-        callback(null, resources)
+        return callback(null, resources)
       }
     })
   },
 
   checkResourceFiles(resources, allFiles, basePath, callback) {
     // check the paths are all relative to current directory
-    const containsRelativePath = (resource) => {
+    let file
-      const dirs = resource.path.split('/')
+    if (callback == null) {
-      return dirs.indexOf('..') !== -1
+      callback = function (error) {}
     }
-    if (resources.some(containsRelativePath)) {
+    for (file of Array.from(resources || [])) {
-      return callback(new Error('relative path in resource file list'))
+      for (const dir of Array.from(
+        __guard__(file != null ? file.path : undefined, (x) => x.split('/'))
+      )) {
+        if (dir === '..') {
+          return callback(new Error('relative path in resource file list'))
+        }
+      }
     }
     // check if any of the input files are not present in list of files
-    const seenFiles = new Set(allFiles)
+    const seenFile = {}
-    const missingFiles = resources
+    for (file of Array.from(allFiles)) {
+      seenFile[file] = true
+    }
+    const missingFiles = Array.from(resources)
+      .filter((resource) => !seenFile[resource.path])
       .map((resource) => resource.path)
-      .filter((path) => !seenFiles.has(path))
+    if ((missingFiles != null ? missingFiles.length : undefined) > 0) {
-    if (missingFiles.length > 0) {
       logger.err(
         { missingFiles, basePath, allFiles, resources },
         'missing input files for project'
@@ -107,7 +144,13 @@ module.exports = {
         )
       )
     } else {
-      callback()
+      return callback()
     }
   }
 }
+
+function __guard__(value, transform) {
+  return typeof value !== 'undefined' && value !== null
+    ? transform(value)
+    : undefined
+}