Problem with Record Collection of Basic Javascript course

Hi, I’m having a problem to get the Record Collection exercise approved, even when the code seems to be performing the required tasks.
On the screenshot, the console shows that after running the update method with the given arguments the Bon Jovi album has not the “tracks” property anymore , but the test seems not to consider it. I don’t know what else to do.
I will appretiate any solution for this, thanks.


// Setup
var recordCollection = {
2548: {
  albumTitle: 'Slippery When Wet',
  artist: 'Bon Jovi',
  tracks: ['Let It Rock', 'You Give Love a Bad Name']
},
2468: {
  albumTitle: '1999',
  artist: 'Prince',
  tracks: ['1999', 'Little Red Corvette']
},
1245: {
  artist: 'Robert Palmer',
  tracks: []
},
5439: {
  albumTitle: 'ABBA Gold'
}
};

// Only change code below this line
function updateRecords(records, id, prop, value) {

if(value==="")
  delete recordCollection[id][prop];
else if(prop!=="tracks")
  recordCollection[id][prop]=value;
else if(prop==="tracks"){
  if(recordCollection[id].hasOwnProperty(prop)===false)
    recordCollection[id][prop]=[value];
  else if(value!=="")
    recordCollection[id][prop].push(value);
} 
return recordCollection;
}
console.log(updateRecords(recordCollection, 2548, "tracks", ""));
updateRecords(recordCollection, 5439, 'artist', 'ABBA');

  **Your browser information:**

User Agent is: Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/91.0.4472.124 Safari/537.36

Challenge: Record Collection

Link to the challenge:

You must use the function argument

instead of directly referencing the global object. It’s always best avoid directly mutating global variables when able.

Thanks for the fast answer. Yes, I forgot to use the record argument to change the object. Now it gives me different errors, wichs makes me more confused since the code logic still being similar but these errors didnt appear before

You’re still returning the global variable. It should look OK if you fix that, I think.

Yes, it was that. Now it worked. Thank you so much

1 Like

This topic was automatically closed 182 days after the last reply. New replies are no longer allowed.