Packages/TYPO3.CMS.git
2 years ago[FOLLOWUP][TASK] Remove migrated TYPO3_CONF_VARS/SYS/setDBinit 55/54255/4
Benni Mack [Mon, 2 Oct 2017 08:00:16 +0000 (10:00 +0200)]
[FOLLOWUP][TASK] Remove migrated TYPO3_CONF_VARS/SYS/setDBinit

This patch removes the obsolete "setDBinit" option which was migrated
in TYPO3 v8.

Resolves: #82421
Releases: master
Change-Id: I96477e4dd7e151bf74caeafa7bdc38a1cde2f484
Reviewed-on: https://review.typo3.org/54255
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
2 years ago[!!!][TASK] Cleanup global debug functions 56/54256/4
Benni Mack [Mon, 2 Oct 2017 08:10:28 +0000 (10:10 +0200)]
[!!!][TASK] Cleanup global debug functions

The global "xdebug()" function is removed, as it is a stub for "debug" nowadays.
This function exists since SVN days,  and no comment or change why this is
available.

Additionally, the global "debug()" function had 6 parameters - the last three
were not evaluated and thus, were removed.

Resolves: #82640
Releases: master
Change-Id: I3e583c06282842e3dcb14f4b68589fd706be2519
Reviewed-on: https://review.typo3.org/54256
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: TYPO3com <no-reply@typo3.com>
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] Stabilize flaky TimeTracker unit test 60/54260/2
Christian Kuhn [Mon, 2 Oct 2017 12:50:28 +0000 (14:50 +0200)]
[BUGFIX] Stabilize flaky TimeTracker unit test

The time tracker sometimes switches between two milliseconds
in its operation. Test a bit weaker to cope with that.

Change-Id: I20d58296749bf33b12ab2da72691ecf4853842eb
Resolves: #82635
Releases: master, 8.7
Reviewed-on: https://review.typo3.org/54260
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
2 years ago[!!!][TASK] Remove tce_db options "prErr" and "uPT" 44/54244/4
Benni Mack [Sat, 30 Sep 2017 20:15:52 +0000 (22:15 +0200)]
[!!!][TASK] Remove tce_db options "prErr" and "uPT"

The two GET/POST parameters "prErr" (Print Errors) and "uPT" (Update Pagetree)
are removed.

The GET/POST parameter "uPT" is there to explicitly request a refresh of
the page tree if visible on the next module load.

The GET/POST parameter "prErr" adds flash messages to the session queue
which should be a default behaviour anyway.

Please note that this only affects the SimpleDataHandlerController (BE module "tce_db") and no datahandler calls itself.

The first parameter of DataHandler->printLogErrorMessages() is removed
as it is unused.

Resolves: #82629
Releases: master
Change-Id: Ifc925e49b49f1537a65141810403a7758e0b448b
Reviewed-on: https://review.typo3.org/54244
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Susanne Moog <susanne.moog@typo3.org>
Reviewed-by: Frank Naegler <frank.naegler@typo3.org>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
2 years ago[BUGFIX] CKEditor should not create uploads subfolder 58/54258/2
Benni Mack [Mon, 2 Oct 2017 08:43:11 +0000 (10:43 +0200)]
[BUGFIX] CKEditor should not create uploads subfolder

The system extension rte_ckeditor creates an upload folder which is always
empty and not in use by the extension or something else.

Resolves: #82633
Releases: master, 8.7
Change-Id: Ic4877a9cbdbb078e5a7745f609f81aeaa5c991c3
Reviewed-on: https://review.typo3.org/54258
Reviewed-by: Frans Saris <franssaris@gmail.com>
Tested-by: Frans Saris <franssaris@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[!!!][TASK] Move EXT:info_pagetsconfig into EXT:info 32/53532/9
Georg Ringer [Sun, 16 Jul 2017 14:13:49 +0000 (16:13 +0200)]
[!!!][TASK] Move EXT:info_pagetsconfig into EXT:info

Migrate the extension info_pagetsconfig into the extension
info where also all other controllers are.

Resolves: #82505
Releases: master
Change-Id: I916d2ef6c00f6a5fa2cfca9c6b3f340aa4aefb44
Reviewed-on: https://review.typo3.org/53532
Reviewed-by: Mona Muzaffar <mona.muzaffar@gmx.de>
Tested-by: Mona Muzaffar <mona.muzaffar@gmx.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[BUGFIX] Fix wrong matcher class name 57/54257/3
Benni Mack [Mon, 2 Oct 2017 08:33:27 +0000 (10:33 +0200)]
[BUGFIX] Fix wrong matcher class name

Resolves: #82632
Related: #82014
Releases: master
Change-Id: I987bdda0bdc5bc91c1422b7558bc220f516e79ea
Reviewed-on: https://review.typo3.org/54257
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: TYPO3com <no-reply@typo3.com>
2 years ago[CLEANUP] Review comment of PageRepository::getPage() 78/52878/9
Elmar Hinz [Sat, 20 May 2017 14:09:58 +0000 (16:09 +0200)]
[CLEANUP] Review comment of PageRepository::getPage()

Note:

Type hints are not trivial to add to the signature. Although the
method always returns an array, there are unit tests that make use
of the assumption that the method is allowed to return null. Fixing
this goes beyond the goals of this ticket.

Releases: master, 8.7
Resolves: #81280
Change-Id: If47041fb6fc4835de879e9077b0359063083cbd2
Reviewed-on: https://review.typo3.org/52878
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
2 years ago[CLEANUP] Review comment of fetch_the_id 92/52892/7
Elmar Hinz [Sun, 21 May 2017 17:50:21 +0000 (19:50 +0200)]
[CLEANUP] Review comment of fetch_the_id

Change-Id: I590025affea93d84a616b92f9b51211448ec3280
Releases: master
Resolves: #81280
Reviewed-on: https://review.typo3.org/52892
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
2 years ago[BUGFIX] EXT:form - open the ElementBrowser in modal window 15/54115/3
Ralf Zimmermann [Mon, 11 Sep 2017 11:50:23 +0000 (13:50 +0200)]
[BUGFIX] EXT:form - open the ElementBrowser in modal window

Migrate the JS ElementBrowser to the modal which was introduced with
issue #82427.

Resolves: #82443
Releases: master
Change-Id: Ib71a15a7ce9638de4eb423d896fefa47bfa15e96
Reviewed-on: https://review.typo3.org/54115
Reviewed-by: Bjoern Jacob <bjoern.jacob@tritum.de>
Tested-by: Bjoern Jacob <bjoern.jacob@tritum.de>
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] Merge EXT:cshmanual into EXT:documentation 80/53180/10
Georg Ringer [Fri, 9 Jun 2017 19:18:42 +0000 (21:18 +0200)]
[TASK] Merge EXT:cshmanual into EXT:documentation

To reduce the amount of system extensions the extension
cshmanual can be merged into EXT:documentation.

Resolves: #81574
Releases: master
Change-Id: I5cd728938ffd319f5c13bf0af4caeda48c08ee37
Reviewed-on: https://review.typo3.org/53180
Reviewed-by: Mona Muzaffar <mona.muzaffar@gmx.de>
Tested-by: Mona Muzaffar <mona.muzaffar@gmx.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[BUGFIX] Bring RTE linkbrowser options back 81/53481/5
Frans Saris [Fri, 14 Jul 2017 18:57:32 +0000 (20:57 +0200)]
[BUGFIX] Bring RTE linkbrowser options back

The CKEditor link browser handled the configuration array still as a
TypoScript array with dots. But the parsed YAML config is a plain array
now.

This patch makes sure the BrowserLinkController handles this config like
a normal array. Further it fixes the missing JavaScript part to set the
link title based on selected css class.

Resolves: #81381
Releases: master, 8.7
Change-Id: Ie4749f820082e185208bf811aad7133bfc82d37a
Reviewed-on: https://review.typo3.org/53481
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Jigal van Hemert <jigal.van.hemert@typo3.org>
Reviewed-by: Markus Sommer <markus.sommer@typo3.org>
Tested-by: Markus Sommer <markus.sommer@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] Increase sequence if uid gets forced for PostgreSQL 19/51819/11
Manuel Selbach [Fri, 24 Feb 2017 09:03:54 +0000 (10:03 +0100)]
[BUGFIX] Increase sequence if uid gets forced for PostgreSQL

If a uid gets forcefully set during a insert into the database
MySQL will automatically update the auto_increment primary key.
Other databases might need post processing, i.e PostgreSQL as
it uses a sequence to generate auto increment uids. Sequences
do not get transparently updated on inserts.

Resolves: #79976
Releases: master, 8.7
Change-Id: I033deb63bf9bc2e072408885a534752573ba4c7c
Reviewed-on: https://review.typo3.org/51819
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: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
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[BUGFIX] Avoid unnecessary I/O in GeneralUtility::createVersionNumberedFilename() 21/54221/3
Faton Haliti [Wed, 27 Sep 2017 07:56:13 +0000 (09:56 +0200)]
[BUGFIX] Avoid unnecessary I/O in GeneralUtility::createVersionNumberedFilename()

With this commit we prevent to check for existence of file
when we don't need to, because mode is not either 'embed' or
'querystring'

Resolves: #82565
Releases: master, 8.7
Change-Id: I829b8e87d951a6f170bc03d8b7836f671b51bdf4
Reviewed-on: https://review.typo3.org/54221
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Anders Kostending <aha@systime.dk>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Oliver Klee <typo3-coding@oliverklee.de>
Reviewed-by: Wolfgang Klinger <wolfgang@wazum.com>
Reviewed-by: Frans Saris <franssaris@gmail.com>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
Reviewed-by: Frank Naegler <frank.naegler@typo3.org>
Tested-by: Frank Naegler <frank.naegler@typo3.org>
2 years ago[TASK] Remove hardcoded check on typo3/sysext/ 30/54230/3
Benni Mack [Thu, 28 Sep 2017 14:53:28 +0000 (16:53 +0200)]
[TASK] Remove hardcoded check on typo3/sysext/

