Packages/TYPO3.CMS.git
2 years ago[TASK] Split fluid styled content rendering definitions 55/51055/4
Benjamin Kott [Wed, 28 Dec 2016 23:36:11 +0000 (00:36 +0100)]
[TASK] Split fluid styled content rendering definitions

Moving the rendering definitions for content elements to dedicated
files provides a better overview over the configuration and creates
a single responsibility per configuration file.

Resolves: #79102
Releases: master

Change-Id: Iecdecaf224c3db2f8d4f319963e09f7ea0dab45b
Reviewed-on: https://review.typo3.org/51055
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Michael Oehlhof <typo3@oehlhof.de>
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: Susanne Moog <susanne.moog@typo3.org>
Reviewed-by: Tobias Wollender <tliegl@gmail.com>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
2 years ago[TASK] EXT:Scheduler: Removed unneccesary Annotation from CronCommand 39/51039/2
Michael Oehlhof [Fri, 23 Dec 2016 13:18:26 +0000 (14:18 +0100)]
[TASK] EXT:Scheduler: Removed unneccesary Annotation from CronCommand

The unneccesary @return annotation is removed from the
function __construct.

Resolves: #79087
Releases: master
Change-Id: Ib588d8e641e7ec4a81b36ffef151b2c885458fd2
Reviewed-on: https://review.typo3.org/51039
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
2 years ago[TASK] Update ckeditor to 4.6.1 39/50939/2
Andreas Fernandez [Mon, 12 Dec 2016 18:14:42 +0000 (19:14 +0100)]
[TASK] Update ckeditor to 4.6.1

New Features:
* The callback parameter in the CKEDITOR.ajax.post method became optional.

Fixed Issues:
* [Blink, WebKit] Fixed: Cannot select all editor content when a widget or
a non-editable element is the first or last element of the content. Also
fixes this issue in the Select All plugin.
* [Blink, WebKit, IE8] Fixed: Browser hangs when a table is inserted in
the place of a selected list with an empty last item.
* Fixed: Improved the Color Button plugin which will now normalize the CSS
background property if it only contains a color value. This fixes missing
background colors when using Paste from Word.
+ [Blink, WebKit] Fixed: Error thrown occasionally by an uninitialized
editable for multiple CKEditor instances on the same page.

Resolves: #78966
Releases: master
Change-Id: I2e711c363dcc8a4a00f7e88828105de987968da4
Reviewed-on: https://review.typo3.org/50939
Tested-by: TYPO3com <no-reply@typo3.com>
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>
2 years ago[!!!][TASK] felogin: remove default css 53/51053/9
Josef Glatz [Thu, 22 Dec 2016 12:43:02 +0000 (13:43 +0100)]
[!!!][TASK] felogin: remove default css

There is no need for this css style. Whenever you start a
new installation, you write your CSS for your website by your
own. Therefore it makes no sense to only style the label tag
of EXT:felogin plugins with `display: block;`.

Resolves: #79100
Releases: master
Change-Id: If346eef97a6066077f08d9fb8b944a2543d7cafe
Reviewed-on: https://review.typo3.org/51053
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Andreas Fernandez <typo3@scripting-base.de>
2 years ago[TASK] Update phpunit to version 5.7.4 34/51034/2
Anja Leichsenring [Thu, 22 Dec 2016 21:43:13 +0000 (22:43 +0100)]
[TASK] Update phpunit to version 5.7.4

Resolves: #79082
Releases: master
Change-Id: I1de2911243d5dfd8a172d9dac60cbe93f36c26f8
Reviewed-on: https://review.typo3.org/51034
Tested-by: TYPO3com <no-reply@typo3.com>
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>
2 years ago[FOLLOWUP][TASK] Extract testing framework for TYPO3 35/51035/11
Anja Leichsenring [Thu, 22 Dec 2016 21:55:00 +0000 (22:55 +0100)]
[FOLLOWUP][TASK] Extract testing framework for TYPO3

Acceptance Tests are not found, path needs adjustment,
References need to be adjusted to the new paths.

Resolves: #79025
Releases: master
Change-Id: Id2fcfeb4e5171c89c421e0af75ddaa7e6acf6cf3
Reviewed-on: https://review.typo3.org/51035
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Mathias Schreiber <mathias.schreiber@typo3.com>
Tested-by: Mathias Schreiber <mathias.schreiber@typo3.com>
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Andreas Fernandez <typo3@scripting-base.de>
2 years ago[TASK] Integrate driver into functional test bootstrap 14/50914/4
Manuel Selbach [Fri, 9 Dec 2016 09:47:11 +0000 (10:47 +0100)]
[TASK] Integrate driver into functional test bootstrap

According to doctrine migration functional tests should be processed against
different dbms. Thus, we have to integrate the driver to the bootstrap. The
driver can be set by environment variable name 'typo3DatabaseDriver'.

Change-Id: Ia725285a695fbb0e11555c7371cfd2165ba30d77
Resolves: #78930
Releases: master
Reviewed-on: https://review.typo3.org/50914
Reviewed-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Tested-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Reviewed-by: Manuel Selbach <manuel_selbach@yahoo.de>
Tested-by: Manuel Selbach <manuel_selbach@yahoo.de>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
2 years ago[TASK] Ensure consistent handling of BLOBs 83/50883/8
Morton Jonuschat [Sun, 4 Dec 2016 19:40:00 +0000 (11:40 -0800)]
[TASK] Ensure consistent handling of BLOBs

Set PDO attributes to ensure that BLOB fields are returned as strings to PHP,
the way MySQL does. Update INSERT/UPDATE statements that deal with BLOB field
to declare the proper parameter type so that the driver can do the required
encoding/escaping when sending the data to the DBMS.

Releases: master
Resolves: #78884
Change-Id: I1618da9130549bd95ce0410420ecdee40cc3632d
Reviewed-on: https://review.typo3.org/50883
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Manuel Selbach <manuel_selbach@yahoo.de>
Tested-by: Manuel Selbach <manuel_selbach@yahoo.de>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
2 years ago[TASK] Update datatables to 1.10.13 36/50936/2
Andreas Fernandez [Mon, 12 Dec 2016 17:55:39 +0000 (18:55 +0100)]
[TASK] Update datatables to 1.10.13

See a list of fixes at https://cdn.datatables.net/1.10.13/

Resolves: #78963
Releases: master
Change-Id: Ic59257fd90ce552d8b59596aaf47ba92d59087b7
Reviewed-on: https://review.typo3.org/50936
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Markus Sommer <markussom@posteo.de>
Tested-by: Markus Sommer <markussom@posteo.de>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
2 years ago[TASK] Testing Framework: Move fluid base test and ensure BC 23/51023/5
Susanne Moog [Thu, 22 Dec 2016 14:45:51 +0000 (15:45 +0100)]
[TASK] Testing Framework: Move fluid base test and ensure BC

After the merge of #79025 now the fluid base test case for viewhelpers
should be moved to the testing framework.

A compat layer for the old namespaces is introduced.

Additionally some small leftovers in documentation and CI configuration were fixed.

Change-Id: I78b9670a1ee1fadb4f4c36265c3a51cd1e879e50
Releases: master
Resolves: #79075
Reviewed-on: https://review.typo3.org/51023
Tested-by: Helmut Hummel <typo3@helhum.io>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
Reviewed-by: Nicole Cordes <typo3@cordes.co>
Tested-by: Nicole Cordes <typo3@cordes.co>
Reviewed-by: Helmut Hummel <typo3@helhum.io>
2 years ago[BUGFIX] Remove wrong queryBuilder usage in prepareCacheFlush 24/51024/3
Wouter Wolters [Thu, 22 Dec 2016 15:05:59 +0000 (16:05 +0100)]
[BUGFIX] Remove wrong queryBuilder usage in prepareCacheFlush

Due to a wrong queryBuilder being used the parameters stay empty.
Use the correct queryBuilder to prevent SQL errors.

Resolves: #79077
Releases: master
Change-Id: I62b28f7eeb0a1cc4d481e63a46f9784159116b6e
Reviewed-on: https://review.typo3.org/51024
Reviewed-by: Steffen Ritter <info@rs-websystems.de>
Tested-by: Steffen Ritter <info@rs-websystems.de>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Oliver Hader <oliver.hader@typo3.org>
Tested-by: Oliver Hader <oliver.hader@typo3.org>
2 years ago[!!!][TASK] Extract testing framework for TYPO3 25/50125/18
Susanne Moog [Sun, 18 Dec 2016 12:12:41 +0000 (13:12 +0100)]
[!!!][TASK] Extract testing framework for TYPO3

Add testing framework component for independent usage

Resolves: #79025
Releases: master
Change-Id: I23cfd2ed42108d7d80ec776e778a1ac2d5293e55
Reviewed-on: https://review.typo3.org/50125
Reviewed-by: Oliver Hader <oliver.hader@typo3.org>
Tested-by: TYPO3com <no-reply@typo3.com>
Tested-by: Oliver Hader <oliver.hader@typo3.org>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
2 years ago[BUGFIX] Show loading label while scanning broken extensions 79/50979/4
Nicole Cordes [Thu, 15 Dec 2016 22:30:04 +0000 (23:30 +0100)]
[BUGFIX] Show loading label while scanning broken extensions

When switching to Bootstrap in the Install Tool, the class adjustment
for the loading message in LoadExtensions.html was missing. This patch
ensures the correct class is used to show the message.

