OskarStark
Repos
267
Followers
203
Following
59

GithubAction for PHP-CS-Fixer

153
26

GithubAction for PHPStan

86
20

The Symfony PHP framework

27459
8581

The Symfony documentation

1988
4728

The missing Symfony Admin Generator

2072
1251

Events

composer(deps-dev): bump phpunit/phpunit from 8.5.29 to 8.5.30 (#125)

Bumps phpunit/phpunit from 8.5.29 to 8.5.30.


updated-dependencies:

  • dependency-name: phpunit/phpunit dependency-type: direct:development update-type: version-update:semver-patch ...

Signed-off-by: dependabot[bot] support@github.com

Signed-off-by: dependabot[bot] support@github.com Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

Created at 7 hours ago
OskarStark delete branch dependabot/composer/phpunit/phpunit-8.5.30
Created at 7 hours ago
pull request closed
composer(deps-dev): bump phpunit/phpunit from 8.5.29 to 8.5.30

Bumps phpunit/phpunit from 8.5.29 to 8.5.30.

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
Created at 7 hours ago

composer(deps-dev): bump phpunit/phpunit from 8.5.29 to 8.5.30

Bumps phpunit/phpunit from 8.5.29 to 8.5.30.


updated-dependencies:

  • dependency-name: phpunit/phpunit dependency-type: direct:development update-type: version-update:semver-patch ...

Signed-off-by: dependabot[bot] support@github.com

Merge pull request #239 from gansel-rechtsanwaelte/dependabot/composer/phpunit/phpunit-8.5.30

composer(deps-dev): bump phpunit/phpunit from 8.5.29 to 8.5.30

Created at 9 hours ago
OskarStark delete branch dependabot/composer/phpunit/phpunit-8.5.30
Created at 9 hours ago
composer(deps-dev): bump phpunit/phpunit from 8.5.29 to 8.5.30

Bumps phpunit/phpunit from 8.5.29 to 8.5.30.

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
Created at 9 hours ago
issue comment
Missing interface after minor upgrade

Yes same for me in another project

Created at 1 day ago
closed issue
[Ux] Add/Update docs to show new `make:twig-component` in MakerBundle

Re: https://github.com/symfony/maker-bundle/pull/1019

@weaverryan suggested updating the docs for this

make:twig-component released in MakerBundle v1.46.0

cc @kbond

Created at 2 days ago
closed issue
[HttpKernel] Use Accept-Language header even if there are no enabled lo…

| Q | A | ------------ | --- | Feature PR | symfony/symfony#47377 | PR author(s) | @MatTheCat | Merged in | 6.2

Created at 2 days ago
issue comment
[HttpKernel] Use Accept-Language header even if there are no enabled lo…

Not really

Created at 2 days ago
issue comment
Replace switch statement with match expression

Should I also replace it with:

Yes please 💪

Created at 2 days ago
issue comment
Missing interface after minor upgrade

The service is aliased, but I think @OskarStark implements a class with this interface.

Exactly and it does not have the "hack" like other classes with the class_exists wrapper

Created at 2 days ago

Explaining how to inject the form factory

The other sentence I deleted was just repeating what was already said a few lines above.

Update service documentation with udpate recipe files

Update example to use Address format

It seems strange to me that the example for the From headers uses the full Address format, whereas the sender envelope does not.

I believe it is possible to use it here, so thought I would expand on the example to show how you can set both the display name and email address when configuring the sender envelope globally.

[DI] add docs for SubscribedService::$attributes

[HttpKernel] Document ControllerEvent::getAttributes()

minor #17283 [HttpKernel] Document ControllerEvent::getAttributes() (wouterj)

This PR was squashed before being merged into the 6.2 branch.

Discussion

[HttpKernel] Document ControllerEvent::getAttributes()

Fixes #16946

Commits

56168bee4 [HttpKernel] Document ControllerEvent::getAttributes()

minor #17211 [DependencyInjection] Add docs for SubscribedService::$attributes (kbond)

This PR was merged into the 6.2 branch.

Discussion

[DependencyInjection] Add docs for SubscribedService::$attributes

Fixes #17208.

Commits

b6aa12741 [DI] add docs for SubscribedService::$attributes

Add some minor notes

Frontend Encore document passing data via serialize twig filter

minor #16403 Frontend Encore document passing data via serialize twig filter (94noni)

This PR was squashed before being merged into the 5.4 branch.

Discussion

Frontend Encore document passing data via serialize twig filter

I think this feature added in v5.3 is a great addition, specialy in this doc article ref https://symfony.com/blog/new-in-symfony-5-3-twig-serialize-filter + https://github.com/symfony/symfony/pull/40323

Cc @jrushlow @weaverryan

Commits

2a1314e49 Frontend Encore document passing data via serialize twig filter

Add the versionadded directive

Merge branch '5.4' into 6.0

  • 5.4: Add the versionadded directive Frontend Encore document passing data via serialize twig filter

Remove a versionadded directive

Merge branch '6.0' into 6.1

  • 6.0: Frontend Encore document passing data via serialize twig filter

Merge branch '6.1' into 6.2

  • 6.1: Frontend Encore document passing data via serialize twig filter

[Console] Add note about URL cutting of default output wrapper

minor #16476 [Console] Add note about URL cutting of default output wrapper (fchris82)

This PR was submitted for the 6.1 branch but it was merged into the 6.2 branch instead.

Discussion

[Console] Add note about URL cutting of default output wrapper

Add note about the new output wrapper function. Related MR: https://github.com/symfony/symfony/pull/45318

Commits

0bfa8ace9 [Console] Add note about URL cutting of default output wrapper

Reword

minor #16364 [Form] Explaining how to inject the form factory (ThomasLandauer)

This PR was merged into the 4.4 branch.

Discussion

[Form] Explaining how to inject the form factory

The other sentence I deleted was just repeating what was already said a few lines above.

Commits

ceedccae7 Explaining how to inject the form factory

Merge branch '4.4' into 5.4

  • 4.4: Explaining how to inject the form factory
Created at 3 days ago

Explaining how to inject the form factory

The other sentence I deleted was just repeating what was already said a few lines above.

Update service documentation with udpate recipe files

Update example to use Address format

It seems strange to me that the example for the From headers uses the full Address format, whereas the sender envelope does not.

I believe it is possible to use it here, so thought I would expand on the example to show how you can set both the display name and email address when configuring the sender envelope globally.

Frontend Encore document passing data via serialize twig filter

minor #16403 Frontend Encore document passing data via serialize twig filter (94noni)

This PR was squashed before being merged into the 5.4 branch.

Discussion

Frontend Encore document passing data via serialize twig filter

I think this feature added in v5.3 is a great addition, specialy in this doc article ref https://symfony.com/blog/new-in-symfony-5-3-twig-serialize-filter + https://github.com/symfony/symfony/pull/40323

Cc @jrushlow @weaverryan

Commits

2a1314e49 Frontend Encore document passing data via serialize twig filter

Add the versionadded directive

Merge branch '5.4' into 6.0

  • 5.4: Add the versionadded directive Frontend Encore document passing data via serialize twig filter

Remove a versionadded directive

minor #16364 [Form] Explaining how to inject the form factory (ThomasLandauer)

This PR was merged into the 4.4 branch.

Discussion

[Form] Explaining how to inject the form factory

The other sentence I deleted was just repeating what was already said a few lines above.

Commits

ceedccae7 Explaining how to inject the form factory

Merge branch '4.4' into 5.4

  • 4.4: Explaining how to inject the form factory

Update autowiring.rst

I removed this sentence because there is no obligation that the interface should be in the same file as the class ====> and that interface is also discovered in the same file

minor #16338 [DependencyInjection] Update autowiring.rst (amine-betari)

This PR was submitted for the 5.3 branch but it was merged into the 5.4 branch instead.

Discussion

[DependencyInjection] Update autowiring.rst

I removed this sentence because there is no obligation that the interface should be in the same file as the class ====> and that interface is also discovered in the same file

Commits

f90b0a567 Update autowiring.rst

Note that env vars are not always compatible with options

There are cases where environment variables cannot be used in place of regular configuration options. This commit makes the limitation explicit.

From https://github.com/symfony/symfony/issues/39902

minor #16383 Note that env vars are not always compatible with options (melkamar)

This PR was submitted for the 6.0 branch but it was merged into the 5.4 branch instead.

Discussion

Note that env vars are not always compatible with options

There are cases where environment variables cannot be used in place of regular configuration options. This commit makes the limitation explicit.

From https://github.com/symfony/symfony/issues/39902

Commits

6b73dda6d Note that env vars are not always compatible with options

Reword

[Security] Added a note regarding the loginUser() method

minor #16071 [Security] Added a note regarding the loginUser() method (artyuum)

This PR was submitted for the 5.3 branch but it was squashed and merged into the 5.4 branch instead.

Discussion

[Security] Added a note regarding the loginUser() method

As @javiereguiluz stated, this method won't work for 100% of Symfony users. This is my case. I'm testing some protected API routes and the firewall is using a custom authenticator that relies on a custom header called "x-api-key". After spending hours trying to understand what I did wrong in my tests, I decided to dig into the code that defines the loginUser() method and noticed that it can only work with session-based authentication.

This little note could have saved me some time, so I believe it could help future users as well who are in the same case as me.

Commits

4d29c99c2 [Security] Added a note regarding the loginUser() method

Minor reword

Add information about strict_mode for encore

minor #16207 Add information about strict_mode for encore (Philosoft)

This PR was submitted for the 6.0 branch but it was squashed and merged into the 5.4 branch instead.

Discussion

Add information about strict_mode for encore

I guess it should be both in 5.4 and 6.0, but actually will be useful down to whenever https://github.com/symfony/webpack-encore-bundle/pull/54 was merged

Personally, it took me quite a long time to determine and fix the cause of functional tests failing in CI. I know that stuff is kind of mentioned in config/packages/webpack_encore.yaml, but here are my excuses

  • there is certain time gap between you first install webpack, write your functional tests (involved with actual HTML rendering), add some styles/scripts to template and setup CI to run these tests
  • this stuff is not mentioned in the docs at all - fgrep -RH strict_mode . gives 0 results

Commits

2b6371f27 Add information about strict_mode for encore

Created at 3 days ago

Explaining how to inject the form factory

The other sentence I deleted was just repeating what was already said a few lines above.

Update service documentation with udpate recipe files

minor #16364 [Form] Explaining how to inject the form factory (ThomasLandauer)

This PR was merged into the 4.4 branch.

Discussion

[Form] Explaining how to inject the form factory

The other sentence I deleted was just repeating what was already said a few lines above.

Commits

ceedccae7 Explaining how to inject the form factory

minor #17167 Update service documentation with udpate recipe files (alexander-schranz)

This PR was merged into the 4.4 branch.

Discussion

Update service documentation with udpate recipe files

The asterisk was removed some time ago as not required and the Tests and Migrations are also were also removed from exclude.

See discussion about asterisk here: https://github.com/symfony/recipes/pull/802#discussion_r455856843

Commits

b89847e83 Update service documentation with udpate recipe files

Created at 3 days ago

Explaining how to inject the form factory

The other sentence I deleted was just repeating what was already said a few lines above.

Update service documentation with udpate recipe files

Update example to use Address format

It seems strange to me that the example for the From headers uses the full Address format, whereas the sender envelope does not.

I believe it is possible to use it here, so thought I would expand on the example to show how you can set both the display name and email address when configuring the sender envelope globally.

Frontend Encore document passing data via serialize twig filter

minor #16403 Frontend Encore document passing data via serialize twig filter (94noni)

This PR was squashed before being merged into the 5.4 branch.

Discussion

Frontend Encore document passing data via serialize twig filter

I think this feature added in v5.3 is a great addition, specialy in this doc article ref https://symfony.com/blog/new-in-symfony-5-3-twig-serialize-filter + https://github.com/symfony/symfony/pull/40323

Cc @jrushlow @weaverryan

Commits

2a1314e49 Frontend Encore document passing data via serialize twig filter

Add the versionadded directive

Merge branch '5.4' into 6.0

  • 5.4: Add the versionadded directive Frontend Encore document passing data via serialize twig filter

Remove a versionadded directive

Merge branch '6.0' into 6.1

  • 6.0: Frontend Encore document passing data via serialize twig filter

minor #16364 [Form] Explaining how to inject the form factory (ThomasLandauer)

This PR was merged into the 4.4 branch.

Discussion

[Form] Explaining how to inject the form factory

The other sentence I deleted was just repeating what was already said a few lines above.

Commits

ceedccae7 Explaining how to inject the form factory

Merge branch '4.4' into 5.4

  • 4.4: Explaining how to inject the form factory

Update autowiring.rst

I removed this sentence because there is no obligation that the interface should be in the same file as the class ====> and that interface is also discovered in the same file

minor #16338 [DependencyInjection] Update autowiring.rst (amine-betari)

This PR was submitted for the 5.3 branch but it was merged into the 5.4 branch instead.

Discussion

[DependencyInjection] Update autowiring.rst

I removed this sentence because there is no obligation that the interface should be in the same file as the class ====> and that interface is also discovered in the same file

Commits

f90b0a567 Update autowiring.rst

Note that env vars are not always compatible with options

There are cases where environment variables cannot be used in place of regular configuration options. This commit makes the limitation explicit.

From https://github.com/symfony/symfony/issues/39902

minor #16383 Note that env vars are not always compatible with options (melkamar)

This PR was submitted for the 6.0 branch but it was merged into the 5.4 branch instead.

Discussion

Note that env vars are not always compatible with options

There are cases where environment variables cannot be used in place of regular configuration options. This commit makes the limitation explicit.

From https://github.com/symfony/symfony/issues/39902

Commits

6b73dda6d Note that env vars are not always compatible with options

Reword

[Security] Added a note regarding the loginUser() method

minor #16071 [Security] Added a note regarding the loginUser() method (artyuum)

This PR was submitted for the 5.3 branch but it was squashed and merged into the 5.4 branch instead.

Discussion

[Security] Added a note regarding the loginUser() method

As @javiereguiluz stated, this method won't work for 100% of Symfony users. This is my case. I'm testing some protected API routes and the firewall is using a custom authenticator that relies on a custom header called "x-api-key". After spending hours trying to understand what I did wrong in my tests, I decided to dig into the code that defines the loginUser() method and noticed that it can only work with session-based authentication.

This little note could have saved me some time, so I believe it could help future users as well who are in the same case as me.

Commits

4d29c99c2 [Security] Added a note regarding the loginUser() method

Minor reword

Add information about strict_mode for encore

Created at 3 days ago

Explaining how to inject the form factory

The other sentence I deleted was just repeating what was already said a few lines above.

Update service documentation with udpate recipe files

Update example to use Address format

It seems strange to me that the example for the From headers uses the full Address format, whereas the sender envelope does not.

I believe it is possible to use it here, so thought I would expand on the example to show how you can set both the display name and email address when configuring the sender envelope globally.

Frontend Encore document passing data via serialize twig filter

minor #16403 Frontend Encore document passing data via serialize twig filter (94noni)

This PR was squashed before being merged into the 5.4 branch.

Discussion

Frontend Encore document passing data via serialize twig filter

I think this feature added in v5.3 is a great addition, specialy in this doc article ref https://symfony.com/blog/new-in-symfony-5-3-twig-serialize-filter + https://github.com/symfony/symfony/pull/40323

Cc @jrushlow @weaverryan

Commits

2a1314e49 Frontend Encore document passing data via serialize twig filter

Add the versionadded directive

minor #16364 [Form] Explaining how to inject the form factory (ThomasLandauer)

This PR was merged into the 4.4 branch.

Discussion

[Form] Explaining how to inject the form factory

The other sentence I deleted was just repeating what was already said a few lines above.

Commits

ceedccae7 Explaining how to inject the form factory

Merge branch '4.4' into 5.4

  • 4.4: Explaining how to inject the form factory

Update autowiring.rst

I removed this sentence because there is no obligation that the interface should be in the same file as the class ====> and that interface is also discovered in the same file

minor #16338 [DependencyInjection] Update autowiring.rst (amine-betari)

This PR was submitted for the 5.3 branch but it was merged into the 5.4 branch instead.

Discussion

[DependencyInjection] Update autowiring.rst

I removed this sentence because there is no obligation that the interface should be in the same file as the class ====> and that interface is also discovered in the same file

Commits

f90b0a567 Update autowiring.rst

Note that env vars are not always compatible with options

There are cases where environment variables cannot be used in place of regular configuration options. This commit makes the limitation explicit.

From https://github.com/symfony/symfony/issues/39902

minor #16383 Note that env vars are not always compatible with options (melkamar)

This PR was submitted for the 6.0 branch but it was merged into the 5.4 branch instead.

Discussion

Note that env vars are not always compatible with options

There are cases where environment variables cannot be used in place of regular configuration options. This commit makes the limitation explicit.

From https://github.com/symfony/symfony/issues/39902

Commits

6b73dda6d Note that env vars are not always compatible with options

Reword

[Security] Added a note regarding the loginUser() method

minor #16071 [Security] Added a note regarding the loginUser() method (artyuum)

This PR was submitted for the 5.3 branch but it was squashed and merged into the 5.4 branch instead.

Discussion

[Security] Added a note regarding the loginUser() method

As @javiereguiluz stated, this method won't work for 100% of Symfony users. This is my case. I'm testing some protected API routes and the firewall is using a custom authenticator that relies on a custom header called "x-api-key". After spending hours trying to understand what I did wrong in my tests, I decided to dig into the code that defines the loginUser() method and noticed that it can only work with session-based authentication.

This little note could have saved me some time, so I believe it could help future users as well who are in the same case as me.

Commits

4d29c99c2 [Security] Added a note regarding the loginUser() method

Minor reword

Add information about strict_mode for encore

minor #16207 Add information about strict_mode for encore (Philosoft)

This PR was submitted for the 6.0 branch but it was squashed and merged into the 5.4 branch instead.

Discussion

Add information about strict_mode for encore

I guess it should be both in 5.4 and 6.0, but actually will be useful down to whenever https://github.com/symfony/webpack-encore-bundle/pull/54 was merged

Personally, it took me quite a long time to determine and fix the cause of functional tests failing in CI. I know that stuff is kind of mentioned in config/packages/webpack_encore.yaml, but here are my excuses

  • there is certain time gap between you first install webpack, write your functional tests (involved with actual HTML rendering), add some styles/scripts to template and setup CI to run these tests
  • this stuff is not mentioned in the docs at all - fgrep -RH strict_mode . gives 0 results

Commits

2b6371f27 Add information about strict_mode for encore

Tweaks

[Mailer] Update mailer.rst

Created at 3 days ago
OskarStark delete branch dependabot/github_actions/actions/stale-6
Created at 4 days ago

github-actions(deps): bump actions/stale from 5 to 6

Bumps actions/stale from 5 to 6.


updated-dependencies:

  • dependency-name: actions/stale dependency-type: direct:production update-type: version-update:semver-major ...

Signed-off-by: dependabot[bot] support@github.com

Merge pull request #238 from gansel-rechtsanwaelte/dependabot/github_actions/actions/stale-6

github-actions(deps): bump actions/stale from 5 to 6

Created at 4 days ago
github-actions(deps): bump actions/stale from 5 to 6

Bumps actions/stale from 5 to 6.

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
Created at 4 days ago