Packages/TYPO3.CMS.git
3 years ago[BUGFIX] Use correct tokenizer for JavaScript in T3Editor 41/45341/3
Daniel Goerz [Thu, 17 Dec 2015 20:05:46 +0000 (21:05 +0100)]
[BUGFIX] Use correct tokenizer for JavaScript in T3Editor

Change-Id: Iaf83e3f2d0a6a4b0cf795e4a6ef2a127cf0421df
Resolves: #72303
Releases: master
Reviewed-on: https://review.typo3.org/45341
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Michael Oehlhof <typo3@oehlhof.de>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
3 years ago[TASK] Move last documentation file from folder "master" 42/45342/2
Wouter Wolters [Thu, 17 Dec 2015 20:10:43 +0000 (21:10 +0100)]
[TASK] Move last documentation file from folder "master"

Resolves: #72304
Releases: master
Change-Id: I177c641ac18b4270b6ab6ce019054bffd37942fb
Reviewed-on: https://review.typo3.org/45342
Reviewed-by: Michael Oehlhof <typo3@oehlhof.de>
Tested-by: Michael Oehlhof <typo3@oehlhof.de>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
3 years ago[!!!][BUGFIX] Severe data-loss on workspaces publishing action 21/45321/6
Oliver Hader [Wed, 16 Dec 2015 19:12:42 +0000 (20:12 +0100)]
[!!!][BUGFIX] Severe data-loss on workspaces publishing action

If pages records in a given scenario are published this causes
a severe data-loss for the whole TYPO3 installation since all
records are deleted. Actually they are marked as deleted, but
that's not less problematic.

The scenario for this in a draft workspace is having reordered
sub-pages (move-placeholder) and a parent-page that is marked
for deletion. On publishing the parent-page, the delete process
iterates over all pages on the root-level due to some essential
missing checks and an implicit type-cast from null to interger
zero (0) on the pages.pid value.

The accordant places are validated now. In addition to that the
possibility to delete everything implicitly from the root-page
is disabled to prevent other programmatic flaws like this.

Resolves: #72273
Releases: master, 6.2
Change-Id: I175f220cc0939124e34713fff07685ba902ad385
Reviewed-on: https://review.typo3.org/45321
Reviewed-by: Alexander Opitz <opitz.alexander@googlemail.com>
Tested-by: Alexander Opitz <opitz.alexander@googlemail.com>
Reviewed-by: Oliver Hader <oliver.hader@typo3.org>
Tested-by: Oliver Hader <oliver.hader@typo3.org>
3 years ago[BUGFIX] DBAL: use correct default value for native connection 38/45338/3
Melanie Kalka [Thu, 17 Dec 2015 15:08:31 +0000 (16:08 +0100)]
[BUGFIX] DBAL: use correct default value for native connection

DBAL uses an empty string for the portnumber parameter
for mysqli real_connect, which results in a PHP warning,
because an integer (or null) is required.

Changing the fallback to null resolves this issue and
results in a working database connection.

Resolves: #72285
Releases: master, 6.2
Change-Id: Ie0e04f3ab04996ab634f99c3d2ab545d4a3b3819
Reviewed-on: https://review.typo3.org/45338
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Andreas Fernandez <typo3@scripting-base.de>
3 years ago[TASK] Extend workspace functional tests on placeholder deletion 29/45329/5
Oliver Hader [Thu, 17 Dec 2015 10:47:26 +0000 (11:47 +0100)]
[TASK] Extend workspace functional tests on placeholder deletion

Actions performed in a workspace:
* move existing page below sibling on same level (move-placeholder)
* create new page on parent page (new-placeholder)
* delete the parent page

Attention:
The test assertions reflect the status quo which is faulty and has
to be fixed on a separate bugfix.

Resolves: #72291
Releases: master, 6.2
Change-Id: I85e10569c36a4c669a479434a5ce973d3fb9fe5c
Reviewed-on: https://review.typo3.org/45329
Reviewed-by: Oliver Hader <oliver.hader@typo3.org>
Tested-by: Oliver Hader <oliver.hader@typo3.org>
3 years ago[BUGFIX] Use correct uid for table list in Page module 35/45335/2
Markus Klein [Thu, 17 Dec 2015 14:32:14 +0000 (15:32 +0100)]
[BUGFIX] Use correct uid for table list in Page module

Regression fix.

Resolves: #72298
Releases: master
Change-Id: I98d70bfaf423d29277607e335abcc286448113f2
Reviewed-on: https://review.typo3.org/45335
Reviewed-by: Gregor Sievert <gregor@gregorsievert.de>
Tested-by: Gregor Sievert <gregor@gregorsievert.de>
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Andreas Fernandez <typo3@scripting-base.de>
3 years ago[BUGFIX] substituteMarkerArrayCached() must accept special chars 19/45319/6
Markus Klein [Wed, 16 Dec 2015 18:04:07 +0000 (19:04 +0100)]
[BUGFIX] substituteMarkerArrayCached() must accept special chars

Add a bunch of unittests and streamline the code as well
by removing a useless preg_match_all() call.
Rename some variables and add comments.

Resolves: #72252
Releases: master, 6.2
Change-Id: I2a31a1c2ab6d83528428693213b922f0e1bc6fe5
Reviewed-on: https://review.typo3.org/45319
Reviewed-by: Tobias Klepp <tobias.klepp@stimme-der-hoffnung.de>
Tested-by: Tobias Klepp <tobias.klepp@stimme-der-hoffnung.de>
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Andreas Fernandez <typo3@scripting-base.de>
3 years ago[BUGFIX] Add buttons with `form` attribute to SplitButton save handling 27/45327/4
Andreas Fernandez [Thu, 17 Dec 2015 10:34:39 +0000 (11:34 +0100)]
[BUGFIX] Add buttons with `form` attribute to SplitButton save handling

Some save buttons do not follow the `_save` naming convention, but have
a `form` attribute instead. IE does not support such attribute which
breaks the save buttons. Now all buttons with a `form` attribute are
added to the save button handling. To prevent side effects, the buttons
must be in the module header now.

Resolves: #72288
Related: #71598
Releases: master
Change-Id: I180725ebd03a7c480ec22058d6ed32a1cc5c5f2b
Reviewed-on: https://review.typo3.org/45327
Reviewed-by: Roman Schürmann <roman.schuermann@wmdb.de>
Tested-by: Roman Schürmann <roman.schuermann@wmdb.de>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
3 years ago[TASK] Update bootstrap to 3.3.6 07/45307/4
Andreas Fernandez [Wed, 16 Dec 2015 08:22:05 +0000 (09:22 +0100)]
[TASK] Update bootstrap to 3.3.6

Resolves: #72259
Releases: master
Change-Id: Ideaaec0afa3d2553a0b4f6222c61057dd60f76c0
Reviewed-on: https://review.typo3.org/45307
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Cedric Ziel <cedric@cedric-ziel.com>
Tested-by: Cedric Ziel <cedric@cedric-ziel.com>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
3 years ago[BUGFIX] Disclose exceptions on CLI in production context 15/45315/2
Helmut Hummel [Wed, 16 Dec 2015 13:49:40 +0000 (14:49 +0100)]
[BUGFIX] Disclose exceptions on CLI in production context

It is pointless to hide the exception message on CLI
in the production context. On CLI there are privileged
users only anyway and hiding this information from them
leads to wasted hours of debugging.

Output the necessary information also in ProductionExceptionHandler

Resolves: #72265
Releases: master, 6.2
Change-Id: I778b057fc7e170af2a2fcdb1befb2a4400449ce7
Reviewed-on: https://review.typo3.org/45315
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Reviewed-by: Michael Oehlhof <typo3@oehlhof.de>
Reviewed-by: Stephan Großberndt <stephan@grossberndt.de>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Michael Oehlhof <typo3@oehlhof.de>
Reviewed-by: Cedric Ziel <cedric@cedric-ziel.com>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
3 years ago[BUGFIX] Missing check before foreach loop 28/45328/2
Oliver Hader [Thu, 17 Dec 2015 10:41:06 +0000 (11:41 +0100)]
[BUGFIX] Missing check before foreach loop

PHP Warning: Invalid argument supplied for foreach()
in typo3/sysext/core/Classes/DataHandling/DataHandler.php line 5285

Resolves: #72289
Releases: master, 6.2
Change-Id: If83dee7261ce3dd6d1fbf3bb81d8f2e53741c4c7
Reviewed-on: https://review.typo3.org/45328
Reviewed-by: Michael Oehlhof <typo3@oehlhof.de>
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Andreas Fernandez <typo3@scripting-base.de>
Reviewed-by: Alexander Opitz <opitz.alexander@googlemail.com>
Tested-by: Alexander Opitz <opitz.alexander@googlemail.com>
3 years ago[TASK] Provide labels for all log types 95/45295/4
Daniel Goerz [Tue, 15 Dec 2015 19:41:51 +0000 (20:41 +0100)]
[TASK] Provide labels for all log types

Change-Id: Ief05c75376ef8b3b2fc8fa8c3ca52e2efbf24198
Resolves: #72256
Releases: master, 6.2
Reviewed-on: https://review.typo3.org/45295
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
3 years ago[TASK] Add cache buster to favicon 18/45318/2
Anja Leichsenring [Wed, 16 Dec 2015 17:51:41 +0000 (18:51 +0100)]
[TASK] Add cache buster to favicon

