Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

500er Fehler bei Klick auf Spalte (Einstellungen) #1190

Closed
tobiasstern opened this issue Oct 29, 2024 · 5 comments · Fixed by #1193 or hitobito/hitobito_jubla#148 · May be fixed by hitobito/hitobito#3021, hitobito/hitobito_jubla#157 or #1277
Closed
Labels
bug Something isn't working interest-sac SAC-MVP

Comments

@tobiasstern
Copy link
Contributor

Auf diversen UI wird ein Error 500 ausgegeben, wenn man auf einen Spaltentitel klickt.

Betroffen sind:

Ausserdem weitere Anpassungen die nicht direkt als Bug gelten:

  • Das Seitenmenu der Einstellungen alphabetisch ordnen
  • Im Seitenmenu der Einstellungen die Seite aktiv zeigen, welche gerade geladen ist.
  • Auf jeder Seite der Einstellungen die Inhalte per default alphabetisch sortiert nach der ersten Spalte laden (aktuell keine Sortierung)
  • Bei den Kurskategorien https://sac-cas.puzzle.ch/de/event_kind_categories die Spalte Sortierschlüssel auch sortierbar machen
@Largo
Copy link
Contributor

Largo commented Oct 30, 2024

Sentry Link

PG::UndefinedTable: ERROR: missing FROM-clause entry for table "label_format_translations"
LINE 1: ...ranslations"."name" FROM (SELECT label_formats.*, "label_for...

 SELECT * FROM (
    SELECT DISTINCT ON ("label_formats"."id") 
      label_formats.*, 
      "label_format_translations"."name" 
    FROM "label_formats" 
    INNER JOIN "label_format_translations" 
      ON "label_format_translations"."label_format_id" = "label_formats"."id" 
    WHERE "label_format_translations"."locale" = $1
  ) label_formats 
  WHERE "label_formats"."person_id" = $2 
  ORDER BY label_format_translations.name asc NULLS LAST, label_formats.name 
  LIMIT $3

@Largo Largo self-assigned this Oct 30, 2024
Largo added a commit to hitobito/hitobito that referenced this issue Oct 30, 2024
@Largo Largo assigned njaeggi and unassigned Largo Oct 30, 2024
@Largo
Copy link
Contributor

Largo commented Oct 30, 2024

Evtl Zusammenhang mit hitobito/hitobito#2946

@Largo
Copy link
Contributor

Largo commented Oct 31, 2024

@tobiasstern
Copy link
Contributor Author

Erfolgreich getestet.

Die Punkte

- Das Seitenmenu der Einstellungen alphabetisch ordnen
- Im Seitenmenu der Einstellungen die Seite aktiv zeigen, welche gerade geladen ist.
- Auf jeder Seite der Einstellungen die Inhalte per default alphabetisch sortiert nach der ersten Spalte laden (aktuell keine Sortierung)

wurden aus Zeitgründen nicht gemacht.

@tobiasstern tobiasstern removed their assignment Nov 8, 2024
@codez
Copy link
Contributor

codez commented Nov 22, 2024

@tobiasstern

Im Seitenmenu der Einstellungen die Seite aktiv zeigen, welche gerade geladen ist.

Gefixt mit hitobito/hitobito#3022

Das Seitenmenu der Einstellungen alphabetisch ordnen

Das ist bisher bewusst nicht so, die Einträge wurde initial logisch gruppiert. Diese Logik hat sich in den letzten Jahren leider ziemlich aufgeweicht und es wirkt etwas beliebig. Ich habe mal mit hitobito/hitobito#3022 versucht, das wieder etwas zu gruppieren.

Auf jeder Seite der Einstellungen die Inhalte per default alphabetisch sortiert nach der ersten Spalte laden (aktuell keine Sortierung)

Grundsätzlich gefixt mit hitobito/hitobito#3021. Allerdings wurde auch hier bei einigen Entitäten explizit abweichendes Verhalten definiert. Z.B. werden Kursarten standardmässig nach Kurzname geordnet und Kurskategorien nach Sortierschlüssel.

@codez codez linked a pull request Nov 22, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment