Author: norman
Date: Sun Aug  2 10:59:55 2009
New Revision: 800026

URL: http://svn.apache.org/viewvc?rev=800026&view=rev
Log:
Correctly display Cc and To headers

Modified:
    labs/hupa/src/main/java/org/apache/hupa/client/mvp/IMAPMessagePresenter.java

Modified: 
labs/hupa/src/main/java/org/apache/hupa/client/mvp/IMAPMessagePresenter.java
URL: 
http://svn.apache.org/viewvc/labs/hupa/src/main/java/org/apache/hupa/client/mvp/IMAPMessagePresenter.java?rev=800026&r1=800025&r2=800026&view=diff
==============================================================================
--- 
labs/hupa/src/main/java/org/apache/hupa/client/mvp/IMAPMessagePresenter.java 
(original)
+++ 
labs/hupa/src/main/java/org/apache/hupa/client/mvp/IMAPMessagePresenter.java 
Sun Aug  2 10:59:55 2009
@@ -85,31 +85,23 @@
 
        private void updateDisplay() {
                display.getFrom().setText(message.getHeader().getFrom());
-               
-               StringBuffer ccBuf = new StringBuffer();
-               ArrayList<String> ccList = message.getHeader().getCc();
-               for (int i = 0; i < ccList.size(); i++) {
-                       ccBuf.append(ccList.get(i));
-                       if (i < ccList.size()) {
-                               ccBuf.append(", ");
-                       }
-               }
-               display.getCc().setText(ccBuf.toString());;
-               
-               StringBuffer toBuf = new StringBuffer();
-               ArrayList<String> toList = message.getHeader().getCc();
-               for (int i = 0; i < toList.size(); i++) {
-                       toBuf.append(toList.get(i));
-                       if (i < toList.size()) {
-                               toBuf.append(", ");
-                       }
-               }
-
-               display.getTo().setText(toBuf.toString());;
+               
display.getCc().setText(arrayToString(message.getHeader().getCc()));
+               
display.getTo().setText(arrayToString(message.getHeader().getTo()));
                display.getSubject().setText(message.getHeader().getSubject());
                
display.getContent().setHTML(message.getMessageContent().getText());
                
display.setAttachments(message.getMessageContent().getMessageAttachments(),user.getSessionId(),
 folder.getFullName(),message.getUid());
        }
+       
+       private String arrayToString(ArrayList<String> aList) {
+               StringBuffer sb = new StringBuffer();
+               for (int i = 0; i < aList.size(); i++) {
+                       sb.append(aList.get(i));
+                       if (i < aList.size()-1) {
+                               sb.append(", ");
+                       }
+               }
+               return sb.toString();
+       }
 
        @Override
        public Place getPlace() {



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to