Packages/TYPO3.CMS.git
4 years ago[TASK] Update to phpunit 4.8 80/42880/2
Anja Leichsenring [Wed, 26 Aug 2015 09:35:10 +0000 (11:35 +0200)]
[TASK] Update to phpunit 4.8

Change-Id: I73b083192ec92bc709796cdafd84884747298c70
Releases: master
Resolves: #69325
Reviewed-on: http://review.typo3.org/42880
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
Reviewed-by: Alexander Opitz <opitz.alexander@googlemail.com>
Tested-by: Alexander Opitz <opitz.alexander@googlemail.com>
4 years ago[CLEANUP] Fix namespace in SplitProcessor example 81/42881/2
Sascha Egerer [Wed, 26 Aug 2015 09:44:15 +0000 (11:44 +0200)]
[CLEANUP] Fix namespace in SplitProcessor example

Change-Id: I97d6f501b775834a6cdd77f3c4f25b47d63b2065
Resolves: #69324
Releases: master
Reviewed-on: http://review.typo3.org/42881
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Andreas Fernandez <typo3@scripting-base.de>
Reviewed-by: Sascha Egerer <sascha@sascha-egerer.de>
Tested-by: Sascha Egerer <sascha@sascha-egerer.de>
4 years ago[CLEANUP] Remove invalid leading slash in SplitProcessor example 79/42879/3
Sascha Egerer [Wed, 26 Aug 2015 09:17:56 +0000 (11:17 +0200)]
[CLEANUP] Remove invalid leading slash in SplitProcessor example

Change-Id: Ia2e3a1b26455a42e814f4a579d61d83c7f5a8f63
Resolves: #69324
Releases: master
Reviewed-on: http://review.typo3.org/42879
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Andreas Fernandez <typo3@scripting-base.de>
4 years ago[TASK] Moved marker substitution functionality to own class 81/41381/7
Benjamin Mack [Wed, 15 Jul 2015 22:37:05 +0000 (00:37 +0200)]
[TASK] Moved marker substitution functionality to own class

The marker substitution functionality has been moved from
core/Classes/Html/HtmlParser.php to own class
core/Classes/Utility/MarkerUtility.php

The following methods within HtmlParser have been marked as deprecated.

* HtmlParser::getSubpart()
* HtmlParser::substituteSubpart()
* HtmlParser::substituteSubpartArray()
* HtmlParser::substituteMarker()
* HtmlParser::substituteMarkerArray()
* HtmlParser::substituteMarkerAndSubpartArrayRecursive()

Resolves: #69262
Releases: master
Change-Id: Ia0ffb3715b3ecef9739db4f6aad53bae9c224b90
Reviewed-on: http://review.typo3.org/41381
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
4 years ago[BUGFIX] Indexed Search: Escape search strings for LIKE queries 06/42806/4
Morton Jonuschat [Fri, 21 Aug 2015 09:49:22 +0000 (11:49 +0200)]
[BUGFIX] Indexed Search: Escape search strings for LIKE queries

LIKE queries support special placeholders (_ and %). These characters
need proper escaping before being used in database queries. Use the
escapeStrForLike() method to provide properly escaped strings to the
query.

Resolves: #69227
Releases: master
Change-Id: I29c41bfecdbc13e8156ab9257b906696bc7a5e4f
Reviewed-on: http://review.typo3.org/42806
Tested-by: Philipp Gampe <philipp.gampe@typo3.org>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Alexander Opitz <opitz.alexander@googlemail.com>
Tested-by: Alexander Opitz <opitz.alexander@googlemail.com>
4 years ago[TASK] Add unit tests for HtmlParser->splitIntoBlock() 40/42840/5
Michael Oehlhof [Sat, 22 Aug 2015 18:03:04 +0000 (20:03 +0200)]
[TASK] Add unit tests for HtmlParser->splitIntoBlock()

Resolves: #69264
Releases: master
Change-Id: I73e1bd8894180658fd6911dbca0104518ceaf585
Reviewed-on: http://review.typo3.org/42840
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Alexander Opitz <opitz.alexander@googlemail.com>
Tested-by: Alexander Opitz <opitz.alexander@googlemail.com>
4 years ago[!!!][TASK] Streamline ElementBrowser mode handling 75/42875/4
Markus Klein [Tue, 25 Aug 2015 16:37:23 +0000 (18:37 +0200)]
[!!!][TASK] Streamline ElementBrowser mode handling

The ElementBrowser now has a main entry point which takes
care of dispatching to the the various modes.
The main_*() methods of ElementBrowser are therefore made protected.

The $mode member of ElementBrowserController is not needed
by outside code and is therefore protected as well.

Additionally, the $content member of ElementBrowserController and
the printContent() method are removed.

Resolves: #69315
Releases: master
Change-Id: Ib9790915e72d3a5a460211a93ceeda24ee165c3e
Reviewed-on: http://review.typo3.org/42875
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
4 years ago[TASK] Add unit tests for HtmlParser->removeFirstAndLastTag() 44/42844/4
Michael Oehlhof [Sun, 23 Aug 2015 08:15:42 +0000 (10:15 +0200)]
[TASK] Add unit tests for HtmlParser->removeFirstAndLastTag()

Resolves: #69266
Releases: master
Change-Id: I8c35c1c175e632d2cba9fb6a495daf09486dc290
Reviewed-on: http://review.typo3.org/42844
Reviewed-by: Alexander Opitz <opitz.alexander@googlemail.com>
Tested-by: Alexander Opitz <opitz.alexander@googlemail.com>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
4 years ago[TASK] Add missing breaking changes documentation 14/42814/9
Alexander Schnitzler [Fri, 21 Aug 2015 14:04:19 +0000 (16:04 +0200)]
[TASK] Add missing breaking changes documentation

This patch adds missing documentation for the breaking changes
introduced in https://review.typo3.org/42644.

It also adds an early return for the FileListIsStartModuleUpdate
to prevent unnecessary database operations.

Releases: master
Resolves: #69237
Change-Id: Iaa113741919013b8a0a9513c1dcca2e5db094b04
Reviewed-on: http://review.typo3.org/42814
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
4 years ago[FOLLOWUP][!!!][TASK] Trees do not use global SOBE anymore 69/42869/2
Markus Klein [Tue, 25 Aug 2015 12:07:10 +0000 (14:07 +0200)]
[FOLLOWUP][!!!][TASK] Trees do not use global SOBE anymore

Fix wrong instanceof check.

Resolves: #69276
Releases: master
Change-Id: I9beb681c595cfa3e63cfe4ffdbf33557041511b4
Reviewed-on: http://review.typo3.org/42869
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
4 years ago[!!!][TASK] Trees do not use global SOBE anymore 53/42853/4
Markus Klein [Mon, 24 Aug 2015 07:19:42 +0000 (09:19 +0200)]
[!!!][TASK] Trees do not use global SOBE anymore

The back-reference to the underlying instance of ElementBrowser is now
stored as member of the tree.

Resolves: #69276
Releases: master
Change-Id: Ie48a77cd01269ea7e13905d651663424dbb616a1
Reviewed-on: http://review.typo3.org/42853
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Alexander Opitz <opitz.alexander@googlemail.com>
Tested-by: Alexander Opitz <opitz.alexander@googlemail.com>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
4 years ago[BUGFIX] Correctly assign workspace for rootline retrieval 65/42865/3
Markus Klein [Tue, 25 Aug 2015 10:55:26 +0000 (12:55 +0200)]
[BUGFIX] Correctly assign workspace for rootline retrieval

Resolves: #69306
Releases: master, 6.2
Change-Id: If59ec36222c1124c2918b39fc014e4c8a2d7bf2f
Reviewed-on: http://review.typo3.org/42865
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
4 years ago[TASK] Use routing for folder navigation tree 47/42847/7
Helmut Hummel [Sun, 23 Aug 2015 06:59:22 +0000 (08:59 +0200)]
[TASK] Use routing for folder navigation tree

Resolves: #69293
Releases: master
Change-Id: I069f3758fc96148043890c6448fda8bc964fd91b
Reviewed-on: http://review.typo3.org/42847
Reviewed-by: Alexander Opitz <opitz.alexander@googlemail.com>
Tested-by: Alexander Opitz <opitz.alexander@googlemail.com>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
4 years ago[BUGFIX] Password in form is transferred as asterisks 58/42858/2
Xavier Perseguers [Mon, 24 Aug 2015 14:27:22 +0000 (16:27 +0200)]
[BUGFIX] Password in form is transferred as asterisks

