feature #32845 [HttpKernel][FrameworkBundle] Add alternative convention for bundle directories (yceruto)

nicolas-grekas · nicolas-grekas · commit 9f08dcecab52 · 2019-08-13T15:29:11.000+02:00
This PR was squashed before being merged into the 4.4 branch (closes #32845). Discussion ---------- [HttpKernel][FrameworkBundle] Add alternative convention for bundle directories | Q | A | ------------- | --- | Branch? | 4.4 | Bug fix? | no | New feature? | yes | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | symfony/symfony#32453 | License | MIT | Doc PR | TODO We already know that bundles must be compatible with many Symfony's versions, so it is very likely that current bundles won't be able to use this feature soon, unless they create symbolic links to support both structures. The point is that this is already happening, so in the future when our bundles stop to support <=4.3 then you'll be sure to change the current directory structure. We have recently added the `getPublicDir()` method in symfony/symfony#31975, here I'm removing it in favor of hardcoding a new convention. I've added some functional tests in which I've changed everything to this structure: ``` -- ModernBundle |-- config/ |-- public/ |-- src/ |-- ModernBundle.php |-- templates/ |-- translations/ ``` WDYT? Commits ------- 6996e1c [HttpKernel][FrameworkBundle] Add alternative convention for bundle directories
diff --git a/Bundle/Bundle.php b/Bundle/Bundle.php
@@ -135,11 +135,6 @@ public function registerCommands(Application $application)
     {
     }
 
-    public function getPublicDir(): string
-    {
-        return 'Resources/public';
-    }
-
     /**
      * Returns the bundle's container extension class.
      *
diff --git a/Bundle/BundleInterface.php b/Bundle/BundleInterface.php
@@ -19,8 +19,6 @@
  * BundleInterface.
  *
  * @author Fabien Potencier <fabien@symfony.com>
- *
- * @method string getPublicDir() Returns relative path for the public assets directory
  */
 interface BundleInterface extends ContainerAwareInterface
 {
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -4,9 +4,8 @@ CHANGELOG
 4.4.0
 -----
 
- * Implementing the `BundleInterface` without implementing the `getPublicDir()` method is deprecated.
-   This method will be added to the interface in 5.0.
  * The `DebugHandlersListener` class has been marked as `final`
+ * Added new Bundle directory convention consistent with standard skeletons
 
 4.3.0
 -----

Original file line number	Diff line number	Diff line change
`@@ -135,11 +135,6 @@ public function registerCommands(Application $application)`
`135`	`135`	`{`
`136`	`136`	`}`
`137`	`137`
`138`		`- public function getPublicDir(): string`
`139`		`- {`
`140`		`- return 'Resources/public';`
`141`		`- }`
`142`		`-`
`143`	`138`	`/**`
`144`	`139`	`* Returns the bundle's container extension class.`
`145`	`140`	`*`
Original file line number	Diff line number	Diff line change
`@@ -19,8 +19,6 @@`
`19`	`19`	`* BundleInterface.`
`20`	`20`	`*`
`21`	`21`	`* @author Fabien Potencier <fabien@symfony.com>`
`22`		`- *`
`23`		`- * @method string getPublicDir() Returns relative path for the public assets directory`
`24`	`22`	`*/`
`25`	`23`	`interface BundleInterface extends ContainerAwareInterface`
`26`	`24`	`{`