- 07 May, 2022 2 commits
-
-
This was probably forgotten when adding native types to this class. Resolves: #97572 Related: #97205 Releases: main Change-Id: I6c11ba669bb152f731f81e245eb05ef8c88da867 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/74545 Tested-by:
Nikita Hovratov <nikita.h@live.de> Tested-by:
core-ci <typo3@b13.com> Tested-by:
Stefan Bürk <stefan@buerk.tech> Reviewed-by:
Nikita Hovratov <nikita.h@live.de> Reviewed-by:
Stefan Bürk <stefan@buerk.tech>
-
phpstan parallel execution is limited to a maximum of 5 processes to be a nice neighbor when executed during core CI runs. This however leads to longer execution times when executed on local developer machines with many available CPU's. This patch introduces two separated phpstan config file entrypoints, one for local and one for CI. They are automatically selected in runTests.sh based on existence of a CI env variable. Resolves: #97571 Releases: main, 11.5 Change-Id: I14bf13412de7a2471703e7982dad4030ae6ba735 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/74536 Tested-by:
core-ci <typo3@b13.com> Tested-by:
Stefan Bürk <stefan@buerk.tech> Tested-by:
Nikita Hovratov <nikita.h@live.de> Tested-by:
Oliver Klee <typo3-coding@oliverklee.de> Tested-by:
Christian Kuhn <lolli@schwarzbu.ch> Reviewed-by:
Stefan Bürk <stefan@buerk.tech> Reviewed-by:
Oliver Klee <typo3-coding@oliverklee.de> Reviewed-by:
Christian Kuhn <lolli@schwarzbu.ch>
-
- 06 May, 2022 10 commits
-
-
Latest docker:20.10-dind (currently 20.10.15) triggers an error on CI: Error response from daemon: failed to create shim task: OCI runtime create failed: runc create failed: unable to start container process: unable to apply cgroup configuration: mkdir /sys/fs/cgroup/rdma/docker: permission denied: unknown The image is the 'sub-container' in CI on runner hosts, that runs all the test images like the php and database images. This error *may* have to do with sysbox, which is a security layer on runner hosts to separate CI jobs from each other. We however currently don't know exactly what is going on. For the time being, we pin the dind image to its previous verion docker:20.10.14-dind Change-Id: Ie59be69680e1f444c115f2249ca8709bbfdd1e3e Releases: main, 11.5, 10.4 Resolves: #97570 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/74541 Tested-by:
core-ci <typo3@b13.com> Tested-by:
Christian Kuhn <lolli@schwarzbu.ch> Tested-by:
Stefan Bürk <stefan@buerk.tech> Reviewed-by:
Christian Kuhn <lolli@schwarzbu.ch> Reviewed-by:
Stefan Bürk <stefan@buerk.tech>
-
This patch is cleaning up a lot of loose ends in the svg tree component. It aims to make the component look more clean and polished. - Unify focus styling with selected, hover and versioning - Ensure the whole node looks focused instead of only some text - Replace expand icon to match core icon styling - Hide expand icon if page tree is stopped - Move trigger to enter stopped pagetree to the front - Remove blurry lines for background borders - Make the edit input fields use as much space as is available - Load common icons always on setup to avoid node refresh on initial loads - Reposition the locked icon (another editor is editing) - Correct keyboard navigation behaviour and focus setting - Keep focus after leaving the edit mode without changes - Adjust positioning to rely on calculation instead of hardcoded values - Align drag delete target - Make drag positions more visible to the user when moving a page - Improve styling of tooltips when dragging elements - Add border to select tree component to make scroll boundaries visible Affected Areas: - PageTree - FileTree - SelectTree Releases: main Resolves: #97344 Change-Id: I6034e0cbc1079b93b24c5c1a99e0299946874b2f Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/74203 Tested-by:
Benjamin Franzke <bfr@qbus.de> Tested-by:
core-ci <typo3@b13.com> Tested-by:
Georg Ringer <georg.ringer@gmail.com> Reviewed-by:
Benjamin Franzke <bfr@qbus.de> Reviewed-by:
Georg Ringer <georg.ringer@gmail.com>
-
Stefan Bürk authored
'doctrine/dbal' 3.x deprecated the 'getSchemaManager()' method in favour of the 'createSchemaManager()' method on connections. This has been aligned in core with the doctrine/dbal raise in #96287. However, two occurances with corresponding todo comments has been overseen. This patch replace the calls and solves the comments of the left-overs. Additionally, unrelated wrapper method in ext:install database checks has aligned to the same name to avoid confusion when looking for the deprecated method. Resolves: #97552 Related: #96287 Releases: main Change-Id: If73e9439acdcbf89db968ba1e6db1c4d60513861 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/74524 Tested-by:
core-ci <typo3@b13.com> Tested-by:
Benni Mack <benni@typo3.org> Tested-by:
Christian Kuhn <lolli@schwarzbu.ch> Tested-by:
Stefan Bürk <stefan@buerk.tech> Reviewed-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>
-
Also, first class callable syntax is added for the csvValues method. Resolves: #97207 Related: #97210 Related: #97372 Releases: main Change-Id: Ib37a497c8df38fa9ef2f071c10439de4d269b25c Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/74286 Tested-by:
Nikita Hovratov <nikita.h@live.de> Tested-by:
core-ci <typo3@b13.com> Tested-by:
Stefan Bürk <stefan@buerk.tech> Reviewed-by:
Nikita Hovratov <nikita.h@live.de> Reviewed-by:
Oliver Klee <typo3-coding@oliverklee.de> Reviewed-by:
Stefan Bürk <stefan@buerk.tech>
-
Nikita Hovratov authored
In order to fix select fields overflowing into each other when they have very long option titles, the CSS rule "overflow: hidden" has been added. As it seems, this wasn't even necessary as the rule "min-width: 120px" already does the trick. The CSS rule "overflow: hidden" is now removed, which enables the accessibility box-shadows from bootstrap 5 again. Resolves: #97565 Related: #94688 Releases: main, 11.5 Change-Id: Ifd8495430ecdc404fa1dbebe9633ef5f401a6af6 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/74530 Tested-by:
core-ci <typo3@b13.com> Tested-by:
Benjamin Franzke <bfr@qbus.de> Tested-by:
Christian Kuhn <lolli@schwarzbu.ch> Tested-by:
Nikita Hovratov <nikita.h@live.de> Reviewed-by:
Benjamin Franzke <bfr@qbus.de> Reviewed-by:
Christian Kuhn <lolli@schwarzbu.ch> Reviewed-by:
Nikita Hovratov <nikita.h@live.de>
-
Also, clean up related tests' doc blocks. Resolves: #97382 Related: #97210 Related: #97372 Releases: main Change-Id: I5f9a99f35bf2e1afbbdeeec1cc1de98157e581ea Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/74316 Tested-by:
Nikita Hovratov <nikita.h@live.de> Tested-by:
core-ci <typo3@b13.com> Tested-by:
Stefan Bürk <stefan@buerk.tech> Reviewed-by:
Nikita Hovratov <nikita.h@live.de> Reviewed-by:
Oliver Klee <typo3-coding@oliverklee.de> Reviewed-by:
Stefan Bürk <stefan@buerk.tech>
-
This change adapts ExtendedFileUtility logs to use interpolation from PSR-3 instead of %s to identify variables in log_details properly later-on. This makes all log messages from ExtendedFileUtility PSR-3 compliant. In addition, details_nr is not in use anymore since TYPO3 v6, so the arbitrary ID is removed when logging. Also, all messages do not contain a "!" at the end anymore, as there is no need to emphasize errors through the log message itself (that's why there is type "Error" for logging). Same goes for "." in order to be consistent. Resolves: #97558 Releases: main Change-Id: I39d474b7c7edd3707d9959b5e750c64f23bcb1ad Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/74527 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>
-
Also, a little cleanup in DefaultFunctionsProvider. The method getCompatVersionFunction() was calling convertVersionNumberToInteger() with an inconsistent type as a result of TypoScript parsing. We therefore need to force-cast it to a string, which is what convertVersionNumberToInteger() wants. Resolves: #97383 Related: #97210 Related: #97372 Releases: main Change-Id: Id47e42fd70d8ec89f6255616a26e6666c461efe7 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/74317 Tested-by:
Nikita Hovratov <nikita.h@live.de> Tested-by:
core-ci <typo3@b13.com> Tested-by:
Stefan Bürk <stefan@buerk.tech> Reviewed-by:
Nikita Hovratov <nikita.h@live.de> Reviewed-by:
Oliver Klee <typo3-coding@oliverklee.de> Reviewed-by:
Stefan Bürk <stefan@buerk.tech>
-
Adds type declarations to DebugUtility. Fixes some calls to these methods to avoid spurious nulls. Improves documentation in renderDump() to explain why certain parameters are nullable. Resolves: #97208 Related: #97210 Related: #97372 Releases: main Change-Id: Iecad6ef3e7432cf1f4123a4cdb1da15f14d04394 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/73991 Reviewed-by:
Oliver Klee <typo3-coding@oliverklee.de> Reviewed-by:
Nikita Hovratov <nikita.h@live.de> Reviewed-by:
Stefan Bürk <stefan@buerk.tech> Tested-by:
Nikita Hovratov <nikita.h@live.de> Tested-by:
core-ci <typo3@b13.com> Tested-by:
Stefan Bürk <stefan@buerk.tech>
-
Nikita Hovratov authored
This adds two recently merged utility classes with changed method signatures to the breaking rst file. Resolves: #97567 Related: #97378 Related: #97381 Releases: main Change-Id: Id50f4806346e4fdd1786f3d93a09d05ebc1e1790 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/74532 Tested-by:
Oliver Klee <typo3-coding@oliverklee.de> Tested-by:
core-ci <typo3@b13.com> Tested-by:
Nikita Hovratov <nikita.h@live.de> Reviewed-by:
Oliver Klee <typo3-coding@oliverklee.de> Reviewed-by:
Nikita Hovratov <nikita.h@live.de>
-
- 05 May, 2022 7 commits
-
-
Document https://review.typo3.org/c/Packages/TYPO3.CMS/+/73048 in the extensions manual Resolves: #97404 Releases: main Change-Id: If241f826fcc8972877eb9b4f6c7281a814b65ef9 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/74331 Tested-by:
Björn Jacob <bjoern.jacob@tritum.de> Tested-by:
core-ci <typo3@b13.com> Tested-by:
Annett Jähnichen <mcmietz@web.de> Tested-by:
Jonas Eberle <flightvision@googlemail.com> Tested-by:
Benni Mack <benni@typo3.org> Tested-by:
Stefan Bürk <stefan@buerk.tech> Tested-by:
Lina Wolf <112@linawolf.de> Reviewed-by:
Björn Jacob <bjoern.jacob@tritum.de> Reviewed-by:
Jonas Eberle <flightvision@googlemail.com> Reviewed-by:
Benni Mack <benni@typo3.org> Reviewed-by:
Stefan Bürk <stefan@buerk.tech> Reviewed-by:
Lina Wolf <112@linawolf.de>
-
This change deprecates the following properties: ContentObjectRenderer->lastTypoLinkUrl ContentObjectRenderer->lastTypoLinkTarget ContentObjectRenderer->lastTypoLinkLD (LD stands for "LinkData") All this information is available in the newly introduced LinkResultInterface, that can be built with the LinkFactory directly, or via ContentObjectRenderer->createLink(). Resolves: #97549 Releases: main Change-Id: I80b21d54e75714ecc148cece35e064720a57df44 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/74510 Reviewed-by:
Oliver Bartsch <bo@cedev.de> Reviewed-by:
Jochen <rothjochen@gmail.com> Reviewed-by:
Stefan Bürk <stefan@buerk.tech> Tested-by:
core-ci <typo3@b13.com> Tested-by:
Jochen <rothjochen@gmail.com> Tested-by:
Stefan Bürk <stefan@buerk.tech>
-
Currently, a user can reset their password, but never gets the information what their username is. Many users forget their login username and reset their password using their email address. This patch adds the username in brackets after the first and last name to the salutation in the recovery email. Resolves: #96705 Releases: main, 11.5 Change-Id: Idb18e13e453edbab8645b079b7573e91191e479b Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/74256 Tested-by:
Anja Leichsenring <aleichsenring@ab-softlab.de> Tested-by:
core-ci <typo3@b13.com> Tested-by:
Stefan Bürk <stefan@buerk.tech> Reviewed-by:
Anja Leichsenring <aleichsenring@ab-softlab.de> Reviewed-by:
Stefan Bürk <stefan@buerk.tech>
-
This change adapts DataHandler logs to use interpolation from PSR-3 instead of %s to identify variables in log_details properly later-on. This makes all log messages from DataHandler and Workspaces PSR-3 compliant. In addition, all messages do not contain a "!" at the end anymore, as there is no need to emphasize errors through the log message itself (that's why there is type "Error" for logging). Same goes for "." in order to be consistent. Resolves: #97556 Releases: main Change-Id: Ia509b6e121be64cf89d111bdc260332d343b164c Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/74526 Tested-by:
core-ci <typo3@b13.com> Tested-by:
Jochen <rothjochen@gmail.com> Tested-by:
Christian Kuhn <lolli@schwarzbu.ch> Tested-by:
Georg Ringer <georg.ringer@gmail.com> Reviewed-by:
Jochen <rothjochen@gmail.com> Reviewed-by:
Christian Kuhn <lolli@schwarzbu.ch> Reviewed-by:
Georg Ringer <georg.ringer@gmail.com>
-
a || (b && !a) can be simplified to a || b (because the second sub-condition is only evaluated if a is false). Resolves: #97421 Releases: main, 11.5 Change-Id: I04d62fd46736bb4a0b9dfce54851aaa8f99d20e1 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/74345 Tested-by:
core-ci <typo3@b13.com> Tested-by:
Jonas Eberle <flightvision@googlemail.com> Tested-by:
Stefan Bürk <stefan@buerk.tech> Reviewed-by:
Jonas Eberle <flightvision@googlemail.com> Reviewed-by:
Stefan Bürk <stefan@buerk.tech>
-
Stefan Bürk authored
Raise phpstan to newest release, which find more things in a proper way. This includes some opcache and autoloading fixes which may hopefully help with lately occurrence of long-running phpstan ci runs. Used commands: > composer req --dev phpstan/phpstan:^1.6.7 > Build/Scripts/runTests.sh -s clean > Build/Scripts/runTests.sh -s phpstanGenerateBaseline Resolves: #97551 Releases: main, 11.5 Change-Id: If7493bc4138e99ff173047cba358d3be43cf2357 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/74523 Tested-by:
core-ci <typo3@b13.com> Tested-by:
Simon Schaufelberger <simonschaufi+typo3@gmail.com> Tested-by:
Benni Mack <benni@typo3.org> Tested-by:
Oliver Klee <typo3-coding@oliverklee.de> Tested-by:
Stefan Bürk <stefan@buerk.tech> Reviewed-by:
Simon Schaufelberger <simonschaufi+typo3@gmail.com> Reviewed-by:
Benni Mack <benni@typo3.org> Reviewed-by:
Oliver Klee <typo3-coding@oliverklee.de> Reviewed-by:
Stefan Bürk <stefan@buerk.tech>
-
Benni Mack authored
This option is obsolete nowadays, as PSR-7 is setting the headers at the end of the script / when emitting a Response, this way the header can be modified at any time. Resolves: #97550 Releases: main Change-Id: I0602e4fd0a20461d49a554f66f36442894e96fd4 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/74518 Tested-by:
core-ci <typo3@b13.com> Tested-by:
Georg Ringer <georg.ringer@gmail.com> Tested-by:
Oliver Bartsch <bo@cedev.de> Tested-by:
Josef Glatz <josefglatz@gmail.com> Tested-by:
Benni Mack <benni@typo3.org> Reviewed-by:
Georg Ringer <georg.ringer@gmail.com> Reviewed-by:
Oliver Bartsch <bo@cedev.de> Reviewed-by:
Benni Mack <benni@typo3.org>
-
- 04 May, 2022 9 commits
-
-
"AbstractOEmbedHelper::getOEmbedData()" requests data and decode it as json. json_decode() may fail and thus having a invalid type decoded. In the complete chain this lead to a "access array offset on value type bool" warning. This patch checks the decoded value before returning, thus ensure valid return type is "array" or "null". Furthermore this avoids a signature change of the corresponding method "AbstractOEmbedHelper::getOEmbedData()". Additionally this paves the way for adding proper return type in a dedicated patch. Streamline usage in VimeoHelper along the way. Resolves: #97428 Releases: main, 11.5 Change-Id: Icda6b2baa88ddcbf070ceae75eafcb2e9ccb41f1 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/74353 Reviewed-by:
Oliver Klee <typo3-coding@oliverklee.de> Reviewed-by:
Oliver Bartsch <bo@cedev.de> Reviewed-by:
Stefan Bürk <stefan@buerk.tech> Tested-by:
core-ci <typo3@b13.com> Tested-by:
Oliver Bartsch <bo@cedev.de> Tested-by:
Stefan Bürk <stefan@buerk.tech>
-
Benni Mack authored
The BackendUtility method "getPreviewUrl" along with its hooks and the helper method "ADMCMD_previewCmds" has been marked as deprecated in favor of having everything in PreviewUriBuilder. This class now contains two new Events for manipulating the generated preview URI. In addition, the unused "viewUrl" property of EditDocumentController is removed. The language parameter is now additionally encapsulated in a separate property. Resolves: #97544 Related: #91123 Releases: main Change-Id: I5d6fbc293cd5bbfa1d4b522eb2f02bb6c763bb1a Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/74499 Tested-by:
core-ci <typo3@b13.com> Tested-by:
Oliver Bartsch <bo@cedev.de> Tested-by:
Nikita Hovratov <nikita.h@live.de> Tested-by:
Benni Mack <benni@typo3.org> Reviewed-by:
Oliver Bartsch <bo@cedev.de> Reviewed-by:
Nikita Hovratov <nikita.h@live.de> Reviewed-by:
Benni Mack <benni@typo3.org>
-
Increase code coverage with some more tests. Resolves: #97444 Related: #97303 Releases: main, 11.5 Change-Id: Icaa20aea45bc34b5618bfc55710014d928444510 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/74371 Tested-by:
core-ci <typo3@b13.com> Tested-by:
Oliver Klee <typo3-coding@oliverklee.de> Tested-by:
Stefan Bürk <stefan@buerk.tech> Reviewed-by:
Oliver Klee <typo3-coding@oliverklee.de> Reviewed-by:
Stefan Bürk <stefan@buerk.tech>
-
This patches replaces several wrong or incomplete type annotations in FormDefinition. Additionally, the `$renderables` property is now explicitly declared and annotated as `array<int, Page>` since it only contains pages. Resolves: #97545 Releases: main, 11.5 Change-Id: I182a861276c65142be7266ec7b10420f8532372c Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/74508 Tested-by:
core-ci <typo3@b13.com> Tested-by:
Björn Jacob <bjoern.jacob@tritum.de> Tested-by:
Oliver Hader <oliver.hader@typo3.org> Reviewed-by:
Björn Jacob <bjoern.jacob@tritum.de> Reviewed-by:
Oliver Hader <oliver.hader@typo3.org>
-
Currently acceptance tests do not work on arm64 (Apple M1) due to the lack of support in the selenium docker image. This has been fixed by detecting arm64 and switch to arm64 compatible image (seleniarm/standalone-chromium). Resolves: #97541 Releases: main, 11.5, 10.4 Change-Id: I531e7e1d7f0f11a1c6d850699eee3b4a9aa3e5d0 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/74502 Tested-by:
core-ci <typo3@b13.com> Tested-by:
Josef Glatz <josefglatz@gmail.com> Tested-by:
Oliver Bartsch <bo@cedev.de> Reviewed-by:
Josef Glatz <josefglatz@gmail.com> Reviewed-by:
André Buchmann <andy.schliesser@gmail.com> Reviewed-by:
Oliver Bartsch <bo@cedev.de>
-
Oliver Bartsch authored
Namespaces in the extension scanner files must not contain a leading backslash. Resolves: #97547 Releases: main, 11.5 Change-Id: Idb608b3e02c8901972a068c531ee681e8b2dd49e Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/74511 Tested-by:
core-ci <typo3@b13.com> Tested-by:
Benjamin Franzke <bfr@qbus.de> Tested-by:
Josef Glatz <josefglatz@gmail.com> Tested-by:
Oliver Bartsch <bo@cedev.de> Reviewed-by:
Benjamin Franzke <bfr@qbus.de> Reviewed-by:
Josef Glatz <josefglatz@gmail.com> Reviewed-by:
Oliver Bartsch <bo@cedev.de>
-
Benni Mack authored
The introduced LinkResult object and its functionality has had a fallback built via ArrayAccess for the LinkBuilderInterface to be compatible with array-ish LinkResult objects to access the previously returned array until TYPO3 v11 LTS. This change now removes the legacy layer, as TYPO3 only works with the LinkResult object directly everywhere. Resolves: #97546 Related: #96107 Related: #96317 Releases: main Change-Id: I733c85359b2f8b5886626a67fe106806f5405c7b Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/74509 Reviewed-by:
Oliver Hader <oliver.hader@typo3.org> Reviewed-by:
Georg Ringer <georg.ringer@gmail.com> Reviewed-by:
Oliver Bartsch <bo@cedev.de> Reviewed-by:
Benni Mack <benni@typo3.org> Tested-by:
core-ci <typo3@b13.com> Tested-by:
Georg Ringer <georg.ringer@gmail.com> Tested-by:
Oliver Bartsch <bo@cedev.de> Tested-by:
Benni Mack <benni@typo3.org>
-
Switchable controller actions are removed with TYPO3 12.0. The deprecation note is adjusted to reflect it. Resolves: #97523 Related: #89463 Related: #96282 Releases: main, 11.5, 10.4 Change-Id: Iad04bbb573f2b87ce87a8faabe25c97a8812f937 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/74496 Tested-by:
Oliver Klee <typo3-coding@oliverklee.de> Tested-by:
core-ci <typo3@b13.com> Tested-by:
Simon Schaufelberger <simonschaufi+typo3@gmail.com> Tested-by:
Stefan Bürk <stefan@buerk.tech> Tested-by:
Josef Glatz <josefglatz@gmail.com> Tested-by:
Oliver Bartsch <bo@cedev.de> Reviewed-by:
Oliver Klee <typo3-coding@oliverklee.de> Reviewed-by:
Simon Schaufelberger <simonschaufi+typo3@gmail.com> Reviewed-by:
Stefan Bürk <stefan@buerk.tech> Reviewed-by:
Josef Glatz <josefglatz@gmail.com> Reviewed-by:
Oliver Bartsch <bo@cedev.de>
-
Annett Jähnichen authored
The patch simplifies and optimizes the readability of the page module interface. It is based on the UX Team's concept of improving user experience for editors: https://typo3.org/article/structured-content-initiative-feedback-wanted Included changes: * refactored and optimised underlying CSS for future adaptions * visually simplified content element boxes and their header buttons * visually differenciated hidden content elements with better opacity and dotted border * placed the "new content element" buttons centered in preparation for further concept implementations in dedicated patches * added a new button for content element context menu in the content element header right button bar This patch was initiated by the UX Team and implemented by Benjamin Kott. Resolves: #97306 Resolves: #95247 Related: #94629 Releases: main Change-Id: I9b4e21a7fc68ebcb17d18f65bf00096f3a295cfb Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/74254 Tested-by:
Benjamin Franzke <bfr@qbus.de> Tested-by:
core-ci <typo3@b13.com> Tested-by:
Georg Ringer <georg.ringer@gmail.com> Tested-by:
Michael Telgkamp <michael.telgkamp@mindscreen.de> Tested-by:
Annett Jähnichen <mcmietz@web.de> Reviewed-by:
Benjamin Franzke <bfr@qbus.de> Reviewed-by:
Georg Ringer <georg.ringer@gmail.com> Reviewed-by:
Michael Telgkamp <michael.telgkamp@mindscreen.de> Reviewed-by:
Annett Jähnichen <mcmietz@web.de>
-
- 03 May, 2022 5 commits
-
-
When TSFE is fetching the $page record from PageRepository, it checks the $TSFE->id parameter, which was originally the GET parameter. In TYPO3 v9, when "pages_language_overlay" was merged into "pages", an intermediate step was added to access translated pages via the GET parameter as well. This concept however was never needed, as the page ID of the default language was always resolved with Site Handling already in the PageRouter. The double-check in TSFE can be removed, as site handling has taken over since TYPO3 v10, and was never accessed since its introduction in TYPO3 v9 (mainly because the code was added with the migration of "pages_language_overlay" into "pages" into TYPO3 v9.0, whereas site handling was only added in TYPO3 v9.2+). Resolves: #97533 Releases: main Change-Id: I290200f90b4d23589a4f7aec33a0516ecfab7739 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/74500 Tested-by:
core-ci <typo3@b13.com> Tested-by:
Oliver Bartsch <bo@cedev.de> Tested-by:
Benjamin Franzke <bfr@qbus.de> Reviewed-by:
Oliver Bartsch <bo@cedev.de> Reviewed-by:
Benjamin Franzke <bfr@qbus.de>
-
Calling a site using http://example.org/index.php/invalid/ leads to ~/index.php/whatever/ being used as internal script path, which causes errors or internal side-effects. This behavior seems to occur only on web-servers using Apache with PHP-CGI or PHP-FPM, using PHP setting `cgi.fix_pathinfo = 1`. In case `cgi.fix_pathinfo` is enabled, the current script name is retrieved from `$_SERVER['SCRIPT_FILENAME']` instead. Resolves: #97543 Releases: main, 11.5, 10.4 Change-Id: Ia5f6b705253d42d4fc409b90b21d0363c4b97974 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/74504 Tested-by:
core-ci <typo3@b13.com> Tested-by:
Stefan Bürk <stefan@buerk.tech> Tested-by:
Benjamin Franzke <bfr@qbus.de> Reviewed-by:
Markus Klein <markus.klein@typo3.org> Reviewed-by:
Stefan Bürk <stefan@buerk.tech> Reviewed-by:
Benjamin Franzke <bfr@qbus.de>
-
FileRepository::findByRelation does not apply any workspace context when invoked in backend context. Resolves: #59334 Releases: main, 11.5 Change-Id: I98f3f0640773c7d87879adb2b75cdb92a410bbc3 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/74185 Tested-by:
André Kraus <info@andrekraus.com> Tested-by:
core-ci <typo3@b13.com> Tested-by:
Andreas Fernandez <a.fernandez@scripting-base.de> Tested-by:
Oliver Hader <oliver.hader@typo3.org> Reviewed-by:
André Kraus <info@andrekraus.com> Reviewed-by:
Benni Mack <benni@typo3.org> Reviewed-by:
Oliver Hader <oliver.hader@typo3.org>
-
Benni Mack authored
The option searchSkipExtendToSubpagesChecking in Indexed search is removed in favor of using a better functionality of getTreeList() which has been implemented in TYPO3 Core, considering "extendToSubpages" at any times. Resolves: #97530 Releases: main Change-Id: Iabe82f2846aaa6d586ef30c0128e957b24531562 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/74377 Tested-by:
Oliver Bartsch <bo@cedev.de> 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 Bartsch <bo@cedev.de> Reviewed-by:
Stefan Bürk <stefan@buerk.tech> Reviewed-by:
Benni Mack <benni@typo3.org>
-
Benni Mack authored
Various methods can now use the Context API, making the dependency to TSFE less relevant for future changes. Resolves: #97531 Releases: main Change-Id: I3daaddd9684e36a5bf58343cd1fdc43d29713a3a Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/73910 Tested-by:
Oliver Bartsch <bo@cedev.de> Tested-by:
core-ci <typo3@b13.com> Tested-by:
Susanne Moog <look@susi.dev> Tested-by:
Benni Mack <benni@typo3.org> Reviewed-by:
Oliver Bartsch <bo@cedev.de> Reviewed-by:
Susanne Moog <look@susi.dev> Reviewed-by:
Benni Mack <benni@typo3.org>
-
- 02 May, 2022 2 commits
-
-
The method parseDataStructureByIdentifier defines that files can also be loaded with the "EXT:" prefix. This is currently not supported and not documented, thus should be removed. The only valid syntax is with "FILE:EXT:" Resolves: #92483 Releases: main, 11.5 Change-Id: I5156c40215753566251c56ca0cbd9b94b66ce4a0 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/74441 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>
-
rowCount() on result sets for SELECT queries are not reliable across all DBMS, which doctrine/dbal comments in the docblock. See: https://github.com/doctrine/dbal/blob/110ab13/src/Driver/Result.php#L77 Because of "rowCount()"'s unreliable behaviour for some dbms like sqlite, this should be avoided to ensure cross dbms support. '\TYPO3\CMS\IndexedSearch\Domain\Repository\IndexSearchRepository' uses 'rowCount()' in 'doSearch()' which fails on sqlite, making it impossible to get any result in the frontend search. Doing a count() query as replacement is not possible and needs a bigger architectural change, which is not feasable as bugfix. Thus fallback to retrieve the result into an array and counting the array is used as a intermediate solution to ensure index_search is working across all supported database systems. The architectural change will be done in a dedicated patch. Resolves: #97460 Releases: main, 11.5 Change-Id: Id7afa9f95fefbba07acb7c0dda1267dd2fce92db Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/74380 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>
-
- 29 Apr, 2022 5 commits
-
-
Streamline the first step of the duplicate-form-wizard to the new layout of new-form-wizard. Resolves: #97506 Releases: main Change-Id: I31482b6cf8c188ece8821aa3a4b2e1cf0da4fe09 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/74480 Tested-by:
Björn Jacob <bjoern.jacob@tritum.de> Tested-by:
core-ci <typo3@b13.com> Tested-by:
waldhacker <hello@waldhacker.dev> Reviewed-by:
Björn Jacob <bjoern.jacob@tritum.de> Reviewed-by:
waldhacker <hello@waldhacker.dev>
-
If a field contains a label that is too long, this creates an ugly line break in the form element header. And also the label of the field is displayed redundantly below the header. So we output the field type instead of the label. Resolves: #97488 Releases: main Change-Id: Ia33b0b41697ecf4cb43bad51069596322fa4a704 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/74489 Tested-by:
core-ci <typo3@b13.com> Tested-by:
Andreas Steiger <typo3@andreassteiger.de> Tested-by:
Björn Jacob <bjoern.jacob@tritum.de> Tested-by:
waldhacker <hello@waldhacker.dev> Reviewed-by:
Andreas Steiger <typo3@andreassteiger.de> Reviewed-by:
Björn Jacob <bjoern.jacob@tritum.de> Reviewed-by:
waldhacker <hello@waldhacker.dev>
-
Core requirements have been changed and publicly announced with: https://typo3.org/article/new-system-requirements-for-upcoming-typo3-v12 This patch removes supportive code for MSSQL due dropped support for this dbmns. MSSQL specific report and status checks are removed with #97411. Resolves: #97462 Related: #96553 Related: #97411 Releases: main Change-Id: I131805157b7780a4de6d0f3ab8e9d0d6883b7872 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/74382 Tested-by:
core-ci <typo3@b13.com> Tested-by:
Benni Mack <benni@typo3.org> Tested-by:
Oliver Bartsch <bo@cedev.de> Tested-by:
Andreas Fernandez <a.fernandez@scripting-base.de> Reviewed-by:
Benni Mack <benni@typo3.org> Reviewed-by:
Oliver Bartsch <bo@cedev.de> Reviewed-by:
Oliver Klee <typo3-coding@oliverklee.de> Reviewed-by:
Andreas Fernandez <a.fernandez@scripting-base.de>
-
This patch adds a new field called "fragment" to the EXT:form Redirect finisher. It will be appended to the redirect URL (e.g. #c127). The fragment can be either a content element identifier or a custom fragment such as `foo`. Resolves: #83912 Releases: main Change-Id: I724e378894782215cc7db21160daef930d0cfdb2 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/73721 Tested-by:
core-ci <typo3@b13.com> Tested-by:
Björn Jacob <bjoern.jacob@tritum.de> Tested-by:
Mathias Brodala <mbrodala@pagemachine.de> Tested-by:
waldhacker <hello@waldhacker.dev> Reviewed-by:
Björn Jacob <bjoern.jacob@tritum.de> Reviewed-by:
Mathias Brodala <mbrodala@pagemachine.de> Reviewed-by:
waldhacker <hello@waldhacker.dev>
-
Adds a label to the delete button and improves the styling. Exceptions are the delete buttons in the validators and in the form finishers. As part of an upcoming patch the finishers will be expanded and the styling of the delete buttons will be adapted to the layout. Resolves: #97487 Releases: main Change-Id: I3d068cc883c7b74c3ceed1ab7a1d205ea795c53a Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/74472 Tested-by:
Björn Jacob <bjoern.jacob@tritum.de> Tested-by:
core-ci <typo3@b13.com> Tested-by:
Andreas Steiger <typo3@andreassteiger.de> Tested-by:
waldhacker <hello@waldhacker.dev> Reviewed-by:
Björn Jacob <bjoern.jacob@tritum.de> Reviewed-by:
Andreas Steiger <typo3@andreassteiger.de> Reviewed-by:
waldhacker <hello@waldhacker.dev>
-