Due to a wrong usage of $.inArray() which possibly returns
"0" if element is found at the first position within an array,
TCA fields with a single eval statement "password" are not
properly transferred to TYPO3.

Change-Id: Ic2647fbefc0ea7c9fef88288946af91343fcc55c
Resolves: #69153
Releases: master
Reviewed-on: http://review.typo3.org/42858
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Andreas Fernandez <typo3@scripting-base.de>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Alexander Opitz <opitz.alexander@googlemail.com>
Tested-by: Alexander Opitz <opitz.alexander@googlemail.com>
4 years ago[TASK] Prettify Extbase Debugger Utility 90/42790/4
Benjamin Kott [Thu, 20 Aug 2015 21:58:22 +0000 (23:58 +0200)]
[TASK] Prettify Extbase Debugger Utility

Releases: master
Resolves: #69299
Change-Id: Ibee145703e1689ca804384b20b9af1b0b586b8d8
Reviewed-on: http://review.typo3.org/42790
Reviewed-by: Helmut Hummel <helmut.hummel@typo3.org>
Tested-by: Helmut Hummel <helmut.hummel@typo3.org>
Reviewed-by: Alexander Opitz <opitz.alexander@googlemail.com>
Tested-by: Alexander Opitz <opitz.alexander@googlemail.com>
4 years ago[!!!][TASK] Fix configuration for backend module icons 54/42854/6
Helmut Hummel [Mon, 24 Aug 2015 13:45:23 +0000 (15:45 +0200)]
[!!!][TASK] Fix configuration for backend module icons

http://review.typo3.org/38004 removed the typo3/mod directory
with the main modules and their configuration and added the configuration
to `$GLOBALS['TBE_MODULES']`directly.
However for that a sub array key `configuration` was used, which is
inconsistent with all our current API and its usages across other
modules, leading to the situation that this configuration was basically
ignored.

On top of that, http://review.typo3.org/39316 introduced the possibility
to use sprite icons as module icons, which used the configuration key
`icon` in the above mentioned sub array key `configuration`,
making this key the only option evaluated in the sub array.

This change removes this sub array and moves the sprite icon
configuration on top for consistency. For that the key must be changed
to 'iconIdentifier' as 'icon' was already taken for Extbase module
icon configuration.

Resolves: #69291
Releases: master
Change-Id: I282841fdb09c9c2773ca6e5518fdda863f9f8ef5
Reviewed-on: http://review.typo3.org/42854
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
4 years ago[TASK] Introduce runtime cache for getIndpEnv() 32/42732/5
Markus Klein [Tue, 18 Aug 2015 20:54:48 +0000 (22:54 +0200)]
[TASK] Introduce runtime cache for getIndpEnv()

This method is independent of runtime state, but just reflects
environment state, which does not change during a request.

Caching the result of the method gives a performance boost for common
BE request as documented in the ticket. (85% fewer calls to the method.)

Resolves: #69173
Releases: master
Change-Id: Icb3fd5fb56434c6db2323a9038b7d486426cab7e
Reviewed-on: http://review.typo3.org/42732
Reviewed-by: Alexander Opitz <opitz.alexander@googlemail.com>
Tested-by: Alexander Opitz <opitz.alexander@googlemail.com>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Xavier Perseguers <xavier@typo3.org>
Reviewed-by: Helmut Hummel <helmut.hummel@typo3.org>
Tested-by: Helmut Hummel <helmut.hummel@typo3.org>
4 years ago[TASK] Cleanup navigation frame module registration 56/42856/5
Helmut Hummel [Mon, 24 Aug 2015 14:06:47 +0000 (16:06 +0200)]
[TASK] Cleanup navigation frame module registration

http://review.typo3.org/37611 made the navigation frame dispatched.
For this it leveraged the addNavigationComponent API,
which was introduced only for ExtJS tree components.

Since we want to get rid of ExtJS (and this API specifically),
we need to introduce a different way to make modules or routes
navigation components.

This change reverts the above changes to the API and instead uses
the `navigationFrameModule` option which was introduced in
https://review.typo3.org/#/c/30593

To make this option work with top level modules and inheritance of
the navigation frame,
the evaluation of the option is moved to the module loader, which
has the benefit to have the code only in one place while it needed
to be in two places before.

Resolves: #69270
Releases: master
Change-Id: I7cb2ca2ee6d04001af424c88a1db828d1712caa1
Reviewed-on: http://review.typo3.org/42856
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
4 years ago[TASK] Deprecate BackendUtility::getPathType_web_nonweb 46/42846/2
Wouter Wolters [Sun, 23 Aug 2015 12:58:20 +0000 (14:58 +0200)]
[TASK] Deprecate BackendUtility::getPathType_web_nonweb

Resolves: #69269
Releases: master
Change-Id: I99f2a486a6bdd5cfb813c5068d226167e4b3ad7f
Reviewed-on: http://review.typo3.org/42846
Reviewed-by: Alexander Opitz <opitz.alexander@googlemail.com>
Tested-by: Alexander Opitz <opitz.alexander@googlemail.com>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
4 years ago[BUGFIX] Copy transOrigDiffSourceField into workspace placeholder 31/42631/4
Morton Jonuschat [Fri, 14 Aug 2015 18:34:18 +0000 (20:34 +0200)]
[BUGFIX] Copy transOrigDiffSourceField into workspace placeholder

The transOrigDiffSourceField field needs to be copied into
workspace move placeholders when it exists as it can be defined
as NOT NULL in the database schema. Not copying it into the
placeholder leads to errors when MySQL is running in strict mode.

Resolves: #69073
Releases: master
Change-Id: Ifcf524675b1d6709d0d77012fdaff35017c00c90
Reviewed-on: http://review.typo3.org/42631
Reviewed-by: Oliver Hader <oliver.hader@typo3.org>
Tested-by: Oliver Hader <oliver.hader@typo3.org>
4 years ago[TASK] Remove unused CSH file locallang_csh_sysws.xlf 83/42783/2
Wouter Wolters [Thu, 20 Aug 2015 20:59:26 +0000 (22:59 +0200)]
[TASK] Remove unused CSH file locallang_csh_sysws.xlf

Resolves: #69217
Releases: master
Change-Id: I0b76541fff254a1b242a105e4104ea00a523f2a7
Reviewed-on: http://review.typo3.org/42783
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
Reviewed-by: Alexander Opitz <opitz.alexander@googlemail.com>
Tested-by: Alexander Opitz <opitz.alexander@googlemail.com>
Reviewed-by: Michael Oehlhof <typo3@oehlhof.de>
Tested-by: Michael Oehlhof <typo3@oehlhof.de>
4 years ago[BUGFIX] Assure a backend domain is set in workspace preview 94/42394/5
Daniel Goerz [Fri, 7 Aug 2015 19:27:44 +0000 (21:27 +0200)]
[BUGFIX] Assure a backend domain is set in workspace preview

Change-Id: Idc17243f8741057ee6ea1746ec6c344aec28d57a
Resolves: #45519
Resolves: #31718
Releases: master, 6.2
Reviewed-on: http://review.typo3.org/42394
Reviewed-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Tested-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Reviewed-by: Frederic Gaus <frederic.gaus@flagbit.de>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
4 years ago[BUGFIX] Don't cut off icons in New Record wizard 12/42812/3
Andreas Fernandez [Fri, 21 Aug 2015 13:05:00 +0000 (15:05 +0200)]
[BUGFIX] Don't cut off icons in New Record wizard

If an icon larger than 16x16 pixels is used for the record icon in TCA,
it gets cutted and only the top left part is displayed. Apply proper
CSS rules to fix this behaviour.

Resolves: #69236
Releases: master, 6.2
Change-Id: I9f7643817f05fd6fba73b7884e70dbed2db84999
Reviewed-on: http://review.typo3.org/42812
Reviewed-by: Frans Saris <franssaris@gmail.com>
Tested-by: Frans Saris <franssaris@gmail.com>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
4 years ago[BUGFIX] Replace ###ID### in openCustomUrlInContentFrame correctly 33/42733/3
Christian Seifert [Tue, 18 Aug 2015 20:31:19 +0000 (20:31 +0000)]
[BUGFIX] Replace ###ID### in openCustomUrlInContentFrame correctly

