Skip to content

Commit 1295472

Browse files
refactor: code
1 parent 2f9d62b commit 1295472

1 file changed

Lines changed: 26 additions & 27 deletions

File tree

src/index.js

Lines changed: 26 additions & 27 deletions
Original file line numberDiff line numberDiff line change
@@ -806,16 +806,16 @@ class CopyPlugin {
806806
/**
807807
* @type {ObjectPattern}
808808
*/
809-
const normalizedPattern =
809+
const pattern =
810810
typeof item === "string" ? { from: item } : { ...item };
811811
const context =
812-
typeof normalizedPattern.context === "undefined"
812+
typeof pattern.context === "undefined"
813813
? compiler.context
814-
: path.isAbsolute(normalizedPattern.context)
815-
? normalizedPattern.context
816-
: path.join(compiler.context, normalizedPattern.context);
814+
: path.isAbsolute(pattern.context)
815+
? pattern.context
816+
: path.join(compiler.context, pattern.context);
817817

818-
normalizedPattern.context = context;
818+
pattern.context = context;
819819

820820
/**
821821
* @type {Array<CopiedResult | undefined> | undefined}
@@ -831,7 +831,7 @@ class CopyPlugin {
831831
cache,
832832
concurrency,
833833
/** @type {ObjectPattern & { context: string }} */
834-
(normalizedPattern),
834+
(pattern),
835835
index,
836836
);
837837
} catch (error) {
@@ -855,13 +855,13 @@ class CopyPlugin {
855855
(result) => result !== undefined,
856856
);
857857

858-
if (typeof normalizedPattern.transformAll !== "undefined") {
859-
if (typeof normalizedPattern.to === "undefined") {
858+
if (typeof pattern.transformAll !== "undefined") {
859+
if (typeof pattern.to === "undefined") {
860860
compilation.errors.push(
861861
/** @type {WebpackError} */
862862
(
863863
new Error(
864-
`Invalid "pattern.to" for the "pattern.from": "${normalizedPattern.from}" and "pattern.transformAll" function. The "to" option must be specified.`,
864+
`Invalid "pattern.to" for the "pattern.from": "${pattern.from}" and "pattern.transformAll" function. The "to" option must be specified.`,
865865
)
866866
),
867867
);
@@ -906,26 +906,25 @@ class CopyPlugin {
906906
const cacheItem = cache.getItemCache(
907907
`transformAll|${getSerializeJavascript()({
908908
version,
909-
from: normalizedPattern.from,
910-
to: normalizedPattern.to,
911-
transformAll: normalizedPattern.transformAll,
909+
from: pattern.from,
910+
to: pattern.to,
911+
transformAll: pattern.transformAll,
912912
})}`,
913913
mergedEtag,
914914
);
915915
let transformedAsset = await cacheItem.getPromise();
916916

917917
if (!transformedAsset) {
918-
transformedAsset = { filename: normalizedPattern.to };
918+
transformedAsset = { filename: pattern.to };
919919

920920
try {
921-
transformedAsset.data =
922-
await normalizedPattern.transformAll(
923-
filteredCopiedResult.map((asset) => ({
924-
data: asset.source.buffer(),
925-
sourceFilename: asset.sourceFilename,
926-
absoluteFilename: asset.absoluteFilename,
927-
})),
928-
);
921+
transformedAsset.data = await pattern.transformAll(
922+
filteredCopiedResult.map((asset) => ({
923+
data: asset.source.buffer(),
924+
sourceFilename: asset.sourceFilename,
925+
absoluteFilename: asset.absoluteFilename,
926+
})),
927+
);
929928
} catch (error) {
930929
compilation.errors.push(
931930
/** @type {WebpackError} */ (error),
@@ -935,9 +934,9 @@ class CopyPlugin {
935934
}
936935

937936
const filename =
938-
typeof normalizedPattern.to === "function"
939-
? await normalizedPattern.to({ context })
940-
: normalizedPattern.to;
937+
typeof pattern.to === "function"
938+
? await pattern.to({ context })
939+
: pattern.to;
941940

942941
if (template.test(filename)) {
943942
const contentHash = CopyPlugin.getContentHash(
@@ -967,15 +966,15 @@ class CopyPlugin {
967966
transformedAsset.source = new RawSource(
968967
transformedAsset.data,
969968
);
970-
transformedAsset.force = normalizedPattern.force;
969+
transformedAsset.force = pattern.force;
971970

972971
await cacheItem.storePromise(transformedAsset);
973972
}
974973

975974
filteredCopiedResult = [transformedAsset];
976975
}
977976

978-
const priority = normalizedPattern.priority || 0;
977+
const priority = pattern.priority || 0;
979978

980979
if (!copiedResultMap.has(priority)) {
981980
copiedResultMap.set(priority, new Map());

0 commit comments

Comments
 (0)