Fix setting prefixed standard fields on the repository form.

Review Request #11005 — Created April 28, 2020 and submitted — Latest diff uploaded

Information

Review Board
release-3.0.x

Reviewers

A recent change fixed performing HTTP PUT operations on the repository
resource without providing entire payloads for the form. It computed the
initial data based on a repository and populated the form, trying to
create a solid starting point.

This had a bug, though, that prevented actually submitting the
repository form via the administration page. It no longer considered
that some standard fields, like path, mirror_path, username, etc.,
might be submitted with an SCMTool prefix (e.g., git-mirror_path), and
this caused it to override these provided values with the initial data,
preventing these fields from being changed again.

This bug did not make it into a release, or into production on
RBCommons.

This change fixes the new logic on the repository form to check whether
the prefixed form of a key was posted on the form data before trying to
figure out a fallback value for it. If it's explicitly posted, we use
the existing value. Otherwise, we now normalize a prefixed form, if one
was provided, and then fall back on the initial value.

Unit tests passed.

I was able to successfully modify the formerly-broken fields for
several existing repositories.

Commits

Files

    Loading...