When running TYPO3 with custom components (subtree), the typo3/sysext/
directory is not always necessary, and the hardcoded check is obsolete.

Resolves: #82622
Releases: master, 8.7
Change-Id: I3e939c80fff30d82dd4eb624c2f4bc5db1696809
Reviewed-on: https://review.typo3.org/54230
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Frank Naegler <frank.naegler@typo3.org>
Tested-by: Frank Naegler <frank.naegler@typo3.org>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
2 years ago[BUGFIX] Handle empty string icon-overlay identifier 12/54212/4
Romain Canon [Sat, 23 Sep 2017 13:07:02 +0000 (15:07 +0200)]
[BUGFIX] Handle empty string icon-overlay identifier

An overlay identifier can be an empty string, for instance when the icon API is
called with the Fluid icon view helper:

```
<core:icon identifier="tx-my-icon"
           overlay="{f:if(condition: myCondition, then: 'my-overlay-icon')}" />
```

In the example below, if the condition `myCondition` is `false`, an empty string
is used for the overlay parameter.

---

Previously, an empty string as overlay identifier would lead to the basic "not
found icon" being used as overlay.

With this commit, the behaviour for an empty string will now be the same as when
 `null` is used.

---

Resolves: #82547
Releases: master, 8.7
Change-Id: I6fb8207f302873e0188d71a210485bbf8565eff9
Reviewed-on: https://review.typo3.org/54212
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Romain Canon <romain.hydrocanon@gmail.com>
Tested-by: Romain Canon <romain.hydrocanon@gmail.com>
Reviewed-by: Frank Naegler <frank.naegler@typo3.org>
Tested-by: Frank Naegler <frank.naegler@typo3.org>
2 years ago[TASK] Use HTML5 validation for Install Tool test email address 16/54216/6
Georg Ringer [Tue, 26 Sep 2017 05:56:23 +0000 (07:56 +0200)]
[TASK] Use HTML5 validation for Install Tool test email address

Let the browser do the validation for the test email address.

Resolves: #82562
Releases: master
Change-Id: I8932eab709cb89d669b8931872d7e1cbda4d0fb1
Reviewed-on: https://review.typo3.org/54216
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
Reviewed-by: Mathias Schreiber <mathias.schreiber@typo3.com>
Tested-by: Mathias Schreiber <mathias.schreiber@typo3.com>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Frank Naegler <frank.naegler@typo3.org>
Tested-by: Frank Naegler <frank.naegler@typo3.org>
2 years ago[BUGFIX] Fix handling of Storage.Client 29/54229/2
Andreas Fernandez [Thu, 28 Sep 2017 13:17:14 +0000 (15:17 +0200)]
[BUGFIX] Fix handling of Storage.Client

The "Storage" module gets the missing "unset" method to remove values
from localStorage. The "isset" method is fixed to determine whether
a value is really existing.

Resolves: #82613
Releases: master, 8.7
Change-Id: Icb4b862b380811a7ba7f897be3c4809b718d2810
Reviewed-on: https://review.typo3.org/54229
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Benni Mack <benni@typo3.org>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
Reviewed-by: Frank Naegler <frank.naegler@typo3.org>
Tested-by: Frank Naegler <frank.naegler@typo3.org>
2 years ago[!!!][TASK] Remove RDCT redirect functionality 25/54225/5
Benni Mack [Thu, 28 Sep 2017 07:51:09 +0000 (09:51 +0200)]
[!!!][TASK] Remove RDCT redirect functionality

The very legacy RDCT code and the corresponding DB table
"cache_md5params" has been long superseeded and is be
completely moved into its own extension "RDCT" published
on GitHub https://github.com/FriendsOfTYPO3/rdct and in TER.

An upgrade wizard checks if the DB table is filled,
and then installs EXT:rdct.

Note #1: I decided to trigger the hard breaking way,
instead of deprecation as the whole logic (evalution,
static method call in GeneralUtility removed) is interconnected
and only makes sense if all places are there.

Note #2: This functionality has not been in use in the
TYPO3 Core since 4.x - at all, but is in place for
other extensions.

Resolves: #82572
Releases: master
Change-Id: I1514293500c385f7b8c1fff440cca3ab3845615b
Reviewed-on: https://review.typo3.org/54225
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
Reviewed-by: Frank Naegler <frank.naegler@typo3.org>
Tested-by: Frank Naegler <frank.naegler@typo3.org>
2 years ago[TASK] Update nikic/php-parser from v3.0.6 to v3.1.1 17/54217/3
Georg Ringer [Tue, 26 Sep 2017 07:49:30 +0000 (09:49 +0200)]
[TASK] Update nikic/php-parser from v3.0.6 to v3.1.1

Resolves: #82620
Releases: master
Change-Id: I50d8186fc84590e063bff5dabcf2e52ea4fb49a7
Reviewed-on: https://review.typo3.org/54217
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Frank Naegler <frank.naegler@typo3.org>
Tested-by: Frank Naegler <frank.naegler@typo3.org>
2 years ago[TASK] EXT:form - explain initializeFormElement hook 34/54234/2
Ralf Zimmermann [Fri, 29 Sep 2017 11:32:16 +0000 (13:32 +0200)]
[TASK] EXT:form - explain initializeFormElement hook

Add more explanations about the initializeFormElement
hook into the docs.

Resolves: #82615
Releases: master, 8.7
Change-Id: I670625533f95b92a37f5536096eb2cc4211d33d9
Reviewed-on: https://review.typo3.org/54234
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] Flexform support for type:passthrough 91/47491/15
Frank Naegler [Fri, 1 Apr 2016 14:05:31 +0000 (16:05 +0200)]
[TASK] Flexform support for type:passthrough

Resolves: #20705
Releases: master
Change-Id: I3476f6945b645a3b906e49efbdeaa385411c50ea
Reviewed-on: https://review.typo3.org/47491
Tested-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Reviewed-by: Frank Naegler <frank.naegler@typo3.org>
Tested-by: Frank Naegler <frank.naegler@typo3.org>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
2 years ago[BUGFIX] Fix broken reload of save+preview links 18/54218/9
Frank Naegler [Tue, 26 Sep 2017 10:43:14 +0000 (12:43 +0200)]
[BUGFIX] Fix broken reload of save+preview links

Resolves: #82456
Related: #81250
Releases: master
Change-Id: Idab208c3273adbd5800edab51bd9ca7bbec19af8
Reviewed-on: https://review.typo3.org/54218
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Jasmina Ließmann <code@frauliessmann.de>
Reviewed-by: Mathias Schreiber <mathias.schreiber@typo3.com>
Tested-by: Mathias Schreiber <mathias.schreiber@typo3.com>
Reviewed-by: Helmut Hummel <typo3@helhum.io>
Tested-by: Helmut Hummel <typo3@helhum.io>
2 years ago[BUGFIX] Allows setting extended search per default 25/51925/7
Dominik Winkelbauer [Thu, 2 Mar 2017 10:12:04 +0000 (11:12 +0100)]
[BUGFIX] Allows setting extended search per default

Makes it possible to enable extended search per default.

Change-Id: I793faa6d7f325661c193bf4661bfd924a6cfb69a
Resolves: #80091
Releases: master,8.7
Reviewed-on: https://review.typo3.org/51925
Reviewed-by: Christian Futterlieb <christian@futterlieb.ch>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
Reviewed-by: Frank Naegler <frank.naegler@typo3.org>
Tested-by: Frank Naegler <frank.naegler@typo3.org>
2 years ago[BUGFIX] Ignore missing folders from psr-4 definitions 07/54207/6
Helmut Hummel [Fri, 22 Sep 2017 12:16:49 +0000 (14:16 +0200)]
[BUGFIX] Ignore missing folders from psr-4 definitions

Even when generating an optimized class map for psr-4
classes, Composer gracefully ignores folders defined
in psr-4 autoload section, if they do not exist.

TYPO3 however fails with an exception.

Also change the logic to always generate a class map
for improved class loading performance. This happened
anyway already when calling the autoload generation twice.

Resolves: #82542
Releases: master, 8.7
Change-Id: I4c3ab8ff7affeba25313d93e40e35f78bb78492d
Reviewed-on: https://review.typo3.org/54207
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: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Nicole Cordes <typo3@cordes.co>
Tested-by: Nicole Cordes <typo3@cordes.co>
2 years ago[BUGFIX] Add backend authentication to Extbase CommandController 22/54222/8
Wolfgang Klinger [Wed, 27 Sep 2017 08:09:42 +0000 (10:09 +0200)]
[BUGFIX] Add backend authentication to Extbase CommandController

Ensure backwards compatibility with existing Extbase CommandControllers
by making sure the _cli_ user is always authenticated, so e.g. calls to
the DataHandler do not require explicit authentication in the extensions
CommandController.

Resolves: #82566
Releases: master, 8.7
Change-Id: Ice9b039558b68050c08a53f1c50dc726ec657522
Reviewed-on: https://review.typo3.org/54222
Reviewed-by: Stephan Großberndt <stephan@grossberndt.de>
Tested-by: Stephan Großberndt <stephan@grossberndt.de>
Reviewed-by: Wolfgang Klinger <wolfgang@wazum.com>
Tested-by: Wolfgang Klinger <wolfgang@wazum.com>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
2 years ago[BUGFIX] Use localize command when translating page record 45/52945/6
Frans Saris [Fri, 26 May 2017 13:51:03 +0000 (15:51 +0200)]
[BUGFIX] Use localize command when translating page record

