- Mar 22, 2024
-
-
Christian Ludwig authored
Fixes the Doctrine error: Incorrect integer value: '' for column 'editlock' at row 1 Releases: main, 12.4, 11.5 Resolves: #103461 Resolves: #103462 Change-Id: I0e3f0344157f66c07681bd29a47f4f51e32e4225 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83564 Tested-by:
Oliver Bartsch <bo@cedev.de> Reviewed-by:
Oliver Bartsch <bo@cedev.de> Tested-by:
Stefan Bürk <stefan@buerk.tech> Tested-by:
core-ci <typo3@b13.com> Reviewed-by:
Stefan Bürk <stefan@buerk.tech>
-
Stefan Bürk authored
New core-testing-* images have been build and published in the TYPO3 testing-infrastructure repository and pulled to the core ci gitlab runner instances. [1] This change modifies `Build/Scripts/runTests.sh` to set new image version to ensure the usage for all changes based on this change. For information what changed in the image see the patches in the testing-infrastrucure repository linked below. * core-testing-php82 => 1.12 * core-testing-php83 => 1.13 [1] https://git.typo3.org/typo3/CI/testing-infrastructure/-/commit/ebdaefdb54a4fe970abf0cb1d32567bbd6339de5 Resolves: #103469 Releases: main, 12.4, 11.5 Change-Id: I35ffd10e7979d4fd59aab5a7fa5d52330e71e0e1 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83569 Reviewed-by:
Oliver Klee <typo3-coding@oliverklee.de> Reviewed-by:
Stefan Bürk <stefan@buerk.tech> Tested-by:
Stefan Bürk <stefan@buerk.tech> Tested-by:
Oliver Klee <typo3-coding@oliverklee.de> Tested-by:
Oliver Bartsch <bo@cedev.de> Reviewed-by:
Oliver Bartsch <bo@cedev.de> Tested-by:
core-ci <typo3@b13.com> Reviewed-by:
Benjamin Franzke <ben@bnf.dev> Tested-by:
Benjamin Franzke <ben@bnf.dev>
-
Benni Mack authored
Various places in smaller extensions were reported in phpstan that had some inconsistencies in their code style, which were unnecessary and are now avoided. Resolves: #103383 Releases: main Change-Id: I1d9653e62e753c87e20bf8f2f9ff299a85c8726e Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83449 Tested-by:
Benjamin Franzke <ben@bnf.dev> Tested-by:
core-ci <typo3@b13.com> Reviewed-by:
Stefan Bürk <stefan@buerk.tech> Tested-by:
Benni Mack <benni@typo3.org> Reviewed-by:
Benjamin Franzke <ben@bnf.dev> Reviewed-by:
Benni Mack <benni@typo3.org> Tested-by:
Stefan Bürk <stefan@buerk.tech>
-
Stefan Bürk authored
`SiteBasedTestTrait` created a instance of the `SiteConfiguration` using the PHP `new` keyword for writing functional test instance site configurations. It is possible, that the `SiteConfiguration` is retrieved from the `Dependency Injection container` by code executed before the test releated site configuration is written, which does not reset the instance based `SiteConfiguration->$firstLevelCache` property and later retrieving site configurations using the `SiteFinder`, for example in rootline and other operations, cannot find the test instance configurations. This change now uses the `SiteConfiguration` from the test instance `Dependency Injection container` which ensures that the internal first level cache is properly reset on `write()` actions. Resolves: #103470 Releases: main, 12.4 Change-Id: I70368e56abd10e55f06d3813371c4538ad704136 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83572 Reviewed-by:
Benjamin Franzke <ben@bnf.dev> Tested-by:
core-ci <typo3@b13.com> Tested-by:
Stefan Bürk <stefan@buerk.tech> Reviewed-by:
Benni Mack <benni@typo3.org> Tested-by:
Benjamin Franzke <ben@bnf.dev> Reviewed-by:
Stefan Bürk <stefan@buerk.tech> Tested-by:
Benni Mack <benni@typo3.org>
-
Oliver Bartsch authored
When removeByType is used to remove restrictions from the container, it is now ensured that also cross classes are properly respected. Resolves: #103456 Releases: main, 12.4 Change-Id: I4841fe1cba7388358771cf01ee220afa4482349a Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83562 Tested-by:
Stefan Bürk <stefan@buerk.tech> Reviewed-by:
Oliver Bartsch <bo@cedev.de> Tested-by:
Oliver Bartsch <bo@cedev.de> Tested-by:
core-ci <typo3@b13.com> Tested-by:
Benni Mack <benni@typo3.org> Reviewed-by:
Benni Mack <benni@typo3.org> Reviewed-by:
Stefan Bürk <stefan@buerk.tech>
-
Oliver Bartsch authored
Unset non matching key arrays and additionally secure the array access with a corresponding check. Resolves: #102241 Releases: main, 12.4 Change-Id: If034faf04f52f6e8882dd0b610fa5dfdf9ec2298 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83559 Tested-by:
Oliver Bartsch <bo@cedev.de> Tested-by:
Benni Mack <benni@typo3.org> Reviewed-by:
Stefan Bürk <stefan@buerk.tech> Tested-by:
Stefan Bürk <stefan@buerk.tech> Tested-by:
core-ci <typo3@b13.com> Reviewed-by:
Benni Mack <benni@typo3.org> Reviewed-by:
Oliver Bartsch <bo@cedev.de>
-
Benni Mack authored
BackendLayoutView is used to build the BackendLayout structure, and does a few things too much: a) it uses ArrayUtility over a simple isset() check for pageTsconfig b) it resolves LLL labels directly, even if MoveContent wizard and Page module handle that as well c) it re-builds logic only used in Move Content which is not needed due to a properly resolved colPos list already All of this internal code can be removed and thus reduced to a minimum, with Page module, FormEngine (pages+tt_content) and Move Content Wizard continue to work, because FormEngine and all other modules already use the proper API for rendering already. Resolves: #103463 Releases: main Change-Id: Ib3878d08936e60a9cbd2f405260f3badcefcf8ae Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83565 Reviewed-by:
Andreas Kienast <a.fernandez@scripting-base.de> Tested-by:
Andreas Kienast <a.fernandez@scripting-base.de> Reviewed-by:
Stefan Bürk <stefan@buerk.tech> Tested-by:
Oliver Bartsch <bo@cedev.de> Tested-by:
core-ci <typo3@b13.com> Reviewed-by:
Oliver Bartsch <bo@cedev.de>
-
- Mar 21, 2024
-
-
Torben Hansen authored
In order to ensure unique HMACs in TYPO3, all usages of `TYPO3\CMS\Core\Utility\GeneralUtility::hmac` have been replaced with the `hmac` function in `\TYPO3\CMS\Core\Crypto\HashService`. This change finally deprecates `GeneralUtility::hmac()`. Resolves: #102762 Relates: #102761 Releases: main Change-Id: I3624ea8e2d6ec91abddfa1580ad68c1189b9445c Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83462 Tested-by:
core-ci <typo3@b13.com> Reviewed-by:
Oliver Klee <typo3-coding@oliverklee.de> Reviewed-by:
Benni Mack <benni@typo3.org> Tested-by:
Oliver Klee <typo3-coding@oliverklee.de> Tested-by:
Benni Mack <benni@typo3.org>
-
Torben Hansen authored
A todo for the `$uid` property in AbstractDomainObject states, that a type declaration should be introduced in v13. This has been achieved with #102878, so the todo can be removed. Resolves: #103460 Related: #102878 Releases: main Change-Id: I96535dc78e1568198c70d39143e7c27e0ed1ce4c Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83563 Tested-by:
Benni Mack <benni@typo3.org> Tested-by:
core-ci <typo3@b13.com> Reviewed-by:
Benni Mack <benni@typo3.org>
-
Lina Wolf authored
Resolves: #103424 Releases: main, 12.4 Change-Id: I3c2da18a3d09e78fe9b72113e1a7332942bde715 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83521 Tested-by:
Lina Wolf <112@linawolf.de> Tested-by:
core-ci <typo3@b13.com> Reviewed-by:
Lina Wolf <112@linawolf.de> Tested-by:
Garvin Hicking <gh@faktor-e.de> Reviewed-by:
Garvin Hicking <gh@faktor-e.de>
-
Andreas Kienast authored
This commit checks whether `eval` and `is_in` are not empty before they are used as "input params" for the JavaScript part of the FormEngine. Also, a plausibility check is added to ensure `eval=is_in` and a non-falsy `is_in` configuration are given. Resolves: #103184 Releases: main, 12.4 Change-Id: I23da4483f1b35b0720ce59d07847fe01ead9a461 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83085 Reviewed-by:
Andreas Kienast <a.fernandez@scripting-base.de> Tested-by:
core-ci <typo3@b13.com> Reviewed-by:
Markus Klein <markus.klein@typo3.org> Tested-by:
Oliver Bartsch <bo@cedev.de> Tested-by:
Andreas Kienast <a.fernandez@scripting-base.de> Reviewed-by:
Oliver Bartsch <bo@cedev.de> Tested-by:
Markus Klein <markus.klein@typo3.org>
-
Andreas Kienast authored
This commit adds a missing `form-check-label` CSS class to the label element of the "Toggle all" field in the backend module of EXT:linkvalidator to have pointer cursor there as well. Resolves: #103452 Releases: main Change-Id: I471a0607f6a3ce4c9c603aee59b0619a4d54cbf9 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83558 Tested-by:
Oliver Bartsch <bo@cedev.de> Tested-by:
core-ci <typo3@b13.com> Reviewed-by:
Oliver Bartsch <bo@cedev.de> Reviewed-by:
Benjamin Franzke <ben@bnf.dev> Tested-by:
Benjamin Franzke <ben@bnf.dev>
-
Benjamin Franzke authored
YamlFileLoader needs to be instantiated via `self::new` to ensure that a logger instance is injected. The issues was introduced with #103187 where YamlFileLoader was added to be provided by the EXT:install ServiceProvider. This is actually not quite right, as a service is always to be provided by the extension that ships the service, not by the consumer. Therefore the YamlFileLoader service definition is now moved into EXT:core. Resolves: #103444 Related: #103187 Releases: main Change-Id: Ifa3b690a43239c26903443789f59fcd2768fd9d5 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83553 Tested-by:
core-ci <typo3@b13.com> Tested-by:
Andreas Kienast <a.fernandez@scripting-base.de> Reviewed-by:
Andreas Kienast <a.fernandez@scripting-base.de> Tested-by:
Oliver Bartsch <bo@cedev.de> Tested-by:
Benjamin Franzke <ben@bnf.dev> Reviewed-by:
Benjamin Franzke <ben@bnf.dev> Reviewed-by:
Oliver Bartsch <bo@cedev.de>
-
Andreas Kienast authored
TYPO3's TCA type field is rendered in the FormEngine along with a `<typo3-formengine-updater>` element that listens on changes on the linked type field. This is a rare case: if FormEngine is programmatically rendered via an own controller and the value of the `type` field is hardcoded via `overrideValues` in the FormEngine compiler, the field itself is not rendered. However, the `<typo3-formengine-updater>` element is still there, leaving an "empty" field group. This patch now checks whether something is available for rendering before rendering the `<typo3-formengine-updater>` element. Resolves: #103449 Releases: main Change-Id: I708b49665dfaa5b836954e40e99cbcb9ed44ec03 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83554 Tested-by:
core-ci <typo3@b13.com> Reviewed-by:
Oliver Bartsch <bo@cedev.de> Tested-by:
Oliver Bartsch <bo@cedev.de> Tested-by:
Benjamin Franzke <ben@bnf.dev> Reviewed-by:
Benjamin Franzke <ben@bnf.dev>
-
Christian Kuhn authored
With TF no longer calling releaseLocks(), it can be removed. > composer u typo3/testing-framework:dev-main Change-Id: I93f68435db5eb09e7057c9de1d714a0d66d67f40 Resolves: #103442 Related: #103410 Releases: main Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83531 Tested-by:
core-ci <typo3@b13.com> Tested-by:
Christian Kuhn <lolli@schwarzbu.ch> Reviewed-by:
Christian Kuhn <lolli@schwarzbu.ch>
-
- Mar 20, 2024
-
-
Jasmina Ließmann authored
Resolves: #103429 Releases: main, 12.4 Change-Id: Id8ab8e745a316f8af0abcfe495b7174f0afe1b96 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83530 Tested-by:
Christian Kuhn <lolli@schwarzbu.ch> Reviewed-by:
Christian Kuhn <lolli@schwarzbu.ch> Tested-by:
Benni Mack <benni@typo3.org> Reviewed-by:
Benni Mack <benni@typo3.org> Reviewed-by:
Jasmina Ließmann <minapokhalo+typo3@gmail.com> Tested-by:
Jasmina Ließmann <minapokhalo+typo3@gmail.com> Tested-by:
core-ci <typo3@b13.com>
-
Andreas Kienast authored
This commit adds `_ga` to the `excludedParameters` configuration used for the cHash calculation. Resolves: #103435 Releases: main, 12.4, 11.5 Change-Id: I5c79c9264d1d8f09f45db947d714d27b3da4fb00 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83529 Tested-by:
JAKOTA Design Group GmbH <suchowski@jakota.de> Reviewed-by:
Oliver Klee <typo3-coding@oliverklee.de> Tested-by:
core-ci <typo3@b13.com> Reviewed-by:
David Steeb <david.steeb@b13.com> Tested-by:
Andreas Kienast <a.fernandez@scripting-base.de> Reviewed-by:
Andreas Kienast <a.fernandez@scripting-base.de> Reviewed-by:
Benni Mack <benni@typo3.org> Tested-by:
David Steeb <david.steeb@b13.com> Reviewed-by:
JAKOTA Design Group GmbH <suchowski@jakota.de> Tested-by:
Benni Mack <benni@typo3.org>
-
Christian Kuhn authored
Releases: main Resolves: #103431 Change-Id: I640fa48610e85bccf43929cafd31cd0158a1a679 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83527 Tested-by:
Garvin Hicking <gh@faktor-e.de> Tested-by:
Benni Mack <benni@typo3.org> Reviewed-by:
Garvin Hicking <gh@faktor-e.de> Tested-by:
core-ci <typo3@b13.com> Reviewed-by:
Benni Mack <benni@typo3.org>
-
- Mar 19, 2024
-
-
Christian Kuhn authored
In version 12, the introduction of the new TypoScript parser was accompanied by the implementation of factories for PageTsConfig and UserTsConfig. A factory for Frontend TypoScript has not been added, though: Frontend TypoScript creation ended up in TSFE->getFromCache(). At this point, establishing a proper factory was unfeasible due to the numerous dependencies of TypoScript creation to TSFE internals. With recent refactorings around TSFE, coupled with lots of state now being represented as request attributes, it's now possible to decompose getFromCache() and establish a FrontendTypoScriptFactory. getFromCache() is a complex beast: It influences Frontend rendering performance a lot, and tries to trigger the least amount of calculations, especially in 'fully cached pages' context. This results in high required complexity due to lots of state with diverse cross dependencies. The method composes of three main steps: 1. Bootstrap TypoScript setting ("constants") and calculate setup condition verdicts. This creates required TypoScript related state needed to calculate the page cache identifier. 2. Access page cache, lock page rendering if needed, and see if a possible page cache content contains uncached ("_INT") sections. 3. Calculate at least setup "config." depending on given type/typeNum, but create full TypoScript setup if a cached page contains uncached sections or could not be retrieved from cache. The patch extracts parts 1 and 3 to FrontendTypoScriptFactory. Part 2 is moved into PrepareTypoScriptFrontendRendering middleware. This approach allowed these related refactorings: * The release of rendering locks is now consolidated within the PrepareTypoScriptFrontendRendering middleware. This guarantees locks are released, even in scenarios where lower middleware components encounter errors. This addresses an issue where locks retained during crashes, leading to deadlock situations in subsequent requests. * Dependencies to TSFE within ext:redirects RedirectService are reduced, and it no longer locks page rendering. * The Extbase BackendConfigurationManager utilizes the new factory, eliminating the need for its own implementation. The patch unlocks further refactorings: It especially allows removing the cache related properties from TSFE by representing them as request attributes. Subsequent patches will address this task accordingly. Resolves: #103410 Related: #97816 Related: #98914 Related: #102932 Releases: main Change-Id: I7fd158cffeebe6b2c64e0e3595284b8780fb73cf Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83179 Tested-by:
Christian Kuhn <lolli@schwarzbu.ch> Reviewed-by:
Stefan Bürk <stefan@buerk.tech> Reviewed-by:
Garvin Hicking <gh@faktor-e.de> Tested-by:
Stefan Bürk <stefan@buerk.tech> Tested-by:
core-ci <typo3@b13.com> Tested-by:
Garvin Hicking <gh@faktor-e.de> Reviewed-by:
Christian Kuhn <lolli@schwarzbu.ch>
-
Benjamin Franzke authored
* Always clean-up containers when Control-C is pressed * Enforce removing networks even if some containers have not been fully stopped (does not work with older docker versions, therefore only enabled for podman) * Bail out if `waitFor` failed instead of resuming with subsequent commands * Avoid waiting unnecessarily for VNC-web port 7900 in headless mode * Do not derive `--pull=never` from the `$CI` environment variable, as `$CI` is set both in gitlab [1] and github [2] actions. runTests.sh might be used in github actions, therefore this flag is moved into our gitlab-ci specific configuration. * Fix typos [1] https://docs.gitlab.com/ee/ci/variables/predefined_variables.html [2] https://docs.github.com/en/actions/learn-github-actions/variables#default-environment-variables Resolves: #103420 Releases: main, 12.4, 11.5 Change-Id: Ie8fcdc5ebc5ba651242bb54faffdfd8de0dc6026 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83515 Tested-by:
Christian Kuhn <lolli@schwarzbu.ch> Reviewed-by:
Christian Kuhn <lolli@schwarzbu.ch> Reviewed-by:
Stefan Bürk <stefan@buerk.tech> Reviewed-by:
Benjamin Franzke <ben@bnf.dev> Tested-by:
core-ci <typo3@b13.com> Tested-by:
Stefan Bürk <stefan@buerk.tech> Tested-by:
Benjamin Franzke <ben@bnf.dev>
-
Marcin Sągol authored
This commit introduces a description for the newly added tableRestrictions palette in the Backend (BE) user groups form, which groups the tables_select and tables_modify fields together. The description clarifies that tables designated for modification are automatically considered as tables for listing as well. This enhancement aims to improve user experience by eliminating the redundant step of setting tables for listing when they have already been marked for modification. Resolves: #103426 Related: #103413 Releases: main Change-Id: I0410f5d406e105f83e5cbaf13eec6322d4f74233 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83522 Reviewed-by:
Christian Kuhn <lolli@schwarzbu.ch> Tested-by:
core-ci <typo3@b13.com> Reviewed-by:
Stefan Bürk <stefan@buerk.tech> Tested-by:
Christian Kuhn <lolli@schwarzbu.ch> Tested-by:
Stefan Bürk <stefan@buerk.tech>
-
Jasmina Ließmann authored
In order to get a better overview of the selected table restrictions, the two corresponding fields are placed next to each other. Resolves: #103413 Releases: main Change-Id: Idb14eea022b6b6732090e482e2584e8292b10d89 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83500 Reviewed-by:
Markus Klein <markus.klein@typo3.org> Tested-by:
Markus Klein <markus.klein@typo3.org> Reviewed-by:
Christian Kuhn <lolli@schwarzbu.ch> Tested-by:
Jasmina Ließmann <minapokhalo+typo3@gmail.com> Tested-by:
Christian Kuhn <lolli@schwarzbu.ch> Reviewed-by:
Jasmina Ließmann <minapokhalo+typo3@gmail.com> Tested-by:
core-ci <typo3@b13.com>
-
Christian Kuhn authored
These two classes are part of the new BE tree, and are documented API for extensions to add additional labels and status to tree nodes. They should not be marked `@internal`. Resolves: #103422 Related: #103186 Related: #103211 Change-Id: I63f466a2726e39058f400599eac70059fb5a6594 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83520 Reviewed-by:
Garvin Hicking <gh@faktor-e.de> Tested-by:
Christian Kuhn <lolli@schwarzbu.ch> Tested-by:
Jasmina Ließmann <minapokhalo+typo3@gmail.com> Tested-by:
core-ci <typo3@b13.com> Tested-by:
Oliver Klee <typo3-coding@oliverklee.de> Reviewed-by:
Oliver Klee <typo3-coding@oliverklee.de> Tested-by:
Garvin Hicking <gh@faktor-e.de> Reviewed-by:
Christian Kuhn <lolli@schwarzbu.ch> Reviewed-by:
Jasmina Ließmann <minapokhalo+typo3@gmail.com>
-
Benjamin Franzke authored
This commit updates ckeditor5 to v41.2.1. It fixed a recent regression where importing documents from Word with suggestions caused CKEditor 5 to crash. See changelog at https://github.com/ckeditor/ckeditor5/releases/tag/v41.2.1. Executed commands: npm install --save \ @ckeditor/ckeditor5-alignment@^41.2.1 \ @ckeditor/ckeditor5-autoformat@^41.2.1 \ @ckeditor/ckeditor5-basic-styles@^41.2.1 \ @ckeditor/ckeditor5-block-quote@^41.2.1 \ @ckeditor/ckeditor5-clipboard@^41.2.1 \ @ckeditor/ckeditor5-code-block@^41.2.1 \ @ckeditor/ckeditor5-core@^41.2.1 \ @ckeditor/ckeditor5-editor-classic@^41.2.1 \ @ckeditor/ckeditor5-engine@^41.2.1 \ @ckeditor/ckeditor5-essentials@^41.2.1 \ @ckeditor/ckeditor5-find-and-replace@^41.2.1 \ @ckeditor/ckeditor5-font@^41.2.1 \ @ckeditor/ckeditor5-heading@^41.2.1 \ @ckeditor/ckeditor5-horizontal-line@^41.2.1 \ @ckeditor/ckeditor5-html-support@^41.2.1 \ @ckeditor/ckeditor5-indent@^41.2.1 \ @ckeditor/ckeditor5-link@^41.2.1 \ @ckeditor/ckeditor5-list@^41.2.1 \ @ckeditor/ckeditor5-paragraph@^41.2.1 \ @ckeditor/ckeditor5-paste-from-office@^41.2.1 \ @ckeditor/ckeditor5-remove-format@^41.2.1 \ @ckeditor/ckeditor5-show-blocks@^41.2.1 \ @ckeditor/ckeditor5-source-editing@^41.2.1 \ @ckeditor/ckeditor5-special-characters@^41.2.1 \ @ckeditor/ckeditor5-style@^41.2.1 \ @ckeditor/ckeditor5-table@^41.2.1 \ @ckeditor/ckeditor5-theme-lark@^41.2.1 \ @ckeditor/ckeditor5-ui@^41.2.1 \ @ckeditor/ckeditor5-undo@^41.2.1 \ @ckeditor/ckeditor5-utils@^41.2.1 \ @ckeditor/ckeditor5-word-count@^41.2.1 npm install --save-dev \ @ckeditor/ckeditor5-dev-utils@^39.6.3 npm run build Resolves: #103421 Releases: main, 12.4 Change-Id: I19bc3c823bffa99a6055514edcfb8d03c37e4098 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83512 Reviewed-by:
Andreas Kienast <a.fernandez@scripting-base.de> Tested-by:
core-ci <typo3@b13.com> Tested-by:
Benjamin Franzke <ben@bnf.dev> Reviewed-by:
Benjamin Franzke <ben@bnf.dev> Tested-by:
Andreas Kienast <a.fernandez@scripting-base.de> Tested-by:
Benni Mack <benni@typo3.org> Reviewed-by:
Benni Mack <benni@typo3.org>
-
Benjamin Franzke authored
All applicable acceptance tests are now also executed in composer mode. Tests that check classic-mode specific functions are excluded and are therefore now tagged as classic-mode test. The composer mode instance is generated with our CLI setup tools, allowing to mimic the realworld case where a instance is creating via our setup tools instead of from fixtures (which we still use for classic mode tests, but try to reducde/avoid for composer mode tests). The existing existing classic-mode test acceptance test execution will keep running as-is. Resolves: #103297 Releases: main, 12.4, 11.5 Change-Id: I64973f110931b51ed2ef7ef8f8cc3411834fcf37 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83027 Tested-by:
Stefan Bürk <stefan@buerk.tech> Reviewed-by:
Christian Kuhn <lolli@schwarzbu.ch> Tested-by:
Christian Kuhn <lolli@schwarzbu.ch> Reviewed-by:
Stefan Bürk <stefan@buerk.tech> Tested-by:
core-ci <typo3@b13.com> Reviewed-by:
Benjamin Franzke <ben@bnf.dev> Tested-by:
Benjamin Franzke <ben@bnf.dev>
-
- Mar 18, 2024
-
-
Willi Wehmeier authored
The used ll keys used in the javascript code for the inline pagetitle edit feature in the page and recordlist module aren't set correctly in the corresponding templates. This patch solves that mismatch. Resolves: #103419 Releases: main, 12.4 Change-Id: I5dfb242bb596bef73a3d9474e8cd31ded723fea0 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83502 Reviewed-by:
Benni Mack <benni@typo3.org> Tested-by:
core-ci <typo3@b13.com> Tested-by:
Andreas Kienast <a.fernandez@scripting-base.de> Tested-by:
Simon Schaufelberger <simonschaufi+typo3@gmail.com> Reviewed-by:
Simon Schaufelberger <simonschaufi+typo3@gmail.com> Tested-by:
Benni Mack <benni@typo3.org> Reviewed-by:
Andreas Kienast <a.fernandez@scripting-base.de>
-
Oliver Bartsch authored
The $request might not be initialized, that's why the getRequest() method has to be used in ContentObjectRenderer. This is now done for newly introduced occurrences, accessing the class property directly. Resolves: #103407 Releases: main Change-Id: I466f31316b2885132288cbc8745f3dc4760bdcf2 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83497 Tested-by:
core-ci <typo3@b13.com> Reviewed-by:
Oliver Klee <typo3-coding@oliverklee.de> Reviewed-by:
Benjamin Franzke <ben@bnf.dev> Tested-by:
Stefan Bürk <stefan@buerk.tech> Tested-by:
Benjamin Franzke <ben@bnf.dev> Reviewed-by:
Stefan Bürk <stefan@buerk.tech> Reviewed-by:
Benni Mack <benni@typo3.org>
-
- Mar 17, 2024
-
-
Oliver Klee authored
In #88366, the caches have been renamed, e.g., from `cache_core` to `core`. Some occurrences of the old names in comments were missed at that time. This commit streamlines those occurrences to also use the new names. Resolves: #103409 Related: #88366 Releases: main, 12.4, 11.5 Change-Id: I69039160b3d3cc5da00d4f3d3e57fce755d22a63 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83498 Reviewed-by:
Christian Kuhn <lolli@schwarzbu.ch> Tested-by:
core-ci <typo3@b13.com> Tested-by:
Christian Kuhn <lolli@schwarzbu.ch>
-
- Mar 16, 2024
-
-
Oliver Bartsch authored
Resolves: #103391 Releases: main, 12.4 Change-Id: I48d55c7b66669f9836886f257aefba2171b6910e Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83455 Tested-by:
Jochen Roth <rothjochen@gmail.com> Tested-by:
core-ci <typo3@b13.com> Reviewed-by:
Jochen Roth <rothjochen@gmail.com> Tested-by:
Andreas Kienast <a.fernandez@scripting-base.de> Reviewed-by:
Oliver Bartsch <bo@cedev.de> Tested-by:
Oliver Bartsch <bo@cedev.de> Reviewed-by:
Andreas Kienast <a.fernandez@scripting-base.de>
-
- Mar 15, 2024
-
-
Oliver Bartsch authored
Resolves: #103385 Releases: main, 12.4, 11.5 Change-Id: Ifd93dfa4049d0d3550d052fa960ae79cc5a27c34 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83451 Reviewed-by:
Oliver Klee <typo3-coding@oliverklee.de> Tested-by:
Garvin Hicking <gh@faktor-e.de> Tested-by:
Stefan Bürk <stefan@buerk.tech> Tested-by:
Georg Ringer <georg.ringer@gmail.com> Tested-by:
core-ci <typo3@b13.com> Reviewed-by:
Georg Ringer <georg.ringer@gmail.com> Reviewed-by:
Stefan Bürk <stefan@buerk.tech> Reviewed-by:
Garvin Hicking <gh@faktor-e.de>
-
Willi Wehmeier authored
This change makes the toggle button accessible via keyboard and sets the focus manually, so that the pagetree can be toggled via pressing space or enter on the keyboard. Resolves: #102426 Releases: main, 12.4 Change-Id: I6916cf1f19de288e961ecb0dab85c76241097d33 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/81817 Tested-by:
core-ci <typo3@b13.com> Reviewed-by:
Andreas Kienast <a.fernandez@scripting-base.de> Tested-by:
Benjamin Kott <benjamin.kott@outlook.com> Reviewed-by:
Benjamin Kott <benjamin.kott@outlook.com> Tested-by:
Andreas Kienast <a.fernandez@scripting-base.de>
-
Willi Wehmeier authored
This patch adds a label to the action-menu used by most of the core extensions. Resolves: #102548 Releases: main, 12.4 Change-Id: I57db0e4d35b439e8841d7f4281bc09b593bb9dc4 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/82008 Reviewed-by:
Andreas Kienast <a.fernandez@scripting-base.de> Tested-by:
core-ci <typo3@b13.com> Tested-by:
Benjamin Kott <benjamin.kott@outlook.com> Reviewed-by:
Benjamin Kott <benjamin.kott@outlook.com> Tested-by:
Andreas Kienast <a.fernandez@scripting-base.de>
-
Oliver Hader authored
Several tests for URI route generation are stricter and assert that either the whole URI matches, or that at least the `?cHash=` suffix is applied for the corresponding test permutations. Besides that, individual URI assertions are streamlined to use AbstractEnhancerLinkGeneratorTestCase::assertGeneratedUriEquals. Resolves: #103396 Releases: main, 12.4, 11.5 Change-Id: I65ae10fa2984021da653bac33579feb6d10a6464 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83463 Reviewed-by:
Oliver Hader <oliver.hader@typo3.org> Tested-by:
Oliver Hader <oliver.hader@typo3.org> Reviewed-by:
Stefan Bürk <stefan@buerk.tech> Tested-by:
Benni Mack <benni@typo3.org> Tested-by:
core-ci <typo3@b13.com> Tested-by:
Stefan Bürk <stefan@buerk.tech> Reviewed-by:
Benni Mack <benni@typo3.org>
-
Andreas Kienast authored
If an exception cannot get written into the database, e.g. due to connectivity issues, exceptions are already caught. However, this doesn't work on low-level errors that may arise if e.g. the DI container is defunct, required by `ConnectionPool` internals. The exception handling is changed to cover all `\Throwable`s. Resolves: #103402 Releases: main Change-Id: I52ce0e91e7f3297893928d6d1b79616b34e85798 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83471 Reviewed-by:
Christoph Lehmann <christoph.lehmann@networkteam.com> Reviewed-by:
Oliver Hader <oliver.hader@typo3.org> Tested-by:
core-ci <typo3@b13.com> Tested-by:
Benjamin Franzke <ben@bnf.dev> Tested-by:
Oliver Hader <oliver.hader@typo3.org> Reviewed-by:
Andreas Kienast <a.fernandez@scripting-base.de> Reviewed-by:
Benjamin Franzke <ben@bnf.dev> Tested-by:
Andreas Kienast <a.fernandez@scripting-base.de>
-
- Mar 14, 2024
-
-
Willi Wehmeier authored
The thumbnail images in the inline file relations have the properties alt and title set to "id=x" which is only helpful when used to show the record uid in the tooltip. For screenreader users the info in the alt attribute isn't helpful at all. That's why the alt text should be empty so the thumbnail doesn't get read out by screenreaders at all. Resolves: #103397 Releases: main, 12.4 Change-Id: I233a54705c9c81e31b8852a8454165593fa8e061 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83467 Reviewed-by:
Jasmina Ließmann <minapokhalo+typo3@gmail.com> Tested-by:
Jasmina Ließmann <minapokhalo+typo3@gmail.com> Tested-by:
Oliver Bartsch <bo@cedev.de> Tested-by:
core-ci <typo3@b13.com> Reviewed-by:
Andreas Kienast <a.fernandez@scripting-base.de> Reviewed-by:
Oliver Bartsch <bo@cedev.de>
-
Garvin Hicking authored
Two errors were made in 13.1 changelogs: Filename Features-93942-CropSVGImagesNatively.rst has an extra "s" and should be "Feature". Feature-103309-AddMoreExpressionMethodsToExpressionBuilder.rst used non-existant "info::" admonition, should be "note::". Resolves: #103394 Releases: main Change-Id: I377f523f966332541ceaef6af10efae710d14b02 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83461 Reviewed-by:
Chris Müller <typo3@krue.ml> Tested-by:
Stefan Bürk <stefan@buerk.tech> Reviewed-by:
Stefan Bürk <stefan@buerk.tech> Reviewed-by:
Benni Mack <benni@typo3.org> Tested-by:
core-ci <typo3@b13.com> Tested-by:
Benni Mack <benni@typo3.org> Tested-by:
Chris Müller <typo3@krue.ml>
-
Andreas Kienast authored
The eslint configuration now takes the rule `@typescript-eslint/prefer-string-starts-ends-with` [1] into account, enforcing the usage of `startsWith()` and `endWith()` over index checking in strings. [1] https://typescript-eslint.io/rules/prefer-string-starts-ends-with Resolves: #103374 Releases: main, 12.4 Change-Id: I88640f71cbe960db2a2e2b162410ff5a79e7c55c Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83437 Tested-by:
Andreas Kienast <a.fernandez@scripting-base.de> Reviewed-by:
Anja Leichsenring <aleichsenring@ab-softlab.de> Tested-by:
core-ci <typo3@b13.com> Tested-by:
Anja Leichsenring <aleichsenring@ab-softlab.de> Reviewed-by:
Benjamin Franzke <ben@bnf.dev> Tested-by:
Garvin Hicking <gh@faktor-e.de> Reviewed-by:
Garvin Hicking <gh@faktor-e.de> Reviewed-by:
Andreas Kienast <a.fernandez@scripting-base.de>
-
- Mar 13, 2024
-
-
Willi Wehmeier authored
The "+" labelled button in the File selector modal was just labelled with the filename as the title which doesn't give any meaning to screenreader users. To show the purpose of the button to screenreader users the title has been changed to additionally describe the action. Also an aria-label with the same value has been added. This is not ideal, because some screenreaders will read out the title twice, but on the other hand other screenreaders ignore the title or the aria-label attribute. Resolves: #103292 Releases: main, 12.4 Change-Id: Ia53fd77623e3cbd3a63a48f1bd4edc457f453c6b Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83224 Tested-by:
Michael Telgkamp <michael.telgkamp@mindscreen.de> Tested-by:
core-ci <typo3@b13.com> Reviewed-by:
Michael Telgkamp <michael.telgkamp@mindscreen.de> Tested-by:
Oliver Bartsch <bo@cedev.de> Tested-by:
Benjamin Franzke <ben@bnf.dev> Reviewed-by:
Oliver Bartsch <bo@cedev.de> Reviewed-by:
Benjamin Franzke <ben@bnf.dev>
-
Oliver Bartsch authored
This prevents the element browser from closing as soon as one file or folder was added. This restores previous behaviour, allowing the user to add multiple references without having to open the browser again and again. Resolves: #103369 Releases: main, 12.4 Change-Id: Id57ebe3ed115c0929beb06fd7dd136fe0f08e415 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83434 Reviewed-by:
Jochen Roth <rothjochen@gmail.com> Tested-by:
Andreas Kienast <a.fernandez@scripting-base.de> Reviewed-by:
Andreas Kienast <a.fernandez@scripting-base.de> Tested-by:
Oliver Bartsch <bo@cedev.de> Tested-by:
core-ci <typo3@b13.com> Tested-by:
Jochen Roth <rothjochen@gmail.com> Reviewed-by:
Oliver Bartsch <bo@cedev.de>
-
Ayke Halder authored
In the backend CEs are elevated by a surrounding box-shadow. This box-shadow was clipped off on top, left and right side. Resolves: #103381 Releases: main, 12.4 Change-Id: I3052ff7ced18e509aefb2f05c6b1bde8725572b5 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83445 Reviewed-by:
Oliver Bartsch <bo@cedev.de> Tested-by:
Garvin Hicking <gh@faktor-e.de> Reviewed-by:
Garvin Hicking <gh@faktor-e.de> Tested-by:
Oliver Bartsch <bo@cedev.de> Tested-by:
core-ci <typo3@b13.com>
-