Project

General

Profile

Anomalie #23028

Les badges poshprofil ne sont pas pris en compte dans le Dispatcher NG

Added by Renaud Dussol over 6 years ago. Updated about 6 years ago.

Status:
Fermé
Priority:
Haut
Assigned To:
Target version:
Start date:
02/09/2018
Due date:
% Done:

100%

Distribution:

Description

Lorsqu'on applique un badge à une ressource dans PoshProfil
Celui-ci n'est pas visible dans Dispatcher NG
Pour l'instant je te l'assigne car je ne sais pas trop ou cela se passe

Associated revisions

History

#1 Updated by Renaud Dussol over 6 years ago

  • Assigned To changed from Christophe LEON to Renaud Dussol

- Pour Nice, la possibilité d'avoir un message de type badge est intéressant (jsute avec un badge sur l'icone de l'appli et un texte qui s'affiche au survol, sans message autre de type popup ou postit)

Je propose donc :
- d'ajouter dans la gestion des messages de xdesktop un 3eme type de message de type "badge" (qui ne serait qu'applicatif du coup)
- si un message est de ce type, il ne s'affiche ni en popup ni en postit, simplement le badge se met sur l'icone de l'appli

#2 Updated by Renaud Dussol over 6 years ago

  • Status changed from Nouveau to En attente d'informations
  • Assigned To changed from Renaud Dussol to Christophe LEON

Dans un premier temps, je vais juste traiter la partie restitution (d'autant plus que la partie saisie est sur xdesktop, je n'ai pas encore de dépot miroir pour xdesktop)

Je vais juste ajouter la possibilité qu'un message soit de type "badge", auquel cas ni le popup ni le postit ne s'afficherait, mais l'icone aurait bien un badge avec le texte du message en survol.

En attendant de faire la partie saisie sur xdesktop, on modifierait le type de message directement dans la base. Je sais c'est pas terrible terrible mais bon...

Il faut quand même que je commite un changement sur edispatcher-messages

Dis-moi si tu es OK sur le principe. A priori cela ne devrait gêner en rien le comportement actuel, on aurait juste un cas supplémentaire possible.

#3 Updated by Renaud Dussol over 6 years ago

  • Priority changed from Normal to Haut

Je passe en prioritaire pour que ça arrive en haut de liste car PCI souhaiterait que ce soit dans la prochaine livraison
Je vais regarder le code en attendant ton avis et j'updaterai la demande au fur et à mesure

#4 Updated by Renaud Dussol over 6 years ago

Fait sur notre serveur de test :

Les fichiers à modifier sont :

- ng/public/elements/behaviors/scripts/messages.js
Ajouter dans la fonction displayMode un test sur le mode "badge"
lignes 285-295

        this.displayMode=function() {
                if (this.data.display_mode === "popup") {
                    return "popup";
                }
                else if (this.data.display_mode === "badge") {
                    return "badge";
                }
                else {
                    return "postit";
                }
        }

et une fonction prototype isBadge sur le modèle de isPopup :
l.358 :

Postit.prototype.isBadge = function() {
         var dp=this.displayMode();
             return dp==="badge";
 }

-fichier ng/public/elements/edispatcher-messages/edispatcher-messages.html

Dans la fonction addmessages, lors de l'assignation de la variable bAddToList (l. 433)

Initialiser tout d'abord à false afin d'éviter par défaut que les messages de type badge se retrouvent dans la liste des messages à traiter

  if (message.isBadge()) {
   bAddToList=false;
  }

Ensuite lors du traitement des messages de type app (l. 453)

Dans la condition if (!message.isPopup() && this.appComponent()) {

ajouter la condition && !message.isBadge()

if (!message.isPopup() && !message.isBadge() && this.appComponent()) {

Regarde si ces modifs ne risquent pas de produire d'effet de bord

J'ai créé 4 messages des autres possibilités différentes (popup et postit applicatif ou non) et ils se sont tous bien affichés, donc cela ne semble pas poser de problème

#5 Updated by Renaud Dussol over 6 years ago

  • % Done changed from 0 to 80

Si OK pour toi, je commite

#6 Updated by Christophe LEON about 6 years ago

Ok commit et je testerais sur notre dev

#7 Updated by Renaud Dussol about 6 years ago

  • Assigned To changed from Christophe LEON to Renaud Dussol

#8 Updated by Renaud Dussol about 6 years ago

  • Status changed from En attente d'informations to Résolu
  • % Done changed from 80 to 100

#9 Updated by Renaud Dussol about 6 years ago

Je rappelle que pour tester, il faut assigner manuellement la valeur "badge" au champ dispay_mode dans la table postit sur un message

#10 Updated by Christophe LEON about 6 years ago

  • Target version set to Envole 5.8

#11 Updated by Arnaud FORNEROT about 6 years ago

  • Status changed from Résolu to Fermé

Also available in: Atom PDF