Packages/TYPO3.CMS.git
6 years ago[TASK] dbal: Remove obsolete CSS HTML template 28/23428/3
Stefan Rotsch [Wed, 28 Aug 2013 15:51:30 +0000 (17:51 +0200)]
[TASK] dbal: Remove obsolete CSS HTML template

The HTML template was used in the old install tool, is
unused now and can be removed.

Change-Id: I046c772acd2a9db166036216b3d8ed78ffe813e4
Resolves: #51469
Releases: 6.2
Reviewed-on: https://review.typo3.org/23428
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
6 years ago[FEATURE] Enable automatic t3d import for extensions 97/23397/3
Susanne Moog [Wed, 28 Aug 2013 10:36:45 +0000 (12:36 +0200)]
[FEATURE] Enable automatic t3d import for extensions

As preparation for the new distribution (introduction package ...)
handling extensions should be able to provide initial t3d imports.
These are then imported on the root level upon installing the extension.

The t3d file has to be called data.t3d and located in
<extension>/Initialisation/data.t3d.

The data is imported only once (even if you reinstall the extension),
execution state is saved via the core registry.

Resolves: #51437
Releases: 6.2
Change-Id: Ibfe631d173b962781018c6c9e929f4f020e48aa5
Reviewed-on: https://review.typo3.org/23397
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
6 years ago[FEATURE] Allow signalSlots to modify arguments 46/23146/8
Felix Oertel [Sat, 17 Aug 2013 08:36:38 +0000 (10:36 +0200)]
[FEATURE] Allow signalSlots to modify arguments

To allow signalSlots to modify the given arguments,
even if not an object (which could be modified by
reference anyway), the signalSlot should be able
to return the arguments.

This will make the signal slot handling diverge
from Flow and violates or at least bends the
signal slot pattern.

To make this clear, we should rename the
signal slot dispatcher in an upcoming change.

Resolves: #51138
Releases: 6.2
Change-Id: I36928adceed672580c76a68891dfb825e43a5a9e
Reviewed-on: https://review.typo3.org/23146
Reviewed-by: Stefan Neufeind
Reviewed-by: Helmut Hummel
Tested-by: Helmut Hummel
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
6 years ago[TASK] sys_file/sys_file_reference override strategy 31/21931/9
Andreas Wolf [Fri, 5 Jul 2013 13:15:56 +0000 (15:15 +0200)]
[TASK] sys_file/sys_file_reference override strategy

Until now, the sys_file_reference records use the regular "null value"
checkbox to switch between the value from sys_file and a custom value
the user can supply. This is however confusing for users, as there is no
explanation what the checkbox really does and also the placeholder text
is not removed when checking the checkbox (but an empty string is stored
in the database and thus used for output afterwards).

This commit changes the appearance to have a checkbox on top and toggle
between a classical read-only field and a real input field.

Change-Id: I2370b17c2cdf475aa6742cbca7cd12b6c8558130
Resolves: #49685
Releases: 6.2, 6.1, 6.0
Reviewed-on: https://review.typo3.org/21931
Reviewed-by: Steffen Ritter
Tested-by: Steffen Ritter
6 years ago[FEATURE] Introduce handling of missing files 13/22913/8
Frans Saris [Wed, 7 Aug 2013 09:51:34 +0000 (11:51 +0200)]
[FEATURE] Introduce handling of missing files

To keep track of missing files we introduce a missing
flag is sys_files.
When indexer detects file is missing the flag is set
and reset when file is present again.

In backend a message is shown when a reference to a
missing file is used.

Resolves: #50827
Change-Id: Iaaf9455698507c5cd574f6928fa5a563a49ad121
Reviewed-on: https://review.typo3.org/22913
Reviewed-by: Steffen Ritter
Reviewed-by: Kai Ole Hartwig
Tested-by: Kai Ole Hartwig
Reviewed-by: Mattias Nilsson
Tested-by: Mattias Nilsson
Tested-by: Steffen Ritter
6 years ago[TASK] Functional tests: error if DB permissions are missing 88/23388/3
Sascha Egerer [Tue, 27 Aug 2013 19:55:31 +0000 (21:55 +0200)]
[TASK] Functional tests: error if DB permissions are missing

The functional test framework needs create and drop
table database permissions.

Added a check that outputs an error message if the DB user
is not able to connect to the database or if the user
has not the required permissions to create a database.

Resolves: #51381
Releases: 6.2
Change-Id: I94c4eeb4356fdba06026ec727283807112c53926
Reviewed-on: https://review.typo3.org/23388
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
6 years ago[TASK] Cleanup saltedpasswords extension configuration 95/23395/2
Nicole Cordes [Wed, 28 Aug 2013 09:49:33 +0000 (11:49 +0200)]
[TASK] Cleanup saltedpasswords extension configuration

Remove the note "Install tool has hardcoded md5 hashing,
enabling this setting will prevent use of a install-tool-created
BE user." for backend.BE.forceSalted

Resolves: #51355
Releases: 6.2
Change-Id: Ic6dd03d37f5c4fe1027dece048693c74ab4dc98e
Reviewed-on: https://review.typo3.org/23395
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Reviewed-by: Kai Ole Hartwig
Tested-by: Kai Ole Hartwig
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
6 years ago[FEATURE] Use salted Install Tool password 39/22739/4
Nicole Cordes [Wed, 31 Jul 2013 22:18:45 +0000 (00:18 +0200)]
[FEATURE] Use salted Install Tool password

To enhanced the security this patch changes the Install Tool password
from md5 hash to a salted hashed password. Therefore the default
password in the FactoryConfiguration.php is changed. Old md5 hashes get
converted automatically during the boot process of the Install Tool. The
output of the calculated hash is reintroduced when an error occured.
The report modules were adjusted to be able to check salted hashed
passwords.

Resolves: #50613
Releases: 6.2
Change-Id: If02a43780c9c819ebd6da7cbf0acad305f805330
Reviewed-on: https://review.typo3.org/22739
Reviewed-by: Kai Ole Hartwig
Tested-by: Kai Ole Hartwig
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
6 years ago[BUGFIX] Only log file/directory actions which were done 03/23303/3
Helmut Hummel [Sun, 25 Aug 2013 20:01:16 +0000 (22:01 +0200)]
[BUGFIX] Only log file/directory actions which were done

Some file/directory actions were logged as they were
successfully performed, even when they failed.

Resolves: #51328
Releases: 6.2, 6.1, 6.0

Change-Id: Iab03e668d3f7d0e8feec09851ea50a59ab4b71a0
Reviewed-on: https://review.typo3.org/23303
Reviewed-by: Steffen Müller
Tested-by: Steffen Müller
Reviewed-by: Georg Ringer
Tested-by: Georg Ringer
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
6 years ago[FEATURE] Create BE-user in installation wizard with a salted password 38/22738/7
Nicole Cordes [Wed, 31 Jul 2013 21:55:33 +0000 (23:55 +0200)]
[FEATURE] Create BE-user in installation wizard with a salted password

In the installation wizard (123 steps) the admin user is created with a
md5 hashed password. For security reason this should be changed to a
salted password. As this function is used in #31407 as well it was
refactored to the AbstractAction class.

Resolves: #50612
Releases: 6.2
Change-Id: Ide966eb1c1e05d2190f0d22446bc2b3e92643451
Reviewed-on: https://review.typo3.org/22738
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Reviewed-by: Stefan Neufeind
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
6 years ago[TASK] Display count of failed error checks in install tool menu 80/23380/3
Stefan Rotsch [Tue, 27 Aug 2013 19:12:46 +0000 (21:12 +0200)]
[TASK] Display count of failed error checks in install tool menu

With displaying the count of failed error checks in the navigation menu
(like the count of unread messages in an email inbox) a user will more
likely enter the "System environment" section of the install tool and
take care about the found errors and warnings.

Change-Id: I3d9b628b428da13992c3dc1e7e5ab6beaeda5ce1
Resolves: #51410
Releases: 6.2
Reviewed-on: https://review.typo3.org/23380
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
6 years ago[TASK] ext:saltedpasswords: Remove isUsageEnabled for backend 75/23375/7
Nicole Cordes [Tue, 27 Aug 2013 18:16:41 +0000 (20:16 +0200)]
[TASK] ext:saltedpasswords: Remove isUsageEnabled for backend

