Some systems allow to connect with empty username / password
configuration to the database. Although this is not wrong the install
tool should always ask for database credentials for new installations.
This patch prevents the install tool from setting an empty username and
password as default values without manually submitting the form.

Resolves: #51540
Resolves: #51956
Releases: 6.2
Change-Id: Ic245635bf649f4ad477d793eda658096ed659aa0
Reviewed-on: https://review.typo3.org/23866
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Reviewed-by: Markus Klein
Tested-by: Markus Klein

Change-Id: I6f6c8a2ddc26a6c7584e6a067c887768cf3ae5b1
Resolves: #52316
Related: #52305
Releases: 6.2, 6.1, 6.0
Reviewed-on: https://review.typo3.org/24084
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn

The extbase reflection and object caches are "stable" cache entries
and only need to be deleted if extensions change or are updates. The
extension manager takes care of flushing in this case.
Currently, the default lifetime of those cache entries is only one
hour, so they are recalculated pretty often. This is especially a
problem if the site was not accessed for more than an hour, since
tons of cache entries exceeded their lifetime and require
recalculation.
The patch moves the cache configuration of the required extbase
extension to DefaultConfiguration and sets unlimited default
lifetime for object and reflection cache.

Change-Id: I671692645aeb104f7fa73595cc79b25ed5984b85
Resolves: #52305
Related: #51116
Related: #52304
Releases: 6.2, 6.1, 6.0
Reviewed-on: https://review.typo3.org/24069
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn

The language cache by default uses the FileBackend to store its data.
Language cache entries need to be deleted only if new extensions are
loaded and if new language overlays are fetched. They do not need
tagging and can have an unlimited lifetime.
Switching to SimpleFileBackend removes the tagging and sets unlimited
lifetime by to reduce read and write load on this cache.

Change-Id: I5c4778f4c38ae369b6873574e961fa65208d77a1
Resolves: #52295
Related: #52125
Releases: 6.2, 6.1, 6.0
Reviewed-on: https://review.typo3.org/24064
Reviewed-by: Wouter Wolters
Reviewed-by: Xavier Perseguers
Tested-by: Xavier Perseguers
Reviewed-by: Thomas Maroschik
Tested-by: Thomas Maroschik
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn

On first installation only existing empty databases can be selected.
The hint for "Existing tables will be removed" is obsolete.

Change-Id: I05c0fb460672b8b4bbffe321945c46d2c7be2b2f
Resolves: #52314
Releases: 6.2
Reviewed-on: https://review.typo3.org/24078
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn

The MessageDisplay class that is used to display a hint in the extension
settings of the dbal extension was removed with commit
2e3c9d5e.

This commit re-adds the class to prevent a PHP error.

Resolves: #52167
Releases: 6.2
Change-Id: I1fa8749ffb93eab45646948ff336bf3d331aa92c
Reviewed-on: https://review.typo3.org/23949
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn

Change-Id: I856db58b5843615ddae80e5f978de79981027711
Resolves: #52270
Releases: 6.2
Reviewed-on: https://review.typo3.org/24056
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn

docs.typo3.org is new official documentation platform supporting both
OpenOffice and Sphinx manuals.

Fixes: #52226
Releases: 6.2, 6.1, 6.0
Change-Id: Ie5f27bf15f5174a6e1bb53bedd575ceeb362add8
Reviewed-on: https://review.typo3.org/24017
Reviewed-by: Georg Ringer
Tested-by: Georg Ringer
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Reviewed-by: Xavier Perseguers
Tested-by: Xavier Perseguers

Exporting arrays for caches is now done with var_export().

Resolves: #51116
Releases: 6.2, 6.1, 6.0
Change-Id: I7519fd9c2ab21cc22a3f4e092b611c6bca58c30f
Reviewed-on: https://review.typo3.org/23969
Reviewed-by: Xavier Perseguers
Tested-by: Xavier Perseguers
Reviewed-by: Helmut Hummel
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn

Exception code "1314354065" is used twice: once in the abstract
Resource repository and the other one in the File Collection repository.
Since exception code must be unique, the patch simply renames one
occurrence.

Change-Id: I41474db5de7e8f575275d57e9585252857334ee2
Releases: 6.0, 6.1, 6.2
Resolves: #52243
Reviewed-on: https://review.typo3.org/24038
Reviewed-by: Kilian Hann
Tested-by: Kilian Hann
Reviewed-by: Xavier Perseguers
Tested-by: Xavier Perseguers

There are two very weird settings is GFX array that previously
decided on how "masks" are applied to images if combining
multiple images. GFX/im_negate_mask and GFX/im_imvMaskState are
also manipulated during boostrap and depend on each other.

Fiddling with those settings in ImageMagick 6 and GraphicsMagick
with its supported versions, it turns out that they are obsolete
and can be removed completly. The patch does this.

Change-Id: Ic24c877287382f9e718974a0df3577dad2eeaac9
Resolves: #52088
Releases: 6.2
Reviewed-on: https://review.typo3.org/23880
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn

The new property mapper is enabled by default now.
Due to this the hint inside the exception is misleading.
It hints now to the new @ignorevalidation annotation.

The old one annotation will stay.

Change-Id: I6a7f213f38a72a3b7676af5b1bf07f879fb99253
Resolves: #52198
Releases: 6.2
Reviewed-on: https://review.typo3.org/23999
Reviewed-by: Georg Ringer
Tested-by: Georg Ringer

Change-Id: Id1ecce2b4fe789b655f63cd8b5f78ab20eadfbd5
Resolves: #52201
Related: #44825
Releases: 6.2, 6.1, 6.0
Reviewed-on: https://review.typo3.org/23998
Reviewed-by: Andy Grunwald
Reviewed-by: Oliver Hader
Reviewed-by: Nicole Cordes
Tested-by: Nicole Cordes
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring

When ext:sys_action is installed it will give a warning about
not valid HTML. This is because the id="" and class="" are glued
to each other. Fix this by moving the space from the front to the
back.

Change-Id: Ia916a9e084cb9f3595164215cf22a100d64dc7a9
Resolves: #52202
Releases: 6.2
Reviewed-on: https://review.typo3.org/24001
Reviewed-by: Georg Ringer
Tested-by: Georg Ringer

The change I7770044c11a6c1a1bcb51a3bfc11b85923653196 added meta
information for the FAL IRRE headers.
It uses a definition list to show certain fields of the related record
inside the title (thus visible if collapsed).
The HTML markup leads to problems in other places of the backend, most
noteable in the workspace module.

Introduce a separate user function only for inline labels,
using the same signature as the normal label_userFunc.

Resolves: #51411
Releases: 6.2
Change-Id: I070a9db3d009c743cb927be90a0aa3903a1bae45
Reviewed-on: https://review.typo3.org/23528
Reviewed-by: Markus Klein
Tested-by: Markus Klein
Reviewed-by: Georg Ringer
Tested-by: Georg Ringer

Fixed workspace filters in case of workspace preview
in TYPO3\CMS\Frontend\Page\PageRepository::enableFields().

Additionally cleared up the corresponding comments.

Change-Id: I088928a88cb673f18f218ef691a6c528019317c0
Fixes: #37065
Releases: 6.2, 6.1, 6.0, 4.5
Reviewed-on: https://review.typo3.org/11127
Reviewed-by: Sascha Egerer
Tested-by: Sascha Egerer
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind

Currently the persistence settings for all tx_beuser models are defined
in module.tx_beuser.persistence.classes.
If any other module tries to access the corresponding backend user
repository from tx_beuser, then no mappings will be loaded by extbase,
resulting in a wrong cache entry for the model and the attempt to
access a nonexisting table.

Move the class mapping part of the TS template into
  config.tx_extbase.persistence.classes
