Fix saving extension settings.

Review Request #5967 — Created June 10, 2014 and submitted — Latest diff uploaded

Information

Djblets
release-0.8.x
aabd620...

Reviewers

A recent change added a request parameter to SiteSettingsForm, and
changed the calls in siteconfig to pass the data and files
values as keyword arguments.

This broke the form, because extensions use the configure_extension
view, which passes data as positional arguments.

While that view could also just be updated for this, the convention is
to pass data and files as positional arguments, and most callers do
so. To provide the best compatibility, we now do this as well, and leave
optional keyword arguments (request) as keyword arguments accessed
through kwargs.

Successfully saved extension settings.

Unit tests pass.

    Loading...