Currently, when using the dropdown to create a page translation within
the Page Module, DataHandler is called with just a simple "new" record
of type "pages_language_overlay".

However, a classic "localize" DataHandler command is more appropriate,
allowing to utilize several localization functionalities built in.

In order to achieve this, the option tags in the selector contain the full
URL to the DataHandler call, as well as the redirect to FormEngine (which
in turn redirects back to the current page module URL).

Resolves: #81345
Releases: master, 8.7
Change-Id: I740c2d1d11494963283bc14b1f534bd344d0764a
Reviewed-on: https://review.typo3.org/52945
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: TYPO3com <no-reply@typo3.com>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
2 years ago[CLEANUP] Remove invalid paths in PSR-4 section 11/54211/2
Georg Ringer [Fri, 22 Sep 2017 17:42:42 +0000 (19:42 +0200)]
[CLEANUP] Remove invalid paths in PSR-4 section

Remove not existing paths from root composer.json.
For 8.7 the path to context_help must be kept.

Resolves: #82546
Releases: master, 8.7
Change-Id: Iea32a481011d906ea7057072413e6ae01a2ce771
Reviewed-on: https://review.typo3.org/54211
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Tested-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Reviewed-by: Jan Stockfisch <jan.stockfisch@googlemail.com>
Reviewed-by: Frank Naegler <frank.naegler@typo3.org>
Tested-by: Frank Naegler <frank.naegler@typo3.org>
2 years ago[BUGFIX] Do not ignore empty value in OptionViewHelper 22/54122/2
Raphael Graf [Mon, 11 Sep 2017 18:55:51 +0000 (20:55 +0200)]
[BUGFIX] Do not ignore empty value in OptionViewHelper

Resolves: #82457
Releases: master
Change-Id: I42a1fdb85e87db4601ceefdd52e0dd4f1722bd23
Reviewed-on: https://review.typo3.org/54122
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Joerg Boesche <typo3@joergboesche.de>
Reviewed-by: David Hoeckele <hoeckele@netvertising.ch>
Reviewed-by: Markus Sommer <markus.sommer@typo3.org>
Tested-by: Markus Sommer <markus.sommer@typo3.org>
Reviewed-by: Sascha Rademacher <s.rademacher@neusta.de>
Tested-by: Sascha Rademacher <s.rademacher@neusta.de>
Reviewed-by: Jan Stockfisch <jan.stockfisch@googlemail.com>
Reviewed-by: Frank Naegler <frank.naegler@typo3.org>
Tested-by: Frank Naegler <frank.naegler@typo3.org>
2 years ago[TASK] Provide uid in ElementInformation popup 87/54187/3
Georg Ringer [Wed, 20 Sep 2017 18:39:19 +0000 (20:39 +0200)]
[TASK] Provide uid in ElementInformation popup

The uid of a record is an important information and should
be provided as well.

Resolves: #82526
Releases: master, 8.7
Change-Id: Ic44ca6a24e21c09ced78203ed4a73907906bf568
Reviewed-on: https://review.typo3.org/54187
Tested-by: TYPO3com <no-reply@typo3.com>
Tested-by: Riccardo De Contardi <erredeco@gmail.com>
Reviewed-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Reviewed-by: Sascha Rademacher <s.rademacher@neusta.de>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
2 years ago[BUGFIX] Disable install tool access in switch user mode 81/54181/6
Mona Muzaffar [Wed, 20 Sep 2017 11:45:09 +0000 (13:45 +0200)]
[BUGFIX] Disable install tool access in switch user mode

During switch user mode the install tool modules are never visible.

Resolves: #82517
Releases: master
Change-Id: Ie367cb7f0208a7414ada38d40f8cdd3ab287da52
Reviewed-on: https://review.typo3.org/54181
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Jan Stockfisch <jan.stockfisch@googlemail.com>
Tested-by: Jan Stockfisch <jan.stockfisch@googlemail.com>
Reviewed-by: Sascha Rademacher <s.rademacher@neusta.de>
Tested-by: Sascha Rademacher <s.rademacher@neusta.de>
Reviewed-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Tested-by: Morton Jonuschat <m.jonuschat@mojocode.de>
2 years ago[!!!][TASK] Deprecate GeneralUtility::devLog() 59/24559/29
Markus Klein [Sun, 10 Sep 2017 01:15:09 +0000 (03:15 +0200)]
[!!!][TASK] Deprecate GeneralUtility::devLog()

The core now directly logs to the Logging API.

Resolves: #52694
Releases: master
Change-Id: Ia3c8f1c3608aff540a990e9078132b233bbedde6
Reviewed-on: https://review.typo3.org/24559
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Frank Naegler <frank.naegler@typo3.org>
Tested-by: Frank Naegler <frank.naegler@typo3.org>
Reviewed-by: Helmut Hummel <typo3@helhum.io>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
2 years ago[BUGFIX] Fix broken redis tests 95/54195/2
Frank Naegler [Thu, 21 Sep 2017 10:09:19 +0000 (12:09 +0200)]
[BUGFIX] Fix broken redis tests

Resolves: #82533
Releases: master, 8.7, 7.6
Change-Id: Ia47f604b9cb7fa53d3707e6150554709cbfdb763
Reviewed-on: https://review.typo3.org/54195
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: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
2 years ago[BUGFIX] Do not recalculate PIDs for every query in BE search 30/53930/4
Susanne Moog [Wed, 6 Sep 2017 13:21:08 +0000 (15:21 +0200)]
[BUGFIX] Do not recalculate PIDs for every query in BE search

The backend search "Show all" link recalculated allowed PIDs for
the current user for all search operations resulting in an
exponentially increasing amount of sql queries depending on the
number of pages in an installation.

As the list of allowed pages for the user does not change during
the request, it is enough to calculate it once.

Change-Id: I9262a5d8425d04cb4ff1e35b2c3cc0ec96b8fa53
Resolves: #82083
Releases: master, 8.7
Reviewed-on: https://review.typo3.org/53930
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: Joerg Boesche <typo3@joergboesche.de>
Tested-by: Joerg Boesche <typo3@joergboesche.de>
Reviewed-by: Helmut Hummel <typo3@helhum.io>
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Tested-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
2 years ago[BUGFIX] MenuProcessor: Allow sub keys of `addQueryString` 83/54183/2
Andreas Fernandez [Wed, 20 Sep 2017 12:20:55 +0000 (14:20 +0200)]
[BUGFIX] MenuProcessor: Allow sub keys of `addQueryString`

THis patch allows sub keys of `addQueryString`, e.g. `exclude`.

Resolves: #82520
Releases: master, 8.7
Change-Id: I8a6b5f924292cf56326cd18d8bb9b7887d095288
Reviewed-on: https://review.typo3.org/54183
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: Frank Naegler <frank.naegler@typo3.org>
Tested-by: Frank Naegler <frank.naegler@typo3.org>
2 years ago[BUGFIX] Fix undefined variable to evaluate `fileFolder_recursions` correctly 74/54174/4
Andreas Fernandez [Tue, 19 Sep 2017 14:24:38 +0000 (16:24 +0200)]
[BUGFIX] Fix undefined variable to evaluate `fileFolder_recursions` correctly

Resolves: #82512
Releases: master, 8.7
Change-Id: Ibfbd1a8e347f5b25871f5d073994bdb2af264051
Reviewed-on: https://review.typo3.org/54174
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: Frank Naegler <frank.naegler@typo3.org>
Tested-by: Frank Naegler <frank.naegler@typo3.org>
2 years ago[BUGFIX] fix spelling of content-type http-header in response classes 00/54200/2
Stephan Jorek [Thu, 21 Sep 2017 14:51:48 +0000 (16:51 +0200)]
[BUGFIX] fix spelling of content-type http-header in response classes

In the constructors of the two new http-response classes ...

* TYPO3\CMS\Core\Http\HtmlResponse
* TYPO3\CMS\Core\Http\JsonResponse

... change the spelling of the value of the lowercased header-names
for the default/fallback 'Content-Type' header from 'Content-type'
to 'Content-Type'. Mind the capital 'T' in '...-Type'.

Resolves: #82537
Releases: master
Change-Id: Ib7b3d0f6c6de0a1490345e9e537d3545b80b4d82
Reviewed-on: https://review.typo3.org/54200
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>
2 years ago[BUGFIX] Fix exception in FE when using predefined admPanel config 23/54123/2
Markus Klein [Mon, 11 Sep 2017 11:14:27 +0000 (13:14 +0200)]
[BUGFIX] Fix exception in FE when using predefined admPanel config

Enforcing the admPanel for users via user TSconfig is working again
without crashing the FE.

We make sure that when the admPanel configuration is fetched, we
do not fall back to the toplevel value if the requested sub-value
does not exist.

Resolves: #82460
Releases: master, 8.7, 7.6
Change-Id: Ibd288805aa6c91d74eee8a9c80ebcce62d1f860f
Reviewed-on: https://review.typo3.org/54123
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Joerg Boesche <typo3@joergboesche.de>
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: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
2 years ago[BUGFIX] Change username from _cli_scheduler to _cli_ 88/54188/5
Raphael Graf [Wed, 20 Sep 2017 18:51:51 +0000 (20:51 +0200)]
[BUGFIX] Change username from _cli_scheduler to _cli_

and remove the whole paragraph about specific rights of the user.