Resolves: #79010
Related: #78376
Releases: master, 7.6
Change-Id: I9c467dd3e0cf7c801ac25774425df7e1d85b4c0a
Reviewed-on: https://review.typo3.org/50979
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
2 years ago[BUGFIX] Add missing getTemplatePathAndFilename() in TemplatePaths 04/51004/7
Daniel Lorenz [Tue, 20 Dec 2016 11:43:27 +0000 (12:43 +0100)]
[BUGFIX] Add missing getTemplatePathAndFilename() in TemplatePaths

The StandaloneView getTemplatePathAndFilename() calls
a method that the fluid standalone extension doesn't implement anymore,
hence the method is added to the Core's TemplatePaths class.

Additionally, a lot of type checks are added to ensure that the correct
object types are present in all usages.

Resolves: #79045
Releases: master
Change-Id: I3ec766a29c31d299451f33dd079a56e9a5855c69
Reviewed-on: https://review.typo3.org/51004
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Nicole Cordes <typo3@cordes.co>
Tested-by: Nicole Cordes <typo3@cordes.co>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
2 years ago[TASK] Update TYPO3 version to TYPO3 8.6-dev 09/51009/2
Wouter Wolters [Tue, 20 Dec 2016 17:46:36 +0000 (18:46 +0100)]
[TASK] Update TYPO3 version to TYPO3 8.6-dev

Resolves: #79055
Releases: master
Change-Id: I09921a6c4fc013af2a01b498712d282559028b37
Reviewed-on: https://review.typo3.org/51009
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
2 years ago[BUGFIX] Add default values from setup.txt and raise override to 20 77/50977/4
Boris Schauer [Thu, 15 Dec 2016 13:36:40 +0000 (14:36 +0100)]
[BUGFIX] Add default values from setup.txt and raise override to 20

Update the indexed_search extension documentation with
proper TypoScript values for overriding templates.

Resolves: #79006
Releases: master, 7.6

Change-Id: Ie034d6b192675fa1a453622b90296d39c80d2447
Reviewed-on: https://review.typo3.org/50977
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Stephan Großberndt <stephan@grossberndt.de>
Reviewed-by: Mathias Schreiber <mathias.schreiber@wmdb.de>
Tested-by: Mathias Schreiber <mathias.schreiber@wmdb.de>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
2 years ago[BUGFIX] Set title for select icons 97/50997/2
Marco Huber [Mon, 19 Dec 2016 16:50:02 +0000 (17:50 +0100)]
[BUGFIX] Set title for select icons

Use the select options label as title instead of
an uninitialized variable.

Resolves: #79037
Releases: master, 7.6
Change-Id: I690b687f5dd854556ac799d89c8462f4ce9966a4
Reviewed-on: https://review.typo3.org/50997
Tested-by: TYPO3com <no-reply@typo3.com>
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>
2 years ago[TASK] Set TYPO3 version to 8.5.1-dev 08/51008/2
TYPO3 Release Team [Tue, 20 Dec 2016 14:11:14 +0000 (15:11 +0100)]
[TASK] Set TYPO3 version to 8.5.1-dev

Change-Id: I3f4991ebbf3117a062c8d3d63fd731990401ebe3
Reviewed-on: https://review.typo3.org/51008
Reviewed-by: TYPO3 Release Team <typo3cms@typo3.org>
Tested-by: TYPO3 Release Team <typo3cms@typo3.org>
2 years ago[RELEASE] Release of TYPO3 8.5.0 07/51007/2 8.5.0 TYPO3_8-5-0
TYPO3 Release Team [Tue, 20 Dec 2016 14:10:25 +0000 (15:10 +0100)]
[RELEASE] Release of TYPO3 8.5.0

Change-Id: I0bc2ac481447671f0354375d9433ed0b9a2f47c8
Reviewed-on: https://review.typo3.org/51007
Reviewed-by: TYPO3 Release Team <typo3cms@typo3.org>
Tested-by: TYPO3 Release Team <typo3cms@typo3.org>
2 years ago[BUGFIX] setPartialRootPaths in the backend tabs menu 48/50948/4
Alexander Opitz [Tue, 13 Dec 2016 09:16:43 +0000 (10:16 +0100)]
[BUGFIX] setPartialRootPaths in the backend tabs menu

Adding the root paths to the partials, which aren't autoguessed anymore.

Resolves: #78969
Releases: master
Change-Id: I578fd21ed0b232198301f73c702569980ed82906
Reviewed-on: https://review.typo3.org/50948
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Mathias Schreiber <mathias.schreiber@wmdb.de>
Tested-by: Mathias Schreiber <mathias.schreiber@wmdb.de>
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: Susanne Moog <susanne.moog@typo3.org>
Reviewed-by: Oliver Hader <oliver.hader@typo3.org>
Tested-by: Oliver Hader <oliver.hader@typo3.org>
2 years ago[DOCS] 8.5 Documentation 91/50991/4
Mathias Schreiber [Sat, 17 Dec 2016 16:55:23 +0000 (17:55 +0100)]
[DOCS] 8.5 Documentation

Added all documentation for CMS v8.5

Resolves: #
Releases: master
Change-Id: I6756ecd38a2d50553bcfc8687707b5c08f894435
Reviewed-on: https://review.typo3.org/50991
Reviewed-by: Michael Oehlhof <typo3@oehlhof.de>
Tested-by: Michael Oehlhof <typo3@oehlhof.de>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
2 years ago[FOLLOWUP][TASK] Remove support for transForeignTable in TCA 03/51003/2
Oliver Hader [Tue, 20 Dec 2016 10:29:18 +0000 (11:29 +0100)]
[FOLLOWUP][TASK] Remove support for transForeignTable in TCA

Some variable names have not been adjusted correctly.

Resolves: #78191
Releases: master
Change-Id: I107b9c5d146fb4a027d6457194c1c4e46f06cd8c
Reviewed-on: https://review.typo3.org/51003
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: TYPO3com <no-reply@typo3.com>
Tested-by: Susanne Moog <susanne.moog@typo3.org>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
2 years ago[BUGFIX] Allow t3:// syntax to be parsed by lib.parseFunc 00/51000/5
Benni Mack [Mon, 19 Dec 2016 20:36:21 +0000 (21:36 +0100)]
[BUGFIX] Allow t3:// syntax to be parsed by lib.parseFunc

When introducing the new t3:// linking syntax for typolink,
we also allowed to use them in the RTE without using <link>
anymore, which is really cool. However, the frontend (= TypoScript)
cannot deal with that yet.

A clean function within parseFunc should "transformLinks" with typolink.
For the time being, a parseFunc.tags.A is introduced to allow
the transformations to be rendered correctly.

It *could* have a side-effect that certain attributes
within parseFunc.tags.A are rendered twice, but can be solved
at a later point by sanitizing the attributes quite simple.

Resolves: #79041
Releases: master
Change-Id: Ib8da2a88b67a633fc89a5c1ddee90c8e14f5b27b
Reviewed-on: https://review.typo3.org/51000
Reviewed-by: Oliver Hader <oliver.hader@typo3.org>
Tested-by: Oliver Hader <oliver.hader@typo3.org>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Andreas Fernandez <typo3@scripting-base.de>
2 years ago[TASK] Re-set RTEHtmlArea as default RTE 99/50999/2
Benni Mack [Mon, 19 Dec 2016 19:54:10 +0000 (20:54 +0100)]
[TASK] Re-set RTEHtmlArea as default RTE

Due to the unfinished integration of CKEditor
in the TYPO3 Core, the default installation
should still use HtmlArea as default RTE
for new installations.

Resolves: #79040
Releases: master
Change-Id: If153ac427dad596ba470bb346d2257c9e0282a44
Reviewed-on: https://review.typo3.org/50999
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Oliver Hader <oliver.hader@typo3.org>
Reviewed-by: Frans Saris <franssaris@gmail.com>
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Susanne Moog <susanne.moog@typo3.org>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
2 years ago[TASK] Installation process hangs with PHP 7.1 98/50998/3
Oliver Hader [Mon, 19 Dec 2016 17:56:03 +0000 (18:56 +0100)]
[TASK] Installation process hangs with PHP 7.1

The lengthy switch/case part of the Lexer class seems to be
too much for PHP 7.1.0. Albeit this seems to be a PHP bug,
this patch is provided to help others and to clean up the
mentioned switch/case code.

Change-Id: I0e8e4ca55f2e15d0445874b44a04742645670570
Resolves: #79039
Releases: master
Reviewed-on: https://review.typo3.org/50998
Tested-by: TYPO3com <no-reply@typo3.com>
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>
2 years ago[TASK] Streamline EXT:form unit test 94/50994/2
Oliver Hader [Sun, 18 Dec 2016 12:21:09 +0000 (13:21 +0100)]
[TASK] Streamline EXT:form unit test

EXT:form's FormManagerControllerTest tries to mock
static method calls of BackendUtility. The methods
to be mocked are now wrapped directly instead of
"wrapping" the reference to the class name.

Change-Id: I7c6381fe093ecc28b85e235fcb425f6048c8ff0c
Resolves: #79023
Releases: master
Reviewed-on: https://review.typo3.org/50994
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Reviewed-by: Oliver Hader <oliver.hader@typo3.org>
Tested-by: Oliver Hader <oliver.hader@typo3.org>
2 years ago[BUGFIX] Exception in EXT:form due to invalid array lookup 95/50995/5
Oliver Hader [Sun, 18 Dec 2016 13:15:43 +0000 (14:15 +0100)]
[BUGFIX] Exception in EXT:form due to invalid array lookup

Due to the changes of ArrayUtility in issue #77732
EXT:form now has to deal with these problems:

+ core's ArrayUtility uses a slash "/" as delimiter,
  but Extbase's ArrayUtility used a dot "."
+ core's ArrayUtility throws an exception if value is
  not defined, Extbase's ArrayUtility returned null

