Amend report

validata_ui/templates/validation_report.html

@@ -7,6 +7,7 @@
         font-size: 1.2em;
         font-weight: bold;
     }
+
     @media print {
         .hidden-print {
             display: none;
@@ -93,60 +94,77 @@
 
     <h2>La table est invalide</h2>
 
+    <p class="text">{{ report.table['error-stats']['total'] }} erreur(s) détectée(s).</p>
+
+    {% set structure_errors = report.table['error-stats']['structure-errors'] %}
+    {% if structure_errors['nb'] != 0 %}
+    Erreur(s) de structure ({{ structure_errors['nb'] }}) :
+    <ul>
+        {% for elt in structure_errors['distribution'] %}
+        <li>{{ elt[0] }} ({{ elt[1] }})</li>
+        {% endfor %}
+    </ul>
+    {% endif %}
+
+    {% if report.table.do_display_body_errors %}
+    {% set value_errors = report.table['error-stats']['value-errors'] %}
+    {% if value_errors %}
+    {% if value_errors['nb'] != 0 %}
+    Erreur(s) de valeur ({{ value_errors['nb'] }}) :
+    <ul>
+        {% for elt in value_errors['distribution'] %}
+        <li>{{ elt[0] }} ({{ elt[1] }})</li>
+        {% endfor %}
+    </ul>
+    {% endif %}
+    {% endif %}
+    {% endif %}
+
     {# table checks #}
 
     {% if report.table.errors.structure %}
     <div>
-        <h3>Problèmes de structure</h3>
+        <h3>Structure</h3>
         {% for err in report.table.errors.structure %}
         <div class="alert alert-danger">
             {{ err.message | safe}}
         </div>
         {% endfor %}
     </div>
+    {% else %}
+    <p class="text">Aucune erreur de structure</p>
     {% endif %}
 
     <!-- row checks -->
 
+    {# We do display body errors! #}
+    {% if report.table.do_display_body_errors %}
+    <h3>Contenu</h3>
+
+    {# No errors - display preview #}
     {% if not report.table.errors.body %}
-    <h3>Problèmes de contenu</h3>
     <p class="text">
         Aucune erreur de contenu
     </p>
-
     {{ tables.preview(source_data) }}
+    
+    {% else %}
 
+    {# display errors #}
+    {{ tables.body_errors(report, source_data) }}
+    
     {% endif %}
 
-    {% if report.table.errors.body %}
-
-    {% if not report.table.display_body_errors %}
-    <p class="text">
-        Veuillez corriger ce(s) erreur(s) pour visualiser les problèmes de contenu.
-    </p>
-
+    {# do not display errors #}
+    {% else %}
     <p class="text">
-        <a href="#" id="show_body_errors">Affichez
-            tout de même les problèmes de contenu</a>
+        Veuillez corriger ce(s) erreur(s) pour visualiser les éventuels problèmes de contenu.
     </p>
-    <div class="alert alert-warning">
-        Attention, en cas de problèmes de structure signalés ci-dessus, certains messages d'erreurs ne seront pas
-        pertinents
-    </div>
-
-    <div id="body_errors" style="display: none">
-        {{ tables.body_errors(report, source_data) }}
-    </div>
-    {% else %}
-    <div>
-        {{ tables.body_errors(report, source_data) }}
-    </div>
-    {% endif %}
     {% endif %}
 
 
-    {% endif %}
     {% endblock %}
+
     {% block footer %}
     <script>
         $(function () {
@@ -156,18 +174,7 @@
                 placement: 'auto',
                 trigger: 'hover'
             });
-
-            // Show body errors
-            $('#show_body_errors').on('click', function () {
-                $('#body_errors').show();
-                $(this).remove();
-            })
         })
     </script>
 
-{#
-{{ report.table.errors.structure|length }} erreur(s) détectée(s)
+    {% endblock %}
\ No newline at end of file
-{{ report.table.errors.body|length }} erreur(s) détectée(s)
-
-#}
-{% endblock %}

validata_ui/views.py

@@ -139,8 +139,9 @@ def create_validata_report(goodtables_report, schema):
             report['table']['errors']['body'].append(err)
 
     # Checks if there are structure errors different to invalid-column-delimiter
-    report['table']['display_body_errors'] = all(err['code'] == 'invalid-column-delimiter'
-                                                 for err in report['table']['errors']['structure'])
+    structure_errors = report['table']['errors']['structure']
+    report['table']['do_display_body_errors'] = len(structure_errors) == 0 or \
+        all(err['code'] == 'invalid-column-delimiter' for err in structure_errors)
 
     # Group body errors by row id
     rows = []