Adding an URL by getModuleUrl including an array having an ID
parameter id=###ID### results in an escaped string.
Instead of "###ID###" the value "%23%23%23ID%23%23%23" is
submitted to the JavaScript function.

This patch fixes the old replacement function.
Now, you can again add click menu items linking to modules and
passing the selected page uid.

Resolves: #69170
Releases: master, 6.2
Change-Id: Ib4c0a1556dd1ed583e8faefe431e2d4eaaf864ad
Reviewed-on: http://review.typo3.org/42733
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Christian Seifert <christian-f-seifert@gmx.de>
Tested-by: Christian Seifert <christian-f-seifert@gmx.de>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
4 years ago[FEATURE] Add MediaViewHelper 24/38624/11
Frans Saris [Fri, 10 Apr 2015 19:19:23 +0000 (21:19 +0200)]
[FEATURE] Add MediaViewHelper

In order to comfortably render video, audio and all other file types
with a registered Renderer class (RenderingRegistry introduced with
#61800) in FE, we add a new ViewHelper.

The new MediaViewHelper first checks if there is a Renderer present for
the given file. If not, it will as fallback render a image tag.
This way it is a replacement for the ImageViewHelper in most cases.

Releases: master
Resolves: #66366
Change-Id: I4f92b5f1030238aacb544e45a35189f4e3d0732e
Reviewed-on: http://review.typo3.org/38624
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
Reviewed-by: Sascha Egerer <sascha@sascha-egerer.de>
Reviewed-by: Markus Sommer <markussom@posteo.de>
Tested-by: Markus Sommer <markussom@posteo.de>
Reviewed-by: Frans Saris <franssaris@gmail.com>
Tested-by: Frans Saris <franssaris@gmail.com>
4 years ago[TASK] Remove unused TCA description file for EM 81/42781/3
Wouter Wolters [Thu, 20 Aug 2015 20:57:33 +0000 (22:57 +0200)]
[TASK] Remove unused TCA description file for EM

Resolves: #69216
Releases: master
Change-Id: I5ed4b8f05da06328699fff86eacddc816d7ad145
Reviewed-on: http://review.typo3.org/42781
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
Reviewed-by: Jan Helke <typo3@helke.de>
Tested-by: Jan Helke <typo3@helke.de>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
4 years ago[CLEANUP] Removed unused CSC constant 10/42810/2
Markus Klein [Fri, 21 Aug 2015 12:23:49 +0000 (14:23 +0200)]
[CLEANUP] Removed unused CSC constant

The constant "addIntextClearer" is not evaluated, remove it.

Resolves: #69235
Releases: master
Change-Id: I22d273ff12edc377f7f5661fa813de7cb4438573
Reviewed-on: http://review.typo3.org/42810
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Alexander Opitz <opitz.alexander@googlemail.com>
Tested-by: Alexander Opitz <opitz.alexander@googlemail.com>
4 years ago[FOLLOWUP][TASK] Combine several calls to allowTableOnStandardPages 02/42802/2
Georg Ringer [Fri, 21 Aug 2015 08:12:54 +0000 (10:12 +0200)]
[FOLLOWUP][TASK] Combine several calls to allowTableOnStandardPages

The function accepts a comma separated string to add multiple tables.

Change-Id: I51fb442969bb569cf6936c3272551daa861fdbd8
Resolves: #69218
Releases: master
Reviewed-on: http://review.typo3.org/42802
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Jan Helke <typo3@helke.de>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
4 years ago[FOLLOWUP][BUGFIX] Show first tab in Element Wizard as fallback 00/42800/2
Markus Klein [Fri, 21 Aug 2015 07:44:10 +0000 (09:44 +0200)]
[FOLLOWUP][BUGFIX] Show first tab in Element Wizard as fallback

A wrong variable initialization is fixed.

Resolves: #59810
Releases: master, 6.2
Change-Id: Ied65743326e52e7e56e24a19db57ba40f6b4be47
Reviewed-on: http://review.typo3.org/42800
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Mathias Schreiber <mathias.schreiber@wmdb.de>
Tested-by: Mathias Schreiber <mathias.schreiber@wmdb.de>
Reviewed-by: Helmut Hummel <helmut.hummel@typo3.org>
Tested-by: Helmut Hummel <helmut.hummel@typo3.org>
4 years ago[TASK] Combine several calls to allowTableOnStandardPages 85/42785/2
Wouter Wolters [Thu, 20 Aug 2015 21:43:41 +0000 (23:43 +0200)]
[TASK] Combine several calls to allowTableOnStandardPages

The function accepts a comma separated string to add multiple tables.

Resolves: #69218
Releases: master
Change-Id: Icff55095786e338816cf925c5e89837a436cd201
Reviewed-on: http://review.typo3.org/42785
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
4 years ago[BUGFIX] Show first tab in Element Wizard as fallback 67/42767/5
Christian Seifert [Thu, 20 Aug 2015 22:59:29 +0000 (22:59 +0000)]
[BUGFIX] Show first tab in Element Wizard as fallback

Using the extension gridelements it is possible to restrict the
selectable content elements instead of the full menu configured
by TSconfig. At first, you might have 5 menus available and you
click on the last menu tab (maybe the fifth), so the backend
stores 5 as current tab.
Adding content elements within a gridelement, which restricts the
range of selectable CEs, you can only select the first tab (p.e.).
The original code assumes that the fifth tab is still there and
tries to select it.

This patch checks first against the existence of the selected tab.
If there is no "fifth" tab p.e. the index is set to 1.

Resolves: #59810
Releases: master, 6.2
Change-Id: I829489737de4631dfe3362e4fb11909be9315864
Reviewed-on: http://review.typo3.org/42767
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Jan Helke <typo3@helke.de>
Tested-by: Jan Helke <typo3@helke.de>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
4 years ago[TASK] Add missing TYPO3_MODE checks in ext_localconf.php 96/42796/2
Wouter Wolters [Thu, 20 Aug 2015 22:34:46 +0000 (00:34 +0200)]
[TASK] Add missing TYPO3_MODE checks in ext_localconf.php

Resolves: #69222
Releases: master
Change-Id: I4e0d737a8035df1c178a584eed203dd48176babe
Reviewed-on: http://review.typo3.org/42796
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
4 years ago[TASK] Remove unused options in global TBE_STYLES 80/42780/2
Wouter Wolters [Thu, 20 Aug 2015 20:38:21 +0000 (22:38 +0200)]
[TASK] Remove unused options in global TBE_STYLES

Since version 7.0 the following options are unused.

* colorschemes
* borderschemes

Resolves: #69215
Releases: master
Change-Id: I6b676aec33e5ce585dd2b6ad5f23f9f449032297
Reviewed-on: http://review.typo3.org/42780
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Michael Oehlhof <typo3@oehlhof.de>
Tested-by: Michael Oehlhof <typo3@oehlhof.de>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
4 years ago[TASK] Use ::class for cleaner command registration in ext:lowlevel 95/42795/2
Wouter Wolters [Thu, 20 Aug 2015 22:17:06 +0000 (00:17 +0200)]
[TASK] Use ::class for cleaner command registration in ext:lowlevel

Resolves: #69221
Releases: master
Change-Id: I3f3970ed213822da6d8e36954ebc1ae8e4392cbe
Reviewed-on: http://review.typo3.org/42795
Reviewed-by: Michael Oehlhof <typo3@oehlhof.de>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
4 years ago[TASK] Use ::class for hook registration in ext:core 87/42787/2
Wouter Wolters [Thu, 20 Aug 2015 21:52:25 +0000 (23:52 +0200)]
[TASK] Use ::class for hook registration in ext:core

Resolves: #69220
Releases: master
Change-Id: I6a23589bbe29996080fa406fe8084c6817e39a7e
Reviewed-on: http://review.typo3.org/42787
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
4 years ago[BUGFIX] Tests: Create NEWid values the same way as DataHandler 25/42625/3
Morton Jonuschat [Fri, 14 Aug 2015 08:24:34 +0000 (10:24 +0200)]
[BUGFIX] Tests: Create NEWid values the same way as DataHandler

The NEWid values generated in the functions tests of the core use a
different kind of NEWid value that is longer than the values created
within DataHandler. The length of the values generated in the tests
exceeds the length of the database column in the sys_log table, leading
to errors when running MySQL in strict mode.

This change uses the same code as DataHandler to generate NEWid values.

Resolves: #69039
Releases: master
Change-Id: I4380fb333a15d37b9a48b98b211a9143286ec73b
Reviewed-on: http://review.typo3.org/42625
Reviewed-by: Stephan Großberndt <stephan@grossberndt.de>
Reviewed-by: Alexander Opitz <opitz.alexander@googlemail.com>
Tested-by: Alexander Opitz <opitz.alexander@googlemail.com>
Reviewed-by: Nicole Cordes <typo3@cordes.co>
Tested-by: Nicole Cordes <typo3@cordes.co>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
4 years ago[BUGFIX] Add values to submit buttons 64/42764/3
Wouter Wolters [Wed, 19 Aug 2015 17:25:57 +0000 (19:25 +0200)]
[BUGFIX] Add values to submit buttons

Submit buttons need to have a value set,
otherwise submit checks in the PHP code won't work.

Some useless POST checks are removed after the removal of the
type=image input fields for saving data.

Resolves: #69194
Releases: master
Change-Id: If42da73242d82d96d0042e0bedce69ad2a302427
Reviewed-on: http://review.typo3.org/42764
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Benjamin Kott <info@bk2k.info>
Tested-by: Benjamin Kott <info@bk2k.info>
4 years ago[FOLLOWUP][BUGFIX] Use correct max size for fields in TCA 73/42773/2
Georg Ringer [Thu, 20 Aug 2015 12:25:02 +0000 (14:25 +0200)]
[FOLLOWUP][BUGFIX] Use correct max size for fields in TCA

Some fields are using a varchar(255) in the DB but the max size
is defined with 256 in the TCA. Change the TCA to 255 to avoid errors.

Change-Id: I9a8dcfaf4563442932ae7f259858ad070415ea24
Resolves: #69209
Releases: master
Reviewed-on: http://review.typo3.org/42773
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Andreas Fernandez <typo3@scripting-base.de>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
4 years ago[BUGFIX] Use correct max size for fields in TCA 72/42772/2
Georg Ringer [Thu, 20 Aug 2015 12:12:28 +0000 (14:12 +0200)]
[BUGFIX] Use correct max size for fields in TCA

Some fields are using a varchar(255) in the DB but the max size
is defined with 256 in the TCA. Change the TCA to 255 to avoid errors.

Change-Id: Iad09512e98bb9e2fb9b52e32e0f5f4f8d9b49035
Resolves: #69209
Releases: master
Reviewed-on: http://review.typo3.org/42772
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
4 years ago[FEATURE] Get list of pages based on pageuids from page repository 01/41601/14
Patrick Broens [Wed, 15 Jul 2015 18:36:51 +0000 (20:36 +0200)]
[FEATURE] Get list of pages based on pageuids from page repository

The frontend page repository is now capable of retrieving a list
of pages based on a single page uid or an array of page uids
like HMENU.
This functionality is needed to retrieve pages for the rendering of
several frontend menus by the content element "Special Menus"

Resolves: #68300
Releases: master
Change-Id: I906c07478f3f07ece71aa428cc03cc3a237dfd27
Reviewed-on: http://review.typo3.org/41601
Reviewed-by: Frans Saris <franssaris@gmail.com>
Tested-by: Frans Saris <franssaris@gmail.com>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Helmut Hummel <helmut.hummel@typo3.org>
Tested-by: Helmut Hummel <helmut.hummel@typo3.org>
4 years ago[TASK] Add unit tests for HtmlParser::substituteSubpartArray() 37/42737/4
Michael Oehlhof [Tue, 18 Aug 2015 22:56:20 +0000 (00:56 +0200)]
[TASK] Add unit tests for HtmlParser::substituteSubpartArray()

Resolves: #69175
Releases: master
Change-Id: Id85a7ab6bc7f3b3281f394fd9a41ecd64ab51ec7
Reviewed-on: http://review.typo3.org/42737
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Alexander Opitz <opitz.alexander@googlemail.com>
Tested-by: Alexander Opitz <opitz.alexander@googlemail.com>
4 years ago[TASK] Add unit tests for HtmlParser::substituteMarker() 65/42765/4
Michael Oehlhof [Wed, 19 Aug 2015 19:59:51 +0000 (21:59 +0200)]
[TASK] Add unit tests for HtmlParser::substituteMarker()

Resolves: #69196
Releases: master
Change-Id: I46a1f5f1ed0b7d2e0a2ceeb22768140771d99ffb
Reviewed-on: http://review.typo3.org/42765
Reviewed-by: Alexander Opitz <opitz.alexander@googlemail.com>
Tested-by: Alexander Opitz <opitz.alexander@googlemail.com>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
4 years ago[TASK] Respect editlock of pages in inline editing 34/41734/13
Andreas Fernandez [Mon, 20 Jul 2015 12:30:03 +0000 (14:30 +0200)]
[TASK] Respect editlock of pages in inline editing

Disable drag and drop of content elements in Page module
and inline page title editing if page's editlock is set.

Resolves: #68388
Related: #68279
Releases: master
Change-Id: Ibcb285babb84f0777eb621f244f2a9e40498596b
Reviewed-on: http://review.typo3.org/41734
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Tested-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
4 years ago[TASK] Add unit tests for HtmlParser::substituteMarkerArray() 66/42766/2
Michael Oehlhof [Wed, 19 Aug 2015 20:37:30 +0000 (22:37 +0200)]
[TASK] Add unit tests for HtmlParser::substituteMarkerArray()

Resolves: #69197
Releases: master
Change-Id: I551ce23be294b152117943f5b6e89ed360a8db50
Reviewed-on: http://review.typo3.org/42766
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Alexander Opitz <opitz.alexander@googlemail.com>
Tested-by: Alexander Opitz <opitz.alexander@googlemail.com>
4 years ago[BUGFIX] Restore save+view|close|new functionality 47/42647/4
Markus Klein [Wed, 19 Aug 2015 15:47:01 +0000 (17:47 +0200)]
[BUGFIX] Restore save+view|close|new functionality

Resolves: #68981
Releases: master
Change-Id: I203ed2a58c30cf0da89bedb266453eeff32b34cc
Reviewed-on: http://review.typo3.org/42647
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Tested-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Reviewed-by: Daniel Maier <dani-maier@gmx.de>
Tested-by: Daniel Maier <dani-maier@gmx.de>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
4 years ago[BUGFIX] DataHandler: Assign reference default values 50/42550/8
Morton Jonuschat [Wed, 12 Aug 2015 20:55:59 +0000 (22:55 +0200)]
[BUGFIX] DataHandler: Assign reference default values

In MySQL stric mode scenarios, relation types not using MM tables
or the foreign_field definition (IRRE) might end up in no
references at all, which is only the trigger for the actual
problem. These references result in a blank string ("") that
is casted in non-strict modes to the accordant field type
defined by the DMBS schema.

In strict mode this has to be handled more specific. Since the
system cannot determine the real meaning automatically, values
for reference that are produced in the mentioned scenario are
now casted to zero (0) or to the default value defined in the
accordant TCA configuration of the field.

Resolves: #69004
Releases: master
Change-Id: I6608f1fe0f63a99caa1208b1d3d76354b6f659cf
Reviewed-on: http://review.typo3.org/42550
Reviewed-by: Oliver Hader <oliver.hader@typo3.org>
Tested-by: Oliver Hader <oliver.hader@typo3.org>
4 years ago[TASK] Add missing unit tests for SilentConfigurationUpgradeService 87/42487/5
Michael Oehlhof [Mon, 10 Aug 2015 18:06:24 +0000 (20:06 +0200)]
[TASK] Add missing unit tests for SilentConfigurationUpgradeService

Missing unit tests for GFX settings are added:
disableImageMagickAndGdlibIfImageProcessingIsDisabled()
disableImageMagickDetailSettingsIfImageMagickIsDisabled()
setImageMagickDetailSettings()

Resolves: #68913
Releases: master
Change-Id: I45a4e7ed4d614bd6e657c714a09d8251f25eff25
Reviewed-on: http://review.typo3.org/42487
Reviewed-by: Michael Oehlhof <typo3@oehlhof.de>
Tested-by: Michael Oehlhof <typo3@oehlhof.de>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
4 years ago[BUGFIX] Log exception as array instead of serialized 17/42717/3
Stephan Großberndt [Tue, 18 Aug 2015 14:54:05 +0000 (16:54 +0200)]
[BUGFIX] Log exception as array instead of serialized

Log exceptions thrown by scheduler tasks as array instead of
serializing the whole exception in order to keep the size down.

Resolves: #69157
Releases: master, 6.2
Change-Id: Ifc6457f3de114779a522baa77443e5019f0dc4c1
Reviewed-on: http://review.typo3.org/42717
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
4 years ago[TASK] Add unit tests for HtmlParser::substituteSubpart() 18/42718/5
Michael Oehlhof [Tue, 18 Aug 2015 16:02:24 +0000 (18:02 +0200)]
[TASK] Add unit tests for HtmlParser::substituteSubpart()

Resolves: #69160
Releases: master
Change-Id: I240dd575b6b105846493a608b8039b0883e2a866
Reviewed-on: http://review.typo3.org/42718
Reviewed-by: Michael Oehlhof <typo3@oehlhof.de>
Tested-by: Michael Oehlhof <typo3@oehlhof.de>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
4 years ago[TASK] Add unit tests for HtmlParser::getSubpart() 67/42667/5
Michael Oehlhof [Sun, 16 Aug 2015 23:15:59 +0000 (01:15 +0200)]
[TASK] Add unit tests for HtmlParser::getSubpart()

Resolves: #69116
Releases: master
Change-Id: I07d4d04d2108976287617b458c0f9857099c2ada
Reviewed-on: http://review.typo3.org/42667
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Michael Oehlhof <typo3@oehlhof.de>
Tested-by: Michael Oehlhof <typo3@oehlhof.de>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
4 years ago[TASK] Remove addModulePath call for ext:taskcenter 34/42734/2
Wouter Wolters [Tue, 18 Aug 2015 21:18:25 +0000 (23:18 +0200)]
[TASK] Remove addModulePath call for ext:taskcenter

Wrong module name is used.
Is not in tools but in user.

Resolves: #69174
Releases: master
Change-Id: Ic2d4a287529b0898c529fb1e04ee7f0b37e4586e
Reviewed-on: http://review.typo3.org/42734
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
4 years ago[TASK] Streamline tt_content TCA types configuration 29/42729/3
Oliver Hader [Tue, 18 Aug 2015 19:31:06 +0000 (21:31 +0200)]
[TASK] Streamline tt_content TCA types configuration

The 3rd parameter of the TCA types configuration is deprecated.
Therefore the structure needs to be changed - accordingly to
the messages being shown in the deprecation log.

This behavior can only be reproduced if EXT:css_styled_content
is deactivated - since it overrides the mentioned types section.

Resolves: #69171
Releases: master
Change-Id: I9669b1c994b0a711862715f1fb4da672df8630a2
Reviewed-on: http://review.typo3.org/42729
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
4 years ago[TASK] Remove not used include for CSS files in ext:core 16/42716/2
Wouter Wolters [Tue, 18 Aug 2015 13:55:47 +0000 (15:55 +0200)]
[TASK] Remove not used include for CSS files in ext:core

The mentioned directory doesn't exists.

Resolves: #69156
Releases: master
Change-Id: I0a2847932178c4d6b0fe1297b82116eb4ded7535
Reviewed-on: http://review.typo3.org/42716
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
4 years ago[!!!][TASK] Remove non-tabbed view of Content Element Wizard 22/42722/5
Benjamin Mack [Tue, 18 Aug 2015 17:11:29 +0000 (19:11 +0200)]
[!!!][TASK] Remove non-tabbed view of Content Element Wizard

Resolves: #69168
Releases: master
Change-Id: I88a73fde52af453181fb0b1bc33ea325e5424765
Reviewed-on: http://review.typo3.org/42722
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
4 years ago[!!!][BUGFIX] Correct "required"-parameter in TextfieldViewHelper 45/40645/6
Frederic Gaus [Thu, 25 Jun 2015 13:42:08 +0000 (15:42 +0200)]
[!!!][BUGFIX] Correct "required"-parameter in TextfieldViewHelper

Releases: master
Resolves: #67098
Change-Id: Ifc322aba2cf95849e1541cac4df65d30aeb56bc3
Reviewed-on: http://review.typo3.org/40645
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
4 years ago[TASK] Move EXT:impexp entrypoint to Routing 27/42727/3
Benjamin Mack [Tue, 18 Aug 2015 18:46:56 +0000 (20:46 +0200)]
[TASK] Move EXT:impexp entrypoint to Routing

Resolves: #69167
Releases: master
Change-Id: I7e20ed767803621cb555dcf3bc47c269505390ea
Reviewed-on: http://review.typo3.org/42727
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
4 years ago[BUGFIX] Exclude computed field names on fetching record overlays 60/42160/4
Albrecht Köhnlein [Fri, 31 Jul 2015 08:12:23 +0000 (10:12 +0200)]
[BUGFIX] Exclude computed field names on fetching record overlays

During versioning, the dynamic fields _ORIG_pid and _ORIG_uid are added
to $row, but they do not exist in database and cause an SQL error.

Resolves: #68643
Releases: master, 6.2
Change-Id: I33a56b2071a04b5224821805e60722ce8dd3dd14
Reviewed-on: http://review.typo3.org/42160
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
Reviewed-by: Oliver Hader <oliver.hader@typo3.org>
Tested-by: Oliver Hader <oliver.hader@typo3.org>
4 years ago[TASK] Move EXT:version entrypoint to Routing 26/42726/4
Benjamin Mack [Tue, 18 Aug 2015 18:04:31 +0000 (20:04 +0200)]
[TASK] Move EXT:version entrypoint to Routing

Resolves: #69166
Releases: master
Change-Id: I7fe803ff352464b984ff7008e6ed1aad2c1bdada
Reviewed-on: http://review.typo3.org/42726
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
4 years ago[TASK] Streamline Version Module 04/42004/8
Benjamin Kott [Mon, 27 Jul 2015 22:14:36 +0000 (00:14 +0200)]
[TASK] Streamline Version Module

- Streamline appearance
- Remove unused JavaScript
- Remove unused CSS
- Use IconFactory

Resolves: #48125
Releases: master
Change-Id: I9efe60a30c895c68c2e4da8105f243f181418525
Reviewed-on: http://review.typo3.org/42004
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
4 years ago[TASK] Streamline addModulePath call 25/42725/2
Benjamin Mack [Tue, 18 Aug 2015 18:03:19 +0000 (20:03 +0200)]
[TASK] Streamline addModulePath call

The addModulePath functionality is also copied inside
addModule in ExtensionManagementUtility and can be reused.

During that, the EXT:recordlist/ext_tables.php call, is duplicated
and addModulePath can be removed there.

Releases: master
Resolves: #69165
Change-Id: I92dd5b566283d4114216242bf297ef42d6b709a0
Reviewed-on: http://review.typo3.org/42725
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
4 years ago[TASK] Add missing unit tests for SilentConfigurationUpgradeService (5) 32/42632/4
Michael Oehlhof [Fri, 14 Aug 2015 19:55:29 +0000 (21:55 +0200)]
[TASK] Add missing unit tests for SilentConfigurationUpgradeService (5)

Missing unit tests for configureBackendLoginSecurity() are added.

Resolves: #69075
Releases: master
Change-Id: I94898193d5e8bcee5ed5b99b5ac7e69a8a5787ea
Reviewed-on: http://review.typo3.org/42632
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
4 years ago[TASK] Move CSH Javascript to EXT:cshmanual 19/42719/6
Benjamin Mack [Tue, 18 Aug 2015 16:52:34 +0000 (18:52 +0200)]
[TASK] Move CSH Javascript to EXT:cshmanual

Releases: master
Resolves: #69161
Change-Id: If0df623ea07c06781aad7f89496919ed2a5f8729
Reviewed-on: http://review.typo3.org/42719
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
4 years ago[FOLLOWUP][TASK] Migrate tce_db, tce_file and FormEngine entry points to routing 24/42724/2
Wouter Wolters [Tue, 18 Aug 2015 17:58:43 +0000 (19:58 +0200)]
[FOLLOWUP][TASK] Migrate tce_db, tce_file and FormEngine entry points to routing

Fix broken unit-tests.

Resolves: #69162
Releases: master
Change-Id: I4331db64694bb3921cdce1795f155afeddba5d5b
Reviewed-on: http://review.typo3.org/42724
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
4 years ago[TASK] Migrate tce_db, tce_file and FormEngine entry points to routing 21/42721/5
Benjamin Mack [Tue, 18 Aug 2015 16:56:54 +0000 (18:56 +0200)]
[TASK] Migrate tce_db, tce_file and FormEngine entry points to routing

Resolves: #69162
Releases: master
Change-Id: I1cd25425cf344212c192c78bdf4e2f002fb66173
Reviewed-on: http://review.typo3.org/42721
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
4 years ago[!!!][TASK] Remove Extbase ModuleRunner 02/42702/5
Benjamin Mack [Mon, 17 Aug 2015 13:00:46 +0000 (15:00 +0200)]
[!!!][TASK] Remove Extbase ModuleRunner

The ModuleRunner and its interface were initially designed
to allow backend modules other than the "classic" modules that
need a conf.php and index.php to be a flexible entry point

Resolves: #69148
Releases: master
Change-Id: I695e41544f2154115b95c1b120da7c227abd4bcf
Reviewed-on: http://review.typo3.org/42702
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
4 years ago[TASK] Do not store files directly into typo3temp 34/42634/7
Markus Klein [Fri, 14 Aug 2015 21:13:23 +0000 (23:13 +0200)]
[TASK] Do not store files directly into typo3temp

Use a subfolder within typo3temp if files are meant to be persisted
there. Only allow Install Tool, UnitTests and temporary files for a
request to reside in the root typo3temp folder.

Resolves: #69078
Releases: master
Change-Id: I6202e84fefc7daff9dfcecb1c074d8e64de24718
Reviewed-on: http://review.typo3.org/42634
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Tested-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
4 years ago[TASK] Tests: Include all NOT NULL fields in EXT:extbase fixtures 30/42630/2
Morton Jonuschat [Fri, 14 Aug 2015 09:00:09 +0000 (11:00 +0200)]
[TASK] Tests: Include all NOT NULL fields in EXT:extbase fixtures

The fixtures used by the functional tests are incomplete and don't
contain all fields that are defined as NOT NULL in the database schema.
Add the missing information to the fixtures to successfully create the
test fixtures in MySQL strict mode.

Resolves: #69072
Releases: master
Change-Id: I793ca8d0cb07ae2c36c1bc74cb473b01ac45594f
Reviewed-on: http://review.typo3.org/42630
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Alexander Opitz <opitz.alexander@googlemail.com>
Tested-by: Alexander Opitz <opitz.alexander@googlemail.com>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
4 years ago[TASK] Tests: Include all NOT NULL fields in EXT:workspaces fixtures 28/42628/3
Morton Jonuschat [Fri, 14 Aug 2015 13:40:21 +0000 (15:40 +0200)]
[TASK] Tests: Include all NOT NULL fields in EXT:workspaces fixtures

The fixtures used by the functional tests are incomplete and don't
contain all fields that are defined as NOT NULL in the database schema.
Add the missing information to the fixtures to make tests pass when
running in MySQL strict mode.

Resolves: #69059
Releases: master
Change-Id: I466cf8fd965918096cc72710a8637e82d26d48c9
Reviewed-on: http://review.typo3.org/42628
Reviewed-by: Frans Saris <franssaris@gmail.com>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
4 years ago[TASK] Tests: Include all NOT NULL fields in CategoryCollection fixtures 48/42648/2
Morton Jonuschat [Thu, 13 Aug 2015 17:34:23 +0000 (19:34 +0200)]
[TASK] Tests: Include all NOT NULL fields in CategoryCollection fixtures

The fixtures used by the functional tests are incomplete and don't
contain all fields that are defined as NOT NULL in the database schema.
Add the missing information so the fixtures can be successfully inserted
when running in MySQL strict mode.

Resolves: #69031
Releases: master
Change-Id: I0347cae10646e29a62f1d8729a89a4ffed491895
Reviewed-on: http://review.typo3.org/42648
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
4 years ago[TASK] Tests: Include all NOT NULL fields in EXT:impexp fixtures 27/42627/2
Morton Jonuschat [Fri, 14 Aug 2015 13:39:44 +0000 (15:39 +0200)]
[TASK] Tests: Include all NOT NULL fields in EXT:impexp fixtures

The fixtures used by the functional tests are incomplete and don't
contain all fields that are defined as NOT NULL in the database schema.
Add the missing information to the fixtures to make tests pass when
running in MySQL strict mode.

Resolves: #69058
Releases: master
Change-Id: Ic55f83c0a41b29f88e8ccbc3220d2641b0b83150
Reviewed-on: http://review.typo3.org/42627
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
4 years ago[TASK] Tests: Include all NOT NULL fields in DataHandling fixtures 24/42624/2
Morton Jonuschat [Fri, 14 Aug 2015 08:23:53 +0000 (10:23 +0200)]
[TASK] Tests: Include all NOT NULL fields in DataHandling fixtures

The fixtures used by the functional tests are incomplete and don't
contain all fields that are defined as NOT NULL in the database schema.
Add the missing information to the fixtures to make tests pass when
running in MySQL strict mode.

Resolves: #69043
Resolves: #69051
Releases: master
Change-Id: I264cd809414b9b45b0bf1b14f33562ed298c4a3f
Reviewed-on: http://review.typo3.org/42624
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
4 years ago[TASK] Use 'wizard_element_browser' instead of 'browser' in FormEngine 92/42692/2
Markus Klein [Mon, 17 Aug 2015 20:12:06 +0000 (22:12 +0200)]
[TASK] Use 'wizard_element_browser' instead of 'browser' in FormEngine

FormEngine is the last place using the module 'browser'.
This module is the ElementBrowserFramesetController,
which uses a frameset. The frameset has no further functionality
and the additional JS provided by the class, is not used anymore.

We can therefore use the 'wizard_element_browser' module directly,
which does not render a frameset.

Resolves: #69140
Releases: master
Change-Id: Ibfc8147a6e822655ebd2e8f347f6f55c0e8382e5
Reviewed-on: http://review.typo3.org/42692
Reviewed-by: Benni Mack <benni@typo3.org>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Alexander Opitz <opitz.alexander@googlemail.com>
Tested-by: Alexander Opitz <opitz.alexander@googlemail.com>
Tested-by: Benni Mack <benni@typo3.org>
4 years ago[TASK] Remove package class source from package manager cache 93/42693/3
Helmut Hummel [Mon, 17 Aug 2015 22:18:57 +0000 (00:18 +0200)]
[TASK] Remove package class source from package manager cache

Writing the package class source to the package manager cache
has become pointless after package management cleanup.
It is only the source of one class and this source stays always
the same. We do not win anything from writing it to the cache.

Resolves: #69142
Releases: master
Change-Id: I8b910c3e133b1ffd9f0efbdbceb6dd067d9d6f78
Reviewed-on: http://review.typo3.org/42693
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
4 years ago[FEATURE] Viewpage language selector 62/27862/7
Erik Frister [Wed, 26 Feb 2014 10:02:06 +0000 (11:02 +0100)]
[FEATURE] Viewpage language selector

Adds a language selector to the viewpage module.

Resolves: #56282
Releases: master
Change-Id: Iae19f533f447c20aab7d672eff9aa1d202365363
Reviewed-on: http://review.typo3.org/27862
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Tested-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
4 years ago[TASK] Add typehints for ElementBrowser usage via SOBE 91/42691/5
Markus Klein [Mon, 17 Aug 2015 22:03:33 +0000 (00:03 +0200)]
[TASK] Add typehints for ElementBrowser usage via SOBE

The TreeViews access the controllers via the global SOBE.
This patch adds typehints to easily identify those usages.

Resolves: #69139
Releases: master
Change-Id: Ib280d4f9a2eeeb347893d1a9ab761760d993cac5
Reviewed-on: http://review.typo3.org/42691
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Tested-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
4 years ago[TASK] Remove width/height fields for links in ElementBrowser 90/42690/4
Markus Klein [Mon, 17 Aug 2015 18:19:15 +0000 (20:19 +0200)]
[TASK] Remove width/height fields for links in ElementBrowser

Resolves: #69137
Releases: master
Change-Id: I6da10b4531bce9e13a9b062e43fa6b4b1e33c6fb
Reviewed-on: http://review.typo3.org/42690
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Tested-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Reviewed-by: Mathias Schreiber <mathias.schreiber@wmdb.de>
Tested-by: Mathias Schreiber <mathias.schreiber@wmdb.de>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
4 years ago[TASK] Update FontAwesome to version 4.4.0 83/42683/3
Andreas Fernandez [Mon, 17 Aug 2015 13:42:57 +0000 (15:42 +0200)]
[TASK] Update FontAwesome to version 4.4.0

Resolves: #69131
Releases: master
Change-Id: I4df75e2f9b89e4ff3d50c31eef565a9b8965b4a8
Reviewed-on: http://review.typo3.org/42683
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Tested-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
4 years ago[BUGFIX] Assure language override without original file 24/37524/5
Stefan Galinski [Wed, 4 Mar 2015 14:50:20 +0000 (15:50 +0100)]
[BUGFIX] Assure language override without original file

The language override of language files is currently broken,
because it only works if a localized language file already
exists inside the extension or l10n directory. This patch assures
that the language override takes place even if no file could
be found in the first place.

Resolves: #65513
Releases: master, 6.2
Change-Id: I9269e60c5788c435b06b820c459fd4f077d066ea
Reviewed-on: http://review.typo3.org/37524
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Daniel Maier <dani-maier@gmx.de>
Reviewed-by: Alexander Opitz <opitz.alexander@googlemail.com>
Tested-by: Alexander Opitz <opitz.alexander@googlemail.com>
Reviewed-by: Stefan Galinski <stefan.galinski@gmail.com>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
4 years ago[CLEANUP] ElementBrowser classes 89/42689/4
Markus Klein [Mon, 17 Aug 2015 17:29:33 +0000 (19:29 +0200)]
[CLEANUP] ElementBrowser classes

Next step in cleanup and separation of concerns.
Removal of unused or redundant code.

Resolves: #69136
Releases: master
Change-Id: I4a81c6856bf44cf15ceea1cad9e41f00ba43f70a
Reviewed-on: http://review.typo3.org/42689
Reviewed-by: Mathias Schreiber <mathias.schreiber@wmdb.de>
Tested-by: Mathias Schreiber <mathias.schreiber@wmdb.de>
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Tested-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
4 years ago[BUGFIX] Extbase: Use fully populated records in Backend::insertObject() 56/42656/3
Morton Jonuschat [Sat, 15 Aug 2015 20:05:47 +0000 (22:05 +0200)]
[BUGFIX] Extbase: Use fully populated records in Backend::insertObject()

To successfully insert new records into a database that does not
silently convert NULL values being inserted into NOT NULL columns (MySQL
in strict mode, PostgreSQL) it is necessary to use a fully
populated record, even if it contains only placeholders for some fields
like relations or reference counters.

Resolves: #69084
Releases: master
Change-Id: I89e79d59ad53666432e73c3b1651323d0d506ac3
Reviewed-on: http://review.typo3.org/42656
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Tested-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Reviewed-by: Alexander Schnitzler <typo3@alexanderschnitzler.de>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
4 years ago[TASK] Make sysext filelist an extbase module 44/42644/9
Alexander Schnitzler [Sat, 15 Aug 2015 12:56:48 +0000 (14:56 +0200)]
[TASK] Make sysext filelist an extbase module

Currently the filelist module is not yet an extbase
module. To be able to adjust the module in the future
more easily the module will be based on extbase.

Releases: master
Resolves: #69083
Change-Id: I74884cd554272b68ab80509b7548647ff01632d6
Reviewed-on: http://review.typo3.org/42644
Reviewed-by: Carsten Bleicker <carsten@bleicker.de>
Tested-by: Carsten Bleicker <carsten@bleicker.de>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Tested-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
4 years ago[BUGFIX] Icon state toggling in record list 81/42681/3
Andreas Fernandez [Mon, 17 Aug 2015 12:44:59 +0000 (14:44 +0200)]
[BUGFIX] Icon state toggling in record list

When the toggle icon is clicked, the state icon class gets removed,
but the called toggleClass() methods add both icon state classes
again. Change the JavaScript to only add the required icon state
class to the button.

Resolves: #69128
Releases: master
Change-Id: I192163ba35ca0a11fb6c8621af8be04796c82b61
Reviewed-on: http://review.typo3.org/42681
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Alexander Opitz <opitz.alexander@googlemail.com>
Tested-by: Alexander Opitz <opitz.alexander@googlemail.com>
4 years ago[BUGFIX] Recycler: Reset internal toggle state after bulk action 82/42682/2
Andreas Fernandez [Mon, 17 Aug 2015 12:58:57 +0000 (14:58 +0200)]
[BUGFIX] Recycler: Reset internal toggle state after bulk action

Reset the internal toggle state in the recycler after a bulk action to get the
button responding after a single click again.

Resolves: #69129
Releases: master
Change-Id: I85d4df932f27cfbd48ac19e4d698e56dc71caec5
Reviewed-on: http://review.typo3.org/42682
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Alexander Opitz <opitz.alexander@googlemail.com>
Tested-by: Alexander Opitz <opitz.alexander@googlemail.com>
4 years ago[TASK] Move record-related entrypoints to backend routing 09/42609/3
Manuel Selbach [Fri, 14 Aug 2015 08:00:21 +0000 (10:00 +0200)]
[TASK] Move record-related entrypoints to backend routing

Resolves: #69038
Releases: master
Change-Id: I6ad5e541b1741f0b29a1f68d631404cb5a6d38d2
Reviewed-on: http://review.typo3.org/42609
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
4 years ago[TASK] Update jQuery to 2.x 94/42494/4
Stefan Neufeind [Wed, 12 Aug 2015 18:40:24 +0000 (20:40 +0200)]
[TASK] Update jQuery to 2.x

Change-Id: Ib39b7e52b794ec375027a3ac1315c6e84e345f93
Resolves: #68917
Releases: master
Reviewed-on: http://review.typo3.org/42494
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Andreas Fernandez <typo3@scripting-base.de>
4 years ago[BUGFIX] Apply default values in DataHandler functional tests 98/42598/5
Oliver Hader [Thu, 13 Aug 2015 17:30:42 +0000 (19:30 +0200)]
[BUGFIX] Apply default values in DataHandler functional tests

If MySQL is used in strict mode, setting up the functional
testing scenario requires proper field values for those
fields not having a default value defined in the schema.
For some field types, like BLOB or TEXT this is not possible
and thus has to be determined from the accordant TCA field
configurations.

Resolves: #69030
Releases: master
Change-Id: Ie915fdaca6c90440259e01896677a2d2395417f4
Reviewed-on: http://review.typo3.org/42598
Reviewed-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Tested-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Reviewed-by: Oliver Hader <oliver.hader@typo3.org>
Tested-by: Oliver Hader <oliver.hader@typo3.org>
4 years ago[TASK] Make usage of TYPO3_PATH_WEB failsafe 76/42676/3
Alexander Schnitzler [Mon, 17 Aug 2015 09:15:14 +0000 (11:15 +0200)]
[TASK] Make usage of TYPO3_PATH_WEB failsafe

Using the TYPO3_PATH_WEB env var with a trailing
slash leads to failing unit/functional tests.

Trimming and appending a slash makes the usage
fail safe.

Releases: master
Resolves: #69124
Change-Id: I5048de4c9d35400c148cb5c40cf516d0d684b47a
Reviewed-on: http://review.typo3.org/42676
Reviewed-by: Helmut Hummel <helmut.hummel@typo3.org>
Tested-by: Helmut Hummel <helmut.hummel@typo3.org>
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Tested-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
4 years ago[BUGFIX] Fix QueryBuilding in IndexSearchRepository 19/42219/4
Marc Neuhaus [Mon, 3 Aug 2015 11:45:57 +0000 (13:45 +0200)]
[BUGFIX] Fix QueryBuilding in IndexSearchRepository

This changeset fixes an issue in the indexSearchRepository that creates
invalid sql queries. Main issue is the missing of several wSelClauses,
that need to be collected, because they are imploded into the query.

Resolves: #67843
Releases: master, 6.2
Change-Id: Ib53b28660acfd92233de748431131f70e69f5125
Reviewed-on: http://review.typo3.org/42219
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Marc Neuhaus <apocalip@gmail.com>
Reviewed-by: Alexander Opitz <opitz.alexander@googlemail.com>
Tested-by: Alexander Opitz <opitz.alexander@googlemail.com>
4 years ago[BUGFIX] Allow attributes with = in cObj MEDIA 99/41699/4
Georg Ringer [Sun, 19 Jul 2015 09:14:54 +0000 (11:14 +0200)]
[BUGFIX] Allow attributes with = in cObj MEDIA

Set limit to 2 in trimExplode to make it possible to use '=' in the
value of a attribute.

Also URL encode flashvars values as per Flash documentation, see
https://helpx.adobe.com/flash/kb/pass-variables-swfs-flashvars.html

Change-Id: I8bebf1afd8ef521268306660129a035e429e5d31
Resolves: #25298
Releases: master
Reviewed-on: http://review.typo3.org/41699
Reviewed-by: Mathias Brodala <mbrodala@pagemachine.de>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Tested-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
4 years ago[BUGFIX] Evaluate TCEFORM fields when changed by a wizard 59/42659/3
Frank Nägler [Sun, 16 Aug 2015 10:41:50 +0000 (12:41 +0200)]
[BUGFIX] Evaluate TCEFORM fields when changed by a wizard

Resolves: #68982
Releases: master
Change-Id: Ia61ba717a080bc4c9110594a03cbaeff3744d087
Reviewed-on: http://review.typo3.org/42659
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Sascha Egerer <sascha@sascha-egerer.de>
Tested-by: Sascha Egerer <sascha@sascha-egerer.de>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Tested-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
4 years ago[BUGFIX] Reestablish DB connection in long-running tasks 46/42546/5
Stefan Neufeind [Wed, 12 Aug 2015 18:31:33 +0000 (20:31 +0200)]
[BUGFIX] Reestablish DB connection in long-running tasks

During a long-running task the connection to the database may be
disconnected, if mysql `wait_timeout` is exceeded. This patch makes
sure the connection is reestablished if there has been a connection
before.

Resolves: #69001
Releases: master, 6.2
Change-Id: I7575ae386ec190d60ddceee146133125083dee41
Reviewed-on: http://review.typo3.org/42546
Reviewed-by: Stephan Großberndt <stephan@grossberndt.de>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Jan Helke <typo3@helke.de>
Tested-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Josef Glatz <jousch@gmail.com>
Tested-by: Josef Glatz <jousch@gmail.com>
Reviewed-by: Alexander Schnitzler <typo3@alexanderschnitzler.de>
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Tested-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
4 years ago[BUGFIX] FAL: Initialize all NOT NULL fields for new metadata records 29/42629/2
Morton Jonuschat [Fri, 14 Aug 2015 16:09:55 +0000 (18:09 +0200)]
[BUGFIX] FAL: Initialize all NOT NULL fields for new metadata records

The columns l10n_diffsource is defined as NOT NULL in the database
schema. When a new record is being created through the method
createMetaDataRecord() this field is not being set, causing errors when
running in MySQL strict mode.

Resolves: #69071
Releases: master
Change-Id: Ie0465abc90c9855c84c8c9bb1c59909316c8dc2e
Reviewed-on: http://review.typo3.org/42629
Reviewed-by: Frans Saris <franssaris@gmail.com>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
4 years ago[TASK] Make list of field selector in record listview scrollable 63/42463/5
Frank Nägler [Sun, 9 Aug 2015 18:14:57 +0000 (20:14 +0200)]
[TASK] Make list of field selector in record listview scrollable

Resolves: #68889
Releases: master
Related: #62323
Change-Id: I77c578f34c30bcbd5793d32ed06604cdf548e17f
Reviewed-on: http://review.typo3.org/42463
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Tested-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
4 years ago[TASK] Improve appearance of linkvalidator 58/42658/3
Daniel Goerz [Sun, 16 Aug 2015 07:02:15 +0000 (09:02 +0200)]
[TASK] Improve appearance of linkvalidator

Change-Id: I70c0d9f640a92d1466294f988fab7b918fe3fb3c
Resolves: #69093
Releases: master
Reviewed-on: http://review.typo3.org/42658
Reviewed-by: Daniel Maier <dani-maier@gmx.de>
Tested-by: Daniel Maier <dani-maier@gmx.de>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
4 years ago[TASK] Add missing unit tests for SilentConfigurationUpgradeService (4) 45/42545/7
Michael Oehlhof [Wed, 12 Aug 2015 17:56:20 +0000 (19:56 +0200)]
[TASK] Add missing unit tests for SilentConfigurationUpgradeService (4)

Missing unit tests for generateEncryptionKeyIfNeeded() are added.

Resolves: #68996
Releases: master
Change-Id: Ia9c88be6dfdef1d7269960f39a17bccedf25ff69
Reviewed-on: http://review.typo3.org/42545
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Tested-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
4 years ago[TASK] Add missing unit tests for SilentConfigurationUpgradeService (6) 33/42633/3
Michael Oehlhof [Fri, 14 Aug 2015 20:18:48 +0000 (22:18 +0200)]
[TASK] Add missing unit tests for SilentConfigurationUpgradeService (6)

Missing unit tests for removeObsoleteLocalConfigurationSettings()
are added.

Resolves: #69077
Releases: master
Change-Id: Icb78d8ba81bdd5f936238629f9fc4c565df44637
Reviewed-on: http://review.typo3.org/42633
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Tested-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
4 years ago[BUGFIX] Extbase: Optional arguments must not throw an Exception 46/39946/3
Anja Leichsenring [Thu, 4 Jun 2015 09:27:42 +0000 (11:27 +0200)]
[BUGFIX] Extbase: Optional arguments must not throw an Exception

Example:
public function barAction(\Vendor\Ext\Domain\Model\Foo $foo = NULL)

If the property mapper receives NULL, 0 or a valid value
the property mapping does not fail. But if the value is
a valid uid (>0) and there is no record with that uid
the whole property mapping fails.

While this is totally valid for required arguments,
optional arguments must be skipped.

Resolves: #55861
Resolves: #55837
Resolves: #62951
Releases: master, 6.2
Change-Id: I4c954bac50be1d34ea09bfc21f8978e9165683d1
Reviewed-on: http://review.typo3.org/39946
Reviewed-by: Alexander Schnitzler <typo3@alexanderschnitzler.de>
Tested-by: Alexander Schnitzler <typo3@alexanderschnitzler.de>
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Tested-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
4 years ago[TASK] Replace sprite icons for history with IconFactory 35/42535/8
Markus Guenther [Wed, 12 Aug 2015 18:53:48 +0000 (20:53 +0200)]
[TASK] Replace sprite icons for history with IconFactory

Replaces all IconUtility::getSpriteIcon calls for the icon
actions-markstate and actions-unmarkstate with the new IconFactory.

Change-Id: I8caec19854a56fcd4fe2b55e7b159936430a1ec9
Resolves: #68975
Releases: master
Reviewed-on: http://review.typo3.org/42535
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Tested-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Reviewed-by: Daniel Maier <dani-maier@gmx.de>
Tested-by: Daniel Maier <dani-maier@gmx.de>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
4 years ago[FEATURE] Added support for multiple FlashMessage queues 89/36589/10
Stefan Neufeind [Tue, 14 Jul 2015 18:43:06 +0000 (20:43 +0200)]
[FEATURE] Added support for multiple FlashMessage queues

Each controller defaults to its own queue.
Allow to access other queues like queues from different
controllers or split message into multiple queues for
specific parts in one controller

The FlashMessageService already supports multiple queues.
It just has a different default-queue.

Since FlashMessageService already is a Singleton, avoid
separate caching of queues in the ControllerContext.

Change-Id: I5e0bb867c6b02979b899e46bb42bf209b6a8e47a
Resolves: #64726
Releases: master
Reviewed-on: http://review.typo3.org/36589
Reviewed-by: Alexander Schnitzler <typo3@alexanderschnitzler.de>
Tested-by: Alexander Schnitzler <typo3@alexanderschnitzler.de>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
4 years ago[TASK] Properly encode database input in the right place 14/42614/4
Helmut Hummel [Fri, 14 Aug 2015 13:45:15 +0000 (15:45 +0200)]
[TASK] Properly encode database input in the right place

This adds code that prevents potential SQL injections.
The core is not exploitable, as cleaning is done in other code parts.

Resolves: #69061
Releases: master, 6.2
Change-Id: Iba42adc6dd4abd3976b57f1dc84ba6585ea7bbd4
Reviewed-on: http://review.typo3.org/42614
Reviewed-by: Frank Nägler <frank.naegler@typo3.org>
Tested-by: Frank Nägler <frank.naegler@typo3.org>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Reviewed-by: Nicole Cordes <typo3@cordes.co>
Tested-by: Nicole Cordes <typo3@cordes.co>