Remove all calls on SaltedPasswordsUtility::isUsageEnabled('BE')
as backend is enabled by default. Besides remove all
ExtensionManagementUtility::isLoaded('saltedpasswords') as
extension is loaded by default.

Change-Id: Ie2332fc3c6c454888afc8c9956b9869309623584
Resolves: #51356
Releases: 6.2
Reviewed-on: https://review.typo3.org/23375
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
6 years ago[BUGFIX] utf8/multibyte character breaks menu filelist 78/23278/2
Peter Niederlag [Thu, 22 Aug 2013 11:17:25 +0000 (13:17 +0200)]
[BUGFIX] utf8/multibyte character breaks menu filelist

Resolves: #51190
Releases: 6.2, 6.1
Change-Id: I105e23b12659495b4058edbc8091eee4374d58b9
Reviewed-on: https://review.typo3.org/23278
Reviewed-by: Philipp Gampe
Tested-by: Philipp Gampe
Reviewed-by: Oliver Klee
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
6 years ago[BUGFIX] Fix incorrect call to substr_count 74/23374/3
Michiel Roos [Tue, 27 Aug 2013 18:09:16 +0000 (20:09 +0200)]
[BUGFIX] Fix incorrect call to substr_count

The call to substr_count was incorrect.

Change-Id: I9f452d22f03a99fbf0b862021b100dcf6d53d792
Resolves: #51409
Releases: 6.2
Reviewed-on: https://review.typo3.org/23374
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
6 years ago[TASK] Update SwiftMailer to 5.0.1 70/23370/2
Jigal van Hemert [Tue, 27 Aug 2013 16:00:04 +0000 (18:00 +0200)]
[TASK] Update SwiftMailer to 5.0.1

Feature wise equal to 4.3.1, but now with MIT license.

Resolves: #51406
Releases: 6.2

Change-Id: Ie0cd306b544c2f56853ca7e16ba6cdd5cdf84a6e
Reviewed-on: https://review.typo3.org/23370
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Reviewed-by: Jigal van Hemert
Tested-by: Jigal van Hemert
6 years ago[TASK] Remove misleading icon from buttons in install tool 68/23368/2
Stefan Rotsch [Tue, 27 Aug 2013 15:53:14 +0000 (17:53 +0200)]
[TASK] Remove misleading icon from buttons in install tool

Nearly all buttons in the install tool currently show a green checkmark
icon. This is misleading, as in most cases nothing has been checked.
The button simply leads to the next step or executes the desired action.

With removing the icon, it will be more clearer that clicking a button
will just perform the intended action, without any check being
performed beforehand that this action will actually succeed.

Change-Id: If00d4ccde24a201f552ead6b9d8991c8e58b473f
Resolves: #51407
Releases: 6.2
Reviewed-on: https://review.typo3.org/23368
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
6 years ago[TASK] Make the extension titles link to the configuration 63/22663/7
Nicole Cordes [Mon, 29 Jul 2013 16:13:53 +0000 (18:13 +0200)]
[TASK] Make the extension titles link to the configuration

In the extension manager make the extension titles link to the
configuration again. This will provide an additional clickable area and
it allows the integrators to continue using the known behavior.

Resolves: #50234
Releases: 6.2, 6.1, 6.0
Change-Id: I120e8fb47628fe118291cbdd57951a0238ebcd71
Reviewed-on: https://review.typo3.org/22663
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Nicole Cordes
Tested-by: Nicole Cordes
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
6 years ago[TASK] Move default salt methods to factory function 69/23369/7
Nicole Cordes [Tue, 27 Aug 2013 15:25:24 +0000 (17:25 +0200)]
[TASK] Move default salt methods to factory function

To use saltedpasswords in the install tool it is necessary to move the
default salt methods registration from ext_localconf.php to the salt
factory. This patch adds an initialize function to set the default salt
methods.

Resolves: #51404
Releases: 6.2
Change-Id: Iaa4547255c8d6e3b0b0fd75c0aaadfa54109ba27
Reviewed-on: https://review.typo3.org/23369
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
6 years ago[BUGFIX] Install Tool: Correct Browser-Title if locked 73/23373/2
Christoph Buchli [Tue, 27 Aug 2013 17:12:13 +0000 (19:12 +0200)]
[BUGFIX] Install Tool: Correct Browser-Title if locked

If the Install Tool is locked, the login-screen states whether the
cause to this is if the ENABLE_INSTALL_TOOL file doesn't exist (and
therefore it's locked) or if no password is set.

Currently, the Browser-Title states "No install tool password set"
even if it's set but the ENABLE_INSTALL_TOOL file doesn't exist.
This Patch sets the correct title in the browser-window:
"The Install Tool is locked".

Resolves: #51408
Release: 6.2
Change-Id: Ia9d25a884667404408a2ef6a375d95f7509d8d9d
Reviewed-on: https://review.typo3.org/23373
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
6 years ago[BUGFIX] Next/last button not linked on last page 63/23363/2
Jigal van Hemert [Tue, 27 Aug 2013 13:49:50 +0000 (15:49 +0200)]
[BUGFIX] Next/last button not linked on last page

The button icon viewhelper now supports an empty uri parameter. The
link on the next / last button of the backend paginate widget in case
you are on the very last page can be removed.

Change-Id: I238c77efa8a9d4ff4a1864f6df876a695dc690e5
Resolves: #51395
Releases: 6.2
Reviewed-on: https://review.typo3.org/23363
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
6 years ago[TASK] Cleanup FactoryConfiguration for saltedpasswords 66/23366/3
Nicole Cordes [Tue, 27 Aug 2013 14:05:24 +0000 (16:05 +0200)]
[TASK] Cleanup FactoryConfiguration for saltedpasswords

This patch rewrites the configuration of ext:saltedpasswords into
a readable array which gets serialized afterwards. The check for
enabling salted passwords in the backend is obsolete and removed.
The values of the "checkConfiguration" user functions superfluous
and removed as well.

Resolves: #51354
Releases: 6.2
Change-Id: I539070d1642660ad0a5641d8971bdabc7cf92223
Reviewed-on: https://review.typo3.org/23366
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
6 years ago[FEATURE] Install Tool: Check for broken ext indicates loading 65/23365/3
Christoph Buchli [Tue, 27 Aug 2013 14:03:40 +0000 (16:03 +0200)]
[FEATURE] Install Tool: Check for broken ext indicates loading

"Check for broken extensions" in important actions of the install
tool checks for broken extenions over an Ajax-Request.

Currently, the user clicks the button to check for broken extensions
and doesn't get any feedback whether there is something going on or not.
This patch implements a loading-message with a Spinner that is displayed
as soon as the button to check for broken extensions is clicked.
Furthermore the button is hidden as long as the request is beign executed
and displayed again if there is anything more to check (e.g. broken
extensions are found).

Resolves: #51379
Release: 6.2
Change-Id: I13e5024a1e2666e54e782ff1bab9418ba8b180f6
Reviewed-on: https://review.typo3.org/23365
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Reviewed-by: Christoph Buchli
Tested-by: Christoph Buchli
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
6 years ago[TASK] Refactor misleading method name 64/23364/3
Stefan Rotsch [Tue, 27 Aug 2013 13:51:00 +0000 (15:51 +0200)]
[TASK] Refactor misleading method name

Purpose of the initialize() method in the AbstractAction class is the
initialization of a handle. It is not intended to be used for generally
initializing classes derived from AbstractAction.

By refactoring the method name, the method's purpose will be indicated
more clearer.

Change-Id: I1603090e4dfccb5b57da4fa9c3792ec73647c524
Resolves: #51396
Releases: 6.2
Reviewed-on: https://review.typo3.org/23364
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
6 years ago[FEATURE] Install tool database port detection 32/23332/3
Michiel Roos [Mon, 26 Aug 2013 19:05:28 +0000 (21:05 +0200)]
[FEATURE] Install tool database port detection

The patch adds better auto detection of port settings to
install tool step action "database connect". If everything
works out, the database connect step is not rendered anymore
during upgrading which is the case without the patch.

- Split port off from IP only if IPv4 is used
- Get default or configured port for connection test
- Store port, host and socket on success and redirect to self

