Projet

Général

Profil

Anomalie #23028

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

Ajouté par Renaud Dussol il y a environ 6 ans. Mis à jour il y a environ 6 ans.

Statut:
Fermé
Priorité:
Haut
Assigné à:
Version cible:
Début:
09/02/2018
Echéance:
% réalisé:

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

Révisions associées

Historique

#1 Mis à jour par Renaud Dussol il y a environ 6 ans

  • Assigné à changé de Christophe LEON à 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 Mis à jour par Renaud Dussol il y a environ 6 ans

  • Statut changé de Nouveau à En attente d'informations
  • Assigné à changé de Renaud Dussol à 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 Mis à jour par Renaud Dussol il y a environ 6 ans

  • Priorité changé de Normal à 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 Mis à jour par Renaud Dussol il y a environ 6 ans

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 Mis à jour par Renaud Dussol il y a environ 6 ans

  • % réalisé changé de 0 à 80

Si OK pour toi, je commite

#6 Mis à jour par Christophe LEON il y a environ 6 ans

Ok commit et je testerais sur notre dev

#7 Mis à jour par Renaud Dussol il y a environ 6 ans

  • Assigné à changé de Christophe LEON à Renaud Dussol

#8 Mis à jour par Renaud Dussol il y a environ 6 ans

  • Statut changé de En attente d'informations à Résolu
  • % réalisé changé de 80 à 100

#9 Mis à jour par Renaud Dussol il y a environ 6 ans

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

#10 Mis à jour par Christophe LEON il y a environ 6 ans

  • Version cible mis à Envole 5.8

#11 Mis à jour par Arnaud FORNEROT il y a environ 6 ans

  • Statut changé de Résolu à Fermé

Formats disponibles : Atom PDF