Compare commits
76 Commits
rename_sta
...
feature/li
| Author | SHA1 | Date | |
|---|---|---|---|
| b82c6fea38 | |||
| cb374bfa77 | |||
| 2b41490806 | |||
| 7186fa2cbe | |||
| da1deac44e | |||
| faae37e6ae | |||
| 6aefb046b6 | |||
| 2350cca32c | |||
| 671d259c44 | |||
| 28a1bb4b62 | |||
| 898e9b8163 | |||
| 48bf8526b9 | |||
| 7e4d2fa29b | |||
| 779604750e | |||
| aca9a2f307 | |||
| d14d9eba4c | |||
| 081ea4de1c | |||
| a075811173 | |||
| d4143da9fc | |||
| b0c9b89e94 | |||
|
|
94363d49ce | ||
|
|
8bca1bb3c7 | ||
|
|
1ce8eb15c0 | ||
|
|
4d2ffeea27 | ||
|
|
8860947d38 | ||
|
|
6df72c95cb | ||
| 2afada0bce | |||
|
|
a42a65b40f | ||
| 5609a735f4 | |||
| 6654779e67 | |||
| 7befde104d | |||
| 96819a7427 | |||
| a437af554b | |||
| 650fe0a87b | |||
|
|
ddf035c50f | ||
|
|
886baa163e | ||
|
|
1146506ca2 | ||
|
|
9610024739 | ||
|
|
c8755e4339 | ||
|
|
0bc1fe7413 | ||
|
|
8ce761c248 | ||
|
|
39a2021cc3 | ||
| 957a1b9255 | |||
| afc07d4561 | |||
| af06598172 | |||
| 4213ca60ac | |||
| bf2f15fa5c | |||
| c1eb2d7871 | |||
| b29e894b22 | |||
| 0f096d18aa | |||
| 9b484787a4 | |||
| 8dd3b4e9af | |||
| 0d0199ca62 | |||
| 5f58d660c0 | |||
| e84f25ca1d | |||
| dfb8eeef97 | |||
| 0225fb3396 | |||
| 7377ddaea3 | |||
| 67c393ecf1 | |||
| dbb3ecd5bf | |||
| 966cd46228 | |||
| 1ee9b3c46f | |||
| 8f57f5fc5b | |||
| cd7195b3aa | |||
| 020dff0871 | |||
| 1dbdbc7f3c | |||
| 4d1232b764 | |||
|
fe2e02934a
|
|||
|
add1a88ce4
|
|||
| 3c23918e1f | |||
| fa0a2a9df9 | |||
|
|
9feaf6686f | ||
| 7087be672a | |||
|
|
969141601d | ||
| d46d937e93 | |||
| 4d713b3763 |
5
.gitignore
vendored
5
.gitignore
vendored
@@ -10,3 +10,8 @@ keys/
|
||||
.idea/
|
||||
|
||||
*.kate-swp
|
||||
node_modules/
|
||||
package-lock.json
|
||||
package.json
|
||||
# Diagram cache directory
|
||||
media/diagram_cache/
|
||||
|
||||
105
Documentation/DIAGRAM_CACHING.md
Normal file
105
Documentation/DIAGRAM_CACHING.md
Normal file
@@ -0,0 +1,105 @@
|
||||
# Diagram POST Caching Implementation
|
||||
|
||||
This feature replaces the URL-encoded GET approach for diagram generation with POST requests and local filesystem caching.
|
||||
|
||||
## Changes Overview
|
||||
|
||||
### New Files
|
||||
- `diagramm_proxy/__init__.py` - Module initialization
|
||||
- `diagramm_proxy/diagram_cache.py` - Caching logic and POST request handling
|
||||
- `abschnitte/management/commands/clear_diagram_cache.py` - Management command for cache clearing
|
||||
|
||||
### Modified Files
|
||||
- `abschnitte/utils.py` - Updated `render_textabschnitte()` to use caching
|
||||
- `.gitignore` - Added cache directory exclusion
|
||||
|
||||
## Configuration Required
|
||||
|
||||
Add to your Django settings file (e.g., `VorgabenUI/settings.py`):
|
||||
|
||||
```python
|
||||
# Diagram cache settings
|
||||
DIAGRAM_CACHE_DIR = 'diagram_cache' # relative to MEDIA_ROOT
|
||||
|
||||
# Ensure MEDIA_ROOT and MEDIA_URL are configured
|
||||
MEDIA_ROOT = os.path.join(BASE_DIR, 'media')
|
||||
MEDIA_URL = '/media/'
|
||||
```
|
||||
|
||||
### URL Configuration
|
||||
|
||||
Ensure media files are served in development. In your main `urls.py`:
|
||||
|
||||
```python
|
||||
from django.conf import settings
|
||||
from django.conf.urls.static import static
|
||||
|
||||
# ... existing urlpatterns ...
|
||||
|
||||
# Serve media files in development
|
||||
if settings.DEBUG:
|
||||
urlpatterns += static(settings.MEDIA_URL, document_root=settings.MEDIA_ROOT)
|
||||
```
|
||||
|
||||
## How It Works
|
||||
|
||||
1. When a diagram is rendered, the system computes a SHA256 hash of the diagram content
|
||||
2. It checks if a cached SVG exists for that hash
|
||||
3. If cached: serves the existing file
|
||||
4. If not cached: POSTs content to Kroki server, saves the response, and serves it
|
||||
5. Diagrams are served from `MEDIA_URL/diagram_cache/{type}/{hash}.svg`
|
||||
|
||||
## Benefits
|
||||
|
||||
- **No URL length limitations** - Content is POSTed instead of URL-encoded
|
||||
- **Improved performance** - Cached diagrams are served directly from filesystem
|
||||
- **Reduced server load** - Kroki server is only called once per unique diagram
|
||||
- **Persistent cache** - Survives application restarts
|
||||
- **Better error handling** - Graceful fallback on generation failures
|
||||
|
||||
## Usage
|
||||
|
||||
### Viewing Diagrams
|
||||
No changes required - diagrams will be automatically cached on first render.
|
||||
|
||||
### Clearing Cache
|
||||
|
||||
Clear all cached diagrams:
|
||||
```bash
|
||||
python manage.py clear_diagram_cache
|
||||
```
|
||||
|
||||
Clear diagrams of a specific type:
|
||||
```bash
|
||||
python manage.py clear_diagram_cache --type plantuml
|
||||
python manage.py clear_diagram_cache --type mermaid
|
||||
```
|
||||
|
||||
## Testing
|
||||
|
||||
1. Create or view a page with diagrams
|
||||
2. Verify diagrams render correctly
|
||||
3. Check that `media/diagram_cache/` directory is created with cached SVGs
|
||||
4. Refresh the page - second load should be faster (cache hit)
|
||||
5. Check logs for cache hit/miss messages
|
||||
6. Test cache clearing command
|
||||
|
||||
## Migration Notes
|
||||
|
||||
- Existing diagrams will be regenerated on first view after deployment
|
||||
- The old URL-based approach is completely replaced
|
||||
- No database migrations needed
|
||||
- Ensure `requests` library is installed (already in requirements.txt)
|
||||
|
||||
## Troubleshooting
|
||||
|
||||
### Diagrams not rendering
|
||||
- Check that MEDIA_ROOT and MEDIA_URL are configured correctly
|
||||
- Verify Kroki server is accessible at `http://svckroki:8000`
|
||||
- Check application logs for error messages
|
||||
- Ensure media directory is writable
|
||||
|
||||
### Cache not working
|
||||
- Verify Django storage configuration
|
||||
- Check file permissions on media/diagram_cache directory
|
||||
- Review logs for cache-related errors
|
||||
@@ -2,3 +2,5 @@
|
||||
|
||||
There are examples for importing text in the "Documentation"-directory. Actual documentation follows.
|
||||
|
||||
Documentation on Confluence so far.
|
||||
This commit should be signed.
|
||||
|
||||
@@ -126,6 +126,13 @@ STATICFILES_DIRS= (
|
||||
os.path.join(BASE_DIR,"static"),
|
||||
)
|
||||
|
||||
# Media files (User-uploaded content)
|
||||
MEDIA_URL = '/media/'
|
||||
MEDIA_ROOT = os.path.join(BASE_DIR, 'media')
|
||||
|
||||
# Diagram cache settings
|
||||
DIAGRAM_CACHE_DIR = 'diagram_cache' # relative to MEDIA_ROOT
|
||||
|
||||
# Default primary key field type
|
||||
# https://docs.djangoproject.com/en/5.2/ref/settings/#default-auto-field
|
||||
|
||||
|
||||
@@ -139,6 +139,13 @@ STATICFILES_DIRS= (
|
||||
os.path.join(BASE_DIR,"static"),
|
||||
)
|
||||
|
||||
# Media files (User-uploaded content)
|
||||
MEDIA_URL = '/media/'
|
||||
MEDIA_ROOT = os.path.join(BASE_DIR, 'media')
|
||||
|
||||
# Diagram cache settings
|
||||
DIAGRAM_CACHE_DIR = 'diagram_cache' # relative to MEDIA_ROOT
|
||||
|
||||
# Default primary key field type
|
||||
# https://docs.djangoproject.com/en/5.2/ref/settings/#default-auto-field
|
||||
|
||||
|
||||
@@ -34,5 +34,11 @@ urlpatterns = [
|
||||
path('referenzen/', referenzen.views.tree, name="referenz_tree"),
|
||||
path('referenzen/<str:refid>/', referenzen.views.detail, name="referenz_detail"),
|
||||
re_path(r'^diagramm/(?P<path>.*)$', DiagrammProxyView.as_view()),
|
||||
] + static(settings.STATIC_URL, document_root=settings.STATIC_ROOT)
|
||||
]
|
||||
|
||||
# Serve static files
|
||||
urlpatterns += static(settings.STATIC_URL, document_root=settings.STATIC_ROOT)
|
||||
|
||||
# Serve media files (including cached diagrams)
|
||||
if settings.DEBUG:
|
||||
urlpatterns += static(settings.MEDIA_URL, document_root=settings.MEDIA_ROOT)
|
||||
|
||||
1
abschnitte/management/__init__.py
Normal file
1
abschnitte/management/__init__.py
Normal file
@@ -0,0 +1 @@
|
||||
# Management commands
|
||||
1
abschnitte/management/commands/__init__.py
Normal file
1
abschnitte/management/commands/__init__.py
Normal file
@@ -0,0 +1 @@
|
||||
# Commands package
|
||||
23
abschnitte/management/commands/clear_diagram_cache.py
Normal file
23
abschnitte/management/commands/clear_diagram_cache.py
Normal file
@@ -0,0 +1,23 @@
|
||||
from django.core.management.base import BaseCommand
|
||||
from diagramm_proxy.diagram_cache import clear_cache
|
||||
|
||||
|
||||
class Command(BaseCommand):
|
||||
help = 'Clear cached diagrams'
|
||||
|
||||
def add_arguments(self, parser):
|
||||
parser.add_argument(
|
||||
'--type',
|
||||
type=str,
|
||||
help='Diagram type to clear (e.g., plantuml, mermaid)',
|
||||
)
|
||||
|
||||
def handle(self, *args, **options):
|
||||
diagram_type = options.get('type')
|
||||
if diagram_type:
|
||||
self.stdout.write(f'Clearing cache for {diagram_type}...')
|
||||
clear_cache(diagram_type)
|
||||
else:
|
||||
self.stdout.write('Clearing all diagram caches...')
|
||||
clear_cache()
|
||||
self.stdout.write(self.style.SUCCESS('Cache cleared successfully'))
|
||||
@@ -1,3 +1,820 @@
|
||||
from django.test import TestCase
|
||||
from django.test import TestCase, TransactionTestCase
|
||||
from django.core.management import call_command
|
||||
from django.conf import settings
|
||||
from django.core.files.storage import default_storage
|
||||
from unittest.mock import patch, Mock, MagicMock
|
||||
from io import StringIO
|
||||
import os
|
||||
import hashlib
|
||||
import tempfile
|
||||
import shutil
|
||||
|
||||
# Create your tests here.
|
||||
from .models import AbschnittTyp, Textabschnitt
|
||||
from .utils import render_textabschnitte, md_table_to_html
|
||||
from diagramm_proxy.diagram_cache import (
|
||||
get_cached_diagram, compute_hash, get_cache_path, clear_cache
|
||||
)
|
||||
|
||||
|
||||
class AbschnittTypModelTest(TestCase):
|
||||
"""Test cases for AbschnittTyp model"""
|
||||
|
||||
def setUp(self):
|
||||
"""Set up test data"""
|
||||
self.abschnitttyp = AbschnittTyp.objects.create(
|
||||
abschnitttyp="text"
|
||||
)
|
||||
|
||||
def test_abschnitttyp_creation(self):
|
||||
"""Test that AbschnittTyp is created correctly"""
|
||||
self.assertEqual(self.abschnitttyp.abschnitttyp, "text")
|
||||
|
||||
def test_abschnitttyp_str(self):
|
||||
"""Test string representation of AbschnittTyp"""
|
||||
self.assertEqual(str(self.abschnitttyp), "text")
|
||||
|
||||
def test_abschnitttyp_verbose_name_plural(self):
|
||||
"""Test verbose name plural"""
|
||||
self.assertEqual(
|
||||
AbschnittTyp._meta.verbose_name_plural,
|
||||
"Abschnitttypen"
|
||||
)
|
||||
|
||||
def test_abschnitttyp_primary_key(self):
|
||||
"""Test that abschnitttyp field is the primary key"""
|
||||
pk_field = AbschnittTyp._meta.pk
|
||||
self.assertEqual(pk_field.name, 'abschnitttyp')
|
||||
|
||||
def test_create_multiple_abschnitttypen(self):
|
||||
"""Test creating multiple AbschnittTyp objects"""
|
||||
types = ['liste ungeordnet', 'liste geordnet', 'tabelle', 'diagramm', 'code']
|
||||
for typ in types:
|
||||
AbschnittTyp.objects.create(abschnitttyp=typ)
|
||||
|
||||
self.assertEqual(AbschnittTyp.objects.count(), 6) # Including setUp type
|
||||
|
||||
|
||||
class TextabschnittModelTest(TestCase):
|
||||
"""Test cases for Textabschnitt abstract model using VorgabeLangtext"""
|
||||
|
||||
def setUp(self):
|
||||
"""Set up test data"""
|
||||
from dokumente.models import Dokumententyp, Dokument, Vorgabe, VorgabeLangtext, Thema
|
||||
from datetime import date
|
||||
|
||||
self.typ_text = AbschnittTyp.objects.create(abschnitttyp="text")
|
||||
self.typ_code = AbschnittTyp.objects.create(abschnitttyp="code")
|
||||
|
||||
# Create required dokumente objects
|
||||
self.dokumententyp = Dokumententyp.objects.create(
|
||||
name="Test Type", verantwortliche_ve="TEST"
|
||||
)
|
||||
self.dokument = Dokument.objects.create(
|
||||
nummer="TEST-001",
|
||||
name="Test Doc",
|
||||
dokumententyp=self.dokumententyp,
|
||||
aktiv=True
|
||||
)
|
||||
self.thema = Thema.objects.create(name="Test Thema")
|
||||
self.vorgabe = Vorgabe.objects.create(
|
||||
dokument=self.dokument,
|
||||
nummer=1,
|
||||
order=1,
|
||||
thema=self.thema,
|
||||
titel="Test Vorgabe",
|
||||
gueltigkeit_von=date.today()
|
||||
)
|
||||
|
||||
def test_textabschnitt_creation(self):
|
||||
"""Test that Textabschnitt can be instantiated via concrete model"""
|
||||
from dokumente.models import VorgabeLangtext
|
||||
|
||||
abschnitt = VorgabeLangtext.objects.create(
|
||||
abschnitt=self.vorgabe,
|
||||
abschnitttyp=self.typ_text,
|
||||
inhalt="Test content",
|
||||
order=1
|
||||
)
|
||||
self.assertEqual(abschnitt.abschnitttyp, self.typ_text)
|
||||
self.assertEqual(abschnitt.inhalt, "Test content")
|
||||
self.assertEqual(abschnitt.order, 1)
|
||||
|
||||
def test_textabschnitt_default_order(self):
|
||||
"""Test that order defaults to 0"""
|
||||
from dokumente.models import VorgabeLangtext
|
||||
|
||||
abschnitt = VorgabeLangtext.objects.create(
|
||||
abschnitt=self.vorgabe,
|
||||
abschnitttyp=self.typ_text,
|
||||
inhalt="Test"
|
||||
)
|
||||
self.assertEqual(abschnitt.order, 0)
|
||||
|
||||
def test_textabschnitt_blank_fields(self):
|
||||
"""Test that abschnitttyp and inhalt can be blank/null"""
|
||||
from dokumente.models import VorgabeLangtext
|
||||
|
||||
abschnitt = VorgabeLangtext.objects.create(
|
||||
abschnitt=self.vorgabe
|
||||
)
|
||||
self.assertIsNone(abschnitt.abschnitttyp)
|
||||
self.assertIsNone(abschnitt.inhalt)
|
||||
|
||||
def test_textabschnitt_ordering(self):
|
||||
"""Test that Textabschnitte can be ordered"""
|
||||
from dokumente.models import VorgabeLangtext
|
||||
|
||||
abschnitt1 = VorgabeLangtext.objects.create(
|
||||
abschnitt=self.vorgabe,
|
||||
abschnitttyp=self.typ_text,
|
||||
inhalt="First",
|
||||
order=2
|
||||
)
|
||||
abschnitt2 = VorgabeLangtext.objects.create(
|
||||
abschnitt=self.vorgabe,
|
||||
abschnitttyp=self.typ_text,
|
||||
inhalt="Second",
|
||||
order=1
|
||||
)
|
||||
abschnitt3 = VorgabeLangtext.objects.create(
|
||||
abschnitt=self.vorgabe,
|
||||
abschnitttyp=self.typ_text,
|
||||
inhalt="Third",
|
||||
order=3
|
||||
)
|
||||
|
||||
ordered = VorgabeLangtext.objects.filter(abschnitt=self.vorgabe).order_by('order')
|
||||
self.assertEqual(list(ordered), [abschnitt2, abschnitt1, abschnitt3])
|
||||
|
||||
def test_textabschnitt_foreign_key_protection(self):
|
||||
"""Test that AbschnittTyp is protected from deletion"""
|
||||
from dokumente.models import VorgabeLangtext
|
||||
from django.db.models import ProtectedError
|
||||
|
||||
abschnitt = VorgabeLangtext.objects.create(
|
||||
abschnitt=self.vorgabe,
|
||||
abschnitttyp=self.typ_text,
|
||||
inhalt="Test"
|
||||
)
|
||||
|
||||
# Try to delete the AbschnittTyp
|
||||
with self.assertRaises(ProtectedError):
|
||||
self.typ_text.delete()
|
||||
|
||||
|
||||
class RenderTextabschnitteTest(TestCase):
|
||||
"""Test cases for render_textabschnitte function"""
|
||||
|
||||
def setUp(self):
|
||||
"""Set up test data"""
|
||||
from dokumente.models import Dokumententyp, Dokument, Vorgabe, VorgabeLangtext, Thema
|
||||
from datetime import date
|
||||
|
||||
self.typ_text = AbschnittTyp.objects.create(abschnitttyp="text")
|
||||
self.typ_unordered = AbschnittTyp.objects.create(abschnitttyp="liste ungeordnet")
|
||||
self.typ_ordered = AbschnittTyp.objects.create(abschnitttyp="liste geordnet")
|
||||
self.typ_table = AbschnittTyp.objects.create(abschnitttyp="tabelle")
|
||||
self.typ_code = AbschnittTyp.objects.create(abschnitttyp="code")
|
||||
self.typ_diagram = AbschnittTyp.objects.create(abschnitttyp="diagramm")
|
||||
|
||||
# Create required dokumente objects
|
||||
self.dokumententyp = Dokumententyp.objects.create(
|
||||
name="Test Type", verantwortliche_ve="TEST"
|
||||
)
|
||||
self.dokument = Dokument.objects.create(
|
||||
nummer="TEST-001",
|
||||
name="Test Doc",
|
||||
dokumententyp=self.dokumententyp,
|
||||
aktiv=True
|
||||
)
|
||||
self.thema = Thema.objects.create(name="Test Thema")
|
||||
self.vorgabe = Vorgabe.objects.create(
|
||||
dokument=self.dokument,
|
||||
nummer=1,
|
||||
order=1,
|
||||
thema=self.thema,
|
||||
titel="Test Vorgabe",
|
||||
gueltigkeit_von=date.today()
|
||||
)
|
||||
|
||||
def test_render_empty_queryset(self):
|
||||
"""Test rendering an empty queryset"""
|
||||
from dokumente.models import VorgabeLangtext
|
||||
|
||||
result = render_textabschnitte(VorgabeLangtext.objects.none())
|
||||
self.assertEqual(result, [])
|
||||
|
||||
def test_render_text_markdown(self):
|
||||
"""Test rendering plain text with markdown"""
|
||||
from dokumente.models import VorgabeLangtext
|
||||
|
||||
abschnitt = VorgabeLangtext.objects.create(
|
||||
abschnitt=self.vorgabe,
|
||||
abschnitttyp=self.typ_text,
|
||||
inhalt="# Heading\n\nThis is **bold** text.",
|
||||
order=1
|
||||
)
|
||||
|
||||
result = render_textabschnitte(VorgabeLangtext.objects.filter(abschnitt=self.vorgabe))
|
||||
self.assertEqual(len(result), 1)
|
||||
typ, html = result[0]
|
||||
self.assertEqual(typ, "text")
|
||||
self.assertIn("<h1>Heading</h1>", html)
|
||||
self.assertIn("<strong>bold</strong>", html)
|
||||
|
||||
def test_render_text_with_footnotes(self):
|
||||
"""Test rendering text with footnotes"""
|
||||
from dokumente.models import VorgabeLangtext
|
||||
|
||||
abschnitt = VorgabeLangtext.objects.create(
|
||||
abschnitt=self.vorgabe,
|
||||
abschnitttyp=self.typ_text,
|
||||
inhalt="This is text[^1].\n\n[^1]: This is a footnote.",
|
||||
order=1
|
||||
)
|
||||
|
||||
result = render_textabschnitte(VorgabeLangtext.objects.filter(abschnitt=self.vorgabe))
|
||||
typ, html = result[0]
|
||||
self.assertIn("footnote", html.lower())
|
||||
|
||||
def test_render_unordered_list(self):
|
||||
"""Test rendering unordered list"""
|
||||
from dokumente.models import VorgabeLangtext
|
||||
|
||||
abschnitt = VorgabeLangtext.objects.create(
|
||||
abschnitt=self.vorgabe,
|
||||
abschnitttyp=self.typ_unordered,
|
||||
inhalt="Item 1\nItem 2\nItem 3",
|
||||
order=1
|
||||
)
|
||||
|
||||
result = render_textabschnitte(VorgabeLangtext.objects.filter(abschnitt=self.vorgabe))
|
||||
typ, html = result[0]
|
||||
self.assertEqual(typ, "liste ungeordnet")
|
||||
self.assertIn("<ul>", html)
|
||||
self.assertIn("<li>Item 1</li>", html)
|
||||
self.assertIn("<li>Item 2</li>", html)
|
||||
self.assertIn("<li>Item 3</li>", html)
|
||||
|
||||
def test_render_ordered_list(self):
|
||||
"""Test rendering ordered list"""
|
||||
from dokumente.models import VorgabeLangtext
|
||||
|
||||
abschnitt = VorgabeLangtext.objects.create(
|
||||
abschnitt=self.vorgabe,
|
||||
abschnitttyp=self.typ_ordered,
|
||||
inhalt="First item\nSecond item\nThird item",
|
||||
order=1
|
||||
)
|
||||
|
||||
result = render_textabschnitte(VorgabeLangtext.objects.filter(abschnitt=self.vorgabe))
|
||||
typ, html = result[0]
|
||||
self.assertEqual(typ, "liste geordnet")
|
||||
self.assertIn("<ol>", html)
|
||||
self.assertIn("<li>First item</li>", html)
|
||||
self.assertIn("<li>Second item</li>", html)
|
||||
self.assertIn("<li>Third item</li>", html)
|
||||
|
||||
def test_render_table(self):
|
||||
"""Test rendering table"""
|
||||
from dokumente.models import VorgabeLangtext
|
||||
|
||||
table_content = """| Header 1 | Header 2 |
|
||||
|----------|----------|
|
||||
| Cell 1 | Cell 2 |
|
||||
| Cell 3 | Cell 4 |"""
|
||||
|
||||
abschnitt = VorgabeLangtext.objects.create(
|
||||
abschnitt=self.vorgabe,
|
||||
abschnitttyp=self.typ_table,
|
||||
inhalt=table_content,
|
||||
order=1
|
||||
)
|
||||
|
||||
result = render_textabschnitte(VorgabeLangtext.objects.filter(abschnitt=self.vorgabe))
|
||||
typ, html = result[0]
|
||||
self.assertEqual(typ, "tabelle")
|
||||
self.assertIn('<table class="table table-bordered table-hover">', html)
|
||||
self.assertIn("<thead>", html)
|
||||
self.assertIn("<th>Header 1</th>", html)
|
||||
self.assertIn("<th>Header 2</th>", html)
|
||||
self.assertIn("<tbody>", html)
|
||||
self.assertIn("<td>Cell 1</td>", html)
|
||||
self.assertIn("<td>Cell 2</td>", html)
|
||||
|
||||
def test_render_code_block(self):
|
||||
"""Test rendering code block"""
|
||||
from dokumente.models import VorgabeLangtext
|
||||
|
||||
code_content = "def hello():\n print('Hello, World!')"
|
||||
|
||||
abschnitt = VorgabeLangtext.objects.create(
|
||||
abschnitt=self.vorgabe,
|
||||
abschnitttyp=self.typ_code,
|
||||
inhalt=code_content,
|
||||
order=1
|
||||
)
|
||||
|
||||
result = render_textabschnitte(VorgabeLangtext.objects.filter(abschnitt=self.vorgabe))
|
||||
typ, html = result[0]
|
||||
self.assertEqual(typ, "code")
|
||||
self.assertIn("<pre><code>", html)
|
||||
self.assertIn("</code></pre>", html)
|
||||
self.assertIn("hello", html)
|
||||
|
||||
@patch('abschnitte.utils.get_cached_diagram')
|
||||
def test_render_diagram_success(self, mock_get_cached):
|
||||
"""Test rendering diagram with successful caching"""
|
||||
from dokumente.models import VorgabeLangtext
|
||||
|
||||
mock_get_cached.return_value = "diagram_cache/plantuml/abc123.svg"
|
||||
|
||||
diagram_content = """plantuml
|
||||
@startuml
|
||||
Alice -> Bob: Hello
|
||||
@enduml"""
|
||||
|
||||
abschnitt = VorgabeLangtext.objects.create(
|
||||
abschnitt=self.vorgabe,
|
||||
abschnitttyp=self.typ_diagram,
|
||||
inhalt=diagram_content,
|
||||
order=1
|
||||
)
|
||||
|
||||
result = render_textabschnitte(VorgabeLangtext.objects.filter(abschnitt=self.vorgabe))
|
||||
typ, html = result[0]
|
||||
self.assertEqual(typ, "diagramm")
|
||||
self.assertIn('<img', html)
|
||||
self.assertIn('width="100%"', html)
|
||||
self.assertIn('diagram_cache/plantuml/abc123.svg', html)
|
||||
|
||||
# Verify get_cached_diagram was called correctly
|
||||
mock_get_cached.assert_called_once()
|
||||
args = mock_get_cached.call_args[0]
|
||||
self.assertEqual(args[0], "plantuml")
|
||||
self.assertIn("Alice -> Bob", args[1])
|
||||
|
||||
@patch('abschnitte.utils.get_cached_diagram')
|
||||
def test_render_diagram_with_options(self, mock_get_cached):
|
||||
"""Test rendering diagram with custom options"""
|
||||
from dokumente.models import VorgabeLangtext
|
||||
|
||||
mock_get_cached.return_value = "diagram_cache/mermaid/xyz789.svg"
|
||||
|
||||
diagram_content = """mermaid
|
||||
option: width="50%" height="300px"
|
||||
graph TD
|
||||
A-->B"""
|
||||
|
||||
abschnitt = VorgabeLangtext.objects.create(
|
||||
abschnitt=self.vorgabe,
|
||||
abschnitttyp=self.typ_diagram,
|
||||
inhalt=diagram_content,
|
||||
order=1
|
||||
)
|
||||
|
||||
result = render_textabschnitte(VorgabeLangtext.objects.filter(abschnitt=self.vorgabe))
|
||||
typ, html = result[0]
|
||||
self.assertIn('width="50%"', html)
|
||||
self.assertIn('height="300px"', html)
|
||||
|
||||
@patch('abschnitte.utils.get_cached_diagram')
|
||||
def test_render_diagram_error(self, mock_get_cached):
|
||||
"""Test rendering diagram when caching fails"""
|
||||
from dokumente.models import VorgabeLangtext
|
||||
|
||||
mock_get_cached.side_effect = Exception("Connection error")
|
||||
|
||||
diagram_content = """plantuml
|
||||
@startuml
|
||||
A -> B
|
||||
@enduml"""
|
||||
|
||||
abschnitt = VorgabeLangtext.objects.create(
|
||||
abschnitt=self.vorgabe,
|
||||
abschnitttyp=self.typ_diagram,
|
||||
inhalt=diagram_content,
|
||||
order=1
|
||||
)
|
||||
|
||||
result = render_textabschnitte(VorgabeLangtext.objects.filter(abschnitt=self.vorgabe))
|
||||
typ, html = result[0]
|
||||
self.assertIn("Error generating diagram", html)
|
||||
self.assertIn("Connection error", html)
|
||||
self.assertIn('class="text-danger"', html)
|
||||
|
||||
def test_render_multiple_abschnitte(self):
|
||||
"""Test rendering multiple Textabschnitte in order"""
|
||||
from dokumente.models import VorgabeLangtext
|
||||
|
||||
abschnitt1 = VorgabeLangtext.objects.create(
|
||||
abschnitt=self.vorgabe,
|
||||
abschnitttyp=self.typ_text,
|
||||
inhalt="First section",
|
||||
order=1
|
||||
)
|
||||
abschnitt2 = VorgabeLangtext.objects.create(
|
||||
abschnitt=self.vorgabe,
|
||||
abschnitttyp=self.typ_unordered,
|
||||
inhalt="Item 1\nItem 2",
|
||||
order=2
|
||||
)
|
||||
abschnitt3 = VorgabeLangtext.objects.create(
|
||||
abschnitt=self.vorgabe,
|
||||
abschnitttyp=self.typ_code,
|
||||
inhalt="print('hello')",
|
||||
order=3
|
||||
)
|
||||
|
||||
result = render_textabschnitte(
|
||||
VorgabeLangtext.objects.filter(abschnitt=self.vorgabe).order_by('order')
|
||||
)
|
||||
|
||||
self.assertEqual(len(result), 3)
|
||||
self.assertEqual(result[0][0], "text")
|
||||
self.assertEqual(result[1][0], "liste ungeordnet")
|
||||
self.assertEqual(result[2][0], "code")
|
||||
|
||||
def test_render_abschnitt_without_type(self):
|
||||
"""Test rendering Textabschnitt without AbschnittTyp"""
|
||||
from dokumente.models import VorgabeLangtext
|
||||
|
||||
abschnitt = VorgabeLangtext.objects.create(
|
||||
abschnitt=self.vorgabe,
|
||||
abschnitttyp=None,
|
||||
inhalt="Content without type",
|
||||
order=1
|
||||
)
|
||||
|
||||
result = render_textabschnitte(VorgabeLangtext.objects.filter(abschnitt=self.vorgabe))
|
||||
typ, html = result[0]
|
||||
self.assertEqual(typ, '')
|
||||
self.assertIn("Content without type", html)
|
||||
|
||||
def test_render_abschnitt_with_empty_content(self):
|
||||
"""Test rendering Textabschnitt with empty content"""
|
||||
from dokumente.models import VorgabeLangtext
|
||||
|
||||
abschnitt = VorgabeLangtext.objects.create(
|
||||
abschnitt=self.vorgabe,
|
||||
abschnitttyp=self.typ_text,
|
||||
inhalt=None,
|
||||
order=1
|
||||
)
|
||||
|
||||
result = render_textabschnitte(VorgabeLangtext.objects.filter(abschnitt=self.vorgabe))
|
||||
self.assertEqual(len(result), 1)
|
||||
typ, html = result[0]
|
||||
self.assertEqual(typ, "text")
|
||||
|
||||
|
||||
class MdTableToHtmlTest(TestCase):
|
||||
"""Test cases for md_table_to_html function"""
|
||||
|
||||
def test_simple_table(self):
|
||||
"""Test converting a simple markdown table to HTML"""
|
||||
md = """| Name | Age |
|
||||
|------|-----|
|
||||
| John | 30 |
|
||||
| Jane | 25 |"""
|
||||
|
||||
html = md_table_to_html(md)
|
||||
|
||||
self.assertIn('<table class="table table-bordered table-hover">', html)
|
||||
self.assertIn("<thead>", html)
|
||||
self.assertIn("<th>Name</th>", html)
|
||||
self.assertIn("<th>Age</th>", html)
|
||||
self.assertIn("<tbody>", html)
|
||||
self.assertIn("<td>John</td>", html)
|
||||
self.assertIn("<td>30</td>", html)
|
||||
self.assertIn("<td>Jane</td>", html)
|
||||
self.assertIn("<td>25</td>", html)
|
||||
|
||||
def test_table_with_multiple_rows(self):
|
||||
"""Test table with multiple rows"""
|
||||
md = """| A | B | C |
|
||||
|---|---|---|
|
||||
| 1 | 2 | 3 |
|
||||
| 4 | 5 | 6 |
|
||||
| 7 | 8 | 9 |"""
|
||||
|
||||
html = md_table_to_html(md)
|
||||
|
||||
self.assertEqual(html.count("<tr>"), 4) # 1 header + 3 body rows
|
||||
self.assertEqual(html.count("<td>"), 9) # 3x3 cells
|
||||
self.assertEqual(html.count("<th>"), 3) # 3 headers
|
||||
|
||||
def test_table_with_spaces(self):
|
||||
"""Test table with extra spaces"""
|
||||
md = """ | Header 1 | Header 2 |
|
||||
| --------- | ---------- |
|
||||
| Value 1 | Value 2 | """
|
||||
|
||||
html = md_table_to_html(md)
|
||||
|
||||
self.assertIn("<th>Header 1</th>", html)
|
||||
self.assertIn("<th>Header 2</th>", html)
|
||||
self.assertIn("<td>Value 1</td>", html)
|
||||
self.assertIn("<td>Value 2</td>", html)
|
||||
|
||||
def test_table_with_empty_cells(self):
|
||||
"""Test table with empty cells"""
|
||||
md = """| Col1 | Col2 | Col3 |
|
||||
|------|------|------|
|
||||
| A | | C |
|
||||
| | B | |"""
|
||||
|
||||
html = md_table_to_html(md)
|
||||
|
||||
self.assertIn("<td>A</td>", html)
|
||||
self.assertIn("<td></td>", html)
|
||||
self.assertIn("<td>C</td>", html)
|
||||
self.assertIn("<td>B</td>", html)
|
||||
|
||||
def test_table_insufficient_lines(self):
|
||||
"""Test that ValueError is raised for insufficient lines"""
|
||||
md = """| Header |"""
|
||||
|
||||
with self.assertRaises(ValueError) as context:
|
||||
md_table_to_html(md)
|
||||
|
||||
self.assertIn("at least header + separator", str(context.exception))
|
||||
|
||||
def test_table_empty_string(self):
|
||||
"""Test that ValueError is raised for empty string"""
|
||||
with self.assertRaises(ValueError):
|
||||
md_table_to_html("")
|
||||
|
||||
def test_table_only_whitespace(self):
|
||||
"""Test that ValueError is raised for only whitespace"""
|
||||
with self.assertRaises(ValueError):
|
||||
md_table_to_html(" \n \n ")
|
||||
|
||||
|
||||
class DiagramCacheTest(TestCase):
|
||||
"""Test cases for diagram caching functionality"""
|
||||
|
||||
def setUp(self):
|
||||
"""Set up test environment"""
|
||||
# Create a temporary directory for testing
|
||||
self.test_media_root = tempfile.mkdtemp()
|
||||
self.original_media_root = settings.MEDIA_ROOT
|
||||
settings.MEDIA_ROOT = self.test_media_root
|
||||
|
||||
def tearDown(self):
|
||||
"""Clean up test environment"""
|
||||
# Restore original settings
|
||||
settings.MEDIA_ROOT = self.original_media_root
|
||||
# Remove test directory
|
||||
if os.path.exists(self.test_media_root):
|
||||
shutil.rmtree(self.test_media_root)
|
||||
|
||||
def test_compute_hash(self):
|
||||
"""Test that compute_hash generates consistent SHA256 hashes"""
|
||||
content1 = "test content"
|
||||
content2 = "test content"
|
||||
content3 = "different content"
|
||||
|
||||
hash1 = compute_hash(content1)
|
||||
hash2 = compute_hash(content2)
|
||||
hash3 = compute_hash(content3)
|
||||
|
||||
# Same content should produce same hash
|
||||
self.assertEqual(hash1, hash2)
|
||||
# Different content should produce different hash
|
||||
self.assertNotEqual(hash1, hash3)
|
||||
# Hash should be 64 characters (SHA256 hex)
|
||||
self.assertEqual(len(hash1), 64)
|
||||
|
||||
def test_get_cache_path(self):
|
||||
"""Test that get_cache_path generates correct paths"""
|
||||
diagram_type = "plantuml"
|
||||
content_hash = "abc123"
|
||||
|
||||
path = get_cache_path(diagram_type, content_hash)
|
||||
|
||||
self.assertIn("diagram_cache", path)
|
||||
self.assertIn("plantuml", path)
|
||||
self.assertIn("abc123.svg", path)
|
||||
|
||||
@patch('diagramm_proxy.diagram_cache.requests.post')
|
||||
@patch('diagramm_proxy.diagram_cache.default_storage')
|
||||
def test_get_cached_diagram_miss(self, mock_storage, mock_post):
|
||||
"""Test diagram generation on cache miss"""
|
||||
# Setup mocks
|
||||
mock_storage.exists.return_value = False
|
||||
mock_storage.path.return_value = os.path.join(
|
||||
self.test_media_root, 'diagram_cache/plantuml/test.svg'
|
||||
)
|
||||
mock_response = Mock()
|
||||
mock_response.content = b'<svg>test</svg>'
|
||||
mock_post.return_value = mock_response
|
||||
|
||||
diagram_content = "@startuml\nA -> B\n@enduml"
|
||||
|
||||
# Call function
|
||||
result = get_cached_diagram("plantuml", diagram_content)
|
||||
|
||||
# Verify POST request was made
|
||||
mock_post.assert_called_once()
|
||||
call_args = mock_post.call_args
|
||||
# Check URL in positional args (first argument)
|
||||
self.assertIn("plantuml/svg", call_args[0][0])
|
||||
|
||||
# Verify storage.save was called
|
||||
mock_storage.save.assert_called_once()
|
||||
|
||||
@patch('diagramm_proxy.diagram_cache.default_storage')
|
||||
def test_get_cached_diagram_hit(self, mock_storage):
|
||||
"""Test diagram retrieval on cache hit"""
|
||||
# Setup mock - diagram exists in cache
|
||||
mock_storage.exists.return_value = True
|
||||
|
||||
diagram_content = "@startuml\nA -> B\n@enduml"
|
||||
|
||||
# Call function
|
||||
result = get_cached_diagram("plantuml", diagram_content)
|
||||
|
||||
# Verify no save was attempted (cache hit)
|
||||
mock_storage.save.assert_not_called()
|
||||
|
||||
# Verify result contains expected path elements
|
||||
self.assertIn("diagram_cache", result)
|
||||
self.assertIn("plantuml", result)
|
||||
self.assertIn(".svg", result)
|
||||
|
||||
@patch('diagramm_proxy.diagram_cache.requests.post')
|
||||
@patch('diagramm_proxy.diagram_cache.default_storage')
|
||||
def test_get_cached_diagram_request_error(self, mock_storage, mock_post):
|
||||
"""Test that request errors are properly raised"""
|
||||
import requests
|
||||
|
||||
mock_storage.exists.return_value = False
|
||||
mock_storage.path.return_value = os.path.join(
|
||||
self.test_media_root, 'diagram_cache/plantuml/test.svg'
|
||||
)
|
||||
mock_post.side_effect = requests.RequestException("Connection error")
|
||||
|
||||
with self.assertRaises(requests.RequestException):
|
||||
get_cached_diagram("plantuml", "@startuml\nA -> B\n@enduml")
|
||||
|
||||
@patch('diagramm_proxy.diagram_cache.default_storage')
|
||||
def test_clear_cache_specific_type(self, mock_storage):
|
||||
"""Test clearing cache for specific diagram type"""
|
||||
# Create real test cache structure for this test
|
||||
cache_dir = os.path.join(self.test_media_root, 'diagram_cache', 'plantuml')
|
||||
os.makedirs(cache_dir, exist_ok=True)
|
||||
|
||||
# Create test files
|
||||
test_file1 = os.path.join(cache_dir, 'test1.svg')
|
||||
test_file2 = os.path.join(cache_dir, 'test2.svg')
|
||||
open(test_file1, 'w').close()
|
||||
open(test_file2, 'w').close()
|
||||
|
||||
# Mock storage methods
|
||||
mock_storage.exists.return_value = True
|
||||
mock_storage.path.return_value = cache_dir
|
||||
|
||||
# Clear cache
|
||||
clear_cache('plantuml')
|
||||
|
||||
# Verify files are deleted
|
||||
self.assertFalse(os.path.exists(test_file1))
|
||||
self.assertFalse(os.path.exists(test_file2))
|
||||
|
||||
@patch('diagramm_proxy.diagram_cache.default_storage')
|
||||
def test_clear_cache_all_types(self, mock_storage):
|
||||
"""Test clearing cache for all diagram types"""
|
||||
# Create real test cache structure with multiple types
|
||||
cache_root = os.path.join(self.test_media_root, 'diagram_cache')
|
||||
for diagram_type in ['plantuml', 'mermaid', 'graphviz']:
|
||||
cache_dir = os.path.join(cache_root, diagram_type)
|
||||
os.makedirs(cache_dir, exist_ok=True)
|
||||
test_file = os.path.join(cache_dir, 'test.svg')
|
||||
open(test_file, 'w').close()
|
||||
|
||||
# Mock storage methods
|
||||
mock_storage.exists.return_value = True
|
||||
mock_storage.path.return_value = cache_root
|
||||
|
||||
# Clear all cache
|
||||
clear_cache()
|
||||
|
||||
# Verify all files are deleted
|
||||
for diagram_type in ['plantuml', 'mermaid', 'graphviz']:
|
||||
test_file = os.path.join(cache_root, diagram_type, 'test.svg')
|
||||
self.assertFalse(os.path.exists(test_file))
|
||||
|
||||
|
||||
class ClearDiagramCacheCommandTest(TestCase):
|
||||
"""Test cases for clear_diagram_cache management command"""
|
||||
|
||||
def setUp(self):
|
||||
"""Set up test environment"""
|
||||
self.test_media_root = tempfile.mkdtemp()
|
||||
self.original_media_root = settings.MEDIA_ROOT
|
||||
settings.MEDIA_ROOT = self.test_media_root
|
||||
|
||||
def tearDown(self):
|
||||
"""Clean up test environment"""
|
||||
settings.MEDIA_ROOT = self.original_media_root
|
||||
if os.path.exists(self.test_media_root):
|
||||
shutil.rmtree(self.test_media_root)
|
||||
|
||||
def test_command_without_type(self):
|
||||
"""Test running command without specifying type"""
|
||||
# Create test cache
|
||||
cache_dir = os.path.join(self.test_media_root, 'diagram_cache', 'plantuml')
|
||||
os.makedirs(cache_dir, exist_ok=True)
|
||||
test_file = os.path.join(cache_dir, 'test.svg')
|
||||
open(test_file, 'w').close()
|
||||
|
||||
# Run command
|
||||
out = StringIO()
|
||||
call_command('clear_diagram_cache', stdout=out)
|
||||
|
||||
# Check output
|
||||
self.assertIn('Clearing all diagram caches', out.getvalue())
|
||||
self.assertIn('Cache cleared successfully', out.getvalue())
|
||||
|
||||
def test_command_with_type(self):
|
||||
"""Test running command with specific diagram type"""
|
||||
# Create test cache
|
||||
cache_dir = os.path.join(self.test_media_root, 'diagram_cache', 'mermaid')
|
||||
os.makedirs(cache_dir, exist_ok=True)
|
||||
test_file = os.path.join(cache_dir, 'test.svg')
|
||||
open(test_file, 'w').close()
|
||||
|
||||
# Run command
|
||||
out = StringIO()
|
||||
call_command('clear_diagram_cache', type='mermaid', stdout=out)
|
||||
|
||||
# Check output
|
||||
self.assertIn('Clearing cache for mermaid', out.getvalue())
|
||||
self.assertIn('Cache cleared successfully', out.getvalue())
|
||||
|
||||
|
||||
class IntegrationTest(TestCase):
|
||||
"""Integration tests with actual dokumente models"""
|
||||
|
||||
def setUp(self):
|
||||
"""Set up test data using dokumente models"""
|
||||
from dokumente.models import (
|
||||
Dokumententyp, Dokument, Vorgabe, VorgabeLangtext, Thema
|
||||
)
|
||||
from datetime import date
|
||||
|
||||
# Create required objects
|
||||
self.dokumententyp = Dokumententyp.objects.create(
|
||||
name="Test Policy",
|
||||
verantwortliche_ve="TEST"
|
||||
)
|
||||
|
||||
self.dokument = Dokument.objects.create(
|
||||
nummer="TEST-001",
|
||||
name="Test Document",
|
||||
dokumententyp=self.dokumententyp,
|
||||
aktiv=True
|
||||
)
|
||||
|
||||
self.thema = Thema.objects.create(name="Test Thema")
|
||||
self.vorgabe = Vorgabe.objects.create(
|
||||
dokument=self.dokument,
|
||||
nummer=1,
|
||||
order=1,
|
||||
thema=self.thema,
|
||||
titel="Test Vorgabe",
|
||||
gueltigkeit_von=date.today()
|
||||
)
|
||||
|
||||
# Create AbschnittTypen
|
||||
self.typ_text = AbschnittTyp.objects.create(abschnitttyp="text")
|
||||
|
||||
# Create VorgabeLangtext (which inherits from Textabschnitt)
|
||||
self.langtext = VorgabeLangtext.objects.create(
|
||||
abschnitt=self.vorgabe,
|
||||
abschnitttyp=self.typ_text,
|
||||
inhalt="# Test\n\nThis is a **test** vorgabe.",
|
||||
order=1
|
||||
)
|
||||
|
||||
def test_render_vorgabe_langtext(self):
|
||||
"""Test rendering VorgabeLangtext through render_textabschnitte"""
|
||||
from dokumente.models import VorgabeLangtext
|
||||
|
||||
result = render_textabschnitte(
|
||||
VorgabeLangtext.objects.filter(abschnitt=self.vorgabe).order_by('order')
|
||||
)
|
||||
|
||||
self.assertEqual(len(result), 1)
|
||||
typ, html = result[0]
|
||||
self.assertEqual(typ, "text")
|
||||
self.assertIn("<h1>Test</h1>", html)
|
||||
self.assertIn("<strong>test</strong>", html)
|
||||
self.assertIn("vorgabe", html)
|
||||
|
||||
def test_textabschnitt_inheritance(self):
|
||||
"""Test that VorgabeLangtext properly inherits Textabschnitt fields"""
|
||||
self.assertEqual(self.langtext.abschnitttyp, self.typ_text)
|
||||
self.assertIn("test", self.langtext.inhalt)
|
||||
self.assertEqual(self.langtext.order, 1)
|
||||
|
||||
@@ -3,6 +3,10 @@ import base64
|
||||
import zlib
|
||||
import re
|
||||
from textwrap import dedent
|
||||
from django.conf import settings
|
||||
|
||||
# Import the caching function
|
||||
from diagramm_proxy.diagram_cache import get_cached_diagram
|
||||
|
||||
DIAGRAMMSERVER="/diagramm"
|
||||
|
||||
@@ -25,15 +29,23 @@ def render_textabschnitte(queryset):
|
||||
elif typ == "tabelle":
|
||||
html = md_table_to_html(inhalt)
|
||||
elif typ == "diagramm":
|
||||
temp=inhalt.splitlines()
|
||||
diagramtype=temp.pop(0)
|
||||
diagramoptions='width="100%"'
|
||||
if temp[0][0:6].lower() == "option":
|
||||
diagramoptions=temp.pop(0).split(":",1)[1]
|
||||
rest="\n".join(temp)
|
||||
html = '<p><img '+diagramoptions+' src="'+DIAGRAMMSERVER+"/"+diagramtype+"/svg/"
|
||||
html += base64.urlsafe_b64encode(zlib.compress(rest.encode("utf-8"),9)).decode()
|
||||
html += '"></p>'
|
||||
temp = inhalt.splitlines()
|
||||
diagramtype = temp.pop(0)
|
||||
diagramoptions = 'width="100%"'
|
||||
if temp and temp[0][0:6].lower() == "option":
|
||||
diagramoptions = temp.pop(0).split(":", 1)[1]
|
||||
rest = "\n".join(temp)
|
||||
|
||||
# Use caching instead of URL encoding
|
||||
try:
|
||||
cache_path = get_cached_diagram(diagramtype, rest)
|
||||
# Generate URL to serve from media/static
|
||||
diagram_url = settings.MEDIA_URL + cache_path
|
||||
html = f'<p><img {diagramoptions} src="{diagram_url}"></p>'
|
||||
except Exception as e:
|
||||
# Fallback to error message
|
||||
html = f'<p class="text-danger">Error generating diagram: {str(e)}</p>'
|
||||
|
||||
elif typ == "code":
|
||||
html = "<pre><code>"
|
||||
html += markdown(inhalt, extensions=['tables', 'attr_list'])
|
||||
|
||||
102
admin/css/vorgabe_border.css
Normal file
102
admin/css/vorgabe_border.css
Normal file
@@ -0,0 +1,102 @@
|
||||
/* Better visual separation for Vorgaben inlines */
|
||||
.inline-group[data-inline-model="vorgabe"] {
|
||||
border: 2px solid #ddd;
|
||||
border-radius: 8px;
|
||||
margin-bottom: 20px;
|
||||
padding: 15px;
|
||||
background-color: #f9f9f9;
|
||||
box-shadow: 0 2px 4px rgba(0,0,0,0.1);
|
||||
}
|
||||
|
||||
.inline-group[data-inline-model="vorgabe"] .inline-related {
|
||||
border: 1px solid #ccc;
|
||||
border-radius: 6px;
|
||||
margin-bottom: 10px;
|
||||
background-color: white;
|
||||
padding: 10px;
|
||||
}
|
||||
|
||||
.inline-group[data-inline-model="vorgabe"] h3 {
|
||||
background-color: #007cba;
|
||||
color: white;
|
||||
padding: 8px 12px;
|
||||
margin: -15px -15px 10px -15px;
|
||||
border-radius: 6px 6px 0 0;
|
||||
font-weight: bold;
|
||||
}
|
||||
|
||||
.inline-group[data-inline-model="vorgabe"] .collapse .inline-related {
|
||||
border-left: 3px solid #007cba;
|
||||
}
|
||||
|
||||
/* Better spacing for nested inlines */
|
||||
.inline-group[data-inline-model="vorgabe"] .inline-group {
|
||||
margin-top: 10px;
|
||||
}
|
||||
|
||||
.inline-group[data-inline-model="vorgabe"] .inline-group h3 {
|
||||
background-color: #f0f8ff;
|
||||
color: #333;
|
||||
padding: 6px 10px;
|
||||
margin: 0 0 8px 0;
|
||||
border-left: 3px solid #007cba;
|
||||
}
|
||||
|
||||
/* Highlight active/expanded vorgabe */
|
||||
.inline-group[data-inline-model="vorgabe"] .inline-related:not(.collapsed) {
|
||||
border-color: #007cba;
|
||||
box-shadow: 0 0 8px rgba(0,124,186,0.2);
|
||||
}
|
||||
|
||||
/* Highlight actively edited vorgabe */
|
||||
.inline-group[data-inline-model="vorgabe"] .inline-related.active-edit {
|
||||
border-color: #28a745;
|
||||
box-shadow: 0 0 12px rgba(40,167,69,0.3);
|
||||
background-color: #f8fff9;
|
||||
}
|
||||
|
||||
/* Toggle hint styling */
|
||||
.toggle-hint {
|
||||
font-size: 0.8em;
|
||||
color: #666;
|
||||
font-weight: normal;
|
||||
}
|
||||
|
||||
/* Better fieldset styling for vorgabe inlines */
|
||||
.inline-group[data-inline-model="vorgabe"] .fieldset {
|
||||
border: 1px solid #e0e0e0;
|
||||
border-radius: 4px;
|
||||
padding: 10px;
|
||||
margin-bottom: 10px;
|
||||
background-color: #fafafa;
|
||||
}
|
||||
|
||||
.inline-group[data-inline-model="vorgabe"] .fieldset h2 {
|
||||
background-color: #e3f2fd;
|
||||
color: #1565c0;
|
||||
padding: 5px 10px;
|
||||
margin: -10px -10px 10px -10px;
|
||||
border-radius: 4px 4px 0 0;
|
||||
font-size: 0.9em;
|
||||
font-weight: bold;
|
||||
}
|
||||
|
||||
/* Better form layout */
|
||||
.inline-group[data-inline-model="vorgabe"] .form-row {
|
||||
border-bottom: 1px solid #eee;
|
||||
padding: 8px 0;
|
||||
}
|
||||
|
||||
.inline-group[data-inline-model="vorgabe"] .form-row:last-child {
|
||||
border-bottom: none;
|
||||
}
|
||||
|
||||
/* Wide fields styling */
|
||||
.inline-group[data-inline-model="vorgabe"] .wide .form-row > div {
|
||||
width: 100%;
|
||||
}
|
||||
|
||||
.inline-group[data-inline-model="vorgabe"] .wide textarea {
|
||||
width: 100%;
|
||||
min-height: 80px;
|
||||
}
|
||||
25
admin/js/vorgabe_toggle.js
Normal file
25
admin/js/vorgabe_toggle.js
Normal file
@@ -0,0 +1,25 @@
|
||||
(function($) {
|
||||
'use strict';
|
||||
|
||||
$(document).ready(function() {
|
||||
// Add toggle buttons for each vorgabe inline
|
||||
$('.inline-group[data-inline-model="vorgabe"]').each(function() {
|
||||
var $group = $(this);
|
||||
var $headers = $group.find('h3');
|
||||
|
||||
$headers.css('cursor', 'pointer').append(' <span class="toggle-hint">(klicken zum umschalten)</span>');
|
||||
|
||||
$headers.on('click', function(e) {
|
||||
e.preventDefault();
|
||||
var $inline = $(this).closest('.inline-related');
|
||||
$inline.find('.collapse').toggleClass('collapsed');
|
||||
});
|
||||
});
|
||||
|
||||
// Highlight active vorgabe when editing
|
||||
$('.inline-group[data-inline-model="vorgabe"] .inline-related').on('click', function() {
|
||||
$('.inline-group[data-inline-model="vorgabe"] .inline-related').removeClass('active-edit');
|
||||
$(this).addClass('active-edit');
|
||||
});
|
||||
});
|
||||
})(django.jQuery);
|
||||
@@ -18,14 +18,14 @@ spec:
|
||||
fsGroupChangePolicy: "OnRootMismatch"
|
||||
initContainers:
|
||||
- name: loader
|
||||
image: git.baumann.gr/adebaumann/vgui-data-loader:0.5
|
||||
image: git.baumann.gr/adebaumann/vui-data-loader:0.9
|
||||
command: [ "sh","-c","cp -n preload/preload.sqlite3 /data/db.sqlite3; chown -R 999:999 /data; ls -la /data; sleep 10; exit 0" ]
|
||||
volumeMounts:
|
||||
- name: data
|
||||
mountPath: /data
|
||||
containers:
|
||||
- name: web
|
||||
image: git.baumann.gr/adebaumann/vui:0.929
|
||||
image: git.baumann.gr/adebaumann/vui:0.942
|
||||
imagePullPolicy: Always
|
||||
ports:
|
||||
- containerPort: 8000
|
||||
|
||||
Binary file not shown.
BIN
data/db.sqlite3
BIN
data/db.sqlite3
Binary file not shown.
1
diagramm_proxy/__init__.py
Normal file
1
diagramm_proxy/__init__.py
Normal file
@@ -0,0 +1 @@
|
||||
# Diagram proxy module
|
||||
91
diagramm_proxy/diagram_cache.py
Normal file
91
diagramm_proxy/diagram_cache.py
Normal file
@@ -0,0 +1,91 @@
|
||||
import hashlib
|
||||
import os
|
||||
import requests
|
||||
from pathlib import Path
|
||||
from django.conf import settings
|
||||
from django.core.files.storage import default_storage
|
||||
from django.core.files.base import ContentFile
|
||||
import logging
|
||||
|
||||
logger = logging.getLogger(__name__)
|
||||
|
||||
# Configure cache directory
|
||||
CACHE_DIR = getattr(settings, 'DIAGRAM_CACHE_DIR', 'diagram_cache')
|
||||
KROKI_UPSTREAM = "http://svckroki:8000"
|
||||
|
||||
def get_cache_path(diagram_type, content_hash):
|
||||
"""Generate cache file path for a diagram."""
|
||||
return os.path.join(CACHE_DIR, diagram_type, f"{content_hash}.svg")
|
||||
|
||||
def compute_hash(content):
|
||||
"""Compute SHA256 hash of diagram content."""
|
||||
return hashlib.sha256(content.encode('utf-8')).hexdigest()
|
||||
|
||||
def get_cached_diagram(diagram_type, diagram_content):
|
||||
"""
|
||||
Retrieve diagram from cache or generate it via POST.
|
||||
|
||||
Args:
|
||||
diagram_type: Type of diagram (e.g., 'plantuml', 'mermaid')
|
||||
diagram_content: Raw diagram content
|
||||
|
||||
Returns:
|
||||
Path to cached diagram file (relative to MEDIA_ROOT)
|
||||
"""
|
||||
content_hash = compute_hash(diagram_content)
|
||||
cache_path = get_cache_path(diagram_type, content_hash)
|
||||
|
||||
# Check if diagram exists in cache
|
||||
if default_storage.exists(cache_path):
|
||||
logger.debug(f"Cache hit for {diagram_type} diagram: {content_hash[:8]}")
|
||||
return cache_path
|
||||
|
||||
# Generate diagram via POST request
|
||||
logger.info(f"Cache miss for {diagram_type} diagram: {content_hash[:8]}, generating...")
|
||||
try:
|
||||
url = f"{KROKI_UPSTREAM}/{diagram_type}/svg"
|
||||
response = requests.post(
|
||||
url,
|
||||
data=diagram_content.encode('utf-8'),
|
||||
headers={'Content-Type': 'text/plain'},
|
||||
timeout=30
|
||||
)
|
||||
response.raise_for_status()
|
||||
|
||||
# Ensure cache directory exists
|
||||
cache_dir = os.path.dirname(default_storage.path(cache_path))
|
||||
os.makedirs(cache_dir, exist_ok=True)
|
||||
|
||||
# Save to cache
|
||||
default_storage.save(cache_path, ContentFile(response.content))
|
||||
logger.info(f"Diagram cached successfully: {cache_path}")
|
||||
|
||||
return cache_path
|
||||
|
||||
except requests.RequestException as e:
|
||||
logger.error(f"Error generating diagram: {e}")
|
||||
raise
|
||||
|
||||
def clear_cache(diagram_type=None):
|
||||
"""
|
||||
Clear cached diagrams.
|
||||
|
||||
Args:
|
||||
diagram_type: If specified, only clear diagrams of this type
|
||||
"""
|
||||
if diagram_type:
|
||||
cache_path = os.path.join(CACHE_DIR, diagram_type)
|
||||
else:
|
||||
cache_path = CACHE_DIR
|
||||
|
||||
if default_storage.exists(cache_path):
|
||||
full_path = default_storage.path(cache_path)
|
||||
# Walk through and delete files
|
||||
for root, dirs, files in os.walk(full_path):
|
||||
for file in files:
|
||||
file_path = os.path.join(root, file)
|
||||
try:
|
||||
os.remove(file_path)
|
||||
logger.info(f"Deleted cached diagram: {file_path}")
|
||||
except OSError as e:
|
||||
logger.error(f"Error deleting {file_path}: {e}")
|
||||
@@ -4,6 +4,7 @@ from nested_admin import NestedStackedInline, NestedModelAdmin, NestedTabularInl
|
||||
from django import forms
|
||||
from mptt.forms import TreeNodeMultipleChoiceField
|
||||
from mptt.admin import DraggableMPTTAdmin
|
||||
from adminsortable2.admin import SortableInlineAdminMixin, SortableAdminBase
|
||||
|
||||
# Register your models here.
|
||||
from .models import *
|
||||
@@ -20,21 +21,33 @@ from referenzen.models import Referenz
|
||||
# 'frage': forms.Textarea(attrs={'rows': 1, 'cols': 100}),
|
||||
# }
|
||||
|
||||
class ChecklistenfragenInline(NestedTabularInline):
|
||||
class ChecklistenfragenInline(NestedStackedInline):
|
||||
model=Checklistenfrage
|
||||
extra=0
|
||||
fk_name="vorgabe"
|
||||
# form=ChecklistenForm
|
||||
classes = ['collapse']
|
||||
verbose_name_plural = "Checklistenfragen"
|
||||
fieldsets = (
|
||||
(None, {
|
||||
'fields': ('frage',),
|
||||
'classes': ('wide',),
|
||||
}),
|
||||
)
|
||||
|
||||
|
||||
class VorgabeKurztextInline(NestedTabularInline):
|
||||
class VorgabeKurztextInline(NestedStackedInline):
|
||||
model=VorgabeKurztext
|
||||
extra=0
|
||||
sortable_field_name = "order"
|
||||
show_change_link=True
|
||||
classes = ['collapse']
|
||||
#inline=inhalt
|
||||
verbose_name_plural = "Kurztext-Abschnitte"
|
||||
fieldsets = (
|
||||
(None, {
|
||||
'fields': ('abschnitttyp', 'inhalt', 'order'),
|
||||
'classes': ('wide',),
|
||||
}),
|
||||
)
|
||||
|
||||
class VorgabeLangtextInline(NestedStackedInline):
|
||||
model=VorgabeLangtext
|
||||
@@ -42,42 +55,75 @@ class VorgabeLangtextInline(NestedStackedInline):
|
||||
sortable_field_name = "order"
|
||||
show_change_link=True
|
||||
classes = ['collapse']
|
||||
#inline=inhalt
|
||||
verbose_name_plural = "Langtext-Abschnitte"
|
||||
fieldsets = (
|
||||
(None, {
|
||||
'fields': ('abschnitttyp', 'inhalt', 'order'),
|
||||
'classes': ('wide',),
|
||||
}),
|
||||
)
|
||||
|
||||
class GeltungsbereichInline(NestedTabularInline):
|
||||
class GeltungsbereichInline(NestedStackedInline):
|
||||
model=Geltungsbereich
|
||||
extra=0
|
||||
sortable_field_name = "order"
|
||||
show_change_link=True
|
||||
classes = ['collapse']
|
||||
classes = ['collapse']
|
||||
#inline=inhalt
|
||||
verbose_name_plural = "Geltungsbereich-Abschnitte"
|
||||
fieldsets = (
|
||||
(None, {
|
||||
'fields': ('abschnitttyp', 'inhalt', 'order'),
|
||||
'classes': ('wide',),
|
||||
}),
|
||||
)
|
||||
|
||||
class EinleitungInline(NestedTabularInline):
|
||||
model = Einleitung
|
||||
extra = 0
|
||||
sortable_field_name = "order"
|
||||
show_change_link = True
|
||||
classes = ['collapse']
|
||||
class EinleitungInline(NestedStackedInline):
|
||||
model = Einleitung
|
||||
extra = 0
|
||||
sortable_field_name = "order"
|
||||
show_change_link = True
|
||||
classes = ['collapse']
|
||||
verbose_name_plural = "Einleitungs-Abschnitte"
|
||||
fieldsets = (
|
||||
(None, {
|
||||
'fields': ('abschnitttyp', 'inhalt', 'order'),
|
||||
'classes': ('wide',),
|
||||
}),
|
||||
)
|
||||
|
||||
class VorgabeForm(forms.ModelForm):
|
||||
# referenzen = TreeNodeMultipleChoiceField(queryset=Referenz.objects.all(), required=False)
|
||||
referenzen = TreeNodeMultipleChoiceField(queryset=Referenz.objects.all(), required=False)
|
||||
class Meta:
|
||||
model = Vorgabe
|
||||
fields = '__all__'
|
||||
|
||||
class VorgabeInline(NestedTabularInline): # or StackedInline for more vertical layout
|
||||
class VorgabeInline(SortableInlineAdminMixin, NestedStackedInline):
|
||||
model = Vorgabe
|
||||
form = VorgabeForm
|
||||
extra = 0
|
||||
#show_change_link = True
|
||||
inlines = [VorgabeKurztextInline,VorgabeLangtextInline,ChecklistenfragenInline]
|
||||
sortable_field_name = "order"
|
||||
show_change_link = False
|
||||
can_delete = False
|
||||
inlines = [VorgabeKurztextInline, VorgabeLangtextInline, ChecklistenfragenInline]
|
||||
autocomplete_fields = ['stichworte','referenzen','relevanz']
|
||||
#search_fields=['nummer','name']ModelAdmin.
|
||||
list_filter=['stichworte']
|
||||
#classes=["collapse"]
|
||||
# Remove collapse class so Vorgaben show by default
|
||||
|
||||
fieldsets = (
|
||||
('Grunddaten', {
|
||||
'fields': (('order', 'nummer'), ('thema', 'titel')),
|
||||
'classes': ('wide',),
|
||||
}),
|
||||
('Gültigkeit', {
|
||||
'fields': (('gueltigkeit_von', 'gueltigkeit_bis'),),
|
||||
'classes': ('wide',),
|
||||
}),
|
||||
('Verknüpfungen', {
|
||||
'fields': (('referenzen', 'stichworte', 'relevanz'),),
|
||||
'classes': ('wide',),
|
||||
}),
|
||||
)
|
||||
|
||||
class StichworterklaerungInline(NestedStackedInline):
|
||||
class StichworterklaerungInline(NestedTabularInline):
|
||||
model=Stichworterklaerung
|
||||
extra=0
|
||||
sortable_field_name = "order"
|
||||
@@ -100,28 +146,104 @@ class PersonAdmin(admin.ModelAdmin):
|
||||
|
||||
|
||||
@admin.register(Dokument)
|
||||
class DokumentAdmin(NestedModelAdmin):
|
||||
class DokumentAdmin(SortableAdminBase, NestedModelAdmin):
|
||||
actions_on_top=True
|
||||
inlines = [EinleitungInline,GeltungsbereichInline,VorgabeInline]
|
||||
#filter_horizontal=['autoren','pruefende']
|
||||
list_display=['nummer','name','dokumententyp']
|
||||
inlines = [EinleitungInline, GeltungsbereichInline, VorgabeInline]
|
||||
filter_horizontal=['autoren','pruefende']
|
||||
list_display=['nummer','name','dokumententyp','gueltigkeit_von','gueltigkeit_bis','aktiv']
|
||||
search_fields=['nummer','name']
|
||||
list_filter=['dokumententyp','aktiv','gueltigkeit_von']
|
||||
|
||||
fieldsets = (
|
||||
('Grunddaten', {
|
||||
'fields': ('nummer', 'name', 'dokumententyp', 'aktiv'),
|
||||
'classes': ('wide',),
|
||||
}),
|
||||
('Verantwortlichkeiten', {
|
||||
'fields': ('autoren', 'pruefende'),
|
||||
'classes': ('wide', 'collapse'),
|
||||
}),
|
||||
('Gültigkeit & Metadaten', {
|
||||
'fields': ('gueltigkeit_von', 'gueltigkeit_bis', 'signatur_cso', 'anhaenge'),
|
||||
'classes': ('wide', 'collapse'),
|
||||
}),
|
||||
)
|
||||
|
||||
class Media:
|
||||
# js = ('admin/js/vorgabe_collapse.js',)
|
||||
js = ('admin/js/vorgabe_collapse.js',)
|
||||
css = {
|
||||
'all': ('admin/css/vorgabe_border.css',
|
||||
# 'admin/css/vorgabe_collapse.css',
|
||||
)
|
||||
'all': ('admin/css/vorgabe_border.css',)
|
||||
}
|
||||
|
||||
|
||||
#admin.site.register(Stichwort)
|
||||
|
||||
@admin.register(VorgabenTable)
|
||||
class VorgabenTableAdmin(admin.ModelAdmin):
|
||||
list_display = ['order', 'nummer', 'dokument', 'thema', 'titel', 'gueltigkeit_von', 'gueltigkeit_bis']
|
||||
list_display_links = ['dokument']
|
||||
list_editable = ['order', 'nummer', 'thema', 'titel', 'gueltigkeit_von', 'gueltigkeit_bis']
|
||||
list_filter = ['dokument', 'thema', 'gueltigkeit_von', 'gueltigkeit_bis']
|
||||
search_fields = ['nummer', 'titel', 'dokument__nummer', 'dokument__name']
|
||||
autocomplete_fields = ['dokument', 'thema', 'stichworte', 'referenzen', 'relevanz']
|
||||
ordering = ['order']
|
||||
list_per_page = 100
|
||||
|
||||
fieldsets = (
|
||||
('Grunddaten', {
|
||||
'fields': ('order', 'nummer', 'dokument', 'thema', 'titel')
|
||||
}),
|
||||
('Gültigkeit', {
|
||||
'fields': ('gueltigkeit_von', 'gueltigkeit_bis')
|
||||
}),
|
||||
('Verknüpfungen', {
|
||||
'fields': ('referenzen', 'stichworte', 'relevanz'),
|
||||
'classes': ('collapse',)
|
||||
}),
|
||||
)
|
||||
|
||||
@admin.register(Thema)
|
||||
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(Thema)
|
||||
#admin.site.register(Referenz, DraggableM§PTTAdmin)
|
||||
admin.site.register(Vorgabe)
|
||||
|
||||
|
||||
#admin.site.register(Changelog)
|
||||
|
||||
70
dokumente/management/commands/sanity_check_vorgaben.py
Normal file
70
dokumente/management/commands/sanity_check_vorgaben.py
Normal file
@@ -0,0 +1,70 @@
|
||||
from django.core.management.base import BaseCommand
|
||||
from django.db import transaction
|
||||
from dokumente.models import Vorgabe
|
||||
import datetime
|
||||
|
||||
|
||||
class Command(BaseCommand):
|
||||
help = 'Run sanity checks on Vorgaben to detect conflicts'
|
||||
|
||||
def add_arguments(self, parser):
|
||||
parser.add_argument(
|
||||
'--fix',
|
||||
action='store_true',
|
||||
help='Attempt to fix conflicts (not implemented yet)',
|
||||
)
|
||||
parser.add_argument(
|
||||
'--verbose',
|
||||
action='store_true',
|
||||
help='Show detailed output',
|
||||
)
|
||||
|
||||
def handle(self, *args, **options):
|
||||
self.verbose = options['verbose']
|
||||
|
||||
self.stdout.write(self.style.SUCCESS('Starting Vorgaben sanity check...'))
|
||||
|
||||
# Run the sanity check
|
||||
conflicts = Vorgabe.sanity_check_vorgaben()
|
||||
|
||||
if not conflicts:
|
||||
self.stdout.write(self.style.SUCCESS('✓ No conflicts found in Vorgaben'))
|
||||
return
|
||||
|
||||
self.stdout.write(
|
||||
self.style.WARNING(f'Found {len(conflicts)} conflicts:')
|
||||
)
|
||||
|
||||
for i, conflict in enumerate(conflicts, 1):
|
||||
self._display_conflict(i, conflict)
|
||||
|
||||
if options['fix']:
|
||||
self.stdout.write(self.style.ERROR('Auto-fix not implemented yet'))
|
||||
|
||||
def _display_conflict(self, index, conflict):
|
||||
"""Display a single conflict"""
|
||||
v1 = conflict['vorgabe1']
|
||||
v2 = conflict['vorgabe2']
|
||||
|
||||
self.stdout.write(f"\n{index}. {conflict['message']}")
|
||||
|
||||
if self.verbose:
|
||||
self.stdout.write(f" Vorgabe 1: {v1.Vorgabennummer()}")
|
||||
self.stdout.write(f" Valid from: {v1.gueltigkeit_von} to {v1.gueltigkeit_bis or 'unlimited'}")
|
||||
self.stdout.write(f" Title: {v1.titel}")
|
||||
|
||||
self.stdout.write(f" Vorgabe 2: {v2.Vorgabennummer()}")
|
||||
self.stdout.write(f" Valid from: {v2.gueltigkeit_von} to {v2.gueltigkeit_bis or 'unlimited'}")
|
||||
self.stdout.write(f" Title: {v2.titel}")
|
||||
|
||||
# Show the overlapping period
|
||||
overlap_start = max(v1.gueltigkeit_von, v2.gueltigkeit_von)
|
||||
overlap_end = min(
|
||||
v1.gueltigkeit_bis or datetime.date.max,
|
||||
v2.gueltigkeit_bis or datetime.date.max
|
||||
)
|
||||
|
||||
if overlap_end != datetime.date.max:
|
||||
self.stdout.write(f" Overlap: {overlap_start} to {overlap_end}")
|
||||
else:
|
||||
self.stdout.write(f" Overlap starts: {overlap_start} (no end)")
|
||||
19
dokumente/migrations/0008_dokument_aktiv.py
Normal file
19
dokumente/migrations/0008_dokument_aktiv.py
Normal file
@@ -0,0 +1,19 @@
|
||||
# Generated by Django 5.2.5 on 2025-10-27 19:48
|
||||
|
||||
from django.db import migrations, models
|
||||
|
||||
|
||||
class Migration(migrations.Migration):
|
||||
|
||||
dependencies = [
|
||||
('dokumente', '0007_alter_changelog_options_and_more'),
|
||||
]
|
||||
|
||||
operations = [
|
||||
migrations.AddField(
|
||||
model_name='dokument',
|
||||
name='aktiv',
|
||||
field=models.BooleanField(blank=True, default=False),
|
||||
preserve_default=False,
|
||||
),
|
||||
]
|
||||
@@ -0,0 +1,23 @@
|
||||
# Generated by Django 5.2.5 on 2025-10-28 14:51
|
||||
|
||||
from django.db import migrations, models
|
||||
|
||||
|
||||
class Migration(migrations.Migration):
|
||||
|
||||
dependencies = [
|
||||
('dokumente', '0008_dokument_aktiv'),
|
||||
]
|
||||
|
||||
operations = [
|
||||
migrations.AlterModelOptions(
|
||||
name='vorgabe',
|
||||
options={'ordering': ['order'], 'verbose_name_plural': 'Vorgaben'},
|
||||
),
|
||||
migrations.AddField(
|
||||
model_name='vorgabe',
|
||||
name='order',
|
||||
field=models.IntegerField(default=0),
|
||||
preserve_default=False,
|
||||
),
|
||||
]
|
||||
@@ -47,6 +47,7 @@ class Dokument(models.Model):
|
||||
gueltigkeit_bis = models.DateField(null=True, blank=True)
|
||||
signatur_cso = models.CharField(max_length=255, blank=True)
|
||||
anhaenge = models.TextField(blank=True)
|
||||
aktiv = models.BooleanField(blank=True)
|
||||
|
||||
def __str__(self):
|
||||
return f"{self.nummer} – {self.name}"
|
||||
@@ -56,9 +57,10 @@ class Dokument(models.Model):
|
||||
verbose_name="Dokument"
|
||||
|
||||
class Vorgabe(models.Model):
|
||||
order = models.IntegerField()
|
||||
nummer = models.IntegerField()
|
||||
dokument = models.ForeignKey(Dokument, on_delete=models.CASCADE, related_name='vorgaben')
|
||||
thema = models.ForeignKey(Thema, on_delete=models.PROTECT)
|
||||
thema = models.ForeignKey(Thema, on_delete=models.PROTECT, blank=False)
|
||||
titel = models.CharField(max_length=255)
|
||||
referenzen = models.ManyToManyField(Referenz, blank=True)
|
||||
gueltigkeit_von = models.DateField()
|
||||
@@ -76,7 +78,7 @@ class Vorgabe(models.Model):
|
||||
if not self.gueltigkeit_bis:
|
||||
return "active"
|
||||
|
||||
if self.gueltigkeit_bis > check_date:
|
||||
if self.gueltigkeit_bis >= check_date:
|
||||
return "active"
|
||||
|
||||
return "expired" if not verbose else "Ist seit dem "+self.gueltigkeit_bis.strftime('%d.%m.%Y')+" nicht mehr in Kraft."
|
||||
@@ -84,9 +86,126 @@ class Vorgabe(models.Model):
|
||||
def __str__(self):
|
||||
return f"{self.Vorgabennummer()}: {self.titel}"
|
||||
|
||||
@staticmethod
|
||||
def sanity_check_vorgaben():
|
||||
"""
|
||||
Sanity check for Vorgaben:
|
||||
If there are two Vorgaben with the same number, Thema and Dokument,
|
||||
their valid_from and valid_to date ranges shouldn't intersect.
|
||||
|
||||
Returns:
|
||||
list: List of dictionaries containing conflicts found
|
||||
"""
|
||||
conflicts = []
|
||||
|
||||
# Group Vorgaben by dokument, thema, and nummer
|
||||
from django.db.models import Count
|
||||
from itertools import combinations
|
||||
|
||||
# Find Vorgaben with same dokument, thema, and nummer
|
||||
duplicate_groups = (
|
||||
Vorgabe.objects.values('dokument', 'thema', 'nummer')
|
||||
.annotate(count=Count('id'))
|
||||
.filter(count__gt=1)
|
||||
)
|
||||
|
||||
for group in duplicate_groups:
|
||||
# Get all Vorgaben in this group
|
||||
vorgaben = Vorgabe.objects.filter(
|
||||
dokument=group['dokument'],
|
||||
thema=group['thema'],
|
||||
nummer=group['nummer']
|
||||
)
|
||||
|
||||
# Check all pairs for date range intersections
|
||||
for vorgabe1, vorgabe2 in combinations(vorgaben, 2):
|
||||
if Vorgabe._date_ranges_intersect(
|
||||
vorgabe1.gueltigkeit_von, vorgabe1.gueltigkeit_bis,
|
||||
vorgabe2.gueltigkeit_von, vorgabe2.gueltigkeit_bis
|
||||
):
|
||||
conflicts.append({
|
||||
'vorgabe1': vorgabe1,
|
||||
'vorgabe2': vorgabe2,
|
||||
'conflict_type': 'date_range_intersection',
|
||||
'message': f"Vorgaben {vorgabe1.Vorgabennummer()} and {vorgabe2.Vorgabennummer()} "
|
||||
f"überschneiden sich in der Geltungsdauer"
|
||||
})
|
||||
|
||||
return conflicts
|
||||
|
||||
def clean(self):
|
||||
"""
|
||||
Validate the Vorgabe before saving.
|
||||
"""
|
||||
from django.core.exceptions import ValidationError
|
||||
|
||||
# Check for conflicts with existing Vorgaben
|
||||
conflicts = self.find_conflicts()
|
||||
if conflicts:
|
||||
conflict_messages = [c['message'] for c in conflicts]
|
||||
raise ValidationError({
|
||||
'__all__': conflict_messages
|
||||
})
|
||||
|
||||
def find_conflicts(self):
|
||||
"""
|
||||
Find conflicts with existing Vorgaben.
|
||||
|
||||
Returns:
|
||||
list: List of conflict dictionaries
|
||||
"""
|
||||
conflicts = []
|
||||
|
||||
# Find Vorgaben with same dokument, thema, and nummer (excluding self)
|
||||
existing_vorgaben = Vorgabe.objects.filter(
|
||||
dokument=self.dokument,
|
||||
thema=self.thema,
|
||||
nummer=self.nummer
|
||||
).exclude(pk=self.pk)
|
||||
|
||||
for other_vorgabe in existing_vorgaben:
|
||||
if self._date_ranges_intersect(
|
||||
self.gueltigkeit_von, self.gueltigkeit_bis,
|
||||
other_vorgabe.gueltigkeit_von, other_vorgabe.gueltigkeit_bis
|
||||
):
|
||||
conflicts.append({
|
||||
'vorgabe1': self,
|
||||
'vorgabe2': other_vorgabe,
|
||||
'conflict_type': 'date_range_intersection',
|
||||
'message': f"Vorgabe {self.Vorgabennummer()} in Konflikt mit "
|
||||
f"bestehender {other_vorgabe.Vorgabennummer()} "
|
||||
f" - Geltungsdauer übeschneidet sich"
|
||||
})
|
||||
|
||||
return conflicts
|
||||
|
||||
@staticmethod
|
||||
def _date_ranges_intersect(start1, end1, start2, end2):
|
||||
"""
|
||||
Check if two date ranges intersect.
|
||||
None end date means open-ended range.
|
||||
|
||||
Args:
|
||||
start1, start2: Start dates
|
||||
end1, end2: End dates (can be None for open-ended)
|
||||
|
||||
Returns:
|
||||
bool: True if ranges intersect
|
||||
"""
|
||||
# If either start date is None, treat it as invalid case
|
||||
if not start1 or not start2:
|
||||
return False
|
||||
|
||||
# If end date is None, treat it as far future
|
||||
end1 = end1 or datetime.date.max
|
||||
end2 = end2 or datetime.date.max
|
||||
|
||||
# Ranges intersect if start1 <= end2 and start2 <= end1
|
||||
return start1 <= end2 and start2 <= end1
|
||||
|
||||
class Meta:
|
||||
verbose_name_plural="Vorgaben"
|
||||
|
||||
ordering = ['order']
|
||||
|
||||
class VorgabeLangtext(Textabschnitt):
|
||||
abschnitt=models.ForeignKey(Vorgabe,on_delete=models.CASCADE)
|
||||
@@ -123,6 +242,12 @@ class Checklistenfrage(models.Model):
|
||||
verbose_name_plural="Fragen für Checkliste"
|
||||
verbose_name="Frage für Checkliste"
|
||||
|
||||
class VorgabenTable(Vorgabe):
|
||||
class Meta:
|
||||
proxy = True
|
||||
verbose_name = "Vorgabe (Tabellenansicht)"
|
||||
verbose_name_plural = "Vorgaben (Tabellenansicht)"
|
||||
|
||||
class Changelog(models.Model):
|
||||
dokument = models.ForeignKey(Dokument, on_delete=models.CASCADE, related_name='changelog')
|
||||
autoren = models.ManyToManyField(Person)
|
||||
|
||||
151
dokumente/templates/standards/incomplete_vorgaben.html
Normal file
151
dokumente/templates/standards/incomplete_vorgaben.html
Normal file
@@ -0,0 +1,151 @@
|
||||
{% extends "base.html" %}
|
||||
{% block content %}
|
||||
<h1 class="mb-4">Unvollständige Vorgaben</h1>
|
||||
|
||||
{% 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>
|
||||
|
||||
<!-- 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="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>
|
||||
{% 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 %}
|
||||
@@ -8,7 +8,7 @@
|
||||
<!-- Autoren, Prüfende etc. -->
|
||||
<p><strong>Autoren:</strong> {{ standard.autoren.all|join:", " }}</p>
|
||||
<p><strong>Prüfende:</strong> {{ standard.pruefende.all|join:", " }}</p>
|
||||
<p><strong>Gültigkeit:</strong> {{ standard.gueltigkeit_von }} bis {{ standard.gueltigkeit_bis }}</p>
|
||||
<p><strong>Gültigkeit:</strong> {{ standard.gueltigkeit_von }} bis {{ standard.gueltigkeit_bis|default_if_none:"auf weiteres" }}</p>
|
||||
|
||||
<!-- Start Einleitung -->
|
||||
{% if standard.einleitung_html %}
|
||||
|
||||
@@ -2,10 +2,10 @@
|
||||
{% block content %}
|
||||
<h1>Standards Informatiksicherheit</h1>
|
||||
<ul>
|
||||
{% for standard in standards %}
|
||||
{% for dokument in dokumente %}
|
||||
<li>
|
||||
<a href="{% url 'standard_detail' nummer=standard.nummer %}">
|
||||
{{ standard.nummer }} – {{ standard.name }}
|
||||
<a href="{% url 'standard_detail' nummer=dokument.nummer %}">
|
||||
{{ dokument.nummer }} – {{ dokument.name }}
|
||||
</a>
|
||||
</li>
|
||||
{% endfor %}
|
||||
|
||||
1138
dokumente/tests.py
1138
dokumente/tests.py
File diff suppressed because it is too large
Load Diff
@@ -3,6 +3,7 @@ from . import views
|
||||
|
||||
urlpatterns = [
|
||||
path('', views.standard_list, name='standard_list'),
|
||||
path('unvollstaendig/', views.incomplete_vorgaben, name='incomplete_vorgaben'),
|
||||
path('<str:nummer>/', views.standard_detail, name='standard_detail'),
|
||||
path('<str:nummer>/history/<str:check_date>/', views.standard_detail),
|
||||
path('<str:nummer>/history/', views.standard_detail, {"check_date":"today"}, name='standard_history'),
|
||||
|
||||
123
dokumente/utils.py
Normal file
123
dokumente/utils.py
Normal file
@@ -0,0 +1,123 @@
|
||||
"""
|
||||
Utility functions for Vorgaben sanity checking
|
||||
"""
|
||||
import datetime
|
||||
from django.db.models import Count
|
||||
from itertools import combinations
|
||||
from dokumente.models import Vorgabe
|
||||
|
||||
|
||||
def check_vorgabe_conflicts():
|
||||
"""
|
||||
Check for conflicts in Vorgaben.
|
||||
|
||||
Main rule: If there are two Vorgaben with the same number, Thema and Dokument,
|
||||
their valid_from and valid_to date ranges shouldn't intersect.
|
||||
|
||||
Returns:
|
||||
list: List of conflict dictionaries
|
||||
"""
|
||||
conflicts = []
|
||||
|
||||
# Find Vorgaben with same dokument, thema, and nummer
|
||||
duplicate_groups = (
|
||||
Vorgabe.objects.values('dokument', 'thema', 'nummer')
|
||||
.annotate(count=Count('id'))
|
||||
.filter(count__gt=1)
|
||||
)
|
||||
|
||||
for group in duplicate_groups:
|
||||
# Get all Vorgaben in this group
|
||||
vorgaben = Vorgabe.objects.filter(
|
||||
dokument=group['dokument'],
|
||||
thema=group['thema'],
|
||||
nummer=group['nummer']
|
||||
)
|
||||
|
||||
# Check all pairs for date range intersections
|
||||
for vorgabe1, vorgabe2 in combinations(vorgaben, 2):
|
||||
if date_ranges_intersect(
|
||||
vorgabe1.gueltigkeit_von, vorgabe1.gueltigkeit_bis,
|
||||
vorgabe2.gueltigkeit_von, vorgabe2.gueltigkeit_bis
|
||||
):
|
||||
conflicts.append({
|
||||
'vorgabe1': vorgabe1,
|
||||
'vorgabe2': vorgabe2,
|
||||
'conflict_type': 'date_range_intersection',
|
||||
'message': f"Vorgaben {vorgabe1.Vorgabennummer()} and {vorgabe2.Vorgabennummer()} "
|
||||
f"have intersecting validity periods"
|
||||
})
|
||||
|
||||
return conflicts
|
||||
|
||||
|
||||
def date_ranges_intersect(start1, end1, start2, end2):
|
||||
"""
|
||||
Check if two date ranges intersect.
|
||||
None end date means open-ended range.
|
||||
|
||||
Args:
|
||||
start1, start2: Start dates
|
||||
end1, end2: End dates (can be None for open-ended)
|
||||
|
||||
Returns:
|
||||
bool: True if ranges intersect
|
||||
"""
|
||||
# If either start date is None, treat it as invalid case
|
||||
if not start1 or not start2:
|
||||
return False
|
||||
|
||||
# If end date is None, treat it as far future
|
||||
end1 = end1 or datetime.date.max
|
||||
end2 = end2 or datetime.date.max
|
||||
|
||||
# Ranges intersect if start1 <= end2 and start2 <= end1
|
||||
return start1 <= end2 and start2 <= end1
|
||||
|
||||
|
||||
def format_conflict_report(conflicts, verbose=False):
|
||||
"""
|
||||
Format conflicts into a readable report.
|
||||
|
||||
Args:
|
||||
conflicts: List of conflict dictionaries
|
||||
verbose: Whether to show detailed information
|
||||
|
||||
Returns:
|
||||
str: Formatted report
|
||||
"""
|
||||
if not conflicts:
|
||||
return "✓ No conflicts found in Vorgaben"
|
||||
|
||||
lines = [f"Found {len(conflicts)} conflicts:"]
|
||||
|
||||
for i, conflict in enumerate(conflicts, 1):
|
||||
lines.append(f"\n{i}. {conflict['message']}")
|
||||
|
||||
if verbose:
|
||||
v1 = conflict['vorgabe1']
|
||||
v2 = conflict['vorgabe2']
|
||||
|
||||
lines.append(f" Vorgabe 1: {v1.Vorgabennummer()}")
|
||||
lines.append(f" Valid from: {v1.gueltigkeit_von} to {v1.gueltigkeit_bis or 'unlimited'}")
|
||||
lines.append(f" Title: {v1.titel}")
|
||||
|
||||
lines.append(f" Vorgabe 2: {v2.Vorgabennummer()}")
|
||||
lines.append(f" Valid from: {v2.gueltigkeit_von} to {v2.gueltigkeit_bis or 'unlimited'}")
|
||||
lines.append(f" Title: {v2.titel}")
|
||||
|
||||
# Show the overlapping period
|
||||
v1 = conflict['vorgabe1']
|
||||
v2 = conflict['vorgabe2']
|
||||
overlap_start = max(v1.gueltigkeit_von, v2.gueltigkeit_von)
|
||||
overlap_end = min(
|
||||
v1.gueltigkeit_bis or datetime.date.max,
|
||||
v2.gueltigkeit_bis or datetime.date.max
|
||||
)
|
||||
|
||||
if overlap_end != datetime.date.max:
|
||||
lines.append(f" Overlap: {overlap_start} to {overlap_end}")
|
||||
else:
|
||||
lines.append(f" Overlap starts: {overlap_start} (no end)")
|
||||
|
||||
return "\n".join(lines)
|
||||
@@ -1,5 +1,6 @@
|
||||
from django.shortcuts import render, get_object_or_404
|
||||
from .models import Dokument
|
||||
from django.contrib.auth.decorators import login_required, user_passes_test
|
||||
from .models import Dokument, Vorgabe, VorgabeKurztext, VorgabeLangtext, Checklistenfrage
|
||||
from abschnitte.utils import render_textabschnitte
|
||||
|
||||
from datetime import date
|
||||
@@ -9,9 +10,9 @@ calendar=parsedatetime.Calendar()
|
||||
|
||||
|
||||
def standard_list(request):
|
||||
standards = Dokument.objects.all()
|
||||
dokumente = Dokument.objects.all()
|
||||
return render(request, 'standards/standard_list.html',
|
||||
{'dokumente': standards}
|
||||
{'dokumente': dokumente}
|
||||
)
|
||||
|
||||
|
||||
@@ -56,3 +57,48 @@ def standard_checkliste(request, nummer):
|
||||
})
|
||||
|
||||
|
||||
def is_staff_user(user):
|
||||
return user.is_staff
|
||||
|
||||
@login_required
|
||||
@user_passes_test(is_staff_user)
|
||||
def incomplete_vorgaben(request):
|
||||
"""
|
||||
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').prefetch_related(
|
||||
'referenzen', 'stichworte', 'checklistenfragen', 'vorgabekurztext_set', 'vorgabelangtext_set'
|
||||
)
|
||||
|
||||
# Build table data
|
||||
vorgaben_data = []
|
||||
for vorgabe in all_vorgaben:
|
||||
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
|
||||
})
|
||||
|
||||
# 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', {
|
||||
'vorgaben_data': vorgaben_data,
|
||||
})
|
||||
|
||||
@@ -17,6 +17,9 @@
|
||||
<div class="collapse navbar-collapse" id="navbarNavAltMarkup">
|
||||
<div class="navbar-nav">
|
||||
<a class="nav-item nav-link active" href="/dokumente">Standards</a>
|
||||
{% if user.is_staff %}
|
||||
<a class="nav-item nav-link" href="/dokumente/unvollstaendig/">Unvollständig</a>
|
||||
{% endif %}
|
||||
<a class="nav-item nav-link" href="/referenzen">Referenzen</a>
|
||||
<a class="nav-item nav-link" href="/stichworte">Stichworte</a>
|
||||
<a class="nav-item nav-link" href="/search">Suche</a>
|
||||
@@ -28,6 +31,6 @@
|
||||
<div class="flex-fill">{% block content %}Main Content{% endblock %}</div>
|
||||
<div class="col-md-2">{% block sidebar_right %}{% endblock %}</div>
|
||||
</div>
|
||||
<div>VorgabenUI v0.927</div>
|
||||
<div>VorgabenUI v0.942</div>
|
||||
</body>
|
||||
</html>
|
||||
|
||||
@@ -12,9 +12,9 @@
|
||||
{% endfor %}
|
||||
{% endif %}
|
||||
|
||||
{% if resultat.kurztext %}
|
||||
<h2>Vorgaben mit "{{ suchbegriff }}" im Kurztext</h2>
|
||||
{% for standard, vorgaben in resultat.kurztext.items %}
|
||||
{% if resultat.all %}
|
||||
<h2>Vorgaben mit "{{ suchbegriff }}"</h2>
|
||||
{% for standard, vorgaben in resultat.all.items %}
|
||||
<h4>{{ standard }}</h4>
|
||||
<ul>
|
||||
{% for vorgabe in vorgaben %}
|
||||
@@ -24,18 +24,7 @@
|
||||
{% endfor %}
|
||||
{% endif %}
|
||||
|
||||
{% if resultat.langtext %}
|
||||
<h2>Vorgaben mit "{{ suchbegriff }}" im Langtext</h2>
|
||||
{% for standard, vorgaben in resultat.langtext.items %}
|
||||
<h4>{{ standard }}</h4>
|
||||
<ul>
|
||||
{% for vorgabe in vorgaben %}
|
||||
<li><a href="{% url 'standard_detail' nummer=vorgabe.dokument.nummer %}#{{vorgabe.Vorgabennummer}}">{{vorgabe}}</a></li>
|
||||
{% endfor %}
|
||||
</ul>
|
||||
{% endfor %}
|
||||
{% endif %}
|
||||
{% if not resultat.langtext and not resultat.kurztext and not resultat.geltungsbereich %}
|
||||
{% if not resultat.all %}
|
||||
<h2>Keine Resultate für "{{suchbegriff}}"</h2>
|
||||
{% endif %}
|
||||
{% endblock %}
|
||||
@@ -2,6 +2,12 @@
|
||||
{% block content %}
|
||||
<h1 class="mb-4">Suche</h1>
|
||||
|
||||
{% if error_message %}
|
||||
<div class="alert alert-danger">
|
||||
<strong>Fehler:</strong> {{ error_message }}
|
||||
</div>
|
||||
{% endif %}
|
||||
|
||||
<!-- Search form -->
|
||||
<form action="." method="post">
|
||||
{% csrf_token %}
|
||||
@@ -13,29 +19,10 @@
|
||||
id="query"
|
||||
name="q"
|
||||
placeholder="Suchbegriff eingeben …"
|
||||
required>
|
||||
value="{{ search_term|default:'' }}"
|
||||
required
|
||||
maxlength="200">
|
||||
</div>
|
||||
|
||||
<!-- Check-box group -->
|
||||
<fieldset class="mb-4">
|
||||
<legend class="h6 mb-2">In folgenden Bereichen suchen:</legend>
|
||||
|
||||
<div class="form-check">
|
||||
<input class="form-check-input" type="checkbox" value="kurztext" id="kurztext" name="suchbereich[]" checked>
|
||||
<label class="form-check-label" for="kurztext">Kurztext</label>
|
||||
</div>
|
||||
|
||||
<div class="form-check">
|
||||
<input class="form-check-input" type="checkbox" value="langtext" id="langtext" name="suchbereich[]" checked>
|
||||
<label class="form-check-label" for="langtext">Langtext</label>
|
||||
</div>
|
||||
|
||||
<div class="form-check">
|
||||
<input class="form-check-input" type="checkbox" value="geltungsbereich" id="geltungsbereich" name="suchbereich[]">
|
||||
<label class="form-check-label" for="geltungsbereich">Geltungsbereich</label>
|
||||
</div>
|
||||
</fieldset>
|
||||
|
||||
<button type="submit" class="btn btn-primary">Suchen</button>
|
||||
</form>
|
||||
{% endblock %}
|
||||
|
||||
312
pages/tests.py
Normal file
312
pages/tests.py
Normal file
@@ -0,0 +1,312 @@
|
||||
from django.test import TestCase, Client
|
||||
from django.core.exceptions import ValidationError
|
||||
from django.utils import timezone
|
||||
from datetime import date, timedelta
|
||||
from dokumente.models import Dokument, Vorgabe, VorgabeKurztext, VorgabeLangtext, Geltungsbereich, Dokumententyp, Thema
|
||||
from stichworte.models import Stichwort
|
||||
from unittest.mock import patch
|
||||
import re
|
||||
|
||||
|
||||
class SearchViewTest(TestCase):
|
||||
def setUp(self):
|
||||
self.client = Client()
|
||||
|
||||
# Create test data
|
||||
self.dokumententyp = Dokumententyp.objects.create(
|
||||
name="Test Typ",
|
||||
verantwortliche_ve="Test VE"
|
||||
)
|
||||
|
||||
self.thema = Thema.objects.create(
|
||||
name="Test Thema",
|
||||
erklaerung="Test Erklärung"
|
||||
)
|
||||
|
||||
self.dokument = Dokument.objects.create(
|
||||
nummer="TEST-001",
|
||||
dokumententyp=self.dokumententyp,
|
||||
name="Test Dokument",
|
||||
gueltigkeit_von=date.today(),
|
||||
aktiv=True
|
||||
)
|
||||
|
||||
self.vorgabe = Vorgabe.objects.create(
|
||||
order=1,
|
||||
nummer=1,
|
||||
dokument=self.dokument,
|
||||
thema=self.thema,
|
||||
titel="Test Vorgabe Titel",
|
||||
gueltigkeit_von=date.today()
|
||||
)
|
||||
|
||||
# Create text content
|
||||
self.kurztext = VorgabeKurztext.objects.create(
|
||||
abschnitt=self.vorgabe,
|
||||
inhalt="Dies ist ein Test Kurztext mit Suchbegriff"
|
||||
)
|
||||
|
||||
self.langtext = VorgabeLangtext.objects.create(
|
||||
abschnitt=self.vorgabe,
|
||||
inhalt="Dies ist ein Test Langtext mit anderem Suchbegriff"
|
||||
)
|
||||
|
||||
self.geltungsbereich = Geltungsbereich.objects.create(
|
||||
geltungsbereich=self.dokument,
|
||||
inhalt="Test Geltungsbereich mit Suchbegriff"
|
||||
)
|
||||
|
||||
def test_search_get_request(self):
|
||||
"""Test GET request returns search form"""
|
||||
response = self.client.get('/search/')
|
||||
self.assertEqual(response.status_code, 200)
|
||||
self.assertContains(response, 'Suche')
|
||||
self.assertContains(response, 'Suchbegriff')
|
||||
|
||||
def test_search_post_valid_term(self):
|
||||
"""Test POST request with valid search term"""
|
||||
response = self.client.post('/search/', {'q': 'Test'})
|
||||
self.assertEqual(response.status_code, 200)
|
||||
self.assertContains(response, 'Suchresultate für Test')
|
||||
|
||||
def test_search_case_insensitive(self):
|
||||
"""Test that search is case insensitive"""
|
||||
# Search for lowercase
|
||||
response = self.client.post('/search/', {'q': 'test'})
|
||||
self.assertEqual(response.status_code, 200)
|
||||
self.assertContains(response, 'Suchresultate für test')
|
||||
|
||||
# Search for uppercase
|
||||
response = self.client.post('/search/', {'q': 'TEST'})
|
||||
self.assertEqual(response.status_code, 200)
|
||||
self.assertContains(response, 'Suchresultate für TEST')
|
||||
|
||||
# Search for mixed case
|
||||
response = self.client.post('/search/', {'q': 'TeSt'})
|
||||
self.assertEqual(response.status_code, 200)
|
||||
self.assertContains(response, 'Suchresultate für TeSt')
|
||||
|
||||
def test_search_in_kurztext(self):
|
||||
"""Test search in Kurztext content"""
|
||||
response = self.client.post('/search/', {'q': 'Suchbegriff'})
|
||||
self.assertEqual(response.status_code, 200)
|
||||
self.assertContains(response, 'TEST-001')
|
||||
|
||||
def test_search_in_langtext(self):
|
||||
"""Test search in Langtext content"""
|
||||
response = self.client.post('/search/', {'q': 'anderem'})
|
||||
self.assertEqual(response.status_code, 200)
|
||||
self.assertContains(response, 'TEST-001')
|
||||
|
||||
def test_search_in_titel(self):
|
||||
"""Test search in Vorgabe title"""
|
||||
response = self.client.post('/search/', {'q': 'Titel'})
|
||||
self.assertEqual(response.status_code, 200)
|
||||
self.assertContains(response, 'TEST-001')
|
||||
|
||||
def test_search_in_geltungsbereich(self):
|
||||
"""Test search in Geltungsbereich content"""
|
||||
response = self.client.post('/search/', {'q': 'Geltungsbereich'})
|
||||
self.assertEqual(response.status_code, 200)
|
||||
self.assertContains(response, 'Standards mit')
|
||||
|
||||
def test_search_no_results(self):
|
||||
"""Test search with no results"""
|
||||
response = self.client.post('/search/', {'q': 'NichtVorhanden'})
|
||||
self.assertEqual(response.status_code, 200)
|
||||
self.assertContains(response, 'Keine Resultate für "NichtVorhanden"')
|
||||
|
||||
def test_search_expired_vorgabe_not_included(self):
|
||||
"""Test that expired Vorgaben are not included in results"""
|
||||
# Create expired Vorgabe
|
||||
expired_vorgabe = Vorgabe.objects.create(
|
||||
order=2,
|
||||
nummer=2,
|
||||
dokument=self.dokument,
|
||||
thema=self.thema,
|
||||
titel="Abgelaufene Vorgabe",
|
||||
gueltigkeit_von=date.today() - timedelta(days=10),
|
||||
gueltigkeit_bis=date.today() - timedelta(days=1)
|
||||
)
|
||||
|
||||
VorgabeKurztext.objects.create(
|
||||
abschnitt=expired_vorgabe,
|
||||
inhalt="Abgelaufener Inhalt mit Test"
|
||||
)
|
||||
|
||||
response = self.client.post('/search/', {'q': 'Test'})
|
||||
self.assertEqual(response.status_code, 200)
|
||||
|
||||
# Should only find the active Vorgabe, not the expired one
|
||||
self.assertContains(response, 'Test Vorgabe Titel')
|
||||
# The expired vorgabe should not appear in results
|
||||
self.assertNotContains(response, 'Abgelaufene Vorgabe')
|
||||
|
||||
def test_search_empty_term_validation(self):
|
||||
"""Test validation for empty search term"""
|
||||
response = self.client.post('/search/', {'q': ''})
|
||||
self.assertEqual(response.status_code, 200)
|
||||
self.assertContains(response, 'Fehler:')
|
||||
self.assertContains(response, 'Suchbegriff darf nicht leer sein')
|
||||
|
||||
def test_search_no_term_validation(self):
|
||||
"""Test validation when no search term is provided"""
|
||||
response = self.client.post('/search/', {})
|
||||
self.assertEqual(response.status_code, 200)
|
||||
self.assertContains(response, 'Fehler:')
|
||||
self.assertContains(response, 'Suchbegriff darf nicht leer sein')
|
||||
|
||||
def test_search_html_tags_stripped(self):
|
||||
"""Test that HTML tags are stripped from search input"""
|
||||
response = self.client.post('/search/', {'q': '<script>alert("xss")</script>Test'})
|
||||
self.assertEqual(response.status_code, 200)
|
||||
# Should search for "alert('xss')Test" after HTML tag removal
|
||||
self.assertContains(response, 'Suchresultate für alert("xss")Test')
|
||||
|
||||
def test_search_invalid_characters_validation(self):
|
||||
"""Test validation for invalid characters"""
|
||||
response = self.client.post('/search/', {'q': 'Test| DROP TABLE users'})
|
||||
self.assertEqual(response.status_code, 200)
|
||||
self.assertContains(response, 'Fehler:')
|
||||
self.assertContains(response, 'Ungültige Zeichen im Suchbegriff')
|
||||
|
||||
def test_search_too_long_validation(self):
|
||||
"""Test validation for overly long search terms"""
|
||||
long_term = 'a' * 201 # 201 characters, exceeds limit of 200
|
||||
response = self.client.post('/search/', {'q': long_term})
|
||||
self.assertEqual(response.status_code, 200)
|
||||
self.assertContains(response, 'Fehler:')
|
||||
self.assertContains(response, 'Suchbegriff ist zu lang')
|
||||
|
||||
def test_search_max_length_allowed(self):
|
||||
"""Test that exactly 200 characters are allowed"""
|
||||
max_term = 'a' * 200 # Exactly 200 characters
|
||||
response = self.client.post('/search/', {'q': max_term})
|
||||
self.assertEqual(response.status_code, 200)
|
||||
# Should not show validation error
|
||||
self.assertNotContains(response, 'Fehler:')
|
||||
|
||||
def test_search_german_umlauts_allowed(self):
|
||||
"""Test that German umlauts are allowed in search"""
|
||||
response = self.client.post('/search/', {'q': 'Test Müller äöü ÄÖÜ ß'})
|
||||
self.assertEqual(response.status_code, 200)
|
||||
# Should not show validation error
|
||||
self.assertNotContains(response, 'Fehler:')
|
||||
|
||||
def test_search_special_characters_allowed(self):
|
||||
"""Test that allowed special characters work"""
|
||||
response = self.client.post('/search/', {'q': 'Test-Test, Test: Test; Test! Test? (Test) [Test] {Test} "Test" \'Test\''})
|
||||
self.assertEqual(response.status_code, 200)
|
||||
# Should not show validation error
|
||||
self.assertNotContains(response, 'Fehler:')
|
||||
|
||||
def test_search_input_preserved_on_error(self):
|
||||
"""Test that search input is preserved on validation errors"""
|
||||
response = self.client.post('/search/', {'q': '<script>Test</script>'})
|
||||
self.assertEqual(response.status_code, 200)
|
||||
# The input should be preserved (escaped) in the form
|
||||
# Since HTML tags are stripped, we expect "Test" to be searched
|
||||
self.assertContains(response, 'Suchresultate für Test')
|
||||
|
||||
def test_search_xss_prevention_in_results(self):
|
||||
"""Test that search terms are escaped in results to prevent XSS"""
|
||||
# Create content with potential XSS
|
||||
self.kurztext.inhalt = "Content with <script>alert('xss')</script> term"
|
||||
self.kurztext.save()
|
||||
|
||||
response = self.client.post('/search/', {'q': 'term'})
|
||||
self.assertEqual(response.status_code, 200)
|
||||
# The script tag should be escaped in the output
|
||||
# Note: This depends on how the template renders the content
|
||||
self.assertContains(response, 'Suchresultate für term')
|
||||
|
||||
@patch('pages.views.pprint.pp')
|
||||
def test_search_result_logging(self, mock_pprint):
|
||||
"""Test that search results are logged for debugging"""
|
||||
response = self.client.post('/search/', {'q': 'Test'})
|
||||
self.assertEqual(response.status_code, 200)
|
||||
# Verify that pprint.pp was called with the result
|
||||
mock_pprint.assert_called_once()
|
||||
|
||||
def test_search_multiple_documents(self):
|
||||
"""Test search across multiple documents"""
|
||||
# Create second document
|
||||
dokument2 = Dokument.objects.create(
|
||||
nummer="TEST-002",
|
||||
dokumententyp=self.dokumententyp,
|
||||
name="Zweites Test Dokument",
|
||||
gueltigkeit_von=date.today(),
|
||||
aktiv=True
|
||||
)
|
||||
|
||||
vorgabe2 = Vorgabe.objects.create(
|
||||
order=1,
|
||||
nummer=1,
|
||||
dokument=dokument2,
|
||||
thema=self.thema,
|
||||
titel="Zweite Test Vorgabe",
|
||||
gueltigkeit_von=date.today()
|
||||
)
|
||||
|
||||
VorgabeKurztext.objects.create(
|
||||
abschnitt=vorgabe2,
|
||||
inhalt="Zweiter Test Inhalt"
|
||||
)
|
||||
|
||||
response = self.client.post('/search/', {'q': 'Test'})
|
||||
self.assertEqual(response.status_code, 200)
|
||||
# Should find results from both documents
|
||||
self.assertContains(response, 'TEST-001')
|
||||
self.assertContains(response, 'TEST-002')
|
||||
|
||||
|
||||
class SearchValidationTest(TestCase):
|
||||
"""Test the validate_search_input function directly"""
|
||||
|
||||
def test_validate_search_input_valid(self):
|
||||
"""Test valid search input"""
|
||||
from pages.views import validate_search_input
|
||||
|
||||
result = validate_search_input("Test Suchbegriff")
|
||||
self.assertEqual(result, "Test Suchbegriff")
|
||||
|
||||
def test_validate_search_input_empty(self):
|
||||
"""Test empty search input"""
|
||||
from pages.views import validate_search_input
|
||||
|
||||
with self.assertRaises(ValidationError) as context:
|
||||
validate_search_input("")
|
||||
|
||||
self.assertIn("Suchbegriff darf nicht leer sein", str(context.exception))
|
||||
|
||||
def test_validate_search_input_html_stripped(self):
|
||||
"""Test that HTML tags are stripped"""
|
||||
from pages.views import validate_search_input
|
||||
|
||||
result = validate_search_input("<script>alert('xss')</script>Test")
|
||||
self.assertEqual(result, "alert('xss')Test")
|
||||
|
||||
def test_validate_search_input_invalid_chars(self):
|
||||
"""Test validation of invalid characters"""
|
||||
from pages.views import validate_search_input
|
||||
|
||||
with self.assertRaises(ValidationError) as context:
|
||||
validate_search_input("Test| DROP TABLE users")
|
||||
|
||||
self.assertIn("Ungültige Zeichen im Suchbegriff", str(context.exception))
|
||||
|
||||
def test_validate_search_input_too_long(self):
|
||||
"""Test length validation"""
|
||||
from pages.views import validate_search_input
|
||||
|
||||
with self.assertRaises(ValidationError) as context:
|
||||
validate_search_input("a" * 201)
|
||||
|
||||
self.assertIn("Suchbegriff ist zu lang", str(context.exception))
|
||||
|
||||
def test_validate_search_input_whitespace_stripped(self):
|
||||
"""Test that whitespace is stripped"""
|
||||
from pages.views import validate_search_input
|
||||
|
||||
result = validate_search_input(" Test Suchbegriff ")
|
||||
self.assertEqual(result, "Test Suchbegriff")
|
||||
@@ -1,34 +1,71 @@
|
||||
from django.shortcuts import render
|
||||
from django.core.exceptions import ValidationError
|
||||
from django.utils.html import escape
|
||||
import re
|
||||
from abschnitte.utils import render_textabschnitte
|
||||
from dokumente.models import Dokument, VorgabeLangtext, VorgabeKurztext, Geltungsbereich
|
||||
from dokumente.models import Dokument, VorgabeLangtext, VorgabeKurztext, Geltungsbereich, Vorgabe
|
||||
from itertools import groupby
|
||||
import datetime
|
||||
|
||||
|
||||
def startseite(request):
|
||||
standards=list(Dokument.objects.all())
|
||||
standards=list(Dokument.objects.filter(aktiv=True))
|
||||
return render(request, 'startseite.html', {"dokumente":standards,})
|
||||
|
||||
def validate_search_input(search_term):
|
||||
"""
|
||||
Validate search input to prevent SQL injection and XSS
|
||||
"""
|
||||
if not search_term:
|
||||
raise ValidationError("Suchbegriff darf nicht leer sein")
|
||||
|
||||
# Remove any HTML tags to prevent XSS
|
||||
search_term = re.sub(r'<[^>]*>', '', search_term)
|
||||
|
||||
# Allow only alphanumeric characters, spaces, and basic punctuation
|
||||
# This prevents SQL injection and other malicious input while allowing useful characters
|
||||
if not re.match(r'^[a-zA-Z0-9äöüÄÖÜß\s\-\.\,\:\;\!\?\(\)\[\]\{\}\"\']+$', search_term):
|
||||
raise ValidationError("Ungültige Zeichen im Suchbegriff")
|
||||
|
||||
# Limit length to prevent DoS attacks
|
||||
if len(search_term) > 200:
|
||||
raise ValidationError("Suchbegriff ist zu lang")
|
||||
|
||||
return search_term.strip()
|
||||
|
||||
def search(request):
|
||||
if request.method == "GET":
|
||||
return render(request, 'search.html')
|
||||
elif request.method == "POST":
|
||||
suchbegriff=request.POST.get("q")
|
||||
areas=request.POST.getlist("suchbereich[]")
|
||||
result= {}
|
||||
geltungsbereich=set()
|
||||
if "kurztext" in areas:
|
||||
qs = VorgabeKurztext.objects.filter(inhalt__contains=suchbegriff).exclude(abschnitt__gueltigkeit_bis__lt=datetime.date.today())
|
||||
result["kurztext"] = {k: [o.abschnitt for o in g] for k, g in groupby(qs, key=lambda o: o.abschnitt.dokument)}
|
||||
if "langtext" in areas:
|
||||
qs = VorgabeLangtext.objects.filter(inhalt__contains=suchbegriff).exclude(abschnitt__gueltigkeit_bis__lt=datetime.date.today())
|
||||
result['langtext']= {k: [o.abschnitt for o in g] for k, g in groupby(qs, key=lambda o: o.abschnitt.dokument)}
|
||||
if "geltungsbereich" in areas:
|
||||
result["geltungsbereich"]={}
|
||||
geltungsbereich=set(list([x.geltungsbereich for x in Geltungsbereich.objects.filter(inhalt__contains=suchbegriff)]))
|
||||
for s in geltungsbereich:
|
||||
result["geltungsbereich"][s]=render_textabschnitte(s.geltungsbereich_set.order_by("order"))
|
||||
raw_search_term = request.POST.get("q", "")
|
||||
|
||||
try:
|
||||
suchbegriff = validate_search_input(raw_search_term)
|
||||
except ValidationError as e:
|
||||
return render(request, 'search.html', {
|
||||
'error_message': str(e),
|
||||
'search_term': escape(raw_search_term)
|
||||
})
|
||||
|
||||
# Escape the search term for display in templates
|
||||
safe_search_term = escape(suchbegriff)
|
||||
result= {"all": {}}
|
||||
qs = VorgabeKurztext.objects.filter(inhalt__icontains=suchbegriff).exclude(abschnitt__gueltigkeit_bis__lt=datetime.date.today())
|
||||
result["kurztext"] = {k: [o.abschnitt for o in g] for k, g in groupby(qs, key=lambda o: o.abschnitt.dokument)}
|
||||
qs = VorgabeLangtext.objects.filter(inhalt__icontains=suchbegriff).exclude(abschnitt__gueltigkeit_bis__lt=datetime.date.today())
|
||||
result['langtext']= {k: [o.abschnitt for o in g] for k, g in groupby(qs, key=lambda o: o.abschnitt.dokument)}
|
||||
qs = Vorgabe.objects.filter(titel__icontains=suchbegriff).exclude(gueltigkeit_bis__lt=datetime.date.today())
|
||||
result['titel']= {k: list(g) for k, g in groupby(qs, key=lambda o: o.dokument)}
|
||||
for r in result.keys():
|
||||
for s in result[r].keys():
|
||||
result[r][s]=set(result[r][s])
|
||||
return render(request,"results.html",{"suchbegriff":suchbegriff,"resultat":result})
|
||||
if r == 'titel':
|
||||
result["all"][s] = set(result["all"].get(s, set()) | set(result[r][s]))
|
||||
else:
|
||||
result["all"][s] = set(result["all"].get(s, set()) | set(result[r][s]))
|
||||
result["geltungsbereich"]={}
|
||||
geltungsbereich=set(list([x.geltungsbereich for x in Geltungsbereich.objects.filter(inhalt__icontains=suchbegriff)]))
|
||||
for s in geltungsbereich:
|
||||
result["geltungsbereich"][s]=render_textabschnitte(s.geltungsbereich_set.order_by("order"))
|
||||
|
||||
return render(request,"results.html",{"suchbegriff":safe_search_term,"resultat":result})
|
||||
|
||||
|
||||
@@ -13,4 +13,4 @@ class ReferenzerklaerungInline(NestedStackedInline):
|
||||
class ReferenzAdmin(NestedModelAdmin):
|
||||
inlines=[ReferenzerklaerungInline]
|
||||
list_display =['Path']
|
||||
search_fields=("referenz",)
|
||||
search_fields=("referenz","path")
|
||||
|
||||
@@ -6,6 +6,7 @@ charset-normalizer==3.4.3
|
||||
curtsies==0.4.3
|
||||
cwcwidth==0.1.10
|
||||
Django==5.2.5
|
||||
django-admin-sortable2==2.2.8
|
||||
django-js-asset==3.1.2
|
||||
django-mptt==0.17.0
|
||||
django-mptt-admin==2.8.0
|
||||
|
||||
@@ -1,15 +1,40 @@
|
||||
/* Style each Vorgabe inline block */
|
||||
.djn-dynamic-form-Standards-vorgabe {
|
||||
border: 2px solid #ccc;
|
||||
.djn-dynamic-form-Standards-vorgabe,
|
||||
.djn-dynamic-form-dokumente-vorgabe {
|
||||
border: 3px solid #2c5aa0;
|
||||
border-radius: 8px;
|
||||
padding: 15px;
|
||||
margin-bottom: 20px;
|
||||
background-color: #f9f9f9;
|
||||
margin-bottom: 50px;
|
||||
box-shadow: 0 2px 5px rgba(0,0,0,0.1);
|
||||
}
|
||||
|
||||
/* Make Vorgabe title prominent */
|
||||
.djn-dynamic-form-Standards-vorgabe > h3,
|
||||
.djn-dynamic-form-dokumente-vorgabe > h3 {
|
||||
font-size: 18px;
|
||||
font-weight: 700;
|
||||
color: #2c5aa0;
|
||||
margin: -15px -15px 15px -15px;
|
||||
padding: 12px 15px;
|
||||
background: linear-gradient(to bottom, #e8f0f8, #d4e4f3);
|
||||
border-bottom: 2px solid #2c5aa0;
|
||||
border-radius: 5px 5px 0 0;
|
||||
}
|
||||
|
||||
/* Make Vorgabe identifier in tabular view prominent */
|
||||
tbody.djn-dynamic-form-Standards-vorgabe td.original,
|
||||
tbody.djn-dynamic-form-dokumente-vorgabe td.original,
|
||||
tbody.djn-dynamic-form-Standards-vorgabe td.original p,
|
||||
tbody.djn-dynamic-form-dokumente-vorgabe td.original p {
|
||||
font-size: 16px !important;
|
||||
font-weight: 700 !important;
|
||||
color: #2c5aa0 !important;
|
||||
}
|
||||
|
||||
/* Optional: Slight padding for inner fieldsets (e.g., Langtext/Kurztext inlines) */
|
||||
.djn-dynamic-form-Standards-vorgabe .inline-related {
|
||||
.djn-dynamic-form-Standards-vorgabe .inline-related,
|
||||
.djn-dynamic-form-dokumente-vorgabe .inline-related {
|
||||
margin-top: 10px;
|
||||
padding-left: 10px;
|
||||
border-left: 2px dashed #ccc;
|
||||
}
|
||||
}
|
||||
|
||||
@@ -1,21 +1,58 @@
|
||||
window.addEventListener('load', function () {
|
||||
setTimeout(() => {
|
||||
const vorgabenBlocks = document.querySelectorAll('.djn-dynamic-form-Standards-vorgabe');
|
||||
console.log("Found", vorgabenBlocks.length, "Vorgaben blocks");
|
||||
// Try different selectors for nested admin vorgabe elements
|
||||
const selectors = [
|
||||
'.djn-dynamic-form-dokumente-vorgabe',
|
||||
'.djn-dynamic-form-Standards-vorgabe',
|
||||
'.inline-related[data-inline-type="stacked"]',
|
||||
'.nested-inline'
|
||||
];
|
||||
|
||||
let vorgabenBlocks = [];
|
||||
for (const selector of selectors) {
|
||||
vorgabenBlocks = document.querySelectorAll(selector);
|
||||
if (vorgabenBlocks.length > 0) {
|
||||
console.log("Found", vorgabenBlocks.length, "Vorgaben blocks with selector:", selector);
|
||||
break;
|
||||
}
|
||||
}
|
||||
|
||||
if (vorgabenBlocks.length === 0) {
|
||||
console.log("No Vorgaben blocks found, trying fallback...");
|
||||
// Fallback: look for any inline with vorgabe in the class
|
||||
vorgabenBlocks = document.querySelectorAll('[class*="vorgabe"]');
|
||||
}
|
||||
|
||||
vorgabenBlocks.forEach((block, index) => {
|
||||
const header = document.createElement('div');
|
||||
header.className = 'vorgabe-toggle-header';
|
||||
header.innerHTML = `▼ Vorgabe ${index + 1}`;
|
||||
header.style.cursor = 'pointer';
|
||||
|
||||
block.parentNode.insertBefore(header, block);
|
||||
|
||||
header.addEventListener('click', () => {
|
||||
const isHidden = block.style.display === 'none';
|
||||
block.style.display = isHidden ? '' : 'none';
|
||||
header.innerHTML = `${isHidden ? '▼' : '▶'} Vorgabe ${index + 1}`;
|
||||
});
|
||||
// Find the existing title/header within the vorgabe block
|
||||
const existingHeader = block.querySelector('h3, .inline-label, .module h2, .djn-inline-header');
|
||||
|
||||
if (existingHeader) {
|
||||
// Make the existing header clickable for collapse/expand
|
||||
existingHeader.style.cursor = 'pointer';
|
||||
existingHeader.addEventListener('click', (e) => {
|
||||
e.preventDefault();
|
||||
e.stopPropagation();
|
||||
|
||||
// Find all content to collapse - everything except the header itself
|
||||
const allChildren = Array.from(block.children);
|
||||
const contentElements = allChildren.filter(child => child !== existingHeader && !child.contains(existingHeader));
|
||||
|
||||
contentElements.forEach(element => {
|
||||
const isHidden = element.style.display === 'none';
|
||||
element.style.display = isHidden ? '' : 'none';
|
||||
});
|
||||
|
||||
// Update the header text to show collapse state
|
||||
const originalText = existingHeader.textContent.replace(/[▼▶]\s*/, '');
|
||||
const anyHidden = contentElements.some(el => el.style.display === 'none');
|
||||
existingHeader.innerHTML = `${anyHidden ? '▶' : '▼'} ${originalText}`;
|
||||
});
|
||||
|
||||
// Add initial collapse indicator
|
||||
const originalText = existingHeader.textContent;
|
||||
existingHeader.innerHTML = `▼ ${originalText}`;
|
||||
}
|
||||
});
|
||||
}, 500); // wait 500ms to allow nested inlines to render
|
||||
}, 1000); // wait longer to allow nested inlines to render
|
||||
});
|
||||
|
||||
38
test_sanity_check.py
Normal file
38
test_sanity_check.py
Normal file
@@ -0,0 +1,38 @@
|
||||
#!/usr/bin/env python
|
||||
"""
|
||||
Simple script to test Vorgaben sanity checking
|
||||
"""
|
||||
import os
|
||||
import sys
|
||||
import django
|
||||
|
||||
# Setup Django
|
||||
os.environ.setdefault('DJANGO_SETTINGS_MODULE', 'VorgabenUI.settings')
|
||||
django.setup()
|
||||
|
||||
from dokumente.utils import check_vorgabe_conflicts, format_conflict_report
|
||||
|
||||
|
||||
def main():
|
||||
print("Running Vorgaben sanity check...")
|
||||
print("=" * 50)
|
||||
|
||||
# Check for conflicts
|
||||
conflicts = check_vorgabe_conflicts()
|
||||
|
||||
# Generate and display report
|
||||
report = format_conflict_report(conflicts, verbose=True)
|
||||
print(report)
|
||||
|
||||
print("=" * 50)
|
||||
|
||||
if conflicts:
|
||||
print(f"\n⚠️ Found {len(conflicts)} conflicts that need attention!")
|
||||
sys.exit(1)
|
||||
else:
|
||||
print("✅ All Vorgaben are valid!")
|
||||
sys.exit(0)
|
||||
|
||||
|
||||
if __name__ == "__main__":
|
||||
main()
|
||||
Reference in New Issue
Block a user