Ecosyste.ms: Awesome
An open API service indexing awesome lists of open source software.
https://github.com/yceruto/acme-bundle
Upgrade your Symfony Bundle directories to standard skeleton
https://github.com/yceruto/acme-bundle
symfony symfony-bundle symfony-bundle-skeleton
Last synced: 18 days ago
JSON representation
Upgrade your Symfony Bundle directories to standard skeleton
- Host: GitHub
- URL: https://github.com/yceruto/acme-bundle
- Owner: yceruto
- License: mit
- Created: 2019-08-10T16:16:10.000Z (over 5 years ago)
- Default Branch: master
- Last Pushed: 2020-09-02T21:34:37.000Z (about 4 years ago)
- Last Synced: 2024-10-11T07:12:37.862Z (about 1 month ago)
- Topics: symfony, symfony-bundle, symfony-bundle-skeleton
- Language: PHP
- Homepage:
- Size: 7.81 KB
- Stars: 11
- Watchers: 3
- Forks: 0
- Open Issues: 0
-
Metadata Files:
- Readme: README.md
- License: LICENSE.md
Awesome Lists containing this project
README
Upgrading directory structure of Symfony bundles
================================================Short guide explaining how to upgrade your current directory structure
to be consistent with standard skeletons:└── AcmeBundle
├── assets/
├── bin/
├── config/
├── docs/
├── public/
├── src/
│ ├── Model/
│ ├── Service/
│ └── AcmeBundle.php
├── templates/
├── tests/
└── translations/AcmeBundle
----------This is the current structure (see [`master`](https://github.com/yceruto/acme-bundle/tree/master) branch):
└── AcmeBundle
├── DependencyInjection/
├── Model/
├── Resources/
│ ├── assets/
│ ├── bin/
│ ├── config/
│ ├── docs/
│ ├── public/
│ ├── translations/
│ └── views/
├── Service/
├── Tests/
├── AcmeBundle.php
└── composer.phpEven though it has worked since the beginning, this structure is mixing the source code with resource files, configuration, documentation, etc.
which is not good enough as it is easy to get lost in large bundles with tons of directories and files at the repository root.**composer.json file:**
"autoload": {
"psr-4": {
"Acme\\AcmeBundle\\": ""
}
}Regarding autoloading, it has a minor impact when building optimized autoloaders with composer as the test classes will be in the classmap,
that's not a huge deal if it's a few classes, but it's not zero-impact.In the next sections we will change the structure to solve these issues without breaking the bundle functionality.
#### Revamped Version
> The following steps relate to this repository sample, adjust them to suit your case.
Before start, let's install the dependencies and run all tests to make sure everything were well.
$ cd AcmeBundle
$ composer install
$ ./vendor/bin/simple-phpunitLet's start by creating a directory `src/` and moving our source code there:
$ mkdir src/
$ mv AcmeBundle.php DependencyInjection/ Model/ Resources/ Service/ src/We will then rename the current `Tests/` directory to `tests/` (lowercase):
$ mv Tests/ tests/
Update your the `composer.json` to reflect the new PSR-4 autoload paths:
"autoload": {
"psr-4": {
"Acme\\AcmeBundle\\": "src/"
}
},
"autoload-dev": {
"psr-4": {
"Acme\\AcmeBundle\\Tests\\": "tests/"
}
}> run `composer dump-autoload` to update the autoload files with the new psr-4 map.
and update the `phpunit.xml` and `phpunit.xml.dist` files as well:
tests
src
Move `assets/`, `bin/` and `docs/` directories at the root-level since they aren't part of any Symfony directory convention:
$ mv src/Resources/assets/ src/Resources/bin/ src/Resources/docs/ ./
Finally, check if the tests have passed `./vendor/bin/simple-phpunit` if it's green, it means you're almost done!
This is how it should look so far:
└── AcmeBundle
├── assets/
├── bin/
├── docs/
├── src/
│ ├── DependencyInjection/
│ ├── Model/
│ ├── Resources/
│ │ ├── config/
│ │ ├── public/
│ │ ├── translations/
│ │ └── views/
│ ├── Service/
│ └── AcmeBundle.php
└── tests/Up to now, this structure is still compatible with all Symfony versions.
See it on [`revamped`](https://github.com/yceruto/acme-bundle/tree/revamped) branch.
#### Upgraded Version
This version will be supported since Symfony 4.4, where a new directory convention for bundles was introduced,
allowing you to have `config/`, `public/`, `translations/` and `templates/` directories at the root of your bundle.It means that the `src/Resources/` directory is no longer needed, let's move these directories:
$ mv src/Resources/config/ src/Resources/public/ src/Resources/translations/ ./
$ mv src/Resources/views templates
$ rmdir src/ResourcesAt this time, the directory structure would look like this (consistent with [the standard PHP packages skeleton](https://github.com/php-pds/skeleton)):
└── AcmeBundle
├── assets/
├── bin/
├── config/
├── docs/
├── public/
├── src/
│ ├── DependencyInjection/
│ ├── Model/
│ ├── Service/
│ └── AcmeBundle.php
├── templates/
├── tests/
└── translations/But, to make this work properly, it is necessary to change the root path of the bundle:
class AcmeBundle extends Bundle
{
public function getPath(): string
{
return \dirname(__DIR__);
}
}See it on [`upgraded`](https://github.com/yceruto/acme-bundle/tree/upgraded) branch.
#### Mixed Version
If you want your bundle to be compatible with older versions of Symfony, use symlink as a workaround:
$ mkdir src/Resources && cd src/Resources
$ ln -s ../../config/ config
$ ln -s ../../public/ public
$ ln -s ../../translations/ translations
$ ln -s ../../templates/ viewsThe final structure would be this one:
└── AcmeBundle
├── assets/
├── bin/
├── config/
├── docs/
├── public/
├── src/
│ ├── DependencyInjection/
│ ├── Model/
│ ├── Resources/
│ │ ├── config/ (symlink)
│ │ ├── public/ (symlink)
│ │ ├── translations/ (symlink)
│ │ └── views/ (symlink)
│ ├── Service/
│ └── AcmeBundle.php
├── templates/
├── tests/
└── translations/Last, let's define the bundle path according to the current Symfony version installed:
use Symfony\Component\HttpKernel\Kernel;
class AcmeBundle extends Bundle
{
public function getPath(): string
{
return Kernel::VERSION_ID >= 40400 ? \dirname(__DIR__) : __DIR__;
}
}
That's it!See it on [`mixed`](https://github.com/yceruto/acme-bundle/tree/mixed) branch.
License
-------This software is published under the [MIT License](LICENSE.md)