Skip to content

Commit 4dbc856

Browse files
authored
Merge pull request #23 from vegarringdal/new_loader
chore(loader) clean up module path checking
2 parents 1863ef2 + f4542ba commit 4dbc856

File tree

2 files changed

+9
-20
lines changed

2 files changed

+9
-20
lines changed

dist/commonjs/fuse-box-aurelia-loader.js

Lines changed: 4 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -259,14 +259,10 @@ var FuseBoxAureliaLoader = (function (_super) {
259259
var moduleId = Object.keys(FuseBox.packages)
260260
.find(function (name) { return path.startsWith(name + "/"); });
261261
if (moduleId) {
262-
var resources = Object.keys(FuseBox.packages[moduleId].f);
263-
var resourceName_1 = path.replace(moduleId + "/", '');
264-
var resourceEntry = resources.find(function (r) { return r.endsWith(resourceName_1 + '.js'); });
265-
var resourceEntryCheck = resources.find(function (r) { return r.endsWith('/' + resourceName_1 + '.js'); });
266-
if (resourceEntry !== resourceEntryCheck) {
267-
resourceEntry = resourceEntryCheck;
268-
}
269-
retunValue = moduleId + "/" + resourceEntry;
262+
var parentEntry = FuseBox.packages[moduleId].s.entry;
263+
var resourceName = path.replace(moduleId, '');
264+
var entry = parentEntry.replace(/\/([^\/]+)\/?$/, resourceName);
265+
return moduleId + "/" + entry;
270266
}
271267
if (!this.fuseBoxExist(retunValue)) {
272268
debugPrint('error', 'findFuseBoxPath() failed to find', arguments);

src/fuse-box-aurelia-loader.ts

Lines changed: 5 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -122,7 +122,7 @@ export class FuseBoxAureliaLoader extends Loader {
122122
}
123123
});
124124

125-
125+
126126

127127
}
128128

@@ -345,17 +345,10 @@ export class FuseBoxAureliaLoader extends Loader {
345345
.find(name => path.startsWith(`${name}/`));
346346

347347
if (moduleId) {
348-
let resources = Object.keys(FuseBox.packages[moduleId].f);
349-
let resourceName = path.replace(`${moduleId}/`, '');
350-
let resourceEntry = resources.find(r => r.endsWith(resourceName + '.js'));
351-
352-
// fix for if resources is ending the same... this will fail if many sub folders and same file name
353-
// todo: improve, get root of package commonjs file to be sure.
354-
let resourceEntryCheck = resources.find(r => r.endsWith('/' + resourceName + '.js'));
355-
if (resourceEntry !== resourceEntryCheck) {
356-
resourceEntry = resourceEntryCheck;
357-
}
358-
retunValue = `${moduleId}/${resourceEntry}`;
348+
let parentEntry = FuseBox.packages[moduleId].s.entry;
349+
let resourceName = path.replace(moduleId, '');
350+
let entry = parentEntry.replace(/\/([^\/]+)\/?$/, resourceName);
351+
return `${moduleId}/${entry}`;
359352
}
360353

361354
if (!this.fuseBoxExist(retunValue)) {

0 commit comments

Comments
 (0)