Releases: master, 8.7
Resolves: #82527
Change-Id: Icd65ba13c7464fdd0290cec48f08de9d0934e796
Reviewed-on: https://review.typo3.org/54188
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
2 years ago[TASK] Increase size of domain name field 86/54186/2
Georg Ringer [Wed, 20 Sep 2017 18:19:39 +0000 (20:19 +0200)]
[TASK] Increase size of domain name field

Allow 255 chars for a domain to be in sync with RFC 1035.

Resolves: #82515
Releases: master, 8.7, 7.6
Change-Id: I63916cb115b00e50c62c1926cf890e29bf89de77
Reviewed-on: https://review.typo3.org/54186
Reviewed-by: Patrik Karisch <p.karisch@pixelart.at>
Tested-by: Patrik Karisch <p.karisch@pixelart.at>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Alexander Schnitzler <review.typo3.org@alexanderschnitzler.de>
Tested-by: Alexander Schnitzler <review.typo3.org@alexanderschnitzler.de>
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: Susanne Moog <susanne.moog@typo3.org>
2 years ago[!!!][FEATURE] EXT:form - add content element uid to the form identifier 25/54125/4
Ralf Zimmermann [Tue, 12 Sep 2017 10:39:47 +0000 (12:39 +0200)]
[!!!][FEATURE] EXT:form - add content element uid to the form identifier

Make it possible to use the same form multiple times on one page by
adding the content element uid to the form identifier.

Resolves: #81775
Releases: master
Change-Id: I001a654f8ffc47c91c984133d8475605f1b15b7a
Reviewed-on: https://review.typo3.org/54125
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Joerg Boesche <typo3@joergboesche.de>
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
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[BUGFIX] Remove duplicate PageTS cache in Typo3DbBackend 51/54151/3
Alexander Schnitzler [Sun, 17 Sep 2017 10:02:11 +0000 (12:02 +0200)]
[BUGFIX] Remove duplicate PageTS cache in Typo3DbBackend

The Typo3DbBackend fetches PageTS via BackendUtility::getPagesTSconfig
and then stores a copy of it (for each page separately) in a static
property. This is a bug and leads to performance decreases because
the PageTS is already cached via BackendUtility::getPagesTSconfig
and it's done even better than in Typo3DbBackend because the backend
utility store a single cache entry for different pages with the same
PageTS. The Typo3DbBackend does create a copy of the PageTS for each
page.

Resolves: #82496
Releases: master, 8.7
Change-Id: I61bd6fa8eb3ba29baea355ae01a8d71f7ae7e963
Reviewed-on: https://review.typo3.org/54151
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
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] Update ExtensionUtility::registerPlugin() icon parameter docs 87/53987/3
Mathias Brodala [Fri, 8 Sep 2017 08:40:08 +0000 (10:40 +0200)]
[BUGFIX] Update ExtensionUtility::registerPlugin() icon parameter docs

The icon parameter is passed through to TCA "select" items, thus aside
from an icon file path an icon identifier can be used here.

Resolves: #82351
Releases: master, 8.7
Change-Id: Ia84aa9c739c10372f1ad76c5b4dde9f0b1a62c99
Reviewed-on: https://review.typo3.org/53987
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
Tested-by: Susanne Moog <susanne.moog@typo3.org>
2 years ago[BUGFIX] Stabilize flaky acceptance test 80/54180/2
Christian Kuhn [Wed, 20 Sep 2017 11:41:15 +0000 (13:41 +0200)]
[BUGFIX] Stabilize flaky acceptance test

The initial view of the installer enables the 'proceed'
buttons on first step via JS. The ac test has to wait
until the buttons are rendered before proceeding.

Change-Id: I1426f1434d80124ac5036c1270668867c86fc2d6
Resolves: #82516
Related: #82504
Releases: master
Reviewed-on: https://review.typo3.org/54180
Reviewed-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] Support target in fluid_styled_content menus 60/54160/4
Mats Svensson [Tue, 12 Sep 2017 13:06:22 +0000 (15:06 +0200)]
[BUGFIX] Support target in fluid_styled_content menus

Add condition and only render target attribute if set on page.

Resolves: #82464
Releases: master, 8.7
Change-Id: I14c561188f1e8a7389b145adf85c64918e61d461
Reviewed-on: https://review.typo3.org/54160
Reviewed-by: Benjamin Kott <benjamin.kott@outlook.com>
Tested-by: Benjamin Kott <benjamin.kott@outlook.com>
Reviewed-by: Sascha Wilking <sascha.wilking@hmmh.de>
Tested-by: Sascha Wilking <sascha.wilking@hmmh.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[TASK] Remove BackendUserRepository injection in NoteController 66/54166/5
Georg Ringer [Tue, 19 Sep 2017 10:06:54 +0000 (12:06 +0200)]
[TASK] Remove BackendUserRepository injection in NoteController

To improve the performance of sys_note rendering, the injection can
be skipped as the current user id can be used directly.

Resolves: #82506
Releases: master
Change-Id: I40b04ac725628f4dcfb3f1e5a280edf9c8e545f9
Reviewed-on: https://review.typo3.org/54166
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
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] EXT:form - allow validation translation for a field in all forms 88/53988/8
David Gurk [Fri, 8 Sep 2017 07:23:03 +0000 (09:23 +0200)]
[TASK] EXT:form - allow validation translation for a field in all forms

This allows to translate a form element validation messages without
a specific form definition identifier.

Resolves: #82366
Releases: master, 8.7
Change-Id: I1db756c7f3c84d599f5c3c2265d8a7ca07fc2205
Reviewed-on: https://review.typo3.org/53988
Reviewed-by: Mathias Brodala <mbrodala@pagemachine.de>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Daniel Lorenz <daniel.lorenz@extco.de>
Reviewed-by: Ralf Zimmermann <ralf.zimmermann@tritum.de>
Reviewed-by: Sonja Großewinkelmann <s.grossewinkelmann@neusta.de>
Tested-by: Ralf Zimmermann <ralf.zimmermann@tritum.de>
Reviewed-by: Bjoern Jacob <bjoern.jacob@tritum.de>
Tested-by: Bjoern Jacob <bjoern.jacob@tritum.de>
Reviewed-by: Sascha Rademacher <sascha.rademacher+typo3@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] EXT:form - fix documentation paths 32/54132/4
Ralf Zimmermann [Wed, 13 Sep 2017 11:10:16 +0000 (13:10 +0200)]
[TASK] EXT:form - fix documentation paths

Cleanup some leftovers from issue #82336 (review:53946) within
the EXT:form documentation. Basically, long file paths are shortened.

Resolves: #82472
Related: #82336
Releases: master
Change-Id: I6ecac403a91d195d0c35d99281ecbe665ca8ec54
Reviewed-on: https://review.typo3.org/54132
Reviewed-by: Bjoern Jacob <bjoern.jacob@tritum.de>
Tested-by: Bjoern Jacob <bjoern.jacob@tritum.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[BUGFIX] Add &L=0 to Indexed Search results only when needed. 78/53978/5
Łukasz Uznański [Thu, 7 Sep 2017 20:44:21 +0000 (22:44 +0200)]
[BUGFIX] Add &L=0 to Indexed Search results only when needed.

L=0 is added to Indexed Search results links if current page language is
> 0.

Resolves: #81164
Related: #81994
Related: #81986
Releases: master, 8.7
Change-Id: Ia9738b4d64bdcf1f3b2d6fa91d446f2f22c8130b
Reviewed-on: https://review.typo3.org/53978
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Reviewed-by: Sascha Rademacher <sascha.rademacher+typo3@gmail.com>
Reviewed-by: Mona Muzaffar <mona.muzaffar@gmx.de>
Tested-by: Mona Muzaffar <mona.muzaffar@gmx.de>
Reviewed-by: Joerg Boesche <typo3@joergboesche.de>
Tested-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] Install tool: JS driven routing 28/54128/16
Christian Kuhn [Tue, 12 Sep 2017 22:27:21 +0000 (00:27 +0200)]
[TASK] Install tool: JS driven routing

The install tool suffered from three main issues since 6.2 rewrite:
* The "step" installer was re-used for recovery and installation
* The routing logic was server based and threw lots of redirects
  which lead to redirect loops
* The Controller/Action class structure was weird and hard to
  understand

The patch solves this with a rather huge rewrite:
* There are two request handlers: One for the Installer, one for
  the install tool.
* A simple list of controllers. One InstallerController for the
  step installer, the others for the main module points of the
  install tool and a Login and a Layout controller.
* Single action code is moved into controllers.
* Both tool and installer first load only a <head> section that
  contain JS references. All other calls are ajax based and the
  routing is done JS side.
* Installer and install tool no longer share controller code
  or templates, the installer can potentially be fully extracted
  from ext:install in another step.
* Installer.js is the "walk through installation" module of the
  installer.
* Router.js is the routing module for the install tool, all tool
  ajax requests get specific urls from this module and hand over
  errors to the Router.
* Error handling is handled on JS side: If for instance
  the login session expires and user clicks elsewhere, a 403
  response is returned which is handled by JS to route to login.
  This is also the place where a recovery can hook in later.
* The silent configuration updater is executed again which was
  removed during one of the previous master patches.
* The template structure is much easier.
* Various card content which has been calculated when loading
  the card layout is moved to the ajax code for single card
  content. That increases the performance of the main module
  points and makes them pretty snappy.

Change-Id: Ib40f40acba17bb47142c0da1bcfb389ab9b4b3a1
Resolves: #82504
Releases: master
Reviewed-on: https://review.typo3.org/54128
Tested-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
Reviewed-by: Mona Muzaffar <mona.muzaffar@gmx.de>
Tested-by: Mona Muzaffar <mona.muzaffar@gmx.de>
Tested-by: Susanne Moog <susanne.moog@typo3.org>
Reviewed-by: Susanne Moog <susanne.moog@typo3.org>
2 years ago[FOLLOWUP][!!!][TASK] Replace GeneralUtility::sysLog() with Logging API 61/54161/2
Markus Klein [Mon, 18 Sep 2017 08:41:36 +0000 (10:41 +0200)]
[FOLLOWUP][!!!][TASK] Replace GeneralUtility::sysLog() with Logging API

