From: Go MAEDA <maeda@farend.jp>
Date: Thu, 27 Jun 2024 04:49:38 +0000 (+0000)
Subject: Merged r22895 and r22896 from trunk to 5.1-stable (#40860).
X-Git-Tag: 5.1.4~23
X-Git-Url: https://source.dussan.org/?a=commitdiff_plain;h=15223f559e68dc19ee2db63ae080abef1196b244;p=redmine.git

Merged r22895 and r22896 from trunk to 5.1-stable (#40860).


git-svn-id: https://svn.redmine.org/redmine/branches/5.1-stable@22900 e93f8b46-1217-0410-a6f0-8f06a7374b81
---

diff --git a/app/models/issue.rb b/app/models/issue.rb
index dddb9df6b..6e69b5560 100644
--- a/app/models/issue.rb
+++ b/app/models/issue.rb
@@ -2069,7 +2069,7 @@ class Issue < ActiveRecord::Base
       tracker.disabled_core_fields.each do |attribute|
         send "#{attribute}=", nil
       end
-      self.priority_id ||= IssuePriority.default&.id || IssuePriority.active.first.id
+      self.priority_id ||= IssuePriority.default&.id || IssuePriority.active.first&.id
       self.done_ratio ||= 0
     end
   end
diff --git a/test/unit/issue_test.rb b/test/unit/issue_test.rb
index 34739b62f..3be86a1a4 100644
--- a/test/unit/issue_test.rb
+++ b/test/unit/issue_test.rb
@@ -83,6 +83,16 @@ class IssueTest < ActiveSupport::TestCase
     assert_save issue
   end
 
+  def test_create_with_no_priority_defined
+    IssuePriority.delete_all
+    issue = Issue.new(
+      project_id: 1, tracker_id: 1, author_id: 3, subject: 'test_create_with_no_priority_defined'
+    )
+
+    assert_nothing_raised {assert_not issue.save}
+    assert_include 'Priority cannot be blank', issue.errors.full_messages
+  end
+
   def test_default_priority_should_be_set_when_priority_field_is_disabled
     tracker = Tracker.find(1)
     tracker.core_fields = tracker.core_fields - ['priority_id']