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] Updated: (ROL-1780) SiteModel.getNewWeblogs javadoc fix, functionality improovement and patch
Date Thu, 04 Mar 2010 19:57:27 GMT

     [ https://issues.apache.org/jira/browse/ROL-1780?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]

Manos Batsis updated ROL-1780:
------------------------------

    Attachment: sitemodel_getNewWeblogs_roller5trunk_patch.txt

new patch for roller 5 on trunk.

> SiteModel.getNewWeblogs javadoc fix, functionality improovement and patch
> -------------------------------------------------------------------------
>
>                 Key: ROL-1780
>                 URL: https://issues.apache.org/jira/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, 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 is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


Mime
View raw message