#28534: Changing JSONField on inline in admin doesn't always trigger change
-----------------------------+-------------------------------------
     Reporter:  john-parton  |                    Owner:  hui shang
         Type:  Bug          |                   Status:  assigned
    Component:  Forms        |                  Version:  1.11
     Severity:  Normal       |               Resolution:
     Keywords:               |             Triage Stage:  Accepted
    Has patch:  1            |      Needs documentation:  0
  Needs tests:  0            |  Patch needs improvement:  0
Easy pickings:  0            |                    UI/UX:  0
-----------------------------+-------------------------------------

Comment (by john-parton):

 Replying to [comment:5 hui shang]:
 > [https://github.com/django/django/pull/8989#### PR]
 >
 > Hi, @john-partonm, for (1),  json.dumps works well, but I think str() is
 a good choice too, so I use it.

 Does str() always sort dictionary keys? I'm not totally sure, but under
 certain circumstances I think your fix might flag {'a': 1, 'b': 2} as
 being different from {'b': 2, 'a': 1}.

 Thanks for the quick response and your hard work!

-- 
Ticket URL: <https://code.djangoproject.com/ticket/28534#comment:6>
Django <https://code.djangoproject.com/>
The Web framework for perfectionists with deadlines.

-- 
You received this message because you are subscribed to the Google Groups 
"Django updates" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To post to this group, send email to [email protected].
To view this discussion on the web visit 
https://groups.google.com/d/msgid/django-updates/069.4f0327f8ea417f3222c23cc2d3a11a99%40djangoproject.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to