Do not reference the logger inside a constructor as its injection
happens after the constructor has been executed.
Therefore create a dedicated instance for logging.

Resolves: #82430
Releases: master
Change-Id: Ic943ad53e4ae2abed3fabb55e24b17362de92e1c
Reviewed-on: https://review.typo3.org/54161
Reviewed-by: Alexander Schnitzler <review.typo3.org@alexanderschnitzler.de>
Tested-by: TYPO3com <no-reply@typo3.com>
Tested-by: Alexander Schnitzler <review.typo3.org@alexanderschnitzler.de>
Reviewed-by: Mathias Brodala <mbrodala@pagemachine.de>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Helmut Hummel <typo3@helhum.io>
Tested-by: Helmut Hummel <typo3@helhum.io>
2 years ago[TASK] Introduce command registry to aggregate console commands 20/54120/22
Alexander Schnitzler [Mon, 11 Sep 2017 16:52:08 +0000 (18:52 +0200)]
[TASK] Introduce command registry to aggregate console commands

This introduces an iterable command registry that
aggregates commands from Configuration/Commands.php files.
To speed things up for subsequent usage, a first level cache is used.

Resolves: #82455
Releases: master
Change-Id: Ibd123ef947d06939bc84f5ea609996fec85de6e8
Reviewed-on: https://review.typo3.org/54120
Reviewed-by: Helmut Hummel <typo3@helhum.io>
Tested-by: Helmut Hummel <typo3@helhum.io>
Reviewed-by: Mathias Brodala <mbrodala@pagemachine.de>
Tested-by: Mathias Brodala <mbrodala@pagemachine.de>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Alexander Schnitzler <review.typo3.org@alexanderschnitzler.de>
Tested-by: Alexander Schnitzler <review.typo3.org@alexanderschnitzler.de>
2 years ago[BUGFIX] Unify Extbase base command error 26/53926/7
Alexander Schnitzler [Wed, 6 Sep 2017 09:18:57 +0000 (11:18 +0200)]
[BUGFIX] Unify Extbase base command error

This changes the output of "typo3/sysext/core/bin/typo3 extbase"
to show the standard Symfony error message instead of
a misleading one.

Calling a Symfony base command outputs an error message
with a list of possible commands.

We change the command name of the core command to
get the same behavior for the Extbase base command.

Resolves: #81917
Releases: master, 8.7
Change-Id: I9ba05600a461aceb4180d3e2dd124a5ec3006e08
Reviewed-on: https://review.typo3.org/53926
Tested-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Reviewed-by: Alexander Schnitzler <review.typo3.org@alexanderschnitzler.de>
Tested-by: Alexander Schnitzler <review.typo3.org@alexanderschnitzler.de>
Reviewed-by: Helmut Hummel <typo3@helhum.io>
Tested-by: Helmut Hummel <typo3@helhum.io>
2 years ago[!!!][TASK] Replace GeneralUtility::sysLog() with Logging API 90/54090/8
Markus Klein [Sat, 9 Sep 2017 23:49:27 +0000 (01:49 +0200)]
[!!!][TASK] Replace GeneralUtility::sysLog() with Logging API

The function GeneralUtility::sysLog() is deprecated and all calls
are replaced with direct calls to according Logging API methods.

The usual configuration options of the Logging API allows to
define all sorts of destinations for log entries, including
the syslog facility as well as file targets.

Resolves: #82430
Releases: master
Change-Id: Ief3f3f14fd10a3ae90b9df4a5e4e7456c2f7619c
Reviewed-on: https://review.typo3.org/54090
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Alexander Schnitzler <review.typo3.org@alexanderschnitzler.de>
Tested-by: Alexander Schnitzler <review.typo3.org@alexanderschnitzler.de>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
2 years ago[BUGFIX] Fix Fluid template path issues 58/53958/12
Helmut Hummel [Thu, 14 Sep 2017 10:53:23 +0000 (12:53 +0200)]
[BUGFIX] Fix Fluid template path issues

* Widget template override also works with legacy config option
* Runtime cache is removed
* TemplatePathsTests from https://github.com/helhum/fluid_test are
  integrated

Resolves: #82344
Resolves: #82181
Resolves: #73207
Related: #82407
Releases: master, 8.7
Change-Id: Icd0a8aa3f1646315c094ad8768e2ff0bb34033d9
Reviewed-on: https://review.typo3.org/53958
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Claus Due <claus@phpmind.net>
Tested-by: Claus Due <claus@phpmind.net>
Reviewed-by: Nicole Cordes <typo3@cordes.co>
Tested-by: Nicole Cordes <typo3@cordes.co>
Reviewed-by: Helmut Hummel <typo3@helhum.io>
Tested-by: Helmut Hummel <typo3@helhum.io>
2 years ago[FOLLOWUP][TASK] Improve TCA cache 54/54154/2
Oliver Hader [Sun, 17 Sep 2017 18:19:57 +0000 (20:19 +0200)]
[FOLLOWUP][TASK] Improve TCA cache

Allowed classes in the options of unserialize invocation need to
be injected using the 'allowed_classes' array index - the current
implementation is casted to true which allows all classes.

Related: #82408
Releases: master, 8.7
Change-Id: I68fbd873a5a7057630a48586878c77547b532348
Reviewed-on: https://review.typo3.org/54154
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Nicole Cordes <typo3@cordes.co>
Tested-by: Nicole Cordes <typo3@cordes.co>
Reviewed-by: Helmut Hummel <typo3@helhum.io>
Tested-by: Helmut Hummel <typo3@helhum.io>
2 years ago[BUGFIX] Revert "Improve performance of Fluid TemplatePaths" 47/54147/2
Benni Mack [Thu, 14 Sep 2017 14:52:08 +0000 (16:52 +0200)]
[BUGFIX] Revert "Improve performance of Fluid TemplatePaths"

This reverts commit 31310553bdfe724fd3be0d968cd2b4830bec24b0
due to side-effects introduced with this commit.

Change-Id: I67680820b0e139fea49cfcc0818cd97af3f6b9b9
Resolves: #82487
Reverts: #82232
Releases: master, 8.7
Reviewed-on: https://review.typo3.org/54147
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] Revert "Remove runtime cache and early return from TemplatePaths" 44/54144/3
Benni Mack [Thu, 14 Sep 2017 13:13:22 +0000 (15:13 +0200)]
[TASK] Revert "Remove runtime cache and early return from TemplatePaths"

This reverts commit 30c467196034ffdedc35e4be205ab4acec28a8c6
due to several side effects.

Change-Id: I43586802a9838faa2723fdcaa2e9e15492003830
Resolves: #82487
Reverts: #82196
Reverts: #82181
Releases: master, 8.7
Reviewed-on: https://review.typo3.org/54144
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] Remove system environment check for Soap 17/54117/3
Claus Due [Mon, 11 Sep 2017 14:26:28 +0000 (16:26 +0200)]
[TASK] Remove system environment check for Soap

TYPO3 no longer requires the Soap extension and
should not check for it in system environment checks.

This patch removes the check.

Change-Id: I3dc5715889af598054e2451c957a05c13f39b9fa
Resolves: #82452
Releases: master, 8.7
Reviewed-on: https://review.typo3.org/54117
Reviewed-by: Stephan Großberndt <stephan@grossberndt.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: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Alexander Schnitzler <review.typo3.org@alexanderschnitzler.de>
Tested-by: Alexander Schnitzler <review.typo3.org@alexanderschnitzler.de>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
2 years ago[BUGFIX] Use TCA to get `type` field instead of hardcoded fieldname 49/53749/8
Sascha Egerer [Mon, 21 Aug 2017 09:16:29 +0000 (11:16 +0200)]
[BUGFIX] Use TCA to get `type` field instead of hardcoded fieldname

The `type` field is configured via TCA and that configuration must
be used instead of hardcoding the fieldname.

Also change arrangement of the DataProvider load order to load
InitializeProcessedTca directly after ReturnUrl. This avoids the
need of accessing $GLOBALS['TCA'].

Resolves: #79954
Releases: master, 8.7
Change-Id: I67cbab00f6f0157c48c749cd8fbcd6cbc0ce1644
Reviewed-on: https://review.typo3.org/53749
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
2 years ago[BUGFIX] Install tool: Use standalone application in backend context 11/54111/4
Christian Kuhn [Mon, 11 Sep 2017 10:11:30 +0000 (12:11 +0200)]
[BUGFIX] Install tool: Use standalone application in backend context

With recent routing changes the install tool "backend context" has
been switched to a "normal" backend module loaded directly from
within the backend application. The standalone is an own application
with its own bootstrap.
This leads to various issues in the install tool since the backend
and standalone requests lead to different bootstrap states, for
instance TCA is initialized in BE context, but isn't in standalone.
Within backend, this won't change until the backend application can
bootstrap to different states depending on a controller.
To solve install tool related issues for now, the BackendModuleController
of the install tool called from within backend now starts a casual
install tool session and marks this session as "initiated from a
valid system maintainer". It then redirects to the standalone
application which omits login and enable install tool file check if
the session is marked as backend user session.

Change-Id: I352e6d04e7a91c56ccf2383f784ae94464c9aacd
Resolves: #82448
Related: #82306
Releases: master
Reviewed-on: https://review.typo3.org/54111
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[TASK] Move flash message handling from DocumentTemplate to own class 00/53200/3
Georg Ringer [Tue, 13 Jun 2017 12:33:57 +0000 (14:33 +0200)]
[TASK] Move flash message handling from DocumentTemplate to own class

