#11186 closed enhancement (fixed)
Internal error when adding repository with a name that already exists in the database
| Reported by: | Owned by: | Jun Omae | |
|---|---|---|---|
| Priority: | normal | Milestone: | 1.0.2 |
| Component: | admin/web | Version: | 1.0-stable |
| Severity: | normal | Keywords: | repository |
| Cc: | Branch: | ||
| Release Notes: |
Show an explicit message when attempting to add a repository that already exists through the Manage Repository. |
||
| API Changes: | |||
| Internal Changes: | |||
Description
The admin panels such as Component display a TracError when attempting to add an entry that already exists.
In Trac 1.0-stable, a TracError is not raised when attempting to add a repository entry through the Manage Repositories admin panel when the repository name already exists. An Internal Error results.
Attachments (1)
Change History (6)
by , 12 years ago
| Attachment: | InternalError.png added |
|---|
comment:1 by , 12 years ago
comment:2 by , 12 years ago
| Milestone: | → 1.0.2 |
|---|---|
| Owner: | set to |
| Status: | new → assigned |
Looks good! I'll merge it later.
comment:3 by , 12 years ago
I've tried. Your changes works fine. In addition, the "Add Alias" in repository admin has the same issue. It is fixed to catch IntegrityError in [32072a92/jomae.git].
Well, "Modify Repository" has the same issue and makes unfortunately repository names duplicated. Also, 0.12-stable has the issue.
$ sqlite3 /tmp/trac-1.0.1/db/trac.db 'select * from repository' 1|name|blah22 1|dir|/var/svn/blah11 1|type|svn 2|name|blah22 2|dir|/var/svn/blah22 2|type|svn 1|url| 1|description|
comment:4 by , 12 years ago
| Release Notes: | modified (diff) |
|---|---|
| Resolution: | → fixed |
| Status: | assigned → closed |
Committed in [11809]. Thanks, Ryan!
comment:5 by , 12 years ago
| Keywords: | repository added |
|---|
Fix and functional testcase in fd1680f2.