Merge pull request #52 from dovg/TextMessageFix

a bit optimize: don't needed to make Timestamp twice

Author: dovg

doc/ChangeLog

@@ -1,3 +1,8 @@
+2011-11-18	Evgeny V. Kokovikhin
+	
+	* main/Messages/TextMessage.class.php, main/Messages/TextFileReceiver.class.php :
+	a bit optimize: don't needed to make Timestamp twice
+
 2011-11-14      Dmitry V. Snezhinskiy
 
 	* main/Utils/AMQP/Pecl/AMQPPeclChannel.class.php :

main/Messages/TextFileReceiver.class.php

@@ -69,8 +69,7 @@ public function receive($uTimeout = null)
 
 			Assert::isNotNull($time);
 
-			$result = TextMessage::create()->
-				setTimestamp(Timestamp::create($time))->
+			$result = TextMessage::create(Timestamp::create($time))->
 				setText($text);
 
 			return $result;

main/Messages/TextMessage.class.php

@@ -15,17 +15,17 @@ final class TextMessage implements Message
 		private $timestamp	= null;
 		private $text		= null;
 
-		public static function create()
+		public static function create(Timestamp $timestamp = null)
 		{
-			return new self;
+			return new self($timestamp);
 		}
 
-		public function __construct()
+		public function __construct(Timestamp $timestamp = null)
 		{
-			$this->timestamp = Timestamp::makeNow();
+			$this->timestamp = $timestamp ?: Timestamp::makeNow();
 		}
 
-		public function setTimestamp($timestamp)
+		public function setTimestamp(Timestamp $timestamp)
 		{
 			$this->timestamp = $timestamp;