search

hitobito / hitobito_sac_cas

A hitobito wagon defining the organization hierarchy and additional features for SAC/CAS
Other
3 stars 2 forks source link

PEOPLE - Firma ohne Firmennamen führt zu 404 #1155

Open ThomasEllenberger opened 1 week ago

ThomasEllenberger commented 1 week ago

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 5 days ago

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 4 days ago

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.