Optimize and add error codes for repository conflict verification.

Review Request #10373 — Created Jan. 3, 2019 and submitted

Information

Review Board
release-3.0.x
bc03771...

Reviewers

The process of checking repositories for conflicts (by name and by path)
required two SQL queries, one for each field. While not terribly
inefficient, it could certainly be improved. As could the error
information.

This shortens the verification process down to one SQL query, which
checks for both name and path conflicts at once. If both have conflicts,
then they'll both be represented in the ValidationError, instead of
only the first one that's found. That exception now comes with a
repository_exists error code, for later matching.

Constants have also been added for the error messages used in the
exception. This is here for an upcoming change to RepositoryForm that
checks if there's a repository conflict. Unfortunately, in Django 1.6,
the ValidationError codes are discarded by the time this logic would
need to be performed, so we're matching strings instead. In the future,
this could be updated to use the codes and drop the constants.

Unit tests pass.

Used this along with some new logic in upcoming tests. Verified it
works as expected.

chipx86
david
  1. Ship It!
  2. 
      
chipx86
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-3.0.x (a17c59e)
Loading...