From dba0eef8ab3986cdc84d6b9fec38cb9017a989c2 Mon Sep 17 00:00:00 2001
From: xue <>
Date: Fri, 4 Jan 2008 22:20:08 +0000
Subject: fixed #707.

---
 HISTORY                                              | 1 +
 framework/Data/SqlMap/DataMapper/TPropertyAccess.php | 5 +----
 2 files changed, 2 insertions(+), 4 deletions(-)

diff --git a/HISTORY b/HISTORY
index b7166184..0dda22b4 100644
--- a/HISTORY
+++ b/HISTORY
@@ -1,5 +1,6 @@
 Version 3.1.2 To be released
 ============================
+BUG: Ticket#707 - TPropertyAccess sets property twice on object when using setters (Qiang)
 BUG: Ticket#719 - TAutoCompleter should not trigger Validation if CausesValidation=False (Christophe)
 BUG: Ticket#744 - Callback error handling is improved (Qiang)
 BUG: Ticket#750 - The "expire" parameter is used inconsistently in cache modules (Qiang)
diff --git a/framework/Data/SqlMap/DataMapper/TPropertyAccess.php b/framework/Data/SqlMap/DataMapper/TPropertyAccess.php
index c89bcb75..b1444163 100644
--- a/framework/Data/SqlMap/DataMapper/TPropertyAccess.php
+++ b/framework/Data/SqlMap/DataMapper/TPropertyAccess.php
@@ -138,10 +138,7 @@ class TPropertyAccess
 		{
 			$setter = 'set'.$prop;
 			if(is_callable(array($object, $setter)))
-			{
-				if($object->{$setter}($value) === null)
-					$object->{$prop} = $value;
-			}
+				$object->{$setter}($value);
 			else
 				$object->{$prop} = $value;
 		}
-- 
cgit v1.2.3