Change-Id: Ia5b53fc7eff60c5b2613ecee35cd5ee699e141ca
Resolves: #49186
Releases: 6.2
Reviewed-on: https://review.typo3.org/23332
Reviewed-by: Kai Ole Hartwig
Tested-by: Kai Ole Hartwig
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
6 years ago[TASK] Shorten database names in functional tests 56/23356/4
Sascha Egerer [Tue, 27 Aug 2013 11:11:35 +0000 (13:11 +0200)]
[TASK] Shorten database names in functional tests

The functional tests frameworks create a new database per test.
The database name is generated by the original database
name plus a string and a hash.

If the database name length exceeds the maximum length of 64
characters an exception is now thrown.

The generated database name is also been shortened a bit.

Resolves: #51383
Releases: 6.2
Change-Id: I5817e440a3dcba7e53e2a27dca4f7bfc679334f5
Reviewed-on: https://review.typo3.org/23356
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
6 years ago[TASK] Fix functional tests in WorkspaceServiceTest 85/23185/8
Tymoteusz Motylewski [Sun, 18 Aug 2013 20:51:40 +0000 (22:51 +0200)]
[TASK] Fix functional tests in WorkspaceServiceTest

Refactor WorkspaceServiceTest to reflect recent changes
in functional testing.
Enable test so they will be run in Travis.
Still 2 tests are skipped and need further review,
but it can be done in the separate change.

Resolves: #51164
Releases: 6.2
Change-Id: I074629d877b6a4d336bd21aacd20268bf4746811
Reviewed-on: https://review.typo3.org/23185
Reviewed-by: Stefan Neufeind
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
6 years ago[TASK] Remove workaround for PHP bug in validEmail() 57/23357/3
Markus Klein [Tue, 27 Aug 2013 11:35:44 +0000 (13:35 +0200)]
[TASK] Remove workaround for PHP bug in validEmail()

In GeneralUtility::validEmail() there's still a workaround for
a PHP crash (bug 52929).
This was fixed with PHP 5.3.4 and as TYPO3 6.2
requires 5.3.7, this workaround can be removed.

Resolves: #51386
Releases: 6.2
Change-Id: I5b32e90fd1a99c1cb2bdc261965114b06b4d4a58
Reviewed-on: https://review.typo3.org/23357
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
6 years ago[FEATURE] ext:saltedpasswords is required and enabled in backend 33/23333/6
Nicole Cordes [Mon, 26 Aug 2013 19:30:14 +0000 (21:30 +0200)]
[FEATURE] ext:saltedpasswords is required and enabled in backend

This patch adds saltedpasswords as a required system extension,
forcing salted hashes to be enabled for backend authentication.
The install tool checks settings and adapts them if needed.
Rsaauth is added to the list of recommended extensions
in the install tool extension installation upgrade wizard.

Resolves: #51352
Releases: 6.2
Change-Id: Ifd3b9f195101bcdb083b3bf9db4a74c812f0a709
Reviewed-on: https://review.typo3.org/23333
Reviewed-by: Kai Ole Hartwig
Tested-by: Kai Ole Hartwig
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
6 years ago[BUGFIX] Hide translations in categories selector 80/23280/2
Francois Suter [Fri, 23 Aug 2013 16:09:54 +0000 (18:09 +0200)]
[BUGFIX] Hide translations in categories selector

The \TYPO3\CMS\Core\Category\CategoryRegistry::addTcaColumn() method
provides an API for adding a system categories selector to a given
table. However the generated TCA misses a condition in
"foreign_table_where" to hide translated categories.

Resolves: #51304
Releases: 6.0, 6.1, 6.2
Change-Id: I2622371a89c8eabeb29e835c7f6673ffc6349186
Reviewed-on: https://review.typo3.org/23280
Reviewed-by: Philipp Gampe
Reviewed-by: Francois Suter
Tested-by: Francois Suter
6 years ago[BUGFIX] Install tool: Check for broken ext uninstalls correct ext 36/23336/5
Christoph Buchli [Mon, 26 Aug 2013 20:16:15 +0000 (22:16 +0200)]
[BUGFIX] Install tool: Check for broken ext uninstalls correct ext

"Check for broken extensions" in important actions of the install
tool renders uninstall buttons for every broken extension it found.

Currently only the last extension is uninstalled, no matter which
button is used. The patch fixes this by adding a class to identify
the extension in question to each button and adapts the javascript.

Resolves: #51368
Releases: 6.2
Change-Id: I7b754aca58e262dbd726452f816fe4dfaa6cf54d
Reviewed-on: https://review.typo3.org/23336
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Reviewed-by: Stefan Neufeind
Reviewed-by: Susanne Moog
Tested-by: Susanne Moog
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
6 years ago[BUGFIX] Tests in Localization\Parser\LocallangXmlParserTest fail 18/22918/4
Nicole Cordes [Wed, 7 Aug 2013 11:27:50 +0000 (13:27 +0200)]
[BUGFIX] Tests in Localization\Parser\LocallangXmlParserTest fail

If the system is already translated into French two tests in
Localization\Parser\LocallangXmlParserTest fail. This is because another
localization file is taken where test strings are not included.

As the tested functions can't work system independent, the test has to
make sure no system dependencies are used. Therefore the patch
introduces a new language "md5" to prove the workflow.

Resolves: #50870
Releases: 6.2, 6.1, 6.0
Change-Id: I64e8864a7d746c4f5c278f5e67d8dae068d8f920
Reviewed-on: https://review.typo3.org/22918
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
6 years ago[BUGFIX] Escape title tag of image links 03/22803/3
Alexander Stehlik [Sun, 4 Aug 2013 11:54:53 +0000 (13:54 +0200)]
[BUGFIX] Escape title tag of image links

This patch adds a missing call to htmlspecialchars() when
the title tag of image links is initialized.

Resolves: #50760
Releases: 6.2, 6.1, 6.0, 4.7, 4.5
Change-Id: If41f33d9621f7790c0ff0de4aebcd7cdcb59707f
Reviewed-on: https://review.typo3.org/22803
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
6 years ago[BUGFIX] Page tree filtering broken in IE7 & IE8 00/22500/4
Aske Ertmann [Mon, 22 Jul 2013 18:46:23 +0000 (20:46 +0200)]
[BUGFIX] Page tree filtering broken in IE7 & IE8

The page tree doesn't show the result when using the filter
in the page tree in IE7 & IE8. ExtJS somehow breaks silently
when trying to use a non-tag (text) as a drag handle. This fix
works with the original change that introduced this, meaning it
doesn't break the functionality of dragging of locked records.

Change-Id: I3f33fb3e4f6bd7622f502a9265897fee9d3e4ed3
Resolves: #25327
Resolves: #37026
Related: #M17952
Releases: 6.2, 6.1, 6.0, 4.7, 4.5
Reviewed-on: https://review.typo3.org/22500
Reviewed-by: Nicole Cordes
Tested-by: Nicole Cordes
6 years ago[FEATURE] Install tool database select during upgrading 29/23329/5
Michiel Roos [Mon, 26 Aug 2013 14:59:13 +0000 (16:59 +0200)]
[FEATURE] Install tool database select during upgrading

- Detect if SYS/isInitialInstallationInProgress is set
- If set: Show form for creating or selecting empty db
- If not set: Show all databases the user has access to

Change-Id: Ifaa1678f8ab1eaacec944e6dddf7d23babb804cf
Resolves: #51357
Releases: 6.2
Reviewed-on: https://review.typo3.org/23329
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
Reviewed-by: Alexander Böhm
Tested-by: Alexander Böhm
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
6 years ago[TASK] Align inputs and adjust label-width 83/22983/3
Stefan Neufeind [Fri, 9 Aug 2013 07:13:22 +0000 (09:13 +0200)]
[TASK] Align inputs and adjust label-width

Labels should display as a block so inputs are aligned.
Label-width needs to be adjusted as well then.

Change-Id: I4a0e58fc648efd0767d0d3dc814d5aa7c7c444e2
Resolves: #50951
Releases: 6.2
Reviewed-on: https://review.typo3.org/22983
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
6 years ago[TASK] Install tool test action fatals if image not exists 31/23331/3
Christian Kuhn [Mon, 26 Aug 2013 18:35:44 +0000 (20:35 +0200)]
[TASK] Install tool test action fatals if image not exists

The install tool test section creates test images. If permissions
on the target folder are not set up correctly, the file may not
be created, but a correct file location is returned.
In this case not caught exceptions are thrown. This is
now sanitized with a simple file exists view helper to
show some error message instead.

