- Jan 22, 2024
-
-
Christian Kuhn authored
page = PAGE page { 10 = COA 10 { 10 = TEXT 10.value = A 20 = TEXT 20.value = B 30 = TEXT 30.value = C } } page.10.30 > This correctly unsets 'page.10.30', but also 'page.10.10' which should be kept of course. The patch fixes this by avoiding a bogus removeChildByName(). Resolves: #102831 Related: #97816 Releases: main, 12.4 Change-Id: Iab66e9c3187588cb403f7c4e65145863314817d3 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/82559 Tested-by:core-ci <typo3@b13.com> Tested-by:
Christian Kuhn <lolli@schwarzbu.ch> Reviewed-by:
-
Christian Kuhn authored
Extbase BE modules suffer from the conceptual mistake of using FE TypoScript for configuration. Modules not in page context pick the first root template in the page tree as TS entry point. The patch changes this awful handling to also use the attached site object for this page to substitute site settings used as constants in TS correctly. Resolves: #102452 Related: #97816 Releases: main, 12.4 Change-Id: I24e1e814cd20fc4999311d4f0068fb51818ad611 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/82558 Tested-by:
-
Julian Hofmann authored
No semicolon after method argument is needed/allowed. Releases: main, 12.4 Resolves: #102888 Change-Id: Ib04783dc487456df88d5b42ae68882ab8c34eb67 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/82573 Tested-by:
-
Stefan Bürk authored
New core-testing-* images has been build and published in the TYPO3 testing-infrastructure repository and pulled to the core ci gitlab runner instances. [1][2] 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 - at least for the phpXY images. * core-testing-php81 => 2.10 * core-testing-php82 => 1.9 * core-testing-php83 => 1.10 * core-testing-apache24 => 1.3 * core-testing-nodejs18 => 1.2 * core-testing-nodejs18-chrome => 1.2 [1] https://git.typo3.org/typo3/CI/testing-infrastructure/-/commit/2fb328622d88d47afd3e8abc9bf6ff32f7ef3db6 [2] https://git.typo3.org/typo3/CI/testing-infrastructure/-/commit/e1ec0954b823acbab3f37100f8b6d0a03c31f0d8 Resolves: #102886 Releases: main, 12.4, 11.5 Change-Id: I3830a5f53b18e2170c9f973a66007dc3868d99dd Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/82553 Tested-by:
Stefan Bürk <stefan@buerk.tech> Tested-by:
-
- Jan 21, 2024
-
-
Torben Hansen authored
The documentation for redirect mode "Defined by Domain entries" for ext:felogin does not describe, that configured domains must be known to TYPO3. This change updates the documentation, so it is clear, that all domains for the "Defined by Domain entries" redirect mode must be configured in TYPO3 site settings. Resolves: #98372 Releases: main, 12.4 Change-Id: I2a18ce40016d60c3878656cc5194cec6af5ed6a5 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/82572 Tested-by:
Benni Mack <benni@typo3.org> Reviewed-by:
-
Oliver Bartsch authored
This fixes several bugs of the recordlist pagination page input field. 1. The selector for the corresponding form element is no longer hardcoded to "pages", enabling the field for all tables again. 2. A JavaScript warning regarding "NaN" for parseInt() is resolved by using `Number()`. 3. Setting the value is only done in case corresponding `min` / `max` settings exist. Resolves: #102872 Resolves: #102861 Releases: main, 12.4 Change-Id: I2110fef23e17ccbde3e37754be5ef3c5187dfa43 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/82555 Tested-by:
Oliver Bartsch <bo@cedev.de> Reviewed-by:
-
- Jan 20, 2024
-
-
Marcin Sągol authored
Ensure unique values in BackendUserAuthentication::groupData['filemounts'] by processing it through the StringUtility::uniqueList() function. Prior to this change, values were not always unique due to the combination of permissions for backend users and groups. Resolves: #102857 Releases: main, 12.4 Change-Id: I37b4021e22cd3dfaa5323fe6713fae321b72173d Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/82481 Tested-by:
Guido Schmechel <guido.schmechel@brandung.de> Tested-by:
-
- Jan 19, 2024
-
-
Marcin Sągol authored
When we use DB Check module and our project has large number of tables configured in the TCA it is often hard to quickly find on the select list the one we are interested with. This commit adds small changes to `DatabaseIntegrityController` to: - sort (ASC) available tables list by table label - if table label is not defined, use raw table name wrapped with [...] - sort (ASC) fields defined for table by their labels - if field label is not defined, [FIELD: xxx] is used as before Examples for table label $GLOBALS['TCA']['sys_file']['title'] = 'Label'; - result without debug mode enabled in the BE: 'Label' - result with debug mode disabled in BE: 'Label [sys_file]' $GLOBALS['TCA']['sys_file']['title'] = ''; - result without debug mode enabled in the BE: '[sys_file]' - result with debug mode disabled in BE: '[sys_file]' Examples for field label $GLOBALS['TCA']['sys_file']['title']['columns']['field']['label'] = 'Label'; - result without debug mode enabled in the BE: 'Label' - result with debug mode disabled in BE: 'Label [field]' $GLOBALS['TCA']['sys_file']['title']['columns']['field']['label'] = ''; - result without debug mode enabled in the BE: '[FIELD: field]' - result with debug mode disabled in BE: '[FIELD: field] [field]' Resolves: #98110 Releases: main, 12.4 Change-Id: I6511ee44f2503b9908a9bdc9f06b508c2f049483 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/82480 Tested-by:
Andreas Kienast <a.fernandez@scripting-base.de> Reviewed-by:
-
- Jan 18, 2024
-
-
Oliver Bartsch authored
Instead of using a class alias for the `Controller` PHP attribute, is the class now restored. This allows to install TYPO3 even without enabling the `typo3/class-alias-loader` composer plugin when the deprecated `Controller` attribute is used by an extensions. We also adapt our codebase to use the newer `AsController` attribute as an example for extensions authors that copy code from the sourcetree. Resolves: #102858 Related: #102631 Releases: 12.4 Change-Id: Ia12843e9c29dc9bd32c2500e4c1c7d22aba4352a Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/82507 Tested-by:
Josef Glatz <typo3@josefglatz.at> Reviewed-by:
Benjamin Franzke <ben@bnf.dev> Tested-by:
-
Benjamin Franzke authored
The textarea element needs to be available in the DOM before the table wizard element is rendered, as the connected callback can be triggered between the rendering of these distinct elements, which would cause the initialization to fail. Resolves: #102748 Related: #102550 Related: #102330 Releases: main, 12.4 Change-Id: I5ad597fb0f64489c0d51a2d3bbc1a71831925061 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/82479 Tested-by:
-
linawolf authored
If you happen to have inconsistent data in the config.yaml like: ``` routes: - route: robots.txt type: staticText source: 'EXT:my_sitepackage_mysite/Resources/Public/robots.txt' ``` Currently a PHP type error is thrown instead of a proper exception. Resolves: #102773 Releases: main, 12.4 Change-Id: If16a93b851237c9d39893cd3e56fa9693986721d Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/82478 Tested-by:
-
- Jan 16, 2024
-
-
Oliver Bartsch authored
In case a plugin, using the `list` CType has an item group configured, which is not defined in `list_type>config>itemGroups`, EMU::addPlugin() tries to get the label from the `CType>config>itemGroups` definition. This does now properly work by using the correct array access in the if clause as well as the correct fallback value in case the defined item group does not exists in `CType>config>itemGroups`. Resolves: #102841 Related: #102826 Releases: main, 12.4 Change-Id: If50fc533b6b3c107e21d73b7b0b6380c45807093 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/82477 Tested-by:
-
Oliver Hader authored
Change-Id: I60ef2b390e55d5544f599d82f8c71b64093ede8b Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/82467 Tested-by:
Oliver Hader <oliver.hader@typo3.org> Reviewed-by:
-
Oliver Hader authored
Change-Id: Id818722727dd70ecccf819231901c8ba2946049f Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/82466 Reviewed-by:
-
Benjamin Franzke authored
The example value '-t -i' for `transport_sendmail_command` is incorrect, as an executable is expected to be provided, not just command flags. Since this option is not needed for SMTP at all, it is dropped from this example. Releases: main, 12.4, 11.5 Resolves: #102837 Change-Id: I99569f23339ae68077edf16aaa59873aa84ba24d Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/82475 Tested-by:
-
- Jan 14, 2024
-
-
Anja Leichsenring authored
The label has been added, so the output of the help section for the command will now provide basic information about the option. Resolves: #102833 Releases: main, 12.4 Change-Id: I72e82f1516eaf960ce572b8e920be6bcaa747663 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/82474 Reviewed-by:
-
Thore authored
Exception message is wrong. The file did not move, the path is empty in itself or due to error in getFileAbsFileName. Releases: main, 12.4 Resolves: #102829 Change-Id: Ia10cc4a3511d042eecba9d06b34a501ab1998182 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/82473 Tested-by:
Anja Leichsenring <aleichsenring@ab-softlab.de> Tested-by:
-
Nikita Hovratov authored
Fix comment about obsolete type user with dbType json. Resolves: #102832 Releases: main, 12.4 Change-Id: I390a1a7d3a7fe4d511f361efae4c2735f9b34c4a Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/82472 Tested-by:
-
- Jan 12, 2024
-
-
Stefan Bürk authored
Doctrine DBAL 3.7.0 introduced a performance issue [1] which has been already fixed by the Doctrine Team [2]. This change updates the `doctrine/dbal` composer constraint to ensure that this performance issue is gone. Note: The monorepo composer.lock already containted the 3.7.2 bugfis release, therefore this has not been detected in casual core development. This update is to mitigate the performance issue at all costs for TYPO3 users. As a sideeffect, this should put `composerInstallMin` nightly function tests on speed again. Used command(s): > composer require --no-update --no-install \ -d typo3/sysext/redirects \ "doctrine/dbal":"^3.7.2" ; \ composer require --no-update --no-install \ -d typo3/sysext/core \ "doctrine/dbal":"^3.7.2" ; \ composer require --no-update --no-install \ -d typo3/sysext/install \ "doctrine/dbal":"^3.7.2" ; \ composer require --no-update \ "doctrine/dbal":"^3.7.2" ; \ composer update --lock [1] https://github.com/contao/contao/issues/6409 [2] https://github.com/doctrine/dbal/pull/6202 Resolves: #102830 Releases: main, 12.4 Change-Id: If8b09f9556f11cc0fda4690f81343fc6c5d5e476 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/82449 Reviewed-by:Oliver Klee <typo3-coding@oliverklee.de> Tested-by:
Garvin Hicking <gh@faktor-e.de> Tested-by:
-
Benni Mack authored
When using DataHandler->log, the format changed to using an associative array as $data, whereas the message then contains PSR-3 compatible log strings. This was handled in TYPO3 v12 but a few places around Workspaces were missed. This patch adapts these places. Resolves: #102827 Related: #97556 Releases: main, 12.4 Change-Id: Id871d65a52243489d7434025850438e0f7aeb670 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/82391 Reviewed-by:
-
Benni Mack authored
With TYPO3 v12, the SelectItem API was introduced to allow an associative array for registering items in TCA select. However, it was forgotten to be added in registerPlugin and addPlugin, which is now supported as well to be consistent around TYPO3 Core, and the new API. As an addition, Extbase plugins can now also add a description in order to have feature parity with Core API. Core plugins such as indexed search now utilize description and group, moving "EXT:indexed_search" plugin into the forms group for consistency reasons. Resolves: #102826 Related: #99739 Releases: main, 12.4 Change-Id: Ifcfc5f8ce482fab3203fe6a9591cc152d0a4261a Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/82390 Tested-by:
-
- Jan 11, 2024
-
-
Nikita Hovratov authored
Resolves: #102812 Releases: main, 12.4 Change-Id: I43ea655938b8e4e691359bd22113c4f02ea71036 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/82389 Reviewed-by:
Nikita Hovratov <nikita.h@live.de> Tested-by:
-
Andreas Kienast authored
This commit updates all `lit` packages to their respective latest version. Executed commands: npm install --save \ @lit/reactive-element@^2.0.3 \ lit@^3.1.1 \ lit-html@^3.1.1 \ lit-element@^4.0.3 npm install --save-dev \ eslint-plugin-lit@^1.11.0 npm run build Resolves: #102816 Releases: main, 12.4 Change-Id: Ic0a3908b128174953ea10aadf5da1bff1d477868 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/82438 Tested-by: -
Benni Mack authored
Resolves: #101397 Releases: 12.4 Change-Id: Ic8a46c8121b779afbe50f8845ff819a42c604e6a Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/82430 Tested-by:
-
Andreas Kienast authored
This commit updates `typescript` and its `eslint` and `terser` friend packages to their respective latest version. Executed commands: npm install --save-dev \ @typescript-eslint/eslint-plugin@^6.18.0 \ @typescript-eslint/parser@^6.18.0 \ @typescript-eslint/typescript-estree@^6.18.0 \ eslint@^8.56.0 \ grunt-eslint@^24.3.0 \ terser@^5.26.0 \ typescript@^5.3.3 npm run build Resolves: #102788 Releases: main, 12.4 Change-Id: I18132539bbe088f7c6025e0a461649244b0641e8 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/82429 Tested-by:
-
- Jan 10, 2024
-
-
Stefan Bürk authored
With #102479 support for persisting datetime_immutable object has been ensured and covered with a functional test. However, using a empty string as default for a not nullable datetime field is invalid for MariaDB and MySQL which seemed to work until now in test executions. Something has changed, but had no luck to find any traces in the mariadb, mysql or corresponding docker hub changelogs and commits explaining this. The TYPO3 default TCA schema creates nullable fields with a null default for `date, datetime and timestamp` anyway, and therefore we now use that for the test case table structure. It make sense to test a valid structure if working state is expected. Note: It seems that core ci is not blocked by this yet, however local execution are broken. To verify with and without the change use following command(s): > Build/Scripts/runTests.sh -u > Build/Scripts/runTests.sh -s functional -d mysql \ typo3/sysext/core/Tests/Functional/Database/ Resolves: #102807 Releated: #102479 Releases: main, 12.4 Change-Id: Ide32b3f9dc4b049ff5c3982997c85a0bb01e2fce Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/82388 Tested-by: -
Oliver Bartsch authored
In case an invalid link type is used in a link field, the link browser does no longer throw an exception but instead displays a flash message with the corresponding information. Additionally, an undefined array key access is resolved, which might happen e.g. in case such invalid link is being analyzed by link validator. Resolves: #102804 Releases: main, 12.4 Change-Id: I5ad35ff46360cc4b27556b8e2c75da69ecbe76d9 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/82387 Tested-by:
-
- Jan 09, 2024
-
-
Torben Hansen authored
The title of a file reference container is currently wrapped to multiple lines, depending on the length of the title and the current screen resolution. This change removes the fixed column width of the dt element. Additionally, the debug information is now on the same line as the title, which is the same behavior as for other inline elements in TYPO3. Resolves: #102066 Related: #99873 Releases: main, 12.4 Change-Id: Ic2b7d8b008a3dd4a95dce0f55228fd89e9b243af Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/82386 Tested-by:
-
Oliver Bartsch authored
Due to the major update of the `chart.js` package to v4, corresponding `legend` configuration of the BarChartWidget and the DoughnutChartWidget needs to be migrated. Resolves: #102759 Releases: main, 12.4 Change-Id: Iced423d184a86b334390cad21319bf088a8369a1 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/82385 Tested-by:
-
Nikita Hovratov authored
This patch replaces all hard-coded occurrences with a class property, allowing to set the table name as a constructor argument. The page view could be way more flexible if it would allow to render grid column items of other inline tables the same way as for tt_content inside previews. This is done at the back of the mind to enable easy Content Blocks backend previews for inline children. Resolves: #102792 Releases: main, 12.4 Change-Id: I29ee892f1e3dbcb56cd0988039d1996ebf7ba157 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/82414 Tested-by:
-
Stefan Bürk authored
The `\TYPO3\CMS\Core\Service\OpcodeCacheService->isClearable()` method checks for existence of opcache function and if they are listed in the `disable_functions` configuration. It seems, that in some environments both methods are missing and not listed in the disabled methods. Even if this sounds odd when the opcache extension is installed but both methods are not available, we add it to the check to avoid the missleading error message: Call to undefined function TYPO3\CMS\Core\Service\opcache_reset() Resolves: #102734 Releases: main, 12.4, 11.5 Change-Id: Icc17a276e8881f502a687db7ca57fc05ecb8981e Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/82383 Tested-by:
-
Oliver Bartsch authored
Replace the hard-coded `sys_language_uid` name with the actually configured `languageField` value. Resolves: #102758 Releases: main, 12.4 Change-Id: I74d0624c0b79b3085247d5d58f091857ff228411 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/82382 Tested-by:
-
Andreas Kienast authored
This commit updates the `flag-icons` and its related build packages `filesize` and `sharpen` to their respective latest version. See changelog at [1]. Executed commands: npm install --save-dev \ filesize@^10.1.0 \ flag-icons@^7.1.0 \ sharp@^0.33.1 npm run build [1] https://github.com/lipis/flag-icons/blob/v7.1.0/CHANGELOG.md Resolves: #102780 Releases: main, 12.4 Change-Id: Ife74d3679d18ec374e40140183c69f912e5b1439 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/82406 Tested-by: -
Andreas Kienast authored
The supported node version is raised to 18.19, see https://nodejs.org/en/blog/release/v18.19.0 for further details. Resolves: #102787 Releases: main, 12.4 Change-Id: I9a199b748b9c3859188fa7ae7809526afe4fda43 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/82381 Reviewed-by:
-
Stefan Bürk authored
New `core-testing-*` images has been build and released. [1] This change update used image version to: * core-testing-nodejs18:1.1.0 * core-testing-nodejs18-chrome:1.1.0 * core-testing-apache24:1.2.0 Additionally, the allowed npm engine in `Build/package.json` is set accordingly. [1] https://git.typo3.org/typo3/CI/testing-infrastructure/-/commit/614aa2cc6173a49195a74772fb22fbcf0b3d25f2 Resolves: #102782 Releases: main, 12.4 Change-Id: I639f446e46b21fe2969a107eace39f9977eee933 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/82380 Tested-by:
-
- Jan 08, 2024
-
-
Andreas Kienast authored
This commit updates `luxon` to 3.4.4. See changelog at [1]. Executed commands: npm install --save luxon@^3.4.4 npm install --save-dev @types/luxon@^3.4.0 npm run build [1] https://github.com/moment/luxon/blob/3.4.4/CHANGELOG.md Resolves: #102777 Releases: main, 12.4 Change-Id: I15419f8693779b8a51eebd2fb9a3e3d822a247c4 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/82398 Tested-by: -
Andreas Kienast authored
This commit updates the JavaScript testing suite, including all related dependencies. As a new requirement, `@types/mocha` is installed as well. Executed commands: npm install --save-dev \ @open-wc/testing@^4.0.0 \ @web/dev-server-esbuild@^1.0.1 \ @web/dev-server-import-maps@^0.2.0 \ @web/test-runner@^0.18.0 \ @web/test-runner-playwright@^0.11.0 \ @types/mocha@^10.0.6 \ sinon@^17.0.1 npm run build npm run test Resolves: #102778 Releases: main, 12.4 Change-Id: Ic2e044a98a1d2536b75420b6b17ef98923d5b6bc Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/82395 Reviewed-by: -
Andreas Kienast authored
Executed commands: npm install --save sortablejs@^1.15.1 npm install --save-dev @types/sortablejs@^1.15.7 npm run build Resolves: #102776 Releases: main, 12.4 Change-Id: I8a9969d87c87b8df708e9e7d7d9cb08a86ae7e54 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/82393 Tested-by: -
Peter Kraume authored
Resolves: #102772 Releases: main, 12.4 Change-Id: I483a54dc85e0505b336634c55709c543075a3b13 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/82379 Tested-by:
-
Oliver Bartsch authored
Some controllers / modules are not registered as dedicated modules but only as route, e.g. FormEngine. This is now respected by the start module evaluation, making the redirect feature working again for those use cases. Resolves: #102743 Related: #102665 Releases: main, 12.4 Change-Id: I1398d7aacdcd5d8a3c433fdb21f8710e34668d63 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/82378 Tested-by:
-