Change-Id: I26b043a28e376c99d50a9a36703b5927b09ee934
Resolves: #79024
Releases: master
Reviewed-on: https://review.typo3.org/50995
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Ralf Zimmermann <ralf.zimmermann@tritum.de>
Tested-by: Ralf Zimmermann <ralf.zimmermann@tritum.de>
Reviewed-by: Benni Mack <benni@typo3.org>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
2 years ago[BUGFIX] Re-enable styling for workspace preview bar 95/50795/5
Benni Mack [Mon, 28 Nov 2016 16:24:17 +0000 (17:24 +0100)]
[BUGFIX] Re-enable styling for workspace preview bar

The workspace preview functionality (comparing versions etc)
is broken due to missing stylings since TYPO3 8.4

The patch fixes the CSS styles.

Resolves: #78820
Releases: master
Change-Id: I6ede8ee1913e91c0095d695ce2e5d90c16dbd5f7
Reviewed-on: https://review.typo3.org/50795
Reviewed-by: Benjamin Kott <info@bk2k.info>
Tested-by: Benjamin Kott <info@bk2k.info>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Andreas Fernandez <typo3@scripting-base.de>
2 years ago[FOLLOWUP] Add functional test for cache backend flushByTags 24/50924/9
Claus Due [Sun, 11 Dec 2016 00:00:36 +0000 (01:00 +0100)]
[FOLLOWUP] Add functional test for cache backend flushByTags

Followup to https://review.typo3.org/#/c/50537/ to add
functional tests covering the new SQL operations.

Change-Id: I7a686fe2ed10d53506ff36d99a21f10379a9656d
Resolves: #78951
Releases: master
Reviewed-on: https://review.typo3.org/50924
Reviewed-by: Nicole Cordes <typo3@cordes.co>
Tested-by: Nicole Cordes <typo3@cordes.co>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Claus Due <claus@phpmind.net>
Tested-by: Claus Due <claus@phpmind.net>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
2 years ago[BUGFIX] FormArrayUtility not found 93/50993/2
Oliver Hader [Sun, 18 Dec 2016 10:20:23 +0000 (11:20 +0100)]
[BUGFIX] FormArrayUtility not found

The FormArrayUtility of EXT:form cannot be found and
results in a PHP fatal error. Reason for that is the
ArrayUtility cleanup in issue #77732.

Change-Id: Iaa4ad928e664a56aac2d71ed39faecbb67e38864
Resolves: #79022
Releases: master
Reviewed-on: https://review.typo3.org/50993
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: Susanne Moog <susanne.moog@typo3.org>
Reviewed-by: Oliver Hader <oliver.hader@typo3.org>
Tested-by: Oliver Hader <oliver.hader@typo3.org>
2 years ago[BUGFIX] Optimize cache handling in ReflectionService 53/50953/8
Helmut Hummel [Tue, 13 Dec 2016 22:39:13 +0000 (23:39 +0100)]
[BUGFIX] Optimize cache handling in ReflectionService

Method reflection objects are used temporarily to extract information
from methods. This information is eventually stored in the cache.
However the method reflection objects themselves must stay outside the cache
because they cannot be serialized. They also do not provide much value
as the information is stored in the cache anyway.
The only additional info these reflections provided is whether a method
existed or not in the class, which we now also cache as plain array.

Furthermore the information whether cache relevant properties were
updated and the cache needs to be saved again is now reset once it was
written. This ensures that the cache is only written once even the
ReflectionService is used multiple times.

Resolves: #78915
Resolves: #78977
Releases: master, 7.6, 6.2
Change-Id: I1d00f596633adab886c0db0dfddd71358fb67303
Reviewed-on: https://review.typo3.org/50953
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Tymoteusz Motylewski <t.motylewski@gmail.com>
Tested-by: Tymoteusz Motylewski <t.motylewski@gmail.com>
Reviewed-by: Stefan Rotsch <stefan.rotsch@aoe.com>
Tested-by: Stefan Rotsch <stefan.rotsch@aoe.com>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
2 years ago[BUGFIX] Load page info when only getting external tables 55/50955/3
Johannes Schlier [Wed, 14 Dec 2016 12:46:45 +0000 (13:46 +0100)]
[BUGFIX] Load page info when only getting external tables

Previously the PageLayoutView would only set the page info when getting
the tt_content of the page in function ‘getTable_tt_content’.
If a user only has rights to view/edit records from other tables, but
not tt_content, the page info is null, as ‘getTable_tt_content’ is never
called.

Resolves: #78991
Releases: master, 7.6
Change-Id: Iabcad89a23b245e8c6a8e23fa420fcab5c1ea8e4
Reviewed-on: https://review.typo3.org/50955
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Daniel Gorges <daniel.gorges@b13.de>
Tested-by: Daniel Gorges <daniel.gorges@b13.de>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
2 years agoRevert "[BUGFIX] Reflection Cache does not save methodReflections" 63/50963/4
Helmut Hummel [Thu, 15 Dec 2016 00:20:22 +0000 (01:20 +0100)]
Revert "[BUGFIX] Reflection Cache does not save methodReflections"

Internal PHP classes including the \ReflectionMethod are not serlializable. Doing so results in a fatal error at some point.
The reported issue must be solved differently.

This reverts commit a5c1e222bf9858f5a44af1007cd9e1677b9a534d.

Reverts: #78915
Resolves: #78977
Releases: master, 7.6, 6.2
Change-Id: I1161b97eb04d0b1ba8ca59b3008d3f62c6ea1cf5
Reviewed-on: https://review.typo3.org/50963
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Nicole Cordes <typo3@cordes.co>
Tested-by: Nicole Cordes <typo3@cordes.co>
Tested-by: TYPO3com <no-reply@typo3.com>
2 years ago[TASK] Update seiyria-bootstrap-slider to 9.5.3 40/50940/3
Andreas Fernandez [Mon, 12 Dec 2016 18:21:38 +0000 (19:21 +0100)]
[TASK] Update seiyria-bootstrap-slider to 9.5.3

Resolves: #78967
Releases: master
Change-Id: Ic8801470c886701a43acedfeb74a8085352774db
Reviewed-on: https://review.typo3.org/50940
Tested-by: TYPO3com <no-reply@typo3.com>
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: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
2 years ago[BUGFIX] Fix RTE in Inline Elements 54/50954/6
Helmut Hummel [Tue, 13 Dec 2016 23:57:05 +0000 (00:57 +0100)]
[BUGFIX] Fix RTE in Inline Elements

HTMLArea needs a global JavaScript variable to work correctly.
Before the change done for #77600, this variable was always initialized
by including a JS file when any backend editing form was rendered.

In this mentioned cleanup, this initialization code was moved to
the place where the RTE is actually rendered. But because this can
also happen in an Ajax request, where the needed JS calls are returned
as array and evaluated line by line with `eval`, the code to initialize
the global variable did not work, because it was prefixed with `var`

This lead to a JS error at a later point where the global var is accessed.

To fix this, we force to set/ initialize the global variable, by prefixing
it with the "super global" object `window`. Now the global `HTMLArea` variable
is also initialized when this code is executed in a `eval` context,
when loading RTE fields of IRRE records via Ajax.

Resolves: #78972
Related: #77600
Releases: master
Change-Id: I7bf1386d3f6f395223bb82d870de0c7868733f67
Reviewed-on: https://review.typo3.org/50954
Reviewed-by: David Greiner <hallo@davidgreiner.de>
Tested-by: David Greiner <hallo@davidgreiner.de>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Nicole Cordes <typo3@cordes.co>
Tested-by: Nicole Cordes <typo3@cordes.co>
2 years ago[FOLLOWUP][TASK] TCA: Streamline field position in tabs 61/50961/3
Helmut Hummel [Wed, 14 Dec 2016 20:47:16 +0000 (21:47 +0100)]
[FOLLOWUP][TASK] TCA: Streamline field position in tabs

Fix some typos and a leading newline in a PHP file

Related: #78383
Releases: master
Change-Id: I483b09abf02a9f109350284e31cba15848004e5c
Reviewed-on: https://review.typo3.org/50961
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Nicole Cordes <typo3@cordes.co>
Tested-by: Nicole Cordes <typo3@cordes.co>
2 years ago[TASK] Deprecate Extbase's ArrayUtility 96/49696/20
Benni Mack [Wed, 31 Aug 2016 19:17:16 +0000 (21:17 +0200)]
[TASK] Deprecate Extbase's ArrayUtility

The copied methods from ArrayUtility can be removed
and be replaced with some PHP native functionality
or the equivalent functionality.

Resolves: #77732
Releases: master
Change-Id: I6c5f4bced77f40f5000d8fb0abb151d54fbc1c8e
Reviewed-on: https://review.typo3.org/49696
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: Susanne Moog <susanne.moog@typo3.org>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
2 years ago[FEATURE] Add Support for Doctrine DBAL Queries in Extbase 78/50178/8
Benni Mack [Mon, 10 Oct 2016 19:19:40 +0000 (21:19 +0200)]
[FEATURE] Add Support for Doctrine DBAL Queries in Extbase

To fully utilize Extbase in conjunction with Doctrine DBAL,
Extbases' StorageBackend now supports native QueryBuilder
and Doctrine DBAL Statements, which can be used for prepared
statements.

Resolves: #78116
Releases: master
Change-Id: I73b5a9588465e345029b7985327fb0cd0d7875df
Reviewed-on: https://review.typo3.org/50178
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
2 years ago[FOLLOWUP][TASK] Fluidification of ShortcutToolbarItem 43/50943/2
Susanne Moog [Mon, 12 Dec 2016 20:18:16 +0000 (21:18 +0100)]
[FOLLOWUP][TASK] Fluidification of ShortcutToolbarItem

