[
https://issues.apache.org/jira/browse/ROL-1780?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Glen Mazza updated ROL-1780:
----------------------------
Component/s: Data Model & JPA Backend
> SiteModel.getNewWeblogs javadoc fix, functionality improvement and patch
> ------------------------------------------------------------------------
>
> Key: ROL-1780
> URL: https://issues.apache.org/jira/browse/ROL-1780
> Project: Apache Roller
> Issue Type: Improvement
> Components: Data Model & JPA Backend
> Affects Versions: 4.0.1
> Reporter: Manos Batsis
> Assignee: Roller Unassigned
> Priority: Minor
> Attachments: ROL-1780.patch.txt,
> sitemodel_getNewWeblogs_roller5trunk_patch.txt
>
>
> 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 was sent by Atlassian JIRA
(v6.2#6252)
