- Aug 04, 2023
-
-
Stefan Bürk authored
Resolves: #101552 Releases: main, 12.4, 11.5 Change-Id: If9f12fa195f119b715e163c578188d8634b68b50 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/80315 Tested-by:
core-ci <typo3@b13.com> Tested-by:
Christian Kuhn <lolli@schwarzbu.ch> Reviewed-by:
-
Christian Spoo authored
parent::__construct() of MemcachedBackend calls setCompression() which determines compression based on state of $this->usePeclModule. This property needs to be set before parent::__construct() is called. Resolves: #99495 Releases: main, 12.4, 11.5 Change-Id: Ibc6c3a2378c5187748b4394d440232600de193e2 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/80387 Tested-by:
Stefan B�rk <stefan@buerk.tech> Reviewed-by:
-
Tobias Liebig authored
This test checks lowlevel things like usergroup to be changed by GeneralUtility::fixPermissions(). Various tests in this area have been disabled on Windows already. They are problematic in macOS driven docker as well since macOS heavily messes around with access rights. We give up on this now, and reduce the details test checks of GeneralUtility::fixPermissions() a bit by removing that test. Resolves: #101568 Releases: main, 12.4, 11.5 Change-Id: I36e781302285dc1966633723abdc8f15bf84ef06 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/80382 Tested-by:
-
Stephan Jorek authored
Resolves: #101529 Releases: main, 12.4 Change-Id: I56bc70e244d810a275eea6e17f16f0f94b69b42f Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/80384 Tested-by:
-
- Aug 03, 2023
-
-
Simon Schaufelberger authored
While the backend already renders preview images of for example YouTube videos in the media module, it doesn't show the preview image anywhere else, for example in the page module and the Media elements within a "Text and Media" content element. This patch shows the generated preview image within the page module and the "Text and Media" content element as well. Resolves: #95454 Releases: main, 12.4, 11.5 Change-Id: I4e404cf41627d5ee9ac165ee46817a456dbecaa8 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/80353 Tested-by:
Andreas Fernandez <a.fernandez@scripting-base.de> Reviewed-by:
-
Jasmina Ließmann authored
In the page module it is possible to display the content of the languages side by side with the "Language Comparison" mode. With different amounts of text in the content elements of each language, the page TSconfig option `mod.web_layout.defLangBinding = 1` can help to arrange the content in a more structured way. If the `mod.web_layout.defLangBinding = 1` option was activated, the first button for creating new content elements was displayed on a white background. With this patch, the button is now displayed on a gray background, analogous to the rest of the content area. Resolves: #101536 Releases: main, 12.4 Change-Id: I88a47d8010d0520bea84f01f9c21408e0f3908f9 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/80352 Tested-by:
Jasmina Lie�mann <minapokhalo+typo3@gmail.com> Tested-by:
-
Christian Spoo authored
Invoke `str_ends_with` on lowercased filesystem entries in order to check for allowed extensions. This is more stable than `substr`ing the comma-separated list of extensions. Resolves: #101035 Releases: main, 12.4 Change-Id: I6a0eee1f6ddb7406a8da168ed489aba6bfc83263 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/80230 Tested-by:
Oliver Bartsch <bo@cedev.de> Tested-by:
-
Oliver Bartsch authored
Apply type cast to the input value to prevent a TypeError. Resolves: #101555 Releases: main, 12.4 Change-Id: If2db6d0cc793999f2ce93d16cfedadb12325c316 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/80231 Tested-by:
-
Stefan Bürk authored
With PHP8.3 the `ReflectionProperty->setValue()` method emits a E_DEPRECATED if a value should be set to a class and the context is not set as first argument. For instanciated classes the class needs to be provided, which is not possible for a static class or property. The solution for this is to use `null` as first argument as context object. That avoids the deprecation and keeps the backward compatibility. The testing-framework provides a tool to keep and handle static Framework state, which uses reflection under the hood and therefore needs the null context for `setValue()`. To verify manually use: > Build/Scripts/runTests.sh -s functional -p 8.3 Note: Activating in CI runs will be shuffeld in a dedicated change afterwards. Resolves: #101548 Releases: main, 12.4 Change-Id: I13fae7a20162eb3a48910ca91a5a1b178d5a1821 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/80228 Tested-by: -
Jasmina Ließmann authored
Several H1 headings are used in some card views. As a result, the semantic structure for these views is not correct. The relevant H1 headings are changed to h2. Resolves: #101545 Releases: main, 12.4 Change-Id: Ic53d92a52b1ae75adda18a63bc76633d7ffcfa44 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/80227 Tested-by:
-
Christian Kuhn authored
With workspaces and the extbase test related blog_example being fixed, we can finally add a primary key on MM tables that have no uid field. Resolves: #101425 Related: #101528 Related: #101537 Related: #95061 Releases: main, 12.4 Change-Id: I2cd44e3f21dac4b774b8067839a6d2abb8d759fb Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/80308 Reviewed-by:
-
Chris Müller authored
Resolves: #101539 Releases: main, 12.4 Change-Id: Id77ae5fb63c0de3d22cb1a4f4302c76476369b67 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/80305 Tested-by:
-
Stefan Bürk authored
Update the typo3/testing-framework to pull in a needed correction as preparation for functional tests with PHP 8.3. Used command(s): > composer require --dev \ typo3/testing-framework:^8.0.2 Resolves: #101538 Releases: main, 12.4, 11.5 Change-Id: Id26531be8c49a9b1ee9624c178342659b28d3eab Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/80301 Reviewed-by:Oliver Klee <typo3-coding@oliverklee.de> Tested-by:
-
- Aug 02, 2023
-
-
Christian Kuhn authored
Similar to the recently fixed other MM TCA setup of blog_example, the "related_posts" is adapted as well. This removes the last mm table definition from ext_tables.sql - the auto-creation kicks in as expected now. Resolves: #101537 Related: #101528 Releases: main, 12.4 Change-Id: Ia92ce6bbc78a493e1a2028bd9ab7a31952181dd3 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/80299 Reviewed-by:
-
Andreas Fernandez authored
When creating a new site configuration, deleting the default language (English) and importing a new language afterwards now doesn't create an invalid site configuration with languageId 0 missing. Resolves: #100439 Releases: main, 12.4 Change-Id: Ic6a83328e92941e4c9894019aab5162f0b3937d9 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/80226 Reviewed-by:
-
Helmut Hummel authored
While the initial idea to store the current ContentObjectRender (cObj) instance in a request attribute, to pass it along to code that requires both, the request and a cObj, is solid, we missed that cObj itself does not require to hold the request as attribute. The patch changes this: The request attribute is only set for ContentObjects, as these are actually responsible for rendering and eventually evaluating a request or the cObj instance. This makes sure, that an instance of cObj is available for Extbase plugins as well as for Fluid view helpers, which can access the request via RenderingContext. By changing the concept slightly, all places that previously added the cObj to the request can be removed in favor of doing so only in AbstractContentObject and ContentObjectRenderer::callUserFunction. The latter is at least required for TypoScript using the old way of calling Extbase plugins with USER and it's userFunc property. The circular reference is now removed, by not passing the request that contains the cObj attribute to the cObj itself. This means: * When a cObj is available, one can obtain the request using its getter * Such request objects, do NOT contain an attribute containing the cObj * The cObj request attribute can now be obtained in all content objects (such as EXTBASEPLUGIN or FLUIDTEMPLATE) and user functions, that are called from be ContentObjectRenderer. Releases: main, 12.4 Resolves: #101170 Resolves: #100872 Related: #100623 Change-Id: I6acb1a92c8b02f1be2f3a396fe9d9465d6482033 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/79521 Tested-by:
Benni Mack <benni@typo3.org> Tested-by:
-
linawolf authored
They should conform to the settings in `.editorconfig`. When your editor uses the `.editorconfig` file, these files become very hard to edit. I would backport this to make backports easier. Resolves: #101527 Releases: main, 12.4 Change-Id: I41e1782b46bbbf327106089f39dc28d2ca358761 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/80296 Reviewed-by:
-
Benjamin Franzke authored
npm update lit lit-html @lit/reactive-element lit-element grunt build Releases: main, 12.4 Resolves: #101496 Change-Id: Ic8de6a200a70ca2721719b531dde2a146692b317 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/80225 Tested-by:
Anja Leichsenring <aleichsenring@ab-softlab.de> Reviewed-by:
-
Christian Kuhn authored
Extension blog_example is used in various extbase tests. It defines invalid relations from tags table over MM to post and persons. The patch fixes the TCA - those relations now work in BE as well, when the fixture blog_example is activated. Essentially all functional tests that works with this extension break with the fixed TCA now. The major part of the patch is to disentangle the .csv import and export files to make them independent from each other per test-case, and to fix the imported and asserted db state along the way. Resolves: #101528 Releases: main, 12.4 Change-Id: I4fb7d6ac82962f8ef7b630b261d543f4ff626bde Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/80295 Reviewed-by:
-
Benjamin Franzke authored
Imports via bare module specifiers (such that do not start with / or .) are cache busted by their URL mapping in importmap — relative imports (inside lit) are not by default. This is why we used version-suffixed lit package folders in #96511 (pretty much like unpkg.com does). This got removed with #100245, as required updates for configuration files have not been applied automatically and the reason for the version suffix was undocumented and thus unclear. While we could fix those issues and add autoupdate support for configuration files, it is now decided to remap all relative imports, as this logic is present in our build setup anyway and eases future updates by creating smaller diffsets (not every file has to be renamed, only changed ones are updated). Note that technically a new helper grunt task es-module-lexer-init is added, which is required to await the asynchronous es-module-lexer parser initialization (written in web assembly, and the...
-
- Jul 31, 2023
-
-
Christian Kuhn authored
Creating reports.xml in acceptance tests and feeding this to gitlab has been added with issue #96628 in the hope to be useful for finding frequently failing ac tests. The gitlab UI however buries this, it's de facto impossible to make use of this information. We drop this definition again. Resolves: #101514 Related: #96628 Releases: main, 12.4, 11.5 Change-Id: I91f13f402f0b9f72f2ca41252db93bb1714efcd6 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/80287 Tested-by:
-
Christian Kuhn authored
When select fields with db relations allow pages records, the DatabaseBrowser tries to render an icon from the page row. uid of that row may be an int, which needs to be cast to string to be fed to htmlspecialchars(). Resolves: #101513 Related: #100819 Releases: main, 12.4, 11.5 Change-Id: I52ee031cb0eb09f41ae8ecb6d7005f03e65de3bd Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/80284 Tested-by:
-
Nikita Hovratov authored
Resolves: #101502 Related: #97330 Releases: main, 12.4 Change-Id: I1171ec0cf44c61480a0dffb0b6105022c8b8efed Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/80277 Tested-by:
-
Christian Kuhn authored
> composer req --dev typo3/cms-styleguide:^12.0.2 Resolves: #101508 Releases: main, 12.4 Change-Id: I824b05025c32eb72d10d908b9e61d6da6a7ca91a Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/80275 Tested-by:
-
Oliver Klee authored
This method promises to return an int, but actually returns a float. Also add a regression test. > ./Build/Scripts/runTests.sh -s phpstanGenerateBaseline Resolves: #101499 Releases: main, 12.4, 11.5 Change-Id: I85ca63c9b32717641d5926b16ee25e32eeb69310 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/80272 Tested-by:
-
Thomas Anders | Naderio authored
The labels of inline FileReference fields were not concatenated but overwritten, in a loop since #92427. This patch corrects the concatenation and adds line-break divs, otherwise the strings are on a single line. Resolves: #99873 Related: #92427 Releases: main, 12.4, 11.5 Change-Id: I368cd0d2cbf7e9ed7fdd1d003f869cc894d6f13e Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/80267 Tested-by:
Nikita Hovratov <nikita.h@live.de> Reviewed-by:
-
- Jul 30, 2023
-
-
Christian Kuhn authored
Add some waitFor() to the drag&drop tests and give the filter test a generic wait which hopefully stabelizes this one a bit as well. Resolves: #101494 Releases: main, 12.4, 11.5 Change-Id: I73d4d2415f4e100bed7f1d7e5ead1036d157af93 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/80257 Reviewed-by:
-
Christian Kuhn authored
Those need some additional waitFor checks to be more stable since the v12 refactoring with all the new js magic. Resolves: #101493 Releases: main, 12.4 Change-Id: Ib5cf06f92ced20c5ee97a77ef15a870dcbf0c36a Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/80223 Tested-by:
-
Christian Kuhn authored
Similar to #101491, the other Application/Frontend acceptance tests get same treatment to wait for the "View webpage" icon being visible, before trying to click it. Resolves: #101492 Related: #101491 Releases: main, 12.4, 11.5 Change-Id: Ie8dd5338a6773baf8850fec0713dcd917699fbfc Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/80253 Tested-by:
-
Christian Kuhn authored
Fails locally occasionally. The additional explicit waitForElement() before clicking the elemnt stabilized it. Resolves: #101491 Releases: main, 12.4, 11.5 Change-Id: I4072daa0e949d3480f399e99a2f35d8ef889eded Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/80250 Reviewed-by:
-
Oliver Bartsch authored
Additionally remove an outdated warning. Resolves: #101490 Related: #99220 Releases: main, 12.4 Change-Id: I0943ffa2097bd5284aa91d0bb6180a789e28e89f Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/80222 Reviewed-by:
Chris M�ller <typo3@krue.ml> Tested-by:
-
Torben Hansen authored
Add several type casts to avoid notices. Resolves: #101486 Releases: main, 12.4, 11.5 Signed-off-by:
Torben Hansen <derhansen@gmail.com> Change-Id: I87f02e129d91ae9902ad74f306f11915d12037b0 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/80221 Reviewed-by:
-
Christian Kuhn authored
A stupid return vs. continue issue breaks a loop in SysTemplateTreeBuilder early, which leads to `ext_typoscript_[setup|constants].typoscript` not being considered when they have been partially cached already. Resolves: #101407 Related: #97816 Releases: main, 12.4 Change-Id: Ia96bfd10a0f2998b1baa1c5a153417b424ba05a2 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/80220 Tested-by:
-
Christian Kuhn authored
This one was still flaky. Properly waiting for the element should finally fix this, this patch now had a huge number of successful runs locally, while it failed repeatedly without. Resolves: #101487 Related: #101440 Releases: main, 12.4, 11.5 Change-Id: I2cf4b3e878caf430ead27340694a169f98e710e8 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/80242 Reviewed-by:
-
- Jul 29, 2023
-
-
Benjamin Franzke authored
Resolves: #101480 Related: #101453 Releases: main, 12.4, 11.5 Change-Id: I62e7166a6ceceb6e8c6c0d0aa5393def72c75346 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/80219 Tested-by:
Benjamin Franzke <ben@bnf.dev> Tested-by:
-
Benjamin Franzke authored
Releases: main, 12.4, 11.5 Resolves: #101482 Change-Id: Id23a78cdb85c3b58a0ce79eacbc5626c27cde46b Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/80217 Tested-by:
-
- Jul 28, 2023
-
-
Oliver Hader authored
The CSP directives 'report-to', 'require-trusted-types-for' and 'trusted-types' have been added. Albeit there aren't any typed value counterparts yet, they can be wrapped in a RawValue object, e.g. new Mutation( MutationMode::Set, Directive::RequireTrustedTypesFor, new RawValue("'script'") ), The stand-alone directives 'sandbox', 'trusted-types' and 'upgrade-insecure-request' now can be used without any values. The cases for 'unsafe-hashes' and 'strict-dynamic' were accidentally added as directives instead of source keywords and have been removed. The source schemes 'filesystem:' and 'mediastream' have been added. Besides that, the frontend CSP configuration now limits using the `<base>` element to same-origin URIs. The backend CSP configuration is now even stricter since using `<base>`, `<embed>` and `<object>` elements is blocked. Resolves: #101477 Releases: main, 12.4 Change-Id: Ie1ce2e30dc0d79faa5b7a923fa39a88dbee17292 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/80208 Reviewed-by:Oliver Hader <oliver.hader@typo3.org> Tested-by:
-
Oliver Hader authored
Using 'strict-dynamic' (SourceKeyword::strictDynamic) should only be allowed for script-src*[1] directives. Using it for any other directive produces strange side effects, since it implicitly adds a CSP nonce value to that directive when compiling the policy. Thus, this change + ensures 'strict-dynamic' is applicable for script-src* when using Policy::set, Policy::extend or the constructor + removes invalid policy purges when 'strict-dynamic' was used, since that was only valid for CSP level 3 - ignoring these directives is supposed to be handled by the browser[2], not by the server-side API, in order to preserve CSP v1/v2 BC. [1] https://w3c.github.io/webappsec-csp/#allow-all-inline > 'strict-dynamic' only applies to scripts, not other resource types. [2] https://w3c.github.io/webappsec-csp/#strict-dynamic-usage > 8.2. Usage of "'strict-dynamic'" > If present in a script-src or default-src directive, it has two > main effects: > * host-source and scheme-source expressions, as well as the > "'unsafe-inline'" and "'self' keyword-sources will be ignored > when loading script. Resolves: #101460 Releases: main, 12.4 Change-Id: I95f696b92b60efef42367c2536b93b855a52522d Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/80207 Reviewed-by:
-
Benjamin Franzke authored
The importmap tag `backend.contextmenu` was added to EXT:form because it registers a context-menu item provider (for files). It turns out that the provider is based on the EXT:filelist provider and therefore doesn't require own javascript (`@typo3/filelist/context-menu-actions` is used). Resolves: #101445 Releases: main, 12.4 Change-Id: Ibdce39c971aae69889849b52847f62ce1298baf7 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/80216 Tested-by:
-
Stefan Bürk authored
Resolves: #101478 Releases: main, 12.4, 11.5 Change-Id: I40987e100686771365c30652dba5a13bfd509604 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/80214 Tested-by:
-
