448429: end support for Bugzilla 3

Change-Id: I7ec71b76c9de4d685b9929105754fbe98025ca01
Task-Url: https://bugs.eclipse.org/bugs/show_bug.cgi?id=448429

2 files changed, 28 insertions, 28 deletions

diff --git a/org.eclipse.mylyn.bugzilla.core/src/org/eclipse/mylyn/internal/bugzilla/core/CustomTransitionManager.java b/org.eclipse.mylyn.bugzilla.core/src/org/eclipse/mylyn/internal/bugzilla/core/CustomTransitionManager.java
index 48023040f..c9725b830 100644
--- a/org.eclipse.mylyn.bugzilla.core/src/org/eclipse/mylyn/internal/bugzilla/core/CustomTransitionManager.java
+++ b/org.eclipse.mylyn.bugzilla.core/src/org/eclipse/mylyn/internal/bugzilla/core/CustomTransitionManager.java
@@ -76,7 +76,7 @@ public class CustomTransitionManager implements Serializable {
 	/**
 	 * Searches for a valid transition description file. Returns true if a file exists and was sucessfully parsed, false
 	 * otherwise
-	 * 
+	 *
 	 * @param filePath
 	 * @return true if anything was changed, false otherwise.
 	 * @throws CoreException
@@ -204,7 +204,7 @@ public class CustomTransitionManager implements Serializable {
 			operationMapByEndStatus.put("REOPENED", list); //$NON-NLS-1$
 
 			list = new ArrayList<AbstractBugzillaOperation>();
-			list.add(BugzillaOperation.reopen);
+			list.add(BugzillaOperation.unconfirmed);
 			operationMapByEndStatus.put("UNCONFIRMED", list); //$NON-NLS-1$
 
 			list = new ArrayList<AbstractBugzillaOperation>();
@@ -240,7 +240,7 @@ public class CustomTransitionManager implements Serializable {
 	/**
 	 * Sets whether or not this class is valid. If set to false, the filePath will be set to "" so that subsequent calls
 	 * to parse(filePath) will work.
-	 * 
+	 *
 	 * @param val
 	 */
 	public void setValid(boolean val) {
@@ -258,7 +258,7 @@ public class CustomTransitionManager implements Serializable {
 	 * Returns the duplicate status. Standard Bugzilla installations will have a duplicate status of RESOLVED, VERIFIED
 	 * or CLOSED. <br>
 	 * By default, the duplicate status will be RESOLVED.
-	 * 
+	 *
 	 * @return The status to send if a bug is set to Duplicate
 	 */
 	public String getDuplicateStatus() {
@@ -274,7 +274,7 @@ public class CustomTransitionManager implements Serializable {
 	 * permissions setting currently cannot be accessed, so this function does not try to guess and returns either the
 	 * default status (NEW) or whichever status was set by a transition file. <br>
 	 * Fix for bug #318128, set startStatus to NEW if startStatus is somehow null at this point.
-	 * 
+	 *
 	 * @return The valid start status. Default value is NEW.
 	 */
 	public String getStartStatus() {
@@ -380,7 +380,7 @@ public class CustomTransitionManager implements Serializable {
 	/**
 	 * Creates a new status with a single operation with the same name as the status itself. Does nothing if a status of
 	 * that name already exists.
-	 * 
+	 *
 	 * @param status
 	 */
 	private void addNewStatus(String status) {
diff --git a/org.eclipse.mylyn.bugzilla.core/src/org/eclipse/mylyn/internal/bugzilla/core/RepositoryConfiguration.java b/org.eclipse.mylyn.bugzilla.core/src/org/eclipse/mylyn/internal/bugzilla/core/RepositoryConfiguration.java
index c06f7bb9b..087da5b2a 100644
--- a/org.eclipse.mylyn.bugzilla.core/src/org/eclipse/mylyn/internal/bugzilla/core/RepositoryConfiguration.java
+++ b/org.eclipse.mylyn.bugzilla.core/src/org/eclipse/mylyn/internal/bugzilla/core/RepositoryConfiguration.java
@@ -34,7 +34,7 @@ import org.eclipse.mylyn.tasks.core.data.TaskOperation;
 
 /**
  * Class describing the configuration of products and components for a given Bugzilla installation.
- * 
+ *
  * @author Rob Elves
  * @author Charley Wang
  * @author Frank Becker
@@ -354,7 +354,7 @@ public class RepositoryConfiguration implements Serializable {
 	/**
 	 * Create a custom transition manager. If fileName is invalid, the resulting transition manager will also be
 	 * invalid.
-	 * 
+	 *
 	 * @param fileName
 	 * @throws CoreException
 	 */
@@ -727,7 +727,7 @@ public class RepositoryConfiguration implements Serializable {
 			} else {
 				addOperation(bugReport, BugzillaOperation.none);
 				for (AbstractBugzillaOperation b : validTransitions.getValidTransitions(attributeStatus.getValue())) {
-					//Special case: the CLOSED status needs a Resolution input. 
+					//Special case: the CLOSED status needs a Resolution input.
 					//This happens automatically if current status is RESOLVED, else we need to supply one
 					if (b.toString().equals(BugzillaOperation.close.toString())) {
 						if (attributeStatus.getValue().equals("RESOLVED") && b.getInputId() != null) { //$NON-NLS-1$
@@ -806,11 +806,11 @@ public class RepositoryConfiguration implements Serializable {
 		if (operationAttribute == null) {
 			operationAttribute = bugReport.getRoot().createAttribute(key.getKey());
 			operationAttribute.getMetaData()
-					.defaults()
-					.setReadOnly(key.isReadOnly())
-					.setKind(key.getKind())
-					.setLabel(key.toString())
-					.setType(key.getType());
+			.defaults()
+			.setReadOnly(key.isReadOnly())
+			.setKind(key.getKind())
+			.setLabel(key.toString())
+			.setType(key.getType());
 			operationAttribute.setValue("0"); //$NON-NLS-1$
 		}
 		operationAttribute = bugReport.getRoot().getMappedAttribute(TaskAttribute.USER_ASSIGNED);
@@ -840,7 +840,7 @@ public class RepositoryConfiguration implements Serializable {
 			//Handle custom operations. Currently only tuned for transitions based on default status names
 			addOperation(bugReport, BugzillaOperation.none);
 			for (AbstractBugzillaOperation b : validTransitions.getValidTransitions(attributeStatus.getValue())) {
-				//Special case: the CLOSED status needs a Resolution input. 
+				//Special case: the CLOSED status needs a Resolution input.
 				//This happens automatically if current status is RESOLVED, else we need to supply one
 				if (b.toString().equals(BugzillaOperation.close.toString())) {
 					if (attributeStatus.getValue().equals("RESOLVED") && b.getInputId() != null) { //$NON-NLS-1$
@@ -855,7 +855,7 @@ public class RepositoryConfiguration implements Serializable {
 			}
 		} else {
 //			Eclipse Bugzilla State transitions
-//			UNCONFIRMED	
+//			UNCONFIRMED
 //			NEW				ASSIGNED RESOLVED# CLOSED#
 //			ASSIGNED		NEW RESOLVED# CLOSED#
 //			REOPENED		NEW ASSIGNED RESOLVED# CLOSED#
@@ -868,7 +868,7 @@ public class RepositoryConfiguration implements Serializable {
 				addOperation(bugReport, BugzillaOperation.none);
 				addOperation(bugReport, BugzillaOperation.accept);
 				addOperation(bugReport, BugzillaOperation.resolve);
-				// This is not the standard workflow add with bug 326216 to support eclipse.org				
+				// This is not the standard workflow add with bug 326216 to support eclipse.org
 				if (bugzillaVersion.compareMajorMinorOnly(BugzillaVersion.BUGZILLA_3_2) >= 0) {
 					addOperation(bugReport, BugzillaOperation.close_with_resolution);
 				}
@@ -879,7 +879,7 @@ public class RepositoryConfiguration implements Serializable {
 				addOperation(bugReport, BugzillaOperation.none);
 				addOperation(bugReport, BugzillaOperation.accept);
 				addOperation(bugReport, BugzillaOperation.resolve);
-				// This is not the standard workflow add with bug 326216 to support eclipse.org				
+				// This is not the standard workflow add with bug 326216 to support eclipse.org
 				if (bugzillaVersion.compareMajorMinorOnly(BugzillaVersion.BUGZILLA_3_2) >= 0) {
 					addOperation(bugReport, BugzillaOperation.close_with_resolution);
 				}
@@ -891,7 +891,7 @@ public class RepositoryConfiguration implements Serializable {
 			case ASSIGNED:
 				addOperation(bugReport, BugzillaOperation.none);
 				addOperation(bugReport, BugzillaOperation.resolve);
-				// This is not the standard workflow add with bug 326216 to support eclipse.org				
+				// This is not the standard workflow add with bug 326216 to support eclipse.org
 				if (bugzillaVersion.compareMajorMinorOnly(BugzillaVersion.BUGZILLA_3_2) >= 0) {
 					addOperation(bugReport, BugzillaOperation.close_with_resolution);
 				}
@@ -948,11 +948,11 @@ public class RepositoryConfiguration implements Serializable {
 			if (operationAttribute == null) {
 				operationAttribute = bugReport.getRoot().createAttribute(key.getKey());
 				operationAttribute.getMetaData()
-						.defaults()
-						.setReadOnly(key.isReadOnly())
-						.setKind(key.getKind())
-						.setLabel(key.toString())
-						.setType(key.getType());
+				.defaults()
+				.setReadOnly(key.isReadOnly())
+				.setKind(key.getKind())
+				.setLabel(key.toString())
+				.setType(key.getType());
 				operationAttribute.setValue("0"); //$NON-NLS-1$
 			}
 			operationAttribute = bugReport.getRoot().getMappedAttribute(TaskAttribute.USER_ASSIGNED);
@@ -1227,7 +1227,7 @@ public class RepositoryConfiguration implements Serializable {
 	public String getDuplicateStatus() {
 		return validTransitions == null
 				? IBugzillaConstants.BUGZILLA_REPORT_STATUS.RESOLVED.toString()
-				: validTransitions.getDuplicateStatus();
+						: validTransitions.getDuplicateStatus();
 	}
 
 	public String getStartStatus() {
@@ -1235,9 +1235,9 @@ public class RepositoryConfiguration implements Serializable {
 			return version.compareMajorMinorOnly(BugzillaVersion.BUGZILLA_4_0) < 0
 					|| !(getOptionValues(BugzillaAttribute.BUG_STATUS).contains(
 							BUGZILLA_REPORT_STATUS_4_0.IN_PROGRESS.toString()) || getOptionValues(
-							BugzillaAttribute.BUG_STATUS).contains(BUGZILLA_REPORT_STATUS_4_0.CONFIRMED.toString()))
-					? IBugzillaConstants.BUGZILLA_REPORT_STATUS.NEW.toString()
-					: IBugzillaConstants.BUGZILLA_REPORT_STATUS_4_0.CONFIRMED.toString();
+									BugzillaAttribute.BUG_STATUS).contains(BUGZILLA_REPORT_STATUS_4_0.CONFIRMED.toString()))
+									? IBugzillaConstants.BUGZILLA_REPORT_STATUS.NEW.toString()
+											: IBugzillaConstants.BUGZILLA_REPORT_STATUS_4_0.CONFIRMED.toString();
 		} else {
 			return validTransitions.getStartStatus();
 		}