Favicon resource link misses the cache buster to make browsers
aware of changed resources.

Resolves: #72274
Relates: #72165
Releases: master
Change-Id: Ie3a32f2c8d48f0026c27175216207438ca11c7da
Reviewed-on: https://review.typo3.org/45318
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
3 years ago[BUGFIX] Scheduler: Correctly toggle results on task type change 39/45239/6
Joerg Kummer [Mon, 14 Dec 2015 15:11:04 +0000 (16:11 +0100)]
[BUGFIX] Scheduler: Correctly toggle results on task type change

Adapt JS and HTML to solve toggle on load and change task type.

Resolves: #72213
Releases: master
Change-Id: Idb4c964c42cb8bd408ebd98a528a603f1a0bead1
Reviewed-on: https://review.typo3.org/45239
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
3 years ago[BUGFIX] Empty row in table content element shows &nbsp; 14/45314/2
Wouter Wolters [Wed, 16 Dec 2015 12:06:30 +0000 (13:06 +0100)]
[BUGFIX] Empty row in table content element shows &nbsp;

When an empty row in a table content element is shown in the
frontend, with htmlSpecialChars set to 1, &nbsp; is shown.
Use a real space instead.

Resolves: #72263
Releases: master,6.2
Change-Id: I60304607caa4fc90451216426f4ed73f01bf75fc
Reviewed-on: https://review.typo3.org/45314
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Andreas Fernandez <typo3@scripting-base.de>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
3 years ago[BUGFIX] Show backend login spinner when getting RSA pubkey 61/45261/4
Andreas Fernandez [Tue, 15 Dec 2015 08:46:32 +0000 (09:46 +0100)]
[BUGFIX] Show backend login spinner when getting RSA pubkey

The spinner displayed in the login button of the backend module is now
shown right after triggering the first AJAX call. Previously, it was
shown after the RSA public key was fetched.

Resolves: #71291
Releases: master
Change-Id: I24b74e3812ea69bc74f1372e902785352a90822b
Reviewed-on: https://review.typo3.org/45261
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Rudy Gnodde <rgn@windinternet.nl>
Tested-by: Rudy Gnodde <rgn@windinternet.nl>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
3 years ago[TASK] Raise minimum and recommended memory_limit 90/45290/2
Cedric Ziel [Tue, 15 Dec 2015 12:09:14 +0000 (13:09 +0100)]
[TASK] Raise minimum and recommended memory_limit

Releases: master
Resolves: #72237
Change-Id: I4ee1d57c4fad46802115cd530d8a2b312f5d5679
Reviewed-on: https://review.typo3.org/45290
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Michael Oehlhof <typo3@oehlhof.de>
Tested-by: Michael Oehlhof <typo3@oehlhof.de>
Reviewed-by: Xavier Perseguers <xavier@typo3.org>
Tested-by: Xavier Perseguers <xavier@typo3.org>
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Andreas Fernandez <typo3@scripting-base.de>
3 years ago[BUGFIX] Use correct severity in FAL flash messages 94/45294/2
Daniel Goerz [Tue, 15 Dec 2015 18:37:37 +0000 (19:37 +0100)]
[BUGFIX] Use correct severity in FAL flash messages

Additionally trailing slashes are no longer added to
folder identifiers.

Change-Id: Ieb14a6fd653a5f3ce130650f61d5235342c96b33
Resolves: #72246
Releases: master
Reviewed-on: https://review.typo3.org/45294
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Michael Oehlhof <typo3@oehlhof.de>
Tested-by: Michael Oehlhof <typo3@oehlhof.de>
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Andreas Fernandez <typo3@scripting-base.de>
Reviewed-by: Xavier Perseguers <xavier@typo3.org>
Tested-by: Xavier Perseguers <xavier@typo3.org>
3 years ago[TASK] Set TYPO3 version to 7.6.2-dev 88/45288/2
TYPO3 Release Team [Tue, 15 Dec 2015 11:07:07 +0000 (12:07 +0100)]
[TASK] Set TYPO3 version to 7.6.2-dev

Change-Id: I5c698ca457b28d0ea214cb44a48e78ce013968e1
Reviewed-on: https://review.typo3.org/45288
Reviewed-by: TYPO3 Release Team <typo3cms@typo3.org>
Tested-by: TYPO3 Release Team <typo3cms@typo3.org>
3 years ago[RELEASE] Release of TYPO3 7.6.1 87/45287/2 7.6.1 TYPO3_7-6-1
TYPO3 Release Team [Tue, 15 Dec 2015 11:05:24 +0000 (12:05 +0100)]
[RELEASE] Release of TYPO3 7.6.1

Change-Id: Ie4db2833e842ef8d98c61b87177002ee8336aa67
Reviewed-on: https://review.typo3.org/45287
Reviewed-by: TYPO3 Release Team <typo3cms@typo3.org>
Tested-by: TYPO3 Release Team <typo3cms@typo3.org>
3 years ago[SECURITY] Escape caption of media using css_styled_content 84/45284/2
Georg Ringer [Tue, 15 Dec 2015 10:38:47 +0000 (11:38 +0100)]
[SECURITY] Escape caption of media using css_styled_content

The caption must be escaped. As this is only a textarea, the parsefunc
is not needed.

Furthermore, the fields "altText" and "titleText" use htmlspecialchars instead of stripHtml.

Resolves: #41690
Releases: master, 6.2
Security-Commit: 2cfa39c452a35286deffdac4f0a03314e6f6ee69
Security-Bulletins: TYPO3-CORE-SA-2015-010, 011, 012, 013, 014, 015
Change-Id: I17a43cfa860c058054cd73f6ab751edfa91a02ce
Reviewed-on: https://review.typo3.org/45284
Reviewed-by: Oliver Hader <oliver.hader@typo3.org>
Tested-by: Oliver Hader <oliver.hader@typo3.org>
3 years ago[SECURITY] Fix XSS in rtehtmlarea 83/45283/2
Georg Ringer [Tue, 15 Dec 2015 10:38:28 +0000 (11:38 +0100)]
[SECURITY] Fix XSS in rtehtmlarea

The SpellCheckingController needs to quote external parameters.

Resolves: #37399
Releases: master, 6.2
Security-Commit: 0c5bb0ef87817710a7276f205b9efd62c9e68924
Security-Bulletins: TYPO3-CORE-SA-2015-010, 011, 012, 013, 014, 015
Change-Id: I4e341ae19cd16734ce228e6ed1250124ee9811f9
Reviewed-on: https://review.typo3.org/45283
Reviewed-by: Oliver Hader <oliver.hader@typo3.org>
Tested-by: Oliver Hader <oliver.hader@typo3.org>
3 years ago[SECURITY] Fix XSS in bullet list element 82/45282/2
Georg Ringer [Tue, 15 Dec 2015 10:38:11 +0000 (11:38 +0100)]
[SECURITY] Fix XSS in bullet list element

Replace parsefunc with htmlSpecialChars for the lines of bullet list

Resolves: #71683
Releases: master, 6.2
Security-Commit: d756287c89b3c3f1e53c7e4661bea8e8f2f1d2e6
Security-Bulletins: TYPO3-CORE-SA-2015-010, 011, 012, 013, 014, 015
Change-Id: Idabbe0222398e848a7365446a577f10ae804cd30
Reviewed-on: https://review.typo3.org/45282
Reviewed-by: Oliver Hader <oliver.hader@typo3.org>
Tested-by: Oliver Hader <oliver.hader@typo3.org>
3 years ago[SECURITY] Replace parseFunc with htmlspecialchars in element "table" 81/45281/2
Georg Ringer [Tue, 15 Dec 2015 10:37:54 +0000 (11:37 +0100)]
[SECURITY] Replace parseFunc with htmlspecialchars in element "table"

Instead of using the lib.parseFunc htmlspecialchars is used for
the table cell rendering.

Resolves: #25245
Releases: master, 6.2
Security-Commit: 7810e48bcd34b3769b981ad3a77b4057312c73fc
Security-Bulletins: TYPO3-CORE-SA-2015-010, 011, 012, 013, 014, 015
Change-Id: I2f4a6c83e5201685c2549f4fd3f0108c39309c2d
Reviewed-on: https://review.typo3.org/45281
Reviewed-by: Oliver Hader <oliver.hader@typo3.org>
Tested-by: Oliver Hader <oliver.hader@typo3.org>
3 years ago[SECURITY] XSS in Link Validator 80/45280/2
Wouter Wolters [Tue, 15 Dec 2015 10:37:34 +0000 (11:37 +0100)]
[SECURITY] XSS in Link Validator

Properly escape user input when showing broken links
in EXT:linkvalidator

