This project is archived and is in readonly mode.
Added Range#cover?
Reported by Diego Carrion | February 19th, 2011 @ 11:51 PM | in 3.1
Hi, I added Range#cover? case it doesn't exist (Ruby 1.8). This modification allowed a considerable refactoring of ActiveModel::Validations::InclusionValidator#validate_each also included in the attached patch.
Comments and changes to this ticket
-
Santiago Pastorino February 20th, 2011 @ 01:21 AM
- State changed from new to open
- Milestone set to 3.1
- Importance changed from to Low
Diego 'manxa' Carrion that looks great for me :P.
I'm pushing it. -
Repository February 20th, 2011 @ 01:26 AM
- State changed from open to committed
(from [511bf2a0505309c65bbf2ad9394e3c548762ea5d]) refactored ActiveModel::Validations::InclusionValidator#validate_each
[#6455 state:committed]
Signed-off-by: Santiago Pastorino santiago@wyeworks.com
https://github.com/rails/rails/commit/511bf2a0505309c65bbf2ad9394e3...
Create your profile
Help contribute to this project by taking a few moments to create your personal profile. Create your profile »
<h2 style="font-size: 14px">Tickets have moved to Github</h2>
The new ticket tracker is available at <a href="https://github.com/rails/rails/issues">https://github.com/rails/rails/issues</a>
People watching this ticket
Attachments
Referenced by
- 6455 Added Range#cover? [#6455 state:committed]