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.