Resolves: #68735
Releases: master,6.2
Security-Commit: e990111b86a9ac0f6f1a6c7741d28f8097b38d5c
Security-Bulletins: TYPO3-CORE-SA-2015-010, 011, 012, 013, 014, 015
Change-Id: Ibc28e7075c7663e796f6e06d4c2872e9fe345dbb
Reviewed-on: https://review.typo3.org/45280
Reviewed-by: Oliver Hader <oliver.hader@typo3.org>
Tested-by: Oliver Hader <oliver.hader@typo3.org>
3 years ago[SECURITY] Possible XSS in felogin messages 79/45279/2
Georg Ringer [Tue, 15 Dec 2015 10:37:16 +0000 (11:37 +0100)]
[SECURITY] Possible XSS in felogin messages

Change default TypoScript to encode messages in felogin
with htmlspecialchars.

Fix two occurences of _LOCAL_LANG messages where htmlspecialchars
was missing.

Resolves: #25243
Releases: master, 6.2
Security-Commit: 341a017859b2c3c99b675fb787b1c5a7af8cef6f
Security-Bulletins: TYPO3-CORE-SA-2015-010, 011, 012, 013, 014, 015
Change-Id: Icddb2be90bced5ef51439630a5b47bf6bc04f624
Reviewed-on: https://review.typo3.org/45279
Reviewed-by: Oliver Hader <oliver.hader@typo3.org>
Tested-by: Oliver Hader <oliver.hader@typo3.org>
3 years ago[SECURITY] XSS through WS comments 78/45278/2
Wouter Wolters [Tue, 15 Dec 2015 10:36:59 +0000 (11:36 +0100)]
[SECURITY] XSS through WS comments

Resolves: #25227
Releases: master, 6.2
Security-Commit: ee77513838267a6dabba8732fa4afe7970ce4ddc
Security-Bulletins: TYPO3-CORE-SA-2015-010, 011, 012, 013, 014, 015
Change-Id: If50fbffdfe8e6709e48bcf08da31094688089b01
Reviewed-on: https://review.typo3.org/45278
Reviewed-by: Oliver Hader <oliver.hader@typo3.org>
Tested-by: Oliver Hader <oliver.hader@typo3.org>
3 years ago[SECURITY] Link fields accept inline javascript code 77/45277/2
Oliver Hader [Tue, 15 Dec 2015 10:36:44 +0000 (11:36 +0100)]
[SECURITY] Link fields accept inline javascript code

JavaScript can be submitted for every link field and will be
rendered in the frontend passed through typolink. To circumvent
that, the URI scheme and prefix "javascript:" will be disallowed.

The extension "javascript_handler" allows however to bring back
that insecure behavior since some installations might rely on it.

Resolves: #71698
Releases: master, 6.2
Security-Commit: c854186c419f26a109afaf068149a58ef1745f32
Security-Bulletins: TYPO3-CORE-SA-2015-010, 011, 012, 013, 014, 015
Change-Id: I7dbed505624718010023cd8192ff7174a6a43fa6
Reviewed-on: https://review.typo3.org/45277
Reviewed-by: Oliver Hader <oliver.hader@typo3.org>
Tested-by: Oliver Hader <oliver.hader@typo3.org>
3 years ago[SECURITY] Fix several XSS issues 76/45276/2
Nicole Cordes [Tue, 15 Dec 2015 10:36:26 +0000 (11:36 +0100)]
[SECURITY] Fix several XSS issues

Resolves: #59150
Releases: master, 6.2
Security-Bulletins: TYPO3-CORE-SA-2015-010, 011, 012, 013, 014, 015
Change-Id: I93386ba95b02713368b8155b73a6da620c3795c2
Reviewed-on: https://review.typo3.org/45276
Reviewed-by: Oliver Hader <oliver.hader@typo3.org>
Tested-by: Oliver Hader <oliver.hader@typo3.org>
3 years ago[SECURITY] Prevent XSS in TER download dialog 75/45275/2
Nicole Cordes [Tue, 15 Dec 2015 10:36:13 +0000 (11:36 +0100)]
[SECURITY] Prevent XSS in TER download dialog

Due to the json request format during a TER extension installation,
the EM is susceptible to XSS.

Resolves: #71524
Releases: master, 6.2
Security-Commit: 45f7ec93279bff9cbbde1d76a5947b73adce81ab
Security-Bulletins: TYPO3-CORE-SA-2015-010, 011, 012, 013, 014, 015
Change-Id: I0e555c4e711c2d3f956b74ca53b0c0534c131724
Reviewed-on: https://review.typo3.org/45275
Reviewed-by: Oliver Hader <oliver.hader@typo3.org>
Tested-by: Oliver Hader <oliver.hader@typo3.org>
3 years ago[BUGFIX] Remove bad PHP code in LiveSearch 59/45259/2
Markus Klein [Tue, 15 Dec 2015 00:41:24 +0000 (01:41 +0100)]
[BUGFIX] Remove bad PHP code in LiveSearch

Remove a left-over code from the LiveSearch migration to autocomplete
which causes a PHP warning.
The removed code and its results is never used in JS.

Resolves: #72226
Releases: master
Change-Id: Ia27c5cd7e863c4d6c0ca89ed8ca3f78a47ed2bfb
Reviewed-on: https://review.typo3.org/45259
Reviewed-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Tested-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Michael Oehlhof <typo3@oehlhof.de>
Tested-by: Michael Oehlhof <typo3@oehlhof.de>
Reviewed-by: Oliver Hader <oliver.hader@typo3.org>
Tested-by: Oliver Hader <oliver.hader@typo3.org>
3 years ago[BUGFIX] Handle content with no markers in substituteMarkerArrayCached 55/45255/2
Markus Klein [Mon, 14 Dec 2015 22:09:54 +0000 (23:09 +0100)]
[BUGFIX] Handle content with no markers in substituteMarkerArrayCached

If the content has no markers at all, the full content
needs to be returned.
Additionally, the result does not need to be cached
if no markers are present.

Resolves: #72224
Releases: master, 6.2
Change-Id: I8fd39ab705eaa2217bba4fd5a069d940e677ea47
Reviewed-on: https://review.typo3.org/45255
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Tested-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Andreas Fernandez <typo3@scripting-base.de>
3 years ago[BUGFIX] Fix wrong PHPdoc for getIncludeScriptByCommandLineKey 52/45252/2
Wouter Wolters [Mon, 14 Dec 2015 17:55:29 +0000 (18:55 +0100)]
[BUGFIX] Fix wrong PHPdoc for getIncludeScriptByCommandLineKey

Correct wrong PHPdoc for getIncludeScriptByCommandLineKey.
It returns an array and not a string.

Resolves: #72212
Releases: master
Change-Id: If5907d44106b3405ccf462a0b9e902226a7f5199
Reviewed-on: https://review.typo3.org/45252
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
3 years ago[BUGFIX] ExtensionManager: Make viewhelpers compatible with PHP 7.0 17/45217/5
Morton Jonuschat [Fri, 11 Dec 2015 20:34:52 +0000 (21:34 +0100)]
[BUGFIX] ExtensionManager: Make viewhelpers compatible with PHP 7.0

This patch changes viewhelpers that inherit from the fluid
ActionViewHelper but have a different method signature for render() to
inherit from a base viewhelper that doesn't have a default render()
method, removing the incompatibilities.

Resolves: #72183
Releases: master
Change-Id: Ifb3479b39d972056c2cf8c431f6dc580f5b2630c
Reviewed-on: https://review.typo3.org/45217
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Reviewed-by: Alexander Opitz <opitz.alexander@googlemail.com>
Tested-by: Alexander Opitz <opitz.alexander@googlemail.com>
3 years ago[FOLLOWUP][BUGFIX] Fix initialization for checked attributes 46/45246/2
Markus Klein [Mon, 14 Dec 2015 16:28:30 +0000 (17:28 +0100)]
[FOLLOWUP][BUGFIX] Fix initialization for checked attributes

Only specify a *single* type information for parameters, otherwise
Fluid fails to extract the right type and does not validate the boolean
expressions anymore.

Resolves: #67225
Releases: master
Change-Id: Ibf9873a74e94fb731a3440da25024853663e9c7e
Reviewed-on: https://review.typo3.org/45246
Reviewed-by: Frank Nägler <frank.naegler@typo3.org>
Tested-by: Frank Nägler <frank.naegler@typo3.org>
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Andreas Fernandez <typo3@scripting-base.de>
3 years ago[BUGFIX] Set default value for list_type 08/45208/4
Marco Huber [Fri, 11 Dec 2015 11:04:59 +0000 (12:04 +0100)]
[BUGFIX] Set default value for list_type

Set the default value for list_type to an empty string in the
database to match the default value in the TCA.

Resolves: #72171
Related: #32209
Releases: master
Change-Id: I46ed9935571e3b3d8dbfb6bec232ffd8f63a35f0
Reviewed-on: https://review.typo3.org/45208
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Christian Kuhn <lolli@schwarzbu.ch>
Tested-by: Christian Kuhn <lolli@schwarzbu.ch>
3 years ago[BUGFIX] Return passed $text if EXT:context_help is not installed 36/45236/2
Andreas Fernandez [Mon, 14 Dec 2015 11:50:38 +0000 (12:50 +0100)]
[BUGFIX] Return passed $text if EXT:context_help is not installed

Return `$text` instead of an empty string if EXT:context_help
is not installed. This shows the field label again but hides the
help buttons.

