Discussion :

[kryogen]

Bonjour,

J'ai une erreur, "getElementById is not a function" que je ne comprend pas.

voici ma fonction (ajax) qui supprime un document.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
 
function supprimer_archive(num,document)
{
    if (confirm('supprimer cette archive ?'))
    {
    }
    else
    {
    return false;
    }
 
    if (confirm('êtes-vous bien sûr de vouloir supprimer cette archive ?'))
    {
    }
    else
    {
    return false;
    }
 
var num = num;
var document = document;
var dossier = 'archives';
 
    xhr.open('POST','./php/script_archive_supprimer.php',true);
    xhr.onreadystatechange = function()
        {
            if ((xhr.readyState == 1)||(xhr.readyState == 2)||(xhr.readyState == 3))
            {
            }
            else if (xhr.readyState == 4)
            {
                // supprimer le tr
                var node = document.getElementById('tr'+num);
                var parent = node.parentNode;
                parent.removeChild(node);
                // afficher message de confirmation
                alert('l\'archive « '+xhr.responseText+' » a été supprimée de la base.');
            }
        }
    xhr.setRequestHeader('Content-type','application/x-www-form-urlencoded');
    var data = 'document='+escape(document)+'&dossier='+escape(dossier);
    xhr.send(data);
}

le document est bien supprimé. pas de problème.
mais ce petit bout de code :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
 
 // supprimer le tr
var node = document.getElementById('tr'+num);
var parent = node.parentNode;
parent.removeChild(node);

est sensé me supprimer la ligne de mon tableau.
bien sûr elle n'est pas supprimée et message d'erreur dans la console :
"getElementById is not a function" !!

merci pour votre aide...

[vermine]

Bonjour,

Tu as plusieurs formulaires dans ta page ?
Car dans document.getElementById(...);, document fait référence au formulaire. Or toi, tu lui passes une valeur pour la variable document. Que lui passes-tu ?

[kryogen]

non aucun formulaire... je fais référence à un <tr> d'une table...

[Auteur]

bonjour,

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

var num = num;

c'est curieux ceci... Ta variable num est un paramètre, inutile de la déclarer par la suite. D'ailleurs cette ligne ne déclenche pas une erreur ?

Pour supprimer une ligne d'un tableau utilise deleteRow() c'est plus efficace

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

document.getElementById('idTableau').deleteRow(indexLigne);

en reprenant ton code :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
 
var ligne = document.getElementById('tr'+num);
document.getElementById('idDuTableau').deleteRow(ligne.rowIndex);

kryogen : le tr ciblé possède bien un id ?

sinon pour Vermine : getElementById() est une méthode de document exclusivement, donc la syntaxe de kryogen est juste. Par ailleurs un id est unique dans la page donc que l'objet soit dans un formulaire ou pas, le résultat sera le même

[Bovino]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

var num = num;

c'est curieux ceci...

Oui, mais le plus... "exotique", ce serait surtout :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

var document = document;

Effectivement, quand tu fais ensuite

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

document.getElementById

tu parles de quoi, de la variable ou de l'objet document ???

Il ne faut JAMAIS donner ce genre de nom à une variable !!!

[Auteur]

Oui, mais le plus... "exotique", ce serait surtout :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

var document = document;

Effectivement, quand tu fais ensuite

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

document.getElementById

tu parles de quoi, de la variable ou de l'objet document ???

Il ne faut JAMAIS donner ce genre de nom à une variable !!!

ouh là, oui, je ne l'ai pas vu celle-là

Je comprends mieux la remarque de Vermine maintenant

[kryogen]

ok ok
je ne suis qu'un modeste novice en javascript... pitié
bon merci en tout cas je vais tout corriger avec toutes vos remarques...
pour les var num = num, je pensais (à tord) qu'il fallait les redéclarer...
merci encore mes bons maîtres
bon week end !

[Auteur]

Déjà pour bien corriger le code dis nous à quoi fait référence ta variable document
Quels sont les paramètres que tu transmets à ta fonction supprimer_archive() lorsque tu l'appelles ?

[kryogen]

num correspond à la ligne (du tableau listant les documents d'un dossier sur le serveur) à supprimer, exemple : <tr id="tr1"> (id généré en PHP).
document correspond au nom du document que je vais supprimer physiquement sur le serveur de manière asynchrone (exemple : 000001-20080210100245-exemple.pdf).
voilà...

[kryogen]

C'est bon c'est corrigé !
Il est vrai que la remarque de la vermine n'était pas sans intérêt.
En effet y avait une pauvre confusion entre la variable document et document.getElementById.
Faut le reconnaître c'est une horrible erreur de débutant (ce que je suis, faut pas se voiler la face).
donc voici le code qui marche trop cool génial :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
 
///////////////////////////////////////////
// supprimer une archive //////////////////
///////////////////////////////////////////
 
function supprimer_archive(num,intitule)
{
    if (confirm('supprimer cette archive ?'))
    {
    }
    else
    {
    return false;
    }
 
    if (confirm('êtes-vous bien sûr de vouloir supprimer cette archive ?'))
    {
    }
    else
    {
    return false;
    }
 
var dossier = 'archives';
 
    xhr.open('POST','./php/script_archive_supprimer.php',true);
    xhr.onreadystatechange = function()
        {
            if ((xhr.readyState == 1)||(xhr.readyState == 2)||(xhr.readyState == 3))
            {
            }
            else if (xhr.readyState == 4)
            {
                // supprimer le tr
                var node = document.getElementById('tr'+num);
                var parent = node.parentNode;
                parent.removeChild(node);
                // afficher message de confirmation
                alert('l\'archive « '+xhr.responseText+' » a été supprimée de la base.');
            }
        }
    xhr.setRequestHeader('Content-type','application/x-www-form-urlencoded');
    var data = 'intitule='+escape(intitule)+'&dossier='+escape(dossier);
    xhr.send(data);
}

merci à tous
bonne continuation !
youhou !

ps : notez bien la disparition de var num = num