- Nov 13, 2016
-
-
Patrik Karisch authored
Symfony has one of the best BC promises in the PHP world. Which means it will be never a problem to allow all new minor versions of a supported major release, even if there are deprecations. Deprecations don't infer with existing code. Resolves: #77983 Resolves: #76118 Related: #73517 Releases: master, 7.6 Change-Id: I221bb61674b3e4aaa7676d0929dae2ec18e01627 Reviewed-on: https://review.typo3.org/49965 Reviewed-by:
Nicole Cordes <typo3@cordes.co> Tested-by:
TYPO3com <no-reply@typo3.com> Reviewed-by:
Helmut Hummel <typo3@helhum.io> Reviewed-by:
Stefan Neufeind <typo3.neufeind@speedpartner.de> Tested-by:
-
Nicole Cordes authored
Currently it is not possible to install an extension which has special sub-dependencies (e.g. dependencies of sub-extensions). During the installation of an extension, the download information is fetched too early and might not contain all necessary downloads. Furthermore later installation actions add new dependencies, which have to be resolved before any other extension can be installed. This patch ensures all dependencies and their sub-dependencies are fetched before the first installation. All installation information is now correctly ordered, as the last one added has to be the first one in the installation queue. Resolves: #78666 Relates: #66152 Releases: master, 7.6 Change-Id: Idd9242aa1e2ecac3deb542290627fdf9c5479edc Reviewed-on: https://review.typo3.org/50602 Tested-by:
Markus Klein <markus.klein@typo3.org> Tested-by:
-
Benni Mack authored
The system extension "tstemplate" still uses a lot of $GLOBAL objects which are only used within the classes itself. These are solely related to the "web_ts" (Web => Template) module. The code is refactored by using protected object members. Resolves: #78675 Releases: master Change-Id: Ic9bbcd9dfd4d9d9f9d976bead46c5d03d887aec6 Reviewed-on: https://review.typo3.org/50626 Reviewed-by:
-
Markus Klein authored
The XML push-parser now correctly collects the data for description and upload comment. When the data contain line breaks the characterData() method is called multiple times for each line of data. Resolves: #78306 Releases: master, 7.6 Change-Id: Ibfa446b92904572d1f4530b265e94228857a5f02 Reviewed-on: https://review.typo3.org/50516 Reviewed-by:
-
Helmut Hummel authored
If a link to a mount point or a shortcut is created, the id argument in the url points to the id of the real page, where the mount or the shortcut points to. This is why we need to take the id of the final page to be hashed and not the one used to generate the link. We also need to move cHash calculation below shortcut resolving. Resolves: #78238 Releases: master, 7.6, 6.2 Change-Id: I96c4305c6d769a5851c17d6fdcba3cd7584fc3ec Reviewed-on: https://review.typo3.org/50184 Tested-by:
Wouter Wolters <typo3@wouterwolters.nl> Tested-by:
-
- Nov 12, 2016
-
-
Markus Klein authored
Adding an empty alert message to the fieldChangeFunc array causes the fieldChangeFunc_hash to mismatch when pagination or search is used in the link browser popup. Add a check for an empty value, which is already present in another place in Core. Resolves: #75743 Releases: master, 7.6 Change-Id: I1a15400960d577c1a2a0a905643b56533d957100 Reviewed-on: https://review.typo3.org/50586 Tested-by:
-
Markus Hoelzle authored
Close the current document in 'Open documents' list by clicking the "Save and create a new one" button. Resolves: #78545 Releases: master, 7.6 Change-Id: Id401f6471132ef7b66ad35cae4b15c5f384b857e Reviewed-on: https://review.typo3.org/50506 Tested-by:
Michael Oehlhof <typo3@oehlhof.de> Tested-by:
Joerg Boesche <typo3@joergboesche.de> Reviewed-by:
-
Benni Mack authored
The backendUser->uc() call is called with a default value, which is unnecessary and "looks ugly" ;). Additionally $BE_USER calls unpack_uc() itself, basically doing a unserialize() before, even though unpack_uc() does the same thing. Resolves: #78658 Releases: master Change-Id: I492326373b61dba5cba61f24e64b31fddf9a2142 Reviewed-on: https://review.typo3.org/50595 Tested-by:
Georg Ringer <georg.ringer@gmail.com> Tested-by:
-
Wouter Wolters authored
Resolves: #78663 Releases: master Change-Id: Ic7bcc67886d66d31d2062847568d800ab4e48717 Reviewed-on: https://review.typo3.org/50598 Tested-by:
Jigal van Hemert <jigal.van.hemert@typo3.org> Tested-by:
-
Wouter Wolters authored
The property is not defined at all. We can retrieve the iconFactory from the moduleTemplate. Resolves: #78662 Releases: master Change-Id: Id51ac702b9a9e57d731014e86e1637632aadcb38 Reviewed-on: https://review.typo3.org/50597 Reviewed-by:
Frank Naegler <frank.naegler@typo3.org> Tested-by:
-
- Nov 11, 2016
-
-
Frank Naegler authored
This patch optimize and streamline the JavaScript of Install.js The following changes are part of the patch: - Centralized FlashMessage handling - Streamline server side and browser side severities - Optimize performance, remove duplicate jQuery selector - Remove console.debug calls - Remove unused FlashMessage templates from views Resolves: #78376 Releases: master, 7.6 Change-Id: I0652335e5d0bc4666896d5c7b6b528db445ee951 Reviewed-on: https://review.typo3.org/50553 Tested-by:
-
Markus Klein authored
The current solution caused a lot of troubles with server configuration. To avoid any nasty workarounds we simply create a real entry point again. Resolves: #78645 Releases: master, 7.6 Change-Id: I9003ffe8308290a145fe36a1e9ff884016ea0523 Reviewed-on: https://review.typo3.org/50581 Reviewed-by:
Benni Mack <benni@typo3.org> Tested-by:
Oliver Hader <oliver.hader@typo3.org> Tested-by:
Andreas Kiessling <andreas.kiessling@web.de> Reviewed-by:
Alexander Opitz <opitz.alexander@googlemail.com> Tested-by:
-
Claus Due authored
Solves the following issue: * CycleViewHelper produces incorrect output in compiled templates https://github.com/TYPO3/Fluid/pull/193 Change-Id: I3d9ab6a8a2df880c33eb595ca12678dea441dbb7 Releases: master Resolves: #78649 Reviewed-on: https://review.typo3.org/50587 Reviewed-by:
Philipp Gampe <philipp.gampe@typo3.org> Tested-by:
Christian Kuhn <lolli@schwarzbu.ch> Tested-by:
-
- Nov 10, 2016
-
-
Wouter Wolters authored
The title and icon are double escaped. This leads to HTML output as readable code. Move translation into the controller. Resolves: #76277 Releases: master,7.6 Change-Id: I5529e00788ea870475d4b46f2da9dd472d226a97 Reviewed-on: https://review.typo3.org/50548 Tested-by:
Anja Leichsenring <aleichsenring@ab-softlab.de> Tested-by:
-
Benni Mack authored
The CLI command to find files that are referenced more than once in TYPO3 is migrated to Symfony Console to better structure the CLI command and get rid of dependencies of the old CLI command line tools (non-extbase). The command was previously available under ./typo3/cli_dispatch lowlevel_cleaner double_files and is now called via ./typo3/sysext/core/bin/typo3 cleanup:multiplereferencedfiles and allows the following options: --dry-run - do not copy the files and update the references, but just print them --update-refindex - update the reference index, do not ask the user Resolves: #78627 Releases: master Change-Id: I19039790227365ab951a954848565bcb0eb29c6b Reviewed-on: https://review.typo3.org/50558 Tested-by:
-
Benni Mack authored
A bug, introduced with the registerArguments option does not work properly anymore with the "default" option for the <f:case> viewhelper. Casting the argument let's the option be evaluated again. Resolves: #78638 Releases: master Change-Id: I5b245378db4fab430fc0670b3a4a77de6c3b863f Reviewed-on: https://review.typo3.org/50575 Reviewed-by:
Claus Due <claus@phpmind.net> Reviewed-by:
Benjamin Kott <info@bk2k.info> Tested-by:
-
Helmut Hummel authored
To make cHash calculation work again, we need to use the page uid. However the GET argument might contain a page alias and not the id. Use the calculated id instead to avoid wrong cHash calculation. Resolves: #78526 Releases: master, 7.6, 6.2 Change-Id: Iaa89b7a52a581bd5ac7c0b01049a7f9596258051 Reviewed-on: https://review.typo3.org/50468 Tested-by:
Mathias Brodala <mbrodala@pagemachine.de> Reviewed-by:
-
Oliver Hader authored
Extract per-connection concerns from SchemaMigrator into new ConnectionMigrator. This way, schema changes on connections can be handled individually. Resolves: #78046 Releases: master Change-Id: Ic7ff27c2251e9dd9f2fed6366b0d4fcb2c5b41d2 Reviewed-on: https://review.typo3.org/50007 Tested-by:
Susanne Moog <susanne.moog@typo3.org> Tested-by:
-
Benni Mack authored
The CLI command to detect and (as far as possible) remove relations to non-existing records, and offline version versions as well as listing deleted records is migrated to Symfony Console to better structure the CLI command and get rid of dependencies of the old CLI command line tools (non-extbase). The command was previously available under ./typo3/cli_dispatch lowlevel_cleaner missing_relations and is now called via ./typo3/sysext/core/bin/typo3 cleanup:missingrelations and allows the following options: --dry-run - do not delete the references, but just print them --update-refindex - update the reference index, do not ask the user Resolves: #78623 Releases: master Change-Id: Ia81a250deb0b238ed4e4dd09667b381f5cb53cfb Reviewed-on: https://review.typo3.org/50555 Tested-by:
-
- Nov 09, 2016
-
-
Wouter Wolters authored
The method doubleval is an alias of floatval and this could be used. But using a cast to float is quicker. Resolves: #78617 Releases: master Change-Id: If5d6c39583534a1878bd94b41f37f3e523627d77 Reviewed-on: https://review.typo3.org/50549 Tested-by:
-
Christian Kuhn authored
Transmitting the full $row in suggest wizard was introduced to find the relevant flex form data structure in the ajax search action to determine the suggest wizard field configuration The patch adds the data structure identifier instead, the searchAction() uses that to fetch the specified flex form data structure directly. Additionally, field values relevant for the ajax call are now signed with hmac and checked in searchAction(). Change-Id: Ibb5004d4d11487fe999072999c7ce2f847aac750 Resolves: #78616 Related: #78581 Releases: master Reviewed-on: https://review.typo3.org/50544 Reviewed-by:
-
Arnd Messer authored
Remove styles.content.links.target from constants.txt and lib.parseFunc.ts. styles.content.links.target has as description in constants.txt pointing the value is used for frames. Since fluid_styled_content is focussing on HTML5, this value does not make sense at all, since frames have been removed from the specification. styles.content.links.target should be removed from constants.txt in fluid_styled_content including the use of this constant value in lib.parseFunc.ts, the only place where it is used. The typolink viewhelper does not take this into account also. Resolves: #77830 Related: #36216 Releases: master Change-Id: I16e8dc3f77595fde20d8c43d6ab16eb7d590e34b Reviewed-on: https://review.typo3.org/49839 Reviewed-by:
-
- Nov 08, 2016
-
-
Wouter Wolters authored
Apply the same fix as was done for the LiveSearch. Resolves: #78457 Related: #77566 Releases: master,7.6 Change-Id: I861e6267b4804f8009d138e22520f1b95ba421b6 Reviewed-on: https://review.typo3.org/50536 Tested-by:
-
Wouter Wolters authored
Use the native PHP method strpos instead of StringUtility::beginsWith method. The beginsWith method runtime compared to a simple strpos check is much higher. Resolves: #78602 Releases: master Change-Id: I608ed791ddd770a8de493e0796387a18b22ff2ce Reviewed-on: https://review.typo3.org/50534 Reviewed-by:
-
Tymoteusz Motylewski authored
The t3_origuid field is now covered in functional tests for DataHandler in workspaces module (Regular/Publish suite). Releases: master, 7.6 Resolves: #78597 Change-Id: I4caf9490866319470af48b9fb4f177c1c3a0b708 Reviewed-on: https://review.typo3.org/50533 Tested-by:
Tymoteusz Motylewski <t.motylewski@gmail.com> Tested-by:
-
- Nov 07, 2016
-
-
Sascha Egerer authored
In #66995 the extbase ObjectAccess has been cleaned up and introduced a behavior where it is not possible anymore to call the getter of an object that implements \ArrayAccess. The problem is avoided by only using array-accessing if the offset is found to exist. If not, the instance is handled as any other object. Change-Id: Ie6b598d93fd7171d0b6c37259e424794861e31ed Resolves: #78589 Related: #66995 Releases: master Reviewed-on: https://review.typo3.org/50521 Reviewed-by:
-
Claus Due authored
Solves this issue: https://github.com/TYPO3/Fluid/pull/228 Change-Id: Idc377dac0d074194b065448a0ce4b8641a94249a Resolves: #78601 Releases: master Reviewed-on: https://review.typo3.org/50535 Tested-by:
-
Christina Hauk authored
Disable further language menu options in free mode. If language record doesn't have a record in the default language there are no more options in the language menu needed. Resolves: #72797 Releases: master, 7.6 Change-Id: Ic5211f99c3c53f3097057ba580e3394896b81170 Reviewed-on: https://review.typo3.org/50298 Tested-by:
Josef Glatz <josef.glatz@typo3.org> Tested-by:
-
Benni Mack authored
Add deprecation docs and logging information to both PHP classes. Resolves: #78244 Releases: master Change-Id: Iacc45588c61b6cd32b8239391176059c5afbf731 Reviewed-on: https://review.typo3.org/50174 Tested-by:
-
Christian Kuhn authored
Method BackendUtility::getFlexFormDS() does two things at once: It finds a data structure by given data (TCA, row, ...) and then parses it. This construct gives tons of headaches, since the methods never exposes where a specific data structure came from and the lookup mechanism is complex. Especially if a flex form is used in combination with ajax requests later, the core has massive issues since the location can not be found out later again. To solve that, the patch splits getFlexFormDS() into two methods: First method "FlexFormTools->getDataStructureIdentifier()" gets TCA and row and locates a specific structure. It returns an "identifier" that points to that unique data structure. This identifier can be later hand around easily. The second method "FlexFormTools->parseDataStructureByIdentifier()" then gets this identifier again, fetches the data structure the identifier points to, and parses it. Change-Id: I38264e8a4a6f956c12e9e50f6039d3d09af0f03a Resolves: #78581 Releases: master Reviewed-on: https://review.typo3.org/48212 Tested-by:
-
Christian Kuhn authored
The files were substituted with "Settings.cfg" a while ago and are now obsolete according to the docs team. Change-Id: I43f2a6fca83f839359d5c1b0c1bb4f0cf967e253 Resolves: #78595 Releases: master, 7.6 Reviewed-on: https://review.typo3.org/50524 Tested-by:
Mathias Schreiber <mathias.schreiber@wmdb.de> Tested-by:
Martin Bless <m.bless@gmx.de> Tested-by:
-
Michael Oehlhof authored
Resolves: #72311 Releases: master, 7.6 Change-Id: I47b19ccb5e74c5755630dba3626dad29b7b24fc7 Reviewed-on: https://review.typo3.org/50274 Tested-by:
Kurt Ludikovsky <kurt.ludikovsky@ludikovsky.at> Tested-by:
-
Mathias Brodala authored
This moves the toplevel "index.php" to the frontend extension as "frontend.php". The toplevel "index.php" becomes a stub which includes the "frontend.php". This setup works in the following cases: 1) Package "typo3/cms" is the root package 2) The "index.php" is symlinked into the web-dir 3) The "index.php" is copied into the web-dir In all of these cases "typo3" is a toplevel directory. Resolves: #73027 Releases: master Change-Id: I1c0653bcc4691105eb2ccfbea2bcc873ce6189da Reviewed-on: https://review.typo3.org/46390 Reviewed-by:
-
Wouter Wolters authored
substr is called 3 times with the same parameters, use one variable for this. Resolves: #78583 Releases: master Change-Id: Idc57e143c26cd190f8047d59f1e36cb745e9aaf8 Reviewed-on: https://review.typo3.org/50514 Tested-by:
Andreas Fernandez <typo3@scripting-base.de> Tested-by:
-
Claus Due authored
Solves the following issues: * ViewHelpers using compiling traits would call incorrect method if renderStatic was inherited https://github.com/TYPO3/Fluid/pull/226 * f:format.raw would not use the "value" argument if it was specified, only renderChildren used https://github.com/TYPO3/Fluid/pull/217 * Some ViewHelpers using content argument trait would fail to render or render unexpected output in compiled/not-compiled rendering respectively https://github.com/TYPO3/Fluid/pull/210 Change-Id: Ie7bbe6e63f3379a2df9d9cc2fea94cddcf4c1320 Resolves: #78584 Releases: master Reviewed-on: https://review.typo3.org/50515 Tested-by:
Daniel Goerz <ervaude@gmail.com> Tested-by:
-
- Nov 06, 2016
-
-
Claus Due authored
This change implements a runtime cache which stores the view configuration for extension keys. This avoids calling removeDotsFromTS and other methods more than one time per configuration. Change-Id: I551fd8526d9a5576e22edaf45742ccb35c2f3aba Fixes: #78580 Releases: master Reviewed-on: https://review.typo3.org/50510 Reviewed-by:
-
- Nov 05, 2016
-
-
Manuel Selbach authored
According to task #78437 queries should follow the prepared statement principle. Thus the LiveSearch has to be refactored. Change-Id: I301190314d6e2f86512d1f5c41bcf6991009258f Resolves: #78578 Releases: master Reviewed-on: https://review.typo3.org/50507 Tested-by:
-
Andreas Fernandez authored
`list($x, )` does not work on empty arrays, causing `$x` being null then. The code now checks whether there are actually open docs and then does the list() stuff. Resolves: #78051 Releases: master Change-Id: I73d2c3ca40bea8c3dce2ac1de0ec97e136bf228b Reviewed-on: https://review.typo3.org/50512 Reviewed-by:
-
Anja Leichsenring authored
It comes in handy to use the names of enumeration objects sometimes. Two getName() functions provide the name once unchanged, once human readable. Resolves: #78575 Releases: master Change-Id: Ibb0e06c7462835097f9980a854c79c8f90a61672 Reviewed-on: https://review.typo3.org/50503 Tested-by:
-
Andreas Fernandez authored
The properties `$openDocs` and `$recentDocs` are described and treated as arrays, but initialized as null values. This patch changes the declaration to array. Resolves: #78051 Releases: master Change-Id: I859c13478b17ddd9fbb7ab7686099f27b3c0e7ea Reviewed-on: https://review.typo3.org/50511 Reviewed-by:
-
