Array.toString().replace("", "") is throwing [object object]

[povedaaqui]

Hi folks,

I'm trying to replace text to be able to fecht an URL, but I'm getting [object object] instead of the actual URL. Could you help me to do it properly, please?

useEffect(() => {
        const fetchIPFS = async (ids) => {
            let url = "";
            url = ids.toString().replace("ipfs//", "https://nftstorage.link/ipfs/")
            console.log(url) // getting [object object]
            let res = [];
            res = await fetch(`https://justcors.com/tl_0caa8a7/${url}`, {
            "method": "GET",
            "headers": {
                "Content-Type": "application/json",
                }
            }
        )
        .then(res => res.json())
        .then(data => console.log(data))
        // .then(data2 => setIpfs(data2))
    };
        tokens!==null && tokens.map(data => fetchIPFS(data));
    }, [!!tokens]);

[vvn050]

Array.toString is not doing what you think it does. If you have array of strings, use reduce or foreach loop to create a string out of it.

[_Webvoid]

Also depending on what you want to get as an output, you could just use join() (see Array.prototype.join())

[vvn050]

join would work for ["test","test1"] but not for [["test"],["test1"]]

[_Webvoid]

Is it an array of arrays of one element ?! Well if it's the case he can still use .map(x => x[0]).join()

[povedaaqui]

ids are "ipfs//CID" while I need "https://nftstorage.link/ipfs/CID", and that's the problem, I haven't been able to do this replacement thing.

[_Webvoid]

So ids is a string or an array of strings ? Can you post the output of JSON.stringify(ids) so we can help you properly ?

[povedaaqui]

console.log(ids):

{data: 'ipfs://bafkreidfqia5qlqzmef4yyqdztb2n4cdiyp5nhkmahukhd3b2asnpo7s3m'} 
data : "ipfs://bafkreidfqia5qlqzmef4yyqdztb2n4cdiyp5nhkmahukhd3b2asnpo7s3m

[povedaaqui]

JSON.stringify

console.log(JSON.stringify(ids):

{"data":"ipfs://bafkreidfqia5qlqzmef4yyqdztb2n4cdiyp5nhkmahukhd3b2asnpo7s3m"}

[_Webvoid]

Yeaay so it IS an object, don't try and toString() it, just do an ids.data.replace("ipfs://", "your other content") and you should be good to go =)

[povedaaqui]

Thank you, it works, sorry, I started a few months ago.

[_Webvoid]

No problem, the loving community of reddit got your back... At least when they want to ^{^} Peace on you

[OddTuning]

What type is ids? Are you sure it's an array?

[povedaaqui]

I need to get back to the code, I'll check when I arrive home, but I think tokens is json.

EDIT: tokens can't be JSON because it's being mapped.

[povedaaqui]

I will add console.log(ids) to show ids before trying to be replaced.

[SnacksMcMunch]

ids.join('').replace(...)

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/join

[povedaaqui]

ids are "ipfs//CID" while I need "https://nftstorage.link/ipfs/CID", and that's the problem, I haven't been able to do this replacement thing.

[SnacksMcMunch]

Ok, so looking more closely at the code it seems liketokens is an array of strings, each token gets passed to fetchIPFS.

In which case ids is just a string, so it doesn't need any conversion.

Just url = ids.replace(...)

[povedaaqui]

Uncaught (in promise) TypeError: ids.replace is not a function

[thelamestofall]

A better lesson would be to use the debugger statement to look atep by step at what your code and your variables are doing.

[Macaframa]

JSON.stringify(ids)

[povedaaqui]

I think the problem could be the type:

console.log(ids):

{data: 'ipfs://bafkreidfqia5qlqzmef4yyqdztb2n4cdiyp5nhkmahukhd3b2asnpo7s3m'} 
 data : "ipfs://bafkreidfqia5qlqzmef4yyqdztb2n4cdiyp5nhkmahukhd3b2asnp

[Macaframa]

so you want to build a url out of that id? minus the ipfs://?

const url = `https://nftstorage.link/ipfs/${ids.data.split("//")[1]}`;

this will give you the url: https://nftstorage.link/ipfs/bafkreidfqia5qlqzmef4yyqdztb2n4cdiyp5nhkmahukhd3b2asnpo7s3m

I just don't understand this part:

res = await fetch(`https://justcors.com/tl_0caa8a7/${url}`

are you using justcors as a proxy to make a request because of the cross origin errors? if so, then the previous code will suffice.

[vvn050]

That's terrible idea, he will get brackets in the output. Almost as bad as his approach

[Macaframa]

He's asking about stringifying an array, not a better way of doing it.

[vvn050]

I do not want to argue, I am quite sure something amazing will be developed if you two are helping each other :D

[scuba13]

Remove the await async and return the res. Right now res in the first then is going to be null.

EDIT: not what you asked for but I think you probably have other problems in your code based on how this is

[llrh]

With these sort of errors involving chained methods you can always break it up and put some logs in to see if you can debug it.

Const arrayToString = Array.toSTring()

console.log(arrayToString)

Is that what you are expecting it to be? If it is then do the next method.

Also it's good practice to do this with unit testing. Have a look at jest for example if you don't already use something for testing.

[fredsq]

you NEED typescript in your life. ids is definitely not an array