[MMBASE CVS] contributions/CMSContainer_Portlets/portlets-newsletter/src/java/com/finalist/newsletter/publisher NewsletterPublisher.java

[jstroomer]

Update of 
/var/cvs/contributions/CMSContainer_Portlets/portlets-newsletter/src/java/com/finalist/newsletter/publisher
In directory 
james.mmbase.org:/tmp/cvs-serv23292/portlets-newsletter/src/java/com/finalist/newsletter/publisher

Modified Files:
        NewsletterPublisher.java 
Log Message:
CMSC-632 Major refactoring of module section and some minor fixes


See also: 
http://cvs.mmbase.org/viewcvs/contributions/CMSContainer_Portlets/portlets-newsletter/src/java/com/finalist/newsletter/publisher
See also: http://www.mmbase.org/jira/browse/CMSC-632


Index: NewsletterPublisher.java
===================================================================
RCS file: 
/var/cvs/contributions/CMSContainer_Portlets/portlets-newsletter/src/java/com/finalist/newsletter/publisher/NewsletterPublisher.java,v
retrieving revision 1.1
retrieving revision 1.2
diff -u -b -r1.1 -r1.2
--- NewsletterPublisher.java    8 Jan 2008 10:45:00 -0000       1.1
+++ NewsletterPublisher.java    17 Jan 2008 21:47:15 -0000      1.2
@@ -29,10 +29,10 @@
 
    private static Logger log = 
Logging.getLoggerInstance(NewsletterPublisher.class.getName());
 
-   private String publicationNumber;
+   private int publicationNumber;
    private Cloud cloud;
 
-   public NewsletterPublisher(String publicationNumber) {
+   public NewsletterPublisher(int publicationNumber) {
       this.publicationNumber = publicationNumber;
       this.cloud = CloudProviderFactory.getCloudProvider().getCloud();
    }
@@ -40,12 +40,11 @@
    private void createConfirmationList(List<String> subscribers) {
       for (int s = 0; s < subscribers.size(); s++) {
          String userName = subscribers.get(s);
-         NewsletterCommunication.setUserPreference(userName, 
UNSENT_NEWSLETTER, publicationNumber);
+         NewsletterCommunication.setUserPreference(userName, 
UNSENT_NEWSLETTER, String.valueOf(publicationNumber));
       }
    }
 
-   private Message generateNewsletter(String userName, String 
publicationNumber, String mimeType) {
-      log.debug("Request to generate a newsletter for user " + userName + " 
from publication " + publicationNumber + " with mimetype " + mimeType);
+   private Message generateNewsletter(String userName, int publicationNumber, 
String mimeType) {
       NewsletterGeneratorFactory factory = 
NewsletterGeneratorFactory.getInstance();
       NewsletterGenerator generator = 
factory.getNewsletterGenerator(publicationNumber, mimeType);
       if (generator != null) {
@@ -56,7 +55,7 @@
    }
 
    private void removeFromConfirmationList(String userName) {
-      NewsletterCommunication.removeUserPreference(userName, 
UNSENT_NEWSLETTER, publicationNumber);
+      NewsletterCommunication.removeUserPreference(userName, 
UNSENT_NEWSLETTER, String.valueOf(publicationNumber));
    }
 
    @Override
@@ -83,7 +82,8 @@
       message.setFrom(new InternetAddress("[EMAIL PROTECTED]"));
 
       String userEmail = NewsletterCommunication.getUserPreference(userName, 
"email");
-      message.setRecipient(RecipientType.TO, new InternetAddress(userEmail));
+      InternetAddress toAddress = new InternetAddress(userEmail);
+      message.setRecipient(RecipientType.TO, toAddress);
 
       String subject = publicationNode.getStringValue("subject");
       message.setSubject(subject);
_______________________________________________
Cvs mailing list
Cvs@lists.mmbase.org
http://lists.mmbase.org/mailman/listinfo/cvs