No changes to parameters in language labels are allowed therefore
a new label for the bookmark description is needed.

Change-Id: Ib1997282905eb8321b903235581d9931449c85d8
Resolves: #77800
Releases: master
Reviewed-on: https://review.typo3.org/50943
Tested-by: TYPO3com <no-reply@typo3.com>
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>
2 years ago[BUGFIX] Fix exception in QuickEdit mode for empty pages 10/50910/3
Manuel Selbach [Thu, 8 Dec 2016 22:22:15 +0000 (23:22 +0100)]
[BUGFIX] Fix exception in QuickEdit mode for empty pages

If there is no content on a page the parameter $this->eRParts[1] will be
set to e.g. /new/1 which is not an integer value and it will throw an
exception here on certain dbms, thus let's check that before as there could
not be a history on a new record and skip the query to prevent the error.

Change-Id: If33c5de067b7685e5723ceb7aa1586640abc99d8
Resolves: #78925
Releases: master, 7.6, 6.2
Reviewed-on: https://review.typo3.org/50910
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Alexander Stehlik <alexander.stehlik@gmail.com>
Reviewed-by: Tymoteusz Motylewski <t.motylewski@gmail.com>
Tested-by: Tymoteusz Motylewski <t.motylewski@gmail.com>
Reviewed-by: Michael Oehlhof <typo3@oehlhof.de>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
2 years ago[BUGFIX] Order extensions by integerVersion instead of version 00/50900/5
Ulrik Høyer Kold [Thu, 8 Dec 2016 14:53:16 +0000 (15:53 +0100)]
[BUGFIX] Order extensions by integerVersion instead of version

Return the results of findByExtensionKeyOrderedByVersion ordered by
column 'integerVersion' instead of 'version' in order to avoid a wrong
order for version numbers like "1.9.2" and "1.10.0".

Releases: master, 7.6
Resolves: #78923
Change-Id: I89f5cf54aa4ee7ac87c5af2aa4ac3946d53f10d6
Reviewed-on: https://review.typo3.org/50900
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Tymoteusz Motylewski <t.motylewski@gmail.com>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
2 years ago[BUGFIX] Reflection Cache does not save methodReflections 13/50913/3
Valentin Despa [Fri, 9 Dec 2016 09:22:42 +0000 (10:22 +0100)]
[BUGFIX] Reflection Cache does not save methodReflections

The cached data of the Reflection Cache (table cf_extbase_reflection)
is not stored in its complete form.

The data/part of property 'methodReflections' is NOT stored in the cache.
As a result, the Reflection Cache will be deleted/inserted each time the
methodReflections property is needed

Resolves: #78915
Releases: master,7.6,6.2
Change-Id: I7b2f7477132abca2e10826bed6e78cb1195d2b26
Reviewed-on: https://review.typo3.org/50913
Tested-by: TYPO3com <no-reply@typo3.com>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Tymoteusz Motylewski <t.motylewski@gmail.com>
Tested-by: Tymoteusz Motylewski <t.motylewski@gmail.com>
2 years ago[TASK] Lowlevel: Clean up $versionSwapmode 04/50904/3
Benni Mack [Tue, 25 Oct 2016 05:44:41 +0000 (07:44 +0200)]
[TASK] Lowlevel: Clean up $versionSwapmode

The versionSwapmode is a orphan with a multivalue
which can easily be handled as boolean since TYPO3 6.0,
where workspaces mode "1" (page-based branching model)
was not supported anymore.

Resolves: #78947
Releases: master
Change-Id: I9b9887a306769f39f2e085110819cce4fe514067
Reviewed-on: https://review.typo3.org/50904
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: Susanne Moog <susanne.moog@typo3.org>
2 years ago[TASK] Raise Fluid engine dependency version to 2.1.0 27/50927/2
Claus Due [Sun, 11 Dec 2016 12:28:34 +0000 (13:28 +0100)]
[TASK] Raise Fluid engine dependency version to 2.1.0

Includes:

* {parsing off} modifier now possible in templates
* --bootstrap argument for Fluid CLI to enable cache warmup

Change-Id: I6a254a8f40e3144d4bc4dffe94482ebea715f145
Resolves: #78952
Releases: master
Reviewed-on: https://review.typo3.org/50927
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Reviewed-by: Nicole Cordes <typo3@cordes.co>
Tested-by: Nicole Cordes <typo3@cordes.co>
2 years ago[TASK] EXT:lowlevel - Remove performanceStatistics array 03/50903/3
Benni Mack [Tue, 25 Oct 2016 05:36:53 +0000 (07:36 +0200)]
[TASK] EXT:lowlevel - Remove performanceStatistics array

The lowlevel extension contains a performanceStatistics array
which is populated with microseconds, but never used, not even
in extensions in TER.

A RST file will follow when the whole class is removed.

Resolves: #78948
Releases: master
Change-Id: I5bd251c320adfb479e7e76144d69e135ee67d3c5
Reviewed-on: https://review.typo3.org/50903
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: Susanne Moog <susanne.moog@typo3.org>
2 years ago[BUGFIX] ResourceCompressor must deal with absolute web paths 21/50921/2
Markus Klein [Fri, 25 Nov 2016 19:29:55 +0000 (20:29 +0100)]
[BUGFIX] ResourceCompressor must deal with absolute web paths

ResourceCompressor lacks the ability to compress files which are
registered with their absolute web path (as FormEngine does).
Add a check to resolve such paths correctly.

Resolves: #78803
Releases: master
Change-Id: Ifdd60c65d9023aca57b5b20cf3f44e1bef121aa3
Reviewed-on: https://review.typo3.org/50921
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: Susanne Moog <susanne.moog@typo3.org>
2 years ago[BUGFIX] Use proper .xlf file ending for RTE demo config 20/50920/2
Benni Mack [Fri, 9 Dec 2016 15:05:27 +0000 (16:05 +0100)]
[BUGFIX] Use proper .xlf file ending for RTE demo config

Resolves: #78942
Releases: master, 7.6
Change-Id: If233cf0d4812f0d57deccbb6d4ebe33b2393b148
Reviewed-on: https://review.typo3.org/50920
Reviewed-by: Mathias Brodala <mbrodala@pagemachine.de>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Andreas Fernandez <typo3@scripting-base.de>
Reviewed-by: Oliver Klee <typo3-coding@oliverklee.de>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
2 years ago[FOLLOWUP][TASK] Add linkbrowser to CKEditor 12/50912/6
Frans Saris [Fri, 9 Dec 2016 08:25:56 +0000 (09:25 +0100)]
[FOLLOWUP][TASK] Add linkbrowser to CKEditor

Streamlined added config and moved plugin out of the Contrib folder
and pass correct contents language to editor.

Releases: master
Resolves: #78917
Change-Id: Iccd66e2bea7f1503e2c4b3ae68096da70e9822ef
Reviewed-on: https://review.typo3.org/50912
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Daniel Gorges <daniel.gorges@b13.de>
Tested-by: Daniel Gorges <daniel.gorges@b13.de>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
2 years ago[TASK] Remove unused RTE option "keepButtonGroupTogether" 11/50911/2
Benni Mack [Fri, 9 Dec 2016 05:56:27 +0000 (06:56 +0100)]
[TASK] Remove unused RTE option "keepButtonGroupTogether"

The option "keepButtonGroupTogether" was used before 6.0
and then removed without updating the documentation.

The patch removes the last reference within the PHP file
which is handed over to JS (but never used there) and removes
all documentation on that as well.

Resolves: #78927
Releases: master
Change-Id: Ic03dc10da06c7949d02ae04a566808ec17bdbab3
Reviewed-on: https://review.typo3.org/50911
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Frans Saris <franssaris@gmail.com>
Tested-by: Frans Saris <franssaris@gmail.com>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Andreas Fernandez <typo3@scripting-base.de>
2 years ago[TASK] Add linkbrowser to CKEditor 01/50901/4
Frans Saris [Thu, 8 Dec 2016 15:18:40 +0000 (16:18 +0100)]
[TASK] Add linkbrowser to CKEditor

Releases: master
Resolves: #78917
Change-Id: I864b7798cd30f0eec63f4ec719f8dfebca1086ba
Reviewed-on: https://review.typo3.org/50901
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Daniel Gorges <daniel.gorges@b13.de>
Tested-by: Daniel Gorges <daniel.gorges@b13.de>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
2 years ago[BUGFIX] SwapLocking should use proper file name 02/50902/2
Benni Mack [Thu, 8 Dec 2016 16:39:28 +0000 (17:39 +0100)]
[BUGFIX] SwapLocking should use proper file name

When using a swap locking file, the file name should not contain
a colon (":") in the filename.

Resolves: #78924
Releases: master, 7.6
Change-Id: I4012ad01e3ef948df691a6ebf1fab79cc1bebe87
Reviewed-on: https://review.typo3.org/50902
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Stephan Großberndt <stephan@grossberndt.de>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
2 years ago[TASK] Add available commands to console output 75/47475/7
Wouter Wolters [Thu, 31 Mar 2016 20:49:52 +0000 (22:49 +0200)]
[TASK] Add available commands to console output

When no valid key or command is given only the cliKeys are written
on the CLI. Now the available commands are written as well.

Resolves: #75350
Releases: master
Change-Id: If8102649128cf663831022b7b75a5150daa91649
Reviewed-on: https://review.typo3.org/47475
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
2 years ago[BUGFIX] Fix IP lock clause when [FE][lockIP] is 0 90/50890/7
Wouter Wolters [Wed, 7 Dec 2016 09:11:14 +0000 (10:11 +0100)]
[BUGFIX] Fix IP lock clause when [FE][lockIP] is 0

