Document: Refactored __get code to use if conditions instead of a swi…

…tch.

QA #1: Removed an unused reference (Collection)
QA #2: Removed newline after php closing tag

data/entity/Document.php

@@ -8,7 +8,6 @@
 
 namespace lithium\data\entity;
 
-use lithium\util\Collection;
 use UnexpectedValueException;
 
 /**
@@ -133,18 +132,17 @@ public function &__get($name) {
 		$conn  = $model ? $model::connection() : null;
 
 		if ($model && $conn && $schema = $model::schema($name)) {
-			switch (true) {
-				case (isset($this->_updated[$name])) :
-					return $this->_updated[$name];
-				case (!isset($this->_data[$name])) :
-					$schema = array($name => $schema);
-					$pathKey = $this->_pathKey ? $this->_pathKey : null;
-					$options = compact('pathKey', 'schema') + array('first' => true);
-					if (($value = $conn->cast($this, array($name => null), $options)) !== null) {
-						$this->_data[$name] = $value;
-						return $this->_data[$name];
-					}
-					break;
+			if (isset($this->_updated[$name])) {
+				return $this->_updated[$name];
+			}
+			if (!isset($this->_data[$name])) {
+				$schema = array($name => $schema);
+				$pathKey = $this->_pathKey ? $this->_pathKey : null;
+				$options = compact('pathKey', 'schema') + array('first' => true);
+				if (($value = $conn->cast($this, array($name => null), $options)) !== null) {
+					$this->_data[$name] = $value;
+					return $this->_data[$name];
+				}
 			}
 		}
 		return parent::__get($name);
@@ -455,4 +453,4 @@ public function increment($field, $value = 1) {
 	}
 }
 
-?>
+?>