Skip to content

Commit

Permalink
fixed bugs, missing translations added
Browse files Browse the repository at this point in the history
  • Loading branch information
4-dash committed Nov 22, 2024
1 parent b6ce10c commit 081e228
Show file tree
Hide file tree
Showing 7 changed files with 322 additions and 262 deletions.
2 changes: 1 addition & 1 deletion src/feedback/admin.py
Original file line number Diff line number Diff line change
Expand Up @@ -245,7 +245,7 @@ def save_related(self, request, form, formsets, change):
person.save()
count_added += 1
if count_added > 0:
self.message_user(request, _(f"Dieses Fachgebiet wurde {count_added} Personen zugeordnet"))
self.message_user(request, _("Dieses Fachgebiet wurde {count_added} Personen zugeordnet").format(count_added=count_added))


class FragebogenAdmin(admin.ModelAdmin):
Expand Down
3 changes: 2 additions & 1 deletion src/feedback/forms.py
Original file line number Diff line number Diff line change
Expand Up @@ -54,9 +54,10 @@ def __init__(self, *args, **kwargs):
self.fields["ergebnis_empfaenger"].queryset = veranstalter_queryset

# Keine negative Anzahl möglich
self.fields["anzahl"] = forms.IntegerField(min_value=1)
self.fields["anzahl"] = forms.IntegerField(label=_("Anzahl"), min_value=1)

self.fields["auswertungstermin"] = forms.DateField(
label=_("Auswertungstermin"),
help_text=_("Zu diesem Termin werden die Ergebnisse versandt. Nach diesem Datum können keine Evaluationsbögen mehr abgegeben werden und die digitale Evaluation geschlossen."),
widget=forms.DateInput(attrs={"type": "date", "value": Semester.current().standard_ergebnisversand}),
)
Expand Down
28 changes: 11 additions & 17 deletions src/feedback/models/base.py
Original file line number Diff line number Diff line change
Expand Up @@ -36,18 +36,12 @@ class Semester(models.Model):

