Invalid implementation of the theming functions

The Drupal 7 version calls the theming function directly rather than using theme(). See http://drupal.org/node/224333#hook_theme_render_changes for the changes.

eg:

  $block_content .= theme_almanac_block($location, $location_time, $local_sunrise_time, $local_sunset_time,
    $moontimes['rise'], $moontimes['set']);

and

  $page_content .= theme_almanac_page($location, $location_time, $sunrise_times,
      $sunset_times, $moontimes['rise'], $moontimes['set']);

And as a feature request: The tables could also be run though the theme('table') call for doing things the Drupal way. And finally, template file based files would be much nicer than functions for both theming functions.

If I get time I may look at this in the next couple of days.