Removing the deprecated properties.

Removing property copying to helpers.
Making the view test cases pass again.

cake/libs/controller/controller.php

@@ -341,7 +341,7 @@ public function __get($name) {
 			case 'data':
 				return $this->request->{$name};
 			case 'action':
-				return $this->request->params['action'];
+				return isset($this->request->params['action']) ? $this->request->params['action'] : '';
 			case 'params':
 				return $this->request;
 		}

cake/libs/view/helper_collection.php

@@ -68,7 +68,7 @@ public function load($helper, $settings = array(), $enable = true) {
 		}
 		$this->_loaded[$name] = new $helperClass($this->_View, $settings);
 
-		$vars = array('request', 'base', 'webroot', 'here', 'params', 'action', 'data', 'theme', 'plugin');
+		$vars = array('request', 'theme', 'plugin');
 		foreach ($vars as $var) {
 			$this->_loaded[$name]->{$var} = $this->_View->{$var};
 		}

cake/libs/view/view.php

@@ -42,21 +42,6 @@ class View extends Object {
  */
 	public $Helpers;
 
-/**
- * Path parts for creating links in views.
- *
- * @var string Base URL
- * @access public
- */
-	public $base = null;
-
-/**
- * Stores the current URL (for links etc.)
- *
- * @var string Current URL
- */
-	public $here = null;
-
 /**
  * Name of the plugin.
  *
@@ -73,35 +58,13 @@ class View extends Object {
  */
 	public $name = null;
 
-/**
- * Action to be performed.
- *
- * @var string Name of action
- * @access public
- */
-	public $action = null;
-
-/**
- * Array of parameter data
- *
- * @var array Parameter data
- */
-	public $params = array();
-
 /**
  * Current passed params
  *
  * @var mixed
  */
 	public $passedArgs = array();
 
-/**
- * Array of data
- *
- * @var array Parameter data
- */
-	public $data = array();
-
 /**
  * An array of names of built-in helpers to include.
  *
@@ -269,16 +232,24 @@ class View extends Object {
  */
 	public $output = false;
 
+/**
+ * An instance of a CakeRequest object that contains information about the current request.
+ * This object contains all the information about a request and several methods for reading
+ * additional information about the request. 
+ *
+ * @var CakeRequest
+ */
+	public $request;
+
 /**
  * List of variables to collect from the associated controller
  *
  * @var array
  * @access protected
  */
 	private $__passedVars = array(
-		'viewVars', 'autoLayout', 'autoRender', 'ext', 'base', 'webroot',
-		'helpers', 'here', 'layout', 'name', 'layoutPath', 'viewPath',
-		'params', 'request', 'data', 'plugin', 'passedArgs', 'cacheAction'
+		'viewVars', 'autoLayout', 'autoRender', 'ext', 'helpers', 'layout', 'name',
+		'layoutPath', 'viewPath', 'request', 'plugin', 'passedArgs', 'cacheAction'
 	);
 
 /**
@@ -645,17 +616,25 @@ public function error($code, $name, $message) {
 	}
 
 /**
- * Magic accessor for helpers.
+ * Magic accessor for helpers. Provides access to attributes that were deprecated.
  *
- * @return void
+ * @param string $name Name of the attribute to get.
+ * @return mixed
  */
 	public function __get($name) {
 		if (isset($this->Helpers->{$name})) {
 			return $this->Helpers->{$name};
 		}
 		switch ($name) {
+			case 'base':
+			case 'here':
+			case 'webroot':
+			case 'data':
+				return $this->request->{$name};
 			case 'action':
-				return $this->request->params['action'];
+				return isset($this->request->params['action']) ? $this->request->params['action'] : '';
+			case 'params':
+				return $this->request;
 		}
 		return null;
 	}
@@ -712,10 +691,10 @@ protected function _render($___viewFn, $___dataForView, $loadHelpers = true, $ca
 		if ($caching) {
 			if (isset($this->Helpers->Cache)) {
 				$cache =& $this->Helpers->Cache;
-				$cache->base = $this->base;
-				$cache->here = $this->here;
+				$cache->base = $this->request->base;
+				$cache->here = $this->request->here;
 				$cache->helpers = $this->helpers;
-				$cache->action = $this->action;
+				$cache->action = $this->request->action;
 				$cache->controllerName = $this->name;
 				$cache->layout = $this->layout;
 				$cache->cacheAction = $this->cacheAction;

cake/tests/cases/libs/view/view.test.php

@@ -19,10 +19,8 @@
  */
 App::import('Core', array('View', 'Controller'));
 App::import('Helper', 'Cache');
+App::import('Core', array('ErrorHandler'));
 
-if (!class_exists('ErrorHandler')) {
-	App::import('Core', array('Error'));
-}
 
 /**
  * ViewPostsController class
@@ -227,8 +225,9 @@ class ViewTest extends CakeTestCase {
  */
 	function setUp() {
 		Router::reload();
-		$this->Controller = new Controller();
+
 		$request = $this->getMock('CakeRequest');
+		$this->Controller = new Controller($request);
 		$this->PostsController = new ViewPostsController($request);
 		$this->PostsController->viewPath = 'posts';
 		$this->PostsController->index();