Projet

Général

Profil

Anomalie #1935

Bug lors d'une signature placée

Ajouté par Clément Leneveu il y a plus de 2 ans. Mis à jour il y a plus d'un an.

Statut:
Résolu
Priorité:
Normal
Assigné à:
Version cible:
Début:
Echéance:
% réalisé:

100%


Description

avec la version 1.15.3-SNAPSHOT on a un nullpointerexception

Dans le fichier SignRequestService.java ligne 1243 remplacer

    commentService.create(id, commentText, commentPosX, commentPosY, commentPageNumber, spotStepNumber, postit.equals("on"), null, authUserEppn);

par

    commentService.create(id, commentText, commentPosX, commentPosY, commentPageNumber, spotStepNumber, "on".equals(postit), null, authUserEppn);

après on a toujours un message d'erreur Merci de placer la signature en pop-up : le problème vient d'un fonction checkSignsPositions() dans le fichier Workspace.js.

checkSignsPositions() {
        let testSign = Array.from(this.signPosition.signRequestParamses.values());
         if(testSign.filter(s => s.signImageNumber >= 0 && s.isSign).length > 0) {
            for (let i = 0; i < this.currentSignRequestParamses.length; i++) {
                if (this.currentSignRequestParamses[i].ready == null || !this.currentSignRequestParamses[i].ready) {
                    return false;
                }
            }
            return true;
        } else {
            return false;
        }
    }

la condition if (this.currentSignRequestParamses[i].ready == null || !this.currentSignRequestParamses[i].ready) est toujours satisfaite, en conséquence elle renvoie toujours false ce qui entraîne à son tour l'apparition du message.

Je pense (mais je n'ai pas testé) que c'est parce qu'il n'y a pas d'attributs ready dans dans la classe SignRequestParams, d'ailleurs il n'y a pas de champs ready dans la table sign_request_params non plus.

cf. https://github.com/EsupPortail/esup-signature/issues/163


Demandes liées

Lié à Esup Signature - Anomalie #2083: Impossible de signerRésolu

Révisions associées

Révision 27385942 (diff)
Ajouté par Clément Leneveu il y a plus de 2 ans

refs #1935 : big nag nethod, comment the test that always returning
false

Historique

#1 Mis à jour par Clément Leneveu il y a plus de 2 ans

  • Description mis à jour (diff)

#2 Mis à jour par Clément Leneveu il y a plus de 2 ans

  • Version cible mis à 1.15.3

#3 Mis à jour par Clément Leneveu il y a plus d'un an

#4 Mis à jour par Clément Leneveu il y a plus d'un an

  • % réalisé changé de 0 à 100
  • Statut changé de Nouveau à Résolu

Formats disponibles : Atom PDF