- Nov 22, 2017
-
-
Andreas Fernandez authored
(Un)mounting nodes in the page tree is now possible again by fixing calls to the page tree objects. Resolves: #83051 Releases: master, 8.7 Change-Id: I978308e301b712fa75d677337d790e4895dc997a Reviewed-on: https://review.typo3.org/54706 Tested-by:
TYPO3com <no-reply@typo3.com> Reviewed-by:
Wouter Wolters <typo3@wouterwolters.nl> Tested-by:
Riccardo De Contardi <erredeco@gmail.com> Reviewed-by:
Frank Naegler <frank.naegler@typo3.org> Tested-by:
-
Andreas Fernandez authored
There is a left over usage of `top.TYPO3.Storage` in the ExtJS based page tree. This usage is replaced with the `Storage/Persistent` module. Resolves: #83050 Related: #82603 Releases: master Change-Id: I68a33d257599f316eb8e169121ec8408397748e8 Reviewed-on: https://review.typo3.org/54705 Tested-by:
Christian Kuhn <lolli@schwarzbu.ch> Tested-by:
-
Stefan Neufeind authored
Add for/id so clicking the label toggles the checkbox. Resolves: #83040 Releases: master, 8.7, 7.6 Change-Id: I3dc08706556e1a95ddc8d96cae705ee968c48326 Reviewed-on: https://review.typo3.org/54715 Tested-by:
Markus Klein <markus.klein@typo3.org> Tested-by:
Stefan Neufeind <typo3.neufeind@speedpartner.de> Tested-by:
Jigal van Hemert <jigal.van.hemert@typo3.org> Tested-by:
-
Georg Ringer authored
Fix a variable name which has been changed with the latest refactoring to make it possible again to set links without a cHash. Resolves: #83063 Releases: master Change-Id: I684d42f85a94179f19dabef5df47f09ce6b34df8 Reviewed-on: https://review.typo3.org/54719 Reviewed-by:
Andreas Fernandez <typo3@scripting-base.de> Tested-by:
Benni Mack <benni@typo3.org> Tested-by:
-
- Nov 21, 2017
-
-
Andreas Fernandez authored
The returnUrl is not available if the access module is accessed via context menu of the page tree. The module now checks whether the returnUrl is available and renders the close button only in such case. Resolves: #83053 Releases: master, 8.7 Change-Id: I37c5c65c1505b42acf7ac734fbc7a9f53629a6a1 Reviewed-on: https://review.typo3.org/54713 Tested-by:
Georg Ringer <georg.ringer@gmail.com> Tested-by:
-
Oliver Hader authored
GIFBUILDER uses provided file names and text snippets in order to generate the final name for files to be written to. In case text snippets start with a dot, this is also reflected into the file name - preventing some web servers to deliver the file since it's considered to be internal. Resolves: #83029 Releases: master, 8.7 Change-Id: Iebed832c1c2429c7400373a6af4d5ae14a9ea18e Reviewed-on: https://review.typo3.org/54677 Tested-by:
-
- Nov 20, 2017
-
-
Andreas Fernandez authored
Resolves: #83052 Releases: master, 8.7 Change-Id: I9d86c38f8d1749583e154bd52c77b168d20af7dc Reviewed-on: https://review.typo3.org/54709 Reviewed-by:
-
Markus Günther authored
Adding a condition to check if a backend user has permissions on the search to prevent a JS error while initializing the autocomplete module. Fixes: #82969 Releases: master, 8.7, 7.6 Change-Id: If2827902443ff7411d9b2c1050213be3e0d78fb6 Reviewed-on: https://review.typo3.org/54694 Tested-by:
Alexander Opitz <opitz.alexander@googlemail.com> Reviewed-by:
-
rickymk authored
Catch InsufficientFileAccessPermissionsException while extracting metadata. Reason for this exception could be that the file extension is not allowed by the ['BE']['fileDenyPattern'] setting. This patch makes sure that the indexer doesn't break and is able to extract the other files from storage. Resolves: #82060 Release: master, 8.7, 7.6 Change-Id: I37950aa70f4f59b388c6b9203e1922708ad07b71 Reviewed-on: https://review.typo3.org/53834 Tested-by:
Ricky Mathew <ricky.mk@pitsolutions.com> Tested-by:
-
Nicole Cordes authored
During some Doctrine cleanup the constraint for fetching extension by extension key got messed up. This patch removes double named parameter in search constraint. Resolves: #80555 Releases: master, 8.7 Change-Id: I84636b4a86d8b3946c552031e3cb23ae69cd3f2a Reviewed-on: https://review.typo3.org/54699 Reviewed-by:
-
Tymoteusz Motylewski authored
Resolves: #83002 Releases: master Change-Id: I06c09f282df7fdcc0d8af11d65c5113f33a9c849 Reviewed-on: https://review.typo3.org/54651 Reviewed-by:
Susanne Moog <susanne.moog@typo3.org> Tested-by:
-
Velletti authored
Releases: master, 8.7 Resolves: #83031 Change-Id: I918ef9ca5ae28f9c605e3c7940cea53b6ed3325a Reviewed-on: https://review.typo3.org/54679 Reviewed-by:
Tymoteusz Motylewski <t.motylewski@gmail.com> Tested-by:
-
Markus Günther authored
Releases: master, 8.7, 7.6 Fixes: #83044 Change-Id: I2c16f160a18f4c0e58b87261b59fbde90aca2f82 Reviewed-on: https://review.typo3.org/54693 Reviewed-by:
-
Markus Klein authored
Move the hidden form field out of the label tag to ensure clicking the label's content (eg. fa icon) really toggles the checkbox. Resolves: #82998 Releases: master, 8.7, 7.6 Change-Id: Id4293b4109c82a9e6410e113a5b5560ffd1b75d4 Reviewed-on: https://review.typo3.org/54682 Tested-by:
-
Helmut Hummel authored
While fixing #82877 a mistake was introduced which leads to a JavaScript error when filtering the page tree. Problem is that PageTree.select() is called with the filter tree currently active, but this method is only functional for the full page tree since quite a while now. We now avoid calling this method and document the behavior properly in method annotations. Additionally a typo in the variable name is fixed, which lead to a jquery warning when selecting a page. Resolves: #83039 Releases: master, 8.7 Change-Id: I6440fb7f6cfd6ead8b5b29dea7c5081dd688a5c3 Reviewed-on: https://review.typo3.org/54688 Tested-by:
Sebastian Fischer <typo3@evoweb.de> Tested-by:
Andreas Fernandez <typo3@scripting-ba...>
-
Benni Mack authored
Translations of the current page can now be listed and edited through the list module again. This is a follow-up change to the pages_language_overlay removal. Resolves: #83016 Releases: master Change-Id: I38b41ba6b1b3164ab1b9cdfc03bca14780c38c08 Reviewed-on: https://review.typo3.org/54530 Tested-by:
-
- Nov 19, 2017
-
-
Jigal van Hemert authored
The files included in the field tsconfig_includes are now included when collecting the page TSconfig in the TypoScriptFrontendController. Resolves: #82662 Releases: master, 8.7 Change-Id: I226f942045647a3b7c4130b9e5d39b6d982efd1b Reviewed-on: https://review.typo3.org/54276 Tested-by:
-
- Nov 17, 2017
-
-
Robert Wildling authored
The link pointing to the positioning of own handlers (a changelog of v7.6) was missing a version paramater after "core/". Releases: master Resolves: #83025 Change-Id: I52d22ad30b4669b2fe59728551a24f761efd423d Reviewed-on: https://review.typo3.org/54673 Tested-by:
Mathias Brodala <mbrodala@pagemachine.de> Tested-by:
-
Joerg Boesche authored
To modify the display result rows before data is assigned to the fluid view there is a new hook inside the `getDisplayResults()` method called `getDisplayResults_postProc`. Resolves: #82488 Releases: master Change-Id: I2f0ac073692fb28b9c71a18def027c71a418606f Reviewed-on: https://review.typo3.org/54148 Reviewed-by:
-
- Nov 16, 2017
-
-
Benni Mack authored
When working with localizations in a system, the option "Localization view" in the list module is a pre-requisite for all editors, for all installations without languages, the checkbox is not necessary. To streamline this behaviour, the option is removed and the functionality is enabled BY DEFAULT. Resolves: #82991 Releases: master Change-Id: I2005413405d009935e3b7d7d046e4aac284ee013 Reviewed-on: https://review.typo3.org/54633 Tested-by:
Oliver Hader <oliver.hader@typo3.org> Tested-by:
-
Helmut Hummel authored
L10nModeUpdater checks whether fields in translated records are modified, thus not synchronized with the parent record and the new l10n_state for this field is set to "custom" in this case or "parent" if the field is empty. However this is done after the data is synchronized, which leads to wrong data being written by the DataMapProcessor. To fix this, we move setting l10n_state upwards in the code, so that relations and fields from translated records can be updated correctly. Resolves: #83006 Releases: master, 8.7 Change-Id: Ibb06eecb6cf743bd6bba1990ca6f9391c983821f Reviewed-on: https://review.typo3.org/54657 Tested-by:
-
Helmut Hummel authored
TSConfig properties keepItems and removeItems are checked with a weak in_array() test (third argument not set to true), which leads to the situation, that arbitrary string values are treated equal to integer 0 This is now fixed by flipping the array and check for the array index. This works nicely because PHP silently converts strings which look like integer to integer for the keys. Tests are added which cover these cases and demonstrate the failure before the change. Resolves: #82980 Releases: 8.7, master Change-Id: I544a221674fa89d302cb6c1bcca506847c6f7f0f Reviewed-on: https://review.typo3.org/54623 Tested-by:
Joerg Boesche <typo3@joergboesche.de> Reviewed-by:
Henning Liebe <h.liebe@neusta.de> Reviewed-by:
Jan Stockfisch <jan.stockfisch@googlemail.com> Tested-by:
-
Oliver Hader authored
Remove duplicated "image" field definition in test scenario. Related: #69909 Releases: master, 8.7 Change-Id: Ie87e2e62e036fa70fb837f9bd05f2c3ba13019fa Reviewed-on: https://review.typo3.org/54668 Tested-by:
-
Oliver Hader authored
If DataMapProcessor is called with a non-reference id, e.g. zero (0), this submission is considered as a reference. Since there is no database record having UID 0, the synchronization process fails with the following exeception: #1486233164: Child record was not processed To solve this behavior, invalid references (empty/zero) are not considered anymore to compare references. Besides that, values for localized records that are configured to be synchronized are sanitized correctly now. Resolves: #83009 Releases: master, 8.7 Change-Id: Ie370007521c45dac8bca03978a387b4662952b1d Reviewed-on: https://review.typo3.org/54655 Tested-by:
-
Alexander Schnitzler authored
This patch introduces the usage of doctrine annotations. Therefore the AnnotationRegistry is configured during the bootstrap and the annotation "TYPO3\CMS\Extbase\Annotation\Inject" is evaluated when building a ClassSchema for a class. On top of that this patch puts the static variable $ignoredTags into the DocCommentParser class, which saved quite some bytes of RAM when reconstituting the ClassSchema objects from the cache. Resolves: #82869 Releases: master Change-Id: I99d17706395ebc0c308c7f611c59f207fe0b6233 Reviewed-on: https://review.typo3.org/54590 Tested-by:
-
Tymoteusz Motylewski authored
Resolves: #83003 Releases: master, 8.7 Change-Id: Iceaa7ab9bd436322229e591677c0aee16cd69f5c Reviewed-on: https://review.typo3.org/54652 Reviewed-by:
-
Frans Saris authored
Add all pageTS.RTE options to config without dots appended to the key to be able to override the RTE config with pageTS like before. Resolves: #82865 Releases: master, 8,7 Change-Id: Idb7d7d3b39d44ec37be8f93cc9de8b42b9c8a55c Reviewed-on: https://review.typo3.org/54511 Tested-by:
-
Benni Mack authored
DataHandlers' newlog() function is a shorthand method that should act as a simple API, which it does, but it also adds a "[newlog()] " info in front of the error message, which is completely useless and only clutters DB and the log module. Resolves: #83012 Releases: master, 8.7 Change-Id: I9e1ea083461de9cdf66fad1a30019e51130a0186 Reviewed-on: https://review.typo3.org/54660 Reviewed-by:
Frans Saris <franssaris@gmail.com> Reviewed-by:
-
Wouter Wolters authored
In QueryBuilder::quoteIdentifiersForSelect the 3rd parameter of trimExplode is inserted. To prevent notices array_pad is added. Resolves: #82995 Releases: master,8.7 Change-Id: If35efcf5c8c3fd5b0360ae8795f9b92ae87250ab Reviewed-on: https://review.typo3.org/54640 Reviewed-by:
-
Tymoteusz Motylewski authored
Resolves: #82921 Releases: master, 8.7 Change-Id: Id26239e65321aa3653de849932a0be7e626bd653 Reviewed-on: https://review.typo3.org/54558 Tested-by:
-
- Nov 15, 2017
-
-
Tymoteusz Motylewski authored
Fixes PHP notices thrown by accessing non existing array key. Resolves: #83004 Releases: master Change-Id: Ib5ff0ace0c9eb9a9987693176c4bee25026e2eb8 Reviewed-on: https://review.typo3.org/54653 Reviewed-by:
-
Claus Due authored
This patch adds a new hook to determine whether a content record is used or not. Since the default decision is fairly basic (only checks if $columns contains $record['colPos']) a hook makes it possible for third party extensions to change the decision and validate on their own. Included rst file shows how to register to the hook and provides example code how to determine the result. Resolves: #82213 Releases: master Change-Id: Id338254dae6575ba7b91141c10a42b99401614c9 Reviewed-on: https://review.typo3.org/53811 Tested-by:
Claus Due <claus@phpmind.net> Reviewed-by:
Nicole Cordes <typo3@cordes.co> Reviewed-by:
Benjamin Kott <benjamin.kott@outlook.com> Tested-by:
-
Thomas Hohn authored
Before the file is processed a extra check is done to be sure the file exists. If the file doesn't exist it's marked as missing. Resolves: #80061 Releases: master, 8.7 Change-Id: Ifda396b75d2ace0ce126057a4e118856b15c05e0 Reviewed-on: https://review.typo3.org/51969 Reviewed-by:
Thomas Hohn <thomas@hohn.dk> Reviewed-by:
-
Tom Novotny authored
Return all fields to ease the work with hooks. Intention is not having to query again in hook $GLOBALS['TYPO3_CONF_VARS']['EXTCONF']['felogin']['forgotPasswordMail'] to change "Hello [username],..." to "Hello [fist_name] [last_name],..." in password forgot mail to user. Change-Id: I18e18845e61d36aadd043758f902dc6627ad5f37 Resolves: #82996 Releases: master, 8.7 Reviewed-on: https://review.typo3.org/54628 Reviewed-by:
Mathias Schreiber <mathias.schreiber@typo3.com> Tested-by:
-
Joshua Westerheide authored
Check for cases where the files array is nested more complex or keywords (e.g. "error") are used as field names. Run `curl -F "d0[]=@d12.txt" -F "d0[d1][d2][d3]=@d12.txt" -F "error=@error_file.txt" localhost:8080` against a custom php script to generate the testing $_FILES array. Resolves: #82976 Releases: master, 8.7, 7.6 Change-Id: I51fb9da7c87871b6bb1e8ac4de317973f307dd49 Reviewed-on: https://review.typo3.org/54619 Tested-by:
-
Benni Mack authored
When doing translations with pages and inline elements (e.g. pages.media), there is an inconsistency when an additional inline element is added to the TRANSLATED page (e.g. language=1). Current (wrong) when adding a translation to a page - localized IRRE children get pid = default page ID (correct) - localized IRRE children only localization get parentid = default page ID (should be translated page ID) Current (wrong) behaviour when adding a IRRE child only available in a translation: - parentid gets localized page => correct - pid gets set to localized page => must be default language The patch corrects this behaviour by always set - the relation (parentid) to the localized page ID - the page ID (pid) to the default language page ID Additionally, when adding an Inline element (IRRE) via AJAX on page properties, the PID gets resolved correctly to be sent to DataHandler. Resolves: #82983 Releases: master Change-Id: Id8bf39524ee608acf452e2b7103087d4299c1e28 Reviewed-on: https://review.typo3.org/54632 Tested-by:
Daniel Gorges <daniel.gorges@b13.de> Tested-by:
-
- Nov 14, 2017
-
-
Sascha Egerer authored
Exceptions in the ImageViewhelper are currently silently ignored. So you'll get no information if a wrong value has been passed to the src attribute or if there went something wrong. The exception is not rethrown as Fluid viewhelper exception so it will be catched and logged in Production Context by the AbstractViewhelper that calls the render method. Change-Id: I95124ebb945366d260b4351ccb85d61a399c50b5 Resolves: #82918 Releases: master Reviewed-on: https://review.typo3.org/54561 Tested-by:
Manuel Selbach <manuel_selbach@yahoo.de> Reviewed-by:
-
Mathias Schreiber authored
Stabilize the acceptance tests to activate and deactivate a language by checking only for the alerts instead of the enabled/disabled rows. Additionally make the download a language test fail immediately if the language is enabled. Resolves: #82972 Releases: master, 8.7 Change-Id: Ib380b464a24188dbbee37296ddda0f0bb5c44f5e Reviewed-on: https://review.typo3.org/54631 Tested-by:
Stephan Großberndt <stephan@grossberndt.de> Reviewed-by:
-
Susanne Moog authored
OrderBy does not make sense in a count only query and fails on SQL Server, hence it is removed. Change-Id: I676d1913f75c29dd71345dbc0f626e99df0d242f Resolves: #82994 Releases: master Reviewed-on: https://review.typo3.org/54638 Reviewed-by:
-
Mathias Brodala authored
Variadic arguments are optional but do not have a default value, thus properly check this. Also render the variadic operator ("...") in the output. Resolves: #82956 Releases: master, 8.7 Change-Id: I10e3528d9cffcdd03ae4711b139898f569ac3775 Reviewed-on: https://review.typo3.org/54600 Tested-by:Alexander Schnitzler <review.typo3.org@alexanderschnitzler.de> Tested-by:
-