The method getNewSessionRecord doesn't check
lockIP setting 0 correctly. Add a check to disable checking
when lockIP is 0.

Resolves: #78902
Releases: master,7.6
Change-Id: Ic060ed36e9a7d4fdb1534ffe76ce8bbbee31038c
Reviewed-on: https://review.typo3.org/50890
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: Susanne Moog <susanne.moog@typo3.org>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
2 years ago[BUGFIX] Do not fetch pages with pid < 0 in prepareCacheFlush 94/50894/3
Steffen Göde [Thu, 11 Feb 2016 14:46:29 +0000 (15:46 +0100)]
[BUGFIX] Do not fetch pages with pid < 0 in prepareCacheFlush

Do not fetch workspace related pages to prepare cache flushing.

Resolves: #73241
Releases: master,7.6,6.2
Change-Id: I49bac4f1289848ded8c74b77fc4473ef925086dd
Reviewed-on: https://review.typo3.org/50894
Reviewed-by: Oliver Hader <oliver.hader@typo3.org>
Tested-by: Oliver Hader <oliver.hader@typo3.org>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: Susanne Moog <susanne.moog@typo3.org>
2 years ago[FOLLOWUP][TASK] Rename getReponseData to getResponseData 06/50906/2
Anja Leichsenring [Thu, 8 Dec 2016 18:44:22 +0000 (19:44 +0100)]
[FOLLOWUP][TASK] Rename getReponseData to getResponseData

One occurence has been missed.

Resolves: #78874
Releases: master
Change-Id: I14bcaa9fabda7139ee6ad1b8465f72c1ebb9bc2e
Reviewed-on: https://review.typo3.org/50906
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: Susanne Moog <susanne.moog@typo3.org>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
2 years ago[FOLLOWUP][TASK] Spread TypeConverter priorities 99/50899/2
Susanne Moog [Thu, 8 Dec 2016 09:56:50 +0000 (10:56 +0100)]
[FOLLOWUP][TASK] Spread TypeConverter priorities

Adjust priority of UploadedFileReferenceConverter

Change-Id: I701697f4d77f6fe042d3faa32a9815edd1c737f1
Resolves: #75232
Releases: master
Reviewed-on: https://review.typo3.org/50899
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Helmut Hummel <typo3@helhum.io>
Tested-by: Helmut Hummel <typo3@helhum.io>
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: Susanne Moog <susanne.moog@typo3.org>
2 years ago[!!!][TASK] Spread TypeConverter priorities 61/47361/4
Bernhard Kraft [Tue, 22 Mar 2016 19:11:02 +0000 (20:11 +0100)]
[!!!][TASK] Spread TypeConverter priorities

The priorities of the "TypeConverter" classes are densly packed.

With the current situation it is not possible to create a custom
TypeConverter with higher priority than "ObjectConverter" but
lower than "PersistentObjectConverter".

Resolves: #75232
Releases: master
Change-Id: If620f927fad54118c71347bc797b93397815a39f
Reviewed-on: https://review.typo3.org/47361
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: Susanne Moog <susanne.moog@typo3.org>
2 years ago[BUGFIX] rtehtmlarea: Fix isRequiredClass check 92/50692/3
Benjamin Franzke [Thu, 17 Nov 2016 08:34:14 +0000 (09:34 +0100)]
[BUGFIX] rtehtmlarea: Fix isRequiredClass check

Given the following TSConfig:

  RTE {
    classes {
      btn-default {
        name = Default Button
        requires = btn
      }
      btn-primary {
        name = Primary Button
        requires = btn
      }
      btn.selectable = 0
    }
  }

The incorrect isRequiredClass check causes the btn class to be
removed from the RTE toolbarbuttons. This happens because it's
considered unneeded in DOM.removeClass(), which is called by
DOM.addClass() (to remove incompatible classes).

The check in isRequiredClass was probably copied from
some classesRequired lookups, but forgot to inverse the
class that's searched for.

This also fixes the RTE to properly remove the selectable=0 class
(e.g. btn), when a class that requires the aformentioned is removed
(e.g. btn-primary).

Releases: master, 7.6
Resolves: #78730
Change-Id: I5a6ac42c7080835d5018608ab17a39b36a0fb70a
Reviewed-on: https://review.typo3.org/50692
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: Susanne Moog <susanne.moog@typo3.org>
2 years ago[FEATURE] Add a hook to DataHandler - localize - translateToMessage 20/40920/9
Goran Medakovic [Sat, 4 Jul 2015 08:58:07 +0000 (10:58 +0200)]
[FEATURE] Add a hook to DataHandler - localize - translateToMessage

Passing entire $langRec variable instead of just the title.

Resolves: #67909
Releases: master
Change-Id: Ibb7ccf91ea9ed2958043705b36238df44066e9cd
Reviewed-on: https://review.typo3.org/40920
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: Susanne Moog <susanne.moog@typo3.org>
2 years ago[TASK] Deprecate ContentObjectRenderer::fileResource 98/49498/17
Georg Ringer [Tue, 16 Aug 2016 15:40:10 +0000 (17:40 +0200)]
[TASK] Deprecate ContentObjectRenderer::fileResource

Deprecate the mentioned method which does too many
mixed things.

Resolves: #77524
Releases: master
Change-Id: Ib0e3d6d4a4441afb836292070b92f559cac96fb3
Reviewed-on: https://review.typo3.org/49498
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: Susanne Moog <susanne.moog@typo3.org>
2 years ago[TASK] Deprecate method LocalizationController::getRecordUidsToCopy 92/50892/2
Wouter Wolters [Wed, 7 Dec 2016 11:24:48 +0000 (12:24 +0100)]
[TASK] Deprecate method LocalizationController::getRecordUidsToCopy

The method "getRecordUidsToCopy" is not used at any place
in the TYPO3 core.

Resolves: #78872
Releases: master
Change-Id: Ib08234f4f1233a200b14d5aa2effbf82afcf87cc
Reviewed-on: https://review.typo3.org/50892
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Gleb Levitin <gleb.levitin@dkd.de>
Tested-by: Gleb Levitin <gleb.levitin@dkd.de>
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: Susanne Moog <susanne.moog@typo3.org>
2 years ago[TASK] Fluidification of ShortcutToolbarItem 10/49810/7
Patrick Kollodzik [Fri, 2 Sep 2016 22:27:43 +0000 (00:27 +0200)]
[TASK] Fluidification of ShortcutToolbarItem

Because the ShortcutToolbarItem has three different actions,
every action has its own template.

Resolves: #77800
Releases: master
Change-Id: I054569376a0a5f62bb40e7d77f675350e0464423
Reviewed-on: https://review.typo3.org/49810
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Alexander Stehlik <alexander.stehlik@gmail.com>
Tested-by: Alexander Stehlik <alexander.stehlik@gmail.com>
Reviewed-by: Joerg Boesche <typo3@joergboesche.de>
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: Susanne Moog <susanne.moog@typo3.org>
2 years ago[TASK] TCA tree: Simplify json result 13/50813/10
Christian Kuhn [Wed, 30 Nov 2016 16:29:10 +0000 (17:29 +0100)]
[TASK] TCA tree: Simplify json result

The patch changes the ajax result that delivers TCA tree
items to the SVG tree from a nested list of items to a
sorted flat list having a 'depth' argument to indicate the
nesting level.
This "flat" list is the native mode of the d3 tree, with
this change the JS side can be streamlined quite a bit.
Along the way, the item providing on PHP side is streamlined,
documented much better and easier to understand now within the
data provider of FormEngine.
The main tree data backend is still a huge, convoluted, slow
and insane mess that will eventually fully substituted with a
much straighter and quicker approach later. Changes in this area
are kept to a minimum for now.

Change-Id: Ib64b7277f671b632be3977218e5465b534618d63
Resolves: #78905
Related: #76108
Releases: master
Reviewed-on: https://review.typo3.org/50813
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Thomas Maroschik <tmaroschik@dfau.de>
Tested-by: Thomas Maroschik <tmaroschik@dfau.de>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
2 years ago[!!!][TASK] Migrate lowlevel rte_images command to Symfony Console 50/50650/6
Benni Mack [Mon, 14 Nov 2016 07:21:25 +0000 (08:21 +0100)]
[!!!][TASK] Migrate lowlevel rte_images command to Symfony Console

The CLI command to detect RTE magic images (images added
within the RTE) and (as far as possible) remove unused RTE images
as well as copy RTE images referenced multiple times, 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 rte_images

and is now called via
./typo3/sysext/core/bin/typo3 cleanup:rteimages

and allows the following options:
--dry-run - do not update the images and delete lost files, but just print them
--update-refindex - update the reference index, do not ask the user

Resolves: #78895
Releases: master
Change-Id: I6e30b7a32fe14bc89cebd85167961ec8fa996c54
Reviewed-on: https://review.typo3.org/50650
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: Susanne Moog <susanne.moog@typo3.org>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
2 years ago[FOLLOWUP][TASK] Use ISO-8601 dates for display and processing 80/50880/8
Oliver Hader [Sun, 4 Dec 2016 11:51:20 +0000 (12:51 +0100)]
[FOLLOWUP][TASK] Use ISO-8601 dates for display and processing

Adjust acceptance tests for ISO-8601 date/-time values.