Resolves: #72205
Related: #71659
Releases: master
Change-Id: I6dba5e40c05e2555ad73d71eb6d177990225a0b4
Reviewed-on: https://review.typo3.org/45236
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Mathias Brodala <mbrodala@pagemachine.de>
Reviewed-by: Frank Nägler <frank.naegler@typo3.org>
Tested-by: Frank Nägler <frank.naegler@typo3.org>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
3 years ago[TASK] Install: Added clearer to All Configuration search field 30/45230/3
Michael Oehlhof [Sun, 13 Dec 2015 23:18:28 +0000 (00:18 +0100)]
[TASK] Install: Added clearer to All Configuration search field

Resolves: #71381
Releases: master
Change-Id: I7aca3ffbcd85a3a2a564ade34d8da77d5b1d2bc4
Reviewed-on: https://review.typo3.org/45230
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Frank Nägler <frank.naegler@typo3.org>
Tested-by: Frank Nägler <frank.naegler@typo3.org>
3 years ago[BUGFIX] Correctly evaluate content edit permissions for users 31/45231/3
Markus Klein [Sun, 13 Dec 2015 23:34:28 +0000 (00:34 +0100)]
[BUGFIX] Correctly evaluate content edit permissions for users

Users are permitted to edit content of a page, even though
they are not permitted to edit the page itself.

Resolves: #72172
Releases: master
Change-Id: Ie2e60beec5a04632cf1cb7fef32929cab51408c2
Reviewed-on: https://review.typo3.org/45231
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Johannes Kasberger <johannes.kasberger@reelworx.at>
Tested-by: Johannes Kasberger <johannes.kasberger@reelworx.at>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
3 years ago[TASK] Add favicon for the Install Tool 24/45224/3
Georg Ringer [Sat, 12 Dec 2015 19:40:46 +0000 (20:40 +0100)]
[TASK] Add favicon for the Install Tool

Readd a favicon for the Install Tool.

Change-Id: I9b0974ff7c7d7cd2e30ecb0819bc90e28ae57219
Resolves: #72165
Releases: master
Reviewed-on: https://review.typo3.org/45224
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Tested-by: Morton Jonuschat <m.jonuschat@mojocode.de>
3 years ago[FEATURE] Mocking of concrete methods in abstract classes 30/45130/2
Stefan Rotsch [Fri, 4 Dec 2015 14:40:13 +0000 (15:40 +0100)]
[FEATURE] Mocking of concrete methods in abstract classes

Allows mocking of concrete methods when testing protected functions
in abstract classes.

Introduces a new parameter which will be passed to the corresponding
PHPUnit method when creating a mock object of an abstract class.

Change-Id: I9cbe510633300184666c6204a4d0c5be7d417ef4
Resolves: #72062
Releases: master
Reviewed-on: https://review.typo3.org/45130
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Helmut Hummel <helmut.hummel@typo3.org>
Tested-by: Helmut Hummel <helmut.hummel@typo3.org>
3 years ago[TASK] Add a space after colon 27/45227/6
Göran Bodenschatz [Sat, 12 Dec 2015 22:46:53 +0000 (23:46 +0100)]
[TASK] Add a space after colon

According traditional and modern English-language writing there has to
be a space after colon.

Resolves: #72191
Releases: master
Change-Id: I59e01ad167963886903d1096cfe15d26de01e6e2
Reviewed-on: https://review.typo3.org/45227
Reviewed-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Tested-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
3 years ago[BUGFIX] FormEngine: Determine IRRE expanded state in data provider 26/44826/14
Andreas Allacher [Fri, 20 Nov 2015 14:06:00 +0000 (15:06 +0100)]
[BUGFIX] FormEngine: Determine IRRE expanded state in data provider

The expanded/collapsed state of inline and combination child records
is determined within the TcaInlineExpandCollapseState provider instead
of the display container so that the concerns are appropriately
separated.

Change-Id: I8152a9f4f689b1c409a9f14d2e29515ab3d91860
Resolves: #71728
Releases: master
Reviewed-on: https://review.typo3.org/44826
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Markus Sommer <markussom@posteo.de>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
Reviewed-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Tested-by: Morton Jonuschat <m.jonuschat@mojocode.de>
3 years ago[FOLLOWUP][BUGFIX] Wrong overloading of AbstractController::redirect() 16/45216/4
Morton Jonuschat [Fri, 11 Dec 2015 19:47:36 +0000 (20:47 +0100)]
[FOLLOWUP][BUGFIX] Wrong overloading of AbstractController::redirect()

This patch adds the missing parameters for the redirect method so that
the method signature matches between BackendModuleController and
AbstractController. As the parameters are all optional this is a
non-breaking change.

Resolves: #71303
Resolves: #72168
Releases: master
Change-Id: I8c7d55d1a1b01e1e7cdc3859e6a99eded5008802
Reviewed-on: https://review.typo3.org/45216
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
3 years ago[BUGFIX] Indexed Search: Bring back html5 placeholder 20/45220/5
Tymoteusz Motylewski [Fri, 11 Dec 2015 23:58:25 +0000 (00:58 +0100)]
[BUGFIX] Indexed Search: Bring back html5 placeholder

The placeholder for the search box of the AbstractPlugin
based plugin was removed by accident in
https://review.typo3.org/6657

This change brings it back.

Resolves: #68381
Releases: master
Change-Id: If876c204b305d6cc67f27fed5a3ca29a894c4b99
Reviewed-on: https://review.typo3.org/45220
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
3 years ago[TASK] Support \DateTimeInterface in fluid Format\DateViewHelper 18/45218/2
Morton Jonuschat [Fri, 11 Dec 2015 21:38:02 +0000 (22:38 +0100)]
[TASK] Support \DateTimeInterface in fluid Format\DateViewHelper

This patch changes the fluid DateViewHelper to check for date objects
implementing \DateTimeInterface so that \DateTime as well as
\DateTimeImmutable can be supplied as parameter.

Resolves: #72149
Releases: master
Change-Id: I3c3e2648168b283f8ec1fc35845a449c63d0b6ff
Reviewed-on: https://review.typo3.org/45218
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
3 years ago[TASK] Add quotes to classname in exception 25/45225/2
Georg Ringer [Sat, 12 Dec 2015 20:09:11 +0000 (21:09 +0100)]
[TASK] Add quotes to classname in exception

Add quotes to the class name in the exception to improve debugging.

Change-Id: If68cf9e5d7d7277b5172ed9a56083c95ca0c69ae
Resolves: #66510
Releases: master
Reviewed-on: https://review.typo3.org/45225
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
3 years ago[BUGFIX] Fix sorting by extension name 84/45184/5
Patrick Schriner [Wed, 25 Nov 2015 19:14:22 +0000 (20:14 +0100)]
[BUGFIX] Fix sorting by extension name

Introduce a custom column sort for the "Extension" (name) column as that
column can contain HTML which breaks a direct string comparison.

The existing ExtensionManager.compare function is renamed to
versionCompare to circumvent misunderstandings.

Resolves: #72102
Releases: master
Change-Id: I46c120458bc7e9da15c1d7d810b7a003bbbd2937
Reviewed-on: https://review.typo3.org/45184
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
3 years ago[TASK] Fluid: Include jquery libs should be include using https 21/45221/3
Gianluigi Martino [Sat, 12 Dec 2015 13:21:49 +0000 (14:21 +0100)]
[TASK] Fluid: Include jquery libs should be include using https

Change-Id: Ice723e93cab0ef63d17b97df1a81b6e025c620de
Resolves: #72169
Releases: master
Reviewed-on: https://review.typo3.org/45221
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
3 years ago[BUGFIX] RTE LinkHandler handle required classes 72/45172/3
Andreas Allacher [Mon, 7 Dec 2015 17:22:46 +0000 (18:22 +0100)]
[BUGFIX] RTE LinkHandler handle required classes

RTE LinkHandler now also handles
RTE.classes.<class>.requires correctly.

Change-Id: I8f50018f522dd9f0b885af7192c003f7fda325b7
Resolves: #71985
Releases: master
Reviewed-on: https://review.typo3.org/45172
Reviewed-by: Frank Nägler <frank.naegler@typo3.org>
Tested-by: Frank Nägler <frank.naegler@typo3.org>
Reviewed-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Tested-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Tymoteusz Motylewski <t.motylewski@gmail.com>
Tested-by: Tymoteusz Motylewski <t.motylewski@gmail.com>
3 years ago[BUGFIX] Indexed Search: Display links to external files 42/45142/4
Tymoteusz Motylewski [Sat, 5 Dec 2015 13:29:06 +0000 (14:29 +0100)]
[BUGFIX] Indexed Search: Display links to external files

Indexed search is rendering links to external files now.
Indexed search will not check if the file exists before displaying
search results. As a side effect this change also improves performance.

This change is affecting only the AbstractPlugin based plugin.
Extbase version do not have this check.

Resolves: #44381
Releases: master
Change-Id: Iae4e5b2f2cc575853f25c674cbb4307bdf3efa17
Reviewed-on: https://review.typo3.org/45142
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
Reviewed-by: Tymoteusz Motylewski <t.motylewski@gmail.com>
Tested-by: Tymoteusz Motylewski <t.motylewski@gmail.com>
3 years ago[BUGFIX] Make ToggleExtensionInstallationStateViewHelper PHP 7 compliant 60/45160/4
Michael Oehlhof [Sun, 6 Dec 2015 23:12:38 +0000 (00:12 +0100)]
[BUGFIX] Make ToggleExtensionInstallationStateViewHelper PHP 7 compliant

