Fix encoding of floating point values into XML.

Review Request #8249 — Created June 18, 2016 and submitted

Information

Djblets
release-0.9.x
042f04c...

Reviewers

The encoding functionality in XMLEncoderAdapter would throw an exception
when encountering a floating point value, indicating that the value was not
XML serializable. This has been corrected by adding special handing for
floating point values, analogous to how other data types are handled.

Added new unit tests to exercise both the JSON and XML encoder adapters.
Encoding is attempted on a variety of data types.  Prior to the fix, the
XML-based test would fail due to the presence of floating point data types.
Description From Last Updated

Col: 44 W292 no newline at end of file

reviewbotreviewbot

Col: 1 W391 blank line at end of file

reviewbotreviewbot
reviewbot
  1. Tool: Pyflakes
    Processed Files:
        djblets/webapi/tests/test_encoders.py
        djblets/webapi/encoders.py
    
    
    
    Tool: PEP8 Style Checker
    Processed Files:
        djblets/webapi/tests/test_encoders.py
        djblets/webapi/encoders.py
    
    
  2. djblets/webapi/tests/test_encoders.py (Diff revision 1)
     
     
    Col: 44
     W292 no newline at end of file
    
  3. 
      
gmyers
reviewbot
  1. Tool: Pyflakes
    Processed Files:
        djblets/webapi/tests/test_encoders.py
        djblets/webapi/encoders.py
    
    
    
    Tool: PEP8 Style Checker
    Processed Files:
        djblets/webapi/tests/test_encoders.py
        djblets/webapi/encoders.py
    
    
  2. djblets/webapi/tests/test_encoders.py (Diff revision 2)
     
     
    Col: 1
     W391 blank line at end of file
    
  3. 
      
gmyers
reviewbot
  1. Tool: PEP8 Style Checker
    Processed Files:
        djblets/webapi/tests/test_encoders.py
        djblets/webapi/encoders.py
    
    
    
    Tool: Pyflakes
    Processed Files:
        djblets/webapi/tests/test_encoders.py
        djblets/webapi/encoders.py
    
    
  2. 
      
brennie
  1. Ship It!
  2. 
      
gmyers
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-0.9.x (a098afe)
Loading...