Opened 8 years ago

Closed 8 years ago

#7114 closed (fixed)

multiwidget doesn handle has_changed properly

Reported by: Honza Král Owned by: Brian Rosner
Component: Forms Version: newforms-admin
Severity: Keywords: multiwidget has_changed decompress
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:

Description

the patch says it all - _has_changed doesn't call decompress in initial argument

Attachments (1)

patch-7114.diff (564 bytes) - added by Honza Král 8 years ago.

Download all attachments as: .zip

Change History (4)

Changed 8 years ago by Honza Král

Attachment: patch-7114.diff added

comment:1 Changed 8 years ago by Brian Rosner

Needs documentation: unset
Needs tests: unset
Owner: changed from nobody to Brian Rosner
Patch needs improvement: unset
Triage Stage: UnreviewedAccepted

I came across this as well, but there were a few other things I think I came across. I need to dig up that code and rememeber. I will be getting to this tonight or tomorrow.

comment:2 Changed 8 years ago by Brian Rosner

Status: newassigned

comment:3 Changed 8 years ago by Brian Rosner

Resolution: fixed
Status: assignedclosed

(In [7503]) newforms-admin: Fixed #7114 -- MultiWidget now calls decompress on initial data. Thanks Honza Kral.

Note: See TracTickets for help on using tickets.
Back to Top