Gestion des doublons

[ikvat]

Hello toutes tous.
Je développe un composant en utilisant comme base le composant weblinks. Ce composant est une base de données qui répertorie des CD DVD liés à l'opéra. Or, il existe des dizaines d'enregistrements ayant le même titre. Le module weblinks interdit l'entrée de doublons.

Le morceau de code suivant semble gérer la recherche de doublons. Si je le commente, quelle conséquences cela peut-il avoir ?

if ($xid && $xid != (int) $this->id)
        {
            $this->setError(JText::_('COM_ABCDVDS_ERR_TABLES_NAME'));

            return false;
        }
        }

[RobertG]

Dans Joomla!, c'est en principe la gestion des alias qui interdit les doublons, il suffit donc qu'à la saisie, si on ne l'a pas fait dans le titre, on introduise un terme complémentaire pour éviter un tel problème.
Les IDs sont normalement automatiquement gérées par un champ en autoincrément, pas par le code PHP, donc si ta table est bien structurée, ce n'est a priori pas cette partie de code qui doit intervenir.

[ikvat]

Dans Joomla!, c'est en principe la gestion des alias qui interdit les doublons, il suffit donc qu'à la saisie, si on ne l'a pas fait dans le titre, on introduise un terme complémentaire pour éviter un tel problème.
Les IDs sont normalement automatiquement gérées par un champ en autoincrément, pas par le code PHP, donc si ta table est bien structurée, ce n'est a priori pas cette partie de code qui doit intervenir.

Et bien si, le module weblinks qui me sert de base gère effectivement les doublons au niveau des alias, mais également au niveau des titres. En supprimant (commentant) la ligne, les doublons pour le titre ne sont plus un problème. Je voulais savoir si cela posait un problème de sécurité ou autre (même si je n'y crois pas trop).

Au passage, mon module est ici

[roland_d_alsace]

Je suppose que tu as trouvé ces lignes dans la classe dérivé de Jtable et certainement dans la méthode check().

Si c'est le cas, ce bout de code vérifie juste que si ce n'est pas une création, xid est bien identique à l'id de l'enregistrement courant.

Reste à voir comment xid est trouvé.
Il doit bien y avoir une lecture de la table sur un autre critère au préalable.

C'est donc bien un contrôle pour éviter la création d'un doublon, mais c'est une sécurité, car ce cas ne devrait pas se produire en arrivant dans cette méthode qui précède juste la méthode store().

Si tu es développeur tu dois savoir qu'une gestion de doublon ne peut en aucun cas se résumer à 2 lignes de code...

[ikvat]

Je suppose que tu as trouvé ces lignes dans la classe dérivé de Jtable et certainement dans la méthode check().

Si c'est le cas, ce bout de code vérifie juste que si ce n'est pas une création, xid est bien identique à l'id de l'enregistrement courant.

Reste à voir comment xid est trouvé.
Il doit bien y avoir une lecture de la table sur un autre critère au préalable.

C'est donc bien un contrôle pour éviter la création d'un doublon, mais c'est une sécurité, car ce cas ne devrait pas se produire en arrivant dans cette méthode qui précède juste la méthode store().

Si tu es développeur tu dois savoir qu'une gestion de doublon ne peut en aucun cas se résumer à 2 lignes de code...

Je ne suis hélas pas à proprement parlé un "développeur". Disons que je bricole, voir bidouille, au petit bonheur la chance.
Effectivement, c'est bien check pour Jtable.
Ce qui m'a laissé pensé que c'était la méthode de vérification des doublons pour le titre, c'est le commentaire préalable à cette partie du code.
En fait, le code en entier est

        // Check for existing name
        $db = $this->getDbo();

        $query = $db->getQuery(true)
            ->select($db->quoteName('id'))
            ->from($db->quoteName('#__MatableDeBricolo'))
            ->where($db->quoteName('oeuvre') . ' = ' . $db->quote($this->oeuvre))
            ->where($db->quoteName('catid') . ' = ' . (int) $this->catid);
        $db->setQuery($query);

        $xid = (int) $db->loadResult();

        if ($xid && $xid != (int) $this->id)
        {
            $this->setError(JText::_('COM_ABCDVDS_ERR_TABLES_NAME'));

            return false;
        } 

[roland_d_alsace]

Et bien si, le module weblinks qui me sert de base gère effectivement les doublons au niveau des alias, mais également au niveau des titres. En supprimant (commentant) la ligne, les doublons pour le titre ne sont plus un problème. Je voulais savoir si cela posait un problème de sécurité ou autre (même si je n'y crois pas trop).

Au passage, mon module est ici

Généralement c'est dans la même méthode check() que l'alias est calculé.

[roland_d_alsace]

Je ne suis hélas pas à proprement parlé un "développeur". Disons que je bricole, voir bidouille, au petit bonheur la chance.
Effectivement, c'est bien check pour Jtable.
Ce qui m'a laissé pensé que c'était la méthode de vérification des doublons pour le titre, c'est le commentaire préalable à cette partie du code.
En fait, le code en entier est

 // Check for existing name
$db = $this->getDbo();

$query = $db->getQuery(true)
->select($db->quoteName('id'))
->from($db->quoteName('#__MatableDeBricolo'))
->where($db->quoteName('oeuvre') . ' = ' . $db->quote($this->oeuvre))
->where($db->quoteName('catid') . ' = ' . (int) $this->catid);
$db->setQuery($query);

$xid = (int) $db->loadResult();

if ($xid && $xid != (int) $this->id)
{
$this->setError(JText::_('COM_ABCDVDS_ERR_TABLES_NAME'));

return false;
} 

Là effectivement la méthode vérifie s'il n'existe pas déjà un enregistrement ayant la même valeur du champ "œuvre" dans la même catégorie.

Mais en général on attend pas le check pour rejeter, car c'est un peu "tard" c'est juste une sécurité.
Il y a des contrôles de suite à la saisie (client side validation et server side validation).
Voir :

https://docs.joomla.org/Client-side_form_validation

et

https://docs.joomla.org/Server-side_form_validation

[ikvat]

Là effectivement la méthode vérifie s'il n'existe pas déjà un enregistrement ayant la même valeur du champ "œuvre" dans la même catégorie.

Mais en général on attend pas le check pour rejeter, car c'est un peu "tard" c'est juste une sécurité.
Il y a des contrôles de suite à la saisie (client side validation et server side validation).
Voir :

https://docs.joomla.org/Client-side_form_validation

et
https://docs.joomla.org/Server-side_form_validation

Merci, c'est gentil.