freq_name is only used locally

This doesn't need to be an object field.
This commit is contained in:
Andrew Ruthven 2022-12-10 01:48:29 +13:00 committed by Andrew Ruthven
parent aab8ddfd30
commit 6ad794eae6

View File

@ -674,18 +674,20 @@ class RepeatRule {
if ( preg_match('{BYHOUR=([0-9,]+)(;|$)}', $rrule, $m) ) $this->byhour = explode(',',$m[1]);
if ( !isset($this->interval) ) $this->interval = 1;
$freq_name = null;
switch( $this->freq ) {
case 'SECONDLY': $this->freq_name = 'second'; break;
case 'MINUTELY': $this->freq_name = 'minute'; break;
case 'HOURLY': $this->freq_name = 'hour'; break;
case 'DAILY': $this->freq_name = 'day'; break;
case 'WEEKLY': $this->freq_name = 'week'; break;
case 'MONTHLY': $this->freq_name = 'month'; break;
case 'YEARLY': $this->freq_name = 'year'; break;
case 'SECONDLY': $freq_name = 'second'; break;
case 'MINUTELY': $freq_name = 'minute'; break;
case 'HOURLY': $freq_name = 'hour'; break;
case 'DAILY': $freq_name = 'day'; break;
case 'WEEKLY': $freq_name = 'week'; break;
case 'MONTHLY': $freq_name = 'month'; break;
case 'YEARLY': $freq_name = 'year'; break;
default:
/** need to handle the error, but FREQ is mandatory so unlikely */
}
$this->frequency_string = sprintf('+%d %s', $this->interval, $this->freq_name );
$this->frequency_string = sprintf('+%d %s', $this->interval, $freq_name );
if ( DEBUG_RRULE ) printf( "Frequency modify string is: '%s', base is: '%s', TZ: %s\n", $this->frequency_string, $this->base->format('c'), $this->base->getTimeZone() );
$this->Start($return_floating_times);
}