Change-Id: Ib35af231f8f8b8150eb0e132543be060eeff67da
Resolves: #77702
Releases: master
Reviewed-on: https://review.typo3.org/50880
Tested-by: TYPO3com <no-reply@typo3.com>
Tested-by: Susanne Moog <susanne.moog@typo3.org>
Reviewed-by: Oliver Hader <oliver.hader@typo3.org>
Tested-by: Oliver Hader <oliver.hader@typo3.org>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
2 years ago[BUGFIX] Flush all cache tags in bulk when deleting records 37/50537/24
Claus Due [Mon, 7 Nov 2016 16:09:35 +0000 (17:09 +0100)]
[BUGFIX] Flush all cache tags in bulk when deleting records

This patch makes the TYPO3 DB cache backend capable of
flushing tags using an array of tag names that is then turned
into a CSV list and used in an `IN` query. It limits the number
of cache flushes from $numRecords to ceil($numRecords/100).

NB: the desired behavior is introduced by reviving the
flushByTags() method as API for cache frontends and backends.
The method was previously present but was dropped in order to
keep sync with Flow - which then later added the method exactly
because of performance concerns.

TYPO3 however did not revive this method and obviously the sync
with Flow is no longer a concern. So this patch restores the
full API required to flush tags in bulk, adds an implementation
for the TYPO3 DB cache backend and adds delegation to the old
flushByTag method for backends not covered by this patch.

It will be possible to improve other backends as well but this
patch focuses exclusively on the DB cache backend for *NIX,
which is where the bad performance was observed in the wild.

Change-Id: I99d4dd8d0881c3bf9f6240e84b083b72b1831779
Resolves: #78596
Releases: master
Reviewed-on: https://review.typo3.org/50537
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Joerg Boesche <typo3@joergboesche.de>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
2 years ago[CLEANUP] Simplify registerPageTSConfigFile description 84/50884/2
Mathias Brodala [Mon, 5 Dec 2016 10:09:43 +0000 (11:09 +0100)]
[CLEANUP] Simplify registerPageTSConfigFile description

Resolves: #78886
Releases: master, 7.6
Change-Id: I1e5922ed510f0f43a6374f9dde061356290b72d9
Reviewed-on: https://review.typo3.org/50884
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Robert van Kammen <rvkammen@hotmail.com>
Tested-by: Robert van Kammen <rvkammen@hotmail.com>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
2 years ago[CLEANUP] Simplify addStaticFile description 85/50885/2
Mathias Brodala [Mon, 5 Dec 2016 10:25:02 +0000 (11:25 +0100)]
[CLEANUP] Simplify addStaticFile description

Resolves: #78887
Releases: master, 7.6
Change-Id: Ifb6de078816e851915372d1c5611552fd58f2cce
Reviewed-on: https://review.typo3.org/50885
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Robert van Kammen <rvkammen@hotmail.com>
Tested-by: Robert van Kammen <rvkammen@hotmail.com>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
2 years ago[TASK] Remove iconv support and use mbstring-polyfill 07/50607/10
Markus Klein [Sat, 12 Nov 2016 15:21:22 +0000 (16:21 +0100)]
[TASK] Remove iconv support and use mbstring-polyfill

We remove the iconv support in the CharsetConverter as the PHP internal
implementation of some functions differ from their mb_string pendants.
This gives us a real headache in edge-cases.

We already have the mbstring-polyfill as an indirect dependency in
the Core (by symfony), so we simply require it our own now and use
mb_string all over the places.

This renders quite a few methods of CharsetConverter useless
and we therefore deprecate those.

Resolves: #78670
Releases: master
Change-Id: I88479f7939e0afb46f704d4e8f347abd2e10fc6d
Reviewed-on: https://review.typo3.org/50607
Tested-by: TYPO3com <no-reply@typo3.com>
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>
2 years ago[FEATURE] Add option to "system status updates" report to send all tests 11/50811/7
Michael Oehlhof [Wed, 30 Nov 2016 13:59:57 +0000 (14:59 +0100)]
[FEATURE] Add option to "system status updates" report to send all tests

Resolves: #52286
Releases: master
Change-Id: I875a6e7f70007c125408ec6c62c4fba1b2616233
Reviewed-on: https://review.typo3.org/50811
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Alexander Stehlik <alexander.stehlik@gmail.com>
Tested-by: Alexander Stehlik <alexander.stehlik@gmail.com>
Reviewed-by: Manuel Glauser <mail@manuelglauser.ch>
Tested-by: Manuel Glauser <mail@manuelglauser.ch>
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>
2 years ago[FEATURE] Integrate ckeditor in the TYPO3 Backend 88/50788/7
Benni Mack [Mon, 28 Nov 2016 08:04:03 +0000 (09:04 +0100)]
[FEATURE] Integrate ckeditor in the TYPO3 Backend

Integrates CKeditor via bower/grunt as system extension "ckeditor".

Loads and saves data for backend forms currently.

Next steps:
* contents.css
* defaultconfig.js taken care of optimally
* styleSets functionality
* language / labels

What needs to be done
* Make editor configurable (height/width, etc)
* Evaluate UI information based on TSconfig
* Integrate plugin configuration in JS
* Check out best defaults for new editor
* Add labels by TYPO3, not from CKeditor
* Integrate "Toolbar Configuration" into "Tools=>Configuration"

Change-Id: Ia751436bf0ea4479d80e7bc0c7e97c229442dcf2
Reviewed-on: https://review.typo3.org/50788
Reviewed-by: Johannes Goslar <jogo@kronberger-spiele.de>
Tested-by: Johannes Goslar <jogo@kronberger-spiele.de>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: Susanne Moog <susanne.moog@typo3.org>
Reviewed-by: Markus Sommer <markussom@posteo.de>
Tested-by: Markus Sommer <markussom@posteo.de>
Reviewed-by: Oliver Hader <oliver.hader@typo3.org>
Tested-by: Oliver Hader <oliver.hader@typo3.org>
2 years ago[BUGFIX] Prevent duplicate empty lines for encapsLines 11/45111/5
Alexander Stehlik [Sat, 3 Dec 2016 16:44:13 +0000 (17:44 +0100)]
[BUGFIX] Prevent duplicate empty lines for encapsLines

When the last line in the text passed to encaps_lineSplit()
is empty a possible duplicate empty line is removed.

Releases: master
Resolves: #72050
Change-Id: I358ae5e1e6e9731d4649add7951c93e62df3bb7e
Reviewed-on: https://review.typo3.org/45111
Reviewed-by: Manuel Glauser <mail@manuelglauser.ch>
Tested-by: Manuel Glauser <mail@manuelglauser.ch>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Jan Helke <typo3@helke.de>
Tested-by: Jan Helke <typo3@helke.de>
2 years ago[!!!][BUGFIX] Allow override of page creation wizard again 60/50560/7
Benni Mack [Wed, 9 Nov 2016 21:51:18 +0000 (22:51 +0100)]
[!!!][BUGFIX] Allow override of page creation wizard again

The TSconfig option to override the page creation wizard via
`mod.web_list.newPageWiz.overrideWithExtension` is impossible
to use anymore since TYPO3 v8, due to the limitation
of the entry-scripts which are only allowed to be in typo3/index.php
for the backend. The option `mod.newPageWizard.override` was
introduced to register a custom route / module to be linked when
adding new pages.

Resolves: #78549
Releases: master
Change-Id: I1c46c8facad83c0d561028e544f18238abe89f91
Reviewed-on: https://review.typo3.org/50560
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: Susanne Moog <susanne.moog@typo3.org>
2 years agoRevert "[FEATURE] Reusable getCategoryFieldsForTable itemsProcFunc" 78/50878/2
Christian Kuhn [Sat, 3 Dec 2016 18:37:18 +0000 (19:37 +0100)]
Revert "[FEATURE] Reusable getCategoryFieldsForTable itemsProcFunc"

This reverts commit 36f8765aa340c364f7b7edfa967549c683f21747.

Change-Id: Ida5c32b33e8953bd1172c9886af16b66e3c2297f
Reviewed-on: https://review.typo3.org/50878
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
2 years ago[FEATURE] Reusable getCategoryFieldsForTable itemsProcFunc 68/24968/19
Alexander Stehlik [Sat, 3 Dec 2016 17:52:06 +0000 (18:52 +0100)]
[FEATURE] Reusable getCategoryFieldsForTable itemsProcFunc

Allow the the method CategoryRegistry->getCategoryFieldItems()
to be used as itemsProcFunc for select fields in the TCA in
arbitary contexts by introducing a new categoryFieldsTable
configuration in the config section of a column in the TCA.

This configuration key can either consist of a single string containing
the name of the table or a configuration array to define additional
conditions that need to be true so that a configured table is used.

The condition matching is based on the displayCond functionality of
the TCA. To make use of the existing functionality the code for
matching the display conditions is extracted from the form data
provider to a new DisplayConditionEvaluator utility class.

Resolves: #53045
Releases: master
Change-Id: I128cbeb6747a8f83e68cdaaaafbc3ab5901353d4
Reviewed-on: https://review.typo3.org/24968
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Reviewed-by: Jan Helke <typo3@helke.de>
Tested-by: Jan Helke <typo3@helke.de>
2 years ago[BUGFIX] Skip swapping/publishing of deleted records 29/23129/14
Oliver Hader [Sat, 3 Dec 2016 15:44:24 +0000 (16:44 +0100)]
[BUGFIX] Skip swapping/publishing of deleted records

In case a content element and the accordant page have been deleted in
separate actions and get published together, the workspace process will
trigger an error message since the removed content element cannot be
published anymore (since it has been processed already with the page).

To avoid this behavior deleted records are collected and checked in the
workspace swapping/publishing process.

Change-Id: If04a198abf81efdc88e75da79da0c01cfaa361ff
Resolves: #47384
Releases: master, 7.6
Reviewed-on: https://review.typo3.org/23129
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: Susanne Moog <susanne.moog@typo3.org>
2 years ago[TASK] Remove compatiblity code added for security release 63/50863/3
Helmut Hummel [Fri, 2 Dec 2016 22:52:07 +0000 (23:52 +0100)]
[TASK] Remove compatiblity code added for security release

