Integration release of NQt GerritRC-V2.2.1-INT-014BL-QTQAINFRA-357

  Fixed JIRA issues:
  167 Fixed gerrit email verification link requires insecure login
  195 Added support for European style dates
  220 Added a check to prevent NumberFormatException
  270 Fixed review database update from StagingApprove
  335 Fixed automatic update of reviewer list
  340 Don't send mail for publishing comments for stage approved
  347 Fixed late removal of review approvals
  348 Fixed cherry pick footer settings
  350 Consider sanity review column
  352 Fix typo in project config's topic review checkbox label
  355 Hide review panel also on diff and topic pages
  366 Fixed QTQAINFRA-366, buttons are enabled when all changes are max pos reviwed
  372 Validate topic current change set
  375 Topic permalink copy to clipboard corrected
  381 Do not permit trailing slash when pushing
  382   Integration release of NQt Gerrit
  382 Added a check to prevent NullPointerException Added a check to prevent ArrayIndexOutOfBoundsException
  385 Keep highlight on review request
  389 Set patch approval changeOpen to false
  411 Updated to use current patch id in loadFileList method.
  456 Remove leading and trailing whitespace in username when creating a request
  462 Updated for eclipse 3.7, m2e and fixed the full name of the MAVEN2_CLASSPATH_CONTAINER
  468 Fixed the Review score put on wrong change

Change-Id: I067e518444be45b51dd63aea77f37b9edca06540

1 files changed, 0 insertions, 34 deletions

diff --git a/gerrit-server/src/main/java/com/google/gerrit/server/project/TopicControl.java b/gerrit-server/src/main/java/com/google/gerrit/server/project/TopicControl.java
index b93632d659..af6f1319bc 100644
--- a/gerrit-server/src/main/java/com/google/gerrit/server/project/TopicControl.java
+++ b/gerrit-server/src/main/java/com/google/gerrit/server/project/TopicControl.java
@@ -14,7 +14,6 @@
 
 package com.google.gerrit.server.project;
 
-import com.google.gerrit.common.data.ApprovalType;
 import com.google.gerrit.common.data.ApprovalTypes;
 import com.google.gerrit.common.data.PermissionRange;
 import com.google.gerrit.reviewdb.ChangeSet;
@@ -24,7 +23,6 @@ import com.google.gerrit.reviewdb.ReviewDb;
 import com.google.gerrit.reviewdb.Topic;
 import com.google.gerrit.server.CurrentUser;
 import com.google.gerrit.server.IdentifiedUser;
-import com.google.gerrit.server.workflow.TopicCategoryFunction;
 import com.google.gerrit.server.workflow.TopicFunctionState;
 import com.google.gwtorm.client.OrmException;
 import com.google.inject.Inject;
@@ -221,22 +219,6 @@ public class TopicControl {
       return result;
     }
 
-    final List<ChangeSetApproval> all =
-      db.changeSetApprovals().byChangeSet(changeSetId).toList();
-
-    final TopicFunctionState fs =
-        functionStateFactory.create(topic, changeSetId, all);
-
-    for (ApprovalType c : approvalTypes.getApprovalTypes()) {
-      TopicCategoryFunction.forCategory(c.getCategory()).run(c, fs);
-    }
-
-    for (ApprovalType type : approvalTypes.getApprovalTypes()) {
-      if (!fs.isValid(type)) {
-        return new CanSubmitResult("Requires " + type.getCategory().getName());
-      }
-    }
-
     return CanSubmitResult.OK;
   }
 
@@ -282,22 +264,6 @@ public class TopicControl {
       return result;
     }
 
-    final List<ChangeSetApproval> all =
-      db.changeSetApprovals().byChangeSet(changeSetId).toList();
-
-    final TopicFunctionState fs =
-        functionStateFactory.create(topic, changeSetId, all);
-
-    for (ApprovalType c : approvalTypes.getApprovalTypes()) {
-      TopicCategoryFunction.forCategory(c.getCategory()).run(c, fs);
-    }
-
-    for (ApprovalType type : approvalTypes.getApprovalTypes()) {
-      if (!fs.isValid(type)) {
-        return new CanSubmitResult("Requires " + type.getCategory().getName());
-      }
-    }
-
     return CanSubmitResult.OK;
   }
 }