continuum-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Carlos Sanchez" <car...@apache.org>
Subject Re: svn commit: r443424 - in /maven/continuum/trunk/continuum-webapp/src/main: java/org/apache/maven/continuum/web/action/ScheduleAction.java resources/localization/Continuum.properties webapp/editSchedule.jsp
Date Fri, 15 Sep 2006 01:00:36 GMT
Fixed the loop and reopened issue

On 9/14/06, Brett Porter <brett@apache.org> wrote:
> I was referring to the table elements in the JSP, but I'll also note
> that the while loop and switch below are unnecessary.
>
> second = cronEx[0];
> minute = cronEx[1];
>
> etc.
>
> - Brett
>
> On 15/09/2006, at 9:23 AM, Brett Porter wrote:
>
> > This suffers the same limitations that were originally found on the
> > same patch for archiva. Are you sure this is the latest?
> >
> > On 15/09/2006, at 3:36 AM, carlos@apache.org wrote:
> >
> >> +
> >> +                String[] cronEx = schedule.getCronExpression
> >> ().split( " " );
> >> +                int i = 0;
> >> +                while ( i < cronEx.length )
> >> +                {
> >> +                    switch( i )
> >> +                    {
> >> +                        case 0 : second = cronEx[i]; break;
> >> +                        case 1 : minute = cronEx[i]; break;
> >> +                        case 2 : hour = cronEx[i]; break;
> >> +                        case 3 : dayOfMonth = cronEx[i]; break;
> >> +                        case 4 : month = cronEx[i]; break;
> >> +                        case 5 : dayOfWeek = cronEx[i]; break;
> >> +                        case 6 : year = cronEx[i]; break;
> >> +                    }
> >> +                    i++;
> >> +                }
> >> +
> >>
>


-- 
I could give you my word as a Spaniard.
No good. I've known too many Spaniards.
                             -- The Princess Bride

Mime
View raw message