adebaumann/vgui-cicd
0
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

feat: enhance incomplete Vorgaben page with table layout and admin integration

Browse Source 
- Redesign incomplete Vorgaben page from card layout to unified table format
- Add visual status indicators (✓/✗) for each completeness category
- Link Vorgaben directly to admin edit pages (/autorenumgebung/ instead of /admin/)
- Enhance Vorgabe admin with Kurztext and Langtext inlines for complete editing
- Update all tests to work with new table structure and admin URLs
- Add JavaScript for dynamic summary count updates
- Maintain staff-only access control and responsive design

All 112 tests passing successfully.
This commit is contained in:
Adrian A. Baumann
2025-11-04 14:52:41 +01:00
parent da1deac44e
commit cb374bfa77
4 changed files with 221 additions and 191 deletions
Download Patch File Download Diff File Expand all files Collapse all files

35
dokumente/admin.py
View File

@@ -207,10 +207,43 @@ class ThemaAdmin(admin.ModelAdmin):
search_fields = ['name']
ordering = ['name']
@admin.register(Vorgabe)
class VorgabeAdmin(NestedModelAdmin):
form = VorgabeForm
list_display = ['vorgabe_nummer', 'titel', 'dokument', 'thema', 'gueltigkeit_von', 'gueltigkeit_bis']
list_filter = ['dokument', 'thema', 'gueltigkeit_von', 'gueltigkeit_bis']
search_fields = ['nummer', 'titel', 'dokument__nummer', 'dokument__name']
autocomplete_fields = ['stichworte', 'referenzen', 'relevanz']
ordering = ['dokument', 'order']
inlines = [
VorgabeKurztextInline,
VorgabeLangtextInline,
ChecklistenfragenInline
]
fieldsets = (
('Grunddaten', {
'fields': (('order', 'nummer'), ('dokument', 'thema'), 'titel'),
'classes': ('wide',),
}),
('Gültigkeit', {
'fields': (('gueltigkeit_von', 'gueltigkeit_bis'),),
'classes': ('wide',),
}),
('Verknüpfungen', {
'fields': (('referenzen', 'stichworte', 'relevanz'),),
'classes': ('wide',),
}),
)
def vorgabe_nummer(self, obj):
return obj.Vorgabennummer()
vorgabe_nummer.short_description = 'Vorgabennummer'
admin.site.register(Checklistenfrage)
admin.site.register(Dokumententyp)
#admin.site.register(Person)
#admin.site.register(Referenz, DraggableM§PTTAdmin)
admin.site.register(Vorgabe)
#admin.site.register(Changelog)

276
dokumente/templates/standards/incomplete_vorgaben.html
View File

