[Plugin: WP Super Cache] Expiry Time & Garbage Collection

hi. thanks for the note. here’s what i’ve found:

1. the development version is much more helpful because it says “UTC” more often.

2. the Interval is still showing “Once Weekly”. wp_get_schedules() is returning the usual choices. the reason why “Once Weekly” is selected is because none of the choices are selected, viz.,

<select id='cache_scheduled_select' name='cache_schedule_interval' size=1>
<option name='weekly'  /> weekly</option>
<option name='bimonthly'  /> bimonthly</option>
<option name='hourly'  /> hourly</option>
<option name='twicedaily'  /> twicedaily</option>
<option name='daily'  /> daily</option>
</select>

as to why that is, i couldn’t tell you.

3. i’m going to run with email notifications enabled for a while to see when the cron job fires.

4. please consider bumping the version number in the development version, so there’s a visual cue that this isn’t the current release.

best,

/mtr

it’s easy to select any of the options, the problem is that when the page comes back after clicking “Change Expiration” none of them are selected, leading to first one, ‘weekly’ being displayed.

it’s quite perplexing.

here is the problem – the <option/> should use value= rather than name=

--- _wp-cache.php	2012-02-23 00:05:32.000000000 -0600
+++ wp-cache.php	2012-03-01 00:55:30.000000000 -0600
@@ -1544,7 +1544,7 @@
 	$schedules = wp_get_schedules();
 	echo "<tr><td><br /></td><td><label for='cache_scheduled_select'>" . __( 'Interval:', 'wp-super-cache' ) . "</label></td><td><select id='cache_scheduled_select' name='cache_schedule_interval' size=1>";
 	foreach( $schedules as $desc => $details ) {
-		echo "<option name='$desc' " . selected( $desc, $cache_schedule_interval, false ) . " /> {$details[ 'display' ]}</option>";
+		echo "<option value='$desc' " . selected( $desc, $cache_schedule_interval, false ) . " /> {$details[ 'display' ]}</option>";
 	}
 	echo "</select></td></tr>";
 	echo '</table></td></tr>';

and it only took me 30m staring at the ‘inspect element’ screen to notice it. (-;