I was trying out the Geonames "Simple City Search" example and I could not get it to work.

Looked over the code and I discovered that there was no implementation of theme_hook() to register the theme_geonames_table() function for Drupal 6.

The theme hook is also missing from 6.x-2.x-dev

The patch below inserts the implementation of hook_theme() right before the theme_geonames_table() function. Not sure if that's the best place for it, but I will leave it to the maintainers to move it to where they like it.

CommentFileSizeAuthor
#1 geonames.module-370976-2.patch446 bytesjfxberns

Comments

jfxberns’s picture

jfxberns commented
StatusFileSize
new geonames.module-370976-2.patch446 bytes

Here's the patch.

Anonymous’s picture

Anonymous (not verified) commented
Title: No Implementation of hook_theme to support theme_geonames _table() » Implement hook_theme to support theme_geonames_table()
Status: Needs review » Reviewed & tested by the community

Works for me!

lyricnz’s picture

lyricnz commented
Status: Reviewed & tested by the community » Fixed

Committed to 6.x-1.x-dev. Thanks.

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.