Match the declaration of the render function with the function
declaration of the base class.

In addition cleanup some code.

Resolves: #71304
Releases: master
Change-Id: I26527229ccf47928e954c412b4046d55c228e9de
Reviewed-on: https://review.typo3.org/45160
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Frans Saris <franssaris@gmail.com>
Tested-by: Frans Saris <franssaris@gmail.com>
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Andreas Fernandez <typo3@scripting-base.de>
3 years ago[BUGFIX] ExportPageTreeView wrapTitle() signature is incompatible 11/45211/5
Markus Sommer [Fri, 11 Dec 2015 13:11:50 +0000 (14:11 +0100)]
[BUGFIX] ExportPageTreeView wrapTitle() signature is incompatible

ExportPageTreeView contains following method declaration:

 public function wrapTitle($title, $v);

however in the AbstractTreeView the signature is:

 public function wrapTitle($title, $row, $bank = 0);

This will create a PHP warning on PHP7.

This change updates the method signature of the ExportPageTreeView to be
in line with the AbstractTreeView.

Resolves: #72176
Releases: master
Change-Id: Icc6cb69ffd4860f882157239e25135e2a6744a65
Reviewed-on: https://review.typo3.org/45211
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Andreas Fernandez <typo3@scripting-base.de>
Reviewed-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Tested-by: Morton Jonuschat <m.jonuschat@mojocode.de>
3 years ago[BUGFIX] Localize FAL error messages 90/44490/10
Mathias Schreiber [Mon, 2 Nov 2015 21:49:17 +0000 (22:49 +0100)]
[BUGFIX] Localize FAL error messages

FAL error messages were taken directly from the sys_log entries and
then put into flash messages. Now we provide proper translations for
the flash messages.

Resolves: #71255
Releases: master
Change-Id: I56985957fee865d5e86f66a63b22d8e88a097007
Reviewed-on: https://review.typo3.org/44490
Reviewed-by: Michael Oehlhof <typo3@oehlhof.de>
Tested-by: Michael Oehlhof <typo3@oehlhof.de>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Tested-by: Morton Jonuschat <m.jonuschat@mojocode.de>
3 years ago[TASK] Replace GeneralUtility::inList() with comparison 39/37139/7
Jo Hasenau [Mon, 23 Feb 2015 20:27:06 +0000 (21:27 +0100)]
[TASK] Replace GeneralUtility::inList() with comparison

Using direct comparisons instead of GeneralUtility::inList() is faster
and uses less memory.

Releases: master
Resolves: #55626
Change-Id: I0acfe07fbd796ba017dd64d6987139bfb80db898
Reviewed-on: https://review.typo3.org/37139
Reviewed-by: Stephan Großberndt <stephan@grossberndt.de>
Tested-by: Stephan Großberndt <stephan@grossberndt.de>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
3 years ago[BUGFIX] FormEngine: Handle empty arrays in display conditions 14/45214/2
Morton Jonuschat [Fri, 11 Dec 2015 14:38:59 +0000 (15:38 +0100)]
[BUGFIX] FormEngine: Handle empty arrays in display conditions

Records passed to the EvaluateDisplayConditions provider can contain
array values which can be empty in the case of new records. Handle this
condition in the evaluator.

Resolves: #71824
Releases: master
Change-Id: Ie5aa3f74913059f2a1e5933a8314dee6cb0fa1fa
Reviewed-on: https://review.typo3.org/45214
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
3 years ago[CLEANUP] ImportExport classes 03/45203/4
Markus Klein [Wed, 9 Dec 2015 18:00:11 +0000 (19:00 +0100)]
[CLEANUP] ImportExport classes

Resolves: #72156
Releases: master
Change-Id: I3582dfd8304d07536bf68104141a5fcabb12bd89
Reviewed-on: https://review.typo3.org/45203
Reviewed-by: Markus Sommer <markussom@posteo.de>
Tested-by: Markus Sommer <markussom@posteo.de>
Reviewed-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Tested-by: Morton Jonuschat <m.jonuschat@mojocode.de>
3 years ago[TASK] Use IconFactory in System Information Panel 12/45212/2
Frank Nägler [Fri, 11 Dec 2015 13:17:30 +0000 (14:17 +0100)]
[TASK] Use IconFactory in System Information Panel

Resolves: #72178
Releases: master
Change-Id: Ia1de64625280ad0cd50e56c80f3ea6efc2c0e86a
Reviewed-on: https://review.typo3.org/45212
Reviewed-by: Michael Oehlhof <typo3@oehlhof.de>
Tested-by: Michael Oehlhof <typo3@oehlhof.de>
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Andreas Fernandez <typo3@scripting-base.de>
3 years ago[BUGFIX] Track pids of deleted/moved records to clear cache 66/39766/8
Jigal van Hemert [Wed, 27 May 2015 21:45:29 +0000 (23:45 +0200)]
[BUGFIX] Track pids of deleted/moved records to clear cache

Clearing caches is done after all operations. The pid can't be read from
the deleted or from the old location of the moved record so an array
remembers these pids for the clear cache operation.

Resolves: #61017
Releases: master, 6.2
Change-Id: I4552d2b9b2a6ea475563fd62a62f29a57fe9df70
Reviewed-on: https://review.typo3.org/39766
Reviewed-by: Philipp Gampe <philipp.gampe@typo3.org>
Tested-by: Philipp Gampe <philipp.gampe@typo3.org>
Reviewed-by: Jan Helke <typo3@helke.de>
Tested-by: Jan Helke <typo3@helke.de>
3 years ago[TASK] Make JavaScript Icon API a singleton 96/45196/4
Andreas Fernandez [Wed, 9 Dec 2015 15:41:02 +0000 (16:41 +0100)]
[TASK] Make JavaScript Icon API a singleton

The JavaScript Icon API is now a singleton. This ensures the code is only
executed once and especially the cache is shared between all requests.

Resolves: #72126
Releases: master
Change-Id: Iafd61b90d2414f655e6500bc564b90490afc58b0
Reviewed-on: https://review.typo3.org/45196
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Josef Glatz <josef.glatz@typo3.org>
Reviewed-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Tested-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Reviewed-by: Frans Saris <franssaris@gmail.com>
Tested-by: Frans Saris <franssaris@gmail.com>
3 years ago[BUGFIX] Show error message in page module for invalid backend layout 46/45146/4
Georg Ringer [Wed, 9 Dec 2015 15:32:21 +0000 (16:32 +0100)]
[BUGFIX] Show error message in page module for invalid backend layout

If there is a backend layout selected with no colPos defined inside the backend layout option, then the page
module fatals with an error.

A error message is now shown to display that at least one
colPos parameter must be set.

Resolves: #67276
Releases: master
Change-Id: I02120be981607d35efa31f49cc0ee4cc992172e4
Reviewed-on: https://review.typo3.org/45146
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Andreas Fernandez <typo3@scripting-base.de>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Josef Glatz <josef.glatz@typo3.org>
Reviewed-by: Frans Saris <franssaris@gmail.com>
Tested-by: Frans Saris <franssaris@gmail.com>
3 years ago[BUGFIX] Correctly handle whitespace in format.date VH 00/45200/2
Markus Klein [Thu, 10 Dec 2015 14:23:24 +0000 (15:23 +0100)]
[BUGFIX] Correctly handle whitespace in format.date VH

Child content may contain line breaks or other whitespace.
Trim the content before processing it.

Resolves: #72152
Releases: master
Change-Id: Ibf5d3fb2c5ca74bb045c4278f0852c47e2131eab
Reviewed-on: https://review.typo3.org/45200
Reviewed-by: Felix Rauch <rauch@skaiamail.de>
Tested-by: Felix Rauch <rauch@skaiamail.de>
Reviewed-by: Josef Glatz <josef.glatz@typo3.org>
Tested-by: Josef Glatz <josef.glatz@typo3.org>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Frans Saris <franssaris@gmail.com>
Tested-by: Frans Saris <franssaris@gmail.com>
3 years ago[TASK] Configuration: Make searchbox clearable and fix scroll position 02/45202/4
Josef Glatz [Thu, 10 Dec 2015 18:17:13 +0000 (19:17 +0100)]
[TASK] Configuration: Make searchbox clearable and fix scroll position

This patches ensures the page is scrolled to the right position
upon opening a subtree.
Additionally, the "clear search" button is added to the search field.

Releases: master
Resolves: #72155
Change-Id: I4227ab0d362b8be2bb531dd341fb30e5d50d37ab
Reviewed-on: https://review.typo3.org/45202
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
3 years ago[BUGFIX] Determine returnUrl when deleting pages in EditDocumentController 26/45126/9
Andreas Fernandez [Mon, 7 Dec 2015 20:08:31 +0000 (21:08 +0100)]
[BUGFIX] Determine returnUrl when deleting pages in EditDocumentController

