Skip to content

Conversation

janbritz
Copy link
Contributor

Closes: #177

@janbritz janbritz requested review from MartinGauk and MHajoha May 20, 2025 12:41
Copy link
Contributor

@MartinGauk MartinGauk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Das ursächliche Problem ist, dass die Funktion changesMadeCheck ("Prevent user navigate away when upload progress still running.") ebenfalls als beforeunload Event Listener aufgerufen wird. Aus Moodle-Sicht ist der Upload also noch nicht ganz abgeschlossen.

Ich bin mir nicht sicher, ob der Upload tatsächlich noch nicht abgeschlossen ist oder ob nur noch nicht alle Event-Handler aufgerufen wurden (ich vermute Letzteres). Wir befinden uns hier ja selber in einem Event-Handler. Es gibt auch ein uploadCompleted Event, das aber nur bei DnD geworfen wird und nicht wenn eine Datei über das Upload-Modal-Formular hochgeladen wird.

Ich würde sagen, dass du das Formular-Abschicken per setTimeout um 100 ms verzögerst. Auch wenn die 100 ms aus der Luft gegriffen sind, finde ich das etwas sauberer, als die Verarbeitung von beforeunload zu stoppen. Falls der Uploadvorgang immer noch nicht abgeschlossen sein sollte, geht es zumindest nicht unter, dass irgendwas das Abschicken noch verhindert.

@janbritz janbritz force-pushed the feat/upload-package-via-dnd-and-no-warning branch from b24b2b4 to e18e227 Compare June 10, 2025 13:51
@janbritz janbritz requested a review from MartinGauk June 10, 2025 13:57
@janbritz janbritz merged commit 90534ed into dev Jun 11, 2025
6 checks passed
@janbritz janbritz deleted the feat/upload-package-via-dnd-and-no-warning branch June 11, 2025 10:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Paket hochladen per Drag-and-Drop ohne Warnung
3 participants