New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Replace the deprecated Doctrine cache bundle #1916
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
leofeyer
commented
Jul 7, 2020
aschempp
requested changes
Jul 7, 2020
leofeyer
force-pushed
the
fix/doctrine-cache-bundle
branch
from
July 8, 2020 08:11
0b67fbd
to
45ae7f2
Compare
leofeyer
added a commit
that referenced
this pull request
Dec 3, 2020
leofeyer
added a commit
that referenced
this pull request
Dec 11, 2020
Description ----------- | Q | A | -----------------| --- | Fixed issues | Fixes #... | Docs PR or issue | contao/docs#... Commits ------- 13c00f1 Add the feature action to the permission checks (see #2524) Description ----------- | Q | A | -----------------| --- | Fixed issues | Fixes #2505 | Docs PR or issue | n/a See #2505. Commits ------- 6131435 Add feature action to permission check as well (see #2505) 55d85f3 Allow version 3 of toflar/psr6-symfony-http-cache-store (see #2522) Description ----------- Another step towards #2263. Commits ------- 2efea3b Allow version 3 of toflar/psr6-symfony-http-cache-store 1cc3754 Increase undo expiry date (see #2528) Description ----------- | Q | A | -----------------| --- | Fixed issues | - | Docs PR or issue | - I really think the default undo period of 24h is not a good default (which is why I think it should be changed in 4.9 LTS too). For me, there are two use cases our current undo system can cover: 1. The immediate "Oh, I deleted something which I shouldn't have" case: 24 hours are enough 2. The customer did something over the weekend and you get a report on Monday that something was deleted: 24 hours are likely not enough. So 30 days by default seem totally okay to me. It makes no sense to store undo entries that date back a year or so. Commits ------- 7d7f01e Increase undo expiry date 7efdef4 Trigger the onload_callback when featuring news (see #2531) Description ----------- | Q | A | -----------------| --- | Fixed issues | Fixes #2485 | Docs PR or issue | - Commits ------- dee7c1d Trigger the onload_callback when featuring news bc8447d Trigger the onload_callback when featuring news/events (see #2532) Description ----------- | Q | A | -----------------| --- | Fixed issues | Fixes #2485 | Docs PR or issue | - Same PR as #2531 for Contao 4.10 where we also have featured events. Commits ------- c3a849c Trigger the onload_callback when featuring news/events 67c7b18 Backport the doctrine-cache-bundle changes (see #2534) Description ----------- Backports the changes from #1916 to Contao 4.9 to make it compatible with PHP 8 (see #2263). Commits ------- ba9d21f Backport the doctrine-cache-bundle changes d1103e9 CS 3feda71 Add compatibility with PHP 8 (see #2535) Description ----------- See #2263 and #2291 ### TODO * [x] Fix the tests. * [x] Wait if sebastianbergmann/php-code-coverage#837 gets merged and then use PHPUnit 8.5 for all PHP versions. * [ ] <del>Click through the back end and front end and fix all warnings.</del> Commits ------- 8ad6149 Add PHP 8.0 to the CI chain 7b8729e Fix the unit and functional tests 47c14db Use 127.0.0.1 instead of localhost in the functional tests 938836c Always set the ACCEPT_LANGUAGE header in the functional tests a4199ec Use PHPUnit 8.5 for all PHP versions 7ffbd26 Adjust the error level in PHP 8 4ce6b27 Merge branch '4.9' into fix/php8 24b1d71 Update the changelog and the language files (see #2541) Description ----------- - Commits ------- a55a7f2 Update the changelog and the language files d19cb6c Add a missing PHPStan dependency (see #2543) Description ----------- - Commits ------- 288bc91 Add a missing PHPStan dependency 80b6469 Merge branch '4.4' into 4.9 # Conflicts: # installation-bundle/src/Resources/translations/messages.fr.xlf # news-bundle/src/Resources/contao/dca/tl_news.php ffebcd7 Fix issues found by the PhpStorm code inspector (see #2544) Description ----------- Function arguments renaming → forward compat with PHP 8. Commits ------- d6b22e4 Fix issues found by the PhpStorm code inspector 1fb0b96 More fixes e2544f4 And even more e6c4b18 Add global page model for fragments (see #2506) Description ----------- | Q | A | -----------------| --- | Fixed issues | Fixes #1935 As discussed on Mumble on the 2020-11-19, this should make the (global) page model available in the fragment controllers. Commits ------- 03e7c41 Add global page model for fragments (replaces #1935) a5c2326 Cast values for comparison 826748d Added unit tests 00eeb72 CS 29ea202 CS 0e09e4e Merge branch '4.9' into bugfix/page-argument 77c79be Do not try to generate fragments for generated fragments (see #2521) Description ----------- If a fragment has multiple methods with different fragment type (e.g. a content element and a front end module), the first RegisterFragmentsPass creates a child definition that inherits all tags from the original. On the next run, the RegisterFragmentsPass would pick up the child definition and try to create duplicate fragments. Commits ------- cc5d72c Do not try to generate fragments for generated fragments a0b92bb CS 4b8bc8f Merge branch '4.9' into bugfix/fragment-children 8605bc4 Allow caption to be a textarea rather than a text field (see #2527) Description ----------- An image caption shouldn't be limited to 255 chars only, that makes no sense to me. That's the result: <img width="1174" alt="Bildschirmfoto 2020-11-27 um 13 12 19" src="https://user-images.githubusercontent.com/481937/100448322-60b94a80-30b2-11eb-88f2-99f8fb499e4d.png"> Commits ------- 06f57ee Allow caption to be a textarea rather than a text field limited to 255 chars b346f65 CS 0c23a1f Merge branch '4.9' into fix/allow-textarea-for-meta-caption 244d845 Move migrations to core bundle (see #2545) Description ----------- | Q | A | -----------------| --- | Fixed issues | Fixes #2364 Commits ------- 1d6f417 Move migrations to core bundle 20dfde3 Fix missing globals array 51fb5c7 PHPStan 5034731 Pass the filesystem as dependency and remove the Contao 3 migrations 66231e7 Fix compatibility with scssphp 1.4 (see #2547) Description ----------- | Q | A | -----------------| --- | Fixed issues | Fixes #2495 ### ToDos - [ ] Revert contao/conflicts@820074e once this got merged and released (4.9 and upstream). Commits ------- 961abaa Set the file path to fix compatibility with scssphp 1.4 946940d Run the CS fixer (see #2548) Description ----------- - Commits ------- 29af160 Run the CS fixer 0df5442 Do not index preview URLs for searching (see #2550) Description ----------- | Q | A | -----------------| --- | Fixed issues | Fixes #2507 | Docs PR or issue | - @Toflar: If the `X-Robots-Tag` header or the meta robots tag contains `noindex`, the listener will not pass the document to the indexer at all. This means that potentially existing entries will not be deleted. Is this OK? Commits ------- 17d0e8a Do not index preview URLs for searching 137d410 Correctly load the DCA labels (see #2540) Description ----------- | Q | A | -----------------| --- | Fixed issues | Fixes #2536 see #2536 for the discussion Commits ------- cb3740c Correctly load the DCA labels 64d2350 Merge branch '4.9' into bugfix/dca-labels 103649e Fix the cache tag invalidation (see #2551) Description ----------- | Q | A | -----------------| --- | Fixed issues | Fixes #2137 | Docs PR or issue | - This is currently a proof of concept that should work for news and events. @Toflar @ausi Can you confirm that my implementation is what you had in mind? ### TODO * [x] Add a general tag without ID for the top parent record * [x] Implement cache tagging for the other elements and modules Commits ------- b523f61 Fix the cache tag invalidation 64c4cdc CS 4b61ac2 Do not double tag the content elements 5b4a03d Add a tag for the top parent element 996694e Add tagging for the missing modules 601247f Also tag the comments 90caa2a Strip port numbers from root page domains (see #2553) Description ----------- | Q | A | -----------------| --- | Fixed issues | Fixes #2214 | Docs PR or issue | - Tested against https://regex101.com/r/jU0OrQ/2 Commits ------- 52e85cd Strip port numbers from root page domains 48a0c3c Improve the regex Co-authored-by: Martin Auswöger <martin@auswoeger.com> bacb48b Clean up before the release (see #2560) Description ----------- - Commits ------- 7db3c54 Clean up before the release 41b89f2 Update the changelog and the language files (see #2561) Description ----------- - Commits ------- 6c50c48 Update the changelog and the language files 918884c Update the changelog 5eb206f Merge branch '4.9' into 4.10 # Conflicts: # calendar-bundle/src/Resources/contao/languages/fr/tl_calendar_feed.xlf # composer.json # core-bundle/src/Command/InstallCommand.php # core-bundle/src/Command/SymlinksCommand.php # core-bundle/src/DependencyInjection/Compiler/RegisterFragmentsPass.php # core-bundle/src/Migration/Version408/Version480Update.php # core-bundle/src/Resources/contao/dca/tl_page.php # core-bundle/src/Resources/contao/library/Contao/Controller.php # core-bundle/src/Resources/contao/library/Contao/DcaExtractor.php # core-bundle/src/Resources/contao/library/Contao/InsertTags.php # core-bundle/src/Resources/contao/modules/Module.php # core-bundle/tests/Contao/Database/ResultTest.php # core-bundle/tests/Functional/RoutingTest.php # core-bundle/tests/Security/Authentication/RememberMe/ExpiringTokenBasedRememberMeServicesTest.php # installation-bundle/src/Database/Version330Update.php # manager-bundle/composer.json # manager-bundle/src/Resources/skeleton/config/config.yml # manager-bundle/tests/ContaoManager/PluginTest.php # monorepo.yml # news-bundle/src/Resources/contao/dca/tl_news.php # news-bundle/src/Resources/contao/languages/fr/tl_news_feed.xlf 695843a Clean up after the upstream merge (see #2564) Description ----------- - Commits ------- 950581e Clean up after the upstream merge 280339c Update the changelog and the language files (see #2565) Description ----------- - Commits ------- 5d2cada Update the changelog and the language files b7567a8 Merge branch '4.10' into master # Conflicts: # calendar-bundle/composer.json # comments-bundle/composer.json # composer.json # core-bundle/composer.json # core-bundle/src/Resources/contao/languages/fr/tl_article.xlf # core-bundle/src/Resources/contao/languages/fr/tl_form_field.xlf # core-bundle/src/Resources/contao/languages/fr/tl_image_size_item.xlf # core-bundle/src/Resources/contao/languages/fr/tl_layout.xlf # core-bundle/src/Resources/contao/languages/fr/tl_member.xlf # core-bundle/src/Resources/contao/languages/fr/tl_member_group.xlf # core-bundle/src/Resources/contao/languages/fr/tl_style_sheet.xlf # core-bundle/src/Resources/contao/languages/fr/tl_theme.xlf # core-bundle/src/Resources/contao/languages/fr/tl_user.xlf # core-bundle/src/Resources/contao/languages/fr/tl_user_group.xlf # core-bundle/tests/Contao/SearchTest.php # faq-bundle/composer.json # installation-bundle/composer.json # installation-bundle/src/Database/Version330Update.php # installation-bundle/src/Database/Version350Update.php # listing-bundle/composer.json # manager-bundle/composer.json # news-bundle/composer.json # news-bundle/src/Resources/contao/languages/cs/default.xlf # news-bundle/src/Resources/contao/languages/cs/tl_layout.xlf # news-bundle/src/Resources/contao/languages/cs/tl_news_feed.xlf # news-bundle/src/Resources/contao/languages/cs/tl_user_group.xlf # newsletter-bundle/composer.json
AlexejKossmann
pushed a commit
to AlexejKossmann/contao
that referenced
this pull request
Apr 6, 2021
Description ----------- This PR replaces the deprecated Doctrine cache bundle as discussed in contao#1889 (comment). <img width="1076" alt="" src="https://user-images.githubusercontent.com/1192057/86783205-48dc1e00-c060-11ea-8d2e-e2ad0a43d244.png"> Commits ------- d391d94 Replace the deprecated Doctrine cache bundle 45ae7f2 Move the services to the services.yml file and use the app. namespace
AlexejKossmann
pushed a commit
to AlexejKossmann/contao
that referenced
this pull request
Apr 6, 2021
Description ----------- | Q | A | -----------------| --- | Fixed issues | Fixes #... | Docs PR or issue | contao/docs#... Commits ------- 13c00f1 Add the feature action to the permission checks (see contao#2524) Description ----------- | Q | A | -----------------| --- | Fixed issues | Fixes contao#2505 | Docs PR or issue | n/a See contao#2505. Commits ------- 6131435 Add feature action to permission check as well (see contao#2505) 55d85f3 Allow version 3 of toflar/psr6-symfony-http-cache-store (see contao#2522) Description ----------- Another step towards contao#2263. Commits ------- 2efea3b Allow version 3 of toflar/psr6-symfony-http-cache-store 1cc3754 Increase undo expiry date (see contao#2528) Description ----------- | Q | A | -----------------| --- | Fixed issues | - | Docs PR or issue | - I really think the default undo period of 24h is not a good default (which is why I think it should be changed in 4.9 LTS too). For me, there are two use cases our current undo system can cover: 1. The immediate "Oh, I deleted something which I shouldn't have" case: 24 hours are enough 2. The customer did something over the weekend and you get a report on Monday that something was deleted: 24 hours are likely not enough. So 30 days by default seem totally okay to me. It makes no sense to store undo entries that date back a year or so. Commits ------- 7d7f01e Increase undo expiry date 7efdef4 Trigger the onload_callback when featuring news (see contao#2531) Description ----------- | Q | A | -----------------| --- | Fixed issues | Fixes contao#2485 | Docs PR or issue | - Commits ------- dee7c1d Trigger the onload_callback when featuring news bc8447d Trigger the onload_callback when featuring news/events (see contao#2532) Description ----------- | Q | A | -----------------| --- | Fixed issues | Fixes contao#2485 | Docs PR or issue | - Same PR as contao#2531 for Contao 4.10 where we also have featured events. Commits ------- c3a849c Trigger the onload_callback when featuring news/events 67c7b18 Backport the doctrine-cache-bundle changes (see contao#2534) Description ----------- Backports the changes from contao#1916 to Contao 4.9 to make it compatible with PHP 8 (see contao#2263). Commits ------- ba9d21f Backport the doctrine-cache-bundle changes d1103e9 CS 3feda71 Add compatibility with PHP 8 (see contao#2535) Description ----------- See contao#2263 and contao#2291 ### TODO * [x] Fix the tests. * [x] Wait if sebastianbergmann/php-code-coverage#837 gets merged and then use PHPUnit 8.5 for all PHP versions. * [ ] <del>Click through the back end and front end and fix all warnings.</del> Commits ------- 8ad6149 Add PHP 8.0 to the CI chain 7b8729e Fix the unit and functional tests 47c14db Use 127.0.0.1 instead of localhost in the functional tests 938836c Always set the ACCEPT_LANGUAGE header in the functional tests a4199ec Use PHPUnit 8.5 for all PHP versions 7ffbd26 Adjust the error level in PHP 8 4ce6b27 Merge branch '4.9' into fix/php8 24b1d71 Update the changelog and the language files (see contao#2541) Description ----------- - Commits ------- a55a7f2 Update the changelog and the language files d19cb6c Add a missing PHPStan dependency (see contao#2543) Description ----------- - Commits ------- 288bc91 Add a missing PHPStan dependency 80b6469 Merge branch '4.4' into 4.9 # Conflicts: # installation-bundle/src/Resources/translations/messages.fr.xlf # news-bundle/src/Resources/contao/dca/tl_news.php ffebcd7 Fix issues found by the PhpStorm code inspector (see contao#2544) Description ----------- Function arguments renaming → forward compat with PHP 8. Commits ------- d6b22e4 Fix issues found by the PhpStorm code inspector 1fb0b96 More fixes e2544f4 And even more e6c4b18 Add global page model for fragments (see contao#2506) Description ----------- | Q | A | -----------------| --- | Fixed issues | Fixes contao#1935 As discussed on Mumble on the 2020-11-19, this should make the (global) page model available in the fragment controllers. Commits ------- 03e7c41 Add global page model for fragments (replaces contao#1935) a5c2326 Cast values for comparison 826748d Added unit tests 00eeb72 CS 29ea202 CS 0e09e4e Merge branch '4.9' into bugfix/page-argument 77c79be Do not try to generate fragments for generated fragments (see contao#2521) Description ----------- If a fragment has multiple methods with different fragment type (e.g. a content element and a front end module), the first RegisterFragmentsPass creates a child definition that inherits all tags from the original. On the next run, the RegisterFragmentsPass would pick up the child definition and try to create duplicate fragments. Commits ------- cc5d72c Do not try to generate fragments for generated fragments a0b92bb CS 4b8bc8f Merge branch '4.9' into bugfix/fragment-children 8605bc4 Allow caption to be a textarea rather than a text field (see contao#2527) Description ----------- An image caption shouldn't be limited to 255 chars only, that makes no sense to me. That's the result: <img width="1174" alt="Bildschirmfoto 2020-11-27 um 13 12 19" src="https://user-images.githubusercontent.com/481937/100448322-60b94a80-30b2-11eb-88f2-99f8fb499e4d.png"> Commits ------- 06f57ee Allow caption to be a textarea rather than a text field limited to 255 chars b346f65 CS 0c23a1f Merge branch '4.9' into fix/allow-textarea-for-meta-caption 244d845 Move migrations to core bundle (see contao#2545) Description ----------- | Q | A | -----------------| --- | Fixed issues | Fixes contao#2364 Commits ------- 1d6f417 Move migrations to core bundle 20dfde3 Fix missing globals array 51fb5c7 PHPStan 5034731 Pass the filesystem as dependency and remove the Contao 3 migrations 66231e7 Fix compatibility with scssphp 1.4 (see contao#2547) Description ----------- | Q | A | -----------------| --- | Fixed issues | Fixes contao#2495 ### ToDos - [ ] Revert contao/conflicts@820074e once this got merged and released (4.9 and upstream). Commits ------- 961abaa Set the file path to fix compatibility with scssphp 1.4 946940d Run the CS fixer (see contao#2548) Description ----------- - Commits ------- 29af160 Run the CS fixer 0df5442 Do not index preview URLs for searching (see contao#2550) Description ----------- | Q | A | -----------------| --- | Fixed issues | Fixes contao#2507 | Docs PR or issue | - @Toflar: If the `X-Robots-Tag` header or the meta robots tag contains `noindex`, the listener will not pass the document to the indexer at all. This means that potentially existing entries will not be deleted. Is this OK? Commits ------- 17d0e8a Do not index preview URLs for searching 137d410 Correctly load the DCA labels (see contao#2540) Description ----------- | Q | A | -----------------| --- | Fixed issues | Fixes contao#2536 see contao#2536 for the discussion Commits ------- cb3740c Correctly load the DCA labels 64d2350 Merge branch '4.9' into bugfix/dca-labels 103649e Fix the cache tag invalidation (see contao#2551) Description ----------- | Q | A | -----------------| --- | Fixed issues | Fixes contao#2137 | Docs PR or issue | - This is currently a proof of concept that should work for news and events. @Toflar @ausi Can you confirm that my implementation is what you had in mind? ### TODO * [x] Add a general tag without ID for the top parent record * [x] Implement cache tagging for the other elements and modules Commits ------- b523f61 Fix the cache tag invalidation 64c4cdc CS 4b61ac2 Do not double tag the content elements 5b4a03d Add a tag for the top parent element 996694e Add tagging for the missing modules 601247f Also tag the comments 90caa2a Strip port numbers from root page domains (see contao#2553) Description ----------- | Q | A | -----------------| --- | Fixed issues | Fixes contao#2214 | Docs PR or issue | - Tested against https://regex101.com/r/jU0OrQ/2 Commits ------- 52e85cd Strip port numbers from root page domains 48a0c3c Improve the regex Co-authored-by: Martin Auswöger <martin@auswoeger.com> bacb48b Clean up before the release (see contao#2560) Description ----------- - Commits ------- 7db3c54 Clean up before the release 41b89f2 Update the changelog and the language files (see contao#2561) Description ----------- - Commits ------- 6c50c48 Update the changelog and the language files 918884c Update the changelog 5eb206f Merge branch '4.9' into 4.10 # Conflicts: # calendar-bundle/src/Resources/contao/languages/fr/tl_calendar_feed.xlf # composer.json # core-bundle/src/Command/InstallCommand.php # core-bundle/src/Command/SymlinksCommand.php # core-bundle/src/DependencyInjection/Compiler/RegisterFragmentsPass.php # core-bundle/src/Migration/Version408/Version480Update.php # core-bundle/src/Resources/contao/dca/tl_page.php # core-bundle/src/Resources/contao/library/Contao/Controller.php # core-bundle/src/Resources/contao/library/Contao/DcaExtractor.php # core-bundle/src/Resources/contao/library/Contao/InsertTags.php # core-bundle/src/Resources/contao/modules/Module.php # core-bundle/tests/Contao/Database/ResultTest.php # core-bundle/tests/Functional/RoutingTest.php # core-bundle/tests/Security/Authentication/RememberMe/ExpiringTokenBasedRememberMeServicesTest.php # installation-bundle/src/Database/Version330Update.php # manager-bundle/composer.json # manager-bundle/src/Resources/skeleton/config/config.yml # manager-bundle/tests/ContaoManager/PluginTest.php # monorepo.yml # news-bundle/src/Resources/contao/dca/tl_news.php # news-bundle/src/Resources/contao/languages/fr/tl_news_feed.xlf 695843a Clean up after the upstream merge (see contao#2564) Description ----------- - Commits ------- 950581e Clean up after the upstream merge 280339c Update the changelog and the language files (see contao#2565) Description ----------- - Commits ------- 5d2cada Update the changelog and the language files b7567a8 Merge branch '4.10' into master # Conflicts: # calendar-bundle/composer.json # comments-bundle/composer.json # composer.json # core-bundle/composer.json # core-bundle/src/Resources/contao/languages/fr/tl_article.xlf # core-bundle/src/Resources/contao/languages/fr/tl_form_field.xlf # core-bundle/src/Resources/contao/languages/fr/tl_image_size_item.xlf # core-bundle/src/Resources/contao/languages/fr/tl_layout.xlf # core-bundle/src/Resources/contao/languages/fr/tl_member.xlf # core-bundle/src/Resources/contao/languages/fr/tl_member_group.xlf # core-bundle/src/Resources/contao/languages/fr/tl_style_sheet.xlf # core-bundle/src/Resources/contao/languages/fr/tl_theme.xlf # core-bundle/src/Resources/contao/languages/fr/tl_user.xlf # core-bundle/src/Resources/contao/languages/fr/tl_user_group.xlf # core-bundle/tests/Contao/SearchTest.php # faq-bundle/composer.json # installation-bundle/composer.json # installation-bundle/src/Database/Version330Update.php # installation-bundle/src/Database/Version350Update.php # listing-bundle/composer.json # manager-bundle/composer.json # news-bundle/composer.json # news-bundle/src/Resources/contao/languages/cs/default.xlf # news-bundle/src/Resources/contao/languages/cs/tl_layout.xlf # news-bundle/src/Resources/contao/languages/cs/tl_news_feed.xlf # news-bundle/src/Resources/contao/languages/cs/tl_user_group.xlf # newsletter-bundle/composer.json
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR replaces the deprecated Doctrine cache bundle as discussed in #1889 (comment).