This project is archived and is in readonly mode.
ActiveRecord::Validator should be ActiveModel::Validator
Reported by darkliquid | August 19th, 2009 @ 04:25 PM
Given the push to keep everything modular and the fact that validations have been mostly extracted from active record, having this validator base class in the active record namespace seems like a mistake. It has no dependencies on active record that I can tell but validate_with from ActiveModel relies on classes conforming to this interface.
Consequently, this class should be moved into ActiveModel so that programs that don't require ActiveRecord can benefit from the interface defined by the base class.
Comments and changes to this ticket
-
José Valim September 4th, 2009 @ 01:24 PM
- Assigned user set to José Valim
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
Tags
Referenced by
- 3058 [PATCH] Sexy Validations I have also made a change to rectify the bug in this tick...