save reusable ajax callback has mix ofexpected result json and string ...

[hayden-t]

VvvebJs/libs/builder/builder.js

Line 2157 in 078fe4f

if (true || data.success || text == "success") {

it seems the code here is written for 2 different result type, similar in other ajax calls such as delete, there is code to expect/handle json result, the default save.php does not though either throwing a error code on failure which is caught or returning a text string if success.

Not sure which is best but maybe should settle on one and standardise ?

[hayden-t]

as a tip, if anyone does want to handle json response, (and i do like the idea of the code supporting both if possible) change the first bit to

	fetch(saveReusableUrl, {method: "POST",  body: new URLSearchParams(data)})
	.then((response) => {
		if (!response.ok) { throw new Error(response) }
		return response.json() //<--here
	})

and remove the 'true' condition in the line referenced above