to make them available for all extensions and modules.

Resolves: #51234
Releases: 6.2, 6.1, 6.0
Change-Id: I0b39c96c9db89c379e5613231a215f27b2dfbcde
Reviewed-on: https://review.typo3.org/23967
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind

Currently a leading space will be returned by the ATagParams handler when
the $addGlobal parameter is TRUE.

This patch trims any space if no ATagParams have been configured.

Resolves: #52171
Releases: 6.2
Change-Id: I53b3723e530beb96ff0dc9606963c6d9928d4478
Reviewed-on: https://review.typo3.org/23954
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind

AbstractUserAuthentication::getAuthInfoArray() calls
$GLOBALS['TYPO3_DB']->cleanIntList() with a possible NULL argument.
This leads to a PHP warning in GeneralUtility::trimExplode().

Resolves: #50913
Relates: #42921
Releases: 6.2, 6.1, 6.0, 4.7, 4.5
Change-Id: I42ddf8fed715c5f8ce060e6ca5826ef3ed8f223e
Reviewed-on: https://review.typo3.org/22956
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind

This patch fixes the handleRequest() method signature of
the Cli request handler by returning the response object
like it is expected by the RequestHandlerInterface.

When the request is returned the Bootstrap process will
reset the singletons which includes persisting of any
changed objects.

Resolves: #52092
Releases: 6.2
Change-Id: I78a507ab5b68ad67a85bf4a914494b39ea1407e1
Reviewed-on: https://review.typo3.org/23885
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Stefan Neufeind
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring

If there is a local storage "uploads/test/" and a requested file
is in "uploads/other/file.txt", the storage matching will not
consider this storage for this file anymore, although they share a
common prefix, but will use a correct matching storage (e.g. 0).

Releases: 6.2, 6.1, 6.0
Resolves: #52173
Change-Id: Id14ffd6620554c086b6e77dca285ade8dee39ab2
Reviewed-on: https://review.typo3.org/23966
Reviewed-by: Markus Klein
Tested-by: Markus Klein
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind

INT scripts placed in page.headerData.XX were
not replaced any more after introducing USER_INT
capabilities of the pageRenderer in #22273

This change reprocesses _INT placeholders after
the pageRenderer replacements have been done.

Resolves: #44825
Releases: 6.0, 6.1, 6.2
Change-Id: If11b692ed75120dbf122d95ca497b5e382fdf349
Reviewed-on: https://review.typo3.org/23788
Reviewed-by: Markus Klein
Tested-by: Markus Klein
Reviewed-by: Xavier Perseguers
Tested-by: Xavier Perseguers
Reviewed-by: Wouter Wolters
Reviewed-by: Jens Jacobsen
Tested-by: Jens Jacobsen
Reviewed-by: Helmut Hummel
Tested-by: Helmut Hummel

Improve usability of viewpage module with:

* Added a default option with width and height set to auto
* Selection is saved in $GLOBALS['BE_USER']->uc using ExtDirect
StateProvider and restored after reload or page switch
* Frame is resizeable and this custom value is also stored in the user
settings as a new option of the selector
* iframe is visually separated from outer module body

Change-Id: I0f956b3b3b76ef5fea25c239acd8f95d4f93b765
Fixes: #51933
Releases: 6.2
Reviewed-on: https://review.typo3.org/23771
Tested-by: Philipp Gampe
Reviewed-by: Mattias Nilsson
Tested-by: Mattias Nilsson
Tested-by: Markus Klein
Reviewed-by: Georg Ringer
Tested-by: Georg Ringer

sys_category until now misses a specific sort order
and is sorted by title. Adding a sorting property
gives more flexibility in sorting by custom
needs.

tca treeConfig of sys_category as well as categoryRegistry
are adapted accordingly

Change-Id: I663fb24dcd5486f911e256fe1a42c445a63543be
Resolves: #51590
Releases: 6.2
Reviewed-on: https://review.typo3.org/23615
Reviewed-by: Francois Suter
Tested-by: Francois Suter
Reviewed-by: Georg Ringer
Tested-by: Georg Ringer

