Merge pull request #6 from anrchen/issue-7/refactoring

Issue 7/refactoring

src/java/fp/v11/Splints/FootPrints11.java → src/java/fp/v11/splints/FootPrints11.java

@@ -1,5 +1,6 @@
 package fp.v11.splints;
 
+import java.util.Collection;
 import javax.xml.soap.MessageFactory;
 import javax.xml.soap.SOAPBody;
 import javax.xml.soap.SOAPConnection;
@@ -8,6 +9,8 @@
 import javax.xml.soap.SOAPEnvelope;
 import javax.xml.soap.SOAPMessage;
 import javax.xml.soap.Text;
+import org.w3c.dom.Node;
+import org.w3c.dom.NodeList;
 
 /**
  * 
@@ -200,9 +203,10 @@ public String createIssue() {
     }
 
     @Override
-    public void getIssueDetails() {
+    public NodeList getIssueDetails() {
         String projectId = "PROJECT_ID";
         String ticketNumber = "TICKET_NUMBER";
+        NodeList result = null;
 
         try {
             /*
@@ -268,6 +272,7 @@ public void getIssueDetails() {
 
             }
             DumpSOAPElement(replybody, 0);
+            result = replybody.getChildNodes();
 
         } catch (Exception ex) {
 
@@ -276,16 +281,18 @@ public void getIssueDetails() {
         }
 
         System.out.println("Done");
+        return result;
     }
 
     @Override
-    public void linkIssues() {
+    public NodeList linkIssues() {
         String linkType = "LINK_TYPE";
         String projectId1 = "PROJECT_ID1";
         String ticketNumber1 = "TICKET_NUMBER1";
         String projectId2 = "PROJECT_ID2";
         String ticketNumber2 = "TICKET_NUMBER2";
-
+        NodeList result = null;
+
         try {
             /*
             // Comment this out for NO proxy.
@@ -364,7 +371,7 @@ public void linkIssues() {
             //reply.writeTo( System.out );
             // Get result
             SOAPBody replybody = reply.getSOAPPart().getEnvelope().getBody();
-
+            result = replybody.getChildNodes();
             // Check for error
             if (replybody.hasFault()) {
                 throw new Exception(replybody.getFault().getFaultString());
@@ -381,6 +388,7 @@ public void linkIssues() {
         } catch (Exception ex) {
             ex.printStackTrace();
         }
+        return result;
     }
 
     /**

src/java/fp/v11/Splints/ISplints.java → src/java/fp/v11/splints/ISplints.java

@@ -1,5 +1,7 @@
 package fp.v11.splints;
 
+import org.w3c.dom.NodeList;
+
 /**
  * Interface for interacting with Splints
  * @author soen487-w18-team03
@@ -16,11 +18,11 @@ public interface ISplints {
      * Gets issue details
      * 
      */
-    public void getIssueDetails();
+    public NodeList getIssueDetails();
 
     /**
      * Links two issues
      * 
      */
-    public void linkIssues();
+    public NodeList linkIssues();
 }

src/java/fp/v11/Splints/PromptCredentialsProvider.java → src/java/fp/v11/splints/PromptCredentialsProvider.java

@@ -13,8 +13,8 @@ public class PromptCredentialsProvider {
 
     private final Console console;
     private final Scanner scanner;
-    private final String USERNAME_INPUT_TEXT = "Your username: ";
-    private final String PASSWORD_INPUT_TEXT = "Your password: ";
+    public static final String USERNAME_INPUT_TEXT = "Your username: ";
+    public static final String PASSWORD_INPUT_TEXT = "Your password: ";
 
     public PromptCredentialsProvider() {
         this.scanner = new Scanner(System.in);