Change-Id: Ic840b0128e5c69f8ce6f601952b8f17e59713e27
Resolves: #51366
Releases: 6.2
Reviewed-on: https://review.typo3.org/23331
Reviewed-by: Simon Schmidt
Tested-by: Simon Schmidt
Reviewed-by: Steffen Ritter
Tested-by: Steffen Ritter
6 years ago[FEATURE] Add public API to get ApplicationContext 18/23018/8
Steffen Müller [Fri, 9 Aug 2013 22:09:57 +0000 (00:09 +0200)]
[FEATURE] Add public API to get ApplicationContext

Application Context is initialized in Bootstrap. We need a
public API to get the current context without using the bootstrap
itself.

In Flow, the ObjectManager provides this API, but in TYPO3 CMS we
do not have this kind of ObjectManager (except in extbase).

The API call is:
\TYPO3\CMS\Core\Utility\GeneralUtility::getContext();

Resolves: #50983
Releases: 6.2
Change-Id: Ic900a503b3e2bc184d2bfd43345e72180887ef18
Reviewed-on: https://review.typo3.org/23018
Reviewed-by: Helmut Hummel
Tested-by: Helmut Hummel
6 years ago[BUGFIX] Fix broken path to fonts in t3skin 06/23306/2
Steffen Müller [Sun, 25 Aug 2013 22:35:18 +0000 (00:35 +0200)]
[BUGFIX] Fix broken path to fonts in t3skin

Follow-up: #51134
Releases: 6.2
Change-Id: I80c2f2bab106cf78a93e371e613c8049510f7bfd
Reviewed-on: https://review.typo3.org/23306
Reviewed-by: Ernesto Baschny
Tested-by: Ernesto Baschny
6 years ago[FEATURE] Render icon with button view helper 81/23281/2
Francois Suter [Fri, 23 Aug 2013 16:32:05 +0000 (18:32 +0200)]
[FEATURE] Render icon with button view helper

Fluid is currently lacking a view helper to render a skinned icon.
By making the button icon view helper accept an empty URI as an
argument, we let it render an unlinked icon, rather than adding
a new view helper with very similar code.

This patch also cleans up the class phpDoc block, which
contained several errors.

Resolves: #51305
Releases: 6.2
Change-Id: I74b1dc3346e20c9094313a2da451bcb6c1f60f29
Reviewed-on: https://review.typo3.org/23281
Reviewed-by: Frans Saris
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
6 years ago[BUGFIX] Fix failing test 62/23262/2
Anja Leichsenring [Thu, 22 Aug 2013 19:22:57 +0000 (21:22 +0200)]
[BUGFIX] Fix failing test

The test using this file was not adjusted and thus failing.

The test was adjusted and another one added, looking after the new
position.

Resolves: #51273
Relates: #51251
Releases: 6.2
Change-Id: I12aebadfa12633f62b32365b064e802f608f6197
Reviewed-on: https://review.typo3.org/23262
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
6 years ago[TASK] Merge pagegen.php to index_ts.php 46/23246/2
Christian Kuhn [Sun, 18 Aug 2013 17:06:45 +0000 (19:06 +0200)]
[TASK] Merge pagegen.php to index_ts.php

pagegen.php contains rendering code and is called twice
in index_ts.php. The patch merged the content of this
file to index_ts.php to have all global code in one file.

The patch is a straight forward refactoring, only the
time tracking is a bit simplified and an obsolete if
is removed at the former second inclusion of the file.

Change-Id: Id8477423455281bd6df2ce32eb280809045e4541
Resolves: #51254
Releases: 6.2
Reviewed-on: https://review.typo3.org/23246
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
6 years ago[TASK] Move TCA and ext_tables.sql from ext:cms to ext:frontend 44/23244/2
Christian Kuhn [Sun, 18 Aug 2013 16:09:12 +0000 (18:09 +0200)]
[TASK] Move TCA and ext_tables.sql from ext:cms to ext:frontend

EXT:frontend is the successor of ext:cms. This started with the
namespace change so all class files are already located there.
The patch moves TCA and ext_tables.sql to the extension to make
it very clear that functional tests that depend on this data
should also go to ext:frontend later on.
The patch is trivial since both extensions are "required" and
thus always loaded, with ext:frontend directly before ext:cms,
so nothing can break with this patch.

Change-Id: Ifd9e0c9263f6b9a6aadfbfa1b568c4cd085e40d9
Resolves: #51251
Releases: 6.2
Reviewed-on: https://review.typo3.org/23244
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
6 years ago[BUGFIX] Wrong namespace in DataHandler test 45/23245/2
Christian Kuhn [Sun, 18 Aug 2013 17:51:06 +0000 (19:51 +0200)]
[BUGFIX] Wrong namespace in DataHandler test

Change-Id: Ice648bc6d9d0668c7f0a42ae57fab556da7dcaaa
Resolves: #51253
Releases: 6.2
Reviewed-on: https://review.typo3.org/23245
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
6 years agoRevert "[BUGFIX] Check for incompatible SQL modes" 43/23243/2
Stefan Neufeind [Thu, 22 Aug 2013 10:19:18 +0000 (12:19 +0200)]
Revert "[BUGFIX] Check for incompatible SQL modes"

The sysenv-check is not the proper place for a check that needs a working db-connection. But we'll find a different place for this check shortly and then reintroduce it.

This reverts commit bad3f24c1d106ce4d2254c1a8e5710edb769151e

Change-Id: I1ba4a2ed88ad03df122593ac964d997ec05edacf
Reviewed-on: https://review.typo3.org/23243
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
6 years ago[BUGFIX] Missing "Interface" in Backend-Annotation 38/23238/3
Stefan Froemken [Thu, 22 Aug 2013 09:40:09 +0000 (11:40 +0200)]
[BUGFIX] Missing "Interface" in Backend-Annotation

ObjectMapper does not work for Backend-Mappings
because of missing "Interface" in Backend-Annotation

Resolves: #51249
Releases: 6.2, 6.1
Change-Id: I45a6f1c57ddcf71282853c44e261bfa877854b85
Reviewed-on: https://review.typo3.org/23238
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
6 years ago[BUGFIX] Check for incompatible SQL modes 71/19671/13
Sascha Egerer [Sat, 6 Apr 2013 13:05:14 +0000 (15:05 +0200)]
[BUGFIX] Check for incompatible SQL modes

If the SQL mode "STRICT_ALL_TABLES" is enabled you
will not be able to save most records. (Enable it in
your my.cnf and try to save a tt_content record)

This patch adds a check in the install tool for the
incompatible SQL modes "STRICT_ALL_TABLES" and
"NO_BACKSLASH_ESCAPES".

The automatic fix for "NO_BACKSLASH_ESCAPES" has been
removed because we don't want to change the mysql
environment automatically.

Resolves: #20052
Resolves: #18866
Resolves: #18821
Releases: 6.2, 6.1, 6.0
Change-Id: Ifd2d7901935f06534a273374bf48266916e23698
Reviewed-on: https://review.typo3.org/19671
Reviewed-by: Sascha Egerer
Tested-by: Sascha Egerer
Reviewed-by: Markus Klein
Reviewed-by: Alexander Opitz
Tested-by: Alexander Opitz
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
6 years ago[BUGFIX] Storage is offline but is still used 15/22915/4
Frans Saris [Wed, 7 Aug 2013 10:09:54 +0000 (12:09 +0200)]
[BUGFIX] Storage is offline but is still used

Check before generating a publicUrl if storage is online.

Resolves: #49842
Releases: 6.2, 6.1, 6.0
Change-Id: I766e337cf5e7dfcb309bbce0bbd7c4d8df733c84
Reviewed-on: https://review.typo3.org/22915
Reviewed-by: Frans Saris
Tested-by: Frans Saris
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
6 years ago[BUGFIX] Ignore permission checks for processed files 96/23196/3
Helmut Hummel [Wed, 14 Aug 2013 14:49:55 +0000 (16:49 +0200)]
[BUGFIX] Ignore permission checks for processed files

It must be possible that processed files are generated
even if the user has read only permissions on a storage.

Additionally add a method to Storage, to specifically
update a processed file with a local file.

Resolves: #51209
Releases: 6.0, 6.1, 6.2

