[ROL-1780] SiteModel.getNewWeblogs javadoc fix, functionality improvement and patch

• Type: Improvement
• Status: Closed
• Priority: Minor
• Resolution: Abandoned
• Affects Version/s: 4.0.1
• Component/s: Data Model & JPA Backend
• Assignee: Roller Unassigned
• Reporter: Manos Batsis
• Created: 19/Jan/09 13:20
• Updated: 17/Sep/23 20:48
• Resolved: 17/Sep/23 20:48

Description

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();