Implement hook_theme to support theme_geonames_table()

jfxberns - February 8, 2009 - 09:31
Project:GeoNames
Version:6.x-1.x-dev
Component:Code
Category:bug report
Priority:normal
Assigned:Unassigned
Status:closed
Description

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.

#1

jfxberns - February 8, 2009 - 10:10

Here's the patch.

AttachmentSize
geonames.module-370976-2.patch 446 bytes

#2

BWPanda - September 10, 2009 - 00:48
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!

#3

lyricnz - September 10, 2009 - 04:10
Status:reviewed & tested by the community» fixed

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

#4

System Message - September 24, 2009 - 04:20
Status:fixed» closed

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

 
 

Drupal is a registered trademark of Dries Buytaert.