# HG changeset patch
# User Thomas De Schampheleire <thomas.de_schampheleire@nokia.com>
# Date 2018-11-10 21:35:54
# Node ID 471b29a2f302e8cb4586fac1f98f20f8038a7817
# Parent  9de5c174a37c5a97caec77c1d28425ddeaa0e4e4

model: changeset_status: remove support for 'dont_allow_on_closed_pull_request' and 'StatusChangeOnClosedPullRequestError'

Previous commit removed the only usage of this parameter and exception.

diff --git a/kallithea/lib/exceptions.py b/kallithea/lib/exceptions.py
--- a/kallithea/lib/exceptions.py
+++ b/kallithea/lib/exceptions.py
@@ -59,10 +59,6 @@ class UserGroupsAssignedException(Except
     pass
 
 
-class StatusChangeOnClosedPullRequestError(Exception):
-    pass
-
-
 class AttachedForksError(Exception):
     pass
 
diff --git a/kallithea/model/changeset_status.py b/kallithea/model/changeset_status.py
--- a/kallithea/model/changeset_status.py
+++ b/kallithea/model/changeset_status.py
@@ -29,7 +29,6 @@ import logging
 from sqlalchemy.orm import joinedload
 
 from kallithea.model.db import ChangesetStatus, PullRequest, Repository, User, Session
-from kallithea.lib.exceptions import StatusChangeOnClosedPullRequestError
 
 log = logging.getLogger(__name__)
 
@@ -133,7 +132,7 @@ class ChangesetStatusModel(object):
         return status
 
     def set_status(self, repo, status, user, comment, revision=None,
-                   pull_request=None, dont_allow_on_closed_pull_request=False):
+                   pull_request=None):
         """
         Creates new status for changeset or updates the old ones bumping their
         version, leaving the current status at the value of 'status'.
@@ -144,9 +143,6 @@ class ChangesetStatusModel(object):
         :param comment:
         :param revision:
         :param pull_request:
-        :param dont_allow_on_closed_pull_request: don't allow a status change
-            if last status was for pull request and it's closed. We shouldn't
-            mess around this manually
         """
         repo = Repository.guess_instance(repo)
 
@@ -165,15 +161,6 @@ class ChangesetStatusModel(object):
             revisions = pull_request.revisions
         cur_statuses = q.all()
 
-        # if statuses exists and last is associated with a closed pull request
-        # we need to check if we can allow this status change
-        if (dont_allow_on_closed_pull_request and cur_statuses
-            and getattr(cur_statuses[0].pull_request, 'status', '')
-                == PullRequest.STATUS_CLOSED):
-            raise StatusChangeOnClosedPullRequestError(
-                'Changing status on closed pull request is not allowed'
-            )
-
         # update all current statuses with older version
         for st in cur_statuses:
             st.version += 1