Inconsistency in validation functions and database constraints
Bug #35077 reported by
Diogo Matsubara
| Affects | Status | Importance | Assigned to | Milestone | |
|---|---|---|---|---|---|
| Launchpad itself |
Fix Released
|
Low
|
Tim Penhey | ||
Bug Description
We have one single constraint in the database that deals with URL validation. We should have for each validation function in interfaces/
Related branches
lp:~thumper/launchpad/fix-unicode-error-in-urlparse
- Robert Collins (community): Approve
-
Diff: 430 lines (+125/-136)6 files modifiedlib/canonical/launchpad/interfaces/validation.py (+0/-105)
lib/lp/app/validators/tests/test_validators.py (+10/-3)
lib/lp/app/validators/url.py (+112/-25)
lib/lp/blueprints/interfaces/specification.py (+1/-1)
lib/lp/registry/browser/announcement.py (+1/-1)
lib/lp/registry/interfaces/productseries.py (+1/-1)
| Changed in launchpad: | |
| status: | Unconfirmed → Confirmed |
| tags: | added: tech-debt |
| Changed in launchpad-foundations: | |
| importance: | Medium → Low |
| Changed in launchpad: | |
| assignee: | nobody → Tim Penhey (thumper) |
| status: | Triaged → In Progress |
Revision history for this message
| Launchpad QA Bot (lpqabot) wrote | #1 |
| Changed in launchpad: | |
| milestone: | none → 11.05 |
| tags: | added: qa-needstesting |
| Changed in launchpad: | |
| status: | In Progress → Fix Committed |
| tags: |
added: qa-ok removed: qa-needstesting |
| Changed in launchpad: | |
| status: | Fix Committed → Fix Released |
To post a comment you must log in.
Fixed in stable r12795 <http://