1

Tengo el siguiente código en un servicio y me lanza el siguiente error por consola:

Error: Uncaught (in promise): TypeError: Cannot read property 'name' of undefined TypeError: Cannot read property 'name' of undefined

Al parecer no esta reconociendo ninguno de los parametros

Si alguien tiene alguna idea! Gracias de Antemano!

import { Injectable } from '@angular/core';
import { GLOBAL } from './global';

@Injectable()
export class UploadService{
	public url: string;

	constructor(){
	this.url = GLOBAL.url;
	}
//al parecer no reconoce estos parametros
	makeFileRequest(url: string, params: Array<string>, files: Array<File>, token: string, name: string){

	return new Promise(function(resolve, reject){
	var formData: any = new FormData();
	var xhr = new XMLHttpRequest();
   // El for dice que no reconoce el namespace Files
	for(var i = 0; < files.length; i++){

		formData.append(name, files[i], files[i].name);

	}

	xhr.onreadystatechange = function(){

	if(xhr.readyState == 4){

	if(xhr.status == 200){

		resolve(JSON.parse(xhr.response));

	}else{
		reject(xhr.response);

	}
	}
	}

	xhr.open('POST', url, true);
	xhr.setRequestHeader('Authotization', token);
	xhr.send(formData);

	});

	}
}

Desde aquí llamo al método del servicio

this._uploadService.makeFileRequest(this.url+'upload-image-user/'+this.user._id, [], this.filesToUpload, this.token, 'image')
                           .then((result: any) => {
                              console.log(result);
                              this.user.image = result.image;
                              localStorage.setItem('identity', JSON.stringify(this.user));
                           });
      }

14
  • comprueba donde llamas al método makeFileRequest() del servicio si le estas pasando un valor para el parámetro name
    – Orici
    el 27 jun. 2018 a las 21:50
  • El error te lo da cuando llamas a makeFileRequest() sin ningún fichero? Estaría bien que pudieras añadir cómo lo estás llamando.
    – Sema
    el 27 jun. 2018 a las 21:54
  • He agregado mas arriba como lo estoy llamando, si le paso los valores de donde lo llamo el 27 jun. 2018 a las 21:58
  • ¿Qué muestra un console.log(files); antes del for?
    – A. Cedano
    el 27 jun. 2018 a las 22:04
  • Soy nuevo en StackOverflow español y no puedo hacer comentarios, editaré esta respuesta, cuando encontremos la solución files[i] no tiene la propiedad nombre, podrías mostrar cómo estás declarando esta variable donde consumes el servicio? el 27 jun. 2018 a las 22:05

3 respuestas 3

0

Error: Uncaught (in promise): TypeError: Cannot read property 'name' of undefined TypeError: Cannot read property 'name' of undefined

ahí te dice clarito el error, "no se puede leer propiedad 'name' de indefinido" y en el único lugar donde usas la propiedad name es en esta línea:

formData.append(name, files[i], files[i].name);

lo que significa que files[i] es igual a undefined

busca la forma de debugear tu código y lo comprobarás

2
  • estoy intentando el 27 jun. 2018 a las 22:58
  • ahí te dieron algunos tips como console.log(files[i]) tambien puedes usar los debugger que tienen los los navegadores usualmente apretando ctrl + F12
    – LPZadkiel
    el 27 jun. 2018 a las 23:09
0

Prueba con un foreach así:

const formData: FormData = new FormData();
files.forEach(file => {
  formData.append('fileItem[]', file, file.name);
});

En lugar de fileItem puede ir el nombre que tu quieras o la variable.

4
  • lo he intentado asi y sigue dando error me envia que TypeError: files.forEach is not a function el 27 jun. 2018 a las 22:22
  • Pero entonces el problema viene de lo que le envias a la función, como le estas pasando el parametro files? Tu variable files tiene un valor undefined el 27 jun. 2018 a las 22:35
  • Creo que el problema es más bien en como estas llenando tu array de files el 27 jun. 2018 a las 22:47
  • esta en el codigo el 27 jun. 2018 a las 22:59
0

Prueba a modificar el código de dentro de la promesa así:

var fileInput = document.getElementById("myfileinput");
var files = fileInput.files; // Esta línea es la que no tienes
var file;

// loop through files
for (var i = 0; i < files.length; i++) {
  // get item
  file = files.item(i);
  // or
  file = files[i];

  console.log('File to upload: ', file.name);

}

Es decir, creo que lo que te hace falta es añadir a los ficheros:

var files = fileInput.files;

2
  • Lo he intentado y nada el 27 jun. 2018 a las 22:58
  • ¿Podrías subir ese fragmento del código a un stackblitz para reproducirlo?
    – Sema
    el 27 jun. 2018 a las 23:00

Tu Respuesta

By clicking “Publica tu respuesta”, you agree to our terms of service and acknowledge you have read our privacy policy.

¿No es la respuesta que buscas? Examina otras preguntas con la etiqueta o formula tu propia pregunta.