fix. saveData not work

Author: nyufeng

system/Common.php

@@ -38,6 +38,7 @@
  */
 
 use Config\App;
+use Config\View;
 use Config\Logger;
 use Config\Database;
 use Config\Services;
@@ -1069,8 +1070,9 @@ function view(string $name, array $data = [], array $options = []): string
 		 */
 		$renderer = Services::renderer();
 
-		$saveData = true;
-		if (array_key_exists('saveData', $options) && $options['saveData'] === true)
+		$saveData = config(View::class)->saveData;
+
+		if (array_key_exists('saveData', $options))
 		{
 			$saveData = (bool) $options['saveData'];
 			unset($options['saveData']);

system/View/View.php

@@ -58,6 +58,11 @@ class View implements RendererInterface
 	 */
 	protected $data = [];
 
+	/**
+	 * Merge savedData and userData
+	 */
+	protected $tempData = null;
+
 	/**
 	 * The base directory to look in for our Views.
 	 *
@@ -189,11 +194,10 @@ public function render(string $view, array $options = null, bool $saveData = nul
 		// Store the results here so even if
 		// multiple views are called in a view, it won't
 		// clean it unless we mean it to.
-		if ($saveData !== null)
+		if ($saveData === null)
 		{
-			$this->saveData = $saveData;
+			$saveData = $this->saveData;
 		}
-
 		$fileExt                     = pathinfo($view, PATHINFO_EXTENSION);
 		$realPath                    = empty($fileExt) ? $view . '.php' : $view; // allow Views as .html, .tpl, etc (from CI3)
 		$this->renderVars['view']    = $realPath;
@@ -225,11 +229,17 @@ public function render(string $view, array $options = null, bool $saveData = nul
 		}
 
 		// Make our view data available to the view.
-		extract($this->data);
 
-		if (! $this->saveData)
+		if(is_null($this->tempData))
+		{
+			$this->tempData = $this->data;
+		}
+
+		extract($this->tempData);
+
+		if ($saveData)
 		{
-			$this->data = [];
+			$this->data = $this->tempData;
 		}
 
 		ob_start();
@@ -276,7 +286,7 @@ public function render(string $view, array $options = null, bool $saveData = nul
 		{
 			cache()->save($this->renderVars['cacheName'], $output, (int) $this->renderVars['options']['cache']);
 		}
-
+		$this->tempData = null;
 		return $output;
 	}
 
@@ -300,16 +310,22 @@ public function render(string $view, array $options = null, bool $saveData = nul
 	public function renderString(string $view, array $options = null, bool $saveData = null): string
 	{
 		$start = microtime(true);
+
 		if (is_null($saveData))
 		{
-			$saveData = $this->config->saveData;
+			$saveData = $this->saveData;
 		}
 
-		extract($this->data);
+		if(is_null($this->tempData))
+		{
+			$this->tempData = $this->data;
+		}
 
-		if (! $saveData)
+		extract($this->tempData);
+
+		if ($saveData)
 		{
-			$this->data = [];
+			$this->data = $this->tempData;
 		}
 
 		ob_start();
@@ -319,7 +335,7 @@ public function renderString(string $view, array $options = null, bool $saveData
 		@ob_end_clean();
 
 		$this->logPerformance($start, microtime(true), $this->excerpt($view));
-
+		$this->tempData = null;
 		return $output;
 	}
 
@@ -354,8 +370,8 @@ public function setData(array $data = [], string $context = null): RendererInter
 		{
 			$data = \esc($data, $context);
 		}
-
-		$this->data = array_merge($this->data, $data);
+		$this->tempData = $this->tempData ?? $this->data;
+		$this->tempData = array_merge($this->tempData, $data);
 
 		return $this;
 	}
@@ -379,7 +395,8 @@ public function setVar(string $name, $value = null, string $context = null): Ren
 			$value = \esc($value, $context);
 		}
 
-		$this->data[$name] = $value;
+		$this->tempData = $this->tempData ?? $this->data;
+		$this->tempData[$name] = $value;
 
 		return $this;
 	}
@@ -407,7 +424,7 @@ public function resetData(): RendererInterface
 	 */
 	public function getData(): array
 	{
-		return $this->data;
+		return $this->tempData ?? $this->data;
 	}
 
 	//--------------------------------------------------------------------

tests/system/CommonFunctionsTest.php

@@ -25,7 +25,8 @@ class CommonFunctionsTest extends \CodeIgniter\Test\CIUnitTestCase
 	protected function setUp(): void
 	{
 		parent::setUp();
-
+		$renderer = Services::renderer();
+		$renderer->resetData();
 		unset($_ENV['foo'], $_SERVER['foo']);
 	}
 
@@ -429,4 +430,18 @@ public function testTrace()
 		$this->assertStringContainsString('Debug Backtrace', $content);
 	}
 
+	public function testViewNotSaveData()
+	{
+
+		$data     = [
+			'testString' => 'bar',
+			'bar'        => 'baz',
+		];
+		view('\Tests\Support\View\Views\simple', $data, ['saveData' => false]);
+		ob_get_clean();
+		$this->expectException('Exception');
+		view('\Tests\Support\View\Views\simple');
+		ob_get_clean();
+	}
+
 }

tests/system/View/ViewTest.php

@@ -231,13 +231,16 @@ public function testCachedRender()
 	public function testRenderStringSavingData()
 	{
 		$view = new View($this->config, $this->viewsDir, $this->loader);
+		$expected = '<h1>Hello World</h1>';
 
+		//I think saveData is sava current data, is not clean already set data.
 		$view->setVar('testString', 'Hello World');
-		$expected = '<h1>Hello World</h1>';
-		$this->assertEquals($expected, $view->renderString('<h1><?= $testString ?></h1>', [], true));
-		$this->assertArrayHasKey('testString', $view->getData());
 		$this->assertEquals($expected, $view->renderString('<h1><?= $testString ?></h1>', [], false));
 		$this->assertArrayNotHasKey('testString', $view->getData());
+
+		$view->setVar('testString', 'Hello World');
+		$this->assertEquals($expected, $view->renderString('<h1><?= $testString ?></h1>', [], true));
+		$this->assertArrayHasKey('testString', $view->getData());
 	}
 
 	//--------------------------------------------------------------------
@@ -343,4 +346,22 @@ public function testRenderLayoutNoContentSection()
 		$this->assertStringContainsString($expected, $view->render('apples'));
 	}
 
+	public function testRenderSaveDataCover()
+	{
+		$view = new View($this->config, $this->viewsDir, $this->loader);
+		$this->setPrivateProperty($view, 'saveData', true);
+		$view->setVar('testString', 'test');
+		$view->render('simple', null, false);
+		$this->assertEquals(true, $this->getPrivateProperty($view, 'saveData'));
+	}
+
+	public function testRenderSaveDataUseAflterSaveDataFalse()
+	{
+		$view = new View($this->config, $this->viewsDir, $this->loader);
+		$view->setVar('testString', 'test');
+		$view->render('simple', null, true);
+		$view->render('simple', null, false);
+		$this->assertEquals('test', $view->render('simple', null, false));
+	}
+
 }