Sometimes the identifier is longer than 200 chars. This patch increases
the maximum length to 512 characters.

Resolves: #48912
Releases: 6.2, 6.1, 6.0
Change-Id: I67edc1facf972d2ffef09a3204edc556464afda5
Reviewed-on: https://review.typo3.org/23882
Reviewed-by: Markus Klein
Tested-by: Markus Klein
Reviewed-by: Aske Ertmann
Tested-by: Aske Ertmann
Reviewed-by: Jon Langeland
Tested-by: Jon Langeland
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn

When a user tries to rename a folder without the permissions
to do so, the ResourceStorage throws a
"InsufficientFileAccessPermissionsException" instead of a
"InsufficientUserPermissionsException".

Resolves: #52056
Releases: 6.0, 6.1, 6.2
Change-Id: I83e75038bd6763f201ef03ebf37018368e990e1a
Reviewed-on: https://review.typo3.org/23862
Reviewed-by: Helmut Hummel
Reviewed-by: Frans Saris
Tested-by: Frans Saris
Reviewed-by: Fabien Udriot
Reviewed-by: Francois Suter
Tested-by: Francois Suter

Related: #50138
Releases: 6.2
Change-Id: I98bf8a4eadfae7183d8904956b3adcfdface1e5b
Reviewed-on: https://review.typo3.org/23918
Reviewed-by: Xavier Perseguers
Tested-by: Xavier Perseguers
Reviewed-by: Georg Ringer
Tested-by: Georg Ringer

Do not show forgot password form in felogin, if the forgot
password link for the login form has been disabled.

Resolves: #50138
Resolves: #34597
Releases: 6.2
Change-Id: I40b490cb09dc339067ff4ff0bc0d0afe0464a456
Reviewed-on: https://review.typo3.org/22541
Reviewed-by: Markus Klein
Tested-by: Markus Klein
Reviewed-by: Marco Huber
Tested-by: Marco Huber
Reviewed-by: Georg Ringer
Tested-by: Georg Ringer

The writelog method for the BackendUserAuthentication object is now
checking if there is actually an uid available in the user array
before it tries to access it.

This prevents a PHP warning about an illegal string offset when writing
log messages in an authentication service.

Releases: 6.0, 6.1, 6.2
Resolves: #49328
Change-Id: I006c6b0ccd8f03292715a7d7ab8e457914637d93
Reviewed-on: https://review.typo3.org/23116
Reviewed-by: Markus Klein
Tested-by: Markus Klein
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn

This change adds a case to treat "inline" TCA types the same way
"select" is treated when building the record's label value.

Before, if record used field of type "inline" as TCA label field, TYPO3
would display fx "3" (number of related records as stored in field
on parent record).

After, TYPO3 will display fx "Record1, Record2, Record2" if "inline"
field contains three related records named thusly.

Fixes: #52133
Releases: 6.2, 6.1, 6.0, 4.7, 4.5
Change-Id: Ie06f09368e81505cb1e5989b61ae98add54b05ba
Reviewed-on: https://review.typo3.org/23902
Reviewed-by: Benjamin Rau
Tested-by: Benjamin Rau
Reviewed-by: Stefan Froemken
Tested-by: Stefan Froemken
Reviewed-by: Cedric Ziel
Tested-by: Cedric Ziel
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring

Rendering is missing in css_styled_content for menu types
8 and 9 (introduced in 6.0 and 6.2 respectively) for older
versions of the static templates (i.e. below 6.0).

Add them down to 4.5.

Resolves: #51837
Releases: 6.2
Change-Id: I5f6f9068ceb89851c67b3c1ee2bd580936498c70
Reviewed-on: https://review.typo3.org/23674
Reviewed-by: Georg Ringer
Tested-by: Georg Ringer

