Make expandedFields a BaseResource concept.

Review Request #4218 — Created June 7, 2013 and submitted — Latest diff uploaded

Information

Review Board
master

Reviewers

Make expandedFields a BaseResource concept.

DraftReviewRequest had a concept of expanding fields in a request by
populating expandedFields. This turns out to be useful elsewhere, so
move this logic out and into BaseResource.

Now, any resource class can set the list of fields to expand, in order
to provide more detailed information or child resources to the callers.
Tested with some upcoming changes that provide sub-models for
related objects.

Unit tests weren't added explicitly for this new functionality, but
some existing tests do cover it, and new tests for the new objects
will as well. We can't easily get in to where jQuery updates the URL
with the options.data stuff, so there's little to really test there.
    Loading...