- Apr 25, 2024
-
-
Willi Wehmeier authored
The modal form to rename a file needs a proper label for the filename input field. Resolves: #103724 Releases: main, 12.4 Change-Id: I29018d288ced1de6001d38b841ef37a1231f7394 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83982 Reviewed-by:
Oliver Bartsch <bo@cedev.de> Tested-by:
core-ci <typo3@b13.com> Tested-by:
-
- Apr 24, 2024
-
-
David Hedden authored
Provide the current language fallback chain to new instance of LanguageAspect. So it can be used in PageRepository to let extbase records use the fallback chain. Resolves: #91781 Releases: main, 12.4 Change-Id: I291c73c05772533919a7bf6beaa767d156bc9d74 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83981 Tested-by:
Andreas Kienast <a.fernandez@scripting-base.de> Tested-by:
-
Stefan Bürk authored
Wizards not marked as done, but not having updates, display a notice `No changes applied, marking wizard as done.` when using the CLI command `bin/typo3 upgrade:run <wizard-identifier>` and return an error exit code. This also happen when not specifying a single wizard, but running all available ones. The GUI based upgrade execution behaves different, as it does not return an exit code and does not display information about `no changes applied`, like its CLI counterpart. This change: * Adds new concrete exception classes for the `upgrade:run` command. * Modifies `UpgradeWizardCommand::getUpgradeWizard()` to throw now one of the new concrete exception or return a wizard instance to run. * Skips available upgrade wizards not eligible to be executed. Resolves: #103447 Releases: main, 12.4 Change-Id: Ib6eab07919bb2f6e11bac774ecfd333572c2205f Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83979 Tested-by:
Stefan Bürk <stefan@buerk.tech> Tested-by:
-
Stefan Bürk authored
TYPO3 provides the ability to configure different error handlers for specific (or all) error codes, as well as the error handler type to use (core handler or custom). The `\TYPO3\CMS\Core\Error\PageErrorHandler\PageContentErrorHandler` allows to specify a target using the link handler. The default config allows to define `LinkService::TYPE_PAGE` and `LinkService::TYPE_URL`. After implementing and stabilizing the TYPO3 sub-request feature, this particular error handler has been refactored to use an internal sub-request to resolve the error page content: An external URL not matching the same instance or having a page unavailable within the TYPO3 instance will not return any content. This change modifies the `PageContentErrorHandler` to send a Guzzle request for an external URL instead of using internal sub-requests, even if it would access the originating instance again. A limitation is that the requested URL **must** return a HTTP status-code 200. A custom request header is attached to this request. It is then checked to avoid recurring errors or loops in the page-resolving workflow. Resolves: #103399 Related: #98396 Related: #94402 Releases: main, 12.4 Change-Id: If09158abd2aa9246bcb7a4fa41a0ad6e4a0f942c Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83978 Tested-by:
-
- Apr 23, 2024
-
-
Oliver Bartsch authored
To be consistent with the other listings in the backend is the root page title of a site in the site configuration module now linked, allowing to edit the corresponding site configuration. Additionally does the link as well as the edit button now use a dedicated label instead of a generic one. Resolves: #103695 Releases: main, 12.4 Change-Id: Ice46e82587cdc36daa6275f6eec2c46909e8a071 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83976 Tested-by:
-
Oliver Bartsch authored
Since the button to create a new site configuration is already descriptive the non-translated title attribute is removed. Resolves: #103696 Releases: main, 12.4 Change-Id: I28c8945f9b2b8fba29461dbd8033c52eacc1f035 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83975 Reviewed-by:
-
Jasmina Ließmann authored
If no record was found in "Workspaces" backend module, an information is displayed. This text is now displayed in the typical infobox style instead of with specific Bootstrap styling classes. Resolves: #103714 Releases: main, 12.4 Change-Id: I78637defba53cebdfc659b2414c59b4f4dbb7bbb Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83977 Reviewed-by:
Jasmina Ließmann <minapokhalo+typo3@gmail.com> Tested-by:
-
Daniel Siepmann authored
The existing solution using `array_diff` did not work, as it only respects values, not a combination with keys. Given an existing record with some empty values would prevent unsetting another value with an empty. The same would apply for setting another value to an existing value. That's why we change to `array_diff_assoc`, which handles proper diff of value in combination with key. Resolves: #103708 Releases: main, 12.4 Change-Id: I36adbc8279e5160c33e8feffcd8741c30c667adc Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/84012 Tested-by:
Christian Kuhn <lolli@schwarzbu.ch> Tested-by:
-
Andreas Nedbal authored
In the current frontend tooling, we invoke Rollup twice: * via grunt-rollup to build library modules * via the command line to build everything related to CKEditor This commit gets rid of the first way and moves the library compilation to a JS-backed configuration as well, so we can get rid of a grunt package and also unify the tooling a bit. We also make use of @rollup/plugin-terser instead of the deprecated rollup-plugin-terser module that was used in ckeditor build before. To match the new behaviour, the "exec:ckeditor" task also was renamed to "exec:rollup". Executed commands: npm remove grunt-rollup npm remove rollup-plugin-terser npm install @rollup/plugin-terser --save-dev npm run build Resolves: #103686 Releases: main, 12.4 Change-Id: I8057dcceae2d039c637c6db68cf51b3ef1f12406 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/84005 Tested-by:Benjamin Franzke <ben@bnf.dev> Reviewed-by:
-
Ayke Halder authored
There is a checkbox in page module to show hidden content elements. If this checkbox is not selected the hidden content elements must be hidden. Additionally hidden elements must not be accessible/focusable by keyboard. Resolves: #103626 Releases: main, 12.4 Change-Id: Id8f4b98938ae27e3bd3d3e7903c3f50588d554b8 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83974 Tested-by:
-
Oliver Hader authored
Given a static route argument mapper is used - for instance one that allows values in the range of 1 to 100 - then generating an URL from a route with an out of scope `&value=5000` was still generating an URL. However, HTTP request to that URL would result in an 404 error. This change skips routes using out of scope values during the URL generation process to avoid pointing to invalid resources. Resolves: #103400 Releases: main, 12.4, 11.5 Change-Id: I11e12f29e294ec86bec948d3a922d4a56b231771 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/84002 Tested-by:
Oliver Hader <oliver.hader@typo3.org> Reviewed-by:
-
Andreas Kienast authored
Executed commands: npm install --save chart.js@^4.4.2 npm run build Resolves: #103652 Releases: main, 12.4 Change-Id: Iddbfb5be5f998c7b9d693dac94fee88a1f242edf Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83996 Tested-by: -
Andreas Kienast authored
Executed commands: npm install --save-dev \ @typescript-eslint/eslint-plugin@^7.7.0 \ @typescript-eslint/parser@^7.7.0 \ @typescript-eslint/typescript-estree@^7.7.0 \ eslint-plugin-wc@^2.1.0 \ sass@^1.75.0 \ sharp@^0.33.3 \ terser@^5.30.3 \ typescript@^5.4.5 npm run build Resolves: #103653 Releases: main, 12.4 Change-Id: I96bb9e96d12182adc33b5fafb52d53a4fecf6806 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83997 Reviewed-by: -
Andreas Kienast authored
The "No results found" alert is rendered in a flex-row container, which caused a rendering bug where the alert box was stretched to a height of 100%, but its width is limited to its content. Use flex children modifiers to ensure correct positioning. Resolves: #103662 Releases: main, 12.4 Change-Id: I5b17950ca381fda841cd3a901d9f5e1a70e5562d Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83972 Tested-by:
-
Oliver Bartsch authored
To allow the use of "empty" attributes, such as `download` in the CKeditor, the RTEHtmlParser does now keep such attributes while processing. Resolves: #103698 Releases: main, 12.4 Change-Id: I331066c0abaa604ba68c19f0013b5a077fe8f5f4 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83973 Tested-by:
-
Nikita Hovratov authored
It is possible that TCA items array is not set in DataHandler if 1. items is defined through types/*/columnsOverrides 2. DataHandler is triggered with context of another record type (e.g. by a translation request) This patch ensures we are always dealing with an array for items. Also, when passing it as parameter to `ItemProcessingService->getProcessingItems`. Resolves: #103472 Releases: main, 12.4 Change-Id: I364841982b7c2913e339bf5a504d811b3e050049 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83899 Tested-by:
Nikita Hovratov <nikita.h@live.de> Reviewed-by:
-
- Apr 22, 2024
-
-
Sybille Peters authored
The TSconfig configuration option mod.linkvalidator.searchFields is documented in the documentation. Previously, it was only possible to check within fields, if at least one softref was configured. This was changed to include fields with the type "link" (see LinkAnalyzer class). This change updates the documentation to reflect that. Resolves: #103477 Related: #83835 Releases: main, 12.4 Change-Id: I83ceafca206b2b1821e622bc36867eb13a9a5984 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83955 Tested-by:
-
Andreas Kienast authored
Executed commands: npm install --save \ lit@^3.1.3 \ lit-element@^4.0.5 \ lit-html@^3.1.3 npm run build Resolves: #103654 Releases: main, 12.4 Change-Id: Id250be2076f29d095cffed34bffe3f882feb223d Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83952 Tested-by: -
Willi Wehmeier authored
Add accessible aria-labels to the record icon located in the docheader component on the upper right. Additionally the record icon on the bottom right below the edit record form has got an aria-label now. Resolves: #103636 Releases: main, 12.4 Change-Id: I02a473e954f74c1b9fe8da765ca562a8a4ded290 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83900 Tested-by:
-
- Apr 21, 2024
-
-
Chris Müller authored
When providing a custom submodule for the admin panel with JavaScript, the script tag does not have a nonce attribute attached. The $attributes argument of the getAdditionalResourcesForModules() method holds a "useNonce" configuration, which is now also passed to getAdditionalResourcesForModules(), like for other calls in this method. Resolves: #103673 Releases: main, 12.4 Change-Id: I135a263734b7273c2065429185f1dd7287bd19a2 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83898 Tested-by:
-
Stefan Bürk authored
The TYPO3 system extension `redirects` now fully supports the setting of relative links as the `target` URL, which is a documented feature. Previously, an issue arose when attempting to use path links with leading slashes as URLs. This issue has been addressed in #101083, marking the initial step toward resolving the problem of relative target links. Additionally, the fix now allows for linking to arbitrary resources, even those that do not currently exist in the File Abstraction Layer (FAL). The class `TYPO3\CMS\Core\LinkHandling\LegacyLinkNotationConverter` has been changed to limit the handling of arbitrary resources to the smallest scope to reuse existing code and to use them as `LinkService::TYPE_URL` links. The superflous outer catch block has been removed to remove dead code. In addition, a test for coverage has been added that covers DataHandler and the lower LinkHandling layer. Resolves: #103387 Related: #101083 Releases: main, 12.4 Change-Id: I51cad29e5551824a0ad8d4528aa43cff7e0f9da7 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83946 Tested-by:
-
- Apr 20, 2024
-
-
Chris Müller authored
The main key is not "RTE" in YAML, but "editor". Resolves: #103678 Releases: main, 12.4 Change-Id: I3c09ac3f0071be70ba3fb6c130653b8b75dc98b4 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83897 Tested-by:
Benni Mack <benni@typo3.org> Tested-by:
-
- Apr 19, 2024
-
-
Chris Müller authored
This patch streamlines some spellings, break long code lines, so so they are visible without horizontal scrolling and adds more context to the RTE configuration examples. Resolves: #103677 Releases: main, 12.4 Change-Id: I2366888a29b90ee59d76f1b8f6876eb600b730d7 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83941 Tested-by:
-
Andreas Kienast authored
This commit removes the import of `@typo3/backend/tabs.js` in the "New Content Element" wizard for two reasons: 1. the module is not required in this scope 2. the import is apparently defunct, no request is made for that file Resolves: #103684 Releases: main, 12.4 Change-Id: I1f9af422f6c2b14590b96a058bc2ed088d0853d0 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83896 Reviewed-by:
-
Oliver Bartsch authored
When resolving the first and last inline elements in the `FilesControlContainer` and `InlineControlContainer`, it is now properly checked, whether any children exist at all, preventing possible type errors. Additionally a fallback is added to prevent further warnings in the containers while trying to iterate over the children. Resolves: #103646 Releases: main, 12.4 Change-Id: I6f46ee5dc6d5553f3bd43c12447ef2390e1f941f Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83895 Tested-by:
-
Oliver Bartsch authored
All pages rendered in the first column of the Localization Overview now display the correct record icon, which is wrapped with the context menu. Resolves: #103587 Releases: main, 12.4 Change-Id: I51d2ce07b91a47d01351f7c4fabac4868286e92d Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83926 Tested-by:
Sybille Peters <sypets@gmx.de> Reviewed-by:
Jörg Bösche <typo3@joergboesche.de> Tested-by:
-
Christian Rath-Ulrich authored
Add record titles to the record icon alt text. To make the alt text for record icons helpful, the title and type of the record is now added to the existing id=x. To implement this also for the pagetree, the method BackendUtility::titleAttribForPages() has got a new parameter to return the value unescaped. The JsonResponse in the TreeController already escapes the data and additionally the TemplateResult from lit also does escaping. Additionally BackendUtility::getRecordIconAltText() received a new parameter to make it possible to not escape string values. This is necessary to prevent values to get double encoded when used in the fluid context. Resolves: #102472 Releases: main, 12.4 Change-Id: I2476baccc4caf1ffaf27bbb3d5681cd53aea6052 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83877 Reviewed-by:
-
Oliver Bartsch authored
Using `options.folderTree.altElementBrowserMountPoints` it's possible to add additional mount points to the file / folder tree. Adding such entries does now properly work by adding the missing `identifier` key, which is required to add those entries to a storage in `StoragePermissionAspect`. Additionally, null coalescing operators are added at two places to prevent undefined array key errors. Resolves: #103609 Releases: main, 12.4 Change-Id: I275f51b5bb2d2a1583393a9e782366092d4f558a Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83925 Tested-by:
Georg Ringer <georg.ringer@gmail.com> Reviewed-by:
-
- Apr 18, 2024
-
-
Christian Weiske authored
Feature #89650 added support for newlines in TCA field descriptions, but palette descriptions were forgotten. This patch makes newlines in palette descriptions visible in the same way. Resolves: #103659 Related: #89650 Releases: main, 12.4 Change-Id: Ib505f476eb0f3efd0bf75bee7550b9e203ccda2b Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83894 Tested-by:
-
Christian Kuhn authored
Calculation of reference index for workspace relations has a nasty bug in getListOfActiveWorkspaces(): It always returns only the first workspace id when there are multiple. This isn't good in v12 since it creates broken or missing index entries with MM. It is worse in v13, since refactorings of the class use the method for some additional operations. The patch fixes the broken helper method. Resolves: #103655 Related: #96067 Related: #103661 Releases: main, 12.4 Change-Id: I989b22f9c25b22f52030c3dbce742074d2e09f97 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83920 Tested-by:
-
Christian Kuhn authored
Resolving relations in RelationHandler for MM in workspaces is a two fold process: readMM() first finds all relation "candidates". A second step then filters live MM relations from the candidate list that have a workspace overlay. In general, it *should* be possible to combine this rather expensive operation into a single query. This however requires a bigger rewrite of the class, which absoletly needs to happen at some point, especially for performance considerations. There is however a significant bug the patch aims to solve now: The result set of MM relations tends to return relations not only of the workspace-uid that has been requested, but relations of other workspaces as well, due to an insufficient query with improper filter logic in the second part of the process. This patch is important for a bug fix within ReferenceIndex, which - when fixed - triggers the currently broken RelationHandler relation calculation. Resolves: #103661 Related: #103655 Releases: main, 12.4 Change-Id: If42c138321f0043bf9d8c363f0e20e61d649e1e1 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83918 Tested-by:
-
Andreas Kienast authored
Executed commands: npm install --save-dev flag-icons@^7.2.1 npm run build Resolves: #103656 Releases: main, 12.4 Change-Id: Ibebf001a8867104d616777adf47b7c0edaccbf9d Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83893 Tested-by: -
Nikita Hovratov authored
In case of CLI context, normalized params are not available and as they are nullable, a guard condition is required before accessing methods. Note: The access via request object was added in #100047 along with a deprecation of using the ip() function in TSConfig. However, there is a special case, where plain TypoScript is loaded even in CLI context. For example this can happen when EXT:form listens to FlexForm data structure parsing event. Then it needs information from TypoScript on how to manipulate FlexForm based on YAML definitions. Resolves: #103644 Related: #100047 Releases: main, 12.4 Change-Id: Id2d166fd11794db9b7a8102677e5b9305fe18c59 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83892 Tested-by:
-
- Apr 17, 2024
-
-
Rafal Jania authored
Resolves: #103596 Related: #97548 Releases: main, 12.4 Change-Id: I2384d223202af1575a93497f81e6a69599ebbc3c Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83873 Tested-by:
-
Christian Kuhn authored
When a TCA table has no 'ctrl' 'tstamp' field defined, the workspace module crashes with an undefined array access for records of this table. Sanitize this. Resolves: #103650 Releases: main, 12.4 Change-Id: I3f8c6a1788ec2f9e8f03eddaf9c2ed4df5d737e2 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83849 Tested-by:
-
Nikita Hovratov authored
The $event parameter is now added and a superfluous parenthesis is removed in the condition. Also, the $tableName variable is used now. Resolves: #103645 Related: #98375 Releases: main, 12.4 Change-Id: I8f827ac7f6941ab03a6f5c67e9fb3525d1a79868 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83856 Reviewed-by:
Oliver Klee <typo3-coding@oliverklee.de> Tested-by:
-
- Apr 16, 2024
-
-
Benjamin Kott authored
Resolves: #103638 Releases: main, 12.4 Change-Id: Ib5880f2ea62c8d9b54b7db6f6488fc6d5a479827 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83855 Tested-by:
-
Willi Wehmeier authored
Add proper focus styles to the nav links in the Link Browser modal window. Resolves: #103631 Releases: main, 12.4 Change-Id: I0abb597124e335c3955c73438ac3cd2f0dad93bf Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83854 Tested-by:
-
Christian Rath-Ulrich authored
To trigger a search, usually an icon-only button is placed next to the search input field. To improve the accessibility of these icon-only buttons, a visually hidden label is added. Resolves: #102410 Releases: main, 12.4 Change-Id: Ifd7faee3d4c727dd2e86ad7304e19f970b61699e Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83835 Reviewed-by:
Willi Wehmeier <wwwehmeier@gmail.com> Tested-by:
-
Lina Wolf authored
Remove excess bracket Releases: main, 12.4 Resolves: #103635 Change-Id: Ie9ea1f03457177eeb6be7abb788c05bd2bf64e8c Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83853 Tested-by:
-
