Added several validates_length_of

git-svn-id: http://redmine.rubyforge.org/svn/trunk@593 e93f8b46-1217-0410-a6f0-8f06a7374b81

diff --git a/app/models/attachment.rb b/app/models/attachment.rb
index 2be4e16..b1a7c52 100644 (file)
--- a/app/models/attachment.rb
+++ b/app/models/attachment.rb
@@ -22,7 +22,9 @@ class Attachment < ActiveRecord::Base
   belongs_to :author, :class_name => "User", :foreign_key => "author_id"
   
   validates_presence_of :container, :filename
-  
+  validates_length_of :filename, :maximum => 255
+  validates_length_of :disk_filename, :maximum => 255
+    
   cattr_accessor :storage_path
   @@storage_path = "#{RAILS_ROOT}/files"
   

diff --git a/app/models/custom_field.rb b/app/models/custom_field.rb
index 554504a..751cc85 100644 (file)
--- a/app/models/custom_field.rb
+++ b/app/models/custom_field.rb
@@ -29,6 +29,7 @@ class CustomField < ActiveRecord::Base
 
   validates_presence_of :name, :field_format
   validates_uniqueness_of :name
+  validates_length_of :name, :maximum => 30
   validates_format_of :name, :with => /^[\w\s\'\-]*$/i
   validates_inclusion_of :field_format, :in => FIELD_FORMATS.keys
 

diff --git a/app/models/document.rb b/app/models/document.rb
index 141489a..8b5d68e 100644 (file)
--- a/app/models/document.rb
+++ b/app/models/document.rb
@@ -21,4 +21,5 @@ class Document < ActiveRecord::Base
   has_many :attachments, :as => :container, :dependent => :destroy
 
   validates_presence_of :project, :title, :category
+  validates_length_of :title, :maximum => 60
 end

diff --git a/app/models/enumeration.rb b/app/models/enumeration.rb
index de85260..c4f0f98 100644 (file)
--- a/app/models/enumeration.rb
+++ b/app/models/enumeration.rb
@@ -20,6 +20,7 @@ class Enumeration < ActiveRecord::Base
   
   validates_presence_of :opt, :name
   validates_uniqueness_of :name, :scope => [:opt]
+  validates_length_of :name, :maximum => 30
   validates_format_of :name, :with => /^[\w\s\'\-]*$/i
        
        OPTIONS = {

diff --git a/app/models/issue.rb b/app/models/issue.rb
index ec62d0a..a73160e 100644 (file)
--- a/app/models/issue.rb
+++ b/app/models/issue.rb
@@ -38,6 +38,7 @@ class Issue < ActiveRecord::Base
   acts_as_watchable
   
   validates_presence_of :subject, :description, :priority, :tracker, :author, :status
+  validates_length_of :subject, :maximum => 255
   validates_inclusion_of :done_ratio, :in => 0..100
   validates_associated :custom_values, :on => :update
 

diff --git a/app/models/issue_status.rb b/app/models/issue_status.rb
index 3bca612..13ed27f 100644 (file)
--- a/app/models/issue_status.rb
+++ b/app/models/issue_status.rb
@@ -22,6 +22,7 @@ class IssueStatus < ActiveRecord::Base
 
   validates_presence_of :name
   validates_uniqueness_of :name
+  validates_length_of :name, :maximum => 30
   validates_format_of :name, :with => /^[\w\s\'\-]*$/i
   validates_length_of :html_color, :is => 6
   validates_format_of :html_color, :with => /^[a-f0-9]*$/i

diff --git a/app/models/news.rb b/app/models/news.rb
index cd130e9..e9a4884 100644 (file)
--- a/app/models/news.rb
+++ b/app/models/news.rb
@@ -21,6 +21,8 @@ class News < ActiveRecord::Base
   has_many :comments, :as => :commented, :dependent => :delete_all, :order => "created_on"
   
   validates_presence_of :title, :description
+  validates_length_of :title, :maximum => 60
+  validates_length_of :summary, :maximum => 255
   
   # returns latest news for projects visible by user
   def self.latest(user=nil, count=5)

diff --git a/app/models/query.rb b/app/models/query.rb
index b641436..8894665 100644 (file)
--- a/app/models/query.rb
+++ b/app/models/query.rb
@@ -24,6 +24,7 @@ class Query < ActiveRecord::Base
   attr_accessor :executed_by
   
   validates_presence_of :name, :on => :save
+  validates_length_of :name, :maximum => 255
     
   @@operators = { "="   => :label_equals, 
                   "!"   => :label_not_equals,

diff --git a/app/models/role.rb b/app/models/role.rb
index a60319c..98d735e 100644 (file)
--- a/app/models/role.rb
+++ b/app/models/role.rb
@@ -24,6 +24,7 @@ class Role < ActiveRecord::Base
 
   validates_presence_of :name
   validates_uniqueness_of :name
+  validates_length_of :name, :maximum => 30
   validates_format_of :name, :with => /^[\w\s\'\-]*$/i
 
   def <=>(role)

diff --git a/app/models/tracker.rb b/app/models/tracker.rb
index 14ac904..c024c09 100644 (file)
--- a/app/models/tracker.rb
+++ b/app/models/tracker.rb
@@ -24,6 +24,7 @@ class Tracker < ActiveRecord::Base
 
   validates_presence_of :name
   validates_uniqueness_of :name
+  validates_length_of :name, :maximum => 30
   validates_format_of :name, :with => /^[\w\s\'\-]*$/i
 
 private

diff --git a/app/models/version.rb b/app/models/version.rb
index 6234732..f4a40da 100644 (file)
--- a/app/models/version.rb
+++ b/app/models/version.rb
@@ -23,6 +23,7 @@ class Version < ActiveRecord::Base
 
   validates_presence_of :name
   validates_uniqueness_of :name, :scope => [:project_id]
+  validates_length_of :name, :maximum => 30
   validates_format_of :effective_date, :with => /^\d{4}-\d{2}-\d{2}$/, :message => :activerecord_error_not_a_date, :allow_nil => true
   
   def start_date