Standardbeschreibung bei Kursen wird nicht konsistent behandelt

[nchiapol]

(vgl. auch https://github.com/cevi/hitobito_cevi/issues/101#issuecomment-1169713120)

Ist bei Kurstypen eine Standardbeschreibung hinterlegt und diese wird beim einzelnen Kurs überschrieben, so wird im Webinterface nur die Beschreibung des Kurses angezeigt. Beim PDF der Anmeldung ist dann aber auch die Standardbeschreibung vorhanden.

Das Verhalten sollte konsistent sein. Idealerweise wie im Web (d.h. die Standardbeschreibung sollte im PDF nicht angezeigt werden).

[ninijay]

Hoi zäme

Der Fix ist relativ "easy" - denke auch nicht, dass es was bricht, kenne aber die Code-Base zu wenig, um das sicher zu beurteilen.

Der Change ist lediglich 1 Code-Zeile, jedoch im hito main Projekt. Denke man könnte dies auch auf dem einzelnen cevi-wagon machen, um die anderen Kunden nicht zu "belästigen".

Ich habe den Change hier statt in einem Pull-Request gemacht, da mein Zivi-Einsatz bei Cevi am Montag fertig ist und damit Thomas hier den Lead übernehmen kann.

Change

File: app/domain/export/pdf/participation/general_information.rb Zeilen: 18-21 (ca) Code vorher:

def render_general_information
  return if event.kind.try(:general_information).blank?

  with_header(I18n.t('activerecord.attributes.event/kind.general_information')) do
    text event.kind.general_information
  end
end

Code nachher:

def render_general_information
  # Skip default description/general_information if a custom one is present for the "course"
  return unless event.try(:description).blank?
  return if event.kind.try(:general_information).blank?

  with_header(I18n.t('activerecord.attributes.event/kind.general_information')) do
    text event.kind.general_information
  end
end

Ich habe lediglich die Kommentar Zeile und die Zeile return unless event.try(:description).blank? hinzugefügt, um die Standartbeschreibung nicht anzuzeigen, falls eine "Kurs-Beschreibung" vorhanden ist.

Resultat

Statt einem PDF mit diesem Inhalt

Wird nun je nach Fall (welche Beschreibung ausgefüllt wurde) dieser Inhalt generiert:

Kurs-Art mit Standardbeschreibung

Mit Kurs spezifischer Beschreibung

Ohne Kurs spezifischer Beschreibung

Kurs-Art OHNE Standardbeschreibung

Mit Kurs spezifischer Beschreibung

Ohne Kurs spezifischer Beschreibung

PS: Sorry, falls der Code nicht Top ist - Ruby ist nicht meine Main-Sprache 🤣

[tschuepbach]

Vielen Dank Mattia. Aus meiner Sicht spricht nichts gegen eine Implementierung im Core. Sieht das jemand anders?

Wie sieht die Handhabung bei euch aus @richardjubla @Michael-Schaer?

[Michael-Schaer]

Seitens PBS ist die Änderung in Ordnung. Wir verwenden die Beschreibung in der Kursart bisher kaum.

[richardjubla]

Vielen Dank Mattia. Aus meiner Sicht spricht nichts gegen eine Implementierung im Core. Sieht das jemand anders?

Wie sieht die Handhabung bei euch aus @richardjubla @Michael-Schaer?

Danke für den Hinweis @tschuepbach

Die Kursbeschreibung (event_description) bzw. Standardbeschreibung (event_kind_general_information) wird von uns insbesondere für unser Kursangebot stark genutzt. Ich gehe deshalb davon aus, dass uns diese Anpassung "interessiert" und betrifft. Gerne lasse ich den Change von unserer Kursverwaltung prüfen. Die Kursadministration ist interessiert an dieser Anpassung und möchte diese gerne auch nutzen.

Wir beschäftigen uns (auch) mit dem Informationen zu einem Kurs und dem dazugehörigen PDF. Bei einer Anmeldung für einen Kurs mit Priorisierung möchten wir die entsprechenden Informationen ebenfalls im PDF haben: https://github.com/hitobito/hitobito_jubla/issues/55 Da es sich hier (wohl) ebenfalls um eine Core-Funktion handelt gehe ich vor der Umsetzung/Anpassung von einer Koordination durch Hitobito/Puzzle aus. Ich werde unsere Absichten noch in hitobito/hitobito kommunizieren.

[tschuepbach]

@nchiapol: Hier wäre noch ein Pull request zu erstellen. Wird wohl aber eh nicht vor dem release gemerged..