Refactored todo model to check only for pending todos

* extended todo_spec to fit adjustment to .any_target?
* with changelog

app/models/todo.rb

@@ -93,7 +93,7 @@ class Todo < ApplicationRecord
     #
     # target - The value of the `target_type` column, such as `Issue`.
     def any_for_target?(target)
-      exists?(target: target)
+      exists?(target: target, state: :pending)
     end
 
     # Updates the state of a relation of todos to the new state.

changelogs/unreleased/66701-done-quick-action-is-available-even-no-To-Do-exists.yml

+---
+title: Changed todo/done quick actions to work not only for first usage
+merge_request:
+author: Marc Schwede
+type: fixed

spec/models/todo_spec.rb

@@ -273,6 +273,22 @@ describe Todo do
       expect(described_class.any_for_target?(todo.target)).to eq(true)
     end
 
+    it 'returns true if there is at least one todo for a given target with state pending' do
+      issue = create(:issue)
+      create(:todo, state: :done, target: issue)
+      create(:todo, state: :pending, target: issue)
+
+      expect(described_class.any_for_target?(issue)).to eq(true)
+    end
+
+    it 'returns false if there are only todos for a given target with state done' do
+      issue = create(:issue)
+      create(:todo, state: :done, target: issue)
+      create(:todo, state: :done, target: issue)
+
+      expect(described_class.any_for_target?(issue)).to eq(false)
+    end
+
     it 'returns false if there are no todos for a given target' do
       issue = create(:issue)