Determine the returnUrl based on the current module and record. If the
delete action if for a record in the table "pages" the backend needs to
perform a partial reload to avoid trying to access the deleted page. In
all other cases the predetermined returnUrl is ok.

Resolves: #72058
Resolves: #72081
Related: #71425
Releases: master
Change-Id: Id019f2b4a6f60cb850322d6667094b832c62a8e0
Reviewed-on: https://review.typo3.org/45126
Reviewed-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Tested-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
3 years ago[BUGFIX] Move default avatar image from background css to img tag 91/45191/3
Michael Oehlhof [Tue, 8 Dec 2015 23:49:23 +0000 (00:49 +0100)]
[BUGFIX] Move default avatar image from background css to img tag

If a transparent avatar image is assigned the default avatar no longer
shines through.

Resolves: #70220
Releases: master
Change-Id: I889c706616876d45d0d9a371b96591ec27aa264c
Reviewed-on: https://review.typo3.org/45191
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Andreas Fernandez <typo3@scripting-base.de>
Reviewed-by: Frans Saris <franssaris@gmail.com>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
3 years ago[BUGFIX] Use correct title for links to create new records 95/45195/2
Georg Ringer [Wed, 9 Dec 2015 13:34:47 +0000 (14:34 +0100)]
[BUGFIX] Use correct title for links to create new records

If records can not be sorted, the title of the link should just state
'Create new record' instead of 'create new record after this record'.

Change-Id: I270ff1043b31366fbb8c90a259d52583a178f301
Resolves: #69934
Releases: master
Reviewed-on: https://review.typo3.org/45195
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Reviewed-by: Josef Glatz <josef.glatz@typo3.org>
Tested-by: Andreas Fernandez <typo3@scripting-base.de>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
3 years ago[BUGFIX] Hide CSH button if context_help is disabled 97/45197/4
Gianluigi Martino [Wed, 9 Dec 2015 15:38:00 +0000 (16:38 +0100)]
[BUGFIX] Hide CSH button if context_help is disabled

Do not show the help icon if the extension "context_help" is not installed.

Change-Id: Ia23c5c1a2d61a1e24e9de96def2fdb6a0a69b99f
Resolves: #71659
Releases: master
Reviewed-on: https://review.typo3.org/45197
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
Reviewed-by: Michael Oehlhof <typo3@oehlhof.de>
Tested-by: Michael Oehlhof <typo3@oehlhof.de>
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Andreas Fernandez <typo3@scripting-base.de>
3 years ago[TASK] Update T3Icons to 0.0.9 87/45187/2
Benjamin Kott [Tue, 8 Dec 2015 19:50:08 +0000 (20:50 +0100)]
[TASK] Update T3Icons to 0.0.9

Resolves: #27475
Resolves: #72056
Releases: master
Change-Id: I3439bfcea56367b44a81db2fa9dc77272cb246e1
Reviewed-on: https://review.typo3.org/45187
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Michael Oehlhof <typo3@oehlhof.de>
Tested-by: Michael Oehlhof <typo3@oehlhof.de>
Reviewed-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Tested-by: Morton Jonuschat <m.jonuschat@mojocode.de>
3 years ago[BUGFIX] LiveSearch: Don't trigger select on valid input 77/45177/3
Michael Oehlhof [Mon, 7 Dec 2015 22:45:36 +0000 (23:45 +0100)]
[BUGFIX] LiveSearch: Don't trigger select on valid input

Disable automatically selecting a valid search result. Otherwise, if
there is only one suggestion provided and the suggestion perfectly
matches the search string, the result is auto selected and no longer
visible/clickable.

Resolves: #71584
Releases: master
Change-Id: I239bf1dac9ce5b4851af4f48f3771d47ae1d8384
Reviewed-on: https://review.typo3.org/45177
Reviewed-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Tested-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
3 years ago[BUGFIX] Fix initialization for checked attributes 46/40646/9
Frederic Gaus [Thu, 25 Jun 2015 14:10:00 +0000 (16:10 +0200)]
[BUGFIX] Fix initialization for checked attributes

Fix the phpDoc for checked attributes in CheckboxViewHelper
and RadioViewHelper

Fix the inititialization of the multiple parameter of the
CheckboxViewHelper

Resolves: #67225
Releases: master
Change-Id: I53dc8793e95f0c45077559ea14f92f268e0a834a
Reviewed-on: https://review.typo3.org/40646
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Reviewed-by: Tomas Norre Mikkelsen <tomasnorre@gmail.com>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Jigal van Hemert <jigal.van.hemert@typo3.org>
Tested-by: Jigal van Hemert <jigal.van.hemert@typo3.org>
Reviewed-by: Frank Nägler <frank.naegler@typo3.org>
Tested-by: Frank Nägler <frank.naegler@typo3.org>
Reviewed-by: Jan Helke <typo3@helke.de>
Tested-by: Jan Helke <typo3@helke.de>
Reviewed-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Tested-by: Morton Jonuschat <m.jonuschat@mojocode.de>
3 years ago[BUGFIX] Remove slash in description 85/45185/2
Boris Schauer [Tue, 8 Dec 2015 14:53:32 +0000 (15:53 +0100)]
[BUGFIX] Remove slash in description

Resolves: #72108
Change-Id: I076bbec8614ae9da807befba27fba7be72a037c2
Reviewed-on: https://review.typo3.org/45185
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Andreas Fernandez <typo3@scripting-base.de>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
3 years ago[BUGFIX] Prefix all CSS for admin panel 73/45173/2
Wouter Wolters [Mon, 7 Dec 2015 17:49:46 +0000 (18:49 +0100)]
[BUGFIX] Prefix all CSS for admin panel

Resolves: #72091
Releases: master
Change-Id: I2666cbc431a6c3a9bda6640fd8f123fd58d86ba4
Reviewed-on: https://review.typo3.org/45173
Reviewed-by: Josef Glatz <josef.glatz@typo3.org>
Tested-by: Josef Glatz <josef.glatz@typo3.org>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
3 years ago[FOLLOWUP][BUGFIX] ImageViewHelper should catch exceptions 66/45166/3
Andreas Allacher [Mon, 7 Dec 2015 09:57:36 +0000 (10:57 +0100)]
[FOLLOWUP][BUGFIX] ImageViewHelper should catch exceptions

We also need to catch RuntimeException and InvalidArgumentException
InvalidArgumentException might be thrown if a storage does not exist.
RuntimeException if a file is outside a storage.

Change-Id: I994d8e7ef8b515f23ccc01847cefac0202f14b6f
Releases: master, 6.2
Resolves: #71686
Reviewed-on: https://review.typo3.org/45166
Reviewed-by: Jan Helke <typo3@helke.de>
Tested-by: Jan Helke <typo3@helke.de>
Reviewed-by: Josef Glatz <josef.glatz@typo3.org>
Reviewed-by: Frank Nägler <frank.naegler@typo3.org>
Tested-by: Frank Nägler <frank.naegler@typo3.org>
3 years ago[BUGFIX] Hide report notice in about module for editors 68/45168/2
Georg Ringer [Mon, 7 Dec 2015 12:25:19 +0000 (13:25 +0100)]
[BUGFIX] Hide report notice in about module for editors

Editors are not allowed to see the reports module, therefore the link
'Please check the status report for more information' does not work.
Furthermore those users can't change anything which is reported, so
don't show this information at all.

Displaying less warnings which can not be changed improve the usability.

Change-Id: I552b34f4bc1e249c51abc166699597b1678a61f9
Resolves: #72084
Releases: master
Reviewed-on: https://review.typo3.org/45168
Reviewed-by: Frank Nägler <frank.naegler@typo3.org>
Reviewed-by: Josef Glatz <josef.glatz@typo3.org>
Tested-by: Frank Nägler <frank.naegler@typo3.org>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
3 years ago[BUGFIX] Use strict authMode access check for list_type 65/45165/2
Georg Ringer [Mon, 7 Dec 2015 09:37:24 +0000 (10:37 +0100)]
[BUGFIX] Use strict authMode access check for list_type

Enforcing a strict authMode check is required to make sure
that editors only see those content elements of type CType='list'
which they are allowed to see.

Change-Id: I47f92a03a3dacbbf686fd9592ea679a40f3a828b
Resolves: #32209
Releases: master, 6.2
Reviewed-on: https://review.typo3.org/45165
Reviewed-by: Josef Glatz <josef.glatz@typo3.org>
Tested-by: Josef Glatz <josef.glatz@typo3.org>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Ralf Merz <info@merzilla.de>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Frank Nägler <frank.naegler@typo3.org>
Tested-by: Frank Nägler <frank.naegler@typo3.org>
3 years ago[BUGFIX] Display prev & next icons in workspace module again 59/45159/3
Georg Ringer [Sun, 6 Dec 2015 20:10:19 +0000 (21:10 +0100)]
[BUGFIX] Display prev & next icons in workspace module again

As the workspace module is not working with the new icon API yet,
show the icons by adding them in the CSS file directly.