On using the element browser in combination with files, the
expand/collapse behavior is wrong. On opening a collapsed folder,
an AJAX call fetches the sub-nodes that will be added to the DOM
tree. However, the calling scope of the original rendering
process and the AJAX call differ a lot.
The only possibility to get this working is to transfer the
scope and rebuild it in the AJAX context.

Change-Id: I1bf7b56ae2d82d15e0246bd651f77793adde705b
Fixes: #45859
Releases: 6.0, 6.1, 6.2
Reviewed-on: https://review.typo3.org/22934
Reviewed-by: Henrik Ziegenhain
Tested-by: Henrik Ziegenhain
Reviewed-by: Markus Schubath
Tested-by: Markus Schubath
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring

Resolves: #50907
Releases: 6.2, 6.1, 6.0
Change-Id: I232498fea18b270e9436aeaf710f728a62365a4b
Reviewed-on: https://review.typo3.org/23874
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters

The usage of the ! operator to force cropping empty image
canvas is inverted between ImageMagick v4 and v6 while
GraphicsMagick accepts both syntaxes. Because im4 is
deprecated, we can always use im6 syntax.

Fixes: #19045
Releases: 6.2, 6.1, 6.0
Change-Id: Iee1260a94f2686a512ce1b1df65c384319a4c4de
Reviewed-on: https://review.typo3.org/13996
Reviewed-by: Markus Klein
Tested-by: Markus Klein
Reviewed-by: Felix Buenemann
Tested-by: Felix Buenemann
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind

It is only possible to do one makeCategorizable() call per
table, because if you do more the later calls would overwrite the
previous category field declaration. That's because of the way
the registry stores the category fields - it can only store one
field per table. This change improves that and allow multiple
fields pers table.

Additionally, a custom label can be set in the options array
for each category field.

Resolves: #51921
Releases: 6.2
Change-Id: I96d1235176109395339e9841564edadc0c71147d
Reviewed-on: https://review.typo3.org/23772
Reviewed-by: Helmut Hummel
Tested-by: Helmut Hummel
Reviewed-by: Markus Klein
Tested-by: Markus Klein

im_mask_temp_ext_noloss setting is a bugfix for a no longer supported
Image Magick 5 version. It can be removed.

Change-Id: I1001c3fc8fa924ec16e24a9e790a0decf2771bcc
Resolves: #52012
Releases: 6.2
Reviewed-on: https://review.typo3.org/23837
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters

Image Magick support for versions 4 and 5 was dropped. The setting
to differentiate combine from composite is obsolete now and composite
is used always.

Change-Id: I0fbc847167d340fd83315285745c63d53225891a
Resolves: #52011
Releases: 6.2
Reviewed-on: https://review.typo3.org/23836
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn

Did you know there is an option to disable condition parsing in
TS config? It is removed with the patch.

Change-Id: If6ce720a4416af80b7deb293160e8b712c04ff0d
Resolves: #52013
Releases: 6.2
Reviewed-on: https://review.typo3.org/23838
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn

Support for Image Magick version 4 and 5 was dropped, allowed
settings for im_version_5 are "im6" and "gm". As soon as one
of this is set for im_version_5, bootstrap automatically sets
"im_no_effects" to 1. In effect, this value is *always* one, so
it is obsolete and can be removed from the system.

Change-Id: Ieb798352aebaae4943018a225518591a952a8bcb
Resolves: #52010
Releases: 6.2
Reviewed-on: https://review.typo3.org/23835
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters

Image Magick versions 6.7 and above need sRGB as colorspace,
otherwise images are too dark. This also works for older versions
like 6.6. Use sRGB as default is image configuration presets.

Change-Id: I7c50ee932b1335892eb228b10fc00136cfc54527
Resolves: #52017
Releases: 6.2
Reviewed-on: https://review.typo3.org/23840
Reviewed-by: Steffen Müller
Tested-by: Steffen Müller
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters