continuum-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Brett Porter <br...@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 Thu, 14 Sep 2006 23:53:43 GMT
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++;
>> +                }
>> +
>>

Mime
View raw message