Change-Id: I847b02f070c55647e06695c1c70b0b7e8e238177
Reviewed-on: https://review.typo3.org/23196
Reviewed-by: Frans Saris
Tested-by: Frans Saris
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
6 years ago[BUGFIX] No version overlay should be done for sys_language 28/11928/6
Lienhart Woitok [Tue, 27 Nov 2012 01:43:04 +0000 (02:43 +0100)]
[BUGFIX] No version overlay should be done for sys_language

The tables sys_language and static_languages do not support
versioning. Therefore, when doing a version preview, no
version overlay should be applied to them.

Change-Id: I37be782601fc37e78a74d2459b6f246c5107b701
Fixes: #37892
Releases: 6.2, 6.1, 6.0
Reviewed-on: https://review.typo3.org/11928
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Reviewed-by: Stefan Neufeind
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
6 years ago[BUGFIX] Files with unclean path indexed multiple times 23/22923/7
Stefan Neufeind [Sat, 6 Apr 2013 15:11:52 +0000 (17:11 +0200)]
[BUGFIX] Files with unclean path indexed multiple times

When adding a file or requesting a file by an identifier
cleanup any . and .. in the path before handing off
to the driver so files are not indexed multiple times.

Fixes: #46989
Releases: 6.2, 6.1, 6.0
Change-Id: I4198a8885a6a148e68e1e0f717775f9af976a9ef
Reviewed-on: https://review.typo3.org/22923
Reviewed-by: Helmut Hummel
Tested-by: Helmut Hummel
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
6 years ago[BUGFIX] FlashMessageQueue PHP strict warning 83/23183/7
Christian Kuhn [Sun, 18 Aug 2013 13:50:54 +0000 (15:50 +0200)]
[BUGFIX] FlashMessageQueue PHP strict warning

FlashMessageQueue exends \SplQueue and overwrites enqueue($value).
In \SplQueue enqueue() has no type hint on $value, but
FlashMessageQueue adds a type hint to FlashMessage. This raises
a PHP strict warning.

Solution is to remove the type hint and add an exception instead
if given argument is not an instance of FlashMessage.

Change-Id: I882aab33a55d84de6614a45c6cc045223adba0cd
Resolves: #51159
Releases: 6.2, 6.1
Reviewed-on: https://review.typo3.org/23183
Reviewed-by: Helmut Hummel
Tested-by: Helmut Hummel
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
6 years ago[TASK] Adapt skinning extension stylesheet paths 44/23144/2
Felix Kopp [Fri, 16 Aug 2013 22:03:29 +0000 (00:03 +0200)]
[TASK] Adapt skinning extension stylesheet paths

In order to clean up EXT: t3skin the standard paths should
be adapted to follow extbase/fluid conventions.

Change-Id: Ic43a42e13c7656064743c936b678a5172e45209b
Resolves: #51134
Releases: 6.2
Reviewed-on: https://review.typo3.org/23144
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
6 years ago[TASK] Implement getDatabase to FunctionalTestCase 87/23187/2
Christian Kuhn [Sun, 18 Aug 2013 23:24:02 +0000 (01:24 +0200)]
[TASK] Implement getDatabase to FunctionalTestCase

Fiddling with database records is a common use case in
functional tests. The patch implements a getDatabase()
method to FunctionalTestCase for easy IDE auto completion
and uses it in activated test cases.

Change-Id: I8900457ff1c31e1f1d731337be17431ccf89b379
Resolves: #51167
Releases: 6.2
Reviewed-on: https://review.typo3.org/23187
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
6 years ago[TASK] Improve documentation in FunctionalTestCase 86/23186/2
Christian Kuhn [Sun, 18 Aug 2013 23:00:32 +0000 (01:00 +0200)]
[TASK] Improve documentation in FunctionalTestCase

Change-Id: Ice1e7760d9d1644411bc95ee2dd15e0885277e82
Releases: 6.2
Resolves: #51165
Reviewed-on: https://review.typo3.org/23186
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
6 years ago[FEATURE] Responsive website preview 68/23168/2
Felix Kopp [Sat, 17 Aug 2013 23:19:54 +0000 (01:19 +0200)]
[FEATURE] Responsive website preview

Bring responsive adaption for preview frame in EXT: viewpage
based on common device widths.

Change-Id: I8e1df6243436d5b76b17ae6364f63c2a93da07c1
Resolves: #51148
Releases: 6.2
Reviewed-on: https://review.typo3.org/23168
Reviewed-by: Philipp Gampe
Tested-by: Philipp Gampe
Tested-by: Stefan Neufeind
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
6 years ago[TASK] FunctionalTestCase refactoring 81/23181/3
Christian Kuhn [Sun, 18 Aug 2013 12:43:48 +0000 (14:43 +0200)]
[TASK] FunctionalTestCase refactoring

* Make properties in utility class non static, store private instance
* Improve documentation
* Better exception code in case test database can not be created

Change-Id: Ib05ad804fd4bbb6772243343e2160df5170d8bfe
Resolves: #51156
Releases: 6.2
Reviewed-on: https://review.typo3.org/23181
Reviewed-by: Tymoteusz Motylewski
Tested-by: Tymoteusz Motylewski
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
6 years ago[CLEANUP] Remove deprecated repository constructor 75/23175/2
Felix Oertel [Sat, 17 Aug 2013 13:08:27 +0000 (15:08 +0200)]
[CLEANUP] Remove deprecated repository constructor

Resolves: #51142
Releases: 6.2
Change-Id: I4d7e49d564df0f59c75802a6526ca0ede1f3477c
Reviewed-on: https://review.typo3.org/23175
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
6 years ago[TASK] FilesContentObject::stdWrapValue(): only execute stdWrap once 40/22740/2
Stefan Neufeind [Thu, 1 Aug 2013 01:03:20 +0000 (03:03 +0200)]
[TASK] FilesContentObject::stdWrapValue(): only execute stdWrap once

Change-Id: I4bad84460e1d1b16e05498e382644cd5946c3be8
Resolves: #50614
Releases: 6.2, 6.1, 6.0
Reviewed-on: https://review.typo3.org/22740
Reviewed-by: Wouter Wolters
Reviewed-by: Philipp Gampe
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
6 years ago[TASK] Use real boolean value for sL second param 53/23153/2
Wouter Wolters [Sat, 17 Aug 2013 13:44:03 +0000 (15:44 +0200)]
[TASK] Use real boolean value for sL second param

Make use of boolean value TRUE as second parameter in sL function.

Change-Id: I6d77affc4f3ff260f617ca67c773fc4ca2e481ce
Resolves: #51143
Releases: 6.2
Reviewed-on: https://review.typo3.org/23153
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
6 years ago[BUGFIX] Language-module icons need to display in correct size 34/22734/4
Stefan Neufeind [Wed, 31 Jul 2013 14:07:39 +0000 (16:07 +0200)]
[BUGFIX] Language-module icons need to display in correct size

Since inline:display-block was missing on those icons, those
icons didn't display in their intended size although they
had a width/height-attribute set.

Change-Id: I5ba318e6a3ca4602714d0c4b5f12674e2e305952
Resolves: #43428
Releases: 6.2, 6.1, 6.0
Reviewed-on: https://review.typo3.org/22734
Reviewed-by: Philipp Gampe
Tested-by: Philipp Gampe
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
6 years agoRevert "[CLEANUP] remove deprecated repository constructor" 73/23173/3
Anja Leichsenring [Sun, 18 Aug 2013 10:35:10 +0000 (12:35 +0200)]
Revert "[CLEANUP] remove deprecated repository constructor"
This breaks a lot of tests as is. Will be reintroduced
together with the neccessary refactorings.

This reverts commit 91bb179484f45fdbdc5a20f938c31e982847b11e

Change-Id: Icb319d4bfce550758241714cd9d344a1038441bc
Reviewed-on: https://review.typo3.org/23173
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
6 years ago[TASK] Simplify travis script 72/23172/2
Christian Kuhn [Sun, 18 Aug 2013 10:10:55 +0000 (12:10 +0200)]
[TASK] Simplify travis script

The chmod for phpunit binaries is not needed after ext:phpunit
fixed this, the execute script itself is simplified a bit to
be better readable.

