- Aug 26, 2021
-
-
Oliver Hader authored
composer req typo3/html-sanitizer:^2.0.10;\ composer req typo3/html-sanitizer:^2.0.10 \ -d typo3/sysext/core --no-update Resolves: #95000 Releases: master, 11.3, 10.4, 9.5 Change-Id: Ia2170f6bd6f3bace862fac124ef8cc2966d35171 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/70763 Tested-by:
Christian Kuhn <lolli@schwarzbu.ch> Tested-by:
core-ci <typo3@b13.com> Tested-by:
Andreas Fernandez <a.fernandez@scripting-base.de> Reviewed-by:
-
Oliver Bartsch authored
The TOTP info modal displays the OTP auth url along with a description. This content however is added to the modal body by a callback. To prevent the "modal content empty" warning from being shown, the modal is now initialized with an empty string as content. Resolves: #94999 Releases: master Change-Id: I0a8c274466d4f62da8edb26a4c2b547b66be3ba6 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/70761 Tested-by:
Jochen <rothjochen@gmail.com> Tested-by:
Oliver Bartsch <bo@cedev.de> Reviewed-by:
-
- Aug 25, 2021
-
-
Wolfgang Klinger authored
This adds the column selector component, introduced in #94218, to the filelist module. As known from recordlist, it can be used to manage the fields to be displayed for each file / folder. Fields to be selected are the combination of special fields (e.g. read/write permissions), sys_file fields and sys_file_metadata fields. Resolves: #84184 Releases: master Change-Id: Ie1369e616f35dc4a92e90409eba202aa4afe15f3 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/56073 Tested-by:
Benni Mack <benni@typo3.org> Reviewed-by:
-
Frank Naegler authored
This patch removes the LOCK TABLE permission check from the PermissionsCheck class that is part of the installer. TYPO3 Core doesn't need the LOCK TABLE permission, hence the check can be removed. Resolves: #94978 Related: #78885 Releases: master, 10.4 Change-Id: I41a07583706fdb9d51bca5b1e74f64f56e38eebe Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/70738 Tested-by:
Stephan Großberndt <stephan.grossberndt@typo3.org> Tested-by:
-
Jochen Roth authored
Now that we have frontend tests in the Backend suite the name Backend for acceptance suite does not fit anymore. So the suite name was changed from Backend to Application. Resolves: #94864 Releases: master Change-Id: Ib52053f6dbbfad9b7fef1ff11e81110cffa010b2 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/70550 Tested-by:
Anja Leichsenring <aleichsenring@ab-softlab.de> Reviewed-by:
-
Daniel Siepmann authored
It can be beneficial to suppress caching using xml2arrayProcess() directly instead of the cache enabled wrapper xml2array() in some situations. The patch makes the worker method public, adapts tests to mock less and moves a data changing detail from xml2array() to xml2arrayProcess(). Resolves: #94993 Releases: master, 10.4 Change-Id: I2b0347ebb1a293d2826b2d13b00f21b56568195c Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/70757 Tested-by:
Daniel Siepmann <coding@daniel-siepmann.de> Tested-by:
crell <larry@garfieldtech.com> Reviewed-by:
-
Christian Kuhn authored
The fluid view classes have a rather complex inheritance and dependency chain, especially when looking at the various extbase related classes, too. One step is TYPO3Fluid\Fluid\View\TemplateView: This empty class adds no value, TYPO3\CMS\Fluid\View\AbstractTemplateView can extend TYPO3Fluid\Fluid\View\AbstractTemplateView instead, without functional change. Resolves: #94973 Releases: master Change-Id: I41888b371d69ca70deb30ff0d47028d4c8e0e6ff Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/70734 Tested-by:
-
Frank Naegler authored
This patch adds the possibility to access site configuration with a placeholder in TCA's `foreign_table_where` query. To access a configuration value the following syntax is available: * ###SITE:rootPageId### * ###SITE:foo.bar.baz### (array path notation) Resolves: #94662 Releases: master Change-Id: I2047085d20c914960bbcd88df5b4c7a7f8d53282 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/70126 Tested-by:
Wouter Wolters <typo3@wouterwolters.nl> Reviewed-by:
-
Benni Mack authored
This change is a precursor to https://review.typo3.org/c/Packages/TYPO3.CMS/+/69174 This change effectively: * Centralizes checks to PackageStates.php in Bootstrap * Centralizes some specific Install-Tool functionality in PackageManager * Adds a cache identifier for ext_localconf/TCA/ext_tables.php from the packagemanager The last adaption allows to have custom cache identifier when the active packages have been updated. Resolves: #94987 Releases: master Change-Id: Ibc34fe1084ee42ef0a86e2942c6aed3a345b7188 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/70752 Tested-by:
Helmut Hummel <typo3@helhum.io> Tested-by:
-
Uwe Trotzek authored
The workspace module gets an additional stage filter to reduce the amount of changes a user is facing. This is very helpful if you are dealing with a lot of changes and want to see only records which are in a certain stage e.g. "Ready to publish". Resolves: #91021 Releases: master Change-Id: I80a3fa5c00ff302c9b2257370bdeda328033c204 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/64197 Tested-by:
-
Benni Mack authored
TemplateView does not use AbstractView, and others do not need this as well, ViewInterface is the one people should rely on. Resolves: #94991 Releases: master Change-Id: Ie09d6dcb3644b4e383bf67e7043765227b622948 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/70726 Tested-by:
-
Helmut Hummel authored
This change allows TYPO3 to detect and trigger a deprecation of the Cache Manager and database connection not only during loading of ext_localconf.php, but as well during building of TCA and loading of ext_tables.php. Code executed in these places has two purposes: 1. Provide/ adapt configuration 2. Register extension APIs (icons, modules, etc.) Both purposes will eventually migrated to DI configuration, which currently already is executed in a "stateless" environment. To prepare extension authors that this change will happen, before we will be able to migrate all current use cases to be registered in DI configuration, usage of DB connection and cache manager is deprecated at these early boot extension points. Releases: master Resolves: #94979 Change-Id: If03910ddd10c06662e1c9f5bea179ff12d64c989 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/69509 Tested-by:
Benjamin Franzke <bfr@qbus.de> Tested-by:
-
Oliver Bartsch authored
Depending on the backend localization, the action labels might be "longer" than initially expected. To prevent the action bar from breaking into another row, the columns width is now automatically calculated while the fiter input is set to flex-grow to automatically use the remaining space. Resolves: #94988 Releases: master Change-Id: I7d19fab9cacb52695c8385258a00f9fef38df33a Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/70753 Tested-by:
-
Oliver Bartsch authored
System news records (sys_news) contain a `content` field, which features a richtext editor. Previously using the default configuration, having most of the available options enabled. Since system news are only displayed on the login screen and are only intended for simple information, most of the options are not relevant. Additionally, some of those options did not work at all, due to the missing frontend context (see e.g. #67556). To ease the use for administrators, a simplified RTE configuration is now used for the `content` field of sys_news records. In case administrators need additional options, this can as usual still be achieved by overriding the corresponding TCA configuration. Resolves: #94980 Releases: master Change-Id: Ib8825344b453a3d19c5cbcd96844be3c47408f9a Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/70742 Tested-by:
-
Oliver Hader authored
Having `additionalParams = &simple[id]=okay&simple[other]=other` with the following route enhancer configuration lead to some flaws: routeEnhancers: TestSimple: type: Simple routePath: '/simple/{simple_id}' _arguments: simple_id: 'simple/id' * generated URI contained `simple__other`, not resolving nesting (`/simple/okay?simple__other=other&cHash=...`) * `PageArguments::$staticArguments` contained parameter `simple_id` (unresolved to actual query parameter, incorrectly marked static) * other `PageArguments` properties contained `simple__other` (unresolved to actual query parameter, ignored nesting) Resolves: #91447 Releases: master, 10.4, 9.5 Change-Id: If96a6245f44a12d6e666d0ead8b1cc9cbbb43170 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/69227 Tested-by:Markus Klein <markus.klein@typo3.org> Tested-by:
Oliver Hader <oliver.hader@typo3.org> Reviewed-by:
-
- Aug 24, 2021
-
-
Wouter Wolters authored
The package symfony/filesystem is used in the class "CliEntryPoint" but has no direct dependency defined in main composer.json. Require the package to not break TYPO3 when packages that rely on symfony/filesystem get removed. composer req symfony/filesystem:^5.3.0 composer req symfony/filesystem:^5.3.0 -d typo3/sysext/core/ --no-update Resolves: #94982 Releases: master, 10.4 Change-Id: Ib116913a709fbc2e635f1a7882f86d6be4c4c704 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/70729 Tested-by:
-
Wouter Wolters authored
Since #94280 it's possible to declare strict types in ext_tables.php files. This patch does this through the core. Resolves: #94984 Related: #94280 Releases: master Change-Id: I8aee34f0f697992b9ce596dfdeb95926f1fd9c31 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/70745 Tested-by:
-
Oliver Bartsch authored
Since #94739 it's possible to declare strict types in ext_localconf.php files. This patch does this through the core. Resolves: #94981 Related: #94739 Releases: master Change-Id: I61a937a487d5d065daccc58ca997a1ffd80f9d35 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/70743 Tested-by:
-
Wouter Wolters authored
With using a global namespace the usage of use statements is not a problem anymore. Resolves: #94280 Releases: master Change-Id: I642b47b4257c22fa59a812763e5f92d5a0d2236a Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/70548 Tested-by:
-
Christoph Schwob authored
Remove duplicate styles but some overrides for e.g. border already needed. Changed class .card-content to .card-body Add card-size-medium to card on ElementInformation to use some empty space, cause there's only one card. Resolves: #94529 Releases: master Change-Id: I1e8d3cdfe8e622806eff4f1e52f3a2bad1740fad Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/70480 Tested-by:
-
Benni Mack authored
This change uses the native web component to render icons, instead of referencing them from Fluid. This change also fixes a small issue regarding language icons and integrity icons. Resolves: #94977 Releases: master Change-Id: I3f99139497a5f4691bea30b53e399a860b050090 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/70735 Tested-by:
-
Benni Mack authored
This change favors native <a> tags for internal TYPO3 links in TYPO3 Backend where a ViewHelper is not needed. Resolves: #94975 Releases: master Change-Id: Ib068aee4bb54e45b158bf79d6e5f0c0d8a597341 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/70736 Tested-by:
-
Oliver Bartsch authored
The workspace review features the collapse/expand functionality for record relations. An example is tt_content which may contain several sys_file_reference relations. They are included in the listing as well but collapsed by default. Since at least v10 those relations are always processed together. This means, in case an editor checks a parent record and executes an action, e.g. publish, the same action is also executed for the relations. However, the UX was previously not really helpful as this was not visible to the editor. Therefore, and to improve the UX in workspace review, those relations are now always "bound together". This means, when selecting the parent record all relations are selected, too. The same goes the other way round. When selecting a relation, all corresponding relations as well as the parent record are selected. As mentioned, this does not change any functionality as this process was already done "under the hood". This process is now just visualized for the editor. Resolves: #94404 Releases: master, 10.4 Change-Id: I28d2c9e67747dde5ffdfb36d24f8cc7f36c246a8 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/70712 Tested-by:
-
Oliver Bartsch authored
The multi record selection, introduced in #94906, is extended for keyboard shortcuts. They can be used to achieve similar results as by using the actions in the multi record selection dropdown. * `shift` key: check / uncheck the range between the last checked checkbox and the current one * `option` (Mac) or `ctrl` (Windows/Linux) key: Toggle the current selection Resolves: #94944 Related: #94906 Releases: master Change-Id: Ia5f6625e6b380d908bddfbf5849f2ea6233fe420 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/70688 Tested-by:
Mathias Brodala <mbrodala@pagemachine.de> Tested-by:
-
Oliver Bartsch authored
The TcaText data provider transforms a given database field value with the RteHtmlParser as soon as `enableRichtext` is set for the field in TCA. This means, the data provider did previously not checked, whether RTE is disabled for the user in general or for this specific field by configuration (e.g. page TSconfig). This therefore led, among others, to "normal" text being falsely wrapped with `<p>` tags. This is now fixed by checking all relevant settings before actually transforming the database field value. Resolves: #94915 Releases: master, 10.4 Change-Id: I5af4e9997c28269be959ea9e63e12cfee5e991af Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/70674 Tested-by:
Daniel Goerz <daniel.goerz@posteo.de> Tested-by:
-
Andrzej Kołbuc authored
Labels are rendered by CshViewHelper which do htmlspecialchars on it. Next step todo is change whole additional fields layout and prepare translations. Resolves: #93888 Releases: master, 10.4 Change-Id: I19d12951a030977384ee3a923823990fe97e9211 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/70536 Tested-by:
kharaf <k.haraf@macopedia.com> Tested-by:
Rudy Gnodde <rudy@famouswolf.com> Reviewed-by:
-
Wouter Wolters authored
The following core extensions are handled: EXT:reports EXT:redirects EXT:recycler EXT:recordlist EXT:opendocs EXT:linkvalidator Resolves: #94855 Releases: master Change-Id: Ifb6b2ca2f6c25dae2c3dab3d2d4ca6571312c9f4 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/70512 Tested-by:
-
- Aug 23, 2021
-
-
Oliver Bartsch authored
The extbase ObjectConverter - typically used when mapping request arguments to target domain models - needs more fine grained handling to drop ObjectManager usages. We missed some details with #94451. The patch takes care of this and adds an explanation together with possible future options to the codebase. This fixes deprecation log entries triggered by the belog module. Resolves: #94807 Related: #94451 Releases: master Change-Id: Ia1f1c15c1814d53b42d83213778029908332ede0 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/70445 Reviewed-by:
-
Alexander Nitsche authored
This is a regression introduced with #94057 that causes resetting excluded records to throw an exception in the export view, since the incoming parameter "exclude" could be of type array (= add excluded records) or integer (= include all records again), but was only checked for null or not null, which was ambiguous in the second case. For clear separation the reset of excluded records is now done via a new parameter "resetExclude". Resolves: #94949 Releases: master Change-Id: Iddb8d21e0f75cc24ef18dd2f851a479cb7dafb3b Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/70690 Tested-by:
Alexander Nitsche <typo3@alexandernitsche.com> Tested-by:
-
Oliver Hader authored
Avoids round trip through Fluid template, but instead assigns stylesheets and inline scripts directly via `PageRenderer`. Resolves: #94969 Releases: master Change-Id: Ic16c853fc8fcb26204c91b6347616513787fcd78 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/70731 Tested-by:
-
Alexander Nitsche authored
This is a regression introduced with #94772 that causes all actions related to export presets to fail because the controller checks $_GET instead of $_POST for incoming preset action parameters. Reduced redundancy in EXT:impexp acceptance tests by introducing AbstractCest::selectInContextMenu() for basic context menu handling and moving waitForAjaxRequestToFinish() to the central AbstractCest. In addition, the declarations of local variables have been bundled. Resolves: #94938 Releases: master Change-Id: I13f506d9c32b09b13034388908d710c284885f56 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/70684 Tested-by:
-
Benni Mack authored
This reverts commit e2e59fb3 due to certain incompatibilities with direct "index.php" calls in certain server setups. Change-Id: I980feb08f3a507d51ad059763748ee64752aa9a3 Resolves: #94968 Reverts: #94537 Releases: master, 10.4, 9.5 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/70694 Tested-by:
Stefan Bürk <stefan@buerk.tech> Tested-by:
-
Alexander Nitsche authored
Adapting the "impexp:export" and "impexp:import" commands to the TYPO3 Core and replacing all lowerCamelCase options with lowercase options. The lowerCamelCase options will be removed in TYPO3 v12. Perform explicit instead of implicit command argument testing to tackle the increased complexity introduced by the handling of deprecated and future-proof arguments. Align naming of command options "pid" and "pageId" Align some command option descriptions. Resolves: #94902 Releases: master Change-Id: Ibd30a89e6ba81f27753979da944bde4d104aeb95 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/70599 Tested-by:
-
Christian Kuhn authored
The frontend editing extension ext:feedit has been extracted from core monorepo with #87498 in v10. This rendered stdWrap.editPanel, stdWrap.editIcons and EDITPANEL content object unused as core functionality. The patch deprecates these parts: Extensions that rely on it - typically ext:feedit and ext:frontend_editing should integrate stdWrap using hooks and registering an own EDITPANEL cObj if needed. Releases: master Resolves: #94953 Related: #87498 Change-Id: I489b4fb2d3e09d8b2fcad91b5ae3bccf28244151 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/70714 Tested-by:
-
Andreas Fernandez authored
The context menu skeleton is partially visible for a very short time, before its items are rendered and event listeners are attached. This is revealed by some flaky acceptance tests and was previously workarounded by an additional ->wait() call in the tests. The context menu stays hidden now until everything is set up and the workaround in the AC tests is removed again. The context menu gets hidden after 500ms once the user moved the mouse out of its boundaries. However, that timeout was never reset which closed another context menu that might have opened within that 500ms time span. To circument this issue, such timeouts are now monitored and cleared again once a new context menu is requested. Resolves: #94891 Releases: master, 10.4 Change-Id: Ia0551d8c0b41edeca27cd0b15e4285a89ad2c171 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/70592 Tested-by:
-
Christian Kuhn authored
TypoScriptFrontendController has a property to count nesting of content objects. This is used in ContentObjectRenderer to prevent recursion scenarios. If that kicks in, the rendering is silently stopped. This is ugly for various reasons: First, creating a recursion via TypoScript is rather unlikely. Second, the property creates a dependency between ContentObjectRenderer and TypoScriptFrontendController. And most importantly: If a TypoScript setup somehow manages to create a recursion, this should create an error, not a silent suppression. The patch drops handling of property cObjectDepthCounter. If a recursion appears now, PHP will error out as soon as max nesting level is reached. This obsoletes the property and its handling. Resolves: #94957 Releases: master Change-Id: I4d25472c6c0680a96cc223b7e81bbcbc813b2f52 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/70719 Tested-by:
-
Anja Leichsenring authored
The information is available in the request and should be used from there. Resolves: #94632 Releases: master Change-Id: I8730e8a96bed7272d6c6239b7d55ea56a03bd2de Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/70065 Reviewed-by:
-
Wouter Wolters authored
The extesion "about" is moved to EXT:backend and can safely be removed in composer.json. Resolves: #94967 Related: #91888 Releases: master Change-Id: I61e00b78ea12c1226fe35244778612e1d82fd054 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/70604 Tested-by:
-
Stefan Bürk authored
If the dropzone was not visible, remove it without animation. Resolves: #94892 Releases: master Change-Id: Icfb0090b9383321cd1d24c0aba7026affdbb58d9 Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/70594 Tested-by:
-
Wouter Wolters authored
Resolves: #94895 Releases: master Change-Id: Ic8a69d570627e3017c53b2ad2959ade0f657fb5f Reviewed-on: https://review.typo3.org/c/Packages/TYPO3.CMS/+/70603 Tested-by:
-