Change-Id: Ib49d8d8ed33034ed0cfdb59a3f2b022fb0ef8fee
Resolves: #71797
Releases: master
Reviewed-on: https://review.typo3.org/45159
Tested-by: Michael Oehlhof <typo3@oehlhof.de>
Reviewed-by: Ralf Merz <info@merzilla.de>
Tested-by: Ralf Merz <info@merzilla.de>
Reviewed-by: Josef Glatz <josef.glatz@typo3.org>
Tested-by: Josef Glatz <josef.glatz@typo3.org>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Frank Nägler <frank.naegler@typo3.org>
Tested-by: Frank Nägler <frank.naegler@typo3.org>
3 years ago[BUGFIX] Removed delete icon in toolbar for sys_file_metadata 62/45162/6
Michael Oehlhof [Mon, 7 Dec 2015 00:28:38 +0000 (01:28 +0100)]
[BUGFIX] Removed delete icon in toolbar for sys_file_metadata

Resolves: #71678
Releases: master
Change-Id: I28e13da627a5b9312f36e1811ede5fbe3bd415bb
Reviewed-on: https://review.typo3.org/45162
Reviewed-by: Steffen Müller <typo3@t3node.com>
Tested-by: Steffen Müller <typo3@t3node.com>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
3 years ago[BUGFIX] Re-validate after remove IRRE element 99/45099/3
Frank Nägler [Wed, 2 Dec 2015 20:41:20 +0000 (21:41 +0100)]
[BUGFIX] Re-validate after remove IRRE element

Resolves: #72014
Releases: master
Change-Id: I035336ec841eef1bc2c8c7875a71f7df316f134e
Reviewed-on: https://review.typo3.org/45099
Reviewed-by: Armin Ruediger Vieweg <armin@v.ieweg.de>
Tested-by: Armin Ruediger Vieweg <armin@v.ieweg.de>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
3 years ago[BUGFIX] Show correct icon for textmedia in page module 61/45161/2
Michael Oehlhof [Mon, 7 Dec 2015 00:06:08 +0000 (01:06 +0100)]
[BUGFIX] Show correct icon for textmedia in page module

Resolves: #70144
Releases: master
Change-Id: I9a5d6360b5f58fe46e65c6be508a3d7c6859015d
Reviewed-on: https://review.typo3.org/45161
Reviewed-by: Josef Glatz <josef.glatz@typo3.org>
Tested-by: Josef Glatz <josef.glatz@typo3.org>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
3 years ago[TASK] Allow to define multiple inlineLocalizeSynchronize commands 54/44354/3
Oliver Hader [Thu, 29 Oct 2015 12:59:53 +0000 (13:59 +0100)]
[TASK] Allow to define multiple inlineLocalizeSynchronize commands

The inlineLocalizeSynchronize command in DataHandler currently
supports these formats:
* [parent][13][inlineLocalizeSynchronize] = field,14
* [parent][13][inlineLocalizeSynchronize] = field,localize
* [parent][13][inlineLocalizeSynchronize] = field,synchronize

Since the sub-commands "localize" and "synchronize" address the
whole parent scope, the particular definition of a child-id (14
in this case) is only considered for one child-element. Localizing
different children would then result in multiple independent
DataHandler processings.

The scope of this change is to extend the API to define multiple
child-ids and define the proper language, which is currently
determined from the parent-element magically.

The current string configuration format is changed to be an array,
legacy configurations are converted to the new format:

[parent][13][inlineLocalizeSynchronize] = [
  field: name of the parent field,
  language: id of the target language,
  action: either "localize" or "synchronize",
  ids: array of child-ids to be localized [1, 2, 3]
]
Either "action" or "ids" must be defined.

Resolves: #71126
Releases: master, 6.2
Change-Id: Ic7e1293a44047bfd69017e240dd8563a1dffa423
Reviewed-on: https://review.typo3.org/44354
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
3 years ago[BUGFIX] DebuggerUtility should return css when requested. 47/45147/2
Benni Mack [Sat, 5 Dec 2015 15:12:16 +0000 (16:12 +0100)]
[BUGFIX] DebuggerUtility should return css when requested.

The DebuggerUtility of Extbase should only echo the CSS when
the debug info is echoed as well, otherwise it should be
returned just like the option describes it.

Resolves: #60893
Releases: master
Change-Id: I7196df201cb5974308aaf103aac5d63e63c02e61
Reviewed-on: https://review.typo3.org/45147
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Frank Nägler <frank.naegler@typo3.org>
Tested-by: Frank Nägler <frank.naegler@typo3.org>
Reviewed-by: Jigal van Hemert <jigal.van.hemert@typo3.org>
Tested-by: Jigal van Hemert <jigal.van.hemert@typo3.org>
3 years ago[BUGFIX] RTE: Use correct settings 36/44736/8
Andreas Allacher [Mon, 16 Nov 2015 16:10:12 +0000 (17:10 +0100)]
[BUGFIX] RTE: Use correct settings

RTE linkhandler properties and classes are now handled correctly.

Change-Id: I943c561e357848b30ae74a3d86062189273b91e5
Releases: master
Resolves: #71248
Reviewed-on: https://review.typo3.org/44736
Tested-by: Renzo Bauen <info@conpassione.ch>
Reviewed-by: Frank Nägler <frank.naegler@typo3.org>
Tested-by: Frank Nägler <frank.naegler@typo3.org>
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
3 years ago[BUGFIX] Render flag icons correctly in backend 56/45156/2
Michael Oehlhof [Sun, 6 Dec 2015 12:22:04 +0000 (13:22 +0100)]
[BUGFIX] Render flag icons correctly in backend

Resolves: #71505
Releases: master
Change-Id: Icd915e9919b46d6e8dca5feb0741a4db6b4c7a5e
Reviewed-on: https://review.typo3.org/45156
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Philipp Gampe <philipp.gampe@typo3.org>
Tested-by: Philipp Gampe <philipp.gampe@typo3.org>
3 years ago[BUGFIX] Re-implement GFX[thumbnails] for Backend 38/45138/2
Benni Mack [Sat, 5 Dec 2015 12:38:59 +0000 (13:38 +0100)]
[BUGFIX] Re-implement GFX[thumbnails] for Backend

The option to disable thumbnails globally for every
backend user in various places does not work anymore,
but is still quite useful. The patch re-introduces the option
to not show the thumbnail in various places.

Resolves: #60497
Releases: master
Change-Id: I28e8bf86563032df8008c7de9aa7861bb5c0e2c0
Reviewed-on: https://review.typo3.org/45138
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Reviewed-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Tested-by: Morton Jonuschat <m.jonuschat@mojocode.de>
3 years ago[BUGFIX] ColorPicker: Use absolute web path for exampleImg 40/45140/2
Benni Mack [Sat, 5 Dec 2015 12:58:19 +0000 (13:58 +0100)]
[BUGFIX] ColorPicker: Use absolute web path for exampleImg

The Example Image of the color picker is currently hardcoded (also with ..)
to the typo3/ directory.
This hinders integrators to allow any other file outside of typo3/ to be shown.

The patch corrects the path resolving part of the Color Picker.

Resolves: #70428
Releases: master
Change-Id: I2fb5a6b7379ee1cb5fd6e76d8081f2f72f91a098
Reviewed-on: https://review.typo3.org/45140
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Andreas Fernandez <typo3@scripting-base.de>
Reviewed-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Tested-by: Morton Jonuschat <m.jonuschat@mojocode.de>
3 years ago[BUGFIX] Let FormEngine Slider wizard save data again 45/45145/2
Benni Mack [Sat, 5 Dec 2015 14:45:59 +0000 (15:45 +0100)]
[BUGFIX] Let FormEngine Slider wizard save data again

Due to a change in FormEngine slider rewrite, the parameters
do not get handed over to the TBE_EDITOR update signal,
thus any changes on the slider do not reflect the actual field
that is sent to the server again.

Using apply() correctly as well as a document.ready()
statement makes this functionality work again.

Resolves: #71977
Releases: master
Change-Id: Ib033015d0c3869da487e9163ceca303bd1b436c8
Reviewed-on: https://review.typo3.org/45145
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Andreas Fernandez <typo3@scripting-base.de>
3 years ago[BUGFIX] Use "id" attribute for HMENU images in HTML5 41/45141/2
Benni Mack [Sat, 5 Dec 2015 13:06:54 +0000 (14:06 +0100)]
[BUGFIX] Use "id" attribute for HMENU images in HTML5

When setting no config.doktype=html5, then the TYPO3
Frontend renders everything as HTML5, except for
the "name" attribute for before/after logic of HMENUs,
which should use "id" attributes.

The "default" option needs to be taken into account
in order to use "id" instead of "name".

Resolves: #62487
Releases: master
Change-Id: If4cccffd499de4c81b862fa2a2e0fb194ac3bcec
Reviewed-on: https://review.typo3.org/45141
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Reviewed-by: Georg Ringer <georg.ringer@gmail.com>
Tested-by: Georg Ringer <georg.ringer@gmail.com>
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Jigal van Hemert <jigal.van.hemert@typo3.org>
Tested-by: Jigal van Hemert <jigal.van.hemert@typo3.org>
3 years ago[TASK] Use root paths for fluid templates in indexed_search 18/45118/3
Daniel Goerz [Thu, 3 Dec 2015 22:23:52 +0000 (23:23 +0100)]
[TASK] Use root paths for fluid templates in indexed_search

