json.push is not a function [closed]

Question

Closed. This question needs debugging details. It is not currently accepting answers.

Edit the question to include desired behavior, a specific problem or error, and the shortest code necessary to reproduce the problem. This will help others answer the question.

Closed 4 years ago.

Improve this question

I am trying to make a command command that allows the author to set up a command for their server, This is meant to update a json file not overwrite it, I am using node.js and discord.js

const fs = require('fs')
module.exports = {
    name: 'setsuggest',
    execute(client, message, args, discord, cmd){
        const channelid = args[0]
        const serverid = message.guild.id
        
    
        fs.readFile('suggest.json', function (err, JsonData) {
            var json = JSON.parse(JsonData)
            let data = { 
                [serverid]: channelid 
            };
            json.push(data)
            fs.writeFile('suggest.json', JSON.stringify(json), function (err) {
                if (err) throw err;
                console.log('Updated')
            })
        })
        
    }
}

What am I doing wrong?

I suspect json is an object not an array. stackoverflow.com/questions/8925820/… — Matt
– Matt, Commented Jun 30, 2021 at 20:03

Flygenring · Accepted Answer · 2021-06-30 20:15:02Z

1

If json is an object, as it seems from your usage, you could use the spread operator to populate a new object with all properties from the parsed object and the add the property from serverid to it:

var json = {...JSON.parse(JsonData), [serverid]: channelid};

Otherwise you can always just assign the property in the parsed object directly:

var json = JSON.parse(JsonData);
json[serverid] = channelid;

edited Jun 30, 2021 at 20:15

answered Jun 30, 2021 at 20:05

Flygenring

3,8481 gold badge35 silver badges40 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

json.push is not a function [closed]

1 Answer 1

Comments

Linked

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Linked

Related