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

Modified Files:
        NewsletterBounceAction.java 
        NewsletterSubscriberSearchForm.java NewsletterTermForm.java 
Log Message:
CMSC-899,Check and improve all lists according to guideline


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-899


Index: NewsletterBounceAction.java
===================================================================
RCS file: 
/var/cvs/contributions/CMSContainer_Portlets/portlets-newsletter/src/java/com/finalist/newsletter/forms/NewsletterBounceAction.java,v
retrieving revision 1.2
retrieving revision 1.3
diff -u -b -r1.2 -r1.3
--- NewsletterBounceAction.java 4 Aug 2008 03:11:05 -0000       1.2
+++ NewsletterBounceAction.java 22 Sep 2008 07:53:42 -0000      1.3
@@ -17,20 +17,20 @@
 import com.finalist.newsletter.domain.NewsletterBounce;
 import com.finalist.newsletter.util.NewsletterBounceUtil;
 
-public class NewsletterBounceAction  extends DispatchAction{
+public class NewsletterBounceAction  extends DispatchAction {
 
-   public ActionForward list(ActionMapping mapping, ActionForm 
form,HttpServletRequest request, HttpServletResponse response)
+   public ActionForward list (ActionMapping mapping, ActionForm form, 
HttpServletRequest request, HttpServletResponse response)
          throws Exception {
       int pageSize = 12;
       int offset = 0;
-      
if(StringUtils.isNotEmpty(PropertiesUtil.getProperty("repository.search.results.per.page")))
 {
+      if 
(StringUtils.isNotEmpty(PropertiesUtil.getProperty("repository.search.results.per.page")))
 {
          pageSize = 
Integer.parseInt(PropertiesUtil.getProperty("repository.search.results.per.page"));
       }
       String strOffset  = request.getParameter("offset");
-      if(StringUtils.isNotEmpty(strOffset)) {
+      if (StringUtils.isNotEmpty(strOffset)) {
          offset = Integer.parseInt(strOffset);
       }
-      List<NewsletterBounce> bounces =  
NewsletterBounceUtil.getBounceRecord(offset*pageSize, pageSize);
+      List < NewsletterBounce > bounces =  
NewsletterBounceUtil.getBounceRecord(offset * pageSize, pageSize);
       int count = NewsletterBounceUtil.getTotalCount();
       request.setAttribute("resultList", bounces);
       request.setAttribute("resultCount", count);
@@ -38,7 +38,7 @@
       return mapping.findForward("success");
    }
     
-    public ActionForward getItem(ActionMapping mapping, ActionForm 
form,HttpServletRequest request, HttpServletResponse response)
+    public ActionForward getItem(ActionMapping mapping, ActionForm form, 
HttpServletRequest request, HttpServletResponse response)
     throws Exception {
        String number  = request.getParameter("objectnumber");       
        NewsletterBounce bounce = 
NewsletterBounceUtil.getNewsletterBounce(Integer.parseInt(number));


Index: NewsletterSubscriberSearchForm.java
===================================================================
RCS file: 
/var/cvs/contributions/CMSContainer_Portlets/portlets-newsletter/src/java/com/finalist/newsletter/forms/NewsletterSubscriberSearchForm.java,v
retrieving revision 1.1
retrieving revision 1.2
diff -u -b -r1.1 -r1.2


Index: NewsletterTermForm.java
===================================================================
RCS file: 
/var/cvs/contributions/CMSContainer_Portlets/portlets-newsletter/src/java/com/finalist/newsletter/forms/NewsletterTermForm.java,v
retrieving revision 1.2
retrieving revision 1.3
diff -u -b -r1.2 -r1.3
--- NewsletterTermForm.java     31 Jul 2008 10:56:31 -0000      1.2
+++ NewsletterTermForm.java     22 Sep 2008 07:53:42 -0000      1.3
@@ -2,6 +2,11 @@
 
 import org.apache.struts.action.ActionForm;
 
+/**
+  *
+  *   newsletter term form
+  *
+  */
 public class NewsletterTermForm extends ActionForm {
 
    /**
@@ -40,12 +45,12 @@
       this.number = number;
       
    }
-   public void reset(){
+   public void reset() {
       name = null;
    }
-   public void clear(){
+   public void clear() {
       name = null;
-      offset = null;;
+      offset = null;
       number = null;
    }
 }
_______________________________________________
Cvs mailing list
[email protected]
http://lists.mmbase.org/mailman/listinfo/cvs

Reply via email to