Project

General

Profile

Bug #3120

Mettre moins en évidence le numéro de dossier informatique.

Added by Mikaël Ates over 10 years ago. Updated about 9 years ago.

Status:
Fermé
Priority:
Normal
Assignee:
Target version:
Start date:
24 June 2013
Due date:
% Done:

100%

Estimated time:
Patch proposed:
Yes
Planning:

Description

Le supprimer ou l'afficher en plus petit.


Files

Associated revisions

Revision 1f4ae3b3 (diff)
Added by Serghei Mihai over 9 years ago

patient record id grayed on all pages in order to avoid confusion

Closes #3120

Revision 3c1f0b4a (diff)
Added by Serghei Mihai over 9 years ago

facturation: gray act ids on invoicing page

Closes #3120

History

#1

Updated by Mikaël Ates over 9 years ago

  • Target version set to 2.0
#2

Updated by Mikaël Ates over 9 years ago

  • Target version changed from 2.0 to 1.2 Autres
#4

Updated by Frédéric Péters over 9 years ago

  • Status changed from Nouveau to En cours
  • % Done changed from 0 to 50

% mis pour indiquer qu'il y a un patch.

#5

Updated by Serghei Mihai over 9 years ago

  • % Done changed from 50 to 100

Appliqué

#7

Updated by Serghei Mihai over 9 years ago

C'est dans mon dépot local pour l'instant.
Je vais pousser aujourd'hui.

#8

Updated by Serghei Mihai over 9 years ago

  • Status changed from En cours to Résolu (à déployer)
#9

Updated by Mikaël Ates over 9 years ago

  • Status changed from Résolu (à déployer) to En cours
  • % Done changed from 100 to 0

Le patch supprime l'affichage du numéro informatique sur le premier onglet du dossier patient ce qui répond partiellement à la demande (incomplète).

Le numéro informatique apparaît à plusieurs endroits dans le logiciel mais ce n'est pas au niveau du premier onglet du dossier patient que cela peut apporter une confusion à l'utilisateur notamment puisqu'il est indiqué "numéro informatique" devant le numéro.

L'idée ce serait plutôt de ne pas l'afficher lorsque cela n'est pas nécessaire et lorsqu'on l'affiche le mettre en grisé. Ainsi il n'y a pas de confusion possible sur le numéro affiché.

Il faudrait donc le restaurer sur le premier onglet du dossier patient et le griser.

L'affichage du numéro informatique a été supprimé a plusieurs endroits dans l'agenda. Il reste à le supprimer sur la page de validation des actes.
Dans les statistiques il n'apparaît pas.

Dans la partie dossier, dans le dossier cf plus haut. Sur les tableaux il est en libellé de la colonne. On laisse ainsi ou on grise le contenu de la colonne.

Dans la saisie des actes et la facturation, il est indiqué entre parenthèse. On grise les parenthèses et le numéro.

#10

Updated by Serghei Mihai over 9 years ago

Modifications apportées.

#11

Updated by Jérôme Schneider over 9 years ago

OK pour moi.

Juste dans calebasse/agenda/templates/agenda/act-validation.html il faudrait remplacer :

{% if acte.patient.paper_id %} {{ acte.patient.paper_id }} {% endif %}

par :

{{ acte.patient.paper_id|default_if_none:"" }}
#12

Updated by Serghei Mihai over 9 years ago

Normalement il n'est pas censé de passer le if is le paper_id est None.

En revanche j'ai rajouté le default_if_none dans le listing des actes.

#13

Updated by Serghei Mihai over 9 years ago

  • Status changed from En cours to Résolu (à déployer)
  • % Done changed from 50 to 100

Appliqué par commit calebasse|commit:1f4ae3b347f15c0133b00b968bfc1ad9aab281d5.

#14

Updated by Mikaël Ates over 9 years ago

  • Status changed from Résolu (à déployer) to En cours

"Dans la saisie des actes et la facturation, il est indiqué entre parenthèse. On grise les parenthèses et le numéro." C'est bon pour la saisie des actes mais pas pour la facturation.

#16

Updated by Mikaël Ates over 9 years ago

Le template est sûrement à factoriser mais cette modif est à appliquer 6 fois dans le fichier:

diff --git a/calebasse/facturation/templates/facturation/detail.html b/calebasse/facturation/templates/facturation/detail.html
index 058c051..4b6eb48 100644
--- a/calebasse/facturation/templates/facturation/detail.html
+++ b/calebasse/facturation/templates/facturation/detail.html
@@ -311,8 +311,8 @@
               {% endif %}
             {% endif %}

-                <h3><a href="../../dossiers/{{ patient.pk }}/view" target="_blank">
-                {% if patient.paper_id %}{{ patient.paper_id }} {% endif %}({{ patient.pk }}) <span class="lastname">{{ patient.last_name }}</span> {{ patient.first_name }}</a></h3>
+                <h3><span class="lightgray">({{ patient.pk }})</span>
+                  <a href="../../dossiers/{{ patient.pk }}/view" target="_blank">{% if patient.paper_id %}{{ patient.paper_id }} {% endif %} <span class="lastname">{{ patient.last_name }}</span> {{ patient.first_name }}</a></h3>
                 <div class="info">
                 </div>
                 {% if patient.pause %}