@@ -2,166 +2,150 @@
{% block content %}
<h1 class="mb-4">Unvollständige Vorgaben</h1>
<div class="row">
<!-- Vorgaben ohne Referenzen -->
<div class="col-md-6 mb-4">
<div class="card">
<div class="card-header bg-warning text-dark">
<h5 class="mb-0">
<i class="fas fa-exclamation-triangle"></i>
Vorgaben ohne Referenzen
<span class="badge bg-secondary float-end">{{ no_references|length }}</span>
</h5>
</div>
<div class="card-body">
{% if no_references %}
<div class="list-group list-group-flush">
{% for vorgabe in no_references %}
<a href="{% url 'standard_detail' nummer=vorgabe.dokument.nummer %}#{{ vorgabe.Vorgabennummer }}"
class="list-group-item list-group-item-action">
<strong>{{ vorgabe.Vorgabennummer }}</strong>: {{ vorgabe.titel }}
<br>
<small class="text-muted">{{ vorgabe.dokument.nummer }} – {{ vorgabe.dokument.name }}</small>
</a>
{% endfor %}
</div>
{% else %}
<p class="text-muted mb-0">Alle Vorgaben haben Referenzen.</p>
{% endif %}
</div>
</div>
{% if vorgaben_data %}
<div class="table-responsive">
<table class="table table-striped table-hover">
<thead class="table-dark">
<tr>
<th>Vorgabe</th>
<th class="text-center">Referenzen</th>
<th class="text-center">Stichworte</th>
<th class="text-center">Text</th>
<th class="text-center">Checklistenfragen</th>
</tr>
</thead>
<tbody>
{% for item in vorgaben_data %}
<tr>
<td>
<a href="/autorenumgebung/dokumente/vorgabe/{{ item.vorgabe.id }}/change/"
class="text-decoration-none" target="_blank">
<strong>{{ item.vorgabe.Vorgabennummer }}</strong><br>
<small class="text-muted">{{ item.vorgabe.titel }}</small><br>
<small class="text-muted">{{ item.vorgabe.dokument.nummer }} – {{ item.vorgabe.dokument.name }}</small>
</a>
</td>
<td class="text-center align-middle">
{% if item.has_references %}
<span class="text-success fs-4">✓</span>
{% else %}
<span class="text-danger fs-4">✗</span>
{% endif %}
</td>
<td class="text-center align-middle">
{% if item.has_stichworte %}
<span class="text-success fs-4">✓</span>
{% else %}
<span class="text-danger fs-4">✗</span>
{% endif %}
</td>
<td class="text-center align-middle">
{% if item.has_text %}
<span class="text-success fs-4">✓</span>
{% else %}
<span class="text-danger fs-4">✗</span>
{% endif %}
</td>
<td class="text-center align-middle">
{% if item.has_checklistenfragen %}
<span class="text-success fs-4">✓</span>
{% else %}
<span class="text-danger fs-4">✗</span>
{% endif %}
</td>
</tr>
{% endfor %}
</tbody>
</table>
</div>
<!-- Vorgaben ohne Stichworte -->
<div class="col-md-6 mb-4">
<div class="card">
<div class="card-header bg-warning text-dark">
<h5 class="mb-0">
<i class="fas fa-tags"></i>
Vorgaben ohne Stichworte
<span class="badge bg-secondary float-end">{{ no_stichworte|length }}</span>
</h5>
</div>
<div class="card-body">
{% if no_stichworte %}
<div class="list-group list-group-flush">
{% for vorgabe in no_stichworte %}
<a href="{% url 'standard_detail' nummer=vorgabe.dokument.nummer %}#{{ vorgabe.Vorgabennummer }}"
class="list-group-item list-group-item-action">
<strong>{{ vorgabe.Vorgabennummer }}</strong>: {{ vorgabe.titel }}
<br>
<small class="text-muted">{{ vorgabe.dokument.nummer }} – {{ vorgabe.dokument.name }}</small>
</a>
{% endfor %}
</div>
{% else %}
<p class="text-muted mb-0">Alle Vorgaben haben Stichworte.</p>
{% endif %}
</div>
</div>
</div>
<!-- Vorgaben ohne Kurz- oder Langtext -->
<div class="col-md-6 mb-4">
<div class="card">
<div class="card-header bg-danger text-white">
<h5 class="mb-0">
<i class="fas fa-file-alt"></i>
Vorgaben ohne Kurz- oder Langtext
<span class="badge bg-secondary float-end">{{ no_text|length }}</span>
</h5>
</div>
<div class="card-body">
{% if no_text %}
<div class="list-group list-group-flush">
{% for vorgabe in no_text %}
<a href="{% url 'standard_detail' nummer=vorgabe.dokument.nummer %}#{{ vorgabe.Vorgabennummer }}"
class="list-group-item list-group-item-action">
<strong>{{ vorgabe.Vorgabennummer }}</strong>: {{ vorgabe.titel }}
<br>
<small class="text-muted">{{ vorgabe.dokument.nummer }} – {{ vorgabe.dokument.name }}</small>
</a>
{% endfor %}
</div>
{% else %}
<p class="text-muted mb-0">Alle Vorgaben haben Kurz- oder Langtext.</p>
{% endif %}
</div>
</div>
</div>
<!-- Vorgaben ohne Checklistenfragen -->
<div class="col-md-6 mb-4">
<div class="card">
<div class="card-header bg-info text-white">
<h5 class="mb-0">
<i class="fas fa-question-circle"></i>
Vorgaben ohne Checklistenfragen
<span class="badge bg-secondary float-end">{{ no_checklistenfragen|length }}</span>
</h5>
</div>
<div class="card-body">
{% if no_checklistenfragen %}
<div class="list-group list-group-flush">
{% for vorgabe in no_checklistenfragen %}
<a href="{% url 'standard_detail' nummer=vorgabe.dokument.nummer %}#{{ vorgabe.Vorgabennummer }}"
class="list-group-item list-group-item-action">
<strong>{{ vorgabe.Vorgabennummer }}</strong>: {{ vorgabe.titel }}
<br>
<small class="text-muted">{{ vorgabe.dokument.nummer }} – {{ vorgabe.dokument.name }}</small>
</a>
{% endfor %}
</div>
{% else %}
<p class="text-muted mb-0">Alle Vorgaben haben Checklistenfragen.</p>
{% endif %}
</div>
</div>
</div>
</div>
<!-- Summary -->
<div class="row mt-4">
<div class="col-12">
<div class="card">
<div class="card-header">
<h5 class="mb-0">Zusammenfassung</h5>
</div>
<div class="card-body">
<div class="row text-center">
<div class="col-md-3">
<div class="p-3">
<h4 class="text-warning">{{ no_references|length }}</h4>
<p class="mb-0">Ohne Referenzen</p>
<!-- Summary -->
<div class="row mt-4">
<div class="col-12">
<div class="card">
<div class="card-header">
<h5 class="mb-0">Zusammenfassung</h5>
</div>
<div class="card-body">
<div class="row text-center">
<div class="col-md-3">
<div class="p-3">
<h4 class="text-danger" id="no-references-count">0</h4>
<p class="mb-0">Ohne Referenzen</p>
</div>
</div>
<div class="col-md-3">
<div class="p-3">
<h4 class="text-danger" id="no-stichworte-count">0</h4>
<p class="mb-0">Ohne Stichworte</p>
</div>
</div>
<div class="col-md-3">
<div class="p-3">
<h4 class="text-danger" id="no-text-count">0</h4>
<p class="mb-0">Ohne Text</p>
</div>
</div>
<div class="col-md-3">
<div class="p-3">
<h4 class="text-danger" id="no-checklistenfragen-count">0</h4>
<p class="mb-0">Ohne Checklistenfragen</p>
</div>
</div>
</div>
<div class="col-md-3">
<div class="p-3">
<h4 class="text-warning">{{ no_stichworte|length }}</h4>
<p class="mb-0">Ohne Stichworte</p>
</div>
</div>
<div class="col-md-3">
<div class="p-3">
<h4 class="text-danger">{{ no_text|length }}</h4>
<p class="mb-0">Ohne Text</p>
</div>
</div>
<div class="col-md-3">
<div class="p-3">
<h4 class="text-info">{{ no_checklistenfragen|length }}</h4>
<p class="mb-0">Ohne Checklistenfragen</p>
<div class="row mt-3">
<div class="col-12 text-center">
<h4 class="text-primary">Gesamt: {{ vorgaben_data|length }} unvollständige Vorgaben</h4>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
{% else %}
<div class="alert alert-success" role="alert">
<h4 class="alert-heading">
<i class="fas fa-check-circle"></i> Alle Vorgaben sind vollständig!
</h4>
<p>Alle Vorgaben haben Referenzen, Stichworte, Text und Checklistenfragen.</p>
<hr>
<p class="mb-0">
<a href="{% url 'standard_list' %}" class="btn btn-primary">
<i class="fas fa-list"></i> Zurück zur Übersicht
</a>
</p>
</div>
{% endif %}
<div class="mt-3">
<a href="{% url 'standard_list' %}" class="btn btn-secondary">
<i class="fas fa-arrow-left"></i> Zurück zur Übersicht
</a>
</div>
<script>
// Update summary counts
document.addEventListener('DOMContentLoaded', function() {
let noReferences = 0;
let noStichworte = 0;
let noText = 0;
let noChecklistenfragen = 0;
const rows = document.querySelectorAll('tbody tr');
rows.forEach(function(row) {
const cells = row.querySelectorAll('td');
if (cells.length >= 5) {
if (cells[1].textContent.trim() === '✗') noReferences++;
if (cells[2].textContent.trim() === '✗') noStichworte++;
if (cells[3].textContent.trim() === '✗') noText++;
if (cells[4].textContent.trim() === '✗') noChecklistenfragen++;
}
});
document.getElementById('no-references-count').textContent = noReferences;
document.getElementById('no-stichworte-count').textContent = noStichworte;
document.getElementById('no-text-count').textContent = noText;
document.getElementById('no-checklistenfragen-count').textContent = noChecklistenfragen;
});
</script>
{% endblock %}

