Fix using ?only-fields= on the user resource with private profiles.

Review Request #8683 — Created Jan. 30, 2017 and submitted — Latest diff uploaded

Information

Review Board
release-2.5.x
b9bee4c...

Reviewers

When serializing a user in the API who has private profiles enabled,
we'd try to remove some data from the resulting payload. However, if the
caller used ?only-fields= and didn't specify those fields, the
serialization process would crash. We now catch any errors here and
ignore them.

Unit tests were added to ensure this doesn't regress.

The new unit test failed before the fix, and passed after.

    Loading...