- Nov 15, 2021
-
-
Stefan Bürk authored
During testing to activate E_ALL and PHP 8.1 testing with activated E_ALL, nightly 'composerMin' tests for PHP8.1 are failing with several E_DEPRECATED warnings, which was not discoverd in composerLock tests. Several composer packages, ex. 'symfony/finder' and 'symfony/routing', are required with different min versions than written in composer.lock file, thus downgrading these components which emits E_DEPRECATED PHP8.1 warnings in nightly composerMin tests execeuted with PHP 8.1. 'symfony/options-resolver' needs to be added directly, as this has been installed as dependency with different lower version constraints for 'friendsofphp/php-cs-fixer' and 'symfony/rate-limiter', thus we need to add it directly with the locked version as min requirement as there are no newer versions of these packages with needed min version as required version. 'bacon/bacon-qr-code' needs to be raised to avoid a float to int PHP8.1 conversion warning. This patch acts as pre-patch to activate E_ALL and PHP 8.1 testing and thus raise version of these components to the currently locked version in 'composer.lock' file, to be conservative. Used composer commands: composer req bacon/bacon-qr-code:^2.0.4 composer req bacon/bacon-qr-code:^2.0.4 \ -d typo3/sysext/core --no-update composer req masterminds/html5:^2.7.5 composer req masterminds/html5:^2.7.5 \ -d typo3/sysext/core --no-update composer req symfony/config:^5.3.4 composer req symfony/config:^5.3.4 \ -d typo3/sysext/core --no-update composer req symfony/console:^5.3.7 composer req symfony/console:^5.3.7 \ -d typo3/sysext/core --no-update composer req symfony/console:^5.3.7 \ -d typo3/sysext/redirects --no-update composer req symfony/dependency-injection:^5.3.7 composer req symfony/dependency-injection:^5.3.7 \ -d typo3/sysext/core --no-update composer req symfony/dependency-injection:^5.3.7 \ -d typo3/sysext/extbase --no-update composer req symfony/dependency-injection:^5.3.7 \ -d typo3/sysext/fluid --no-update composer req symfony/expression-language:^5.3.7 composer req symfony/expression-language:^5.3.7 \ -d typo3/sysext/core --no-update composer req symfony/expression-language:^5.3.7 \ -d typo3/sysext/form --no-update composer req symfony/filesystem:^5.3.4 composer req symfony/filesystem:^5.3.4 \ -d typo3/sysext/core --no-update composer req symfony/finder:^5.3.7 composer req symfony/finder:^5.3.7 \ -d typo3/sysext/core --no-update composer req symfony/finder:^5.3.7 \ -d typo3/sysext/install --no-update composer req symfony/http-foundation:^5.3.7 composer req symfony/http-foundation:^5.3.7 \ -d typo3/sysext/core --no-update composer req symfony/http-foundation:^5.3.7 \ -d typo3/sysext/install --no-update composer req symfony/mailer:^5.3.4 composer req symfony/mailer:^5.3.4 \ -d typo3/sysext/core --no-update composer req symfony/mime:^5.3.7 composer req symfony/mime:^5.3.7 \ -d typo3/sysext/core --no-update composer req symfony/polyfill-intl-normalizer:^1.23.0 composer req symfony/polyfill-intl-normalizer:^1.23.0 \ -d typo3/sysext/core --no-update composer req symfony/polyfill-intl-icu:^1.23.0 composer req symfony/polyfill-intl-icu:^1.23.0 \ -d typo3/sysext/core --no-update composer req symfony/polyfill-intl-idn:^1.23.0 composer req symfony/polyfill-intl-idn:^1.23.0 \ -d typo3/sysext/core --no-update composer req symfony/polyfill-mbstring:^1.23.1 composer req symfony/polyfill-mbstring:^1.23.1 \ -d typo3/sysext/core --no-update composer req symfony/polyfill-mbstring:^1.23.1 \ -d typo3/sysext/frontend --no-update composer req symfony/property-access:^5.3.7 composer req symfony/property-access:^5.3.7 \ -d typo3/sysext/extbase --no-update composer req symfony/property-info:^5.3.7 composer req symfony/property-info:^5.3.7 \ -d typo3/sysext/extbase --no-update composer req symfony/rate-limiter:^5.3.4 composer req symfony/rate-limiter:^5.3.4 \ -d typo3/sysext/core --no-update composer req symfony/routing:^5.3.7 composer req symfony/routing:^5.3.7 \ -d typo3/sysext/core --no-update composer req symfony/var-dumper:^5.3.7 composer req symfony/var-dumper:^5.3.7 \ -d typo3/sysext/adminpanel --no-update composer req symfony/yaml:^5.3.6 composer req symfony/yaml:^5.3.6 \ -d typo3/sysext/core --no-update Following are needed, but are new as direct requirements but locked: composer req symfony/options-resolver:^5.3.7 composer req symfony/options-resolver:^5.3.7 \ -d typo3/sysext/core --no-update Resolves: #95970 Releases: master Change-Id: I43441ff6ba260721026447c035d0f2a56902119b Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72166 Tested-by:
core-ci <typo3@b13.com> Tested-by:
Jochen <rothjochen@gmail.com> Tested-by:
Christian Kuhn <lolli@schwarzbu.ch> Reviewed-by:
Oliver Klee <typo3-coding@oliverklee.de> Reviewed-by:
Jochen <rothjochen@gmail.com> Reviewed-by:
Christian Kuhn <lolli@schwarzbu.ch>
-
Peter Kraume authored
Resolves: #95971 Releases: master Change-Id: I73f4d694d3a641c07c10b31c7e508f20836a955c Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72169 Tested-by:
Oliver Hader <oliver.hader@typo3.org> Tested-by:
Georg Ringer <georg.ringer@gmail.com> Reviewed-by:
Oliver Hader <oliver.hader@typo3.org> Reviewed-by:
Georg Ringer <georg.ringer@gmail.com>
-
Christian Eßl authored
There were several problems with tooltips in the page tree: - htmlspecialchars was applied twice to the tooltip string. Once in BackendUtility::titleAttribForPages, once in TreeController. - When applying double or single quotes to a page title, the tooltips in the page tree would show these as html entities. Double and single quotes are now allowed, while any potential html tags in the title will be stripped from the tooltip. Any of those characters (", ', <, >) left in the tooltip are later escaped to unicode in the json view. Resolves: #91424 Releases: master Change-Id: I53b2ad6bbd4e4131535eb573422a8cbd5d002858 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/64511 Tested-by:
Benni Mack <benni@typo3.org> Tested-by:
core-ci <typo3@b13.com> Tested-by:
Oliver Hader <oliver.hader@typo3.org> Reviewed-by:
Benni Mack <benni@typo3.org> Reviewed-by:
Oliver Hader <oliver.hader@typo3.org>
-
- Nov 13, 2021
-
-
Axel Seemann authored
If you use some mail relay servers for sending emails via SMTP, it is necessary to specify the domain name under which the emails are sent. This is especially important for the Gmail-Relay server smtp-relay.gmail.com but is not limited to this. Resolves: #93635 Releases: master Change-Id: If2fca7d02cc5c8017ae7eebcd616a165c4d9bb1b Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/69652 Tested-by:
Stefan Bürk <stefan@buerk.tech> Tested-by:
core-ci <typo3@b13.com> Tested-by:
Oliver Hader <oliver.hader@typo3.org> Reviewed-by:
Stefan Bürk <stefan@buerk.tech> Reviewed-by:
Lina Wolf <112@linawolf.de> Reviewed-by:
Guido Schmechel <guido.schmechel@brandung.de> Reviewed-by:
Oliver Hader <oliver.hader@typo3.org>
-
Nikita Hovratov authored
In #89871 (65dc0603) a fix was introduced, that forces the link generation to use the requested language of shortcuts instead of the current language of the frontend. In addition, another call to getPageOverlay was added. This is in fact not necessary, as overlaying is already done implicitly in getPageShortcut. This introduced a falsy behaviour, where shortcuts without the current language available would create a URL with the entry point of the fallback language. This can be fixed by simply using the request language from the original $menuItem variable before processing. Resolves: #94677 Related: #89871 Releases: master, 10.4 Change-Id: I5c9bd0439d821d5516a5cb58cdf53df6b63c3aab Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/70836 Tested-by:
core-ci <typo3@b13.com> Tested-by:
Stefan Bürk <stefan@buerk.tech> Tested-by:
Benni Mack <benni@typo3.org> Reviewed-by:
Stefan Bürk <stefan@buerk.tech> Reviewed-by:
Benni Mack <benni@typo3.org>
-
- Nov 12, 2021
-
-
Nikita Hovratov authored
The added tests for #89871 do not prove the correctness of the patch. They go green, without the fix being applied. This is a blocker for another regression bugfix, which depends on these tests. The issue lied in the language menu, not the normal menu. For this, a new method createLanguageMenu has been added to AbstractLocalizedPagesTestCase.php. It is now used to assure the generated links always point to the target language and not to the current one. Resolves: #95966 Related: #89871 Related: #94677 Releases: master Change-Id: I20d202c2eae58597ee59312ef0896debb2f93d39 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72162 Reviewed-by:
Oliver Klee <typo3-coding@oliverklee.de> Reviewed-by:
Stefan Bürk <stefan@buerk.tech> Reviewed-by:
Benni Mack <benni@typo3.org> Tested-by:
core-ci <typo3@b13.com> Tested-by:
Stefan Bürk <stefan@buerk.tech> Tested-by:
Benni Mack <benni@typo3.org>
-
Stefan Bürk authored
doctrine/dbal driver for postgres has been feeding null to preg_match() in some constellations, which emits E_DEPRECATED warning in PHP8.1 with E_ALL. doctrine/dbal:2.13.5 includes the bugfix for this issue and more. This patch acts as pre-patch to activate E_ALL and more PHP81 testing and thus raise the min version to ^2.13.5 to avoid these warnings and make functional tests for postgres with E_ALL happy. Used composer commands: composer req doctrine/dbal:^2.13.5 composer req doctrine/dbal:^2.13.5 -d typo3/sysext/core --no-update composer req doctrine/dbal:^2.13.5 -d typo3/sysext/install --no-update composer req doctrine/dbal:^2.13.5 -d typo3/sysext/redirects --no-update Resolves: #95968 Releases: master Change-Id: Iba44ac6ccfcf7f7286d6d4e5843e25fefd0a607a Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72161 Tested-by:
core-ci <typo3@b13.com> Tested-by:
Anja Leichsenring <aleichsenring@ab-softlab.de> Tested-by:
Nikita Hovratov <nikita.h@live.de> Tested-by:
Simon Gilli <typo3@gilbertsoft.org> Tested-by:
Benni Mack <benni@typo3.org> Reviewed-by:
Anja Leichsenring <aleichsenring@ab-softlab.de> Reviewed-by:
Nikita Hovratov <nikita.h@live.de> Reviewed-by:
Oliver Klee <typo3-coding@oliverklee.de> Reviewed-by:
Simon Gilli <typo3@gilbertsoft.org> Reviewed-by:
Benni Mack <benni@typo3.org>
-
Simon Gilli authored
Change #95839 introduced a regression that causes TranslateViewHelper to fail when an invalid extension key is specified. This patch restores the previous behavior and avoids more regressions by a new test. Resolves: #95957 Releases: master Change-Id: I98675f94cf367b43f4123bb981f60c35286ace7a Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72158 Tested-by:
core-ci <typo3@b13.com> Tested-by:
Helmut Hummel <typo3@helhum.io> Tested-by:
Nikita Hovratov <nikita.h@live.de> Tested-by:
Georg Ringer <georg.ringer@gmail.com> Reviewed-by:
Helmut Hummel <typo3@helhum.io> Reviewed-by:
Nikita Hovratov <nikita.h@live.de> Reviewed-by:
Georg Ringer <georg.ringer@gmail.com>
-
Nikita Hovratov authored
Resolves: #95960 Releases: master Change-Id: I5853d135ce20b1e9b53806a16fb734ab87c41b2b Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72159 Tested-by:
core-ci <typo3@b13.com> Tested-by:
Stefan Bürk <stefan@buerk.tech> Tested-by:
Benni Mack <benni@typo3.org> Reviewed-by:
Oliver Klee <typo3-coding@oliverklee.de> Reviewed-by:
Stefan Bürk <stefan@buerk.tech> Reviewed-by:
Benni Mack <benni@typo3.org>
-
- Nov 11, 2021
-
-
Gabe Troyan authored
Currently, the value of `USdateFormat` is disregarded because the corresponding property, `FormEngineValidation.USmode`, is set to zero regardless in `FormEngineValidation.initialize()`. This change will ensure that `USmode` will remain 1 without conflicting with core functionality that relies on `USmode` being set *before* `initialize()` is called. Resolves: #86679 Releases: master, 10.4 Related-to: #91105 Change-Id: I5584ca7fe92742e28147f9883fedf53e0556f1b1 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72034 Tested-by:
core-ci <typo3@b13.com> Tested-by:
Oliver Bartsch <bo@cedev.de> Tested-by:
Benni Mack <benni@typo3.org> Reviewed-by:
Oliver Bartsch <bo@cedev.de> Reviewed-by:
Benni Mack <benni@typo3.org>
-
Oliver Hader authored
With this change JavaScriptHandler.js is not only an immediately invoked function expression (IIFE) like before but is extended to be used as asynchronous module definition (AMD) as well. This way it can be used in internal TypeScript implementations as well. Resolves: #95953 Releases: master Change-Id: I7aa37ba707298eaa8ea151358c985a2e857cfb0c Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72153 Tested-by:
core-ci <typo3@b13.com> Tested-by:
Markus Klein <markus.klein@typo3.org> Tested-by:
Benni Mack <benni@typo3.org> Reviewed-by:
Markus Klein <markus.klein@typo3.org> Reviewed-by:
Benni Mack <benni@typo3.org>
-
Benni Mack authored
When dealing with 3rd-party authentication services, which might return other values than arrays or without UIDs. This change fixes the relevant places. Resolves: #95958 Releases: master Change-Id: Idd50e3dceea079086a98225727ef7733a9b432c8 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72157 Tested-by:
core-ci <typo3@b13.com> Tested-by:
Oliver Bartsch <bo@cedev.de> Tested-by:
Georg Ringer <georg.ringer@gmail.com> Reviewed-by:
Oliver Bartsch <bo@cedev.de> Reviewed-by:
Oliver Klee <typo3-coding@oliverklee.de> Reviewed-by:
Georg Ringer <georg.ringer@gmail.com>
-
Benni Mack authored
This reverts commit 48e4e495 as this has some side-effects with absolute paths. Change-Id: Ieb696908e5226d57eedeef7cb5fa8cd7b7ec4ea6 Reverts: #95379 Resolves: #95959 Releases: master, 10.4 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72114 Tested-by:
core-ci <typo3@b13.com> Tested-by:
Benni Mack <benni@typo3.org> Reviewed-by:
Benni Mack <benni@typo3.org>
-
Benjamin Franzke authored
Fetch services from the DI container and perform post-creations checks instead of checking class_exists and therefore exclude possible service aliases defined in DI. Releases: master Resolves: #95855 Resolves: #92910 Change-Id: Ic85485edc7604a92bdf72d147ad47fe79954500e Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72029 Tested-by:
core-ci <typo3@b13.com> Tested-by:
Benni Mack <benni@typo3.org> Tested-by:
Oliver Bartsch <bo@cedev.de> Reviewed-by:
Benni Mack <benni@typo3.org> Reviewed-by:
Oliver Bartsch <bo@cedev.de>
-
Robert Kärner authored
Two undefined array key warnings are fixed, which occurred on PHP 8 when accessing the MagicImageService. Resolves: #95942 Releases: master Change-Id: Ifc7c64a7e14b600e88154c82ae575a1c23f9f87c Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72138 Tested-by:
core-ci <typo3@b13.com> Tested-by:
physikbuddha <r.kaerner@oranto.de> Tested-by:
Benni Mack <benni@typo3.org> Tested-by:
Oliver Bartsch <bo@cedev.de> Reviewed-by:
physikbuddha <r.kaerner@oranto.de> Reviewed-by:
Benni Mack <benni@typo3.org> Reviewed-by:
Oliver Bartsch <bo@cedev.de>
-
Oliver Hader authored
Introduces new `\TYPO3\CMS\Dashboard\Widgets\JavaScriptInterface`, based on explicit `\TYPO3\CMS\Core\Page\JavaScriptModuleInstruction` model - to reduce (and avoid) inline JavaScript in the TYPO3 backend. Resolves: #95873 Releases: master Change-Id: I92f0c9d37a16658cac10c6b0c98085b7b670c42f Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72061 Tested-by:
core-ci <typo3@b13.com> Tested-by:
Oliver Bartsch <bo@cedev.de> Tested-by:
Benni Mack <benni@typo3.org> Reviewed-by:
Oliver Bartsch <bo@cedev.de> Reviewed-by:
Benni Mack <benni@typo3.org>
-
- Nov 10, 2021
-
-
Andreas Wolf authored
TCA config property 'localizeReferencesAtParentLocalization' for database relations is only handled with type=select having a 'foreign_table' defined. With the similar type=group, this is only handled if 'foreign_table' is set, which is odd since group is based on 'allowed' over 'foreign_table'. The change adapts DataHandler to consider 'allowed' with type=group tables and checks whether at least one of them is localizable. This solves an @todo from #94570 in the group DataHandling tests. The test set is completed by adding a similar set for type=select. Resolves: #41713 Related: #94570 Releases: master Change-Id: I33df58fd4c9bdf31e27f64532c371c74b3f52627 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72102 Tested-by:
core-ci <typo3@b13.com> Tested-by:
Christian Kuhn <lolli@schwarzbu.ch> Tested-by:
Benni Mack <benni@typo3.org> Reviewed-by:
Christian Kuhn <lolli@schwarzbu.ch> Reviewed-by:
Benni Mack <benni@typo3.org>
-
Oliver Bartsch authored
TableController was removed in #95036. The corresponding service registration is now removed, too. Resolves: #95955 Related: #95036 Releases: master Change-Id: I9e55be091fd911f8f4d3d99f72fd4a30af505390 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72156 Tested-by:
core-ci <typo3@b13.com> Tested-by:
Christian Kuhn <lolli@schwarzbu.ch> Reviewed-by:
Christian Kuhn <lolli@schwarzbu.ch>
-
Imko Schumacher authored
Native time fields (dbtype) in the record list are now displayed independent from the server timezone. Resolves: #92962 Releases: master, 10.4 Change-Id: Ie9075378e67d0c302b126db78751992789fae29c Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/66953 Tested-by:
core-ci <typo3@b13.com> Tested-by:
Jochen <rothjochen@gmail.com> Tested-by:
Benni Mack <benni@typo3.org> Reviewed-by:
Jochen <rothjochen@gmail.com> Reviewed-by:
Markus Klein <markus.klein@typo3.org> Reviewed-by:
Benni Mack <benni@typo3.org>
-
Anja Leichsenring authored
All functional tests in DataHandling category (both workspaces and core) used to load the ext:irre_tutorial test extension in the uppermost abstract class. Not all of the tests use it though, and even declare additional test extensions. In order to clean up and provide test cases only with data they need and use, as a first step each abstract class below the highest level abstract declare their test extensions individually. Resolves: #95950 Releases: master Change-Id: If7922f16def8a0bde1de034f224bcd2b838f0d8f Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72150 Tested-by:
core-ci <typo3@b13.com> Tested-by:
Benni Mack <benni@typo3.org> Tested-by:
Stefan Bürk <stefan@buerk.tech> Tested-by:
Christian Kuhn <lolli@schwarzbu.ch> Reviewed-by:
Benni Mack <benni@typo3.org> Reviewed-by:
Stefan Bürk <stefan@buerk.tech> Reviewed-by:
Christian Kuhn <lolli@schwarzbu.ch>
-
Stefan Bürk authored
Text cropping on texts with way more than 1000 characters triggers regular expression compilation errors. The sweet spot was determined to a value of max 962 characters, thus leading to internal 'preg_match' error state and throwing 'Compilation failed: regular expression is too large' warning. The patch crops text in multiple chunks and reassembles the chunks until crop position was reached to avoid this internal error exceeding the max regular expression pattern length. Further tests are added to cover this scenario. Resolves: #77642 Releases: master, 10.4 Change-Id: I15b778370df0d0a3d3087a0b199c3e16f7a46add Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72048 Tested-by:
core-ci <typo3@b13.com> Tested-by:
Nikita Hovratov <nikita.h@live.de> Tested-by:
Christian Kuhn <lolli@schwarzbu.ch> Reviewed-by:
Nikita Hovratov <nikita.h@live.de> Reviewed-by:
Christian Kuhn <lolli@schwarzbu.ch>
-
Sybille Peters authored
Using references instead of absolute URLs has the following advantages: - If a target section is moved, the URL in a :ref: linking to it is updated automatically - It is possible to maintain the links to the other docs repository in a central place (Settings.cfg). This way, it is possible to link to the same version of a manual in a lower branch (e.g. 10.4) by changing the link in one place The above results in a better user experience (stay in information for same version when linking across docs repos) and less broken links (in the future). Resolves: #95948 Releases: master, 10.4 Change-Id: I84d801a75424bd94b4fa15aecb17a6085af07c36 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72145 Tested-by:
core-ci <typo3@b13.com> Tested-by:
Oliver Bartsch <bo@cedev.de> Tested-by:
Christian Kuhn <lolli@schwarzbu.ch> Reviewed-by:
Oliver Bartsch <bo@cedev.de> Reviewed-by:
Christian Kuhn <lolli@schwarzbu.ch>
-
Tomas Norre Mikkelsen authored
FlexForm Section can be added before content element is saved. This is done by ensuring that processedTca have a value for list_type. Earlier the pi_flexform was ignored as the default values on create before save was empty. Resolves: #81684 Releases: master, 10.4 Change-Id: I62e03cbbd93e8785f58c24dad1e573e8b5e0202b Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/68152 Tested-by:
core-ci <typo3@b13.com> Tested-by:
Benni Mack <benni@typo3.org> Tested-by:
Oliver Bartsch <bo@cedev.de> Tested-by:
Christian Kuhn <lolli@schwarzbu.ch> Reviewed-by:
Benni Mack <benni@typo3.org> Reviewed-by:
Oliver Bartsch <bo@cedev.de> Reviewed-by:
Christian Kuhn <lolli@schwarzbu.ch>
-
Alexander Schnitzler authored
The method signature of Query::locicalOr() 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 logicalOr() 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: #95938 Change-Id: I2a605cb4f42ec5324d973b63cf838278287de516 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72135 Tested-by:
core-ci <typo3@b13.com> Tested-by:
Benni Mack <benni@typo3.org> Tested-by:
Christian Kuhn <lolli@schwarzbu.ch> Reviewed-by:
Benni Mack <benni@typo3.org> Reviewed-by:
Christian Kuhn <lolli@schwarzbu.ch>
-
Sybille Peters authored
- Remove information about bug which no longer exists - Link to redirects documentation - Remove example for redirect module because such a redirect may not be the best example to handle with EXT:redirects because permanent redirects with regex might be better maintained in webserver configuration, examples are best kept in redirects documentation and example for this specific redirect and the regex for it can be easily found elsewhere. Resolves: #95944 Releases: master, 10.4 Change-Id: Ia8f01f37228efc88c2675e342a96738b5e20ae6c Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72141 Tested-by:
core-ci <typo3@b13.com> Tested-by:
Stefan Bürk <stefan@buerk.tech> Tested-by:
Christian Kuhn <lolli@schwarzbu.ch> Reviewed-by:
Stefan Bürk <stefan@buerk.tech> Reviewed-by:
Christian Kuhn <lolli@schwarzbu.ch>
-
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:
Andreas Fernandez <a.fernandez@scripting-base.de> Reviewed-by:
Benni Mack <benni@typo3.org> Reviewed-by:
Oliver Bartsch <bo@cedev.de>
-
Oliver Bartsch authored
Resolves: #95943 Releases: master Change-Id: Ibf5c2c9fe776ab3111c172085335541970a41de7 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72139 Tested-by:
core-ci <typo3@b13.com> Tested-by:
Benni Mack <benni@typo3.org> Tested-by:
Stefan Bürk <stefan@buerk.tech> Tested-by:
Andreas Fernandez <a.fernandez@scripting-base.de> Reviewed-by:
Benni Mack <benni@typo3.org> Reviewed-by:
Oliver Klee <typo3-coding@oliverklee.de> Reviewed-by:
Stefan Bürk <stefan@buerk.tech> Reviewed-by:
Andreas Fernandez <a.fernandez@scripting-base.de>
-
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:
core-ci <typo3@b13.com> Tested-by:
Christian Kuhn <lolli@schwarzbu.ch> Tested-by:
Stefan Bürk <stefan@buerk.tech> Tested-by:
Benni Mack <benni@typo3.org> Reviewed-by:
Oliver Klee <typo3-coding@oliverklee.de> Reviewed-by:
Christian Kuhn <lolli@schwarzbu.ch> Reviewed-by:
Stefan Bürk <stefan@buerk.tech> Reviewed-by:
Benni Mack <benni@typo3.org>
-
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:
core-ci <typo3@b13.com> Tested-by:
Stefan Bürk <stefan@buerk.tech> Tested-by:
Andreas Fernandez <a.fernandez@scripting-base.de> Reviewed-by:
Stefan Bürk <stefan@buerk.tech> Reviewed-by:
Andreas Fernandez <a.fernandez@scripting-base.de>
-
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:
core-ci <typo3@b13.com> Tested-by:
Oliver Bartsch <bo@cedev.de> Reviewed-by:
Simon Gilli <typo3@gilbertsoft.org> Reviewed-by:
Oliver Klee <typo3-coding@oliverklee.de> Reviewed-by:
Oliver Bartsch <bo@cedev.de>
-
Chris Müller authored
Resolves: #95925 Releases: master Change-Id: I125fcb5e60561646ee8398b0536106a2d5c828ed Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72103 Tested-by:
core-ci <typo3@b13.com> Tested-by:
Nikita Hovratov <nikita.h@live.de> Tested-by:
Christian Kuhn <lolli@schwarzbu.ch> Reviewed-by:
Nikita Hovratov <nikita.h@live.de> Reviewed-by:
Christian Kuhn <lolli@schwarzbu.ch>
-
Chris Müller authored
Resolves: #95926 Releases: master Change-Id: I67de7575d4c6b75062cb21e558402f15dfeb70a7 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/72104 Tested-by:
Nikita Hovratov <nikita.h@live.de> Tested-by:
core-ci <typo3@b13.com> Tested-by:
Christian Kuhn <lolli@schwarzbu.ch> Reviewed-by:
Nikita Hovratov <nikita.h@live.de> Reviewed-by:
Christian Kuhn <lolli@schwarzbu.ch>
-
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:
core-ci <typo3@b13.com> Tested-by:
Wouter Wolters <typo3@wouterwolters.nl> Tested-by:
Christian Kuhn <lolli@schwarzbu.ch> Reviewed-by:
Wouter Wolters <typo3@wouterwolters.nl> Reviewed-by:
Christian Kuhn <lolli@schwarzbu.ch>
-
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:
Wouter Wolters <typo3@wouterwolters.nl> Tested-by:
core-ci <typo3@b13.com> Tested-by:
Christian Kuhn <lolli@schwarzbu.ch> Reviewed-by:
Wouter Wolters <typo3@wouterwolters.nl> Reviewed-by:
Nikita Hovratov <nikita.h@live.de> Reviewed-by:
Christian Kuhn <lolli@schwarzbu.ch>
-
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:
Wouter Wolters <typo3@wouterwolters.nl> Tested-by:
core-ci <typo3@b13.com> Tested-by:
Christian Kuhn <lolli@schwarzbu.ch> Reviewed-by:
Wouter Wolters <typo3@wouterwolters.nl> Reviewed-by:
Christian Kuhn <lolli@schwarzbu.ch>
-
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:
core-ci <typo3@b13.com> Tested-by:
Wouter Wolters <typo3@wouterwolters.nl> Tested-by:
Christian Kuhn <lolli@schwarzbu.ch> Reviewed-by:
Wouter Wolters <typo3@wouterwolters.nl> Reviewed-by:
Christian Kuhn <lolli@schwarzbu.ch>
-
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:
core-ci <typo3@b13.com> Tested-by:
Wouter Wolters <typo3@wouterwolters.nl> Tested-by:
Christian Kuhn <lolli@schwarzbu.ch> Reviewed-by:
Wouter Wolters <typo3@wouterwolters.nl> Reviewed-by:
Christian Kuhn <lolli@schwarzbu.ch>
-
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:
core-ci <typo3@b13.com> Tested-by:
Christian Kuhn <lolli@schwarzbu.ch> Tested-by:
Benni Mack <benni@typo3.org> Reviewed-by:
Christian Kuhn <lolli@schwarzbu.ch> Reviewed-by:
Benni Mack <benni@typo3.org>
-
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:
Christian Kuhn <lolli@schwarzbu.ch> Tested-by:
core-ci <typo3@b13.com> Tested-by:
Benni Mack <benni@typo3.org> Reviewed-by:
Christian Kuhn <lolli@schwarzbu.ch> Reviewed-by:
Benni Mack <benni@typo3.org>
-
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:
Christian Kuhn <lolli@schwarzbu.ch> Tested-by:
core-ci <typo3@b13.com> Tested-by:
Benni Mack <benni@typo3.org> Reviewed-by:
Christian Kuhn <lolli@schwarzbu.ch> Reviewed-by:
Benni Mack <benni@typo3.org>
-