Django

Code

Ticket #10370 (closed: fixed)

Opened 1 year ago

Last modified 1 year ago

[patch] re-implement MultiValueDict.itervalues() to match values()

Reported by: rfk Assigned to: nobody
Milestone: 1.1 Component: Core framework
Version: 1.0 Keywords:
Cc: Triage Stage: Accepted
Has patch: 1 Needs documentation: 0
Needs tests: 0 Patch needs improvement: 0

Description

MultiValueDict? modifies values() to yield only the last item from each list, but uses the default itervalues() implementation which yields the entire list. This leads to much confusion since d.values() != list(d.itervalues()).

Attachments

mvd_itervalues.diff (1.2 kB) - added by rfk on 02/27/09 01:27:49.
re-implement MultiValueDict?.itervalues()

Change History

02/27/09 01:27:49 changed by rfk

  • attachment mvd_itervalues.diff added.

re-implement MultiValueDict?.itervalues()

02/28/09 13:13:42 changed by jacob

  • needs_better_patch changed.
  • needs_docs changed.
  • stage changed from Unreviewed to Accepted.
  • needs_tests changed.
  • milestone set to 1.1.

See also #8447.

03/14/09 00:24:23 changed by russellm

  • component changed from Uncategorized to Core framework.

03/31/09 01:30:07 changed by jacob

  • status changed from new to closed.
  • resolution set to fixed.

(In [10241]) Fixed #8847, #10370: added some missing methods to MultiValueDict? after [8399]. Thanks, James Turk and rfk.

Add/Change #10370 ([patch] re-implement MultiValueDict.itervalues() to match values())




Change Properties
Action