To simplify the class \TYPO3\CMS\Backend\Template\DocumentTemplate, the
AJAX based flash message handling is moved to an own class.

Adding a RST file will be added with other removals within DocumentTemplate.

Resolves: #81564
Releases: master
Change-Id: Icff696f2dc8a2a895711218b86ea529128506161
Reviewed-on: https://review.typo3.org/53200
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Reviewed-by: Jan Stockfisch <jan.stockfisch@googlemail.com>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
2 years ago[TASK] Improve TCA cache 62/54062/4
Helmut Hummel [Sat, 9 Sep 2017 12:32:20 +0000 (14:32 +0200)]
[TASK] Improve TCA cache

Currently TCA is cached as serialized string using
the core cache.

Simplify the cache retrieval by taking advantage of
the code cache by using requireOnce and previously
storing the cache as PHP file that returns an array.
This significantly improves performance
with opcode cache enabled.

The cache identifier is changed to avoid conflicts
with previously stored data.

Resolves: #82408
Releases: master, 8.7
Change-Id: I59210fa800d10c14d21aceb7416ea418988d6ca5
Reviewed-on: https://review.typo3.org/54062
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Claus Due <claus@phpmind.net>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Reviewed-by: Helmut Hummel <typo3@helhum.io>
Tested-by: Helmut Hummel <typo3@helhum.io>
2 years ago[FEATURE] Inject logger into newly created objects 01/54101/9
Markus Klein [Sun, 10 Sep 2017 12:06:11 +0000 (14:06 +0200)]
[FEATURE] Inject logger into newly created objects

Instantiating new objects with makeInstance or the ObjectManager
will automatically inject a logger into the new objects, if those
implement the LoggerAwareInterface.

Resolves: #82441
Releases: master
Change-Id: I38e1a4619857f3fc2d88b0b95a9721b289c679d3
Reviewed-on: https://review.typo3.org/54101
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Tested-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Reviewed-by: Helmut Hummel <typo3@helhum.io>
Tested-by: Helmut Hummel <typo3@helhum.io>
2 years ago[!!!][BUGFIX] EXT:form - FlexForm settings overrule TypoScript settings 71/54071/6
Łukasz Uznański [Sat, 9 Sep 2017 16:35:10 +0000 (18:35 +0200)]
[!!!][BUGFIX] EXT:form - FlexForm settings overrule TypoScript settings

Releases: master
Resolves: #82252
Change-Id: I5cddefe1fd3fbf97e2cdc1aea1f373ef2076da72
Reviewed-on: https://review.typo3.org/54071
Reviewed-by: Bjoern Jacob <bjoern.jacob@tritum.de>
Reviewed-by: Joerg Kummer <typo3@enobe.de>
Tested-by: Joerg Kummer <typo3@enobe.de>
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: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
2 years ago[TASK] Use Response objects directly in Install Tool actions 59/54059/6
Benni Mack [Sat, 9 Sep 2017 11:48:42 +0000 (13:48 +0200)]
[TASK] Use Response objects directly in Install Tool actions

The patch introduces three shorthand Response classes for output:
HtmlResponse, JsonResponse and RedirectResponse which are used
throughout all the actions inside the Install Tool.

Resolves: #82415
Releases: master
Change-Id: I9edc15c341084fcb59a3b1dc45b42a6f65d87e7f
Reviewed-on: https://review.typo3.org/54059
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Reviewed-by: Markus Hölzle <typo3@markus-hoelzle.de>
Tested-by: Markus Hölzle <typo3@markus-hoelzle.de>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
2 years ago[BUGFIX] Fluid: Strict type checking when adding debug namespace 32/53832/3
Benni Mack [Wed, 30 Aug 2017 06:13:13 +0000 (08:13 +0200)]
[BUGFIX] Fluid: Strict type checking when adding debug namespace

The check on BE_USER->uc() in frontend context is cleaned up and more
strict, checking whether the object exists currently.

Additionally, the specific request type is used here.

Resolves: #82397
Releases: master, 8.7
Change-Id: Ibe0b533c23e1ba9c35ed2b6b9b0301e1a449c2e3
Reviewed-on: https://review.typo3.org/53832
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Tested-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
2 years ago[!!!][TASK] Remove legacy entrypoint for Install Tool 93/54093/2
Benni Mack [Sun, 10 Sep 2017 09:27:23 +0000 (11:27 +0200)]
[!!!][TASK] Remove legacy entrypoint for Install Tool

The entrypoint located under "typo3/sysext/install/Start/Install.php" has been
removed, as the entrypoint "typo3/install.php" is the new way to access the install
tool. The legacy entrypoint "typo3/install/index.php" will still be kept, as it is
easy for existing users to get a simple redirect (still marked as deprecated).

Resolves: #82433
Releases: master
Change-Id: I36a165649c7ca5a988379b70328e570fc45285b0
Reviewed-on: https://review.typo3.org/54093
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Tested-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
2 years ago[TASK] Provide missing example configuration for TYPO3 on Microsoft IIS 79/53379/4
Benjamin Kott [Sun, 2 Jul 2017 19:29:38 +0000 (21:29 +0200)]
[TASK] Provide missing example configuration for TYPO3 on Microsoft IIS

Releases: master, 8.7
Resolves: #81769
Change-Id: I92df0fc254b4a729298f32d1482ad2a0f39502bf
Reviewed-on: https://review.typo3.org/53379
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Benni Mack <benni@typo3.org>
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
2 years ago[BUGFIX] Fix parameters in recursiveConfigurationFetching() call 95/54095/2
Simon Praetorius [Sun, 10 Sep 2017 09:45:24 +0000 (11:45 +0200)]
[BUGFIX] Fix parameters in recursiveConfigurationFetching() call

Resolves: #82435
Relates: #82399
Releases: master
Change-Id: Ic2bb90883c0a6cec5e6bec051cb308c723392d75
Reviewed-on: https://review.typo3.org/54095
Reviewed-by: Markus Hölzle <typo3@markus-hoelzle.de>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Tested-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
2 years ago[FEATURE] Add crossorigin property to JavaScript files 55/48455/8
Jacob Dreesen [Sat, 9 Sep 2017 16:30:55 +0000 (18:30 +0200)]
[FEATURE] Add crossorigin property to JavaScript files

Add a property 'crossorigin="some-value"' to JavaScript files via TypoScript
page.includeJSlibs.<array>.crossorigin = some-value

The crossorigin property is automatically set to the value "anonymous" for
external JavaScript files with an integrity property if not explicitly set.

This patch affects the TypoScript PAGE properties
* includeJSlibs
* includeJSFooterlibs
* includeJS
* includeJSFooter

Resolves: #76459
Releases: master
Change-Id: Ie8d1d86ca5a8863c2a637395f1f87a463729691c
Reviewed-on: https://review.typo3.org/48455
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Tested-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
2 years ago[BUGFIX] Calculate workspace module route correctly 97/54097/3
Andreas Fernandez [Sun, 10 Sep 2017 10:45:44 +0000 (12:45 +0200)]
[BUGFIX] Calculate workspace module route correctly

Resolves: #82434
Related: #82406
Releases: master
Change-Id: I67182a131c2499713410807099f5d658a3858d63
Reviewed-on: https://review.typo3.org/54097
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@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] Route dispatching: Don't rely on id being an integer 98/54098/2
Andreas Fernandez [Sun, 10 Sep 2017 11:19:43 +0000 (13:19 +0200)]
[BUGFIX] Route dispatching: Don't rely on id being an integer

The route dispatcher can't rely on id being an integer, as file list for
example uses string identifier.

It is now checked whether the id is an int instead.

Resolves: #82440
Related: #82406
Releases: master
Change-Id: Idddc8484e0a597fec9d61c509d06f36736990c67
Reviewed-on: https://review.typo3.org/54098
Reviewed-by: Matthias Vogel <typo3@kanti.de>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Markus Hölzle <typo3@markus-hoelzle.de>
Tested-by: Markus Hölzle <typo3@markus-hoelzle.de>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
2 years ago[TASK] Raise typo3/testing-framework to 1.1.4 96/54096/2
Christian Kuhn [Sun, 10 Sep 2017 10:22:04 +0000 (12:22 +0200)]
[TASK] Raise typo3/testing-framework to 1.1.4

The raise brings an additional UnitTestsDeprecated.xml
setup file dedicated to test deprecated core functionality.

Change-Id: Ic61ce2d76ed7f4e83bc6c2a12dd1337392d6c4aa
Resolves: #82437
Related: #82438
Releases: master, 8.7
Reviewed-on: https://review.typo3.org/54096
Reviewed-by: Markus Hölzle <typo3@markus-hoelzle.de>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
2 years ago[!!!][TASK] Remove old typoscript constants editor defaults option 86/54086/3
Markus Hoelzle [Sat, 9 Sep 2017 21:02:14 +0000 (23:02 +0200)]
[!!!][TASK] Remove old typoscript constants editor defaults option

Remove support for the special constant comment
`###MOD_TS:EDITABLE_CONSTANTS###`. This constant was used to show
all constants before this comment as default constants in constants
editor and is dropped as ancient and widely unknown and rarely used
feature.

Releases: master
Resolves: #82425
Change-Id: I5345871818e1d627cacb8be6db2ab8629770237e
Reviewed-on: https://review.typo3.org/54086
Reviewed-by: Joerg Kummer <typo3@enobe.de>
Tested-by: Joerg Kummer <typo3@enobe.de>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Markus Hölzle <typo3@markus-hoelzle.de>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
2 years ago[TASK] Use general utility to build html attribute string 19/49819/16
Sascha Egerer [Sat, 3 Sep 2016 08:34:26 +0000 (10:34 +0200)]
[TASK] Use general utility to build html attribute string

