Don't show committed patches as needing rebase

Author: x4m

pgcommitfest/commitfest/templates/commitfest.html

@@ -68,7 +68,7 @@ <h3>{{p.is_open|yesno:"Active patches,Closed patches"}}</h3>
     {%with p.cfbot_results as cfb%}
      {%if not cfb %}
       <span class="badge bg-secondary">Not processed</span>
-     {%elif p.needs_rebase_since %}
+     {%elif p.needs_rebase_since and p.status != 4 %}
       <a href="{{cfb.apply_url}}" title="View git apply logs. Needs rebase {% cfsince p.needs_rebase_since %}. {%if p.failing_since and p.failing_since != p.needs_rebase_since %}Failing {% cfsince p.failing_since %}.{%endif%}">
        <span class="badge bg-warning">Needs rebase!</span>
       </a>

pgcommitfest/commitfest/templates/home.html

@@ -174,7 +174,7 @@ <h3>{%if user.is_authenticated%}Open patches you are subscribed to{%elif p.is_op
      {%with p.cfbot_results as cfb%}
       {%if not cfb %}
        <span class="badge bg-secondary">Not processed</span>
-      {%elif p.needs_rebase_since %}
+      {%elif p.needs_rebase_since and p.status != 4 %}
        <a href="{{cfb.apply_url}}" title="View git apply logs. Needs rebase {% cfsince p.needs_rebase_since %}. {%if p.failing_since and p.failing_since != p.needs_rebase_since %}Failing {% cfsince p.failing_since %}.{%endif%}">
         <span class="badge bg-warning">Needs rebase!</span>
        </a>

pgcommitfest/commitfest/templates/patch.html

@@ -24,7 +24,7 @@
     <td>
      {%if not cfbot_branch %}
       <span class="badge bg-secondary">Not processed</span></a>
-     {%elif cfbot_branch.needs_rebase_since %}
+     {%elif cfbot_branch.needs_rebase_since and not current_poc.is_committed %}
       <a href="{{cfbot_branch.apply_url}}">
        <span class="badge bg-warning" title="View git apply logs">Needs rebase!</span></a>
       Needs rebase {% cfsince cfbot_branch.needs_rebase_since %}. {%if cfbot_branch.failing_since and cfbot_branch.failing_since != cfbot_branch.needs_rebase_since %}Failing {% cfsince cfbot_branch.failing_since %}. {%endif%}<br>Additional links previous successfully applied patch (outdated):<br>

pgcommitfest/commitfest/views.py

@@ -431,7 +431,7 @@ def patchlist(request, cf, personalized=False):
                     SELECT 1 FROM commitfest_patch_authors cpa WHERE cpa.patch_id=p.id AND cpa.user_id=%(self)s
                 ) AND (
                     poc.status=%(needs_author)s
-                    OR branch.needs_rebase_since IS NOT NULL
+                    OR (branch.needs_rebase_since IS NOT NULL AND poc.status != %(committed_status)s)
                     OR branch.failing_since + interval '4 days' < now()
                     OR (%(is_committer)s AND poc.status=%(needs_committer)s)
                 )
@@ -451,6 +451,7 @@ def patchlist(request, cf, personalized=False):
         """
         whereparams["needs_author"] = PatchOnCommitFest.STATUS_AUTHOR
         whereparams["needs_committer"] = PatchOnCommitFest.STATUS_COMMITTER
+        whereparams["committed_status"] = PatchOnCommitFest.STATUS_COMMITTED
         whereparams["closed_status"] = CommitFest.STATUS_CLOSED
         is_committer = bool(Committer.objects.filter(user=request.user, active=True))
         whereparams["is_committer"] = is_committer
@@ -1278,7 +1279,18 @@ def close(request, patchid, status):
         "feedback": PatchOnCommitFest.STATUS_RETURNED,
         "committed": PatchOnCommitFest.STATUS_COMMITTED,
     }
-    poc.set_status(status_mapping[status])
+    new_status = status_mapping[status]
+    poc.set_status(new_status)
+
+    # Clear needs_rebase_since if patch is being marked as committed
+    if new_status == PatchOnCommitFest.STATUS_COMMITTED:
+        try:
+            cfbot_branch = poc.patch.cfbot_branch
+            if cfbot_branch.needs_rebase_since:
+                cfbot_branch.needs_rebase_since = None
+                cfbot_branch.save()
+        except CfbotBranch.DoesNotExist:
+            pass
 
     PatchHistory(
         patch=poc.patch,
@@ -1638,7 +1650,11 @@ def cfbot_ingest(message):
     # state so we can skip sending notifications if the needs_rebase status did
     # not change.
     needs_save = False
-    needs_rebase = branch_status["commit_id"] is None
+    # Check if patch is committed - committed patches don't need rebase
+    is_committed = PatchOnCommitFest.objects.filter(
+        patch=patch, status=PatchOnCommitFest.STATUS_COMMITTED
+    ).exists()
+    needs_rebase = branch_status["commit_id"] is None and not is_committed
     if bool(branch_in_db.needs_rebase_since) is not needs_rebase:
         if needs_rebase:
             branch_in_db.needs_rebase_since = datetime.now()