Fix: Refactor CacheManager to use cachepath grunt option internally (#…

…3600)
adaptlearning · Sep 27, 2024 · f1bcd40 · f1bcd40
1 parent 22a6d9c
commit f1bcd40
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 3 deletions.
diff --git a/grunt/helpers/CacheManager.js b/grunt/helpers/CacheManager.js
@@ -24,8 +24,7 @@ module.exports = class CacheManager {
   }
 
   get tempPath() {
-    const tempPath = path.join(os.tmpdir(), 'adapt_framework');
-    return tempPath;
+    return this.grunt.option('cachepath') ?? path.join(os.tmpdir(), 'adapt_framework');
   }
 
   cachePath(basePath, outputFilePath = process.cwd()) {

diff --git a/grunt/tasks/javascript.js b/grunt/tasks/javascript.js
@@ -130,7 +130,7 @@ module.exports = function(grunt) {
     const options = this.options({});
     const isSourceMapped = Boolean(options.generateSourceMaps);
     const basePath = path.resolve(cwd + '/' + options.baseUrl).replace(convertSlashes, '/') + '/';
-    const cachePath = buildConfig.cachepath ?? cacheManager.cachePath(cwd, options.out);
+    const cachePath = cacheManager.cachePath(cwd, options.out);
     if (!isDisableCache) {
       grunt.log.ok(`Cache path: ${cachePath}`);
     }