There was one leftover place, where an attributes string was
concatenated instead using the helper function.

Resolves: #77824
Releases: master
Change-Id: I0d01a5bde47b333dc11d25f6dd28bde9e7810696
Reviewed-on: https://review.typo3.org/49819
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Joerg Kummer <typo3@enobe.de>
Tested-by: Joerg Kummer <typo3@enobe.de>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
2 years ago[TASK] Open the elementbrowser always in modal window 73/54073/12
Benjamin Kott [Sat, 9 Sep 2017 17:19:34 +0000 (19:19 +0200)]
[TASK] Open the elementbrowser always in modal window

Resolves: #82427
Releases: master
Change-Id: Ia0e5f34a0f8bb29a179f1e9f7b90bb1b540b57f7
Reviewed-on: https://review.typo3.org/54073
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: Tymoteusz Motylewski <t.motylewski@gmail.com>
Tested-by: Tymoteusz Motylewski <t.motylewski@gmail.com>
2 years ago[BUGFIX] Fix file upload and directory creation in file browser 94/54094/2
Andreas Fernandez [Sun, 10 Sep 2017 09:46:21 +0000 (11:46 +0200)]
[BUGFIX] Fix file upload and directory creation in file browser

While #81763 was in development, some locations of the file operation
handling were not adjusted. This patch fixes the left over places.

Resolves: #82436
Related: #81763
Releases: master
Change-Id: I450458f5ef687969b22a194a2387953330b8d17d
Reviewed-on: https://review.typo3.org/54094
Tested-by: Riccardo De Contardi <erredeco@gmail.com>
Reviewed-by: Benjamin Kott <benjamin.kott@outlook.com>
Tested-by: Benjamin Kott <benjamin.kott@outlook.com>
Reviewed-by: Tymoteusz Motylewski <t.motylewski@gmail.com>
Tested-by: Tymoteusz Motylewski <t.motylewski@gmail.com>
2 years ago[BUGFIX] JS: Init SelectTree after FormEngine 15/53815/3
Alexander Opitz [Sun, 27 Aug 2017 18:54:26 +0000 (20:54 +0200)]
[BUGFIX] JS: Init SelectTree after FormEngine

SelectTree depends on availability of TYPO3.FormEngine,
so add FormEngine as Dependency.

Resolves: #82217
Releases: master, 8.7
Change-Id: I2d1a0c8c77b1140f9c501ea105cf8a66a6cd6a97
Reviewed-on: https://review.typo3.org/53815
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
2 years ago[!!!][TASK] Remove unused configuration settings 77/54077/4
Christian Kuhn [Sat, 9 Sep 2017 18:07:03 +0000 (20:07 +0200)]
[!!!][TASK] Remove unused configuration settings

The patch drops some unused pre-doctrine DB related TYPO3_CONF_VARS.

Change-Id: I9204ef926b1de145f541e2c84dadc6a968e153b9
Resolves: #82421
Releases: master
Reviewed-on: https://review.typo3.org/54077
Reviewed-by: Jan Helke <typo3@helke.de>
Tested-by: Jan Helke <typo3@helke.de>
Reviewed-by: Joerg Kummer <typo3@enobe.de>
Tested-by: Joerg Kummer <typo3@enobe.de>
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>
Tested-by: TYPO3com <no-reply@typo3.com>
2 years ago[BUGFIX] Cache calls to SchemaManager()->listTableColumns() 91/53391/3
Stephan Großberndt [Tue, 4 Jul 2017 07:51:55 +0000 (09:51 +0200)]
[BUGFIX] Cache calls to SchemaManager()->listTableColumns()

Caching the calls SchemaManager()->listTableColumns() during runtime
leads to improved performance when doing operations on multiple files or
multiple extbase Domain/Model objects at once as the same information
does not have to be fetched over and over.

Releases: master, 8.7
Resolves: #81778
Change-Id: Ieebcf5046d007dfea40acf148045807053873ac8
Reviewed-on: https://review.typo3.org/53391
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Reviewed-by: Philipp Gampe <philipp.gampe@typo3.org>
Reviewed-by: Sebastian Fischer <typo3@evoweb.de>
Reviewed-by: Henning Liebe <h.liebe@neusta.de>
Tested-by: Philipp Gampe <philipp.gampe@typo3.org>
Reviewed-by: Joerg Boesche <typo3@joergboesche.de>
Reviewed-by: Steffen Frese <steffenf14@gmail.com>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
2 years ago[FEATURE] Add frontend debug output as HTTP Response Header 75/54075/4
Benni Mack [Sat, 9 Sep 2017 17:42:29 +0000 (19:42 +0200)]
[FEATURE] Add frontend debug output as HTTP Response Header

The current debug information in the frontend is added as HTML
comment, but breaks non-HTML pages (e.g. JSON output).

The debug information (= parse time of the frontend request) is now sent as HTTP
Response Header found at "X-TYPO3-Parsetime".

Resolves: #82419
Releases: master
Change-Id: Ibd84825fd3ebdcb1c88e0059d00d78b4bb20f53e
Reviewed-on: https://review.typo3.org/54075
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
Reviewed-by: Markus Hölzle <typo3@markus-hoelzle.de>
Tested-by: Markus Hölzle <typo3@markus-hoelzle.de>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
2 years ago[BUGFIX] T3editor: Get default mode if none is defined for a field 85/54085/3
Andreas Fernandez [Sat, 9 Sep 2017 20:33:42 +0000 (22:33 +0200)]
[BUGFIX] T3editor: Get default mode if none is defined for a field

If a field is rendered by t3editor but no mode was defined, the t3editor
falls back to the default mode, which is `html` by default.

Resolves: #82424
Related: #81901
Releases: master
Change-Id: I4b94e5257429219ccd72a917faa170fbc56c34ad
Reviewed-on: https://review.typo3.org/54085
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Reviewed-by: Benjamin Kott <benjamin.kott@outlook.com>
Tested-by: Benjamin Kott <benjamin.kott@outlook.com>
2 years ago[BUGFIX] Remove NOT NULL constraints from l1[08]n_diffsource fields 84/54084/3
Andreas Fernandez [Sat, 9 Sep 2017 20:01:35 +0000 (22:01 +0200)]
[BUGFIX] Remove NOT NULL constraints from l1[08]n_diffsource fields

Having NOT NULL collides with MySQL strict mode, so these constraints
get removed.

Resolves: #82423
Releases: master, 8.7
Change-Id: Ic7a71ee40b6ad276a47405e3fd81c501868abb76
Reviewed-on: https://review.typo3.org/54084
Reviewed-by: Markus Hölzle <typo3@markus-hoelzle.de>
Tested-by: Markus Hölzle <typo3@markus-hoelzle.de>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Benjamin Kott <benjamin.kott@outlook.com>
Tested-by: Benjamin Kott <benjamin.kott@outlook.com>
2 years ago[TASK] Replace `get_class($this)` calls with `static::class` 69/54069/2
Romain Canon [Sat, 9 Sep 2017 16:13:26 +0000 (18:13 +0200)]
[TASK] Replace `get_class($this)` calls with `static::class`

Improves runtime performance of these calls by approximately 1/3.

See https://belineperspectives.com/2017/03/13/get_classthis-vs-staticclass/

Resolves: #82416
Releases: master
Change-Id: I7a069068ed66cbb4ebd83bdad56c621166bf8139
Reviewed-on: https://review.typo3.org/54069
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Simon Praetorius <simon@praetorius.me>
Reviewed-by: Nathan Boiron <nathan.boiron@gmail.com>
Reviewed-by: Matthias Vogel <typo3@kanti.de>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
2 years ago[FOLLOWUP][TASK] Remove TYPO3.jQuery 74/54074/2
Benni Mack [Sat, 9 Sep 2017 17:32:38 +0000 (19:32 +0200)]
[FOLLOWUP][TASK] Remove TYPO3.jQuery

This followup ensures that the Frontend Request including jQuery behaves
the same as the TYPO3 Backend (noConflict is default).

Related: #79221
Resolves: #82378
Releases: master
Change-Id: Ic16fcfb6932cc32d4190e715708c58f234dde5d7
Reviewed-on: https://review.typo3.org/54074
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: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
2 years ago[TASK] Introduce Source Sans Pro and drop Share Font 63/54063/3
Benjamin Kott [Sat, 9 Sep 2017 13:02:51 +0000 (15:02 +0200)]
[TASK] Introduce Source Sans Pro and drop Share Font

To improve readability in the backend, Source Sans Pro typeface is now
introduced and replaces the Share Font.

Resolves: #82410
Releases: master
Change-Id: Id9a71b37bb41ff0e0b7fb514e62aebd380ff3d2b
Reviewed-on: https://review.typo3.org/54063
Tested-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Tested-by: Joerg Kummer <typo3@enobe.de>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Andreas Fernandez <typo3@scripting-base.de>
Reviewed-by: Jan Helke <typo3@helke.de>
Tested-by: Jan Helke <typo3@helke.de>
Reviewed-by: Nathan Boiron <nathan.boiron@gmail.com>
Tested-by: Nathan Boiron <nathan.boiron@gmail.com>
Tested-by: Patricia Tiedemann <pixelaeffchen@googlemail.com>
Reviewed-by: Simon Praetorius <simon@praetorius.me>
Tested-by: Simon Praetorius <simon@praetorius.me>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
2 years ago[TASK] Add descriptions to dropdowns in Install Tool - All Configuration 72/54072/3
Benni Mack [Sat, 9 Sep 2017 16:54:10 +0000 (18:54 +0200)]
[TASK] Add descriptions to dropdowns in Install Tool - All Configuration

