Validate a URL attribute in my model

Question

I've got a model that refers to a table with a thumbnail column. As for validation, I want the thumbnail to accept values of either empty/nil or a valid url (ideally jpg/gif). Below is my attempt but it doesn't seem to be validating as expected. Could someone please have a look at my model below and tell me where I'm going wrong, thanks

class SearchResult < ActiveRecord::Base

  validate :check_thumbnail

  private
    def check_thumbnail
      if self.thumbnail.nil?
        true
      else
        if /https?:\/\/[\S]+/.match(self.thumbnail).nil?
          false
        else
          false
        end
      end
    end
end

Within your else, both if and else parts are false. Other than that have a look at the official validation guide (particularly at allow_nil, allow_blank, and format: guides.rubyonrails.org/active_record_validations.html — Kostas Rousis
– Kostas Rousis, Commented May 15, 2014 at 11:44

peresleguine · Accepted Answer · 2014-05-15 13:57:42Z

3

Using official guide for custom validations I would recommend to create url_validator.rb in your app/validators with the following content:

class UrlValidator < ActiveModel::EachValidator
  def validate_each(record, attribute, value)
    record.errors[attribute] << (options[:message] || 'must be a valid URL') unless url_valid?(value)
  end

  def url_valid?(url)
    uri = URI.parse(url) rescue false
    uri.kind_of?(URI::HTTP) || uri.kind_of?(URI::HTTPS)
  end
end

Thereafter you may validate thumbnail attribute like this:

class SearchResult < ActiveRecord::Base
  validates :thumbnail, url: true, allow_blank: true
end

Also you may improve url_valid? method to consider extension whitelist via regex (however I would create another validator for this, e.g. image_url_validator.rb to keep basic url validator).

EDIT:

app/validators/image_url_validator.rb

class ImageUrlValidator < ActiveModel::EachValidator
  def validate_each(record, attribute, value)
    record.errors[attribute] << (options[:message] || 'must be a valid URL (only jpg and gif)') unless url_valid?(value)
  end

  def url_valid?(url)
    uri = URI.parse(url) rescue false
    (uri.kind_of?(URI::HTTP) || uri.kind_of?(URI::HTTPS)) && uri.to_s.match(/\.jpg|\.gif/)
  end
end

app/models/search_result.rb

class SearchResult < ActiveRecord::Base
  validates :thumbnail, image_url: true, allow_blank: true
end

Key benefit of this approach is that you may reuse this custom validations for different attributes in different models.

edited May 15, 2014 at 13:57

answered May 15, 2014 at 12:22

peresleguine

2,4232 gold badges32 silver badges36 bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

Martyn Over a year ago

Hi, actually I'm getting an error include ActiveModel::Validations validates_with MyValidator. Do I need to include the files somehow? I have them in app/validators/ folder? I didn't see any mention of this in the guide

peresleguine Over a year ago

You don't need to include anything. Just create app/validators/image_url_validator.rb as described and add this line to SearchResult model: validates :thumbnail, image_url: true, allow_blank: true.

peresleguine Over a year ago

In the guide EmailValidator describes this instead of MyValidator: guides.rubyonrails.org/…

Max Williams · Accepted Answer · 2014-05-15 13:03:43Z

1

I would rewrite it as

class SearchResult < ActiveRecord::Base

validate :check_thumbnail

private
  def check_thumbnail
    unless self.thumbnail.blank? || self.thumbnail =~ /https?:\/\/[\S]+/
      self.errors.add(:thumbnail, "is not valid")
    end
  end
end

answered May 15, 2014 at 13:03

Max Williams

33k34 gold badges137 silver badges205 bronze badges

Comments

Rajdeep Singh · Accepted Answer · 2014-05-15 11:53:00Z

-1

Use this regex to validate URL, I've used this myself

/\A(http|https):\/\/[a-z0-9]+([\-\.]{1}[a-z0-9]+)*\.[a-z]{2,5}(:[0-9]{1,5})?(\/.*)?\z/ix

here in your code in the else block, do this way

  else
    URL_REGEX = /\A(http|https):\/\/[a-z0-9]+([\-\.]{1}[a-z0-9]+)*\.[a-z]{2,5}(:[0-9]{1,5})?(\/.*)?\z/ix
    if (URL_REGEX).match(self.thumbnail).nil?
      false
    else
      false
    end
  end

edited May 15, 2014 at 11:53

answered May 15, 2014 at 11:47

Rajdeep Singh

17.9k6 gold badges58 silver badges80 bronze badges

Collectives™ on Stack Overflow

Validate a URL attribute in my model

3 Answers 3

3 Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

3 Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related