diff options
author | Timothy Andrew <mail@timothyandrew.net> | 2017-02-16 12:35:10 +0530 |
---|---|---|
committer | Timothy Andrew <mail@timothyandrew.net> | 2017-02-24 16:50:20 +0530 |
commit | f2ed82fa8486875660b80dd061827ac8b86d00b6 (patch) | |
tree | 07b0221a225f32de7a3063a4bee2ff97463f9c54 /lib/api/helpers | |
parent | 3bd2a98f64a12a2e23a6b9ce77427a9c2033a6bf (diff) | |
download | gitlab-ce-f2ed82fa8486875660b80dd061827ac8b86d00b6.tar.gz |
Implement final review comments from @DouweM and @rymai
- Have `Uniquify` take a block instead of a Proc/function. This is more
idiomatic than passing around a function in Ruby.
- Block a user before moving their issues to the ghost user. This avoids a data
race where an issue is created after the issues are migrated to the ghost user,
and before the destroy takes place.
- No need to migrate issues (to the ghost user) in a transaction, because
we're using `update_all`
- Other minor changes
Diffstat (limited to 'lib/api/helpers')
0 files changed, 0 insertions, 0 deletions