Resolves: #82418
Releases: master
Change-Id: Ic5a59c485df3ce9aaaad0bd51206c0d53f445690
Reviewed-on: https://review.typo3.org/54072
Reviewed-by: Joerg Kummer <typo3@enobe.de>
Tested-by: Joerg Kummer <typo3@enobe.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] Migrate modules to regular backend routing 81/53881/13
Matthias Vogel [Sat, 9 Sep 2017 11:04:20 +0000 (13:04 +0200)]
[!!!][TASK] Migrate modules to regular backend routing

This patch removes the separate request handler for backend modules,
which was accessed via "&M=moduleName" GET parameter. This is now
migrated into the RouteDispatcher which can dispatch modules as well.

Now, modules are called via the "&route" parameter like all other routes.

Additionally, the requested URLs for modules were requested with the additional
"moduleToken" which is now called "token".

This way, special treatment for modules when dispatching is removed,
however the security checks are still in place so this is kept as is.

All places where URLs are generated can now still be accessed via
`BackendUtility::getModuleUrl()` which can deal with routes, module names
and routePaths (from the URL) to keep backwards-compatibility.

Next Steps:
- Migration wizard for bookmarks + Streamline bookmarks code (see todos)
- Check what needs to be added in ExtensionManagementUtility
- Introduce slugs in routes for BE, e.g. /file-edit/{fileId}/ and /module/page/view/{id}
- Document reserved GET parameters "id", "route" and "token"
- Cleanup usage of determineScriptId and getModuleUrl to use new API

Resolves: #82406
Releases: master
Change-Id: If11c3d5289e14bc9ea766468b8e94cce95c23c71
Reviewed-on: https://review.typo3.org/53881
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Tested-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Reviewed-by: Matthias Vogel <typo3@kanti.de>
Tested-by: Matthias Vogel <typo3@kanti.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[BUGFIX] Use correct values in settings InstallTool 49/54049/4
Simon Praetorius [Sat, 9 Sep 2017 08:54:32 +0000 (10:54 +0200)]
[BUGFIX] Use correct values in settings InstallTool

Currently, the InstallTool will use the global TYPO3_CONF_VARS for the
settings form. In backend context, this contains merged TypoScript,
which shouldn't be added to LocalConfiguration.php.

Resolves: #82399
Releases: master
Change-Id: I0aabbde5c8fb8574fe30422fa687849271a3d618
Reviewed-on: https://review.typo3.org/54049
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
2 years ago[FEATURE] Support defer attribute for JavaScript includes in TypoScript 53/54053/3
Simon Praetorius [Sat, 9 Sep 2017 10:13:36 +0000 (12:13 +0200)]
[FEATURE] Support defer attribute for JavaScript includes in TypoScript

When including JavaScript files in TypoScript, the HTML5 attribute
`defer` is now supported.

Resolves: #80542
Releases: master
Change-Id: I81ff3968360585692ec586449f64856dc0541f93
Reviewed-on: https://review.typo3.org/54053
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Tested-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
2 years ago[TASK] Distinguish install tool actions with button colors 55/54055/2
Simon Praetorius [Sat, 9 Sep 2017 11:08:35 +0000 (13:08 +0200)]
[TASK] Distinguish install tool actions with button colors

Resolves: #82405
Releases: master
Change-Id: Id28b655b8ebf4853e960b3dd97414d31d42ff6de
Reviewed-on: https://review.typo3.org/54055
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Joerg Kummer <typo3@enobe.de>
Tested-by: Joerg Kummer <typo3@enobe.de>
Reviewed-by: Patricia Tiedemann <pixelaeffchen@googlemail.com>
Tested-by: Patricia Tiedemann <pixelaeffchen@googlemail.com>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
2 years ago[!!!][TASK] Remove old typoscript constants "TSConstantEditor" 48/54048/6
Markus Hoelzle [Fri, 8 Sep 2017 22:38:12 +0000 (00:38 +0200)]
[!!!][TASK] Remove old typoscript constants "TSConstantEditor"

Remove support for the special constant "TSConstantEditor".
This constant was used to show additional helping information
in the constants editor and is dropped as ancient and widely
unknown and rarely used feature.

Releases: master
Resolves: #82398
Change-Id: I9825ca5a321c6c92159a5cd94ca5ed6c9ea1d0ff
Reviewed-on: https://review.typo3.org/54048
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
Reviewed-by: Markus Hölzle <typo3@markus-hoelzle.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[BUGFIX] Perform rollback only once if undeleting CE with History/Undo 73/53973/6
Lukasz Uznanski [Thu, 7 Sep 2017 19:19:15 +0000 (21:19 +0200)]
[BUGFIX] Perform rollback only once if undeleting CE with History/Undo

Releases: master, 8.7
Resolves: #81649
Change-Id: I1cb9a681b46327af56c2cb080c92b3cc273b0e10
Reviewed-on: https://review.typo3.org/53973
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Joerg Kummer <typo3@enobe.de>
Tested-by: Joerg Kummer <typo3@enobe.de>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
2 years ago[FOLLOWUP][TASK] Render backend icon list by existing files 61/54061/6
Patricia Tiedemann [Sat, 9 Sep 2017 12:19:39 +0000 (14:19 +0200)]
[FOLLOWUP][TASK] Render backend icon list by existing files

* Re-added the "spinning" option for spinners (thx Nathan)
* Changed cache from backend_core to asset (thx @helhum)

Resolves: #82348
Releases: master
Change-Id: I1e59da79dc7072b582866213a571e35bfee4b639
Reviewed-on: https://review.typo3.org/54061
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Tested-by: Stefan Neufeind <typo3.neufeind@speedpartner.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] Render backend icon list by existing files 20/54020/13
Patricia Tiedemann [Fri, 8 Sep 2017 12:50:39 +0000 (14:50 +0200)]
[TASK] Render backend icon list by existing files

Until now all of the backend icons were a manually edited array of file
paths. With this patch defined icon folders in the core are scanned
for existing icon files and build most of that static array
automatically.

For historic reason (and hopefully soon to be removed alltogether) a
second array of statically declared icons is added and merged into.

Some icons that were in the original static list but whose files
actually are missing or misnamed are marked as deprecated.

Resolves: #82348
Releases: master
Change-Id: I8dc4fbbe815d923eda55e1c1b072369e62f66d87
Reviewed-on: https://review.typo3.org/54020
Reviewed-by: Benjamin Kott <benjamin.kott@outlook.com>
Tested-by: Benjamin Kott <benjamin.kott@outlook.com>
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Andreas Fernandez <typo3@scripting-base.de>
2 years ago[BUGFIX] Make Extbase pagination work with QueryBuilder queries 09/53409/12
Tymoteusz Motylewski [Mon, 14 Aug 2017 14:27:22 +0000 (16:27 +0200)]
[BUGFIX] Make Extbase pagination work with QueryBuilder queries

Now you can pass a query built by QueryBuilder to $query->statement(),
and use the QueryResult for pagination.

So inside repository you can create custom query like:

$queryBuilder->select('*')->...
$query = $this->createQuery();
$query->statement($queryBuilder);
return $query->execute();

and pass the result to paginate ViewHelper.

Resolves: #81887
Releases: master, 8.7
Change-Id: Ia00f673039362afaa8ef7232ba4bc89c520d5a46
Reviewed-on: https://review.typo3.org/53409
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Joerg Kummer <typo3@enobe.de>
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
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] Fix using contains in query on property of group 93/53993/8
Łukasz Uznański [Fri, 8 Sep 2017 09:29:36 +0000 (11:29 +0200)]
[BUGFIX] Fix using contains in query on property of group

Related: master, 8.7
Resolves: #80215
Change-Id: I6f5c88650d6167550a46ad2200c846ec9d11d34a
Reviewed-on: https://review.typo3.org/53993
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Joerg Kummer <typo3@enobe.de>
Tested-by: Joerg Kummer <typo3@enobe.de>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
2 years ago[TASK] Migrate cHash configuration on silent upgrade wizard 02/53202/11
Benni Mack [Tue, 13 Jun 2017 15:19:06 +0000 (17:19 +0200)]
[TASK] Migrate cHash configuration on silent upgrade wizard

The cHash calculation is done on every TYPO3 request, which is
really unnecessary overhead, as this could be taken care by the
SilentConfigurationUpgradeService once and store the new value.

Resolves: #81568
Releases: master
Change-Id: I30b562c0b9376d1044979cd7264c67a79209f6b6
Reviewed-on: https://review.typo3.org/53202
Reviewed-by: Nathan Boiron <nathan.boiron@gmail.com>
Tested-by: Nathan Boiron <nathan.boiron@gmail.com>
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] Update localization rendering tests to newest Testing Framework 51/54051/2
Tymoteusz Motylewski [Sat, 9 Sep 2017 10:20:18 +0000 (12:20 +0200)]
[TASK] Update localization rendering tests to newest Testing Framework

addTypoScriptToTemplateRecord method was added to FunctionalTestCase.php
in Testing Framework.
So we can remove it from test cases.
Fix two other germlins also (unneeded comment and unused variable).

Resolves: #82401
Releases: master, 8.7, 7.6
Change-Id: I8144bae5a807d60123ef5228a44eba579a7705c7
Reviewed-on: https://review.typo3.org/54051
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Tested-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Reviewed-by: Tymoteusz Motylewski <t.motylewski@gmail.com>
Tested-by: Tymoteusz Motylewski <t.motylewski@gmail.com>