Skip to content

Commit 4cc579f

Browse files
committed
minor #32452 [Bundles] Rename getPublicPath() as getPublicDir() (javiereguiluz)
This PR was squashed before being merged into the 4.4 branch (closes #32452). Discussion ---------- [Bundles] Rename getPublicPath() as getPublicDir() | Q | A | ------------- | --- | Branch? | 4.4 | Bug fix? | no | New feature? | no <!-- please update src/**/CHANGELOG.md files --> | BC breaks? | no <!-- see https://symfony.com/bc --> | Deprecations? | no <!-- please update UPGRADE-*.md and src/**/CHANGELOG.md files --> | Tests pass? | yes <!-- please add some, will be required by reviewers --> | Fixed tickets | - | License | MIT | Doc PR | I'll add this if approved While documenting #31975 (see symfony/symfony-docs#11930) I realized that the `getPublicPath()` method name is not consistent with the rest of Symfony. In Symfony, "path" is usually associated to routes and we use "dir" for things similar to this: * `getCacheDir()` and `getLogdir()` to override Symfony structure (https://symfony.com/doc/current/configuration/override_dir_structure.html) * `binDir`, `configDir`, `srcDir`, `varDir`, `publicDir` in Symfony Flex recipes (https://github.com/symfony/recipes) to override the dir structure So, this PR proposes to rename `getPublicPath()` as `getPublicDir()` Commits ------- 4ab2f99 [Bundles] Rename getPublicPath() as getPublicDir()
2 parents e851aa5 + b110db4 commit 4cc579f

File tree

3 files changed

+4
-4
lines changed

3 files changed

+4
-4
lines changed

Bundle/Bundle.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -135,7 +135,7 @@ public function registerCommands(Application $application)
135135
{
136136
}
137137

138-
public function getPublicPath(): string
138+
public function getPublicDir(): string
139139
{
140140
return 'Resources/public';
141141
}

Bundle/BundleInterface.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@
2020
*
2121
* @author Fabien Potencier <[email protected]>
2222
*
23-
* @method string getPublicPath() Returns relative path for public assets
23+
* @method string getPublicDir() Returns relative path for the public assets directory
2424
*/
2525
interface BundleInterface extends ContainerAwareInterface
2626
{

Tests/KernelTest.php

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -627,7 +627,7 @@ protected function getBundle($dir = null, $parent = null, $className = null, $bu
627627
{
628628
$bundle = $this
629629
->getMockBuilder('Symfony\Component\HttpKernel\Bundle\BundleInterface')
630-
->setMethods(['getPath', 'getPublicPath', 'getParent', 'getName'])
630+
->setMethods(['getPath', 'getPublicDir', 'getParent', 'getName'])
631631
->disableOriginalConstructor()
632632
;
633633

@@ -651,7 +651,7 @@ protected function getBundle($dir = null, $parent = null, $className = null, $bu
651651

652652
$bundle
653653
->expects($this->any())
654-
->method('getPublicPath')
654+
->method('getPublicDir')
655655
->willReturn('Resources/public')
656656
;
657657

0 commit comments

Comments
 (0)