fix: make getOriginatorPackageName a free function

atom-minimap · Dec 31, 2020 · 3d67d03 · 3d67d03
1 parent 115ae4f
commit 3d67d03
Showing 1 changed file with 11 additions and 11 deletions.
diff --git a/lib/mixins/decoration-management.js b/lib/mixins/decoration-management.js
@@ -329,7 +329,7 @@ export default class DecorationManagement {
     const { type, plugin } = decorationParams
 
     if (plugin == null) {
-      decorationParams.plugin = this.getOriginatorPackageName()
+      decorationParams.plugin = getOriginatorPackageName()
     }
 
     if (decorationParams.scope == null && decorationParams.class != null) {
@@ -422,16 +422,6 @@ export default class DecorationManagement {
     return decoration
   }
 
-  getOriginatorPackageName () {
-    const line = new Error().stack.split('\n')[3]
-    const filePath = line.split('(')[1].replace(')', '')
-    const re = new RegExp(
-      atom.packages.getPackageDirPaths().join('|') + escapeRegExp(path.sep)
-    )
-    const plugin = filePath.replace(re, '').split(path.sep)[0].replace(/minimap-|-minimap/, '')
-    return plugin.indexOf(path.sep) < 0 ? plugin : undefined
-  }
-
   /**
    * Emits a change in the `Minimap` corresponding to the
    * passed-in decoration.
@@ -608,6 +598,16 @@ export default class DecorationManagement {
   }
 }
 
+function getOriginatorPackageName () {
+  const line = new Error().stack.split('\n')[3]
+  const filePath = line.split('(')[1].replace(')', '')
+  const re = new RegExp(
+    atom.packages.getPackageDirPaths().join('|') + escapeRegExp(path.sep)
+  )
+  const plugin = filePath.replace(re, '').split(path.sep)[0].replace(/minimap-|-minimap/, '')
+  return plugin.indexOf(path.sep) < 0 ? plugin : undefined
+}
+
 /**
  * Given two ranges, it returns an array of ranges representing the
  * differences between them.