cocoon-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Patrick Ahles <p.ah...@gmail.com>
Subject Re: svn commit: r312820 - /cocoon/branches/BRANCH_2_1_X/src/java/org/apache/cocoon/generation/CalendarGenerator.java
Date Tue, 11 Oct 2005 13:26:43 GMT
Hi Ralph,

this indeed looks somewhat confusing, but the IF is now _inside_ the
while loop...



On 10/11/05, Ralph Goers <Ralph.Goers@dslextreme.com> wrote:
> Is it just too early in the morning for me?  I don't see any difference
> other than spacing?
>
> Ralph
>
> hepabolu@apache.org wrote:
>
> >Author: hepabolu
> >Date: Tue Oct 11 00:02:02 2005
> >New Revision: 312820
> >
> >URL: http://svn.apache.org/viewcvs?rev=312820&view=rev
> >Log:
> >Bug 35413, patch applied, thanks to Patrick Ahles, now properly applied ;-)
> >
> >Modified:
> >    cocoon/branches/BRANCH_2_1_X/src/java/org/apache/cocoon/generation/CalendarGenerator.java
> >
> >Modified: cocoon/branches/BRANCH_2_1_X/src/java/org/apache/cocoon/generation/CalendarGenerator.java
> >URL: http://svn.apache.org/viewcvs/cocoon/branches/BRANCH_2_1_X/src/java/org/apache/cocoon/generation/CalendarGenerator.java?rev=312820&r1=312819&r2=312820&view=diff
> >==============================================================================
> >--- cocoon/branches/BRANCH_2_1_X/src/java/org/apache/cocoon/generation/CalendarGenerator.java
(original)
> >+++ cocoon/branches/BRANCH_2_1_X/src/java/org/apache/cocoon/generation/CalendarGenerator.java
Tue Oct 11 00:02:02 2005
> >@@ -308,11 +308,11 @@
> >                 this.contentHandler.endElement(URI, DAY_NODE_NAME,
> >                         PREFIX + ':' + DAY_NODE_NAME);
> >                 end.add(Calendar.DAY_OF_MONTH, 1);
> >+                              if (firstDay == end.get(Calendar.DAY_OF_WEEK)) {
> >+                                      this.contentHandler.endElement(URI, WEEK_NODE_NAME,
> >+                              PREFIX + ':' + WEEK_NODE_NAME);
> >+                              }
> >             }
> >-                      if (firstDay == end.get(Calendar.DAY_OF_WEEK)) {
> >-                              this.contentHandler.endElement(URI, WEEK_NODE_NAME,
> >-                        PREFIX + ':' + WEEK_NODE_NAME);
> >-                      }
> >         }
> >         this.contentHandler.endElement(URI, CALENDAR_NODE_NAME,
> >                 PREFIX + ':' + CALENDAR_NODE_NAME);
> >
> >
> >
> >
>


--
Patrick

----
Neutiquam erro. Magister mundi sum!

Mime
View raw message