Change-Id: I9b6d1d1e2e9e4155ee20c3dfda4313d89fed0def
Resolves: #51153
Releases: 6.2
Reviewed-on: https://review.typo3.org/23172
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
6 years ago[TASK] Add first functional tests for DataHandler 71/23171/5
Tymoteusz Motylewski [Sun, 18 Aug 2013 08:08:12 +0000 (10:08 +0200)]
[TASK] Add first functional tests for DataHandler

The patch introduces some common database fixtures that can be
re-used between tests. Two base functional tests in core
data handler are activated and another one in workspace.

Functional test xml is changed to execute all core and workspace
tests. Most of them are not working yet and marked as incomplete
for new.

Resolves: #51151
Releases: 6.2
Change-Id: Ic8e3da4f1064689e6ea7ef2365b0f8b1a3ce91f4
Reviewed-on: https://review.typo3.org/23171
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Reviewed-by: Tymoteusz Motylewski
Tested-by: Tymoteusz Motylewski
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
6 years ago[TASK] Remove dispensable headlines in EXT: beuser 63/23163/2
Felix Kopp [Sat, 17 Aug 2013 20:40:47 +0000 (22:40 +0200)]
[TASK] Remove dispensable headlines in EXT: beuser

The amount of headline in backend user module can be reduced.
There is one headline for one view in Backend User Management.

Change-Id: If6b58b0ac2a09ecc9020714dba01448b6c159cb5
Resolves: #51147
Releases: 6.2
Reviewed-on: https://review.typo3.org/23163
Reviewed-by: Philipp Gampe
Tested-by: Philipp Gampe
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
6 years ago[CLEANUP] remove deprecated repository constructor 51/23151/6
Felix Oertel [Sat, 17 Aug 2013 13:08:27 +0000 (15:08 +0200)]
[CLEANUP] remove deprecated repository constructor

Resolves: #51142
Releases: 6.2
Change-Id: Iff5850940fa6e92ccd890564e2a90d11e9cf5e8d
Reviewed-on: https://review.typo3.org/23151
Reviewed-by: Oliver Klee
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
6 years ago[BUGFIX] TCA: subtypes_addlist not processed 49/9549/5
Benjamin Mack [Sat, 6 Apr 2013 09:54:32 +0000 (11:54 +0200)]
[BUGFIX] TCA: subtypes_addlist not processed

TCA: subtypes_addlist is not processed
if the subtype_value_field is in a palette

Fixes: #30636
Releases: 6.2, 6.1, 6.0, 4.7, 4.5
Change-Id: I6bcaaf36c330962c1d7d9fdd4a73ae502e9710dc
Reviewed-on: https://review.typo3.org/9549
Reviewed-by: Wouter Wolters
Tested-by: Lucas Bremgartner
Reviewed-by: Ernesto Baschny
Tested-by: Ernesto Baschny
Reviewed-by: Philipp Gampe
Tested-by: Philipp Gampe
6 years ago[BUGFIX] Not possible to finish update wizard 79/22679/2
Wouter Wolters [Mon, 29 Jul 2013 23:14:48 +0000 (01:14 +0200)]
[BUGFIX] Not possible to finish update wizard

When running the update wizard you are not able to finish it.
Therefore a back button is introduced when there is no
upgrade left.

Change-Id: Iaf66ced20e0dc0ec2870a559d6beee802011ae75
Resolves: #47735
Releases: 6.2
Reviewed-on: https://review.typo3.org/22679
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Reviewed-by: Philipp Gampe
Tested-by: Philipp Gampe
6 years ago[BUGFIX] Query parameters of external link may get altered 82/22682/2
Stanislas Rolland [Tue, 30 Jul 2013 02:34:54 +0000 (22:34 -0400)]
[BUGFIX] Query parameters of external link may get altered

If an external link is set to the same domain as the BE domain and
the id query parameter is present, any other query parameters will
be altered (and repeated).
Solution: href of external link should remain unaltered.

Resolves: #47844
Releases: 6.0, 6.1, 6.2
Change-Id: I79564ebf63ee4f92bfd3f95b17f72592ea849163
Reviewed-on: https://review.typo3.org/22682
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Philipp Gampe
Tested-by: Philipp Gampe
6 years ago[TASK] Refactor functional base test class 54/23154/7
Christian Kuhn [Sat, 17 Aug 2013 14:10:03 +0000 (16:10 +0200)]
[TASK] Refactor functional base test class

Extract setUp() and tearDown() code to an own class
and start documenting the whole sytem.

Resolves: #51144
Releases: 6.2

Change-Id: Id71e9f9111f62ee04e5f8ebcec121e5e01b958b3
Reviewed-on: https://review.typo3.org/23154
Reviewed-by: Tymoteusz Motylewski
Tested-by: Tymoteusz Motylewski
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
Reviewed-by: Helmut Hummel
Tested-by: Helmut Hummel
6 years ago[TASK] Implement standalone functional test API 17/23117/13
Helmut Hummel [Thu, 25 Jul 2013 08:33:17 +0000 (10:33 +0200)]
[TASK] Implement standalone functional test API

Test encapsulation and a controlled environment is
crucial for solid functional tests.

The patch creates a full TYPO3 CMS instance within
typo3temp/ together with an own database and
LocalConfiguration to run a specific functional test
in this environment. A full TYPO3 CMS bootstrap of
this instance is done. A new environment with a fresh
PHP process is created for each and every single test.

Functional test can use the API by calling parent::setUp()
and parent::tearDown().

The functional suite can be called directly with phpunit
"./typo3conf/ext/phpunit/Composer/vendor/bin/phpunit
-c typo3/sysext/core/Build/FunctionalTests.xml"

Currently the test suite must be called from the document
root folder.

The patch is currently a base patch for the main API implementing
immediatly needed stuff. With further patches sanitizing and
more details will be added.

Change-Id: I54f652f6a346a5155b5c33e4a065ab37898ff5b2
Resolves: #51091
Releases: 6.2
Reviewed-on: https://review.typo3.org/23117
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Reviewed-by: Stefan Neufeind
Reviewed-by: Tymoteusz Motylewski
Tested-by: Tymoteusz Motylewski
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
6 years ago[BUGFIX] Show correct record title for be_groups and be_users 31/9631/17
Markus Klein [Wed, 12 Dec 2012 10:48:42 +0000 (11:48 +0100)]
[BUGFIX] Show correct record title for be_groups and be_users

Currently it lists groups only using field title of
be_groups table regardless of any label definitions in TCA.
Use BackendUtility::getRecordTitle() instead.

Change-Id: I051c29687cb51d1883846b3cb780409c1e1cadc4
Resolves: #34631
Releases: 6.2, 6.1, 6.0
Reviewed-on: https://review.typo3.org/9631
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
6 years ago[TASK] Execute standalone functional tests in travis 38/23138/3
Christian Kuhn [Fri, 16 Aug 2013 14:15:58 +0000 (16:15 +0200)]
[TASK] Execute standalone functional tests in travis

Change-Id: If9195b5ef908ecf09d1a03f07f93cf983beaf474
Resolves: #51121
Releases: 6.2
Reviewed-on: https://review.typo3.org/23138
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
6 years ago[TASK] Reformat .travis.yml to be better readable 36/23136/2
Christian Kuhn [Fri, 16 Aug 2013 13:06:13 +0000 (15:06 +0200)]
[TASK] Reformat .travis.yml to be better readable

Resolves: #51119
Releases: 6.2
Change-Id: Ibbabf4de13aa4bff069021629bedb14fd111f091
Reviewed-on: https://review.typo3.org/23136
Reviewed-by: Stefan Neufeind
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
6 years ago[FEATURE] Prepare generic persistence for overlay handling by signals 28/23128/8
Felix Oertel [Fri, 16 Aug 2013 09:54:08 +0000 (11:54 +0200)]
[FEATURE] Prepare generic persistence for overlay handling by signals

To get a better grip on languages and workspaces, we decided
to separate their handling from the storageBackend.

This is the first step, introducing the signals needed to
invoke external action.

In some cleanup it also straightens up the names of the
member variables.

Resolves: #51106
Releases: 6.2
Change-Id: Ie4a79270c5fe5f559f0950ea157c137c5c9cccd1
Reviewed-on: https://review.typo3.org/23128
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
6 years ago[TASK] Standalone functional test runner 31/23131/3
Christian Kuhn [Fri, 16 Aug 2013 11:09:38 +0000 (13:09 +0200)]
[TASK] Standalone functional test runner

