Packages/TYPO3.CMS.git
5 years ago[BUGFIX] Add missing API method FileInterface::getNameWithoutExtension 54/23454/2
Ernesto Baschny [Tue, 6 Aug 2013 17:17:07 +0000 (19:17 +0200)]
[BUGFIX] Add missing API method FileInterface::getNameWithoutExtension

This method was supposed to be in the interface, but due to a syntax
error in the closing comment tag, it was not (and was not detected as a
PHP syntax error).

This finally adds this method and implements it also in the class
FileReference which hadn't had this method before but also implements
this interface.

Resolves: #51509
Releases: 6.2, 6.1, 6.0
Change-Id: I8ee1ecc155e9cfaeec7ac0ac7086f75df13f4c34
Reviewed-on: https://review.typo3.org/23454
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
5 years ago[BUGFIX] Exclude empty passwords from password hashing check 32/23432/2
Nicole Cordes [Wed, 28 Aug 2013 14:02:36 +0000 (16:02 +0200)]
[BUGFIX] Exclude empty passwords from password hashing check

The test for insecure passwords handles empty passwords as insecure and
recommends to use the saltedpasswords scheduler task to convert all
insecure passwords. But the scheduler task doesn't convert empty
passwords, so a never ending story exists. Therefore this patch exludes
empty passwords being handled as insecure.

Resolves: #36244
Releases: 6.2, 6.1, 6.0, 4.7, 4.5
Change-Id: I6bdd5c8807a07841850c1e4c22afe57eec28b335
Reviewed-on: https://review.typo3.org/23432
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
5 years ago[BUGFIX] utf8/multibyte character breaks menu filelist 78/23378/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/23378
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
5 years ago[TASK] Make the extension titles link to the configuration 76/23376/2
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/23376
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
5 years ago[BUGFIX] Hide translations in categories selector 58/23358/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
Reviewed-on: https://review.typo3.org/23358

