Preserve ordering

Question

Preserve ordering

Closed this issue 6 years ago · 5 comments

The dict should be replaced with collections.OrderedDict.

Answer 1 · 2018-08-17T15:01:41.000Z

Tres kindly implemented a PR for this issue in #15 . I'm not totally against merging it, but I'd like to understand the rationale behind asking for it before we do.

Answer 2 · 2018-08-17T15:37:04.000Z

It's some time ago since i created this issue. I think the comment #15 (comment) explains the use case.

Answer 3 · 2018-08-17T17:20:22.000Z

@delijati can you remember the use case of wanting to parse the items of a mapping in some sort of order?

Answer 4 · 2018-08-18T10:40:41.000Z

Hmm it is really long ago and it was probably for some customer. I think it was for generating reports or sending emails or something like this. The ordering in the form should be preserved in the appstruct or from.

Answer 5 · 2018-08-18T14:32:44.000Z

OK, thanks. I am hesitantly going to reject the patch from @tseaver because there doesn't appear to be a heart-must-continue-beating reason to accept it; if someone else comes up with a defensible case for it, we can reopen.