Defect #9685
Adding multiple times the same related issue relation is possible
| Status: | Closed | Start date: | 2011-11-29 | |
|---|---|---|---|---|
| Priority: | Normal | Due date: | ||
| Assignee: | - | % Done: | 0% |
|
| Category: | Issues | |||
| Target version: | 1.4.0 | |||
| Affected version: | 1.2.2 | Resolution: | Fixed |
Description
Associated revisions
Enforced unicity of relations between 2 issues with a database index (#9685).
History
#1 Updated by Etienne Massip 6 months ago
- Subject changed from Adding multiple times the same related issue is possible to Adding multiple times the same related issue relation is possible
As explained in RoR documentation (both 2.3.x and 3.x, this excerpt is from 3.x'):
Concurrency and integrity Using this validation method in conjunction with ActiveRecord::Base#save does not guarantee the absence of duplicate record insertions, because uniqueness checks on the application level are inherently prone to race conditions. (...) This could even happen if you use transactions with the ‘serializable’ isolation level. The best way to work around this problem is to add a unique index to the database table using (...)#add_index. (...) The bundled ActiveRecord::ConnectionAdapters distinguish unique index constraint errors from other types of database errors by throwing an ActiveRecord::RecordNotUnique exception. For other adapters you will have to parse the (database-specific) exception message to detect such a case.
A simple migration creating an unique index on to and from issue_relation columns should do the job.
#2 Updated by Etienne Massip 6 months ago
- File add_unique_index_in_issue_relations.patch added
Migration patch attached.
#3 Updated by Etienne Massip 6 months ago
- Target version changed from Candidate for next minor release to Candidate for next major release
Ok, since there is a migration, moving to major.
#4 Updated by Jean-Philippe Lang 6 months ago
- Target version changed from Candidate for next major release to 1.4.0
#5 Updated by Etienne Massip 6 months ago
- Status changed from Confirmed to Resolved
- Resolution set to Fixed
Migration committed with r8041.
#6 Updated by Jean-Philippe Lang 4 months ago
- Status changed from Resolved to Closed
