Project

General

Profile

Anomalie #1935

Bug lors d'une signature placée

Added by Clément Leneveu 11 months ago. Updated 17 days ago.

Status:
Résolu
Priority:
Normal
Target version:
Start date:
Due date:
% Done:

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


Related issues

Related to Esup Signature - Anomalie #2083: Impossible de signerRésolu

Associated revisions

Revision 27385942 (diff)
Added by Clément Leneveu 11 months ago

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

History

#1 Updated by Clément Leneveu 11 months ago

  • Description updated (diff)

#2 Updated by Clément Leneveu 11 months ago

  • Target version set to 1.15.3

#3 Updated by Clément Leneveu 17 days ago

#4 Updated by Clément Leneveu 17 days ago

  • % Done changed from 0 to 100
  • Status changed from Nouveau to Résolu

Also available in: Atom PDF