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

Modified Files:
        NewsletterSubscriptionAddRelAction.java 
Log Message:
CMSC-990 modify addsubscriber


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


Index: NewsletterSubscriptionAddRelAction.java
===================================================================
RCS file: 
/var/cvs/contributions/CMSContainer_Portlets/portlets-newsletter/src/java/com/finalist/newsletter/forms/NewsletterSubscriptionAddRelAction.java,v
retrieving revision 1.3
retrieving revision 1.4
diff -u -b -r1.3 -r1.4
--- NewsletterSubscriptionAddRelAction.java     28 Jul 2008 14:46:13 -0000      
1.3
+++ NewsletterSubscriptionAddRelAction.java     28 Jul 2008 15:02:15 -0000      
1.4
@@ -39,18 +39,13 @@
                log.debug("With parameter subscribeNewsletters,go to search 
page");
                if 
(StringUtils.isNotBlank(request.getParameter("newsletterId"))) {
                        int newsletterId = 
Integer.parseInt(request.getParameter("newsletterId"));
-                       Enumeration<String> parameterNames = 
request.getParameterNames();
-                       while (parameterNames.hasMoreElements()) {
-                               String paraName = 
parameterNames.nextElement().trim();
-                               if (paraName.startsWith("chk_")) {
-                                       int chkNumber = 
Integer.parseInt(request.getParameter(paraName));
-                                       if 
(subscriptionServices.noSubscriptionRecord(chkNumber, newsletterId)) {
-                                               
subscriptionServices.addNewRecord(chkNumber, newsletterId);
+                       String[] authIds = request.getParameterValues("chk_");
+                       for (String authId : authIds) {
+                               if 
(subscriptionServices.noSubscriptionRecord(Integer.parseInt(authId), 
newsletterId)) {
+                                       
subscriptionServices.addNewRecord(Integer.parseInt(authId), newsletterId);
                                        }
                                }
                        }
-               }
-
                ActionForward ret = new 
ActionForward(mapping.findForward("success").getPath() + "?newsletterId=" + 
request.getParameter("newsletterId"));
                ret.setRedirect(true);
                return ret;
_______________________________________________
Cvs mailing list
[email protected]
http://lists.mmbase.org/mailman/listinfo/cvs

Reply via email to