The extbase/fluid plugin of indexed search still uses the deprecated
rootPath for templates, partials and layouts. This patch adjusts the
typoscript to use rootPaths.

Change-Id: I7fc164d3ef14aaa1674f3f7940e2cd921a0a9216
Resolves: #72051
Releases: master
Reviewed-on: https://review.typo3.org/45118
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Markus Sommer <markussom@posteo.de>
Tested-by: Markus Sommer <markussom@posteo.de>
Reviewed-by: Benni Mack <benni@typo3.org>
Reviewed-by: Tymoteusz Motylewski <t.motylewski@gmail.com>
Tested-by: Tymoteusz Motylewski <t.motylewski@gmail.com>
3 years ago[TASK] Remove unused properties in $LANG 37/45137/2
Benni Mack [Sat, 5 Dec 2015 12:06:32 +0000 (13:06 +0100)]
[TASK] Remove unused properties in $LANG

The properties $charsetArray and $typo3_help_url
inside LanguageService are not needed and are outdated,
thus they can be removed.

Resolves: #72069
Releases: master
Change-Id: Icc6bc48297b361b2797614214176e463d25f683f
Reviewed-on: https://review.typo3.org/45137
Reviewed-by: Daniel Goerz <ervaude@gmail.com>
Tested-by: Daniel Goerz <ervaude@gmail.com>
Reviewed-by: Michael Oehlhof <typo3@oehlhof.de>
Tested-by: Michael Oehlhof <typo3@oehlhof.de>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
3 years ago[TASK] Make t3skin frame dimension overrides default 36/45136/2
Benni Mack [Sat, 5 Dec 2015 12:04:27 +0000 (13:04 +0100)]
[TASK] Make t3skin frame dimension overrides default

The TBE_STYLES settings in t3skin are not needed since
the default should be actually the whole backend setting.

The patch changes the settings accordingly.

Resolves: #72068
Releases: master
Change-Id: I7de0c3d8da0c3bdc9f58b56960a073193832ce0e
Reviewed-on: https://review.typo3.org/45136
Reviewed-by: Mathias Schreiber <mathias.schreiber@wmdb.de>
Tested-by: Mathias Schreiber <mathias.schreiber@wmdb.de>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
3 years ago[BUGFIX] sys_note not displayed in footer -list 33/45133/2
Gianluigi Martino [Fri, 4 Dec 2015 23:00:33 +0000 (00:00 +0100)]
[BUGFIX] sys_note not displayed in footer -list

No notes displayed in the list view footer

Change-Id: I54461133797db0a7a309b2b31407a41e29d1ba5d
Resolves: #72066
Releases: master
Reviewed-on: https://review.typo3.org/45133
Reviewed-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Tested-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
3 years ago[BUGFIX] Avoid overly large regex in substituteMarkerArrayCached 97/45097/2
Markus Klein [Wed, 2 Dec 2015 20:15:11 +0000 (21:15 +0100)]
[BUGFIX] Avoid overly large regex in substituteMarkerArrayCached

Fetch the actually used markers from the content and only
generate the replace regex for those.
This avoids problems where 1000 markers may be passed in,
but only 10 are actually used.

Resolves: #44270
Releases: master, 6.2
Change-Id: I05f60960949e945249b045a8ae8e8430f7d8f7e6
Reviewed-on: https://review.typo3.org/45097
Reviewed-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Tested-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
3 years ago[BUGFIX] FormEngine: show inline children found by RelationHandler 75/45075/3
Nicole Cordes [Tue, 1 Dec 2015 15:21:36 +0000 (16:21 +0100)]
[BUGFIX] FormEngine: show inline children found by RelationHandler

Currently only inline children with a proper default parent are added to
the result array for alternative languages. This patch ensures that all
inline children found by the RelationHandler are added and shown in the
backend as well.

Resolves: #71712
Releases: master
Change-Id: I21febbe5947bb61c098a95cf498d7dfb874f1212
Reviewed-on: https://review.typo3.org/45075
Reviewed-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Tested-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
3 years ago[TASK] Streamline JavaScript calls to ModuleMenu reload 88/45088/2
Benni Mack [Wed, 2 Dec 2015 13:45:11 +0000 (14:45 +0100)]
[TASK] Streamline JavaScript calls to ModuleMenu reload

There are several versions (partly legacy) of how to
trigger a reload of the Module Menu. This is now
streamlined.

Additionally the NewContentElementController does
not reload the module menu anymore on loading.

Resolves: #72024
Releases: master
Change-Id: I58fc274f22d0312b0616e20e379225f77540fbb3
Reviewed-on: https://review.typo3.org/45088
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Reviewed-by: Frank Nägler <frank.naegler@typo3.org>
Tested-by: Frank Nägler <frank.naegler@typo3.org>
Reviewed-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Tested-by: Morton Jonuschat <m.jonuschat@mojocode.de>
3 years ago[BUGFIX] File Recycler collect deleted files again 76/45076/6
Michael Oehlhof [Tue, 1 Dec 2015 21:46:21 +0000 (22:46 +0100)]
[BUGFIX] File Recycler collect deleted files again

Check if a _recycler_ folder exists along the path from the storage root
to the file. If a matching folder is found move the file or folder to be
deleted to the closest _recycler_ folder instead of immediately removing
them from the local filesystem.

Resolves: #66247
Resolves: #45327
Releases: master
Change-Id: Iac000095b86e7c888461aeb0347b9548d8edcd90
Reviewed-on: https://review.typo3.org/45076
Reviewed-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Tested-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
3 years ago[BUGFIX] Avoid wrong SQL query in fe_login user group redirect 47/39447/6
Frederic Gaus [Mon, 11 May 2015 15:07:11 +0000 (17:07 +0200)]
[BUGFIX] Avoid wrong SQL query in fe_login user group redirect

Ensure that no invalid group data is collected and check
if any group is set at all before running the SQL query.

Change-Id: I5792ff606d60ba5e5a108ed18c2ec91e354f85ff
Resolves: #64966
Releases: master, 6.2
Reviewed-on: https://review.typo3.org/39447
Reviewed-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Tested-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Reviewed-by: Benni Mack <benni@typo3.org>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Markus Klein <markus.klein@typo3.org>
3 years ago[BUGFIX] Ignore placeholders when copying records 76/11076/24
Oliver Hader [Sat, 31 Oct 2015 19:46:46 +0000 (20:46 +0100)]
[BUGFIX] Ignore placeholders when copying records

Copying records would make placeholders for draft elements visible in the
copy. This is due to the fact that t3ver_state information is removed upon
copy and the records itself are not ignored either. To circument that, now
only relevant records are selected for each scope, being live workspace or
any draft workspace. This also means, that new placeholders are copied if
the process has been initiated on a real draft workspace, which does not
happen if the duplication process was triggered in the live workspace.

Resolves: #36946
Resolves: #42075
Releases: master, 6.2
Change-Id: I0952d1a46d1cc02e0ec97f7cbbc9abc700e00c07
Reviewed-on: https://review.typo3.org/11076
Reviewed-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Tested-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Reviewed-by: Oliver Hader <oliver.hader@typo3.org>
Tested-by: Oliver Hader <oliver.hader@typo3.org>
3 years ago[TASK] EXT:lang Add a litle space between the search field and the table 48/45048/2
Michael Oehlhof [Sun, 29 Nov 2015 11:46:29 +0000 (12:46 +0100)]
[TASK] EXT:lang Add a litle space between the search field and the table

Resolves: #71955
Releases: master
Change-Id: I900f5edddaeef523647c15f4a677256f723b281f
Reviewed-on: https://review.typo3.org/45048
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
Tested-by: Ralf Merz <info@merzilla.de>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
3 years ago[TASK] Bring back the toggle button in install tool 10/45110/7
Gianluigi Martino [Thu, 3 Dec 2015 14:47:00 +0000 (15:47 +0100)]
[TASK] Bring back the toggle button in install tool

This patch brings back the toggle button for all configuration in install tool

Resolves: #71387
Releases: master
Change-Id: I827f3389c564857e76f215f6e9c6d22cddb2728f
Reviewed-on: https://review.typo3.org/45110
Reviewed-by: Frank Nägler <frank.naegler@typo3.org>
Tested-by: Frank Nägler <frank.naegler@typo3.org>
Reviewed-by: Andreas Fernandez <typo3@scripting-base.de>
Tested-by: Andreas Fernandez <typo3@scripting-base.de>
3 years ago[BUGFIX] Remove NOT NULL from optional text fields in system extensions 39/45039/6
Tomas Norre Mikkelsen [Sun, 29 Nov 2015 10:00:29 +0000 (11:00 +0100)]
[BUGFIX] Remove NOT NULL from optional text fields in system extensions

Removes NOT NULL requirement from TEXT database columns where the field is
not required to be filled in the backend. This aligns TCA definition and
database configuration, allowing to skip those fields when creating
records as MySQL doesn't allow default values for TEXT columns.

Resolves: #71952
Releases: master
Change-Id: Ic863899c6343a8e8c603a265c85dd906ad834568
Reviewed-on: https://review.typo3.org/45039
Reviewed-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Tested-by: Morton Jonuschat <m.jonuschat@mojocode.de>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>