Record collection problems, please

Tell us what’s happening:
There’s a condition that I can’t pass and I don’t know why.

  **Your code so far**

// Setup
const 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(prop !== 'tracks' && value !== ""){
  records[id][prop]=value;
}else if(prop === "tracks" && records[id].hasOwnProperty("tracks") === false ){
  records[id][prop]=[value];
}else if(prop === "tracks" && value !== ""){
  records[id][prop].push[value];
}else if(value === ""){
  delete records[id][prop];
}
return records;
}

updateRecords(recordCollection, 5439, 'artist', 'ABBA');
updateRecords(recordCollection, 5439, "tracks", "Take a Chance on Me");
updateRecords(recordCollection, 2548, "artist", "");
updateRecords(recordCollection, 1245, "tracks", "Addicted to Love")
  **Your browser information:**

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

Challenge: Record Collection

Link to the challenge:

function updateRecords(records, id, prop, value) {
  if (prop !== 'tracks' && value !== "") {
    records[id][prop] = value;
  } else if (prop === "tracks" && records[id].hasOwnProperty("tracks") === false) {
    records[id][prop] = [value];
  } else if (prop === "tracks" && value !== "") {
    // Is this the correct syntax for using 
    // .push on an array?
    records[id][prop].push[value];
  } else if (value === "") {
    delete records[id][prop];
  }
  return records;
}
2 Likes

Hi @CODEean ,

On line 30, make sure it is .push(value) with () not [] :

1 Like

I’ve seen my mistake,:woman_facepalming:t4: thank you Jeremy.

1 Like

Thank you spark, I’ve corrected myself. :melting_face:

1 Like

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