| @@ -624,7 +624,7 class Issue < ActiveRecord::Base | |||
|
|
624 | 624 | errors.add :base, v.custom_field.name + ' ' + l('activerecord.errors.messages.blank') |
|
|
625 | 625 | end |
|
|
626 | 626 | else |
|
|
627 | if respond_to?(attribute) && send(attribute).blank? | |
|
|
627 | if respond_to?(attribute) && send(attribute).blank? && !disabled_core_fields.include?(attribute) | |
|
|
628 | 628 | errors.add attribute, :blank |
|
|
629 | 629 | end |
|
|
630 | 630 | end |
| @@ -830,6 +830,26 class IssueTest < ActiveSupport::TestCase | |||
|
|
830 | 830 | assert issue.save |
|
|
831 | 831 | end |
|
|
832 | 832 | |
|
|
833 | def test_required_attribute_that_is_disabled_for_the_tracker_should_not_be_required | |
|
|
834 | WorkflowPermission.delete_all | |
|
|
835 | WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 1, | |
|
|
836 | :role_id => 1, :field_name => 'start_date', | |
|
|
837 | :rule => 'required') | |
|
|
838 | user = User.find(2) | |
|
|
839 | ||
|
|
840 | issue = Issue.new(:project_id => 1, :tracker_id => 1, :status_id => 1, | |
|
|
841 | :subject => 'Required fields', :author => user) | |
|
|
842 | assert !issue.save | |
|
|
843 | assert_include "Start date can't be blank", issue.errors.full_messages | |
|
|
844 | ||
|
|
845 | tracker = Tracker.find(1) | |
|
|
846 | tracker.core_fields -= %w(start_date) | |
|
|
847 | tracker.save! | |
|
|
848 | issue = Issue.new(:project_id => 1, :tracker_id => 1, :status_id => 1, | |
|
|
849 | :subject => 'Required fields', :author => user) | |
|
|
850 | assert issue.save | |
|
|
851 | end | |
|
|
852 | ||
|
|
833 | 853 | def test_required_attribute_names_for_multiple_roles_should_intersect_rules |
|
|
834 | 854 | WorkflowPermission.delete_all |
|
|
835 | 855 | WorkflowPermission.create!(:old_status_id => 1, :tracker_id => 1, |
General Comments 0
You need to be logged in to leave comments.
Login now