Fix undefined key error

[packbundle] / Package / PathPackage.php

diff --git a/Package/PathPackage.php b/Package/PathPackage.php
index 50c6172b16d2668439428ed805ab9c0bb0704cdd..c53b10fc268c74ef7fc179592db25c2ca2b47709 100644 (file)
--- a/Package/PathPackage.php
+++ b/Package/PathPackage.php
@@ -1,41 +1,74 @@
-<?php
+<?php declare(strict_types=1);

 
 

-namespace Rapsys\PackBundle\Asset;
+/*
+ * This file is part of the Rapsys PackBundle package.
+ *
+ * (c) Raphaël Gertz <symfony@rapsys.eu>
+ *
+ * For the full copyright and license information, please view the LICENSE
+ * file that was distributed with this source code.
+ */
+
+namespace Rapsys\PackBundle\Package;
+
+use Rapsys\PackBundle\Context\NullContext;

 
 use Symfony\Component\Asset\Context\ContextInterface;
 use Symfony\Component\Asset\Package;
 use Symfony\Component\Asset\VersionStrategy\VersionStrategyInterface;
 
 /**
 
 use Symfony\Component\Asset\Context\ContextInterface;
 use Symfony\Component\Asset\Package;
 use Symfony\Component\Asset\VersionStrategy\VersionStrategyInterface;
 
 /**

- * (@inheritdoc)
+ * {@inheritdoc}

  */
 class PathPackage extends Package {
  */
 class PathPackage extends Package {

-       //The base path
-       protected $basePath;
+       /**
+        * The base url
+        */
+       protected string $baseUrl;

 
        /**
         * {@inheritdoc}
         */
 
        /**
         * {@inheritdoc}
         */

-       public function __construct(string $basePath, VersionStrategyInterface $versionStrategy, ContextInterface $context = null) {
-               parent::__construct($versionStrategy, $context);
+       public function __construct(protected string $basePath, protected VersionStrategyInterface $versionStrategy, protected ?ContextInterface $context = null) {
+               //Without context use a null context
+               $this->context = $this->context ?? new NullContext();
+
+               //Call parent constructor
+               parent::__construct($this->versionStrategy, $this->context);

 
 

-               if (!$basePath) {
+               //Without base path
+               if (empty($basePath)) {
+                       //Set base path

                        $this->basePath = '/';
                        $this->basePath = '/';

+               //With base path

                } else {
                } else {

+                       //With relative base path

                        if ('/' != $basePath[0]) {
                        if ('/' != $basePath[0]) {

+                               //Set base path as absolute

                                $basePath = '/'.$basePath;
                        }
 
                                $basePath = '/'.$basePath;
                        }
 

+                       //Set base path

                        $this->basePath = rtrim($basePath, '/').'/';
                }
                        $this->basePath = rtrim($basePath, '/').'/';
                }

+
+               //Set base url
+               $this->baseUrl = $this->context->getBaseUrl();

        }
 
        /**
        }
 
        /**

-        * @todo Try retrive public dir from the member function BundleNameBundle::getPublicDir() return value ?
-        * @xxx see https://symfony.com/doc/current/bundles.html#overridding-the-bundle-directory-structure

         * {@inheritdoc}
         * {@inheritdoc}

+        *
+        * Returns an absolute or root-relative public path
+        *
+        * Transform @BundleBundle to bundle and remove /Resources/public fragment from path
+        * This bundle name conversion and bundle prefix are the same as in asset:install command
+        *
+        * @link https://symfony.com/doc/current/bundles.html#overridding-the-bundle-directory-structure
+        * @see vendor/symfony/framework-bundle/Command/AssetsInstallCommand.php +113
+        * @see vendor/symfony/framework-bundle/Command/AssetsInstallCommand.php +141

         */
         */

-       public function getUrl($path) {
+       public function getUrl(string $path): string {

                //Match url starting with a bundle name
                if (preg_match('%^@([A-Z][a-zA-Z]*?)(?:Bundle/Resources/public)?/(.*)$%', $path, $matches)) {
                        //Handle empty or without replacement pattern basePath
                //Match url starting with a bundle name
                if (preg_match('%^@([A-Z][a-zA-Z]*?)(?:Bundle/Resources/public)?/(.*)$%', $path, $matches)) {
                        //Handle empty or without replacement pattern basePath


@@ -53,4 +86,15 @@ class PathPackage extends Package {
                //Return parent getUrl result
                return parent::getUrl($path);
        }
                //Return parent getUrl result
                return parent::getUrl($path);
        }

+
+       /**
+        * Returns an absolute public path.
+        *
+        * @param string $path A path
+        * @return string The absolute public path
+        */
+       public function getAbsoluteUrl(string $path): string {
+               //Return concated base url and url from path
+               return $this->baseUrl.self::getUrl($path);
+       }

 }
 }