Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

This project is archived and is in readonly mode.

Added Range#cover?

#6455

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.

Reported by Diego Carrion · February 19th, 2011 @ 11:51 PM

State: committed
Milestone: 3.1
Assigned to: Santiago Pastorino Santiago Pastorino
Importance: Low

Activity

  1. Santiago Pastorino
    Santiago Pastorino
    • 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.

    February 20th, 2011 @ 01:21 AM

  2. Repository
    Repository
    • State changed from open to committed

    (from [511bf2a0505309c65bbf2ad9394e3c548762ea5d]) refactored ActiveModel::Validations::InclusionValidator#validate_each

    [#6455 Added Range#cover? state:committed]

    Signed-off-by: Santiago Pastorino santiago@wyeworks.com
    https://github.com/rails/rails/commit/511bf2a0505309c65bbf2ad9394e3...

    February 20th, 2011 @ 01:26 AM