PHP 8 & Symfony 6 #1153
PHP 8 & Symfony 6 #1153
Conversation
|
Kudos, SonarCloud Quality Gate passed!
|
|
Ci is nog niet blij :( |
NathanHuisman
force-pushed
the
php8-symfony6
branch
2 times, most recently
from
649e802
to
a810a62
Compare
De UserInterface is veranderd, de method 'getUsername' is veranderd naar de method 'getUserIdentifier'.
De constructor signature van UsernamePasswordToken is veranderd
NathanHuisman
force-pushed
the
php8-symfony6
branch
from
a810a62
to
5b914e7
Compare
Verander alle routing annotations naar attributes
Hierbij is ook de manier waarop de app geladen wordt veranderd, configuratie.include.php wordt op dit moment niet geladen TODO: We moeten deze hele logica veranderen om een beetje overeen te komen met best practices.
| @@ -102,7 +102,6 @@ package-lock.json | |||
| ###< phpunit/phpunit ### | |||
|
|
|||
| ###> symfony/phpunit-bridge ### | |||
| .phpunit | |||
| @@ -222,7 +222,7 @@ | |||
| <path value="$PROJECT_DIR$/vendor/twig/string-extra" /> | |||
| </include_path> | |||
| </component> | |||
| <component name="PhpProjectSharedConfiguration" php_language_level="7.3"> | |||
| <component name="PhpProjectSharedConfiguration" php_language_level="8.1"> | |||
There was a problem hiding this comment.
Deze moet ook nog even naar 8.2
There was a problem hiding this comment.
Dependencies moeten naar 6.4 voor upgraden naar Symfony 7
| class: Symfony\Bundle\SecurityBundle\Security | ||
| public: true | ||
|
|
||
| security.authorization_checker: |
There was a problem hiding this comment.
Moet nog even nagaan waar dit precies voor is
| @@ -1,16 +1,16 @@ | |||
| FROM php:7.3-apache | |||
| FROM php:8.1-apache | |||
| { | ||
| public function __construct( | ||
| private readonly EntityManagerInterface $entityManager, | ||
| private readonly ObjectNormalizer $normalizer | ||
| ) { | ||
| } | ||
|
|
||
| public function getSupportedTypes(?string $format): array { |
There was a problem hiding this comment.
Op dit moment de meest veilige optie, ik weet niet hoe het caching mechanisme precies werkt
| "symfony/config": "5.4.*", | ||
| "symfony/dotenv": "5.4.*", | ||
| "sentry/sentry-symfony": "^4.14", | ||
| "symfony/cache": "^6.0", |
| @@ -1,11 +1,6 @@ | |||
| <?xml version="1.0" encoding="UTF-8"?> | |||
| <!-- https://phpunit.readthedocs.io/en/latest/configuration.html --> | |||
| <phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/9.3/phpunit.xsd" backupGlobals="false" colors="true" bootstrap="tests/bootstrap.php"> | |||
| <coverage processUncoveredFiles="true"> | |||
There was a problem hiding this comment.
Coverage is wel handig om te hebben, we moeten maar even kijken hoe we dat het best kunnen configureren
| @@ -12,3 +12,7 @@ | |||
| } elseif (method_exists(Dotenv::class, 'bootEnv')) { | |||
| (new Dotenv())->bootEnv(dirname(__DIR__) . '/.env'); | |||
| } | |||
|
|
|||
| if ($_SERVER['APP_DEBUG']) { | |||
There was a problem hiding this comment.
Waar is dit voor nodig?
| @@ -88,27 +88,6 @@ protected function setUp(): void | |||
| */ | |||
| protected function tearDown(): void | |||
| { | |||
| $status = $this->getStatus(); | |||
There was a problem hiding this comment.
Panther kan als het goed is zelf screenshots maken, moet nog even kijken hoe we dat moeten configureren
In plaats van een flash message, log een error. TODO: Gebrek aan LDAP minder problematisch maken
|
| @@ -5,5 +5,5 @@ SYMFONY_DEPRECATIONS_HELPER=999999 | |||
| PANTHER_WEB_SERVER_DIR=./htdocs/ | |||
| # --force-prefers-reduced-motion om panther scrollen direct te maken. | |||
| PANTHER_CHROME_ARGUMENTS='--window-size=1400,1024 --force-prefers-reduced-motion' | |||
|
|
|||
| PANTHER_ERROR_SCREENSHOT_DIR='./var/error-screenshots' | |||
There was a problem hiding this comment.
Waarom werkt dit niet...
Zodra de stek naar PHP 8 kan natuurlijk