Merge pull request #94 from connorhu/fix-88

add 'double' check to internal isValueModified check.
Fixes  #88

Author: thePanz

lib/Doctrine/Record.php

@@ -1558,7 +1558,7 @@ protected function _isValueModified($type, $old, $new)
 
         if ($type == 'boolean' && (is_bool($old) || is_numeric($old)) && (is_bool($new) || is_numeric($new)) && $old == $new) {
             return false;
-        } else if (in_array($type, array('decimal', 'float')) && is_numeric($old) && is_numeric($new)) {
+        } else if (in_array($type, array('decimal', 'float', 'double')) && is_numeric($old) && is_numeric($new)) {
             return $old * 100 != $new * 100;
         } else if (in_array($type, array('integer', 'int')) && is_numeric($old) && is_numeric($new)) {
             return $old != $new;

tests/RecordTestCase.php

@@ -41,6 +41,7 @@ public function prepareTables()
         $this->tables[] = 'Book';
         $this->tables[] = 'EntityAddress';
         $this->tables[] = 'UnderscoreColumn';
+        $this->tables[] = 'Location2';
         parent::prepareTables();
     }
 
@@ -1022,4 +1023,15 @@ public function testDeleteReturnBooleanAndThrowsException()
           $this->fail();
         }
     }
+    
+    public function testDoubleIsModified()
+    {
+        $location = new Location2();
+        $location->lat = '12.345';
+        
+        $location->save();
+        $location->lat = 12.345;
+        
+        $this->assertFalse($location->isModified());
+    }
 }

tests/models/Location2.php

@@ -0,0 +1,10 @@
+<?php
+class Location2 extends Doctrine_Record
+{
+    public function setTableDefinition()
+    {
+        $this->hasColumn('id', 'integer', 10, array('primary' => true));
+        $this->hasColumn('lat', 'double', 10);
+        $this->hasColumn('lon', 'double', 10);
+    }
+}