Anomalie #4359: Incohérence d'argument lors de l'appel de Dir::preInsertMultipleDocuments() par Dir::insertMultipleDocuments() - Core - Dynacase

Actions

Copier le lien

Anomalie #4359

fermé

Incohérence d'argument lors de l'appel de Dir::preInsertMultipleDocuments() par Dir::insertMultipleDocuments()

Ajouté par Jérôme Augé il y a plus de 12 ans. Mis à jour il y a plus de 12 ans.

Statut:

Intégré

Priorité:

Normal

Assigné à:

Éric Brison

Version cible:

3.2.12

Début:

10/12/2013

Echéance:

% réalisé:

100%

Temps estimé:

Version source:

Solution proposée:

Les Hook pre/postInsertMultipleDocuments seront livrés vide comme tous les autres hooks

Les hook pre/postInsertDocument sont appelé dans la méthode insertMultipleDocument avec l'argument multiple=true.

Principaux fichiers impactés:

Complexité:

Contrôle:

Thème:

Régression:

Description

La méthode Dir::insertMultipleDocuments appelle Dir::preInsertMultipleDocuments en lui passant une liste de document bruts ($tdocs) alors que cette méthode est prévue pour recevoir une liste d'identifiants (comme postInsertMultipleDocuments) :

    function preInsertMultipleDocuments($tdocid)
    {
        $err = '';
        foreach ($tdocid as $docid) {
        //                  ^-- $tdocid doit être une liste d'identifiants
            $err.= $this->preInsertDocument($docid, true);
        }
        return $err;
    }

    ...

    function insertMultipleDocuments($tdocs, $mode = "latest", $noprepost = false, &$tinserted = array() , &$twarning = array()) {
        $err = $this->preInsertMultipleDocuments($tdocs);
        //                                       ^-- On passe une liste de documents bruts
        ...
    }