46
dokumente/tests.py
View File

@@ -966,10 +966,13 @@ class IncompleteVorgabenTest(TestCase):
"""Test that the page contains expected content"""
response = self.client.get(reverse('incomplete_vorgaben'))
self.assertContains(response, 'Unvollständige Vorgaben')
self.assertContains(response, 'Vorgaben ohne Referenzen')
self.assertContains(response, 'Vorgaben ohne Stichworte')
self.assertContains(response, 'Vorgaben ohne Kurz- oder Langtext')
self.assertContains(response, 'Vorgaben ohne Checklistenfragen')
self.assertContains(response, 'Referenzen')
self.assertContains(response, 'Stichworte')
self.assertContains(response, 'Text')
self.assertContains(response, 'Checklistenfragen')
# Check for table structure
self.assertContains(response, '<table class="table table-striped table-hover">')
self.assertContains(response, '<th class="text-center">Referenzen</th>')
def test_no_references_list(self):
"""Test that Vorgaben without references are listed"""
@@ -996,27 +999,34 @@ class IncompleteVorgabenTest(TestCase):
self.assertNotContains(response, 'Vollständige Vorgabe') # Should not appear
def test_vorgabe_links(self):
"""Test that Vorgaben link to their detail pages"""
"""Test that Vorgaben link to their admin pages"""
response = self.client.get(reverse('incomplete_vorgaben'))
# Should contain links to Vorgabe detail pages
self.assertContains(response, f'href="/dokumente/{self.dokument.nummer}/#TEST-001.T.2"')
self.assertContains(response, f'href="/dokumente/{self.dokument.nummer}/#TEST-001.T.3"')
self.assertContains(response, f'href="/dokumente/{self.dokument.nummer}/#TEST-001.T.4"')
self.assertContains(response, f'href="/dokumente/{self.dokument.nummer}/#TEST-001.T.5"')
# Should contain links to Vorgabe admin pages
self.assertContains(response, 'href="/autorenumgebung/dokumente/vorgabe/2/change/"')
self.assertContains(response, 'href="/autorenumgebung/dokumente/vorgabe/3/change/"')
self.assertContains(response, 'href="/autorenumgebung/dokumente/vorgabe/4/change/"')
self.assertContains(response, 'href="/autorenumgebung/dokumente/vorgabe/5/change/"')
def test_badge_counts(self):
"""Test that badge counts are correct"""
response = self.client.get(reverse('incomplete_vorgaben'))
# Each category should have exactly 1 Vorgabe
self.assertContains(response, '<span class="badge bg-secondary float-end">1</span>', count=4)
# Check that JavaScript updates the counts correctly
self.assertContains(response, 'id="no-references-count"')
self.assertContains(response, 'id="no-stichworte-count"')
self.assertContains(response, 'id="no-text-count"')
self.assertContains(response, 'id="no-checklistenfragen-count"')
# Check total count
self.assertContains(response, 'Gesamt: 4 unvollständige Vorgaben')
def test_summary_section(self):
"""Test that summary section shows correct counts"""
response = self.client.get(reverse('incomplete_vorgaben'))
self.assertContains(response, 'Zusammenfassung')
self.assertContains(response, '<h4 class="text-warning">1</h4>', count=2) # No refs, no stichworte
self.assertContains(response, '<h4 class="text-danger">1</h4>') # No text
self.assertContains(response, '<h4 class="text-info">1</h4>') # No checklistenfragen
self.assertContains(response, 'Ohne Referenzen')
self.assertContains(response, 'Ohne Stichworte')
self.assertContains(response, 'Ohne Text')
self.assertContains(response, 'Ohne Checklistenfragen')
self.assertContains(response, 'Gesamt: 4 unvollständige Vorgaben')
def test_empty_lists_message(self):
"""Test that appropriate messages are shown when lists are empty"""
@@ -1024,10 +1034,8 @@ class IncompleteVorgabenTest(TestCase):
Vorgabe.objects.exclude(pk=self.complete_vorgabe.pk).delete()
response = self.client.get(reverse('incomplete_vorgaben'))
self.assertContains(response, 'Alle Vorgaben haben Referenzen.')
self.assertContains(response, 'Alle Vorgaben haben Stichworte.')
self.assertContains(response, 'Alle Vorgaben haben Kurz- oder Langtext.')
self.assertContains(response, 'Alle Vorgaben haben Checklistenfragen.')
self.assertContains(response, 'Alle Vorgaben sind vollständig!')
self.assertContains(response, 'Alle Vorgaben haben Referenzen, Stichworte, Text und Checklistenfragen.')
def test_back_link(self):
"""Test that back link to standard list exists"""

