Projet

Général

Profil

Anomalie #4359

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 5 ans. Mis à jour il y a plus de 5 ans.

Statut:
Intégré
Priorité:
Normal
Assigné à:
Version cible:
Début:
10/12/2013
Echéance:
% réalisé:

100%

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:
Non
Jalons: 3.2 R6

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
        ...
    }

Historique

#1 Mis à jour par Marc Claverie il y a plus de 5 ans

  • Statut changé de Nouveau à À analyser
  • Assigné à mis à Éric Brison
  • Version cible mis à 3.2.12

#2 Mis à jour par Éric Brison il y a plus de 5 ans

  • Solution proposée mis à jour (diff)

#3 Mis à jour par Éric Brison il y a plus de 5 ans

  • Statut changé de À analyser à Intégré
  • % réalisé changé de 0 à 100

Appliqué par commit dynacase-core|commit:48e2b1bb525cc7589a6d14bd45828a2e6b3b99cb.

Formats disponibles : Atom PDF