[development] Geo-IP Modifications to 'watchdog' and 'statistics'
dries.buytaert at gmail.com
Tue Apr 18 10:53:40 UTC 2006
On 4/18/06, Gary (Lists) <listout at accidentaltechie.org> wrote:
> Last night I wrapped up a few changes to my copy of 'watchdog.module' and
> 'statistics.module' to add a graphic country flag and country, city,
> latitude and longitude.
> In case anyone would like to spruce up their 'details' pages, as you see
> from either the watchdog entries 'details' link or the statistics module's
> 'details' links, then you might find this useful.
The screenshots look nice but the code looks a little dangerous as
you're not validating the input. Hackers can inject XSS attacks.
Apart from that, you may want to make the interface changes
translatable. For the former, read up Drupal's input validation
functions. For the latter, read up on the t()-function.
Hope that helps,
Dries Buytaert :: http://buytaert.net/
More information about the development