Remove the overhead that was added to avoid BC breaks for
extension code that subclassed the form view helper.

Enough time is now passed so that extensions can adopt their
subclasses so that the security related hidden field is added as well.

These adaptions will then be compatible with all TYPO3 versions
so that this change here is not marked as breaking.

Resolves: #78869
Releases: master
Change-Id: I910bc26cd57b7629e57332fdab3d57032f0c2478
Reviewed-on: https://review.typo3.org/50863
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: Susanne Moog <susanne.moog@typo3.org>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
2 years ago[TASK] Rename getReponseData to getResponseData 69/50869/2
Markus Sommer [Sat, 3 Dec 2016 11:31:00 +0000 (12:31 +0100)]
[TASK] Rename getReponseData to getResponseData

Resolves: #78874
Releases: master
Change-Id: I56567b000ef156731f38e9945c234822fd926ced
Reviewed-on: https://review.typo3.org/50869
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Reviewed-by: Johannes Goslar <jogo@kronberger-spiele.de>
Reviewed-by: Alexander Rothmund <arothmund@snowflake.ch>
Reviewed-by: Jan Helke <typo3@helke.de>
Tested-by: Jan Helke <typo3@helke.de>
2 years ago[FEATURE] Enable rechecking whether an UpdateWizard should run 24/49724/7
Tobias Adolph [Thu, 1 Sep 2016 15:25:18 +0000 (17:25 +0200)]
[FEATURE] Enable rechecking whether an UpdateWizard should run

UpdateWizards can now be chosen to be rechecked. If the
criteria is met they are displayed as executable - otherwise not.

Resolves: #77757
Releases: master
Change-Id: I2dcd0e62c9ab3e5fe9d498a02dd9773090b110b9
Reviewed-on: https://review.typo3.org/49724
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Reviewed-by: Jan Helke <typo3@helke.de>
Tested-by: Jan Helke <typo3@helke.de>
2 years ago[BUGFIX] Documentation of sessionTimeout 97/50797/3
Alexander Opitz [Mon, 28 Nov 2016 18:14:13 +0000 (19:14 +0100)]
[BUGFIX] Documentation of sessionTimeout

A sessionTimeout <= 0 means that a user will be logged out
instantly after login.

Resolves: #78822
Releases: master, 7.6, 6.2
Change-Id: I1890cb7d52347faab1fa3884eb46c9294d997da3
Reviewed-on: https://review.typo3.org/50797
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
2 years ago[!!!][TASK] TCA: Streamline field position in tabs 64/50364/23
Daniel Windloff [Sat, 3 Dec 2016 14:23:05 +0000 (15:23 +0100)]
[!!!][TASK] TCA: Streamline field position in tabs

In TYPO3 there are some recurring field definitions shared
by a lot of records. These fields are mostly defined in
the control section of TCA.

These fields are used as well by the core as by records
introduced by third party extensions.

