• 
      

    Fix KeyErrors when saving the Avatar Settings form.

    Review Request #9404 — Created Nov. 27, 2017 and submitted

    Information

    Review Board
    release-3.0.x
    5c91c53...

    Reviewers

    The Avatar Settings form's clean_default_service() method attempted to
    access cleaned data for other fields. Whether or not this works depends
    on the order in which the Form class attempts to clean fields, and
    broke in some manual testing I was performing, raising KeyErrors.

    This switches this method to be a clean() method, allowing it to
    operate on cleaned data from multiple fields and ensuring that saving
    never breaks.

    Saved the form many times without hitting these crashes.

    Unit tests pass.

    david
    1. Ship It!
    2. 
        
    chipx86
    Review request changed
    Status:
    Completed
    Change Summary:
    Pushed to release-3.0.x (a37a3fe)