roller-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Manos Batsis (JIRA)" <j...@apache.org>
Subject [jira] Created: (ROL-1780) SiteModel.getNewWeblogs javadoc fix, functionality improovement and patch
Date Mon, 19 Jan 2009 13:22:12 GMT
SiteModel.getNewWeblogs javadoc fix, functionality improovement and patch
-------------------------------------------------------------------------

                 Key: ROL-1780
                 URL: https://issues.apache.org/roller/browse/ROL-1780
             Project: Roller
          Issue Type: Improvement
    Affects Versions: 4.0.1
            Reporter: Manos Batsis
            Assignee: Roller Unassigned
            Priority: Minor


The patch that follows fixes the method's javadoc and adds the ability to just retrieve a
list of blogs if the number of days is a negative integer

Index: /Users/manos/lib/roller_4.1/apps/weblogger/src/java/org/apache/roller/weblogger/ui/rendering/model/SiteModel.java
===================================================================
--- /Users/manos/lib/roller_4.1/apps/weblogger/src/java/org/apache/roller/weblogger/ui/rendering/model/SiteModel.java
(revision 735639)
+++ /Users/manos/lib/roller_4.1/apps/weblogger/src/java/org/apache/roller/weblogger/ui/rendering/model/SiteModel.java
(working copy)
@@ -391,14 +391,19 @@
     /*
      * Get most collection of Website objects,
      * in reverse chrono order by creationDate.
-     * @param offset   Offset into results (for paging)
-     * @param len      Max number of results to return
+     * @param sinceDays   What "new" means in days
+     * @param length      Max number of results to return
      */
     public List getNewWeblogs(int sinceDays, int length) {
         List results = new ArrayList();
-        Calendar cal = Calendar.getInstance();
-        cal.setTime(new Date());
-        cal.add(Calendar.DATE, -1 * sinceDays);
+    	Calendar cal = Calendar.getInstance();
+        if(sinceDays < 0){
+        	cal.set(1970, 0, 1);
+        }
+        else{
+            cal.setTime(new Date());
+            cal.add(Calendar.DATE, -1 * sinceDays);
+        }
         Date startDate = cal.getTime();
         try {            
             Weblogger roller = WebloggerFactory.getWeblogger();


-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message