JavaScript: Doppelte Funktionen & toter Code bereinigen #39

New Issue

greggy · 2026-05-19T14:40:24+02:00

greggy commented

2026-05-19 14:40:24 +02:00

Problem

Aus der Schwachstellenanalyse ergeben sich zwei zusammenhängende JS-Probleme in js/haus-schleusingen.js:

1. Doppelte Funktionsdefinitionen

openLightbox() und closeLightbox() werden zweimal definiert – einmal mit jQuery, einmal vanilla. Die zweite Definition überschreibt die erste.

2. Toter Code im Keyboard-Handler

Außerhalb jedes Event-Listeners steht ein Block, der einen ReferenceError wirft:

if (e.key === "Enter" || e.key === " ") {
  e.preventDefault();
  $(this).trigger("click");
}

e und $(this) sind hier nicht definiert.

Lösung

Doppelte openLightbox()/closeLightbox() Definitionen zusammenführen (nur Vanilla-Version behalten)
Toten Code-Block entfernen
Sicherstellen, dass alle Event-Listener korrekt funktionieren

Akzeptanzkriterien

Keine doppelten Funktionsdefinitionen mehr
Keine ReferenceError in der Console
Lightbox funktioniert weiterhin (Klick + Keyboard)
Floor-Accordion funktioniert weiterhin

## Problem Aus der Schwachstellenanalyse ergeben sich zwei zusammenhängende JS-Probleme in `js/haus-schleusingen.js`: ### 1. Doppelte Funktionsdefinitionen `openLightbox()` und `closeLightbox()` werden **zweimal** definiert – einmal mit jQuery, einmal vanilla. Die zweite Definition überschreibt die erste. ### 2. Toter Code im Keyboard-Handler Außerhalb jedes Event-Listeners steht ein Block, der einen ReferenceError wirft: ```javascript if (e.key === "Enter" || e.key === " ") { e.preventDefault(); $(this).trigger("click"); } ``` `e` und `$(this)` sind hier nicht definiert. ## Lösung - Doppelte `openLightbox()`/`closeLightbox()` Definitionen zusammenführen (nur Vanilla-Version behalten) - Toten Code-Block entfernen - Sicherstellen, dass alle Event-Listener korrekt funktionieren ## Akzeptanzkriterien - [ ] Keine doppelten Funktionsdefinitionen mehr - [ ] Keine ReferenceError in der Console - [ ] Lightbox funktioniert weiterhin (Klick + Keyboard) - [ ] Floor-Accordion funktioniert weiterhin

greggy referenced this issue from a commit

2026-05-19 14:42:10 +02:00

fix: remove duplicate openLightbox/closeLightbox and dead code (#39)

greggy referenced a pull request that will close this issue

2026-05-19 14:55:46 +02:00

fix: JavaScript doppelte Funktionen & toter Code (#39) #40

greggy closed this issue

2026-05-19 15:13:30 +02:00

greggy referenced this issue from a commit

2026-05-19 15:13:31 +02:00

Merge pull request 'fix: JavaScript doppelte Funktionen & toter Code (#39)' (#40) from feature/39-js-duplicate-functions-fix into main

Sign in to join this conversation.