Project

General

Profile

Actions

Bug #12059

closed

AuthSource allow_nil for hosts should be removed

Added by Daniel Lobato Garcia over 8 years ago. Updated almost 6 years ago.

Status:
Closed
Priority:
Normal
Category:
Rails
Target version:
Difficulty:
Triaged:
Fixed in Releases:
Found in Releases:

Description

When we migrated to the new migration syntax, the clause 'validate_length_of :host' was wrongly translated. It contains a clause 'allow_nil' that allows it to be nil, but in Rails 3 the :presence => true clause takes precedence. If you try to run AuthSourceLdap.new(:host => nil) it will fail as it need a host (correctly). This clause has to be removed so that Rails 4 doesn't allow to create AuthSourceLdap without a host, as :allow_nil takes precedence in Rails 4.

Actions

Also available in: Atom PDF