Bug 26539 - Form: clearFormValues() sollte auch _loadedID zurücksetzen
Form: clearFormValues() sollte auch _loadedID zurücksetzen
Status: CLOSED FIXED
Product: UCS@school
Classification: Unclassified
Component: UMC
unspecified
Other Linux
: P3 normal (vote)
: UCS@school 3.0 MS2
Assigned To: Jascha Geerds
Florian Best
:
Depends on:
Blocks: 26540
  Show dependency treegraph
 
Reported: 2012-03-16 13:06 CET by Jascha Geerds
Modified: 2012-06-11 06:29 CEST (History)
1 user (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

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:06:03 CET
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.
Comment 1 Jascha Geerds univentionstaff 2012-03-16 13:58:42 CET
Änderungen sind eingecheckt; Paket ist gebaut.
Comment 2 Jascha Geerds univentionstaff 2012-03-16 17:34:39 CET
Der Patch wurde noch einmal überarbeitet.

Änderungen sind eingecheckt; Paket baut gerade.
Comment 3 Florian Best univentionstaff 2012-05-22 13:22:45 CEST
Es fehlt ein Changelog eintrag.

Die Änderung hätte auch einfacher (ohne this.set) gemacht werden können (this._loadedID = null). Das ist aber OK so.
Comment 4 Jascha Geerds univentionstaff 2012-05-22 13:25:56 CEST
Nach Rücksprache hat sich herausgestellt, dass für diesen Bug kein Changelog-Eintrag nötig ist.
Comment 5 Florian Best univentionstaff 2012-05-22 13:28:15 CEST
(In reply to comment #4)
> Nach Rücksprache hat sich herausgestellt, dass für diesen Bug kein
> Changelog-Eintrag nötig ist.
Ok
Comment 6 Stefan Gohmann univentionstaff 2012-06-11 06:29:55 CEST
UCS@school 3.0 wurde veröffentlicht. Sollte der hier beschriebene Bug mit einer  neueren Version von UCS@school erneut auftreten, so sollte dieser Bug dupliziert werden: "Clone This Bug"