@@ -385,8 +385,8 @@
         <div id="dossiers-concernes">
         {% for patient, detail in patients_stats %}
             <div class="frame dossier" id="to-hide">
-                <h3>{% if patient.pk %}<a href="../../dossiers/{{ patient.pk }}/view" target="_blank">{% endif %}
-                {% if patient.paper_id %}{{ patient.paper_id }} {% endif %}{% if patient.pk %}({{ patient.pk }}) {% endif %}<span class="lastname">{{ patient.last_name }}</span> {{ patient.first_name }}{% if patient.pk %}</a>{% endif %}<
+                <h3><span class="lightgray">({{ patient.pk }})</span>
+                  <a href="../../dossiers/{{ patient.pk }}/view" target="_blank">{% if patient.paper_id %}{{ patient.paper_id }} {% endif %} <span class="lastname">{{ patient.last_name }}</span> {{ patient.first_name }}</a></h3>
                 <div class="info">
                 </div>
                 <ul>
@@ -435,8 +435,8 @@
             {% else %}
             <div class="frame dossier">
             {% endif %}
-                <h3><a href="../../dossiers/{{ patient.pk }}/view" target="_blank">
-                {% if patient.paper_id %}{{ patient.paper_id }} {% endif %}({{ patient.pk }}) <span class="lastname">{{ patient.last_name }}</span> {{ patient.first_name }}</a></h3>
+                <h3><span class="lightgray">({{ patient.pk }})</span>
+                  <a href="../../dossiers/{{ patient.pk }}/view" target="_blank">{% if patient.paper_id %}{{ patient.paper_id }} {% endif %} <span class="lastname">{{ patient.last_name }}</span> {{ patient.first_name }}</a></h3>
                 <div class="info">
                 </div>
                 {% if patient.pause %}
@@ -474,8 +474,8 @@
         <div id="dossiers-concernes">
         {% for patient, detail in patients_stats %}
             <div class="frame dossier" id="to-hide">
-                <h3><a href="../../dossiers/{{ patient.pk }}/view" target="_blank">
-                {% if patient.paper_id %}{{ patient.paper_id }} {% endif %}({{ patient.pk }}) <span class="lastname">{{ patient.last_name }}</span> {{ patient.first_name }}</a></h3>
+                <h3><span class="lightgray">({{ patient.pk }})</span>
+                  <a href="../../dossiers/{{ patient.pk }}/view" target="_blank">{% if patient.paper_id %}{{ patient.paper_id }} {% endif %} <span class="lastname">{{ patient.last_name }}</span> {{ patient.first_name }}</a></h3>
                 <div class="info">
                 </div>
                 <ul>
@@ -512,8 +512,8 @@
             {% else %}
             <div class="frame dossier">
             {% endif %}
-                <h3><a href="../../dossiers/{{ patient.pk }}/view" target="_blank">
-                {% if patient.paper_id %}{{ patient.paper_id }} {% endif %}({{ patient.pk }}) <span class="lastname">{{ patient.last_name }}</span> {{ patient.first_name }}</a></h3>
+                 <h3><span class="lightgray">({{ patient.pk }})</span>
+                  <a href="../../dossiers/{{ patient.pk }}/view" target="_blank">{% if patient.paper_id %}{{ patient.paper_id }} {% endif %} <span class="lastname">{{ patient.last_name }}</span> {{ patient.first_name }}</a></h3>
                 <div class="info">
                 </div>
                 {% if patient.pause %}
@@ -561,8 +561,8 @@
         <div id="dossiers-concernes">
         {% for patient, detail in patients_stats %}
             <div class="frame dossier" id="to-hide">
-                <h3><a href="../../dossiers/{{ patient.pk }}/view" target="_blank">
-                {% if patient.paper_id %}{{ patient.paper_id }} {% endif %}({{ patient.pk }}) <span class="lastname">{{ patient.last_name }}</span> {{ patient.first_name }}</a></h3>
+                <h3><span class="lightgray">({{ patient.pk }})</span>
+                  <a href="../../dossiers/{{ patient.pk }}/view" target="_blank">{% if patient.paper_id %}{{ patient.paper_id }} {% endif %} <span class="lastname">{{ patient.last_name }}</span> {{ patient.first_name }}</a></h3>
                 <div class="info">
                 </div>
                 <ul>
#17

Updated by Frédéric Péters over 9 years ago

Il est plutôt conseillé d'ajouter de la sémantique au niveau des classes, plutôt qu'une description graphique (i.e. class="patient-pk" plutôt que class="lightgray").

#18

Updated by Serghei Mihai over 9 years ago

Remarques prises en compte.
Je prefere les "_" dans les noms des classes css.

#19

Updated by Thomas Noël over 9 years ago

Serghei Mihai a écrit :

Je prefere les "_" dans les noms des classes css.

Heuula ... https://developer.mozilla.org/en-US/docs/Web/Guide/CSS/Underscores_in_class_and_ID_Names

#20

Updated by Serghei Mihai over 9 years ago

  • Status changed from En cours to Résolu (à déployer)

Appliqué par commit calebasse|commit:3c1f0b4a59e4052b3c0bf3bfb8d84ee716d6f685.

#21

Updated by Mikaël Ates about 9 years ago

  • Status changed from Résolu (à déployer) to Fermé

Ok

Also available in: Atom PDF