packageFileName is a proper closure

Author: Waldemar Zurowski

lib/files.js

@@ -4,28 +4,26 @@ const path = require('path');
 
 module.exports = {
   serviceFile: function (root, pkg) {
-    return path.resolve(root, packageFileName(pkg) + '.service');
+    return path.resolve(root, this.packageFileName(pkg) + '.service');
   },
   specsDirectory: function (root) {
     return path.resolve(root, 'SPECS');
   },
   specFile: function (root, pkg) {
-    return path.resolve(root, packageFileName(pkg) + '.spec');
+    return path.resolve(root, this.packageFileName(pkg) + '.spec');
   },
   sourcesDirectory: function (root) {
     return path.resolve(root, 'SOURCES');
   },
   sourcesArchive: function (root, pkg) {
     const sourcesDirectory = this.sourcesDirectory(root);
 
-    return path.resolve(sourcesDirectory, packageFileName(pkg) + '.tar.gz');
+    return path.resolve(sourcesDirectory, this.packageFileName(pkg) + '.tar.gz');
   },
-  packageFileName: packageFileName
+  packageFileName: function (pkg) {
+      return pkg.name[0] === '@'
+          // scoped packages get special treatment
+          ? pkg.name.substr(1).replace(/\//g, '-')
+          : pkg.name;
+  }
 };
-
-function packageFileName (pkg) {
-    return pkg.name[0] === '@'
-        // scoped packages get special treatment
-        ? pkg.name.substr(1).replace(/\//g, '-')
-        : pkg.name;
-}