- Jul 27, 2022
-
-
Stefan Bürk authored
With PHP 8.2 old and unreliable methods `utf8_encode()` and `utf8_decode()` have been deprecated and will be removed in PHP 9.0. There are several ways to replace these methods, although none of them are "ensured" to be native enabled in PHP. We need to replace these methods in tests only, and as we have testing setup fully under control, this patch use `mb_convert_encoding()` from php extension `mbstring` to avoid deprecated methods and corresponding native php deprecation messages in functional test. Resolves: #98038 Releases: main, 11.5 Change-Id: I8a70a203eeae6a314e30cedab8668541145ef6c3 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/75312 Tested-by:
core-ci <typo3@b13.com> Tested-by:
Christian Kuhn <lolli@schwarzbu.ch> Tested-by:
Oliver Klee <typo3-coding@oliverklee.de> Tested-by:
Stefan Bürk <stefan@buerk.tech> Reviewed-by:
-
Stefan Bürk authored
With PHP 8.2 setting undefined properties in class instances is deprecated and emits corresponding deprecation notice. This patch removes a superflous test, setting a undefiend property in `TypoScriptFrontendController` to avoid PHP 8.2 deprecation message. Resolves: #98039 Releases: main, 11.5 Change-Id: I0944a13f6c87c95db01577dbdb499c39675691be Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/75313 Tested-by:
-
Andreas Fernandez authored
To provide a consistent look&feel and to eventually phase out FontAwesome in the TYPO3 backend, all remaining fa-* icons are replaced with their counterparts from the @typo3/icons package. The CSS class `icon-emphasized` is introduced as a drop-in replacement for the `fa-stack` construct: ``` <span class="icon-emphasized"> <core:icon identifier="my-icon-identifier" size="small"/> </span> ``` The removal of the FontAwesome integration will follow in a separate patch. Resolves: #97877 Releases: main Change-Id: I0e9ca1e4a918b3f44969ea9886ec0cd1e56d78be Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/75041 Tested-by:Nikita Hovratov <nikita.h@live.de> Tested-by:
Benni Mack <benni@typo3.org> Reviewed-by:
-
Nikita Hovratov authored
Since the removal of css_styled_content the TypoScript snippet `lib.stdheader` is gone. This patch removes some leftovers in the `addPItoST43` method and some other places. Resolves: #98012 Related: #80876 Releases: main Change-Id: I3e1b2795c05b464f4210abc0968f507f20f9fba7 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/75273 Tested-by:
-
Oliver Klee authored
This avoids problem when GU::trimExplode is switched to native type declarations. Also fix some possible invalid array accesses. Resolves: #97960 Relates: #97602 Releases: main Change-Id: I6a08b7a20f12475fbee7a5a817f1618e9466087c Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/75210 Tested-by:
-
Larry Garfield authored
Not marking this as breaking since it's a test fixture only. Resolves: #98034 Releases: main Change-Id: Id2dd650c617f5be807d3615751466102d274cea3 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/75309 Tested-by:
-
- Jul 26, 2022
-
-
Larry Garfield authored
Doing so also allows for a bit of code cleanup. Used command: > ./Build/Scripts/runTests.sh -s phpstanGenerateBaseline Resolves: #97982 Releases: main Change-Id: Id351fda2040fd7de3931a109b77ff0c947c552b3 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/75251 Tested-by:
-
Larry Garfield authored
Also switches to constructor property promotion in the process. Used command: > ./Build/Scripts/runTests.sh -s phpstanGenerateBaseline Resolves: #98004 Releases: main Change-Id: Ic55548afe59654da4ab7fc7ab5a4cfb408ab38d8 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/75266 Tested-by:
-
Larry Garfield authored
This was most likely a very old copy-paste error. Used command: > ./Build/Scripts/runTests.sh -s phpstanGenerateBaseline Resolves: #98025 Releases: main, 11.5 Change-Id: Idacf7ca70895a9fd1be9c765e714a68ab6e44032 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/75292 Tested-by:
-
Larry Garfield authored
Used command: > ./Build/Scripts/runTests.sh -s phpstanGenerateBaseline Resolves: #98008 Releases: main, 11.5 Change-Id: Iecadad3cc1e5089f0ab7282f5b5302ead66ce6a7 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/75270 Tested-by:
-
Larry Garfield authored
v11 introduced support for using a Channel attribute to define what log channel should be injected into a class, either via the constructor or the LoggingAwareInterface. However, the initial patch lacked any tests. This patch adds basic tests to verify that the attribute works, and indirectly document it. Resolves: #97695 Related: #95079 Related: #95044 Releases: main, 11.5 Change-Id: I7cc97fc839ccd4015ac20b1bce787d4cbbc40a26 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/74750 Tested-by:
-
Larry Garfield authored
The docblock for QueryObjectModelFactory::comparison() expects a string, specifically one of the QueryObjectModelConstants.JCR_OPERATOR_* constants. However, that class seems to no longer exist, and what it actually uses is an integer. (Passing a string through would break in subsequent sub-calls.) This corrects the type to the correct integer constants that are actually used in practice, and uses a language type to ensure it is used properly. Used command: > ./Build/Scripts/runTests.sh -s phpstanGenerateBaseline Resolves: #98007 Releases: main, 11.5 Change-Id: Id00e30370a1944a3158cb46825ccc4e15174f7ab Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/75269 Tested-by:
-
Larry Garfield authored
ExtensionStatus::getStatus()'s doc type does not account for the use of [] as a sentinel value. The extra doc type declaration in FolderBasedFileCollection::fromArray() is incorrect, as it does not account for null return values. Used command: > ./Build/Scripts/runTests.sh -s phpstanGenerateBaseline Resolves: #98003 Releases: main, 11.5 Change-Id: Ifaad71a8e65836e943b1a3813f3294e7421c5402 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/75264 Tested-by:
-
Christian Kuhn authored
Property setValidatedInstancesContainer does not exist in ObjectValidatorInterface and its implementations and is never read in extbase since ever. Drop it. Change-Id: I64b5e1c4e297322fb057f2c6a1cf6f1f8103b90e Resolves: #98031 Releases: main, 11.5 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/75300 Tested-by:
-
Christian Kuhn authored
Add CI unit testing setup with PHP 8.2 in pre-merge and nightly "locked" builds: Nightly "max" and "min" builds are blocked by prophecy for now. Resolves: #98030 Releases: main, 11.5 Change-Id: Ia6814465ade7efd33eaafe9850aa62d2e907ec21 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/75298 Tested-by:
-
Christian Kuhn authored
Prophecy has a minor PHP 8.2 issue as reported with https://github.com/phpspec/prophecy/issues/555 This seems to be triggered by willBeConstructedWith() when SuT has constructors in its inheritance chain. Luckily, we can simplify the test a bit and circumvent the issue for now. Change-Id: I6e50de3345b5d27ec3e46fe78c4855a7a0fb7d43 Resolves: #98028 Releases: main, 11.5 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/75296 Tested-by:
-
Christian Kuhn authored
Brings PHP 8.2 fixes. Setting mikey179/vfsstream as explicit dependency since testing-framework dropped the dependency. Raising testing-framework. > composer req --dev mikey179/vfsstream:^1.6.11 > composer req --dev typo3/testing-framework:dev-main Change-Id: I4761948bd8827ab4638f280d5b69403d300afcb1 Resolves: #98026 Releases: main, 11.5, 10.4 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/75291 Tested-by:
-
Benni Mack authored
TYPO3's TCA definition has a setting called `cruser_id` which automatically is filled with the current TYPO3's Backend User ID, if created via DataHandler. However, this information is also available in the RecordHistory (since its extraction from sys_log in v9) and can be queried, when needed, directly from there. This way, all TCA-based tables, having the setting "cruser_id" do not need to carry this value anymore, as it is duplicated in TYPO3's data model. This change removes the evaluation of "cruser_id", and removes the filling of this property via DataHandler. The DB field is also not set anymore automatically. The Info popup (ElementInformationController) now shows the information from the RecordHistory information. Resolves: #98024 Releases: main Change-Id: I1fe54b0ecfc0ee9b2d1867011dd960e1906fd2d4 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/75123 Tested-by:
-
- Jul 25, 2022
-
-
Nikita Hovratov authored
In a scenario, where a backend user has two entry points, in which the second one intersects the first one (is a direct sub-page), pages were shown multiple times. The reason behind this was, that inside PageTreeRepository->groupAndSortPages() every page was added regardless if it was already added in a previous run. This is the case, when the same pages are found in different depth levels. E.g. 1 - Home (DB mount 1) 2 - Page 1 2 - Page 2 (DB mount 2) 3 - Subpage 1 // will be added for level 2 and 3 3 - Subpage 2 To prevent this, a guard clause was added, which checks whether the page was already added by comparing the uid value. Resolves: #96512 Resolves: #97774 Related: #96367 Releases: main, 11.5, 10.4 Change-Id: I65439011b894f05faef9cb507d905b3d7d3b709f Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/74258 Tested-by:Silvia Bigler <silvia@dunkelweiss.li> Tested-by:
-
Larry Garfield authored
PHPStan says all of these are unnecessary. On visual inspection, it appears to be correct. Used command: > ./Build/Scripts/runTests.sh -s phpstanGenerateBaseline Resolves: #98001 Releases: main, 11.5 Change-Id: Ic18ce25997357c786b17bc5be02cfdb3617c6eef Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/75262 Tested-by:
-
Torben Hansen authored
This change adds type definitions for all class properties in ext:felogin. Resolves: #98009 Releases: main Change-Id: I0c8c504a4b669e7a3e7d2f47c87a5847db59cb07 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/75271 Tested-by:
Oliver Hader <oliver.hader@typo3.org> Tested-by:
Guido Schmechel <guido.schmechel@brandung.de> Reviewed-by:
-
Christian Kuhn authored
Second main usage of the new TypoScript parser is the ext:tstemplate Object Browser. In contrast to the Template Analyzer patch that brought a fully revamped view with new features, this Object Browser patch is mostly just a modernization of the view, so this patch is not marked as feature. The Object Browser now renders both setup and constant tree at the same time. Other than that, most existing features are kept and streamlined along the way. As example, searching for a constant name now reveals constant usages in setup, plus similar small gems. The only missing features are currently the comments rendering, plus rendering of parser errors. Comments are a bigger issue that will be implemented with a revamped Constant Editor, Object Browser will then add comment rendering as side effect. Parts of the parser errors (detecting invalid lines) will be added to Template Analyzer with a dedicated patch soon, further details will be added to Object Browser when suitable. Change-Id: I08b2d7a1c1f60d2ab2184fb005dc1132b261f1aa Resolves: #98013 Related: #97816 Releases: main Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/75259 Tested-by:
-
Christian Kuhn authored
Not setting $GLOBALS['TCA']['myTable']['ctrl']['crdate'] or $GLOBALS['TCA']['myTable']['ctrl']['cruser_id'] or $GLOBALS['TCA']['myTable']['ctrl']['tstamp'] leads to rather obvious PHP 8 "Undefined array key" warnings in DataHandler. Sanitize them. Resolves: #98018 Releases: main, 11.5 Change-Id: Ia3e94ac65feede4668fd1247cd29562f3215f328 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/75283 Tested-by:
-
Christian Kuhn authored
Some of the more advanced fields of sys_template records are rather hard to understand without additional description. The patch moves all more complex fields to an "Advanced Options" tab and uses the TCA "description" feature to add some explanations and hints if and when integrators should bother with them in typical scenarios. Resolves: #98017 Related: #97816 Releases: main, 11.5 Change-Id: Ifb4166dd225ec1c74339726f97ec7d7391e6996e Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/75281 Tested-by:
Georg Ringer <georg.ringer@gmail.com> Reviewed-by:
-
- Jul 24, 2022
-
-
Christian Kuhn authored
The issue has been resolved in sysbox, we don't need the patch level pin anymore. Resolves: #98015 Related: #97570 Releases: main, 11.5, 10.4 Change-Id: I1a2080ec3e8609350c9975bad8f65e69513d5f48 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/75277 Tested-by:
-
Christian Kuhn authored
phpstan almost always triggers a full recalculation ignoring cache in CI. We switch phpstan CI cache config to /tmp which is not cached by CI to save some memory. Resolves: #98014 Releases: main, 11.5 Change-Id: I0bf2c72f529e9f179eb4becaac4d164cad8c318a Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/75275 Tested-by:
-
- Jul 23, 2022
-
-
Christian Kuhn authored
Scenario: * Create a live page that has a 'media' resource * Create a localization of that page * Go to workspace * Edit the localized page * Save * Save again -> crash "Array to string conversion" in DataHandler isSubmittedValueEqualToStoredValue() Investigating this, the issue is not in isSubmittedValueEqualToStoredValue() itself, but within DataMapProcessor, which fails to apply a workspace overlay due to #92209: The default WorkspaceRestriction is not sufficient in this case. The patch changes DataMapProcessor to apply workspace overlays in a different way for this special case. Resolves: #97989 Related: #92209 Releases: main, 11.5 Change-Id: I176aa2d2e209224b48517b872ea39b1e01f06af7 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/75256 Tested-by:
-
- Jul 22, 2022
-
-
Stefan Bürk authored
Localized page preview links in page layout module or view module display page in default language. With #97544 preview uri generation hooks has been replaced with corresponding PSR-14 events. Language information has been divided from the paramaters array to have it as direct property in the new PSR-14 events. `PageRouter->generateUri()` needs proper language information as `$parameter` argument, which is not properly reassembled before generating the preview uri in `PreviewUriBuilder()`, using event information. ext:workspace uses the before event to change the preview uri only added the language id as query argument `_language`, if additional query arguments are provided. This failing to show localized workspaced page in workspace preview. This change reassemble language information back into the `$parameters` array to build localized preview uri again, for live workspace preview and for workspace preview. This is also done in corresponding ext:workspace eventlistener to always provide non-default language query argument with the newer `_language` parameter, even if there are no other additional query arguments. Resolves: #98000 Related: #97544 Releases: main Change-Id: Ie02ee0437da4c692a50dc3b7802301af519793ae Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/75260 Tested-by:
Simon Schaufelberger <simonschaufi+typo3@gmail.com> Tested-by:
-
Christian Kuhn authored
Scenario: * Create a live page, add two images to media field * Go to workspaces, edit page, resort the two images, save * Go to workspace module of that page -> Crash in main ajax call "DiffUtility::makeDiffDisplay(): Argument #1 ($str1) must be of type string, int given" Fix with proper cast. Resolves: #97997 Related: #97598 Related: #97373 Releases: main Change-Id: I1764cab043e9a8c3fd8718c08d13549c9c590f98 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/75258 Tested-by:
Andreas Fernandez <a.fernandez@scripting-base.de> Tested-by:
crell <larry@garfieldtech.com> Reviewed-by:
-
Andreas Fernandez authored
There were a few usages of `btn-checkbox` left in TYPO3 code base which can be savely replaced with Bootstrap's `form-check`. To make it more recognizable in the constants editor, the unchecked state is represented by a square icon, representing an unchecked box. Resolves: #97980 Releases: main Change-Id: I4e0db3d1ce1eba220de5902e5ac15a4b2a356b1b Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/75250 Tested-by:
-
Nikita Hovratov authored
Also, update the comment above. Resolves: #97992 Releases: main Change-Id: I019ba4079857f021828c83658f950ed908990d34 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/75257 Tested-by:
-
Georg Ringer authored
Avoid notices in PageLinkBuilder Resolves: #97968 Releases: main, 11.5 Change-Id: Icc55f4b9fdd6e53333d5337d65b9971a74e956b5 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/75246 Tested-by:
-
Larry Garfield authored
The > operator binds higher than ??, meaning these are always comparing null to 0. That is clearly not the intent. Used command: > ./Build/Scripts/runTests.sh -s phpstanGenerateBaseline Resolves: #97978 Releases: main, 11.5 Change-Id: I3410e087fe7fa214c4cb43312a551d8ca3047e70 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/75248 Tested-by:
-
- Jul 21, 2022
-
-
Christian Kuhn authored
Relations in 'custom value' mode may not have translation parents. This raises a PHP 8 array access issue in workspace module when for instance a page translation defines own relations for 'media': The workspace module main ajax call crashes. Sanitize this. Resolves: #97988 Releases: main, 11.5 Change-Id: Ie28d3db43a0e425a909d78355e4ecd64e1aedf8b Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/75255 Tested-by:
-
Christian Kuhn authored
When the TCA colon feature for ['ctrl']['type'] is used in inline parent child relations, the target record may not yet be persisted and getRecord() returns null. This triggers a PHP 8 array warning. Sanitize this access. Resolves: #97985 Releases: main, 11.5 Change-Id: If26e7aecdfc9504fbf1727813bc825de6522b9f9 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/75253 Tested-by:
Michael Semle <michael.semle@b13.com> Tested-by:
-
Larry Garfield authored
This check likely predates types being specified, which makes it unnecessary. Used command: > ./Build/Scripts/runTests.sh -s phpstanGenerateBaseline Resolves: #97979 Releases: main, 11.5 Change-Id: I78af547b9b3c1412c50cc36373973b160f3a82d4 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/75249 Tested-by:
-
- Jul 19, 2022
-
-
Anja Leichsenring authored
With PHP 8.2, dynamically declared properties raise a deprecation warning, leading to test fails. Resolve these. Resolves: #97969 Releases: main, 11.5 Change-Id: Idb1ad46415fcfa2c2c562863004c9f2b88cff58c Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/75243 Tested-by:
-
Torben Hansen authored
The initialization of the variable `$conditionMatcher` with a `null` value can safely be removed, since the following condition always overwrites the initial variable value. Resolves: #97970 Releases: main, 11.5 Change-Id: Ia7a555c9ac480f2af1e13089edffd6105f37cdc8 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/75244 Tested-by:
-
Anja Leichsenring authored
These raises fix the bulk of PHP 8.2 unit test fails. We need to make webmozart/assert:^1.11.0 explicit to pin it as minimum version for PHP 8.2 composer update --prefer-lowest in nightlies. We don't strictly need the phpunit raise, but pick it as casual dev update along the way. $ composer require --dev phpunit/phpunit:^9.5.21 $ composer require --dev webmozart/assert:^1.11.0 Resolves: #97967 Releases: master, 11.5 Change-Id: I5c79a8577c5eb836566cb5c00bb6b63aa1b7ea1f Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/75241 Tested-by:
-
ToGetATockenOnly authored
Releases: main, 11.5 Resolves: #97965 Change-Id: I6e03354925b4ecb9c3f72554862a01057851f48b Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/75239 Tested-by:
-
