- Apr 22, 2024
-
-
Benjamin Franzke authored
The description has been forgotten to be added in #103437. Releases: main Resolves: #103630 Related: #103437 Change-Id: Ie254e6bd8b0fc35ba2a2ea2ca03b3f0383317c47 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83949 Reviewed-by:
Simon Schaufelberger <simonschaufi+typo3@gmail.com> Tested-by:
Benjamin Franzke <ben@bnf.dev> Tested-by:
core-ci <typo3@b13.com> Tested-by:
Andreas Kienast <a.fernandez@scripting-base.de> Reviewed-by:
Oliver Bartsch <bo@cedev.de> Tested-by:
-
Andreas Kienast authored
The component `@typo3/backend/new-content-element-wizard` is primarily used to render a wizard-like UI for selecting a content element for creation. To make the component usable in other places as well, it get's renamed to `@typo3/backend/new-record-wizard`. Resolves: #103683 Releases: main Change-Id: I9ad19d95fd0c8aa1efb60c335726dac57eb7c54c Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83936 Reviewed-by:
Georg Ringer <georg.ringer@gmail.com> Reviewed-by:
-
Benjamin Franzke authored
TypoScript constants expressions have been extended to support a null coalescing operator (`??`) as a way for providing a migration path from a legacy constant name to a newer name, while providing full backwards compatibility for the legacy constant name, if still defined. Resolves: #103671 Related: #97816 Releases: main Change-Id: I959b07deb40a9a5582599d5c3eee96da831e8149 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83912 Tested-by:
Benni Mack <benni@typo3.org> Tested-by:
Stefan Bürk <stefan@buerk.tech> Reviewed-by:
Torben Hansen <derhansen@gmail.com> Tested-by:
Garvin Hicking <gh@faktor-e.de> Tested-by:
Christian Kuhn <lolli@schwarzbu.ch> 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/+/83927 Tested-by:
-
Oliver Bartsch authored
This extends the PAGEVIEW content object to provide not only the page record, but the full Page Information object, which among others contains the page record, the rootline and additionally also the Page Layout object. Resolves: #103570 Releases: main Change-Id: I904b91f7427e0bcd27c74f3fb3a92a7e320dff8b Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83708 Reviewed-by:
Lisa Kreitz <lisamariakreitz@web.de> 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/+/83444 Reviewed-by:
Markus Klein <markus.klein@typo3.org> Tested-by:
-
- Apr 20, 2024
-
-
Simon Schaufelberger authored
Place the search icon on the right side of the input field. Resolves: #103404 Releases: main Change-Id: I7a0ed1ab77273ad300b406b8c356b8e7f7851474 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83571 Reviewed-by:
-
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/+/83931 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/+/83930 Tested-by:
Jörg Bösche <typo3@joergboesche.de> Reviewed-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/+/83938 Reviewed-by:
Andreas Nedbal <andy@pixelde.su> 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/+/83844 Reviewed-by:
Marcin Sągol <marcin@soee.pl> 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/+/83913 Tested-by:
Sybille Peters <sypets@gmx.de> Reviewed-by:
Jasmina Ließmann <minapokhalo+typo3@gmail.com> Reviewed-by:
-
Torben Hansen authored
Initializing the variable `$nativeDateTimeFieldEmptyValue` to an empty string in the function `checkValueForDatetime` is superfluous, since the variable is initialized again in the following if-condition and only used inside the condition code. The initialization to an empty string can therefore safely be removed. Resolves: #103643 Releases: main Change-Id: Ifd845228dfb2f5eec2f75c4232602921d9dc964b Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83840 Reviewed-by:
-
Stefan Bürk authored
Doctrine DBAL 4.x has been introduced with #102875 and required adoption of breaking changes - which had been done. This included changes of children structures in classes, for example `\Doctrine\DBAL\Schema\Table` no longer returns associative array for `getColumns()` and `getIndexes()`. TYPO3 builds a virtual database schema reading extension `ext_tables.sql` files combined with TCA based default schema definitions to provide the Database Analyzer flow. The collection process build up partly table structures, and having tables multiple time which needs to be merged at some point to get the final virtual structure. With change #103015 this merge process has been moved to an earlier point, overseeing that columns and indexes are no longer associative arrays and using `array_merge()` is no longer suitable for them. In cases, where an extensions change or overrides the table definition of a table this merge process will throw a Doctrine DBAL `ColumnAlreadyExists` exception, because columns and indexes of multiple table parts are not correctly merged anymore. This change introduces two helper method as sidekicks for method `SchemaMigrator::mergeTableDefinitions()`, to merge table columns and indexes correctly and guard it with the added test. Note: Foreign keys constraints are still associative, and the TYPO3 does not support unique constraints at all within `ext_tables.sql` files. Resolves: #103666 Related: #103015 Related: #102875 Releases: main Change-Id: I9b5bca0f4faab198c3aba1c70d249ac08d07d2db Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83934 Tested-by:
Nikita Hovratov <nikita.h@live.de> Reviewed-by:
-
Christian Kuhn authored
There is a relatively obscure and not well known TCA config property for type='select': 'allowNonIdValues'. It can be used in scenarios with 'foreign_table' relations, where select stores the uids of selected records as CSV list: It allows additional items next to "true" relations, for instance items from a static 'items' list. To keep "true" relations and "additional static values" apart, those static values must be some string that can not be interpreted as an integer uid. This only works when select stores selections as CSV list: When using an MM table, relation uids are forced to be integers. This is documented, too. RelationHandler still handles 'allowNonIdValues' in its MM related readMM() method, which is obsolete code since that can not work. The patch clean this up and simplifies readMM() a bit along the way. Resolves: #103676 Releases: main Change-Id: I5f760f4c111ce5ae7a1cc3c9412b8900e4ac82d6 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83929 Tested-by:
-
Christian Kuhn authored
Extend the functional RootlineUtilityTest with more scenarios for resolving category MM, and a "casual" inline relation. Resolves: #103680 Releases: main Change-Id: Ic6e5f9cd604582853431be9415c0fb68b4a41b3c Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83933 Tested-by:
-
Nikita Hovratov authored
Also adjust variable name along the way. Resolves: #103679 Related: #103667 Releases: main Change-Id: I345dabbd388eaac9e5a56af6b70e249c4bac0feb Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83932 Tested-by:
-
Christian Kuhn authored
There is no point to trim() value arrays that have been created with GU::trimExplode() just the line before. Clean up the loop a bit along the way. Resolves: #103674 Releases: main Change-Id: Ia5580b292b1a44a0291468b35bc1e5ca5b2f9823 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83928 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/+/81883 Reviewed-by:
-
Nikita Hovratov authored
New Content Element Wizards are retrieved from both normal Content Types (CType) and Plugin sub-types (list_type). For the latter the bool parameter `$isPluginSubType` has to be set to `true`. Tests are added to ensure correct functionality. Resolves: #103667 Related: #102834 Releases: main Change-Id: Ib1b63b77f5c66edbde7622d4f8324fcfb82cdf53 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83923 Tested-by:
Oliver Klee <typo3-coding@oliverklee.de> Tested-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/+/83841 Tested-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/+/83915 Reviewed-by:
-
Jasmina Ließmann authored
Resolves: #103664 Releases: main Change-Id: I310eee5d304821d5f1b5ba210645f5ed2b4a85d2 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83921 Reviewed-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/+/83875 Tested-by:
-
Jasmina Ließmann authored
In the "Indexing" backend module, words can have various informations, such as frequency and flags. The flags are now displayed as badges and their labels have been updated. Resolves: #103660 Releases: main Change-Id: I73b30e5201619dc2f1cd0831a23d017c5ca7065a Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83916 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/+/83917 Reviewed-by:
-
Jasmina Ließmann authored
The "Indexing" indexed_search backend module allows picking single indexed words and adding them as keywords a pages 'keywords' field. This mixes up two different topics: Search and seo. Good seo administration should be a separate topic that involves much more than just adding some arbitrary keyword to a page. Additionally, meta keywords lost quite a bit of relevance over the years, and it seems odd the indexed_search BE module takes care of this detail. It additionally comes with zero additional explanation for editors, is hidden in detail views of the module, and removes the index of a page when used, which leads to unexpected routing within the module. The patch removes this functionality. This also fixes an issue when trying to save keywords for an index that is not a page, but something else like an image. Resolves: #103657 Resolves: #103618 Releases: main Change-Id: Ifd0e1de9c5d02082e2abb363dcd435ecb6778337 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83878 Reviewed-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/+/83876 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/+/83842 Reviewed-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/+/83740 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/+/83848 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/+/83843 Tested-by:
-
Jasmina Ließmann authored
With #92993 the table 'index_stat_search' was removed. The related field 'index_stat_search_id' in table 'index_stat_word' is left as an orphaned remnant. This patch now removes the field. Resolves: #103649 Related: #92993 Releases: main Change-Id: Idc04a6675c6b75fefd13149cd68a93a2cca3123d Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83847 Reviewed-by:
-
Christian Kuhn authored
TCA type=select has no config field 'foreign_field'. A check in RootlineUtility can be simplified a bit. Resolves: #103647 Releases: main Change-Id: Ia34a79f1797a9dfb8d9fbadb76d12fd1b2b175b6 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83845 Tested-by:
-
Jasmina Ließmann authored
The element information modal displays both field information for the current record and its references. The more fields there are in a table and the more content the specific record contains, the more the reference sections disappear from view. The three sections are now displayed as accordions to enable the user to decide for themselves which sections are relevant to them. In addition, the display of non-existent records in the reference sections has been adjusted and information about the paths/storage locations has been removed, as these can already be viewed in the record's editing mask. Resolves: #102469 Releases: main Change-Id: Ia4ba85921d7be2ab30477e8248d4fb8734f6eaaf Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/81873 Tested-by:
-
Jasmina Ließmann authored
Several minor adjustments have been made to the submodules of the “Indexing” backend module to improve the UX and accessibility. This includes, among other things, the following: * move action links to a dedicated control/button-group column * use badges for label 'Not indexed' * set unique h1 in all submodules * use correct heading levels (h2/h3) * show infobox when either no page has been selected or no results exists and unify locallang labels * add label to back-button in statisticDetailsAction * move back-button to docheader in wordDetailAction Resolves: #103593 Releases: main Change-Id: I078ba09de3c0a9e3ae000231c63b29bc203af8d1 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83733 Tested-by:
-
- Apr 16, 2024
-
-
Stefan Bürk authored
Default value support for TEXT, JSON and BLOB fields has been added with #103578 by implementing the use of default value expression for MySQL. That required to add custom normalization on data schema reads to be comparable. MySQL requires to use a single-quote to quote a single quote in a value string, and due to the expression way this needs to be properly decoded now in two steps: * Revert escape sequences in the retrieved default value * Unquote the unescaped retrieved default value JSON field defaults shows a similar issue for double quotes in the json value and can be fixed in the same way. Added test revealed, that Doctrine DBAL has an issue with double single-quotes for PostgreSQL too. To fix this the issue has been reported [1] and a pull-request provided [2]. This change ensure correct unescaping and unquoting of the retrieved column default value for TEXT, JSON and BLOB column types for MySQL connections, enriched with further tests. The extended PostreSQLSchemaManager now clones method `_getPortableTableColumnDefinition()` to incorporate the bugfix directly until a fixed Doctrine DBAL version has been released. [1] https://github.com/doctrine/dbal/issues/6357 [2] https://github.com/doctrine/dbal/pull/6358 Resolves: #103610 Related: #103578 Releases: main Change-Id: Icb39cdb8c87ae7907f84e5c38adcde4ef545ed1b Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83745 Tested-by:
-
Benjamin Kott authored
Resolves: #103638 Releases: main, 12.4 Change-Id: Ib5880f2ea62c8d9b54b7db6f6488fc6d5a479827 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/83836 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/+/83786 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/+/81800 Reviewed-by:
Michael Telgkamp <michael.telgkamp@mindscreen.de> Tested-by:
Benjamin Kott <benjamin.kott@outlook.com> Tested-by:
-
