Changeset 232

Timestamp:

09/03/03 08:38:23 (5 years ago)

Author:

kake

Message:

Strip whitespace from OS co-ords before storing in database.

Location:

trunk

Files:

• Changes (modified) (1 diff)
• MANIFEST (modified) (1 diff)
• lib/OpenGuides/Template.pm (modified) (2 diffs)
• t/62_bug_os_coords.t (added)

trunk/Changes

@@ -1 +1 @@
 0.24    
         Fix to recent changes so minor changes don't mask major ones.
-        Fix supersearch.cgi to use a template instead of CGI.pm to avoid
+        Fixed supersearch.cgi to use a template instead of CGI.pm to avoid
           weird errors, also turned it into a module and added tests.
+        Strip whitespace from OS co-ords before storing in database.
 
 0.23    4 August 2003

trunk/MANIFEST

@@ -59 +59 @@
 t/51_diff.t
 t/61_bug_textareas.t
+t/62_bug_os_coords.t
 t/templates/15_test.tt
 wiki.cgi

trunk/lib/OpenGuides/Template.pm

@@ -3 +3 @@
 use strict;
 use vars qw( $VERSION );
-$VERSION = '0.11';
+$VERSION = '0.12';
 
 use Carp qw( croak );
@@ -259 +259 @@
         my $os_x = $q->param("os_x");
         my $os_y = $q->param("os_y");
+        # Trim whitespace - trailing whitespace buggers up the integerification
+        # by postgres and it's an easy mistake to make when typing into a form.
+        $os_x =~ s/\s+//;
+        $os_y =~ s/\s+//;
         # Work out latitude and longitude for the preview display.
         if ($os_x and $os_y) {