Remove the limitation on characters that can be used in custom_field, issue_status...

git-svn-id: svn+ssh://rubyforge.org/var/svn/redmine/trunk@4599 e93f8b46-1217-0410-a6f0-8f06a7374b81

diff --git a/app/models/custom_field.rb b/app/models/custom_field.rb
index 7cc6e865d71cbc3db2ea1330788431a6a7d2efb8..3aacd4686500946ce965cf17fb00753b5146f3cd 100644 (file)
--- a/app/models/custom_field.rb
+++ b/app/models/custom_field.rb
@@ -23,7 +23,6 @@ class CustomField < ActiveRecord::Base
   validates_presence_of :name, :field_format
   validates_uniqueness_of :name, :scope => :type
   validates_length_of :name, :maximum => 30
-  validates_format_of :name, :with => /^[\w\s\.\'\-]*$/i
   validates_inclusion_of :field_format, :in => Redmine::CustomFieldFormat.available_formats
 
   def initialize(attributes = nil)

diff --git a/app/models/issue_status.rb b/app/models/issue_status.rb
index f376d5d15bbac9dc546e9a64f7502916e8a1661a..8171cdb79f1bbefdec1dd0516be3851dd009cd8d 100644 (file)
--- a/app/models/issue_status.rb
+++ b/app/models/issue_status.rb
@@ -25,7 +25,6 @@ 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_inclusion_of :default_done_ratio, :in => 0..100, :allow_nil => true
 
   def after_save

diff --git a/app/models/role.rb b/app/models/role.rb
index d1bebdb6d637f5b083defdacbbccf44d0b64ae42..6a4b9e4e3b9d0e3c61e412eab091e1dab3ef0807 100644 (file)
--- a/app/models/role.rb
+++ b/app/models/role.rb
@@ -43,7 +43,6 @@ 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 permissions
     read_attribute(:permissions) || []

diff --git a/app/models/tracker.rb b/app/models/tracker.rb
index 9ee045ce14ad9a0fd95ad36159055960fab85e73..93fdd4035cd16d66c08f7287d6d0c1b60bc83931 100644 (file)
--- a/app/models/tracker.rb
+++ b/app/models/tracker.rb
@@ -31,7 +31,6 @@ 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
 
   def to_s; name end
   

diff --git a/app/models/user.rb b/app/models/user.rb
index 91d6c5fd02b99ef38f2c2858e5ca70b20e15132d..7d0014c8d8c966aee7adbbf69e65a424de17e461 100644 (file)
--- a/app/models/user.rb
+++ b/app/models/user.rb
@@ -68,7 +68,6 @@ class User < Principal
   # Login must contain lettres, numbers, underscores only
   validates_format_of :login, :with => /^[a-z0-9_\-@\.]*$/i
   validates_length_of :login, :maximum => 30
-  validates_format_of :firstname, :lastname, :with => /^[\w\s\'\-\.]*$/i
   validates_length_of :firstname, :lastname, :maximum => 30
   validates_format_of :mail, :with => /^([^@\s]+)@((?:[-a-z0-9]+\.)+[a-z]{2,})$/i, :allow_nil => true
   validates_length_of :mail, :maximum => 60, :allow_nil => true

diff --git a/lib/tasks/migrate_from_mantis.rake b/lib/tasks/migrate_from_mantis.rake
index e9244fcaf7f9340edfe0a4b4d42aa6d60dd182f6..cd82872c18dff078b40a022712d75c13df05ae79 100644 (file)
--- a/lib/tasks/migrate_from_mantis.rake
+++ b/lib/tasks/migrate_from_mantis.rake
@@ -88,13 +88,11 @@ task :migrate_from_mantis => :environment do
       \r
       def firstname\r
         @firstname = realname.blank? ? username : realname.split.first[0..29]\r
-        @firstname.gsub!(/[^\w\s\'\-]/i, '')\r
         @firstname\r
       end\r
       \r
       def lastname\r
         @lastname = realname.blank? ? '-' : realname.split[1..-1].join(' ')[0..29]\r
-        @lastname.gsub!(/[^\w\s\'\-]/i, '')\r
         @lastname = '-' if @lastname.blank?\r
         @lastname\r
       end\r
@@ -224,7 +222,7 @@ task :migrate_from_mantis => :environment do
       end\r
       \r
       def name\r
-        read_attribute(:name)[0..29].gsub(/[^\w\s\'\-]/, '-')\r
+        read_attribute(:name)[0..29]\r
       end\r
     end\r
     \r

diff --git a/lib/tasks/migrate_from_trac.rake b/lib/tasks/migrate_from_trac.rake
index 0510be51e698889e1dd183ed7c8032231dab1106..f124703d4220b8be877eb3b9956ed91867d3c026 100644 (file)
--- a/lib/tasks/migrate_from_trac.rake
+++ b/lib/tasks/migrate_from_trac.rake
@@ -246,8 +246,8 @@ namespace :redmine do
           ln = ($2 || '-').strip
 
           u = User.new :mail => mail.gsub(/[^-@a-z0-9\.]/i, '-'),
-                       :firstname => fn[0, limit_for(User, 'firstname')].gsub(/[^\w\s\'\-]/i, '-'),
-                       :lastname => ln[0, limit_for(User, 'lastname')].gsub(/[^\w\s\'\-]/i, '-')
+                       :firstname => fn[0, limit_for(User, 'firstname')],
+                       :lastname => ln[0, limit_for(User, 'lastname')]
 
           u.login = username[0,limit_for(User, 'login')].gsub(/[^a-z0-9_\-@\.]/i, '-')
           u.password = 'trac'