Introduce base code for a standalone php test runner.

"./typo3conf/ext/phpunit/Composer/vendor/bin/phpunit
-c typo3/sysext/core/Build/FunctionalTests.xml"

This way phpunit is called fully standalone and does not
depend on TYPO3 CMS bootstrap itself.

Resolves: #51111
Releases: 6.2
Change-Id: Ie790eec315f7a273fc62856df3169aa56b2ad60d
Reviewed-on: https://review.typo3.org/23131
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
6 years ago[TASK] Disable scheduler-tests if EXT:scheduler not loaded 33/23133/2
Stefan Neufeind [Fri, 16 Aug 2013 11:53:27 +0000 (13:53 +0200)]
[TASK] Disable scheduler-tests if EXT:scheduler not loaded

Change-Id: I62ec38fc27330e74c4bc3eed6ded3a2f4a6de550
Resolves: #51115
Releases: 6.2, 6.1, 6.0
Reviewed-on: https://review.typo3.org/23133
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
6 years ago[TASK] Enable unittests for EXT:documentation 32/23132/2
Stefan Neufeind [Fri, 16 Aug 2013 11:38:53 +0000 (13:38 +0200)]
[TASK] Enable unittests for EXT:documentation

Travis should start including the unittests for EXT:documentation
in its builds as well.

Change-Id: Ie59945964dbf684d3493b0d7581c41b0cde4750f
Resolves: #51114
Releases: 6.2
Reviewed-on: https://review.typo3.org/23132
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
6 years ago[TASK] Make tests use correct namespaces 27/23127/3
Stefan Neufeind [Fri, 16 Aug 2013 09:41:01 +0000 (11:41 +0200)]
[TASK] Make tests use correct namespaces

Change-Id: I7cc308d3c5c59b1bf8922b6830cf29fb726caa56
Resolves: #51103
Releases: 6.2, 6.1, 6.0
Reviewed-on: https://review.typo3.org/23127
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
6 years ago[TASK] Introduce relative line-heights 21/23021/3
Felix Kopp [Sat, 10 Aug 2013 12:51:34 +0000 (14:51 +0200)]
[TASK] Introduce relative line-heights

The line-height for the backend was fixed to 14px.
But line-heights should be relative to the font-size.

With this standard line-heights are 1.5 x font-size.

Change-Id: I2c098141b4905b1ac7bf756266e77dd2078ea6f2
Resolves: #51000
Releases: 6.2
Reviewed-on: https://review.typo3.org/23021
Reviewed-by: Ingo Pfennigstorf
Tested-by: Ingo Pfennigstorf
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
6 years ago[TASK] UnitTests.xml cleanup 24/23124/2
Christian Kuhn [Fri, 16 Aug 2013 09:28:54 +0000 (11:28 +0200)]
[TASK] UnitTests.xml cleanup

Minor cleanup in UnitTests.xml, BackupGlobals can be enabled
by default.

Resolves: #51101
Releases: 6.2

Change-Id: Ia1271d60bf29a5ac7adb98116d9d7a7a43f00b47
Reviewed-on: https://review.typo3.org/23124
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
6 years ago[BUGFIX] Fix file permission methods in BackendUserAuthentication 37/23037/6
Helmut Hummel [Wed, 7 Aug 2013 16:33:37 +0000 (18:33 +0200)]
[BUGFIX] Fix file permission methods in BackendUserAuthentication

Take default TSConfig file permissions into account:

File permissions configured in User TSConfig,
are not taken into account, because the properties
from the getTSConfig method are returned in a
'properties' key of an array but the top level
array is used. Use getTSConfigProp instead.

Fix the bit wise check for old file permissions:

XOR the value is wrong because it gives the wrong
result if more bits are set. Use AND operation
to properly check if a specific bit is not set.

Check if the user is admin in getFilePermissionsForStorage()

Add tests to confirm the desired behaviour.

Releases: 6.0, 6.1, 6.2
Resolves: #51004
Change-Id: Ia5d6fa1cb47a74306fe5465a0e70c2f2aea2a4b8
Reviewed-on: https://review.typo3.org/23037
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
Reviewed-by: Stefan Neufeind
Reviewed-by: Helmut Hummel
Tested-by: Helmut Hummel
6 years ago[BUGFIX] Avoid usage of subheader in mailform 91/22391/3
Francois Suter [Wed, 17 Jul 2013 20:07:01 +0000 (22:07 +0200)]
[BUGFIX] Avoid usage of subheader in mailform

Since TYPO3 CMS 4.7 the subheader field is rendered by lib.stdheader
if the chosen doctype is HTML 5. However the old mailform (ab)uses this
field for storing the recipient e-mail address. The subheader must
not be displayed in this case.

Resolves: #46094
Releases: 6.2, 6.1, 6.0, 4.7
Change-Id: I19d13e65b941e2514a9c4a12cb37f015661e9d8f
Reviewed-on: https://review.typo3.org/22391
Reviewed-by: Jo Hasenau
Tested-by: Jo Hasenau
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
6 years ago[BUGFIX] storagePid is lost when persistence.recursive is set 86/23086/5
Steffen Müller [Tue, 13 Aug 2013 18:55:01 +0000 (20:55 +0200)]
[BUGFIX] storagePid is lost when persistence.recursive is set

If recursive storagePid is set via TypoScript, the storagePid
itself is missing in the list of pids.

Resolves: #51064
Releases: 6.2, 6.1
Change-Id: I5f06911bb60b847e4a97b5690e57557d4dec66b7
Reviewed-on: https://review.typo3.org/23086
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
6 years ago[BUGFIX] Typing after abbr or acronym tag is difficult 84/22684/2
Stanislas Rolland [Tue, 30 Jul 2013 03:16:15 +0000 (23:16 -0400)]
[BUGFIX] Typing after abbr or acronym tag is difficult

Solution: After inserting the abbreviation, position the cursor
after and outside the inserted tag.

Resolves: #47806
Releases: 6.0, 6.1, 6.2
Change-Id: I4e8c47cf031fe3df2772ed47cb42122b8db9a646
Reviewed-on: https://review.typo3.org/22684
Reviewed-by: Stefan Neufeind
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Stanislas Rolland
Tested-by: Stanislas Rolland
6 years ago[BUGFIX] Fix inconsistencies in getTSConfig in BackenuserAuth 32/23032/4
Helmut Hummel [Sat, 10 Aug 2013 21:18:05 +0000 (23:18 +0200)]
[BUGFIX] Fix inconsistencies in getTSConfig in BackenuserAuth

BackenuserAuthentication::getTSConfig() behaves
inconsistently when an object string is given with
multiple sections but no result in the actual TSConfig
of the user. Other than noted in the method declaration,
the method returns an empty array instead of an array
where the 'properties' key is null.

Additionally we fix a bug which caused wrong results
if "0" was used as a key in configuration and accessed
with something like "permissions.file.0"

Resolves: #51007
Releases: 6.0, 6.1, 6.2
Change-Id: I80fc7398c1955ff77d052377e3ba81bb8aea01cf
Reviewed-on: https://review.typo3.org/23032
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Markus Klein
Tested-by: Markus Klein
6 years ago[FEATURE] Intgrate CSS class "lead" for introductions 50/23050/2
Felix Kopp [Mon, 12 Aug 2013 08:02:53 +0000 (10:02 +0200)]
[FEATURE] Intgrate CSS class "lead" for introductions

The most relevant text should always be highly visible.
Integrate the CSS class "lead" for introductions.

Lead brings a slightly larger font size and should be used
for the first short introduction text only.

"Lead" is a semantical annotation.
Follow-ups: Bring lead texts for all backend modules.

Change-Id: I10305c2d2ed66526d7e882aceff17b5d317cfbb3
Resolves: #51022
Releases: 6.2
Reviewed-on: https://review.typo3.org/23050
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
6 years ago[BUGFIX] Reposition TYPO3 logo text 47/23047/2
Felix Kopp [Sun, 11 Aug 2013 22:21:53 +0000 (00:21 +0200)]
[BUGFIX] Reposition TYPO3 logo text

Brings logo text "TYPO3" in line with module menu text.

