You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@roller.apache.org by "Manos Batsis (JIRA)" <ji...@apache.org> on 2009/01/19 14:24:11 UTC
[jira] Updated: (ROL-1780) SiteModel.getNewWeblogs javadoc fix,
functionality improovement and patch
[ https://issues.apache.org/roller/browse/ROL-1780?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Manos Batsis updated ROL-1780:
------------------------------
Attachment: ROL-1780.patch.txt
Attaching a proper patch; sorry, not used to jira.
> 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
> Attachments: ROL-1780.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 is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.