feat: added answer field
This commit is contained in:
@@ -1,5 +1,6 @@
|
|||||||
from django import forms
|
from django import forms
|
||||||
from .models import Comment, Ticket
|
from .models import Comment, Ticket
|
||||||
|
from django.core.exceptions import ValidationError
|
||||||
|
|
||||||
|
|
||||||
class CommentForm(forms.ModelForm):
|
class CommentForm(forms.ModelForm):
|
||||||
@@ -16,7 +17,26 @@ class CommentForm(forms.ModelForm):
|
|||||||
class TicketForm(forms.ModelForm):
|
class TicketForm(forms.ModelForm):
|
||||||
class Meta:
|
class Meta:
|
||||||
model = Ticket
|
model = Ticket
|
||||||
fields = ["title", "description", "status", "priority", "course"]
|
fields = ["title", "description", "status", "priority", "course", "answer"]
|
||||||
|
widgets = {
|
||||||
|
'answer': forms.Textarea(attrs={
|
||||||
|
'rows': 4,
|
||||||
|
'placeholder': 'Beschreibe die Lösung des Problems...'
|
||||||
|
})
|
||||||
|
}
|
||||||
|
|
||||||
|
def clean(self):
|
||||||
|
cleaned_data = super().clean()
|
||||||
|
status = cleaned_data.get('status')
|
||||||
|
answer = cleaned_data.get('answer')
|
||||||
|
|
||||||
|
# Wenn Status auf "gelöst" gesetzt wird, muss eine Antwort vorhanden sein
|
||||||
|
if status == 'resolved' and not answer:
|
||||||
|
raise ValidationError({
|
||||||
|
'answer': 'Eine Antwort ist erforderlich, wenn der Status auf "Gelöst" gesetzt wird.'
|
||||||
|
})
|
||||||
|
|
||||||
|
return cleaned_data
|
||||||
|
|
||||||
def save(self, commit=True):
|
def save(self, commit=True):
|
||||||
ticket = super().save(commit=False)
|
ticket = super().save(commit=False)
|
||||||
@@ -25,6 +45,11 @@ class TicketForm(forms.ModelForm):
|
|||||||
if ticket.course and ticket.course.tutor:
|
if ticket.course and ticket.course.tutor:
|
||||||
ticket.assigned_to = ticket.course.tutor
|
ticket.assigned_to = ticket.course.tutor
|
||||||
|
|
||||||
|
# Setze answered_at wenn eine Antwort gegeben wird
|
||||||
|
if ticket.answer and not ticket.answered_at:
|
||||||
|
from django.utils import timezone
|
||||||
|
ticket.answered_at = timezone.now()
|
||||||
|
|
||||||
if commit:
|
if commit:
|
||||||
ticket.save()
|
ticket.save()
|
||||||
return ticket
|
return ticket
|
||||||
@@ -51,6 +51,18 @@ class Ticket(models.Model):
|
|||||||
status = models.CharField(max_length=20, choices=STATUS_CHOICES, default="new")
|
status = models.CharField(max_length=20, choices=STATUS_CHOICES, default="new")
|
||||||
priority = models.CharField(max_length=10, choices=PRIORITY_CHOICES, default="medium")
|
priority = models.CharField(max_length=10, choices=PRIORITY_CHOICES, default="medium")
|
||||||
|
|
||||||
|
answer = models.TextField(
|
||||||
|
blank=True,
|
||||||
|
null=True,
|
||||||
|
verbose_name="Antwort/Lösung",
|
||||||
|
help_text="Beschreibung der Lösung (erforderlich bei Status 'Gelöst')"
|
||||||
|
)
|
||||||
|
answered_at = models.DateTimeField(
|
||||||
|
blank=True,
|
||||||
|
null=True,
|
||||||
|
verbose_name="Beantwortet am"
|
||||||
|
)
|
||||||
|
|
||||||
course = models.ForeignKey(
|
course = models.ForeignKey(
|
||||||
Course,
|
Course,
|
||||||
on_delete=models.CASCADE,
|
on_delete=models.CASCADE,
|
||||||
|
|||||||
@@ -30,7 +30,7 @@
|
|||||||
</div>
|
</div>
|
||||||
</nav>
|
</nav>
|
||||||
<!-- Global Container -->
|
<!-- Global Container -->
|
||||||
<div class="max-w-4xl mx-auto mt-8 px-4">
|
<div class="max-w-5xl mx-auto mt-8 px-4">
|
||||||
{% block content %}{% endblock %}
|
{% block content %}{% endblock %}
|
||||||
</div>
|
</div>
|
||||||
</body>
|
</body>
|
||||||
|
|||||||
@@ -2,7 +2,7 @@
|
|||||||
{% block content %}
|
{% block content %}
|
||||||
<!-- Messages -->
|
<!-- Messages -->
|
||||||
{% if messages %}
|
{% if messages %}
|
||||||
<div class="max-w-4xl mx-auto px-4 pt-4">
|
<div class="max-w-6xl mx-auto px-4 pt-4">
|
||||||
{% for message in messages %}
|
{% for message in messages %}
|
||||||
<div class="mb-4 p-3 rounded {% if message.tags == 'error' %}bg-red-100 text-red-700{% elif message.tags == 'success' %}bg-green-100 text-green-700{% else %}bg-yellow-100 text-yellow-700{% endif %}">
|
<div class="mb-4 p-3 rounded {% if message.tags == 'error' %}bg-red-100 text-red-700{% elif message.tags == 'success' %}bg-green-100 text-green-700{% else %}bg-yellow-100 text-yellow-700{% endif %}">
|
||||||
{{ message }}
|
{{ message }}
|
||||||
@@ -10,7 +10,7 @@
|
|||||||
{% endfor %}
|
{% endfor %}
|
||||||
</div>
|
</div>
|
||||||
{% endif %}
|
{% endif %}
|
||||||
<div class="max-w-4xl mx-auto px-4 py-6">
|
<div class="max-w-6xl mx-auto px-4 py-6">
|
||||||
<!-- Ticket Bearbeitung -->
|
<!-- Ticket Bearbeitung -->
|
||||||
<div class="bg-white rounded-lg shadow p-6 mb-6">
|
<div class="bg-white rounded-lg shadow p-6 mb-6">
|
||||||
<h1 class="text-3xl font-bold mb-4">🎫 Ticket #{{ ticket.id }} – {{ ticket.title }}</h1>
|
<h1 class="text-3xl font-bold mb-4">🎫 Ticket #{{ ticket.id }} – {{ ticket.title }}</h1>
|
||||||
@@ -71,13 +71,50 @@
|
|||||||
</select>
|
</select>
|
||||||
</div>
|
</div>
|
||||||
</div>
|
</div>
|
||||||
|
<!-- Zugewiesen an (Nur Anzeige) -->
|
||||||
<div>
|
<div>
|
||||||
<label class="block text-sm font-medium mb-1">Zugewiesen an:</label>
|
<label class="block text-sm font-medium mb-1">Zugewiesen an:</label>
|
||||||
<div id="tutor_display" class="w-full p-2 border border-gray-300 bg-gray-100 rounded shadow-sm text-gray-300">
|
<div id="tutor_display" class="w-full p-2 border border-gray-300 bg-gray-300 rounded shadow-sm">
|
||||||
<span id="tutor_text">{% if ticket.assigned_to %}{{ ticket.assigned_to.username }}{% else %}Niemand zugewiesen{% endif %}</span>
|
<span id="tutor_text">{% if ticket.assigned_to %}{{ ticket.assigned_to.username }}{% else %}Niemand zugewiesen{% endif %}</span>
|
||||||
</div>
|
</div>
|
||||||
<p class="text-xs text-gray-500 mt-1">Wird automatisch basierend auf dem ausgewählten Kurs zugewiesen</p>
|
<p class="text-xs text-gray-500 mt-1">Wird automatisch basierend auf dem ausgewählten Kurs zugewiesen</p>
|
||||||
</div>
|
</div>
|
||||||
|
|
||||||
|
<!-- Antwort/Lösung -->
|
||||||
|
<div>
|
||||||
|
<label class="block text-sm font-medium mb-1">
|
||||||
|
Antwort/Lösung:
|
||||||
|
{% if form.status.value == 'resolved' or ticket.status == 'resolved' %}
|
||||||
|
<span class="text-red-500">*</span>
|
||||||
|
{% endif %}
|
||||||
|
</label>
|
||||||
|
{% if ticket.answer and not view.can_edit %}
|
||||||
|
<!-- Nur Anzeige wenn bereits beantwortet und nicht editierbar -->
|
||||||
|
<div class="w-full p-3 border border-gray-300 bg-gray-50 rounded shadow-sm">
|
||||||
|
<div class="text-gray-700">{{ ticket.answer|linebreaks }}</div>
|
||||||
|
{% if ticket.answered_at %}
|
||||||
|
<div class="text-xs text-gray-500 mt-2">
|
||||||
|
Beantwortet am: {{ ticket.answered_at|date:"d.m.Y H:i" }}
|
||||||
|
</div>
|
||||||
|
{% endif %}
|
||||||
|
</div>
|
||||||
|
{% else %}
|
||||||
|
<!-- Bearbeitbares Feld -->
|
||||||
|
<textarea name="answer"
|
||||||
|
rows="4"
|
||||||
|
{% if not view.can_edit or ticket.status != 'resolved' %}disabled{% endif %}
|
||||||
|
placeholder="Beschreibe die Lösung des Problems..."
|
||||||
|
class="w-full p-2 border border-gray-300 rounded shadow-sm focus:outline-none focus:ring-2 focus:ring-blue-500 focus:border-blue-500 placeholder:text-black {% if not view.can_edit or ticket.status != 'resolved' %}bg-gray-300 cursor-not-allowed{% endif %}">{{ ticket.answer|default:'' }}</textarea>
|
||||||
|
{% if form.answer.errors %}
|
||||||
|
<div class="text-red-600 text-sm mt-1">{{ form.answer.errors }}</div>
|
||||||
|
{% endif %}
|
||||||
|
<p class="text-xs text-gray-500 mt-1">
|
||||||
|
{% if ticket.status != 'resolved' %}
|
||||||
|
Eine Antwort ist erforderlich beim Setzen des Status auf "Gelöst"
|
||||||
|
{% endif %}
|
||||||
|
</p>
|
||||||
|
{% endif %}
|
||||||
|
</div>
|
||||||
{% if view.can_edit %}
|
{% if view.can_edit %}
|
||||||
<button type="submit"
|
<button type="submit"
|
||||||
class="bg-blue-500 hover:bg-blue-600 text-white px-4 py-2 rounded">💾 Speichern</button>
|
class="bg-blue-500 hover:bg-blue-600 text-white px-4 py-2 rounded">💾 Speichern</button>
|
||||||
@@ -178,5 +215,40 @@
|
|||||||
});
|
});
|
||||||
{% endif %}
|
{% endif %}
|
||||||
});
|
});
|
||||||
|
|
||||||
|
// Answer Feld nur bei Status "resolved" aktivieren
|
||||||
|
{% if view.can_edit %}
|
||||||
|
document.addEventListener('DOMContentLoaded', function() {
|
||||||
|
const statusSelect = document.querySelector('select[name="status"]');
|
||||||
|
const answerField = document.querySelector('textarea[name="answer"]');
|
||||||
|
const answerLabel = answerField.previousElementSibling;
|
||||||
|
|
||||||
|
function toggleAnswerField() {
|
||||||
|
if (statusSelect.value === 'resolved') {
|
||||||
|
answerField.disabled = false;
|
||||||
|
answerField.classList.remove('bg-gray-100', 'cursor-not-allowed');
|
||||||
|
answerField.classList.add('bg-white');
|
||||||
|
answerField.required = true;
|
||||||
|
// Pflichtfeld-Stern anzeigen
|
||||||
|
if (!answerLabel.querySelector('.text-red-500')) {
|
||||||
|
answerLabel.innerHTML = answerLabel.textContent + ' <span class="text-red-500">*</span>';
|
||||||
|
}
|
||||||
|
} else {
|
||||||
|
answerField.disabled = true;
|
||||||
|
answerField.classList.add('bg-gray-100', 'cursor-not-allowed');
|
||||||
|
answerField.classList.remove('bg-white');
|
||||||
|
answerField.required = false;
|
||||||
|
// Pflichtfeld-Stern entfernen
|
||||||
|
answerLabel.innerHTML = answerLabel.textContent.replace(' *', '');
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
// Initial prüfen
|
||||||
|
toggleAnswerField();
|
||||||
|
|
||||||
|
// Bei Status-Änderung prüfen
|
||||||
|
statusSelect.addEventListener('change', toggleAnswerField);
|
||||||
|
});
|
||||||
|
{% endif %}
|
||||||
</script>
|
</script>
|
||||||
{% endblock %}
|
{% endblock %}
|
||||||
@@ -1,6 +1,6 @@
|
|||||||
{% extends "ticketsystem/base.html" %}
|
{% extends "ticketsystem/base.html" %}
|
||||||
{% block content %}
|
{% block content %}
|
||||||
<div class="max-w-4xl mx-auto px-4 py-6">
|
<div class="max-w-6xl mx-auto px-4 py-6">
|
||||||
<!-- Header -->
|
<!-- Header -->
|
||||||
<div class="bg-white rounded-lg shadow p-6 mb-6">
|
<div class="bg-white rounded-lg shadow p-6 mb-6">
|
||||||
<div class="flex justify-between items-center">
|
<div class="flex justify-between items-center">
|
||||||
|
|||||||
@@ -1,6 +1,6 @@
|
|||||||
{% extends "ticketsystem/base.html" %}
|
{% extends "ticketsystem/base.html" %}
|
||||||
{% block content %}
|
{% block content %}
|
||||||
<div class="max-w-4xl mx-auto px-4 py-12">
|
<div class="max-w-6xl mx-auto px-4 py-12">
|
||||||
<!-- Header -->
|
<!-- Header -->
|
||||||
<div class="text-center mb-12">
|
<div class="text-center mb-12">
|
||||||
<div class="text-6xl mb-4">🎫</div>
|
<div class="text-6xl mb-4">🎫</div>
|
||||||
|
|||||||
@@ -2,7 +2,7 @@
|
|||||||
{% block content %}
|
{% block content %}
|
||||||
<!-- Messages -->
|
<!-- Messages -->
|
||||||
{% if messages %}
|
{% if messages %}
|
||||||
<div class="max-w-4xl mx-auto px-4 pt-4">
|
<div class="max-w-6xl mx-auto px-4 pt-4">
|
||||||
{% for message in messages %}
|
{% for message in messages %}
|
||||||
<div class="mb-4 p-3 rounded {% if message.tags == 'error' %}bg-red-100 text-red-700{% elif message.tags == 'success' %}bg-green-100 text-green-700{% else %}bg-yellow-100 text-yellow-700{% endif %}">
|
<div class="mb-4 p-3 rounded {% if message.tags == 'error' %}bg-red-100 text-red-700{% elif message.tags == 'success' %}bg-green-100 text-green-700{% else %}bg-yellow-100 text-yellow-700{% endif %}">
|
||||||
{{ message }}
|
{{ message }}
|
||||||
@@ -10,7 +10,7 @@
|
|||||||
{% endfor %}
|
{% endfor %}
|
||||||
</div>
|
</div>
|
||||||
{% endif %}
|
{% endif %}
|
||||||
<div class="max-w-4xl mx-auto px-4 py-6">
|
<div class="max-w-6xl mx-auto px-4 py-6">
|
||||||
<!-- Ticket Erstellen -->
|
<!-- Ticket Erstellen -->
|
||||||
<div class="bg-white rounded-lg shadow p-6">
|
<div class="bg-white rounded-lg shadow p-6">
|
||||||
<h1 class="text-3xl font-bold mb-4">➕ Neues Ticket erstellen</h1>
|
<h1 class="text-3xl font-bold mb-4">➕ Neues Ticket erstellen</h1>
|
||||||
|
|||||||
@@ -107,7 +107,7 @@ class TicketDetailUpdateView(UpdateView):
|
|||||||
response = super().form_valid(form) # Speichert das Ticket
|
response = super().form_valid(form) # Speichert das Ticket
|
||||||
|
|
||||||
# History tracking für geänderte Felder
|
# History tracking für geänderte Felder
|
||||||
tracked_fields = ["title", "description", "status", "priority", "course"]
|
tracked_fields = ["title", "description", "status", "priority", "course", "answer"]
|
||||||
for field in tracked_fields:
|
for field in tracked_fields:
|
||||||
if field in form.changed_data:
|
if field in form.changed_data:
|
||||||
old_value = getattr(original, field)
|
old_value = getattr(original, field)
|
||||||
@@ -123,6 +123,15 @@ class TicketDetailUpdateView(UpdateView):
|
|||||||
elif field == "course":
|
elif field == "course":
|
||||||
old_value = str(old_value)
|
old_value = str(old_value)
|
||||||
new_value = str(new_value)
|
new_value = str(new_value)
|
||||||
|
elif field == "answer":
|
||||||
|
if old_value:
|
||||||
|
old_value = old_value[:50] + "..." if len(old_value) > 50 else old_value
|
||||||
|
else:
|
||||||
|
old_value = "Keine Antwort"
|
||||||
|
if new_value:
|
||||||
|
new_value = new_value[:50] + "..." if len(new_value) > 50 else new_value
|
||||||
|
else:
|
||||||
|
new_value = "Keine Antwort"
|
||||||
|
|
||||||
TicketHistory.objects.create(
|
TicketHistory.objects.create(
|
||||||
ticket=ticket,
|
ticket=ticket,
|
||||||
|
|||||||
Reference in New Issue
Block a user