semester = models.IntegerField(help_text='Aufbau: YYYYS, wobei YYYY = Jahreszahl und S = Semester (0=SS, 5=WS).',
unique=True)
fragebogen = models.CharField(max_length=5, choices=FRAGEBOGEN_CHOICES,
fragebogen = models.CharField(max_length=5, choices=FRAGEBOGEN_CHOICES, verbose_name=_("Fragebogen"),
help_text=_('Verwendete Version des Fragebogens.'))
sichtbarkeit = models.CharField(max_length=3, choices=SICHTBARKEIT_CHOICES,
help_text='Sichtbarkeit der Evaluationsergebnisse.<br /><em>' +
SICHTBARKEIT_CHOICES[0][1] +
':</em> nur für Mitglieder des Feedback-Teams<br /><em>' +
SICHTBARKEIT_CHOICES[1][1] +
':</em> Veranstalter und Mitglieder des Feedback-Teams<br /><em>' +
SICHTBARKEIT_CHOICES[2][1] +
':</em> alle (beschränkt auf das Uninetz)<br />'
sichtbarkeit = models.CharField(max_length=3, choices=SICHTBARKEIT_CHOICES, verbose_name=_("Sichtbarkeit"),
help_text=_('Sichtbarkeit der Evaluationsergebnisse.<br /><em>{s1}:</em> nur für Mitglieder des Feedback-Teams<br /><em>{s2}:</em> Veranstalter und Mitglieder des Feedback-Teams<br /><em>{s3}:</em> alle (beschränkt auf das Uninetz)<br />').format(s1=SICHTBARKEIT_CHOICES[0][1],s2=SICHTBARKEIT_CHOICES[1][1],s3=SICHTBARKEIT_CHOICES[2][1])
)
vollerhebung = models.BooleanField(default=False)
vollerhebung = models.BooleanField(default=False, verbose_name=_("Vollerhebung"))
standard_ergebnisversand = models.DateField(null=True, blank=True, verbose_name=_('Ergebnisversand'), help_text=_('Standarddatum für den Ergebnisversand'))

def _format_generic(self, ss, ws, space, modulus):
Expand Down Expand Up @@ -187,14 +181,14 @@ class Person(models.Model):
'w': 'f',
}

geschlecht = models.CharField(max_length=1, choices=GESCHLECHT_CHOICES, blank=True, verbose_name='Anrede')
geschlecht = models.CharField(max_length=1, choices=GESCHLECHT_CHOICES, blank=True, verbose_name=_('Anrede'))
vorname = models.CharField(_('first name'), max_length=30, blank=True)
nachname = models.CharField(_('last name'), max_length=30, blank=True)
email = models.EmailField(_('E-Mail'), blank=True)
anschrift = models.CharField(_('anschrift'), max_length=80, blank=True,
help_text=_('Tragen Sie bitte nur die Anschrift ohne Namen ein, '
'da der Name automatisch hinzugefügt wird.'))
fachgebiet = models.ForeignKey(Fachgebiet, null=True, blank=True, on_delete=models.CASCADE)
fachgebiet = models.ForeignKey(Fachgebiet, verbose_name=_("Fachgebiet"), null=True, blank=True, on_delete=models.CASCADE)

def full_name(self):
return '%s %s' % (self.vorname, self.nachname)
Expand Down Expand Up @@ -437,7 +431,7 @@ class Veranstaltung(models.Model):
vlNoEx = _('Wenn Ihre Vorlesung keine Übung hat wählen Sie bitte <i>%s</i> aus')
for cur in TYP_CHOICES:
if cur[0] == 'v':
vlNoEx = vlNoEx % cur[1]
vlNoEx = _('Wenn Ihre Vorlesung keine Übung hat wählen Sie bitte <i>{s}</i> aus').format(s=cur[1])
break

typ = models.CharField(verbose_name=_("Typ"), max_length=2, choices=TYP_CHOICES, help_text=vlNoEx)
Expand Down Expand Up @@ -631,7 +625,7 @@ def log(self, interface, is_frontend=False):

def auwertungstermin_to_late_msg(self):
toLateDate = self.semester.last_Auswertungstermin_to_late_human()
return _(f'Der Auswertungstermin muss vor dem {toLateDate} liegen.')
return _('Der Auswertungstermin muss vor dem {toLateDate} liegen.').format(toLateDate=toLateDate)

def has_uebung(self):
"""Gibt True zurück wenn die Veranstaltung eine Übung hat sonst False"""
Expand All @@ -647,7 +641,7 @@ def veranstalter_list(self):


def anzahl_too_few_msg(self) :
return _(f'Anzahl der Bestellungen muss mindestens {self.MIN_BESTELLUNG_ANZAHL} sein. Bei weniger als {self.MIN_BESTELLUNG_ANZAHL} Teilnehmenden ist eine Evaluation leider nicht möglich')
return _('Anzahl der Bestellungen muss mindestens {MIN_BESTELLUNG_ANZAHL} sein. Bei weniger als {MIN_BESTELLUNG_ANZAHL} Teilnehmenden ist eine Evaluation leider nicht möglich').format(MIN_BESTELLUNG_ANZAHL=self.MIN_BESTELLUNG_ANZAHL)


def clean(self, *args, **kwargs):
Expand Down Expand Up @@ -724,8 +718,8 @@ class Tutor(models.Model):
vorname = models.CharField(_('first name'), max_length=30)
nachname = models.CharField(_('last name'), max_length=30)
email = models.EmailField(_('e-mail address'))
anmerkung = models.CharField(max_length=100)
veranstaltung = models.ForeignKey(Veranstaltung, on_delete=models.CASCADE)
anmerkung = models.CharField(max_length=100, verbose_name=_("Anmerkung"))
veranstaltung = models.ForeignKey(Veranstaltung, verbose_name=_("Veranstaltung"), on_delete=models.CASCADE)

def get_barcode_number(self):
"""Gibt die Barcodenummer anhand der Tutorennummer zurück."""
Expand Down
2 changes: 1 addition & 1 deletion src/feedback/views/veranstalter.py
Original file line number Diff line number Diff line change
Expand Up @@ -288,7 +288,7 @@ def send_mail_to_verantwortliche(ergebnis_empfaenger, context, veranstaltung):
if ergebnis_empfaenger is not None:
for e in ergebnis_empfaenger:
send_mail(
_(f'Evaluation der Lehrveranstaltungen - Zusammenfassung der Daten für {veranstaltung.name}'),
_('Evaluation der Lehrveranstaltungen - Zusammenfassung der Daten für {name}').format(name=veranstaltung.name),
_('Nachfolgend finder Sie Informationen zu Ihrer Bestellung'),
settings.DEFAULT_FROM_EMAIL,
[e.email],
Expand Down
Loading

0 comments on commit 081e228

Please sign in to comment.