funcion eliminar imagen

[charlyta]

Hola. El siguiente script cumple su función, que es eliminar la imagen, pero me salta un paso que es la confirmación de si quiero eliminarla y no sé porqué! alguna idea?

Código Javascript:

<script>
function deleteFile(fname,directory,deletecount) {
$("a.ico-edit").live("click",function(event){
   event.stopPropagation();
   if(confirm("Do you want to delete?")) {
    this.click;
       alert("Ok");
        $.ajax({ url: "delete_img.php",
        data: {"file":fname,"directory":directory},
        type: 'post',
        success: function(output) {
          alert(output);
          
$('#' + deletescount).remove();
 
 
 
                 }
    });
   }
   else
   {
       alert("Cancel");
   }       
   event.preventDefault();
   
});
                             });
 
</script>
 

[maycolalvarez]

no entiendo para que es el this.click; además organice el código para poder leerlo adecuadamente

[Alexis_Mejias]

Yo usaria e confirm despues del function deleteFile(fname,directory,deletecount) {
y no tendrias problemas con ese salto que dices que te sucede...

[maycolalvarez]

Iniciado por Alexis_Mejias

Yo usaria e confirm despues del function deleteFile(fname,directory,deletecount) {
y no tendrias problemas con ese salto que dices que te sucede...

no puedes, porque la lógica es preguntar primero y luego enviar la petición ajax que elimine ¿no te parece?

[Alexis_Mejias]

Iniciado por maycolalvarez

no puedes, porque la lógica es preguntar primero y luego enviar la petición ajax que elimine ¿no te parece?

Yo me referia a lo siguiente:

Código PHP:

  function deleteFile(campo1, campo2, campo3)
{
    if (confirm('Are you sure?')){
        //EJEUCION DEL AJAX
    }
    else{
        alert("the file wasn't remove")
    }
} 
   

Se entiende el punto?

[charlyta]

Solucionado!! Pero no creo que pasando los parámetros así sea muy útil, debe haber otra manera

Código Javascript:

<script>
  $(document).ready(function() {
$("a.ico-edit").live("click",function(event){
                                      var fname=$(this).attr("id");var directory=$(this).attr("rel");var deletecount=$(this).attr("value");
   event.stopPropagation();
   if(confirm("Do you want to delete?")) {
    this.click;
       alert("Ok");
        $.ajax({ url: "delete_img.php",
        data: {"file":fname,"directory":directory},
        type: 'post',
        success: function(output) {
          alert(output);
          
$('#' + deletecount).remove();
 
 
 
                 }
    });
   }
   else
   {
       alert("Cancel");
   }       
   event.preventDefault();
   
});
                             });
 
</script>
 

[maycolalvarez]

@Alexis_Mejias OK, te malinterpreté

@charlyta, ¡claro! no me había fijado de que usabas live, y sólo le pasas un parámetro y no 3, de la nueva forma en que lo haces lo garantizas, porque recuerda que el .live se aplica a los futuros objetos, este es uno de los puntos por el cual no recomiendan su uso, pero bueno de esa discusión puedes conseguir mucha tela que cortar, saludos