Title text rollovers

function THEMENAME_station_schedule_item($start, $finish, $program) {
  $class = 'station-sch-box station-sch-scheduled';
  $height = ($finish - $start);
  $link = url('node/'. $program->nid);

  $output = "<div class='{$class}'><a href='{$link}' style='height: {$height}em;' title='". theme('station_hour_range', $start, $finish) ."'>";
  $output .= '<span class="station-sch-time">'. theme('station_hour_range', $start, $finish) .'</span>';
  $output .= '<span class="station-sch-title">'. check_plain($program->title) .'</span>';
  if (!empty($program->field_station_program_dj)) {
    $djs = array();
    foreach ($program->field_station_program_dj as $entry) {
      $user = user_load($entry);
      $djs[] .= $user->name;
    }
    $output .= '<span class="station-sch-djs">'. check_plain(implode(', ', $djs)) .'</span>';
  }
  $output .= "</a></div>\n";
  return $output;
}

I put that in my template.php.

Thoughts, anyone?
Worth modifying the actual version? I can roll a patch later if need be.

bump. i'll roll a patch for the module tomorrow.

Instead of calling theme('station_hour_range', $start, $finish) like I suggested originally, I just used a variable.

Also, there are some weird things going on alternating between single and double quotes throughout the module. I just made it match the above line.

Committed to HEAD! Yes!