Change-Id: Ief21e7ded4f8085e2f661304b55861211f202955
Resolves: #51015
Releases: 6.2
Reviewed-on: https://review.typo3.org/23047
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
6 years ago[TASK] Visual feedback on hover: spot text links 19/23019/2
Felix Kopp [Sat, 10 Aug 2013 11:15:16 +0000 (13:15 +0200)]
[TASK] Visual feedback on hover: spot text links

Text links are not spotted easily. Give TYPO3 an easy visual feedback
for text links on hover. Hover on table rows also pumps up text links.

Change-Id: Ie255df54a5c556160f0e0826c87ff6425907bcb0
Resolves: #50997
Releases: 6.2
Reviewed-on: https://review.typo3.org/23019
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
6 years ago[TASK] EXT: beuser remove spaces from links 48/23048/3
Felix Kopp [Sun, 11 Aug 2013 22:28:45 +0000 (00:28 +0200)]
[TASK] EXT: beuser remove spaces from links

Removes excessive spaces from within links / a-tags.

Change-Id: Ica8b6d365282dc3a0e6588a5099404c6b4bc7c73
Resolves: #51016
Releases: 6.2
Reviewed-on: https://review.typo3.org/23048
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
6 years ago[TASK] Installer: Ask for username during initial setup 82/22982/2
Stefan Neufeind [Fri, 9 Aug 2013 06:55:01 +0000 (08:55 +0200)]
[TASK] Installer: Ask for username during initial setup

Good security-practice is to choose a username different
than "admin".

Change-Id: Ib4a70b491f1f9a9112b8cd37db43664186d7192d
Resolves: #50949
Releases: 6.2
Reviewed-on: https://review.typo3.org/22982
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
6 years ago[FEATURE] EXT: extensionmanager PNG icon 46/23046/3
Felix Kopp [Sun, 11 Aug 2013 21:16:51 +0000 (23:16 +0200)]
[FEATURE] EXT: extensionmanager PNG icon

Use transparent PNG version of backend module icon for
EXT: extensionmanager in module menu instead of GIF.

Change-Id: I424a082c6a294be5f49d4c2a9dd976cb014d5096
Resolves: #51014
Releases: 6.2
Reviewed-on: https://review.typo3.org/23046
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
6 years ago[TASK] EXT: be_user - improve listing 20/23020/4
Felix Kopp [Sat, 10 Aug 2013 11:36:58 +0000 (13:36 +0200)]
[TASK] EXT: be_user - improve listing

Backend user's username and real name are tightly coupled. These
texts should stand right next to each other. Merged username and
real name columns. When real name is unknown, not a problem.

Also adds link to the primary action / edit view to user listing.

Change-Id: I16d06771ad302fe64232e068e12663d87c07d79d
Resolves: #50999
Resolves: #50994
Related: #50998
Releases: 6.2
Reviewed-on: https://review.typo3.org/23020
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
6 years ago[BUGFIX] Allow reading files if storage is not browsable 34/23034/2
Helmut Hummel [Sat, 10 Aug 2013 21:26:42 +0000 (23:26 +0200)]
[BUGFIX] Allow reading files if storage is not browsable

Whether a storage is browsable or not should not
affect the possibility to read a file with a
given identifier.

Releases: 6.0, 6.1, 6.2
Resolves: #51010

Change-Id: Ic754143ed39a277f14c488c5b0d4c050dde027df
Reviewed-on: https://review.typo3.org/23034
Reviewed-by: Steffen Ritter
Tested-by: Steffen Ritter
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
6 years ago[BUGFIX] Take into account all file and folder permissions 35/23035/3
Helmut Hummel [Sat, 10 Aug 2013 17:40:47 +0000 (19:40 +0200)]
[BUGFIX] Take into account all file and folder permissions

When handling the old file permission settings from
user or user group records, all current FAL user
permissions should be set accordingly, to not allow
file actions with FAL which were not allowed before.

In particular no write / change actions should be
allowed if no permissions are granted in the records.

Releases: 6.0, 6.1, 6.2
Resolves: #51005
Related: #51007

Change-Id: I873f23ed02428842621c8256df5792e54abbfe2c
Reviewed-on: https://review.typo3.org/23035
Reviewed-by: Steffen Ritter
Tested-by: Steffen Ritter
Reviewed-by: Helmut Hummel
Tested-by: Helmut Hummel
6 years ago[TASK] Add signal in ResourceFactory for storage creation 33/23033/2
Helmut Hummel [Sun, 11 Aug 2013 16:28:42 +0000 (18:28 +0200)]
[TASK] Add signal in ResourceFactory for storage creation

We miss a central place to adapt or change
configuration for a resource storage.

Add a signal in the resource factory to be able
to post process a storage object after creation.

Releases: 6.0, 6.1, 6.2
Resolves: #51011
Change-Id: Iace95bc50ab149b4e62dd85c01417b3a3e6bf1cf
Reviewed-on: https://review.typo3.org/23033
Reviewed-by: Steffen Ritter
Tested-by: Steffen Ritter
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
6 years ago[BUGFIX] rtehtmlarea acronym error with static_info_tables 6.0+ 64/22664/2
Stanislas Rolland [Mon, 29 Jul 2013 16:33:06 +0000 (12:33 -0400)]
[BUGFIX] rtehtmlarea acronym error with static_info_tables 6.0+

Problem: Editing acronym records fails with Static Info Tables 6.0+ as
tx_staticinfotables_div->selectItemsTCA does not exist anymore.
Solution: Use suggest wizard

Resolves: #50502
Releases: 6.0, 6.1, 6.2
Change-Id: I69e564662df19f0625ec3e71b05e91c7ed5edfc8
Reviewed-on: https://review.typo3.org/22664
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Stanislas Rolland
Tested-by: Stanislas Rolland
6 years ago[BUGFIX] Invalid allowed value for color space 23/23023/4
Tomita Militaru [Sat, 10 Aug 2013 16:29:12 +0000 (19:29 +0300)]
[BUGFIX] Invalid allowed value for color space

The array with allowed color space names contains an invalid value
(grey), which is replaced by the correct one (gray).

Resolves: #50287
Releases: 6.2, 6.1
Change-Id: I96a2a1c4b55fc32e95a68f0f8ff0304d8de7bc51
Reviewed-on: https://review.typo3.org/23023
Reviewed-by: Markus Klein
Tested-by: Markus Klein
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
6 years ago[FEATURE] Added autofocus attribute to textfield and textarea 94/21994/2
Tim Spiekerkoetter [Sat, 6 Jul 2013 10:35:35 +0000 (12:35 +0200)]
[FEATURE] Added autofocus attribute to textfield and textarea

The ViewHelpers f:form.textfield and f:form.textarea were lacking the
HTML5 "autofocus" attribute. It is available for button, input and
textarea. Now Fluid reflects this correctly by adding this attribute to
the initializeArguments methods for TextareaViewHelper and
TextfieldViewHelper.

Resolves: #48689
Releases: 6.2
Change-Id: I00480dc4e76b2854dc0be36d150d32c7b432d93d
Reviewed-on: https://review.typo3.org/21994
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
6 years ago[BUGFIX] Mixed up parameters in call to replaceMarkersInWhereClause 12/23012/3
Anja Leichsenring [Fri, 9 Aug 2013 15:01:47 +0000 (17:01 +0200)]
[BUGFIX] Mixed up parameters in call to replaceMarkersInWhereClause

The patch for #21892 breaks functionality due to a mixed up parameters
list in BackendUtility.

Resolves: #50976
Releases: 6.2
Change-Id: I46585d995ec345561f605a19202d91614ad5e8d8
Reviewed-on: https://review.typo3.org/23012
Reviewed-by: Philipp Gampe
Tested-by: Philipp Gampe
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
6 years ago[BUGFIX] Reports module tries to load not-installed extension 26/22726/4
Wouter Wolters [Tue, 30 Jul 2013 22:28:37 +0000 (00:28 +0200)]
[BUGFIX] Reports module tries to load not-installed extension

When you select a report from an extension, the reports module
will remember this when you re-open it. But if you have
uninstalled the extension in the meantime, the reports module
will throw an exception. This is fixed by checking the
$reportClass. If this is NULL then redirect to indexAction.

Change-Id: Ifad4e6ee8dc651a3d6b1360486fd5a7cb2abf111
Resolves: #48523
Releases: 6.2, 6.1, 6.0
Reviewed-on: https://review.typo3.org/22726
Reviewed-by: Stefan Neufeind
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring