Skip to content

Commit 3f6d268

Browse files
feat: fixed connecting of credits to mediaItems
1 parent ac8ea51 commit 3f6d268

File tree

1 file changed

+11
-5
lines changed

1 file changed

+11
-5
lines changed

lesson_10/libraries/src/loaders/xavier_cruz_loader.ts

Lines changed: 11 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -16,10 +16,14 @@ export class XavierCruzLoader implements Loader {
1616
`Loaded ${credits.length} credits and ${mediaItems.length} media items`,
1717
);
1818

19+
let counter = 0;
1920
for (let i = 0; i < mediaItems.length; i++) {
2021
if (i < mediaItems.length / 2) {
21-
mediaItems[i].addCredit(credits[i] as Credit);
22+
mediaItems[i].addCredit(credits[counter] as Credit);
23+
counter++;
24+
mediaItems[i].addCredit(credits[counter] as Credit);
2225
}
26+
counter++;
2327
}
2428

2529
return [...mediaItems.values()];
@@ -44,14 +48,16 @@ export class XavierCruzLoader implements Loader {
4448
const fileContents = fs.readFileSync(filePath, 'utf-8');
4549

4650
const lines = fileContents.split('\n');
47-
const newString = lines.slice(1);
51+
const contentByLine = lines.slice(1);
4852

49-
for (let i = 0; i < newString.length; i++) {
50-
newString[i] = newString[i].substring(newString[i].indexOf(',') + 1);
53+
for (let i = 0; i < contentByLine.length; i++) {
54+
contentByLine[i] = contentByLine[i].substring(
55+
contentByLine[i].indexOf(',') + 1,
56+
);
5157
}
5258

5359
// help from ChatGPT - Fixing the roleStr as Role issue
54-
const credits: Credit[] = newString.map((credit) => {
60+
const credits: Credit[] = contentByLine.map((credit) => {
5561
const [mediaItemId, roleStr, name] = credit.split(',');
5662

5763
// cast roleStr to Role type

0 commit comments

Comments
 (0)