router.post('/uploadImage', (request, response, next) => {
let formidable = require('formidable');
// parse a file upload
var form = new formidable.IncomingForm();
form.uploadDir = `../imagenes/`;
form.keepExtensions = true;
form.maxFieldsSize = 10 * 1024 * 1024; //10 MB
form.multiples = true;
form.parse(request, (err, fields, files) => {
if (err) {
response.json({
result: "failed",
data: {},
messege: `Cannot upload images.Error is : ${err}`
});
}
var arrayOfFiles = [];
if(files[""] instanceof Array) {
arrayOfFiles = files[""];
} else {
arrayOfFiles.push(files[""]);
}
if (arrayOfFiles.length > 0) {
var fileNames = [];
arrayOfFiles.forEach((eachFile)=> {
// fileNames.push(eachFile.path)
fileNames.push(eachFile.path.split('/')[1]);
});
response.json({
result: "ok",
data: fileNames,
numberOfImages: fileNames.length,
messege: "Upload images successfully"
});
} else {
response.json({
result: "failed",
data: {},
numberOfImages: 0,
messege: "No images to upload !"
});
}
});
});
1 respuesta
El problema que tienes es que files
nunca es un arreglo, siempre es un objeto donde las llaves son el nombre del campo en donde mandaron el fichero y el valor es un objeto o un arreglo dependiendo si mandaron varios ficheros en el mismo campo desde el cliente.
Este código
var arrayOfFiles = [];
if(files[""] instanceof Array) {
arrayOfFiles = files[""];
} else {
arrayOfFiles.push(files[""]);
}
puede escribirse
var arrayOfFiles = Object.keys(files).reduce((curr, key) => {
const group = files[key]; // Para chequear si es fichero único o arreglo
if (Array.isArray(group)) {
curr = curr.concat(group); // Si es arreglo concatenalo al que ya tienes
// o también [...curr, ...group]
} else {
curr.push(group); // Sino agregalo al final
// o también [...curr, group]
}
return curr;
}, []);
O aún más simplificado
var arrayOfFiles = Object.keys(files)
.reduce((curr, key) => Array.isArray(files[key])
? [...curr, ...files[key]]
: [...curr, files[key]]
, []);
Eso te generará un arreglo con todos los ficheros de tu request. Tu solución sería
router.post('/uploadImage', (request, response, next) => {
let formidable = require('formidable');
// parse a file upload
var form = new formidable.IncomingForm();
form.uploadDir = `../imagenes/`;
form.keepExtensions = true;
form.maxFieldsSize = 10 * 1024 * 1024; //10 MB
form.multiples = true;
form.parse(request, (err, fields, files) => {
if (err) {
response.json({
result: "failed",
data: {},
messege: `Cannot upload images.Error is : ${err}`
});
}
var arrayOfFiles = Object.keys(files)
.reduce((curr, key) => Array.isArray(files[key])
? [...curr, ...files[key]]
: [...curr, files[key]]
, []);
if (arrayOfFiles.length > 0) {
var fileNames = [];
arrayOfFiles.forEach((eachFile) => {
// fileNames.push(eachFile.path)
fileNames.push(eachFile.path.split('/')[1]);
});
response.json({
result: "ok",
data: fileNames,
numberOfImages: fileNames.length,
messege: "Upload images successfully"
});
} else {
response.json({
result: "failed",
data: {},
numberOfImages: 0,
messege: "No images to upload !"
});
}
});
});
-
me devuelve el mismo json =´{ "result": "failed", "data": {}, "numberOfImages": 0, "messege": "No images to upload !" }´ el 10 dic. 2018 a las 19:20
-
puede ser que lo este haciendo mal con el programa postman ? el 10 dic. 2018 a las 19:20
-
@joseangel Yo lo acabo de probar y funciona. Revisa que lo copiaste bien y que estes usando el Postman correctamente el 10 dic. 2018 a las 19:21
-
a ver lo que es el servidor internamente recibe el codigo 200 de que esta bien, pero la respuesta dice q hay un error (el response) , miro mi directorio y no esta subida la foto. En cuanto al postman lo tengo en metodo POST , en la seccion Body y en el apartado form-data subiendo un File el 10 dic. 2018 a las 19:25
-
1@joseangel Yo tambien lo probé en Windows. El Postman a veces te resetea los campos de ficheros. Prueba a introducir todos los valores nuevamente. Ese código que no entiendes es exactamente lo mismo que usando
concat
ypush
. Lee developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/… el 10 dic. 2018 a las 19:39
arrayOfFiles
y mirafiles[""]
console.log(arrayOfFiles )
?console.log(arrayOfFiles[0] )
arrayOfFiles.push(files[""]);
creo que debe ser algo comoarrayOfFiles.push(files[0]);