In order to get a consistent behaviour
(from editor's perspective)
these fields should have a generic position in tabs.

Resolves: #78383
Releases: master
Change-Id: I365538b8e6013e3911710b113bec7d41314809b2
Reviewed-on: https://review.typo3.org/50364
Reviewed-by: Alexander Stehlik <alexander.stehlik@gmail.com>
Tested-by: Alexander Stehlik <alexander.stehlik@gmail.com>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Reviewed-by: Jan Helke <typo3@helke.de>
Tested-by: Jan Helke <typo3@helke.de>
2 years ago[BUGFIX] Respect availableLanguages from AdditionalConfiguration.php 71/50871/5
Florian Mast [Sat, 3 Dec 2016 12:45:23 +0000 (13:45 +0100)]
[BUGFIX] Respect availableLanguages from AdditionalConfiguration.php

if $GLOBALS['TYPO3_CONF_VARS']['EXTCONF']['lang']['availableLanguages']
is set in AdditionalConfiguration.php, the language module will now
respect that setting.

Resolves: #78876
Releases: master
Change-Id: Iec6a0f05db835ae949fbacdc2cdf500a0950d57c
Reviewed-on: https://review.typo3.org/50871
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: Susanne Moog <susanne.moog@typo3.org>
2 years ago[TASK] Show correct message in workspace module 75/50875/2
Markus Sommer [Sat, 3 Dec 2016 15:24:58 +0000 (16:24 +0100)]
[TASK] Show correct message in workspace module

Remove HTML in language file

Releases: master
Resolves: #78877
Change-Id: I154b14530b54791c1b44d6862a4c3c6bb472129d
Reviewed-on: https://review.typo3.org/50875
Reviewed-by: Johannes Goslar <jogo@kronberger-spiele.de>
Tested-by: Johannes Goslar <jogo@kronberger-spiele.de>
Reviewed-by: Jan Helke <typo3@helke.de>
Tested-by: Jan Helke <typo3@helke.de>
2 years ago[BUGFIX] Install extension when dependency check is skipped 30/50630/3
Nicole Cordes [Sun, 13 Nov 2016 12:21:16 +0000 (13:21 +0100)]
[BUGFIX] Install extension when dependency check is skipped

You can skip the dependency check if you want to enforce the installation
of an extension. This patch ensures that dependencies which are available
in the system but marked as not compatible are installed anyways.

Resolves: #78677
Releases: master, 7.6
Change-Id: I552ff949524e884bfbb5b8e270cfee0dc4691187
Reviewed-on: https://review.typo3.org/50630
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: TYPO3com <no-reply@typo3.com>
Reviewed-by: Markus Sommer <markussom@posteo.de>
Tested-by: Markus Sommer <markussom@posteo.de>
Reviewed-by: Johannes Goslar <jogo@kronberger-spiele.de>
Tested-by: Johannes Goslar <jogo@kronberger-spiele.de>
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: Susanne Moog <susanne.moog@typo3.org>
2 years ago[TASK] Refactor BackendUtility::getRecordsByField() to prepared statements 64/50664/6
Manuel Selbach [Tue, 15 Nov 2016 14:10:29 +0000 (15:10 +0100)]
[TASK] Refactor BackendUtility::getRecordsByField() to prepared statements

According to task #78437 queries should follow the prepared statement
principle.
Thus method BackendUtiltiy::getRecordsByField() has to be refactored
to retrieve a queryBuilder which holds the parameter assigned to e.g.
additional where clause, etc. otherwise the parameter and the dynamically
generated placeholders in the queryQuilder will be reset.

Change-Id: Id66d7b2fcfc5bcdca4d920b645c2285ded0c160a
Resolves: #78704
Releases: master
Reviewed-on: https://review.typo3.org/50664
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: Susanne Moog <susanne.moog@typo3.org>
2 years ago[FEATURE] Purge language packs in language module 60/50860/4
Michael Oehlhof [Fri, 2 Dec 2016 19:23:30 +0000 (20:23 +0100)]
[FEATURE] Purge language packs in language module

A "remove" button is added to the list of the installed languages.

Resolves: #58637
Releases: master
Change-Id: Ibd8911d2c58e449a026f52f4bd8b0b2d04bc2bbf
Reviewed-on: https://review.typo3.org/50860
Reviewed-by: Markus Sommer <markussom@posteo.de>
Tested-by: Markus Sommer <markussom@posteo.de>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: Susanne Moog <susanne.moog@typo3.org>
2 years ago[BUGFIX] Re-enable environment status report in Backend 25/50825/3
Alexander Stehlik [Fri, 2 Dec 2016 14:32:10 +0000 (15:32 +0100)]
[BUGFIX] Re-enable environment status report in Backend

The condition that checks for CLI mode in the ext_tables.php of
the install tool is fixed.

The EnvironmentStatusReport is now active in the Backend and
not on CLI.

The additional checks added in #78786 are not required any more
and are reverted.

Change-Id: I4d364fc08c68c7c25d9210c08b813247bee7d47f
Resolves: #78854
Relates: #78786
Releases: master, 7.6
Reviewed-on: https://review.typo3.org/50825
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: Susanne Moog <susanne.moog@typo3.org>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
2 years ago[BUGFIX] Fix bug in linkhandler when file is selected 67/50767/2
Sascha Egerer [Thu, 24 Nov 2016 14:18:51 +0000 (15:18 +0100)]
[BUGFIX] Fix bug in linkhandler when file is selected

Selecting a file an the RTE linkhandler works but reopening
the linkhandler ends with a fatal error as a file object
is passed instead of string.

Change-Id: I56972ef8d25e9add0c51e1b72fd7034c1140bc01
Resolves: #78782
Releases: master
Reviewed-on: https://review.typo3.org/50767
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Tymoteusz Motylewski <t.motylewski@gmail.com>
Tested-by: Tymoteusz Motylewski <t.motylewski@gmail.com>
Reviewed-by: Markus Sommer <markussom@posteo.de>
Tested-by: Markus Sommer <markussom@posteo.de>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Jan Helke <typo3@helke.de>
Tested-by: Jan Helke <typo3@helke.de>
2 years ago[BUGFIX] Unify line endings in ext-lang-am.js 67/50867/4
Alexander Stehlik [Sat, 3 Dec 2016 10:21:54 +0000 (11:21 +0100)]
[BUGFIX] Unify line endings in ext-lang-am.js

The ext-lang-am.js now has unique linux linebreaks.
Before it had a mixture of windows and linux linebreaks.

Change-Id: I4f74677c66611e2ca2143cf249934cb69b5e5ea5
Resolves: #78871
Relates: #77808
Releases: master, 7.6
Reviewed-on: https://review.typo3.org/50867
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: Susanne Moog <susanne.moog@typo3.org>
Reviewed-by: Oliver Hader <oliver.hader@typo3.org>
Tested-by: Oliver Hader <oliver.hader@typo3.org>
2 years ago[BUGFIX] Fix install tool image test path 64/49864/7
Johannes Goslar [Mon, 5 Sep 2016 11:54:25 +0000 (13:54 +0200)]
[BUGFIX] Fix install tool image test path

The temppath for gdlib and combineImages is different
then all other image tests
The assets/images path is set in imageMagickConvert
for other image tests

Releases: master
Resolves: #77851
Change-Id: I4fec0422595045cfb5ca67a48f8e6dae79fc08d7
Reviewed-on: https://review.typo3.org/49864
Reviewed-by: Gleb Levitin <gleb.levitin@dkd.de>
Reviewed-by: Manuel Glauser <mail@manuelglauser.ch>
Tested-by: Manuel Glauser <mail@manuelglauser.ch>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
2 years ago[!!!][TASK] Fluidification of Clipboard 67/46367/24
Jan Helke [Fri, 29 Jan 2016 17:32:38 +0000 (18:32 +0100)]
[!!!][TASK] Fluidification of Clipboard

Moves most of the HTML from the PHP to an own Template.
In fact in line 408 and in the function linkItemText are some remaining
html tags, but removing those would require some major refactoring.

Resolves: #73016
Releases: master
Change-Id: Ie9859758885be0516ff8aeb58c473c5634848c81
Reviewed-on: https://review.typo3.org/46367
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Florian Mast <flo.mast@web.de>
Reviewed-by: Markus Sommer <markussom@posteo.de>
Tested-by: Markus Sommer <markussom@posteo.de>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
2 years ago[BUGFIX] Avoid duplicate calls to update reference index 24/50724/3
Claus Due [Mon, 21 Nov 2016 15:52:03 +0000 (16:52 +0100)]
[BUGFIX] Avoid duplicate calls to update reference index

This patch adds a check when processing the update
reference index stack, in order to avoid calling the
update function more than once with the same input
arguments (which are $table and $uid).

Saves around 100 SQL queries when doing a standard
copy of 25 (nested) pages with 25 content elements.

Change-Id: If162218d4eabb749a92270e18c7c39108fd15f34
Resolves: #78761
Releases: master, 7.6
Reviewed-on: https://review.typo3.org/50724
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: Susanne Moog <susanne.moog@typo3.org>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
2 years ago[FOLLOWUP][BUGFIX] Respect SQL table names max length in install tool 65/50865/2
Anja Leichsenring [Sat, 3 Dec 2016 08:27:03 +0000 (09:27 +0100)]
[FOLLOWUP][BUGFIX] Respect SQL table names max length in install tool

Respect the max length of 64 chars in sql if removing (renaming)
an SQL table. Add unit test.

Resolves: #78636
Releases: master
Change-Id: I94f38c40f6539d5e53e2ac9bad5ed1c059836333
Reviewed-on: https://review.typo3.org/50865
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Manuel Glauser <mail@manuelglauser.ch>
Tested-by: Manuel Glauser <mail@manuelglauser.ch>
Reviewed-by: Jan Helke <typo3@helke.de>
Tested-by: Jan Helke <typo3@helke.de>
2 years ago[TASK] Use ISO-8601 dates for display and processing 24/47124/16
Andreas Wolf [Sat, 5 Mar 2016 22:51:56 +0000 (23:51 +0100)]
[TASK] Use ISO-8601 dates for display and processing

This changes the internal date processing for FormEngine and DataHandler
to use ISO-8601 dates including a proper timezone. Dates are converted
to ISO-8601 as early as possible and converted back to UNIX timestamps
as late as possible.

As before, the database values are always values in the server’s
timezone, interpreted as UTC. Also, the client side inputs are
interpreted as UTC.

The main advantage is that once and for all we get rid of the timezone
issues that may potentially arise if the server and client use different
timezones. Additionally, the values are human readable (which is of
course not so much of an issue for hidden fields) and we can directly
use Moment.js for all heavy lifting.

Another big advantage is that we can make the date formats configurable
and decouple displayed dates and internally stored dates (by putting the
Moment objects to the input fields).

Change-Id: I3461915c2beaa96cd29c52c60e65a2e5005065b7
Resolves: #77702
Releases: master
Reviewed-on: https://review.typo3.org/47124
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Oliver Hader <oliver.hader@typo3.org>
Tested-by: Oliver Hader <oliver.hader@typo3.org>
2 years ago[BUGFIX] Make showAccessRestrictedPages work 96/50696/8
Claus Due [Thu, 17 Nov 2016 12:14:24 +0000 (13:14 +0100)]
[BUGFIX] Make showAccessRestrictedPages work

This patch fixes a problem where setting showAccessRestrictedPages
setting in HMENU etc. did not result in the correct behavior according
to documentation at https://docs.typo3.org/typo3cms/TyposcriptReference/MenuObjects/CommonProperties/Index.html.

The patch changes the behavior caused by using `NONE` as
keyword in the setting.

Before: using the option renders `<a>Page title</a>` when page is inaccessible.

After: using the option renders `<a href="index.php?id=123">Page title</a>`
when page is not accessible.

Change-Id: I4364f7ec9f44e1e0b70654830efc236f20e6bf30
Resolves: #17904
Releases: master, 7.6
Reviewed-on: https://review.typo3.org/50696
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Jan Helke <typo3@helke.de>
Tested-by: Jan Helke <typo3@helke.de>
2 years ago[BUGFIX] Flaky child references handling for pages_language_overlay 95/48495/5
Andreas Allacher [Wed, 8 Jun 2016 05:47:17 +0000 (07:47 +0200)]
[BUGFIX] Flaky child references handling for pages_language_overlay

IRRE records have to point to their specific localized parent record.
This works in general for tables containing records of the default
language as well as localizations. For the "pages" table this look-up
has to be extended to query from "pages_language_overlay".

Change-Id: I84b6ee9304f85222430a5bdf62bd2271aae68fd5
Resolves: #76174
Releases: master, 7.6
Reviewed-on: https://review.typo3.org/48495
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: Susanne Moog <susanne.moog@typo3.org>
2 years ago[TASK] Streamline ElementInformation template 51/50851/2
Daniel Windloff [Fri, 2 Dec 2016 12:15:30 +0000 (13:15 +0100)]
[TASK] Streamline ElementInformation template

- Change "Show" button to btn-default
- Use infobox for missing files
- Streamline button order and groups
    - preview / edit
    - history
    - opens list module

Resolves: #78864
Releases: master
Change-Id: I1e74b5e5d75a0678ab385b40f53802a30028980a
Reviewed-on: https://review.typo3.org/50851
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: Susanne Moog <susanne.moog@typo3.org>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
2 years ago[BUGFIX] Respect SQL table names max length in install tool 83/50583/5
Markus Hoelzle [Thu, 10 Nov 2016 19:47:14 +0000 (20:47 +0100)]
[BUGFIX] Respect SQL table names max length in install tool

Respect the max length of 64 chars in sql if removing (renaming)
an SQL table. Add unit test.

Resolves: #78636
Releases: master
Change-Id: I5f5d4b239f467a205edfdceeac97da5625631a83
Reviewed-on: https://review.typo3.org/50583
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Alexander Rothmund <alexander@rothmund.ch>
Tested-by: Alexander Rothmund <alexander@rothmund.ch>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
2 years ago[BUGFIX] Reduce expensive calls in AbstractMenuContentObject 52/50652/4
Claus Due [Mon, 14 Nov 2016 14:21:43 +0000 (15:21 +0100)]
[BUGFIX] Reduce expensive calls in AbstractMenuContentObject

This patch reduces the number of SQL queries and PHP calls
which get performed when rendering menus. The patch has
two parts:

* Runtime cache is used to remember a generated link and
  is given a cache identifier which includes a hash of all
  parameters which may affect the link.
* Runtime cache is used to remember the decision if a page
  is a submenu.

The item rendering function is called every time the same page
is rendered in any menu in the same request, and the decision
function to check if page is a sub-menu is called at least three
times with the same UID when generating a menu. Both of these
implements together reduce the necessary re-calling of methods
which generate the same output given the same arguments.

Change-Id: Idd6225081e8fb3f8160270af3d865b48208b756e
Releases: master, 7.6
Resolves: #78693
Reviewed-on: https://review.typo3.org/50652
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Reviewed-by: Daniel Alder <dalder@snowflake.ch>
Tested-by: Daniel Alder <dalder@snowflake.ch>
Reviewed-by: Joerg Boesche <typo3@joergboesche.de>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
2 years ago[BUGFIX] Reset hidden field information in FormViewhelper 86/50486/2
Nicole Cordes [Wed, 2 Nov 2016 23:26:04 +0000 (00:26 +0100)]
[BUGFIX] Reset hidden field information in FormViewhelper

If a checkbox or a multiple selectbox is rendered, a hidden field is
automatically inserted before to send an empty default value. Field
information are stored within an own viewHelperVariableContainer
variable. After a form is rendered, this information has to be reset to
be able to render another form with same field names. This patch adds
the missing reset of the hidden field information.

Resolves: #78551
Releases: master, 7.6, 6.2
Change-Id: I3cde5618d07cae39b7841875504471cb2341dc17
Reviewed-on: https://review.typo3.org/50486
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Michael Oehlhof <typo3@oehlhof.de>
Tested-by: Michael Oehlhof <typo3@oehlhof.de>
Reviewed-by: Jan Helke <typo3@helke.de>
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Tested-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>