-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Conversation
It is possible to register single files to load every time in Composer. Using Composer to load the AppKernel means we don't have to manually include it in all the entry points of the application.
👍 |
@@ -15,7 +15,6 @@ set_time_limit(0); | |||
* @var Composer\Autoload\ClassLoader $loader | |||
*/ | |||
$loader = require __DIR__.'/../app/autoload.php'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks like it can be simplified too.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is not part of this pull request, additionally I believe the variable is kept so that you can uncomment the code below: https://github.com/symfony/symfony-standard/blob/2.8/web/app.php#L11-L19
ping @symfony/deciders |
@mnapoli I think you missed https://github.com/symfony/symfony-standard/blob/2.8/web/app_dev.php#L27 |
@@ -24,8 +24,6 @@ | |||
$loader = require __DIR__.'/../app/autoload.php'; | |||
Debug::enable(); | |||
|
|||
require_once __DIR__.'/../app/AppKernel.php'; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@aitboudad it’s there
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
:)
👍 |
1 similar comment
👍 |
Thank you @mnapoli. |
This PR was merged into the 2.8 branch. Discussion ---------- Autoload the AppKernel using Composer Fixes #868 Using Composer to load the AppKernel means we don't have to manually include it in all the entry points of the application. Commits ------- ab358a4 #868 Load the AppKernel class using Composer's autoloader
Shouldn't we backport this to older branches too (I don't see a reason why new Symfony 2.3 or 2.7 applications should not benefit from this change). |
It's not a bugfix, so IMO it shouldn't be backported. |
fix blog post cat name fix main page fix admin fix web add multiple root taxon add deploy param sylius override composer update fix migrate script fix vat price fix some service fix template fix product display update composer fix composer fix composer fix sf3 deploy structure fix class autoload symfony/symfony-standard#869 update node-sass fix locale fix bug - email template - auto enable user - missing shop_homepage route - taxon repo bugfix taxation fix wip
fix blog post cat name fix main page fix admin fix web add multiple root taxon add deploy param sylius override composer update fix migrate script fix vat price fix some service fix template fix product display update composer fix composer fix composer fix sf3 deploy structure fix class autoload symfony/symfony-standard#869 update node-sass fix locale fix bug - email template - auto enable user - missing shop_homepage route - taxon repo bugfix taxation fix wip bugfix fix all error small fix translation fix fix css fix css fix gulp run fix cms build gulp gulp improve fix blog post add featuredCover to post missing migrate script add title page add all posts page
fix blog post cat name fix main page fix admin fix web add multiple root taxon add deploy param sylius override composer update fix migrate script fix vat price fix some service fix template fix product display update composer fix composer fix composer fix sf3 deploy structure fix class autoload symfony/symfony-standard#869 update node-sass fix locale fix bug - email template - auto enable user - missing shop_homepage route - taxon repo bugfix taxation fix wip bugfix fix all error small fix translation fix fix css fix css fix gulp run fix cms build gulp gulp improve fix blog post add featuredCover to post missing migrate script add title page add all posts page add taxon name to post grid fix typo add link to heading content thumb better home content page post bc add product external order link add product manual tab fix image manual fix typo fix translate
Fixes #868
Using Composer to load the AppKernel means we don't have to manually include it in all the entry points of the application.