Merge pull request #118 from AlexeyDsov/primitiveHstoreExport

PrimitiveHstore::exportValue now return array instead of Hstore object

Author: AlexeyDsov

core/Form/Primitives/PrimitiveHstore.class.php

@@ -49,7 +49,10 @@ public function getInnerForm()
 
 		public function getValue()
 		{
-			return $this->exportValue();
+			if (!$this->value instanceof Form)
+				return null;
+			
+			return Hstore::make($this->value->export());
 		}
 
 		/**
@@ -103,7 +106,9 @@ public function exportValue()
 			if (!$this->value instanceof Form)
 				return null;
 
-			return Hstore::make($this->value->export());
+			return !$this->value->getErrors()
+				? $this->value->export()
+				: null;
 		}
 
 		/**

doc/ChangeLog

@@ -1,8 +1,12 @@
-2012-08-08	Artem A. Naumenko
+2012-08-10	Alexey S. Denisov
+
+	* core/Form/Primitives/PrimitiveHstore.class.php:
+		PrimitiveHstore exportValue now return array|null instead Hstore object
 
-	* core/DB/NoSQL/RedisNoSQL.class.php
+2012-08-08	Artem A. Naumenko
 
-	added campability with old version of redis servers
+	* core/DB/NoSQL/RedisNoSQL.class.php:
+		added campability with old version of redis servers
 
 2012-07-20	Alexey S. Denisov
 

test/core/PrimitiveHstoreTest.class.php

@@ -1,6 +1,4 @@
 <?php
-	/* $Id$ */
-	
 	final class PrimitiveHstoreTest extends TestCase
 	{
 		protected static $scope =
@@ -47,6 +45,10 @@ public function testImport()
 				$hstore->getList(),
 				self::$scope['properties']
 			);
+			$this->assertEquals(
+				$prm->exportValue(),
+				self::$scope['properties']
+			);
 
 			try {
 				$hstore->get('NotFound');
@@ -81,6 +83,8 @@ public function testInvalidImport()
 				)
 			);
 
+			$this->assertNull($prm->exportValue());
+			
 			$prm->clean();
 		}