5 years ago[BUGFIX] Tests in Localization\Parser\LocallangXmlParserTest fail 50/23350/2
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/23350
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
5 years ago[BUGFIX] Escape title tag of image links 44/23344/2
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/23344
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
5 years ago[BUGFIX] Page tree filtering broken in IE7 & IE8 37/23337/3
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/23337
Reviewed-by: Nicole Cordes
Tested-by: Nicole Cordes
5 years ago[BUGFIX] Ignore permission checks for processed files 14/23214/2
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/23214
Reviewed-by: Helmut Hummel
Tested-by: Helmut Hummel
5 years agoRevert "[BUGFIX] Check for incompatible SQL modes" 42/23242/2
Stefan Neufeind [Thu, 22 Aug 2013 10:18:58 +0000 (12:18 +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 e727d5ff0b8ee93de38983555164b19f6bf87377

Change-Id: I117f38df6796bf04c285d6d5345a0829f32fbfc3
Reviewed-on: https://review.typo3.org/23242
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
5 years ago[BUGFIX] Missing "Interface" in Backend-Annotation 40/23240/2
Stefan Froemken [Thu, 22 Aug 2013 09:53:22 +0000 (11:53 +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/23240
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
5 years ago[BUGFIX] Check for incompatible SQL modes 37/23237/2
Sascha Egerer [Thu, 22 Aug 2013 09:32:54 +0000 (11:32 +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/23237
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
5 years ago[BUGFIX] No version overlay should be done for sys_language 03/23203/2
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/23203
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
5 years ago[BUGFIX] Files with unclean path indexed multiple times 99/23199/2
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/23199
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
5 years ago[BUGFIX] FlashMessageQueue PHP strict warning 97/23197/2
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/23197
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
5 years ago[TASK] FilesContentObject::stdWrapValue(): only execute stdWrap once 79/23179/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/23179
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
5 years ago[BUGFIX] Language-module icons need to display in correct size 77/23177/2
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/23177
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
5 years ago[BUGFIX] TCA: subtypes_addlist not processed 59/23159/2
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/23159
Reviewed-by: Philipp Gampe
Tested-by: Philipp Gampe
5 years ago[BUGFIX] Query parameters of external link may get altered 57/23157/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/23157
Reviewed-by: Philipp Gampe
Tested-by: Philipp Gampe
5 years ago[TASK] Disable scheduler-tests if EXT:scheduler not loaded 34/23134/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/23134
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
5 years ago[BUGFIX] Fix file permission methods in BackendUserAuthentication 26/23126/2
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: I78c5fa13f2110e6044c0a81e2e7e4f3298766226
Reviewed-on: https://review.typo3.org/23126
Reviewed-by: Helmut Hummel
Tested-by: Helmut Hummel
5 years ago[BUGFIX] Fix inconsistencies in getTSConfig in BackenuserAuth 23/23123/2
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: Ie3daa0abc04a927186e9e9f3a88875086b606f93
Reviewed-on: https://review.typo3.org/23123
Reviewed-by: Helmut Hummel
Tested-by: Helmut Hummel
5 years agoRevert "[BUGFIX] Fix inconsistencies in getTSConfig in BackenuserAuth" 19/23119/2
Helmut Hummel [Fri, 16 Aug 2013 07:10:52 +0000 (09:10 +0200)]
Revert "[BUGFIX] Fix inconsistencies in getTSConfig in BackenuserAuth"

This reverts commit c7d04fbda9272f13b52fe57a2b39b8307ff14340

Defective cherry-pick from master causes failing builds on Travis

Change-Id: I46c14840d349188abb1a6ef9b158fbeffb67d2fc
Reviewed-on: https://review.typo3.org/23119
Reviewed-by: Helmut Hummel
Tested-by: Helmut Hummel
5 years agoRevert "[BUGFIX] Fix file permission methods in BackendUserAuthentication" 18/23118/2
Helmut Hummel [Fri, 16 Aug 2013 07:09:33 +0000 (09:09 +0200)]
Revert "[BUGFIX] Fix file permission methods in BackendUserAuthentication"

This reverts commit 01ac0075f41251557b5ddb24388508a927555ef6

Because of a missing dependency this caused failing builds on Travis

Change-Id: Iaedd4d9a251af1a8ca3c7b2fc9d56095d8f6928e
Reviewed-on: https://review.typo3.org/23118
Reviewed-by: Helmut Hummel
Tested-by: Helmut Hummel
5 years ago[BUGFIX] Fix inconsistencies in getTSConfig in BackenuserAuth 71/23071/3
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/23080
Reviewed-by: Helmut Hummel
Tested-by: Helmut Hummel
Reviewed-on: https://review.typo3.org/23071

5 years ago[BUGFIX] Fix file permission methods in BackendUserAuthentication 11/23111/3
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/23111
Reviewed-by: Helmut Hummel
Tested-by: Helmut Hummel
5 years ago[BUGFIX] Avoid usage of subheader in mailform 07/23107/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/23107
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
5 years ago[BUGFIX] storagePid is lost when persistence.recursive is set 89/23089/2
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/23089
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
5 years ago[BUGFIX] Typing after abbr or acronym tag is difficult 73/23073/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/23073
Reviewed-by: Stanislas Rolland
Tested-by: Stanislas Rolland
5 years ago[BUGFIX] Allow reading files if storage is not browsable 65/23065/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/23065
Reviewed-by: Helmut Hummel
Tested-by: Helmut Hummel
5 years ago[BUGFIX] Take into account all file and folder permissions 44/23044/2
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/23044
Reviewed-by: Helmut Hummel
Tested-by: Helmut Hummel
5 years ago[BUGFIX] Failing tests in Resource\Driver\LocalDriverTest on Windows 04/22904/2
Nicole Cordes [Tue, 6 Aug 2013 20:22:06 +0000 (22:22 +0200)]
[BUGFIX] Failing tests in Resource\Driver\LocalDriverTest on Windows

The test createdFilesAreEmpty depends on another test which returns a
mocked object. In the mock two returnValues for getIdentifier() and
getName() were set. But due to PHPUnit architecture mocked functions
belong to the test there were defined and cannot be shared. This is why
the mock has to be initialized within the test itself. The test only
works because on unix system you can open a folder which returns an
empty string.

Furthermore if you use any php function to get a folder path we have to
fix the containing backslashes.

Finally due to some errors we have to rewind any iterator of class
RecursiveIteratorIterator as otherwise a wrong index is returned.

Resolves: #50844
Releases: 6.2, 6.1, 6.0
Change-Id: I13f9efbd1618056fcd66220e3de5e12af79e4dc6
Reviewed-on: https://review.typo3.org/22904
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Jigal van Hemert
Tested-by: Jigal van Hemert
5 years ago[BUGFIX] Missing \TYPO3\CMS\Core\Utility\ in ResourceFactory 42/23042/2
Wouter Wolters [Sun, 11 Aug 2013 19:02:24 +0000 (21:02 +0200)]
[BUGFIX] Missing \TYPO3\CMS\Core\Utility\ in ResourceFactory

After merge of the new signal inside the ResourceFactory
the call to GeneralUtility is wrong.

Resolves: #51012
Releases: 6.1, 6.0
Change-Id: Ia76d26559a8cc3618bd4fcd11c750b3f1cd885c7
Reviewed-on: https://review.typo3.org/23042
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
5 years ago[TASK] Add signal in ResourceFactory for storage creation 38/23038/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/23038
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
5 years ago[BUGFIX] FAL: Image Processing doesn't respect GFX "thumbnails_png" 50/22850/3
Benjamin Mack [Fri, 19 Jul 2013 13:16:05 +0000 (15:16 +0200)]
[BUGFIX] FAL: Image Processing doesn't respect GFX "thumbnails_png"

The Task Interface of the File Abstraction Layer defines the
target file extension for graphcal tasks. Previously this was done
with the option "thumbnails_png" from the TYPO3_CONF_VARS.

However, checking for that parameter is not done anymore within
FAL/AbstractGraphicalTask. Thus, all .gif images are shown as .png
files, even though TYPO3 is not configured like that.

This can be seen when using GM, thumbnails_png=0, and creating
a preview image of an animated GIF. The preview will not be shown
for some images then.

The patch introduces the existing option back in to 6.x/FAL.

Releases: 6.2, 6.1, 6.0
Resolves: #50193
Change-Id: Ie539fe1aa1ab7366129a156be8ce9b05a5f8d5d2
Reviewed-on: https://review.typo3.org/22850
Reviewed-by: Anja Leichsenring
Tested-by: Marc Fell
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
5 years ago[BUGFIX] LocalDriver: Recursive file listing is broken 93/22193/3
Andreas Wolf [Tue, 29 Jan 2013 14:11:24 +0000 (15:11 +0100)]
[BUGFIX] LocalDriver: Recursive file listing is broken

The recursive file listing, introduced as part of the public API in
the fix for #43249, is currently broken.

One problem is that the file information retrieval is incomplete: The
filename of files in subfolders also contains the path to this file from
the current folder, while the identifier is missing that information.
The identifier is thus wrong and the filename contains too much
information (more than the filename).

Additionally, the method getDirectoryItemList() returns the file list
with the filenames as key, which will fail when a file name exists twice
in different folders. Therefore, this patch changes the keys to numeric
values when a recursive folder list is requested.

Change-Id: Iaebd862327d2dfc849044236474f6da2444cd4f5
Resolves: #44910
Releases: 6.2, 6.1, 6.0
Reviewed-on: https://review.typo3.org/22193
Reviewed-by: Nicole Cordes
Tested-by: Nicole Cordes
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
5 years ago[BUGFIX] rtehtmlarea acronym error with static_info_tables 6.0+ 28/23028/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/23028
Reviewed-by: Stanislas Rolland
Tested-by: Stanislas Rolland
5 years ago[BUGFIX] Invalid allowed value for color space 24/23024/2
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/23024
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
5 years ago[BUGFIX] Reports module tries to load not-installed extension 81/22981/2
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/22981
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
5 years ago[BUGFIX] number_format() expects parameter 1 to be double 74/22974/2
Wouter Wolters [Wed, 7 Aug 2013 17:46:54 +0000 (19:46 +0200)]
[BUGFIX] number_format() expects parameter 1 to be double

In stdWrap function numberFormat $content is a string.
Make sure this is a float value when it is passed into
PHP function number_format.

Change-Id: If4a98ae8238264f94f771c997fbe6cac98311912
Resolves: #50868
Releases: 6.2, 6.1, 6.0, 4.7, 4.5
Reviewed-on: https://review.typo3.org/22974
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
5 years ago[BUGFIX] Ignore case in file extension filter 37/22937/2
Alexander Stehlik [Sat, 3 Aug 2013 18:14:10 +0000 (20:14 +0200)]
[BUGFIX] Ignore case in file extension filter

This patch makes the FileExtensionFilter case insensitive.

If the filter is configured to allow files with the ".EXT"
extension it will now also allow files with an ".ext"
extension.

Additionally the order of a data provider method in the
FileExtensionFilterTest has been corrected.

Resolves: #50568
Releases: 6.2, 6.1, 6.0
Change-Id: I5761ab7f06116ef847caeb2ae786e580730c0e6b
Reviewed-on: https://review.typo3.org/22937
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
5 years ago[BUGFIX] Correctly set user storage permissions 30/22930/2
Helmut Hummel [Wed, 7 Aug 2013 11:00:39 +0000 (13:00 +0200)]
[BUGFIX] Correctly set user storage permissions

With FAL a new way to define file permissions
(for storages) has been introduced. You can set
these in User TsConfig instead of doing it
in the user record.

There is a backwards compatibility layer, which should
set the permissions according to the database entry
in case no permissions have been set in TsConfig.
This however does not work, because of a wrong check
for existence of UserTsConfig properties.

Instead of checking for an array (which is always the case),
we have to check if the array is not empty.

Releases: 6.0, 6.1, 6.2
Resolves: #50872
Change-Id: I0ec30a1ae96c4b6916a3e46c7222f832c44f603e
Reviewed-on: https://review.typo3.org/22930
Reviewed-by: Helmut Hummel
Tested-by: Helmut Hummel
5 years ago[TASK] Introduce AbstractHierarchicalFilesystemDriver 19/22919/2
Steffen Ritter [Wed, 7 Aug 2013 09:09:55 +0000 (11:09 +0200)]
[TASK] Introduce AbstractHierarchicalFilesystemDriver

There are many tasks which apply to all drivers working on an
Hierarchical FileSystem (NFS, WebDav, LocalDriver, ....).
When it comes to security we dont want custom drivers to implement
that all on their own, but provide it by the System.

Introducing AbstractHierarchicalFilesystemDriver enables us to
do so, without breaking compatibility (Base class, interface
and Classname stay the same).

Releases: 6.2, 6.1, 6.0
Resolves: #50867
Change-Id: Ib1b45c6ad384f82f7a5138372ed91269b8db38c2
Reviewed-on: https://review.typo3.org/22919
Reviewed-by: Helmut Hummel
Tested-by: Helmut Hummel
5 years ago[BUGFIX] Failing Resource\FactoryTest on Windows systems 12/22912/2
Nicole Cordes [Tue, 6 Aug 2013 20:00:25 +0000 (22:00 +0200)]
[BUGFIX] Failing Resource\FactoryTest on Windows systems

Two tests in class TYPO3\CMS\Core\Tests\Unit\Resource\FactoryTest fail
with warnings. This is because of two single issues.

First of all the test depends on a folder which (might) belong to an own
storage other than storage 0 (fileadmin). This patch changes fileadmin
to the typo3 folder as this one should be not included in any storage.

On the other hand on Windows systems a full path contains always a colon
which breaks any storage check. To solve this we have to fix any
absolute path beforehand as it is done in the sub functions as well.

Resolves: #50843
Releases: 6.2, 6.1, 6.0
Change-Id: I950a3c81222155da403ca1eb7b920e8682033450
Reviewed-on: https://review.typo3.org/22912
Reviewed-by: Nicole Cordes
Tested-by: Nicole Cordes
5 years ago[BUGFIX] Indexing of external files does not work in indexed_search 06/22906/3
Wouter Wolters [Fri, 26 Jul 2013 15:35:17 +0000 (17:35 +0200)]
[BUGFIX] Indexing of external files does not work in indexed_search

After the namespace change a wrong replace was done in
indexed_search. Replace it with the correct class.

Change-Id: I35295f4fecde241ec3004a8ee6b7dd71d5e34e93
Resolves: #47106
Releases: 6.2, 6.1, 6.0
Reviewed-on: https://review.typo3.org/22906
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
5 years ago[BUGFIX] Callback in CrawlerHook of indexed_search sysext buggy 05/22905/3
Marius Büscher [Wed, 31 Jul 2013 08:45:07 +0000 (10:45 +0200)]
[BUGFIX] Callback in CrawlerHook of indexed_search sysext buggy

The property $callBack pointed to a Controller that was not existing.
Now it points back to the Hook.

Resolves: #50562
Releases: 6.2, 6.1, 6.0
Change-Id: I79e0ce73a23d98c5922ead43480202b3445282b2
Reviewed-on: https://review.typo3.org/22905
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
5 years ago[BUGFIX] Backup singletons in unit tests prior to other setUp operations 99/22899/2
Nicole Cordes [Tue, 6 Aug 2013 10:22:51 +0000 (12:22 +0200)]
[BUGFIX] Backup singletons in unit tests prior to other setUp operations

If the singleton management is changed in unit tests, a local copy of
the previous stage is done to re-construct this state in tearDown().
This has to be done before any other instantiation in setUp() is
initiated which might register additional singletons.

Resolves: #50812
Releases: 6.2, 6.1, 6.0
Change-Id: I7788a5788aca6777040b777aeceb0558f96022b2
Reviewed-on: https://review.typo3.org/22899
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
5 years ago[BUGFIX] Test in Configuration\ConfigurationManagerTest fails in backend 98/22898/2
Nicole Cordes [Tue, 6 Aug 2013 20:53:37 +0000 (22:53 +0200)]
[BUGFIX] Test in Configuration\ConfigurationManagerTest fails in backend

When running the tests in the PHPUnit backend, the path of the current
working directory is typo3/ (where mod.php is included). Therefore the
test canWriteConfigurationReturnsTrueIfDirectoryAndFilesAreWritable
fails as it deals with paths relative to the root directory. We have to
use absolute paths to ensure files in typo3temp folder exist.

Resolves: #50845
Releases: 6.2, 6.1
Change-Id: I7859978c31cf523c8803316fdac8c067d00042cc
Reviewed-on: https://review.typo3.org/22898
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
5 years ago[BUGFIX] Fix EmConfUtility::fixEmConf conflicts generation 91/22891/2
Sascha Egerer [Thu, 1 Aug 2013 09:46:42 +0000 (11:46 +0200)]
[BUGFIX] Fix EmConfUtility::fixEmConf conflicts generation

EmConfUtility::fixEmConf generates a string instead
of an array for "$emConf['constraints']['conflicts']"

Resolves: #50628
Releases: 6.2, 6.1, 6.0
Change-Id: I37e026569e761b8550a9b0e6a1cb10835f6c899b
Reviewed-on: https://review.typo3.org/22891
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
5 years ago[BUGFIX] Incorrect check for empty folder 88/22888/2
Philipp Gampe [Mon, 29 Jul 2013 20:15:32 +0000 (22:15 +0200)]
[BUGFIX] Incorrect check for empty folder

The condition in ResourceStorage::deleteFolder is wrong.
Negate the isFolderEmpty() clause.

Resolves: #50125
Releases: 6.2, 6.1, 6.0
Change-Id: Ia60f3e97ff8d1e0cc2671cec1b31cfc995c235b1
Reviewed-on: https://review.typo3.org/22888
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
5 years ago[BUGFIX] Use proper implementation of a wordwrap 90/22890/2
Georg Ringer [Tue, 6 Aug 2013 12:17:09 +0000 (14:17 +0200)]
[BUGFIX] Use proper implementation of a wordwrap

The page module uses a custom implementation of a wordwrap
as this was not supported by the browser in those earlier days.

The big drawback is that there is no UTF8 support for this which
leads to empty preview boxes when e.g. using Japanese.

The 6.1 version of this patch changes the wordWrapper() usage only
and does not deprecate the method.

Change-Id: I7caab8e14a9a0b7ccbd38c692b0cf98635967567
Resolves: #15797
Releases: 6.2, 6.1
Reviewed-on: https://review.typo3.org/22890
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
5 years ago[TASK] Use magic __CLASS__ in getInstance()-methods 86/22886/2
Stefan Neufeind [Thu, 1 Aug 2013 01:20:23 +0000 (03:20 +0200)]
[TASK] Use magic __CLASS__ in getInstance()-methods

Some classes contain static getInstance()-methods that return
an instance of themselves. Instead of using the correct namespace
and classname everywhere, just use __CLASS__.

Change-Id: I22770505acded0c54b07dd0e239976ad01fdf893
Resolves: #50615
Releases: 6.2, 6.1, 6.0
Reviewed-on: https://review.typo3.org/22886
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
5 years ago[BUGFIX] Fix empty href parameter 33/22833/2
Anja Leichsenring [Sat, 3 Aug 2013 13:38:46 +0000 (15:38 +0200)]
[BUGFIX] Fix empty href parameter

In case the link target is not available (maybe hidden), the behaviour
known from the Core is to render the link text, but no a tag. The pageVH
renders the link, but the href parameter will be empty.

Resolves: #50751
Releases: 6.2, 6.1, 6.0
Change-Id: I526aa0e3d42f8aba867b3f996bac2366dc1f6d19
Reviewed-on: https://review.typo3.org/22833
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
5 years ago[BUGFIX] MySQLi: Allow persistent connections again 71/22871/2
Stefan Neufeind [Mon, 5 Aug 2013 08:31:20 +0000 (10:31 +0200)]
[BUGFIX] MySQLi: Allow persistent connections again

For a persistent connection the hostname needs to be
prefixed with "p:". This was already prepared in the
code, however that variable was not yet used.

Change-Id: I01e20fd69239d0772ca9b371af0058316541f2a9
Resolves: #50781
Releases: 6.2, 6.1
Reviewed-on: https://review.typo3.org/22813
Reviewed-by: Philipp Gampe
Tested-by: Stefan Neufeind
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
(cherry picked from commit ac1bc673090986210794b4b2790f295e64a227f8)
Reviewed-on: https://review.typo3.org/22871

5 years ago[BUGFIX] Fix failing test in StorageRepositoryTest 41/22841/3
Anja Leichsenring [Tue, 6 Aug 2013 08:18:00 +0000 (10:18 +0200)]
[BUGFIX] Fix failing test in StorageRepositoryTest

With #50803 the usage of the constant TYPO3_MODE was introduced. This
leads to the failing test, after the constant can not be redefined
for test purposes.

To work around this, a method that can be mocked is introduced, and
specifically set to "FE" for this test to work again.

Resolves: #50809
Relates: #50803
Releases: 6.2, 6.1, 6.0
Change-Id: I7319d8ccef530feff69a7f9885df24dba0beed4d
Reviewed-on: https://review.typo3.org/22841
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
5 years ago[BUGFIX] Fatal error: "enableFields on non-object" in extension manager 31/22831/2
Ernesto Baschny [Mon, 5 Aug 2013 20:28:05 +0000 (22:28 +0200)]
[BUGFIX] Fatal error: "enableFields on non-object" in extension manager

Don't rely on $TSFE object to detect if we are in a frontend context. Rely
on TYPO3_MODE constant instead.

Releases: 6.2, 6.1, 6.0
Resolves: #50803
Change-Id: I06ea2692dab7683c7284e5f3ff45036d72c64999
Reviewed-on: https://review.typo3.org/22831
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
5 years ago[BUGFIX] MySQL: Use ENGINE (not TYPE) for storage-engine 92/22792/3
Stefan Neufeind [Fri, 26 Jul 2013 13:19:35 +0000 (15:19 +0200)]
[BUGFIX] MySQL: Use ENGINE (not TYPE) for storage-engine

Using TYPE is deprecated since a MySQL 4.x and was removed in MySQL
5.5. Core already uses ENGINE but SqlParser turned that into TYPE,
leading to errors.

From https://dev.mysql.com/doc/refman/5.0/en/storage-engines.html:
"The older term TYPE is supported as a synonym for ENGINE for
backward compatibility, but ENGINE is the preferred term and TYPE
is deprecated."

Change-Id: I6607d3e726c43cb74ca00f33ec2332de4f6a76d8
Resolves: #50466
Releases: 6.2, 6.1, 6.0, 4.7, 4.5
(cherry picked from commit a7b47c61008c6ca484a1ba2f252792dc24557559)
Reviewed-on: https://review.typo3.org/22792
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
5 years ago[BUGFIX] ControllerExtensionName can not be resolved in CLI mode 94/22794/2
Achim Fritz [Thu, 18 Jul 2013 18:35:59 +0000 (20:35 +0200)]
[BUGFIX] ControllerExtensionName can not be resolved in CLI mode

This fixes the resolution of the controllerExtensionName in CLI
context.

Change-Id: Ic0363a4e1b055aff813ddf8620403df5a4fa872f
Resolves: #49288
Releases: 6.2, 6.1
Reviewed-on: https://review.typo3.org/22794
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
5 years ago[BUGFIX] selected = 1 doesn't work in FormContentObject 51/22751/2
Wouter Wolters [Sun, 28 Apr 2013 22:01:20 +0000 (00:01 +0200)]
[BUGFIX] selected = 1 doesn't work in FormContentObject

Wrong variable is used to set selected correctly.

Change-Id: I90623339c4d13eb07d7e5ef87e1b4e3aa8cf147e
Resolves: #43893
Releases: 6.2, 6.1, 6.0
Reviewed-on: https://review.typo3.org/20282
Reviewed-by: uwe trotzek
Tested-by: uwe trotzek
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
(cherry picked from commit 8d30ece2f0dfc0306c22a63d9e57a75e32d1365e)
Reviewed-on: https://review.typo3.org/22751

5 years ago[TASK] Provide information about import action in TCEmain to hooks 42/22642/3
Stefan Galinski [Tue, 27 Nov 2012 01:31:50 +0000 (02:31 +0100)]
[TASK] Provide information about import action in TCEmain to hooks

While importing T3D files the TCEmain currently fires the required
processing on records just as if the editor is generating the records. If
extensions hook into TCEmain, there is no way to differenciate this
situation from the regular editing mode - which might provide unexpected
results, i.e. wrong IRRE relations.

Solution is to provide a boolean to indicate that TCEmain has been called
during an import action which extensions could use to recognize this
situation.

Change-Id: I4fe429fb88f8ec24a7fda885f38ddfbdb4a642f5
Resolves: #36031
Releases: 4.5, 4.7, 6.0, 6.1, 6.2
Reviewed-on: https://review.typo3.org/22642
Reviewed-by: Jo Hasenau
Tested-by: Jo Hasenau
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
5 years ago[BUGFIX] RTE wizard can't "save document and view page" 46/22646/2
Stanislas Rolland [Tue, 23 Apr 2013 15:47:46 +0000 (11:47 -0400)]
[BUGFIX] RTE wizard can't "save document and view page"

Including the full fieldConfig from TCA may produce too long an URL.
In IE, the RTE wizard simply does not open.

Solution: Remove the fieldConfig parameter not used by the RTE wizard.

Revolves: #43637
Resolves: #43631
Releases: 4.5, 4.7, 6.0, 6.1, 6.2
Change-Id: Idd855d56d6a7ed92d40d9a788bc11efb3f535ed3
Reviewed-on: https://review.typo3.org/22646
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
5 years ago[TASK] Set TYPO3 version to 6.1.4-dev 20/22720/2
TYPO3 Release Team [Tue, 30 Jul 2013 13:09:29 +0000 (15:09 +0200)]
[TASK] Set TYPO3 version to 6.1.4-dev

Change-Id: I96e4770c075c082efce159fef19c250d408df5f7
Reviewed-on: https://review.typo3.org/22720
Reviewed-by: TYPO3 Release Team
Tested-by: TYPO3 Release Team
5 years ago[RELEASE] Release of TYPO3 6.1.3 19/22719/2 TYPO3_6-1-3
TYPO3 Release Team [Tue, 30 Jul 2013 13:09:00 +0000 (15:09 +0200)]
[RELEASE] Release of TYPO3 6.1.3

Change-Id: I403489edc0c4f0a4c989befb29753b84b7b3d3cf
Reviewed-on: https://review.typo3.org/22719
Reviewed-by: TYPO3 Release Team
Tested-by: TYPO3 Release Team
5 years ago[SECURITY] fileDenyPattern ignored in file-list module 09/22709/2
Jigal van Hemert [Tue, 30 Jul 2013 12:39:27 +0000 (14:39 +0200)]
[SECURITY] fileDenyPattern ignored in file-list module

Change-Id: Icf4b3452c3b039f12335e184161083cbcbd3acec
Fixes: #47452
Releases: 6.0, 6.1, 6.2
Security-Commit: 71fa96798f6d3b85b1ad18f1dca4f86247b9d683
Security-Bulletin: TYPO3-CORE-SA-2013-002
Reviewed-on: https://review.typo3.org/22709
Reviewed-by: Oliver Hader
Tested-by: Oliver Hader
5 years ago[SECURITY] XSS in 3rd party library Flowplayer 08/22708/2
Oliver Hader [Tue, 30 Jul 2013 12:39:14 +0000 (14:39 +0200)]
[SECURITY] XSS in 3rd party library Flowplayer

Change-Id: I6bb53b23ca4df6f8d4b8ee801bd99a1a5c32f5b2
Fixes: #49209
Releases: 4.7, 6.0, 6.1, 6.2
Security-Commit: ee100f7fef667c77b2bb8278d1e4de14d3114285
Security-Bulletin: TYPO3-CORE-SA-2013-002
Reviewed-on: https://review.typo3.org/22708
Reviewed-by: Oliver Hader
Tested-by: Oliver Hader
5 years ago[SECURITY] XSS in 3rd party library Audio Player 07/22707/2
Oliver Hader [Tue, 30 Jul 2013 12:39:06 +0000 (14:39 +0200)]
[SECURITY] XSS in 3rd party library Audio Player

Update player.swf of version 2.0.4.6 from
http://wordpress.org/plugins/audio-player/

Change-Id: Ib9ba8c7f02279c42d619154b9b79cc38a8e2c41b
Fixes: #49210
Releases: 4.5, 4.7, 6.0, 6.1, 6.2
Security-Commit: f580e8e7182e8d2e53aed7889c9f37b5104874f4
Security-Bulletin: TYPO3-CORE-SA-2013-002
Reviewed-on: https://review.typo3.org/22707
Reviewed-by: Oliver Hader
Tested-by: Oliver Hader
5 years ago[BUGFIX] Deleted flag is not updated during file indexing 94/22694/2
Oliver Hader [Mon, 29 Jul 2013 14:31:44 +0000 (16:31 +0200)]
[BUGFIX] Deleted flag is not updated during file indexing

If sys_file records (file objects) have set the deleted flag, it
won't be removed once the file is physically available again
with the same file name (identifier).

Change-Id: Id6dc7466764ac3994606204edadc9c2a7d2106c8
Fixes: #50525
Releases: 6.0, 6.1, 6.2
Reviewed-on: https://review.typo3.org/22694
Reviewed-by: Oliver Hader
Tested-by: Oliver Hader
5 years ago[BUGFIX] MailUtility breakLinesForEmail cuts text wrong and discard rest 77/22677/2
Tim Lochmueller [Fri, 5 Jul 2013 21:26:50 +0000 (23:26 +0200)]
[BUGFIX] MailUtility breakLinesForEmail cuts text wrong and discard rest

Now the substrStart pointer increases in case that the line is longer
than the lineWidth parameter.

Resolves: #49396
Releases: 6.2, 6.1, 6.0
Change-Id: I87b3622c2ada7efdcb4c1992c1cfa43635d68f09
Reviewed-on: https://review.typo3.org/22677
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
5 years agoRevert "[BUGFIX] setRespectSysLanguage(FALSE) doesn't prevent language overlay" 71/22671/2
Anja Leichsenring [Mon, 29 Jul 2013 19:11:37 +0000 (21:11 +0200)]
Revert "[BUGFIX] setRespectSysLanguage(FALSE) doesn't prevent language overlay"

This reverts commit 58c9017fbdeb9108c15e80de4fb726c1f9273b8b

The patch introduces a regression. We have no easy fix for it, so the patch gets reverted. Sorry for now, we come back to the problem.

Change-Id: I8b608146a72cd48988c77ecf1036ac85074e03b0
Reviewed-on: https://review.typo3.org/22671
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
5 years agoRevert "[BUGFIX] sys_file record doesn't get flagged as delete" 66/22666/3
Oliver Hader [Mon, 29 Jul 2013 16:39:53 +0000 (18:39 +0200)]
Revert "[BUGFIX] sys_file record doesn't get flagged as delete"

This reverts commit 640e4d51238cea1315ec40377214f4d533d9b7d0

Change-Id: I2cd6c4a03c3336e73bfe81fa86f222ce5a352f7d
Change-Id: I9c0369ca4d93bd2f2a078c008219136e39037d60
Reviewed-on: https://review.typo3.org/22666
Reviewed-by: Oliver Hader
Tested-by: Oliver Hader
5 years ago[BUGFIX] Use correct url schema in MediaWizardProvider 76/22376/2
Georg Ringer [Tue, 16 Jul 2013 05:00:18 +0000 (07:00 +0200)]
[BUGFIX] Use correct url schema in MediaWizardProvider

If the media provider supports https, the video should be
embedded using https instead of http.

Change-Id: Id6f1677acfd82ba071c2425f345ea20b48de9cfe
Resolves: #50041
Releases: 6.2, 6.1, 6.0
Reviewed-on: https://review.typo3.org/22376
Reviewed-by: Georg Ringer
Tested-by: Georg Ringer
5 years ago[BUGFIX] Workspace selector in top toolbar cannot scroll 52/22652/2
Georg Ringer [Fri, 5 Jul 2013 09:41:09 +0000 (11:41 +0200)]
[BUGFIX] Workspace selector in top toolbar cannot scroll

If there are too many workspaces defined, some of them cannot
be accessed using the workspace selector in in the top toolbar
due to a missing scrolling behaviour of that component.

Fixes: #11014
Releases: 6.2, 6.1, 6.0
Change-Id: Iaae51ebb2524116b8673a9cc81d612fd845b2d29
Reviewed-on: https://review.typo3.org/22652
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
5 years ago[BUGFIX] Detection of curlProxyServer settings is incorrect 50/22650/2
Dmitry Dulepov [Tue, 26 Feb 2013 13:53:17 +0000 (17:53 +0400)]
[BUGFIX] Detection of curlProxyServer settings is incorrect

Parsing of 'curlProxyServer' option during the bootstrap is incorrect.
It produces wrong host and port part in 'proxy_host' and 'proxy_port'
system options.

Change-Id: I12f66acef697dbec9b065edcba8ad0d8f78c4e01
Resolves: #45834
Releases: 6.2, 6.1, 6.0
Reviewed-on: https://review.typo3.org/22650
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
5 years ago[BUGFIX] Class loader tries to load classes it cannot load 28/22628/2
Andreas Wolf [Sat, 20 Jul 2013 10:19:13 +0000 (12:19 +0200)]
[BUGFIX] Class loader tries to load classes it cannot load

If a class file from a third-party package is loaded which does not
follow the Vendor\Product\Classname convention (e.g. Product\Classname),
the class loader still tries to access the third component of the class
name, leading to a warning (and thus an exception, depending on your
configuration).

To fix this problem, the class loader now checks in advance if it can
possibly load this class (or rather if the classname conforms to the
specification for the CMS universe).

Change-Id: I1cd3964b9486ea591fb373533fd6cef4e9ce3989
Resolves: #50217
Releases: 6.2, 6.1, 6.0
Reviewed-on: https://review.typo3.org/22628
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
5 years ago[TASK] Add tests for ContentObjectRenderer::getData() 39/22639/3
Stefan Neufeind [Sun, 28 Jul 2013 09:18:49 +0000 (11:18 +0200)]
[TASK] Add tests for ContentObjectRenderer::getData()

* Adds unittests

Change-Id: I2b4ff89c828384c852e6fbf9f82c55e6ebbac281
Resolves: #50258
Releases: 6.2, 6.1, 6.0
Reviewed-on: https://review.typo3.org/22639
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
5 years ago[BUGFIX] Debug exception handler: set exit code on CLI 30/22630/2
Christian Weiske [Wed, 19 Dec 2012 13:50:40 +0000 (14:50 +0100)]
[BUGFIX] Debug exception handler: set exit code on CLI

Sets the exit code to 1 when an exception is caught by the debug exception
handler, so that the error state can be checked automatically.

Change-Id: Ic89a79f2ded0b4b9ccc58a4789d9416f260fd88a
Fixes: #44118
Releases: 6.2, 6.1, 6.0
Reviewed-on: https://review.typo3.org/22630
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
5 years ago[TASK] Run phpLint and phpUnit in different travis builds 25/22625/2
Christian Kuhn [Sat, 27 Jul 2013 18:15:05 +0000 (20:15 +0200)]
[TASK] Run phpLint and phpUnit in different travis builds

phpLint checks php files for compilation errors in travis before
the unit tests are executed. This takes quite some time.
The patch configures travis to run phpLint and phpUnit in different
builds to speed up the process. Additionally, phpLint environment
does not clone phpunit and the introduction package and does no
database initialization.

Change-Id: Ie25ade5b6a8fcf4c4dc0752fc8662c433ea3c33c
Resolves: #50492
Releases: 6.2, 6.1, 6.0
Reviewed-on: https://review.typo3.org/22624
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Susanne Moog
Tested-by: Susanne Moog
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
(cherry picked from commit cae1c18432a6306a28590605f5e5a70be8a8c63d)
Reviewed-on: https://review.typo3.org/22625

5 years ago[BUGFIX] Make BackendUtility::viewOnClick honor doc 20/22620/2
Kasper Ligaard [Fri, 21 Sep 2012 12:27:28 +0000 (14:27 +0200)]
[BUGFIX] Make BackendUtility::viewOnClick honor doc

Make parameter $alternativeUrl work as documented in the docblock
and also as it worked until #22157.
This patch preserves the hook and basically just puts in an
if-statement. Also adding a unit test.

Change-Id: I7d67af438b0a0af51ca7662b3f8286aeec827c77
Fixes: #41165
Releases: 6.2, 6.1, 6.0
Reviewed-on: https://review.typo3.org/22620
Reviewed-by: Markus Klein
Tested-by: Markus Klein
5 years ago[TASK] Remove empty .gitmodules file 14/22614/2
Christian Kuhn [Fri, 26 Jul 2013 21:02:46 +0000 (23:02 +0200)]
[TASK] Remove empty .gitmodules file

With removal of git submodules the .gitmodules file is empty
and can be removed now.

Change-Id: I453f63c0852825b46e9ea62f87385d9eace25134
Resolves: #50480
Releases: 6.2, 6.1, 6.0, 4.7, 4.5
Reviewed-on: https://review.typo3.org/22613
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
(cherry picked from commit 643f2e9cef2e7997e2c7ae5b8d8bbf279d9983c1)
Reviewed-on: https://review.typo3.org/22614

5 years ago[BUGFIX] Failures in em unit tests due to phpunit update 09/22609/2
Christian Kuhn [Fri, 26 Jul 2013 19:30:43 +0000 (21:30 +0200)]
[BUGFIX] Failures in em unit tests due to phpunit update

Young phpunit versions seem to handle clones a bit differently,
resulting in test failures in extension manager tests. The tests
are adapted a bit to fix this.

Change-Id: If70561743f68a62167e41ee4c23ae96b56f83ab9
Resolves: #50478
Releases: 6.2, 6.1, 6.0
Reviewed-on: https://review.typo3.org/22604
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
(cherry picked from commit eab10d590ceacc7d256a8ea195a2e562099fb52d)
Reviewed-on: https://review.typo3.org/22609
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
5 years ago[BUGFIX] FAL wrong \RecursiveIteratorIterator usage 07/22607/2
Christian Kuhn [Fri, 26 Jul 2013 18:54:23 +0000 (20:54 +0200)]
[BUGFIX] FAL wrong \RecursiveIteratorIterator usage

If using \RecursiveIteratorIterator on \RecursiveDirectoryIterator,
flag SELF_FIRST must be used, otherwise only leaves (files) and not
also directories are returned. This behavior is a bit dependent on
the PHP version used, younger versions (> 5.3.3 seem to do it
correctly).

Furthermore, in copyFolderWithinStorage() $itemSubPath contains
the "relative" path part and not only the node name, checking
equality to "." or ".." fails then if handling sub directories.
So, directories may be created multiple times, resulting in mkdir()
warnings. This is fixed by getting the node name itself and using
that for comparison.

Resolves: #50476
Releases: 6.2, 6.1, 6.0

Change-Id: Ifac4b28f4e02ec8a9bf5891739cf66c1e1074af9
Reviewed-on: https://review.typo3.org/22602
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Anja Leichsenring
Tested-by: Anja Leichsenring
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
(cherry picked from commit ef8204d64f44c4643482c7e6a01313a1b702ca43)
Reviewed-on: https://review.typo3.org/22607

5 years ago[BUGFIX] FAL does not copy subfolders cleanly 05/22605/2
Christian Kuhn [Fri, 26 Jul 2013 15:43:19 +0000 (17:43 +0200)]
[BUGFIX] FAL does not copy subfolders cleanly

If full structures with sub folders are copied around in FAL
local driver, sub folders are not correctly created in the
target directory.

This is detected in unit tests after recent vfsStream updates in
ext:phpunit.

The patch adds a flag to the Iterator code to also include
directories to fix the issue.

Additionally, the existing two unit tests also tested multiple other
methods of the local driver and are functional tests. The patch moves
those two tests to the functional section and adds three simple unit
tests with correct mocking instead.

Change-Id: I30d3c8bd5c6538b3fe23405d97bcddff6c1ff405
Releases: 6.2, 6.1, 6.0
Resolves: #50472
Reviewed-on: https://review.typo3.org/22596
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
(cherry picked from commit b0f8bb0d15106b6f3ee2c5b3b033cbc3e1b635e2)
Reviewed-on: https://review.typo3.org/22605

5 years ago[BUGFIX] Fix failing test 85/22585/2
Anja Leichsenring [Fri, 26 Jul 2013 09:21:45 +0000 (11:21 +0200)]
[BUGFIX] Fix failing test

The test failes due to incorrect mocking, what leads to not claimed
properties for the test.
Additionally the expected call amounts where handled more strictly
where fitting.

Releases: 6.2, 6.1, 6.0
Resolves: #50458
Change-Id: I34214fb9ef9e9935892aa03518505b877538e128
Reviewed-on: https://review.typo3.org/22585
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
5 years ago[BUGFIX] Uninstall extension with dependency throws Exception 78/22578/2
Wouter Wolters [Tue, 9 Jul 2013 16:49:02 +0000 (18:49 +0200)]
[BUGFIX] Uninstall extension with dependency throws Exception

Exception is thrown if uninstalling an extension with
dependecies on that extension. Catch the exception and
show a nice Flashmessage.

Change-Id: I434a4e1e1d83f43a6f5d4c99a82433e35ddd92f9
Resolves: #49722
Releases: 6.2, 6.1, 6.0
Reviewed-on: https://review.typo3.org/22578
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
5 years ago[BUGFIX] Test regression from patch for #47192 75/22575/2
Andreas Wolf [Sat, 20 Jul 2013 21:14:22 +0000 (23:14 +0200)]
[BUGFIX] Test regression from patch for #47192

This fixes a unit test regression introduced with the solution for
 #47192.

Change-Id: I0e70b916f24fada7f76e7fd835d31e54ca41a402
Related: #47192
Releases: 6.2, 6.1, 6.0
Reviewed-on: https://review.typo3.org/22461
Reviewed-on: https://review.typo3.org/22575
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
5 years ago[BUGFIX] Improve method annotation and type hinting in FAL 73/22573/2
Christian Kuhn [Thu, 25 Jul 2013 20:53:04 +0000 (22:53 +0200)]
[BUGFIX] Improve method annotation and type hinting in FAL

Add a missing "@throws" annotation and a namespaced comment.

Change-Id: Ibc929a01dbfdef9cf8730a8da259570786b6118c
Resolves: #50447
Releases: 6.2, 6.1, 6.0
Reviewed-on: https://review.typo3.org/22572
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
(cherry picked from commit 3a23fb06d51fda287a78745a133f293fe8a9db77)
Reviewed-on: https://review.typo3.org/22573

5 years ago[BUGFIX] rsaauth BackendFactory does not unset backend 70/22570/2
Markus Klein [Wed, 24 Jul 2013 18:12:42 +0000 (20:12 +0200)]
[BUGFIX] rsaauth BackendFactory does not unset backend

The BackendFactory of rsaauth tries to forcefully unset an
instantiated backend, but unsets the class name instead of the object.

Resolves: #50411
Releases: 6.2, 6.1, 6.0
Change-Id: I26712c73b74788cad159f95c7e54cbbbf69e8764
Reviewed-on: https://review.typo3.org/22570
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
5 years ago[BUGFIX] require_once of vfsStream triggers travis fail 67/22567/2
Christian Kuhn [Thu, 25 Jul 2013 18:42:59 +0000 (20:42 +0200)]
[BUGFIX] require_once of vfsStream triggers travis fail

vfsStream is now included in ext:phpunit, the pear module is
not maintained and was removed in travis.
This triggeres a failure in the phplint check. All require_once
calls are removed for now, ext:phpunit now also provides an
autoloader for vfsSstream.

Change-Id: If1b07c31e999111e6b5d2804a9f5cc457066b7b0
Resolves: #50442
Releases: 6.2, 6.1, 6.0
Reviewed-on: https://review.typo3.org/22566
Reviewed-by: Jigal van Hemert
Tested-by: Jigal van Hemert
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
(cherry picked from commit 5ebcdc361e0efc5b1119df3bba0c46a948cf3c67)
Reviewed-on: https://review.typo3.org/22567

5 years ago[TASK] Revise typo3/sysext/README.txt 56/22556/2
Wouter Wolters [Wed, 24 Jul 2013 18:00:47 +0000 (20:00 +0200)]
[TASK] Revise typo3/sysext/README.txt

* Replace GIT repository link with the new one
* Remove information about submodules

Change-Id: I60db762ce831c7edea413d9f7780a5617b7f9e96
Resolves: #50410
Releases: 6.2, 6.1, 6.0, 4.7, 4.5
Reviewed-on: https://review.typo3.org/22556
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
5 years ago[TASK] Set TYPO3 version to 6.1.3-dev 18/22518/2
TYPO3 Release Team [Tue, 23 Jul 2013 17:14:09 +0000 (19:14 +0200)]
[TASK] Set TYPO3 version to 6.1.3-dev

Change-Id: I59ed376c317bbda9a43b7a377587dcce3907c714
Reviewed-on: https://review.typo3.org/22518
Reviewed-by: TYPO3 Release Team
Tested-by: TYPO3 Release Team
5 years ago[RELEASE] Release of TYPO3 6.1.2 17/22517/2 TYPO3_6-1-2
TYPO3 Release Team [Tue, 23 Jul 2013 17:13:42 +0000 (19:13 +0200)]
[RELEASE] Release of TYPO3 6.1.2

Change-Id: I488700726d337e3e19e979a52a86ac9c1ab8084a
Reviewed-on: https://review.typo3.org/22517
Reviewed-by: TYPO3 Release Team
Tested-by: TYPO3 Release Team
5 years ago[TASK] Make DiffUtility faster and much more scaleable. 09/22509/2
Kasper Ligaard [Mon, 22 Jul 2013 16:19:49 +0000 (18:19 +0200)]
[TASK] Make DiffUtility faster and much more scaleable.

The method DiffUtility->explodeStringIntoWords() calls array_merge()
inside a loop. The cost of constantly doing a function invocation and
merging arrays needlessly slows the method and the whole diff utility.
Moving the array merging outside the loop, the method becomes faster
and able to handle much larger input.

Change-Id: I4c2d21cff9b18392e89e316fec9dc615e715a073
Resolves: #50318
Releases: 6.2, 6.1, 6.0
Reviewed-on: https://review.typo3.org/22509
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
5 years ago[BUGFIX] Two mailforms on a page cause error on field validation 77/22477/2
Ernesto Baschny [Tue, 27 Nov 2012 01:38:49 +0000 (02:38 +0100)]
[BUGFIX] Two mailforms on a page cause error on field validation

Check formname for uniqueness and if need append a unique hash.

Change-Id: Ifd33c5130c383f3afc3e5e6d80c1be017f25ec5b
Releases: 6.2, 6.1, 6.0, 4.7, 4.5
Fixes: #16114
Reviewed-on: https://review.typo3.org/22477
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
5 years ago[BUGFIX] Width of action column is wrong 53/22453/2
Oliver Hader [Wed, 19 Jun 2013 14:53:35 +0000 (16:53 +0200)]
[BUGFIX] Width of action column is wrong

The width of the action column in the workspaces module is
wrong. 5 icons with a width of 16px each need to be displayed
and results in a required width of 80px.

Fixes: #50210
Releases: 6.0, 6.1, 6.2
Change-Id: I3e1cbfd9be53c57dad86a6003edbf38b559dab73
Reviewed-on: https://review.typo3.org/22453
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
5 years ago[BUGFIX] setRespectSysLanguage(FALSE) doesn't prevent language overlay 21/22421/2
Stanislas Rolland [Mon, 10 Jun 2013 15:25:00 +0000 (11:25 -0400)]
[BUGFIX] setRespectSysLanguage(FALSE) doesn't prevent language overlay

Solution: When this flag is set to FALSE, do not apply language
overlays.

Resolves: #47192
Releases: 6.0, 6.1, 6.2
Change-Id: Iac0d647fde2f930a63bc6982472b1c76e649601d
Reviewed-on: https://review.typo3.org/22421
Reviewed-by: Stanislas Rolland
Tested-by: Stanislas Rolland
5 years ago[BUGFIX] Faulty layout if row details are too long 87/22387/2
Oliver Hader [Wed, 17 Jul 2013 11:49:52 +0000 (13:49 +0200)]
[BUGFIX] Faulty layout if row details are too long

If the row details in in the workspaces difference view are too
long, e.g. a string that does not contain white-spaces like e.g.
a filename, the layout looks broken.
The solution is to introduce a new classname for the content
details and to force word breaking via CSS.

Fixes: #50114
Releases: 6.0, 6.1, 6.2
Change-Id: I9a768aed1c9bb4ebdfea5ba79083b7e5dce262d9
Reviewed-on: https://review.typo3.org/22387
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
5 years ago[TASK] Remove redundant try/catches in ResourceStorage of FAL 71/22371/3
Claus Due [Fri, 21 Jun 2013 19:13:50 +0000 (21:13 +0200)]
[TASK] Remove redundant try/catches in ResourceStorage of FAL

This change simply removes some try/catch statement which did
nothing but re-throw the RuntimeException.

Fixes: #49332
Releases: 6.0, 6.1, 6.2
Change-Id: I62d7e1b0312a16926b06e0bd5eb6fc8be0829d3c
Change-Id: I2c1e2f5947071ec5aadb897b4ca1044231552ed8
Reviewed-on: https://review.typo3.org/22371
Reviewed-by: Markus Klein
Tested-by: Markus Klein
5 years ago[BUGFIX] Category API does not consider multiple tables 66/22366/2
Marc Wöhlken [Fri, 5 Jul 2013 16:23:58 +0000 (18:23 +0200)]
[BUGFIX] Category API does not consider multiple tables

Using the Category API with multiple tables, leads to wrong
results due to a missing check for the correct table.

Resolves: #45332
Releases: 6.2, 6.1, 6.0
Change-Id: Iaf5b453acd434ab8d8f58932a2c96385a66abf37
(cherry picked from commit 7c97fe9f6ac712faba8dac4ecf0e2c127038b5d6)
Reviewed-on: https://review.typo3.org/22366
Reviewed-by: Stefan Neufeind
Tested-by: Stefan Neufeind
5 years ago[BUGFIX] Pagetree filtering crashes on draft moves 59/21959/2
Bart Dubelaar [Thu, 20 Sep 2012 20:06:14 +0000 (22:06 +0200)]
[BUGFIX] Pagetree filtering crashes on draft moves

This bugfix fixes the following pagetree issues encountered in
a draft workspace:

1. Records moved in draft could not be found
2. Pagetree would crash if a moved page occurs in the rootline of
a filtered page.
3. Filtering only looked at live page title not the draft title

Change-Id: I49c0082d013303d39c0c259a42f9bb74346ba7c8
Fixes: #39197
Releases: 6.2, 6.1, 6.0
Reviewed-on: https://review.typo3.org/21959
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters
5 years ago[BUGFIX] Skip link validation of RTE filelink 30/22330/2
Visay Keo [Thu, 20 Jun 2013 12:01:35 +0000 (19:01 +0700)]
[BUGFIX] Skip link validation of RTE filelink

When a text is linked to a file using RTE link wizard,
linkvalidator will detect it as internal link.
The validation check for internal page when it is a file will always fail.

This patch just skip validation if it finds a link to the file.

TODO:
* Proper validation of the file need to be applied.

Resolves: #49286
Releases: 6.0, 6.1, 6.2
Change-Id: I1b4901e2ac311ae93442fef044e4f7d7ae415753
Reviewed-on: https://review.typo3.org/22330
Reviewed-by: Wouter Wolters
Tested-by: Wouter Wolters