Bug 26540 - Form: clearFormValues() sollte auch _loadedID zurücksetzen
Form: clearFormValues() sollte auch _loadedID zurücksetzen
Status: CLOSED FIXED
Product: UCS
Classification: Unclassified
Component: UMC (Generic)
UNSTABLE
Other Linux
: P3 normal (vote)
: UCS 3.0-2
Assigned To: Jascha Geerds
Andreas Büsching
: interim-1
Depends on: 26539
Blocks:
  Show dependency treegraph
 
Reported: 2012-03-16 13:08 CET by Jascha Geerds
Modified: 2012-07-20 15:24 CEST (History)
2 users (show)

See Also:
What kind of report is it?: ---
What type of bug is this?: ---
Who will be affected by this bug?: ---
How will those affected feel about the bug?: ---
User Pain:
Enterprise Customer affected?:
School Customer affected?:
ISV affected?:
Waiting Support:
Flags outvoted (downgraded) after PO Review:
Ticket number:
Bug group (optional):
Max CVSS v3 score:


Attachments
Patch (350 bytes, patch)
2012-03-16 14:03 CET, Jascha Geerds
Details | Diff
Patch (352 bytes, patch)
2012-03-16 17:33 CET, Jascha Geerds
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Jascha Geerds univentionstaff 2012-03-16 13:08:34 CET
+++ This bug was initially created as a clone of Bug #26539 +++

Aktuell ist es so, dass beim Aufruf von form.clearFormValues() lediglich alle
Widgets zurückgesetzt werden. Der Wert "_loadedID" wird jedoch nicht mit
zurückgesetzt. Dies kann dazu führen, dass ggf. der Wert von einem vorherigen
Objekt gespeichert bleibt.

Ich denke es wäre gut wenn die clearFormValues-Funktion auch _loadedID
zurücksetzt.

---
Das sollte auch in UCS geändert werden.
Comment 1 Jascha Geerds univentionstaff 2012-03-16 14:03:56 CET
Created attachment 4275 [details]
Patch

Hier der Fix
Comment 2 Jascha Geerds univentionstaff 2012-03-16 17:33:50 CET
Created attachment 4276 [details]
Patch

Der Patch wurde noch einmal überarbeitet.
Comment 3 Jascha Geerds univentionstaff 2012-06-04 12:21:07 CEST
Der Patch wurde in abgewandelter Form übernommen. Das Paket baut gerade.
Comment 4 Andreas Büsching univentionstaff 2012-06-18 15:12:36 CEST
Der Patch ist drin.

if (this._loadedID) {
	this._loadedID = null;
}

Kein ChangeLog-Eintrag vorhanden
Comment 5 Alexander Kläser univentionstaff 2012-06-19 13:42:09 CEST
(In reply to comment #4)
> ...
> Kein ChangeLog-Eintrag vorhanden

Der wurde ergänzt.
Comment 6 Andreas Büsching univentionstaff 2012-06-19 13:52:39 CEST
(In reply to comment #5)
> (In reply to comment #4)
> > ...
> > Kein ChangeLog-Eintrag vorhanden
> 
> Der wurde ergänzt.

Korrekt
Comment 7 Stefan Gohmann univentionstaff 2012-07-20 15:24:46 CEST
UCS 3.0-2 has been released: 
  http://forum.univention.de/viewtopic.php?f=54&t=1905

If this error occurs again, please use "Clone This Bug".