hitobito/hitobito_sac_cas

PEOPLE - Firma ohne Firmennamen führt zu 404

Opened this issue · 2 comments

ThomasEllenberger commented

Aktuelles Verhalten

Erstelle ich eine neue Person und setze das Firmen Flag, erfasse jedoch keinen Firmennamen, so führt dies zu einem 404 Fehler.
Problem scheint im Core zu existieren und passiert so auch schon in Version 2.2.2

Gewünschtes Verhalten

Her müsste eine korrekte Fehlermeldung erscheinen, dass Firmen zwingend einen Firmennamen haben müssen.

Schritte zu reproduzieren

  1. Neue Person erstellen mit beliebiger Gruppe/Rolle
  2. Firmenflag anwählen
  3. Feld "Firmenname" leer lassen
  4. Speichern

Wann ist der Fehler aufgetreten?

18.10.2024

hitobito version

2.2.2 & 2.3.0

Organisation

Core?

Screenshots, weiterer Kontext

grafik

ToDos

No response

Largo commented

Fehler liegt hier:

def swappable_role_add_fieldset(*keys)
    title = t("roles.person_fields.text_with_alternative_link_html",
      text: t(".#{keys.last}_person"),
      link: link_to(t(".#{keys.first}_person"), "#", data: {swap: "person-fields"}))

    visible = (keys.first == :create_new) ? !entry.person.changed? : entry.person.changed?
    field_set_tag(title, class: "person-fields", style: element_visible(visible)) { yield }
  end

 =  f.labeled_person_field(:person)

Das @objekt hat keine person, aber es wird versucht diese anzuzeigen

Largo commented

Der Bug ist etwas schwer auffindbar, da es sich vielleicht auch um einen Fehler in ActiveRecord handeln koennte. => Edit: Es handelte sich um einen schwer auffindbaren Fehler, der auftrat, weil to_s nil ansatt wie erwartet einen String zurücklieferte.