supress iCal

I wanted to know if you could add a toggle to suppress the iCal option.
I played with the concept and I think it would be as easy as:

In event.theme file:

function theme_event_ical_link($path) {
   if (variable_get('event_show_ical', 'no') != 'no') {
  return '<div class="ical-link">'. l('<img src="'. base_path() . drupal_get_path('module', 'event') .'/images/ical16x16.gif" alt="'.t('Add to iCalendar').'" />', $path, array('title' => t('Add this calendar to your iCalendar')), NULL, NULL, TRUE, TRUE) .'</div>';
  }
}

and in event.module:

function event_admin_overview_settings() {

  $form['event_show_ical'] = array(
    '#type' => 'radios',
    '#title' => t('Show iCal'),
    '#default_value' => variable_get('event_show_ical', 'yes'),
    '#options' => array('yes' => t('Yes'), 'no' => t('No')),
    '#description' => t('Show iCal option in the calendar and block sections'));

I have not created a patch for it, but would love it if you could look at this hack and tell me if I am doing it right.

David
neofactor