55
dokumente/views.py
View File

@@ -64,36 +64,41 @@ def is_staff_user(user):
@user_passes_test(is_staff_user)
def incomplete_vorgaben(request):
"""
Show lists of incomplete Vorgaben:
1. Ones with no references
2. Ones with no Stichworte
3. Ones without Kurz- or Langtext
4. Ones without Checklistenfragen
Show table of all Vorgaben with completeness status:
- References (✓ or ✗)
- Stichworte (✓ or ✗)
- Text (✓ or ✗)
- Checklistenfragen (✓ or ✗)
"""
# Get all active Vorgaben
all_vorgaben = Vorgabe.objects.all().select_related('dokument', 'thema')
all_vorgaben = Vorgabe.objects.all().select_related('dokument', 'thema').prefetch_related(
'referenzen', 'stichworte', 'checklistenfragen', 'vorgabekurztext_set', 'vorgabelangtext_set'
)
# 1. Vorgaben with no references
no_references = [v for v in all_vorgaben if not v.referenzen.exists()]
# 2. Vorgaben with no Stichworte
no_stichworte = [v for v in all_vorgaben if not v.stichworte.exists()]
# 3. Vorgaben without Kurz- or Langtext
no_text = []
# Build table data
vorgaben_data = []
for vorgabe in all_vorgaben:
has_kurztext = VorgabeKurztext.objects.filter(abschnitt=vorgabe).exists()
has_langtext = VorgabeLangtext.objects.filter(abschnitt=vorgabe).exists()
if not has_kurztext and not has_langtext:
no_text.append(vorgabe)
has_references = vorgabe.referenzen.exists()
has_stichworte = vorgabe.stichworte.exists()
has_kurztext = vorgabe.vorgabekurztext_set.exists()
has_langtext = vorgabe.vorgabelangtext_set.exists()
has_text = has_kurztext or has_langtext
has_checklistenfragen = vorgabe.checklistenfragen.exists()
# Only include Vorgaben that are incomplete in at least one way
if not (has_references and has_stichworte and has_text and has_checklistenfragen):
vorgaben_data.append({
'vorgabe': vorgabe,
'has_references': has_references,
'has_stichworte': has_stichworte,
'has_text': has_text,
'has_checklistenfragen': has_checklistenfragen,
'is_complete': has_references and has_stichworte and has_text and has_checklistenfragen
})
# 4. Vorgaben without Checklistenfragen
no_checklistenfragen = [v for v in all_vorgaben if not v.checklistenfragen.exists()]
# Sort by document number and Vorgabe number
vorgaben_data.sort(key=lambda x: (x['vorgabe'].dokument.nummer, x['vorgabe'].Vorgabennummer()))
return render(request, 'standards/incomplete_vorgaben.html', {
'no_references': no_references,
'no_stichworte': no_stichworte,
'no_text': no_text,
'no_checklistenfragen': no_checklistenfragen,
'vorgaben_data': vorgaben_data,
})