- Feb 28, 2014
-
-
Helmut Hummel authored
The security fix introduced a bug that the title is encoded every time the link wizard is opened, leading to multiple encoded strings. Solution is to not encode it centrally but encode it just before using it in the JavaScript context. Fixes: #41413 Releases: 4.5, 4.7, 6.0, 6.1, 6.2 Change-Id: I6b08db290d5457761edc4506105672d79840764d Reviewed-on: https://review.typo3.org/27853 Reviewed-by: Markus Klein Tested-by: Markus Klein
-
- Feb 27, 2014
-
-
Markus Klein authored
This reverts commit b0c54dca. The 401 header code is used with HTTP based authentication schemes, based on RFC 2617. This is not the case here. Resolves: #55966 Reverts: #51803 Releases: 6.2, 6.1, 6.0, 4.5 Change-Id: I81e0e2bb428a9da69155fc597933bd9f023cf226 Reviewed-on: https://review.typo3.org/27898 Reviewed-by: Markus Klein Tested-by: Markus Klein
-
- Feb 25, 2014
-
-
Jigal van Hemert authored
The paginator in the extension list from TER is transformed into using Ajax. This fails because this process searches for links inside a class which is used by the frontend paginator widget. Changing it to the id of the backend paginator widget makes the ajax calls work again. Resolves: #56184 Releases: 6.2, 6.1, 6.0 Change-Id: I06c193b2657eb3edae623dc0126b06c240f486c6 Reviewed-on: https://review.typo3.org/27851 Reviewed-by: Wouter Wolters Tested-by: Wouter Wolters
-
Jigal van Hemert authored
Encoding a few extra character besides the ones according to RFC3986 makes password reset links working again in various mail clients which do not comply to this RFC (and which do not have plans to fix this in the near future). Change-Id: I0b42bef6cb732c5fc6cc2d900407271cb606e301 Fixes: #23984 Releases: 6.2, 6.1, 6.0, 4.7, 4.5 Reviewed-on: https://review.typo3.org/27826 Reviewed-by: Jigal van Hemert Tested-by: Jigal van Hemert
-
- Feb 24, 2014
-
-
Benjamin Kott authored
If a javascript file is forceOnTop the script uses array_unshift() to put the file at the beginning of the section array. If this is the first file that got processed, the array does not exist yet, so the array_unshift function returns NULL and the file will not added to the array. The fix properly initalizes the section sub-array. Resolves: #56242 Releases: 6.2, 6.1 Change-Id: Iadf2b6f2f512929ff5e8598af9972c5337949341 Reviewed-on: https://review.typo3.org/27818 Reviewed-by: Benjamin Mack Tested-by: Benjamin Mack
-
- Feb 21, 2014
-
-
Marc Bastian Heinrichs authored
The count in LazyObjectStorage for relation type HAS_MANY has to use the count on storage items after initialization. Otherwise the wrong count is returned for a not persisted storage. Resolves: #54724 Releases: 6.2, 6.1, 6.0 Change-Id: I817ce86dde11b175e6a5765a8f6518770ba75f28 Reviewed-on: https://review.typo3.org/27443 Reviewed-by: Wouter Wolters Tested-by: Wouter Wolters
-
Markus Klein authored
The language initialization process is currently split in two main methods (TypoScriptFrontendController->initLLvars and ->settingLanguage). Only settingLanguage contains hooks and sets the "sys_language_uid" property (and "sys_language_content") which determine the displayed records in Frontend. On the opposite, initLLvars is not hookable and sets the "lang" property, which may be used by charset conversion methods. In order to determine display language within hooks, the only way currently is to call initLLvars a second time after determining language, but this method populates the "languageDependencies" property without initializing it. So it is filled two times, which leads to incorrect labels, especially if the displayed language is English and the default language is not English (as "languageDependencies" is already filled with "default language"). The three parts of the patch are: * The initLLvars method now pre-initializes the "languageDependencies" property. * The initLLvars is now called from "settingLanguage" method, right after the "settingLanguage_preProcess" hook to avoid a second call it. * The convPOSTCharset call is now moved after the language initialization, as the language determination done by initLLvars can have some impact on charset handling (multi-charset locallang.php legacy). Change-Id: I6924345931342d5114b13e5d6fab417387559b9a Resolves: #49499 Releases: 6.2, 6.1, 6.0 Reviewed-on: https://review.typo3.org/27664 Reviewed-by: Markus Klein Tested-by: Markus Klein
-
- Feb 20, 2014
-
-
Alexander Jahn authored
When loading tt_content as an inline record WITHOUT using AJAX, some additional broken fields are rendered. Change-Id: Ia5aaec79d0fb7c3266ea4f3a0c04d65b1af4b201 Fixes: #39048 Releases: 6.2, 6.1 Reviewed-on: https://review.typo3.org/27280 Reviewed-by: Wouter Wolters Tested-by: Wouter Wolters
-
Markus Klein authored
Setting the forceOnTop property for a JS or CSS file is not observed if concatenateJs/Css is set as well. Resolves: #53116 Resolves: #56019 Releases: 6.2, 6.1 Change-Id: Ica31656bd72f4a0249513206bac45aa16b929761 Reviewed-on: https://review.typo3.org/27672 Reviewed-by: Stefan Neufeind Reviewed-by: Wouter Wolters Tested-by: Wouter Wolters
-
Markus Klein authored
In DatabaseConnection::listQuery strpos() is used with wrong parameter order. Resolves: #56135 Releases: 6.2, 6.1, 6.0 Change-Id: Iaa18d46442a2aac21a836216cb61ae376bbb2090 Reviewed-on: https://review.typo3.org/27750 Reviewed-by: Stefan Neufeind Tested-by: Stefan Neufeind
-
- Feb 19, 2014
-
-
Felix Althaus authored
FileContentParser used to issue a warning trying to index image files with corrupted EXIF data. Suppress these warnings to fail silently. Resolves: #55286 Releases: 6.2, 6.1 Change-Id: I863601f5579ab74a0743ab684cf1d898b633edba Reviewed-on: https://review.typo3.org/27715 Reviewed-by: Markus Klein Tested-by: Markus Klein
-
Markus Klein authored
Resolves: #56067 Releases: 6.2, 6.1, 6.0 Change-Id: I6d1e19026afde81bec46cec3dff9060fa6042c43 Reviewed-on: https://review.typo3.org/27706 Reviewed-by: Markus Klein Tested-by: Markus Klein
-
Wouter Wolters authored
Resolves: #56057 Releases: 6.2, 6.1 Change-Id: I9c1ec60e7518867d27e99681d3d18957a27c4983 Reviewed-on: https://review.typo3.org/27705 Reviewed-by: Wouter Wolters Tested-by: Wouter Wolters
-
- Feb 18, 2014
-
-
Caspar Stuebs authored
Get the labels for the doktypes from TCA Resolves: #52955 Releases: 6.2, 6.1 Change-Id: I6d70c11f1711bf191a4ce14a91796819224649da Reviewed-on: https://review.typo3.org/27704 Reviewed-by: Wouter Wolters Tested-by: Wouter Wolters
-
Alexey Gafiulov authored
encodeURIComponent is added to escape all special characters in parameters for AJAX call. Resolves: #54304 Releases: 6.2, 6.1, 6.0 Change-Id: I3559104e1a26241b519f40a10000637852a4f114 Reviewed-on: https://review.typo3.org/27674 Reviewed-by: Stefan Neufeind Reviewed-by: Markus Klein Tested-by: Markus Klein
-
- Feb 17, 2014
-
-
Tomita Militaru authored
Fixes problem with match on substring when adding a new field using ExtensionManagementUtility::addToAllTCAtypes() Resolves: #52527 Releases: 6.1, 6.0 Change-Id: I8877cdabc00f5ac64f1f7857bf47c275d36aae62 Reviewed-on: https://review.typo3.org/27663 Reviewed-by: Jigal van Hemert Tested-by: Jigal van Hemert Reviewed-by: Markus Klein Tested-by: Markus Klein
-
Frans Saris authored
The thumbnail rendering in filelist clibboard isn't updated to use FAL. And adding folders to clipboard with thumbnails enabled results in a fatal error. This changes makes sure the thumbnail is only rendered for files and updates the thumbnail part so it uses FAL. Resolves: #56037 Releases: 6.2, 6.1 Change-Id: I2a0eccd633f5313fed9ab94b5081c13e08892900 Reviewed-on: https://review.typo3.org/27669 Reviewed-by: Wouter Wolters Reviewed-by: Frans Saris Tested-by: Frans Saris Reviewed-by: Anja Leichsenring Tested-by: Anja Leichsenring
-
Tim Lochmueller authored
Fix a undefined variable and a wrong variable assignment in the ShortcurtToolbarItem. Resolves: #55998 Releases: 6.2, 6.1, 6.0 Change-Id: I0f0ebcc846a9aa56edd05e384d62aad8f0a5b05b Reviewed-on: https://review.typo3.org/27661 Reviewed-by: Markus Klein Tested-by: Markus Klein
-
Tom Ruether authored
To execute more than one different CommandController at the same time, every execute call has to use its own request and response object. Otherwise the request would be already dispatched for the further tasks. Fixes: #55362 Releases: 6.2, 6.1 , 6.0 Change-Id: I62ba9cbd2c47a59c7243a1017716b28526bcf6ea Reviewed-on: https://review.typo3.org/27654 Reviewed-by: Markus Klein Tested-by: Markus Klein
-
- Feb 11, 2014
-
-
Karol Lamparski authored
Resolves: #49440 Releases: 6.2, 6,1 Change-Id: Iea20a7e9c637c2f1f13c39eebe406d07edc8a2c3 Reviewed-on: https://review.typo3.org/27563 Reviewed-by: Wouter Wolters Tested-by: Wouter Wolters
-
Dmitry Dulepov authored
If the config.cache_ClearAtMidnight is set, the current code of TypoScriptFrontendController::get_cache_timeout() will not take content's starttime/endtime into account. Thus if the content (or other configured record) has start or end time before the midnight, it will not be taken into account and the cache timeout will be wrong. Change-Id: I732da8f07270a86c62b7ce5028b12b630bb2e027 Resolves: #53028 Releases: 6.2, 6.1, 6.0 Reviewed-on: https://review.typo3.org/27560 Reviewed-by: Helmut Hummel Tested-by: Helmut Hummel
-
- Feb 10, 2014
-
-
Helmut Hummel authored
Change-Id: Id06f35b5fa4148e7110d9248ceee80e69e8a5327 Reviewed-on: https://review.typo3.org/27530 Reviewed-by: Helmut Hummel Tested-by: Helmut Hummel
-
- Feb 09, 2014
-
-
Markus Klein authored
CONTENT/RECORDS elements prevent to render the same object twice. Resolves: #53768 Resolves: #28745 Releases: 6.2, 6.1, 6.0 Change-Id: I30750f2dc848521999c3734129439d7f6f90aae1 (cherry picked from commit 1b9d3a59) Reviewed-on: https://review.typo3.org/27515 Reviewed-by: Anja Leichsenring Tested-by: Anja Leichsenring
-
Christian Kuhn authored
Two tests check getUrl() by calling some resource on typo3.org. This smells funny and the tests are unstable if network, dns or typo3.org is down or slow. Mocking is not easily possible with the lowlevel nature of getUrl(). The tests are removed for now. Change-Id: I85a83345404c833a67c2f532e820fd28bb7c08e7 Resolves: #55821 Releases: 6.2, 6.1, 6.0 Reviewed-on: https://review.typo3.org/27520 Reviewed-by: Christian Kuhn Tested-by: Christian Kuhn Reviewed-by: Stefan Neufeind Tested-by: Stefan Neufeind
-
Nicole Cordes authored
When creating a new page inside the top level of a DB mount which is only a sub tree, the pages up and down from the DB mount root will be displayed in the position selector if the logged-in user has read permissions for these pages. This is unwanted information disclosure as the permissions should not matter for pages which are outside the DB mount. Resolves: #18797 Releases: 6.2, 6.1, 6.0 Change-Id: I98008bc7f4308c9fb32dae645325e7cb1b44e413 Reviewed-on: https://review.typo3.org/27429 Reviewed-by: Stefan Neufeind Tested-by: Stefan Neufeind
-
Marc Bastian Heinrichs authored
Resolves: #53564 Releases: 6.0, 6.1, 6.2 Change-Id: If7d69e4f87e368c1eef672cb68f1af92d6ae501b Reviewed-on: https://review.typo3.org/27252 Reviewed-by: Stefan Neufeind Tested-by: Stefan Neufeind
-
- Feb 08, 2014
-
-
Wouter Wolters authored
When using a HTTPS backend the download of CSV is not working in Internet Explorer browser versions lower then 9. Add the needed header to fix this problem. Change-Id: Iefa63fb37d57491fb73bfd504b6caed5b76c8cac Resolves: #16491 Releases: 6.2, 6.1, 6.0, 4.7, 4.5 Reviewed-on: https://review.typo3.org/20902 Reviewed-by: Markus Klein Tested-by: Markus Klein Reviewed-by: Christian Kuhn Tested-by: Christian Kuhn (cherry picked from commit 8c699f8c) Reviewed-on: https://review.typo3.org/27476
-
Thorsten Kahler authored
Add missing label action_1_5 and fix label action_1_4. Resolves: #55698 Releases: 6.2, 6.1 Change-Id: I7f04aaf9d8b53cbe3109dbf72f4fddb0c089e9ad Reviewed-on: https://review.typo3.org/27361 Reviewed-by: Wouter Wolters Reviewed-by: Marcin Sągol Reviewed-by: Jan Bartnik Tested-by: Jan Bartnik Reviewed-by: Christian Kuhn Tested-by: Christian Kuhn (cherry picked from commit 2fdd1737) Reviewed-on: https://review.typo3.org/27468
-
- Feb 07, 2014
-
-
Georg Ringer authored
The whole header is clickable, thererfore move the css rule to this place. Change-Id: Ia864faba4976fc5e2e84299a42e2661a9566be1c Resolves: #55611 Releases: 6.2, 6.1 Reviewed-on: https://review.typo3.org/27312 Reviewed-by: Stefan Neufeind Tested-by: Stefan Neufeind
-
- Feb 06, 2014
-
-
Frans Saris authored
added unit tests. This change adds the mocking to the new tests. Resolves: #54131 Releases: 6.1, 6.2 Change-Id: I09abbcea6a2c6310b7a04774808aa92ab8ad1c86 Reviewed-on: https://review.typo3.org/27220 Reviewed-by: Wouter Wolters Reviewed-by: Stefan Neufeind Tested-by: Stefan Neufeind
-
Markus Klein authored
Resolves: #55713 Releases: 6.1, 6.0 Change-Id: I791b64bec8af256454485bdb016d87ec27778df1 Reviewed-on: https://review.typo3.org/27370 Reviewed-by: Markus Klein Tested-by: Markus Klein
-
- Feb 05, 2014
-
-
Bernhard Kraft authored
Change-Id: I3892f88bdd094b390a22fe94d9b089bd9b8aef71 Resolves: #54112 Releases: 6.2, 6.1, 6.0 Reviewed-on: https://review.typo3.org/27133 Reviewed-by: Bernhard Kraft Tested-by: Bernhard Kraft Reviewed-by: Stefan Neufeind Tested-by: Stefan Neufeind
-
- Feb 04, 2014
-
-
Xavier Perseguers authored
When invalid credentials are entered in the Backend login form, various PHP warnings are raised such as PHP Warning: Illegal string offset 'uid' because $user is not an array. Resolves: #55434 Releases: 6.2, 6.1 Change-Id: I62b85816ce04720ed9fd236965c3a6f55effd093 Reviewed-on: https://review.typo3.org/27274 Reviewed-by: Xavier Perseguers Tested-by: Xavier Perseguers Reviewed-by: Stefan Neufeind Tested-by: Stefan Neufeind
-
- Feb 03, 2014
-
-
Markus Klein authored
It is not possible to set $GLOBALS['TSFE']->altPageTitle in USER_INT extensions anymore to modify the page title. This has been possible before in the 4.x branches. Fix this by implementing the title generation like all other header data. Resolves: #54467 Releases: 6.2, 6.1 Change-Id: I3d5be34b95cb295b92732d2f2f9bd1f123812365 Reviewed-on: https://review.typo3.org/27135 Tested-by: Alexander Opitz Reviewed-by: Markus Klein Tested-by: Markus Klein
-
Markus Klein authored
As discussed back in 2010, the value property of the TEXT content object should have stdWrap functionality as well, if the HTML content object is deprecated. Today the 6.0 TSref already mentions stdWrap for value and the HTML cObject has been removed with 6.0 as well. This fix now finally adds the stdWrap. Resolves: #54371 Releases: 6.2, 6.1, 6.0 Change-Id: I1b7068b3715bc9f8aa4608d5f746c16b70562f3b Reviewed-on: https://review.typo3.org/27245 Reviewed-by: Markus Klein Tested-by: Markus Klein
-
Markus Klein authored
Locker has wrong LockState in semaphore mode, if the lock could not be acquired. Resolves: #52048 Releases: 6.2, 6.1, 6.0 Change-Id: Ifdc2f5d9b2a3c35a0ebef54817a56344f27ab15d Reviewed-on: https://review.typo3.org/27293 Reviewed-by: Markus Klein Tested-by: Markus Klein
-
- Feb 02, 2014
-
-
Frans Saris authored
Currently the property mapper uses the classnames found in the the class property annotations to find the right TypeConverter. But for class aliases this breaks as the alias is used and not the original classname. There was already a getClassNameForAlias call but this was to early. This change moves the getClassNameForAlias() call from doMapping() to findTypeConverter(). Releases: 6.2, 6.1 Resolves: #54289 Change-Id: Ie4ab3deb3c192290db62ea06826870d2fadba8a7 Reviewed-on: https://review.typo3.org/27262 Reviewed-by: Helmut Hummel Tested-by: Helmut Hummel
-
- Jan 31, 2014
-
-
Frans Saris authored
BackendUtility::getLabelsFromItemsList() does not take itemsProcFunc into account. This means no value is returned when there is no matching item found. With this change the plain db value is returned when no matching item is found. Resolves: #54131 Releases: 6.2, 6.1 Change-Id: Ieb04394fa104627b765f6bc05d0d5886f29099c9 Reviewed-on: https://review.typo3.org/25866 Reviewed-by: Andreas Wolf Tested-by: Andreas Wolf (cherry picked from commit 3d240dda) Reviewed-on: https://review.typo3.org/27212
-
- Jan 30, 2014
-
-
Wouter Wolters authored
The fix for issue #53862 calls GeneralUtility::idnaEncode without fully qualified class namespace. Follow-up to: c99a07a9 Resolves: #55475 Releases: 6.1, 6.0 Change-Id: I8ba161ee73e7456da53d2182b4a22d87dad9d53c Reviewed-on: https://review.typo3.org/27173 Reviewed-by: Steffen Müller Tested-by: Steffen Müller Reviewed-by: Christian Kuhn Tested-by: Christian Kuhn
-
Stefan Neufeind authored
Change-Id: I252da74973c3dc4157717139c95ad0605e16fce1 Releases: 6.2, 6.1, 4.5 Resolves: #55458 Reviewed-on: https://review.typo3.org/27149 Reviewed-by: Stefan Neufeind Tested-by: Stefan Neufeind
-