- Nov 10, 2021
-
-
Oliver Bartsch authored
The contents of the toolbar dropdown menus for bookmarks and open documents are dynamic. Having e.g. a lot bookmarks previously lead to the most recent bookmarks might be cut of at the bottom of the screen. This is now fixed by using a maximum height and allowing vertical scroll. Resolves: #94641 Releases: master Change-Id: I50b799c628e51fbcb8b59ad5eb64d369647c27fe Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72140 Tested-by:
Andreas Fernandez <a.fernandez@scripting-base.de> Tested-by:
core-ci <typo3@b13.com> Tested-by:
Benni Mack <benni@typo3.org> Tested-by:
Oliver Bartsch <bo@cedev.de> Reviewed-by:
-
Oliver Bartsch authored
Resolves: #95943 Releases: master Change-Id: Ibf5c2c9fe776ab3111c172085335541970a41de7 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72139 Tested-by:
Stefan Bürk <stefan@buerk.tech> Tested-by:
Oliver Klee <typo3-coding@oliverklee.de> Reviewed-by:
-
Benni Mack authored
* EXT:belog * EXT:beuser * EXT:dashboard * EXT:extensionmanager Resolves: #94256 Releases: master Change-Id: Iee778a0a2a0ec3b92453762ea285aa7825e3b8c9 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72040 Tested-by:
Christian Kuhn <lolli@schwarzbu.ch> Tested-by:
-
Oliver Bartsch authored
Additionally, a change in behaviour from #95519 is fixed, while keeping the fallbacks in place. Resolves: #95945 Releases: master Change-Id: I51c630480f6a9781aad02a9210dd143f8abc2d12 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72143 Tested-by:
-
Andreas Fernandez authored
Resolves: #95946 Releases: master Change-Id: Ide10cf2722af3f7cc7b8e4e8830159f7de3510be Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72142 Tested-by:
Simon Gilli <typo3@gilbertsoft.org> Tested-by:
-
Chris Müller authored
Resolves: #95925 Releases: master Change-Id: I125fcb5e60561646ee8398b0536106a2d5c828ed Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72103 Tested-by:
Nikita Hovratov <nikita.h@live.de> Tested-by:
-
Chris Müller authored
Resolves: #95926 Releases: master Change-Id: I67de7575d4c6b75062cb21e558402f15dfeb70a7 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72104 Tested-by:
-
Alexander Stehlik authored
Prevent undefined array key access in saveAdditionalFields() method in ExecuteSchedulableCommandAdditionalFieldProvider. When the command line task is initially saved the arguments array is not submitted. Resolves: #95939 Releases: master Change-Id: Iee2997330a7847659914d9083a228bec87961cae Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72136 Tested-by:
Wouter Wolters <typo3@wouterwolters.nl> Tested-by:
-
Stefan Bürk authored
GeneralUtility::implodeAttributes() calls htmlspecialchars() on the array values to sanitize them. The values may be null instead of a string, thus feeding null to htmlspecialchars() emits a E_DEPRECATED PHP 8.1 error. Sanitize call. Resolves: #95933 Releases: master Change-Id: I57f1cdf2eb2e818283f5e1c1305ce4d2f3b9d440 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72110 Tested-by:
-
Stefan Bürk authored
CropViewHelper::renderStatic() calls ContentObjectRenderer::cropHTML() with result from children closure, which may be null. Feeding null to preg_split(null) emits a E_DEPRECATED PHP 8.1 error. This patch casts the value to string in the ViewHelper, but also add a string sanitize to the cropHTML() method to avoid this the E_DEPRECATED error. Added @todo to add strict types to the signature in v12. Resolves: #95934 Releases: master Change-Id: I2882163bda9bab9012241f11daf65a0a3191041d Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72111 Tested-by:
-
Stefan Bürk authored
TypoScriptTemplateInformationModuleFunctionController::main() calls trim() and explode() with array values which may be null. Feeding null for the string arguments of these functions emits a E_DEPRECATED PHP 8.1 error. Sanitize calls. Resolves: #95936 Releases: master Change-Id: I03d9f5db9e0688c0de21db1305fce431804d1d26 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72133 Tested-by:
-
Stefan Bürk authored
ExtendedTemplateService::ext_regObjectPositions() calls explode() with $constants parameter which may be null. Feeding null to explode(null) emits a E_DEPRECATED PHP 8.1 error. ext_regObjectPositions() gets called from method initialize_editor() in TypoScriptTemplateConstantEditorModuleFunctionController and may feed null as constants argument instead of string value. This patch sanitizes that in initialize_editor() with a type cast. ExtendedTemplateService is marked as @internal, and as there is only one call to ext_regObjectPositions() in the core this patch changes signature of ext_regObjectPositions() to strict types. Resolves: #95935 Releases: master Change-Id: I41cd70782163141482600eb439a47c894b5c965f Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72132 Tested-by:
-
Nikita Hovratov authored
TCA shares same keys across different types, and so does "items" with type check, radio and select. The value is usually set for the array key "1". But type check is special in that regard: the value is stored as a bitset. This means, the value is completely unused and shall not be provided for checkboxes. 1. All occurrences of "items => ['1' => '']" removed. 2. When checkboxes only have 1 item and no configuration whatsoever, "items" can be omitted. These constructs are gone now: 'items' => [ [ 0 => '', 1 => '', ], ] Resolves: #95876 Releases: master Change-Id: I3deb96498e3afb8823cd6397e57e268c68774276 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72056 Tested-by: -
Stefan Bürk authored
InputTextElement::render() calls htmlspecialchars() to output the itemValue. $itemValue can be null null instead of a string at this place and feeding null to htmlspecialchars() emits a E_DEPRECATED PHP 8.1 error. Sanitize call. Resolves: #95929 Releases: master Change-Id: I96eeb6fb0b19e5f751b68e835a59726c7a4657b6 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72106 Tested-by:
-
Stefan Bürk authored
DatabaseRecordList::renderListRow() calls linkUrlMail() with eventually null as second argument, thus invoking parse_url(null) which emits a E_DEPRECATED PHP 8.1 error. The patch sanitizes call in renderListRow(), adds type casts in linkUrlMail() and a @todo to change signature in v12. Resolves: #95937 Releases: master Change-Id: I78c0498cf5e2aa8de70576300a616ef1328d6f66 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72134 Tested-by:
-
Oliver Hader authored
To decrease the amount of inline JavaScript that has been generated by some PHP process assignments and invocations are declared in a more strict way. This would allow to make use of strict content security policy denying invocation of unsafe-inline scripts. Resolves: #91786 Releases: master Change-Id: I89384d661ebd35a5fda10f9587a7f41db4f587aa Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/64123 Tested-by:
Torben Hansen <derhansen@gmail.com> Tested-by:
Oliver Hader <oliver.hader@typo3.org> Reviewed-by:
-
Stefan Bürk authored
TemplateService::render() calls substr_count() on a array field retrieved from the database, which may be null instead of a string. Feeding null to substr_count() emits a E_DEPRECATED PHP 8.1 error. Sanitize call. Resolves: #95932 Releases: master Change-Id: I1429d0f49356922208acf9ccb2ab8b8bbd53043c Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72109 Tested-by:
-
Stefan Bürk authored
TextElement::render() calls strlen() to determine length of form elemet item value. It may be that item value is null instead of a string at this place and feed null to strlen() emits a E_DEPRECATED PHP 8.1 error. Sanitize call. Resolves: #95930 Releases: master Change-Id: Ib7f073d59e4bbc39fb070d86bbfe3a5b69d3bef8 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72107 Tested-by:
-
Stefan Bürk authored
TextElement::render() calls htmlspecialchars() to output the item value. It may be that $itemValue is null instead of a string at this place and feed null to htmlspecialchars() emits a E_DEPRECATED PHP 8.1 error. Sanitize call. Resolves: #95931 Releases: master Change-Id: I0b2066e7161b2e12e3a4b39d3f9669938a04c9bb Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72108 Tested-by:
-
- Nov 09, 2021
-
-
Nikita Hovratov authored
Add core and workspace functional test for TCA option 'localizeReferencesAtParentLocalization' with type=group to nail current state. Resolves: #94570 Related: #41713 Releases: master Change-Id: If349614306118405a2721d11d074eb2c1d0ee13e Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/69904 Tested-by:
-
Florian Rival authored
When using a text content element with a table inside bodytext, you will have a PHP warning: Undefined array key "1." in typo3/sysext/frontend/Classes/ContentObject/ContentObjectRenderer.php This was only a notice with PHP v7 but it's an error with PHP v8. This bugfix corrects the usage of undefined array keys. Resolves: #95858 Releases: master Change-Id: Ie70fb6bdd66eee895e0c290a9bcc662c9f734d2e Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72058 Tested-by:
-
Chris Müller authored
Resolves: #95924 Releases: master Change-Id: Id341014507623e1f1aa07721d81d0667b7cdef04 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72101 Tested-by:
Georg Ringer <georg.ringer@gmail.com> Tested-by:
-
Marcel Schüßler authored
Cast 'tokenValue' to string as it can happen to be an integer. Releases: master Resolves: #95887 Change-Id: I2c053926a7ed24f6cc3634a8c139213de9b65f0f Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72069 Tested-by:
-
Alexander Schnitzler authored
To ease the work with phpstan and IDE's, some types in persistence related classes of extbase are now either fixed or more specified. Releases: master, 10.4 Resolves: #95885 Change-Id: I2a784bd3ffaae651dd67dd939ef13e5efcb2ead6 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72066 Tested-by:
-
Oliver Bartsch authored
It's possible to query for deleted records in DB Check "Full search". Such records can then be restored only or additionally restored and directly edited. The icon for the latter action was broken, since a non existing icon identifier was used. This is now fixed, using the correct icon identifier. Resolves: #95923 Releases: master, 10.4 Change-Id: I7da25198efe731a9ccba9e673c8c55e42760b233 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72098 Tested-by:
Jochen <rothjochen@gmail.com> Tested-by:
-
Oliver Bartsch authored
In case only one localization mode is available, e.g. because a localization mode was already defined for the translated page or only one mode is allowed by configuration, the selection step is superfluous and therefore now skipped. Resolves: #95909 Releases: master Change-Id: I4a842b0758c9c95744f31f8074a09a1c29f53af2 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72079 Tested-by:
-
Alexander Schnitzler authored
The method signature of Query::locicalAnd() is very misleading because it only states there is one method argument $constraint1. Actually, the method does accept any number of arguments, evaluated via func_get_args(). It also accepts any number of arguments if $constraint1 is an array of constraints. This leads to multiple issues when working with phpstan and also when simply using an IDE which does not understand the internal magic. The goal is to state, that logicalAnd() needs at least two constraints but also accepts any number of further constraints. Since that requires a breaking change, the method signature can only be fixed partly for now. $constraint1 remains as is, a 2nd argument $constraint2 is introduced, null by default to make it optional and a variadic argument ...$furtherConstraints is also introduced. This approach keeps the bc promise but also lays the foundation for further changes in the next version. Releases: master, 10.4 Resolves: #95889 Change-Id: Ife1be32d99371b82a84b6f6d800a58d242c8ff0d Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72063 Tested-by:
-
Oliver Bartsch authored
In prior versions, creating a new page via the context menu inserted the new page "inside". This however changed to "after" in v8, leading to misbehavior, especially for non-admins, as they are usually not allowed to create pages on pid=0. This is now fixed by restoring the previous behavior of inserting pages "inside" instead of "after". Resolves: #81405 Releases: master, 10.4 Change-Id: I6a941c3e015189a471d0c136c81f1e5698426dbb Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72093 Tested-by:
Markus Klein <markus.klein@typo3.org> Tested-by:
-
Nikita Hovratov authored
The identifier from the linkHandler page TSconfig was formerly used as the reference table name "ref_table". This identifier is free to choose and shouldn't be interpreted as a name of a table. In order to resolve the correct table name, which is stated in the configuration, the page id of the reference record is now used. This allows the TypolinkSoftReferenceParser to read the according page TSconfig and to set the correct table name. Resolves: #95380 Releases: master Change-Id: I28817a7267a107df27828d99f7e0cf2eeaaf2de1 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/71340 Tested-by:
-
Nikita Hovratov authored
Resolves: #95917 Releases: master Change-Id: I0a02c5f9ac49b4523fe754d5a0a1e86b6a57eb34 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72092 Tested-by:
-
Stefan Bürk authored
GeneralUtility::fixed_lgd_cs() may be called with null instead of a string as first argument, which is passed to mb_strlen(). Feeding null to mb_strlen() emits a E_DEPRECATED PHP 8.1 error. This patch sanitizes the call in BackendUtillity and further ensures that the first parameter $string is a string (until strict type hints can be added). Resolves: #95913 Releases: master Change-Id: Ie3bb44d4d47d4673d2274f10dca80f860d494c32 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72085 Tested-by:
Sybille Peters <sypets@gmx.de> Reviewed-by:
-
Oliver Bartsch authored
The pagination feature in DB Check "Full search" did not work, since the defined offset was not added to the query. This is now fixed by checking for a comma separated string, which would therefore contain both an offset and the limit. In case both values are given, they are now properly added to the query. Resolves: #95922 Releases: master, 10.4 Change-Id: Id461b98a9818c3b55b34f93041a9c0950e68f634 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72095 Tested-by:
-
Stefan Bürk authored
DatabaseRecordList::renderListRow() used the return value of called BackendUtility::getProcessedValueExtra() to feed it into php internal htmlspecialchars. As result can be null instead of a string this emits a E_DEPRECATED PHP 8.1 error. Sanitize call. Resolves: #95916 Releases: master Change-Id: If9fde8e6ab96ab8083d60e1a67fc37160859a7e3 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72087 Reviewed-by:
-
Christian Kuhn authored
Latest 3.141 again leads to trouble with timeout exceptions. 4.0 seems to behave better ... let's raise again to see if that improves the situation. Change-Id: If04b85cf544c1f95d0d65bc95657d5dbf852dacc Resolves: #95920 Related: #95883 Releases: master Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72094 Tested-by:
-
Stefan Bürk authored
Add PHP 8.1 attribute #[\ReturnTypeWillChange] with todo to set correct return type as breaking change in v12 to satisfy PHP 8.1 strict return on SPL classes. Resolves: #95912 Releases: master Change-Id: Id92292d7dacec8c99022a719fe4e407e848f2f81 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72084 Tested-by:
-
Oliver Bartsch authored
Since #91085, sending a remote request in workspaces, e.g. sending records to another stage, is visualized by a progress bar. The configured selector for the progress bar however differs between the review module and the preview module and therefore led to a TypeError. This is now fixed by using the correct selector depending on the current module. Resolves: #95661 Related: #91085 Releases: master Change-Id: Ibeb84879e459ac8e94a0e248d77b2d974b94710c Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72080 Tested-by:
-
Simon Gilli authored
When installing TYPO3 and Extensions into the vendor folder, no language files can be loaded anymore because of an implementation depending on hard coded Extension directories. A new method is implemented in the PackageManager to extract a package key from a path which solves the issue and makes the old implementation superfluous. The code is left there however to not interfere with custom implementations (despite these classes are marked internal). LLL references not using an EXT: path, but a file system path will "work" exactly like before. This should rather be deprecated later on. In fact the whole language file parsing deserves a complete overhaul, which should be tackled in subsequent changes for a new major TYPO3 version. Resolves: #95839 Releases: master Change-Id: I5ca698d445ae6c5125ee42787fd7274dc8d27a38 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/71938 Tested-by:
Helmut Hummel <typo3@helhum.io> Tested-by:
-
Stefan Bürk authored
Set void return type for @internal class ExtensionXmlParser to satisfy PHP 8.1 strict return on SPL classes. Resolves: #95911 Releases: master Change-Id: If112aed90f6e9d6b411d1d66086dab142b68748a Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72083 Tested-by:
-
Stefan Bürk authored
Nl2brViewHelper::renderStatic() calls nl2br with the return of the called $renderChildrenClosure which may return null. Feeding null to nl2br() emits a E_DEPRECATED PHP 8.1 error. Sanatize the call. Resolves: #95915 Releases: master Change-Id: I2313811b42836ec21ba7efcee4c0a6ed9148fcab Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72086 Tested-by:
-
- Nov 08, 2021
-
-
Chris Müller authored
In most module configurations icons with a distinct path are used. This is not necessary, as the module icons are already available from the TYPO3.Icons package and therefore the icon identifier can be used. For EXT:core this was already done. The previously referenced icons are not used anymore and will be removed by a separated patch for v12 once v11 is branched. Resolves: #95888 Releases: master Change-Id: Ic113fc9f40ce80059b65e1290afe70673ba06a33 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72070 Tested-by:
-
