Make base path in View and WebView optional (#251)

CHANGELOG.md

@@ -12,6 +12,7 @@
 - Bug #232: Fix render templates that contain dots in their name (@rustamwin)
 - New #242: Add `View::getLocale()` and `WebView::getLocale()` methods (@Tigrov)
 - Enh #250: Make event dispatcher in `View` and `WebView` optional (@vjik)
+- Enh #251: Make base path in `View` and `WebView` optional (@vjik)
 
 ## 8.0.0 February 16, 2023
 

src/View.php

@@ -33,10 +33,10 @@ final class View implements ViewInterface
     private ThemeState $themeState;
 
     /**
-     * @param string $basePath The full path to the base directory of views.
+     * @param string|null $basePath The full path to the base directory of views.
      * @param EventDispatcherInterface|null $eventDispatcher The event dispatcher instance.
      */
-    public function __construct(string $basePath, ?EventDispatcherInterface $eventDispatcher = null)
+    public function __construct(?string $basePath = null, ?EventDispatcherInterface $eventDispatcher = null)
     {
         $this->basePath = $basePath;
         $this->state = new ViewState();

src/ViewTrait.php

@@ -5,6 +5,7 @@
 namespace Yiisoft\View;
 
 use InvalidArgumentException;
+use LogicException;
 use Psr\EventDispatcher\EventDispatcherInterface;
 use Psr\EventDispatcher\StoppableEventInterface;
 use RuntimeException;
@@ -36,7 +37,7 @@ trait ViewTrait
 {
     private ?EventDispatcherInterface $eventDispatcher;
 
-    private string $basePath;
+    private ?string $basePath = null;
     private ?ViewContextInterface $context = null;
     private string $placeholderSignature;
     private string $sourceLocale = 'en';
@@ -207,6 +208,10 @@ public function getLocale(): string
      */
     public function getBasePath(): string
     {
+        if ($this->basePath === null) {
+            throw new LogicException('The base path is not set.');
+        }
+
         return $this->basePath;
     }
 
@@ -643,7 +648,7 @@ private function findTemplateFile(string $view): string
     {
         if ($view !== '' && $view[0] === '/') {
             // path relative to basePath e.g. "/layouts/main"
-            $file = $this->basePath . '/' . ltrim($view, '/');
+            $file = $this->getBasePath() . '/' . ltrim($view, '/');
         } elseif (($currentViewFile = $this->getRequestedViewFile()) !== null) {
             // path relative to currently rendered view
             $file = dirname($currentViewFile) . '/' . $view;

src/WebView.php

@@ -87,10 +87,10 @@ final class WebView implements ViewInterface
     private const PLACEHOLDER_BODY_END = '<![CDATA[YII-BLOCK-BODY-END-%s]]>';
 
     /**
-     * @param string $basePath The full path to the base directory of views.
+     * @param string|null $basePath The full path to the base directory of views.
      * @param EventDispatcherInterface|null $eventDispatcher The event dispatcher instance.
      */
-    public function __construct(string $basePath, ?EventDispatcherInterface $eventDispatcher = null)
+    public function __construct(?string $basePath = null, ?EventDispatcherInterface $eventDispatcher = null)
     {
         $this->basePath = $basePath;
         $this->state = new WebViewState();

tests/ViewTest.php

@@ -6,6 +6,7 @@
 
 use Exception;
 use InvalidArgumentException;
+use LogicException;
 use PHPUnit\Framework\TestCase;
 use RuntimeException;
 use Yiisoft\Files\FileHelper;
@@ -609,6 +610,15 @@ public function testGetLocale()
         $this->assertSame('en-US', $view->getLocale());
     }
 
+    public function testWithoutBasePath(): void
+    {
+        $view = new View();
+
+        $this->expectException(LogicException::class);
+        $this->expectExceptionMessage('The base path is not set.');
+        $view->getBasePath();
+    }
+
     private function createViewWithBasePath(string $basePath): View
     {
         return new View($basePath, new SimpleEventDispatcher());