Stefano Kowalke [Sun, 17 Apr 2011 13:05:39 +0000 (15:05 +0200)]
[TASK] Add prefix "class." to all unit test files which contain classes.
According to the CGL the files which contains a class must have a prefix "class." in filename.
Change-Id: I5ac0214236dc248a7d2b37d5981278fd65bc6c91
Resolves: #25868
Releases: 4.6
Reviewed-on: http://review.typo3.org/1646
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Tobias Liebig [Thu, 31 Mar 2011 19:48:21 +0000 (21:48 +0200)]
[BUGFIX] Recursion error when including TypoScript files
When editing TypoScript in the template module and using the "Include
TypoScript file content" option with more than 9 files, then the 10th
and all later files gets screwed up.
Thanks to André Spindler for reporting and providing a patch.
Thanks to Kilian Hann (futureconnection AG Zürich) for sponsoring
100 beers during the bug auction.
Change-Id: I2b1363506e5b01a9fb8fa47b792d3351d302eaa9
Resolves: #25374
Reviewed-on: http://review.typo3.org/1362
Reviewed-by: Michael Stucki
Reviewed-by: Susanne Moog
Tested-by: Susanne Moog
Reviewed-by: Sebastian Fischer
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Jigal van Hemert [Sun, 3 Apr 2011 07:41:49 +0000 (09:41 +0200)]
[TASK][!!!] Remove support for safe_mode
Change-Id: Iae45d949855ceec1e9dc9bf6e27d4885749375d5
Resolves: #25332
Reviewed-on: http://review.typo3.org/1447
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Reviewed-by: Susanne Moog
Tested-by: Susanne Moog
Xavier Perseguers [Wed, 13 Apr 2011 20:04:07 +0000 (22:04 +0200)]
[TASK] Remove global declarations inside methods
Global declaration such as global $TCA or global $BE_USER should
be changed to $GLOBALS['TCA'], $GLOBALS['BE_USER'], ...
Change-Id: I68f801ff6934a7e16db78cb3f10d1c2b64aef351
Fixes: #25334
Reviewed-on: http://review.typo3.org/1446
Reviewed-by: Steffen Gebert
Tested-by: Steffen Gebert
Reviewed-by: Susanne Moog
Tested-by: Susanne Moog
Tested-by: Christian Kuhn
Reviewed-by: Christian Kuhn
Andy Grunwald [Tue, 12 Apr 2011 20:06:39 +0000 (22:06 +0200)]
[FEATURE] Move class "tx_scheduler_Module" from "mod1/index.php" for unit test
This change makes it possible to write unit tests for "tx_scheduler_module".
Change-Id: I85b977687c24966aadba11248a58ee081b8bef67
Resolves: #25925
Releases: 4.6
Reviewed-on: http://review.typo3.org/1588
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Steffen Mueller [Sun, 27 Mar 2011 20:46:15 +0000 (22:46 +0200)]
[BUGFIX] Flexformfield "Forgot Password Message" in felogin does not work
The value of the FF field "Forgot Password Message" in felogin is not working. Solution is to rename FF field name from "forgot_message" to "forgot_reset_message".
Change-Id: I5a08196f8a26c73a27b65d3456e242ea86fc5707
Resolves: #23134
Releases: 4.6, 4.5, 4.4, 4.3
Reviewed-on: http://review.typo3.org/1285
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Steffen Gebert [Tue, 5 Apr 2011 15:42:19 +0000 (17:42 +0200)]
[BUGFIX] PHP_SCRIPT_INT unnecessarily requires parameter scriptSuffix
Using PHP_SCRIPT_INT throws an Exception "Expected parameter
$conf['scriptSuffix'] was not given.", in its render() method.
As the parameter is useless and not used in other places, this check
can be removed.
Change-Id: I64fd656e0af78e711290aea597353c7194dd613d
Resolves: #25145
Releases: 4.6, 4.5
Reviewed-on: http://review.typo3.org/1466
Reviewed-by: Björn Pedersen
Reviewed-by: Susanne Moog
Tested-by: Susanne Moog
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Steffen Gebert [Tue, 12 Apr 2011 18:13:33 +0000 (20:13 +0200)]
[FEATURE] Include Request URL in Exception log
Adding the requested URL in the log message for uncaught Exceptions can
ease the task of finding the particular page causing an error.
Change-Id: I39ffb7943c39a406c651dc30b2b2351347f0cad4
Resolves: #25924
Reviewed-on: http://review.typo3.org/1587
Reviewed-by: Stefano Cecere
Tested-by: Tim Duelken
Reviewed-by: Andy Grunwald
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Frederic Gaus, Flagbit GmbH & Co. KG [Tue, 12 Apr 2011 14:52:16 +0000 (16:52 +0200)]
[BUGFIX] remove console.log in pagetree actions.js
Simply removes the console.log debugging output. This line causes
JavaScript errors if there is no console object present.
Change-Id: I060b96f47b95d4265899ccff027009fe54f565cb
Resolves: #25721
Releases: 4.6, 4.5
Reviewed-on: http://review.typo3.org/1584
Reviewed-by: Oliver Hader
Tested-by: Oliver Hader
Oliver Hader [Wed, 9 Mar 2011 14:05:36 +0000 (15:05 +0100)]
[BUGFIX] Visibility of method tx_sysaction_task::fixUsername()
The method fixUsername() is defined as private, but should be
protected instead.
Change-Id: I0513416a0d1cf07793708e86e7f1c7fde619f4d5
Resolves: #25219
Releases: 4.6, 4.5, 4.4
Reviewed-on: http://review.typo3.org/1104
Reviewed-by: Jacob Rasmussen
Tested-by: Jacob Rasmussen
Reviewed-by: Oliver Klee
Reviewed-by: Andy Grunwald
Tested-by: Andy Grunwald
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Susanne Moog [Thu, 31 Mar 2011 19:57:06 +0000 (21:57 +0200)]
[TASK] Add version 4.5 CSS Styled Content Templates and Compat Version
For changes to CSS Styled Content in TYPO3 version 4.6 we need new
static templates and a compatibility version.
Change-Id: I4e36f465c057982ed3b66ceffe8fcc9b4ff6a73c
Resolves: #25606
Reviewed-on: http://review.typo3.org/1363
Reviewed-by: Ernesto Baschny
Tested-by: Ernesto Baschny
Reviewed-by: Christian Kuhn
Tested-by: Christian Kuhn
Sven Juergens [Sun, 10 Apr 2011 07:32:14 +0000 (09:32 +0200)]
[BUGFIX] Missing sprite icons in module db check > record statistics
The module DB check > Record statistics is using old icons for
pages. Additionally the icons for the different doktypes are
broken.
Change-Id: I64de9b55bf0a2fe32378aa9ce9c7bf3756da1224
Resolves: #24835
Releases: 4.6, 4.5, 4.4
Reviewed-on: http://review.typo3.org/1530
Reviewed-by: Jigal van Hemert
Tested-by: Jigal van Hemert
Reviewed-by: Oliver Hader
Tested-by: Oliver Hader
Marco Huber [Mon, 11 Apr 2011 13:15:19 +0000 (15:15 +0200)]
[BUGFIX] Disable the TCA Tree if $PA['fieldConf']['config']['readOnly'] is true.
Disable the TCA Tree if $PA['fieldConf']['config']['readOnly'] is true.
This happens for example in translations of records if
$TCA['my_table']['columns']['my_column']['l10n_display'] = 'defaultAsReadonly'.
Change-Id: I18195bbc3fc6369d56669a5fd5c81103140e7a6c
Resolves: #25888
Releases: 4.6, 4.5
Reviewed-on: http://review.typo3.org/1548
Reviewed-by: Marco Huber
Tested-by: Marco Huber
Reviewed-by: Oliver Hader
Tested-by: Oliver Hader
Markus Klein [Wed, 6 Apr 2011 09:44:24 +0000 (11:44 +0200)]
[BUGFIX] Configuration Tool creates invalid extTables.php file
If extTables.php is empty or not present, this changes takes care of
adding an extra LF after the opening <?php tag.
Change-Id: Ie9b94c75cd33c5a448c2e04759b38558ca1b8845
Resolves: #25781
Releases: 4.6, 4.5, 4.4
Reviewed-on: http://review.typo3.org/1474
Reviewed-by: Georg Ringer
Tested-by: Georg Ringer
Reviewed-by: Jan Loderhose
Tested-by: Jan Loderhose
Reviewed-by: Oliver Hader
Tested-by: Oliver Hader
Alexandre Gravel-Raymond [Fri, 4 Mar 2011 14:55:16 +0000 (15:55 +0100)]
[BUGFIX] Media obj respect moveJsFromHeaderToFooter
If you set config.moveJsFromHeaderToFooter=1 in your TS template, and
add a Media object, the swfobject library is correctly added at the
bottom of the page, but the inline code calling the swfobject library
is added inline at the same place as the media object.
This results in the javascript error "swfobject is not defined" and
the media not appearing on the page.
This change makes use of t3lib_PageRenderer::addJsInlineCode to add
the inline code, so it respects config.moveJsFromHeaderToFooter=1.
Change-Id: I0902ca60dab810dc58e7f1b46a07ad4ae6f1efda
Resolves: #M17868
Reviewed-on: http://review.typo3.org/1062
Reviewed-by: Steffen Kamper
Reviewed-by: Björn Pedersen
Reviewed-by: Jigal van Hemert
Tested-by: Jigal van Hemert
Reviewed-by: Oliver Hader
Tested-by: Oliver Hader
Steffen Gebert [Fri, 1 Apr 2011 10:13:20 +0000 (12:13 +0200)]
[FEATURE] Create an Exception hierarchy for HTTP Status Exceptions
This change creates an Exception hierarchy for the most frequently used
cases, when a request should be canceled with an Error 40x/50x.
Change-Id: If68f5f1f975b34c97b00a6a82c7eb47f38dc08cb
Resolves: #25697
Reviewed-on: http://review.typo3.org/1396
Reviewed-by: Helmut Hummel
Tested-by: Helmut Hummel
Reviewed-by: Susanne Moog
Tested-by: Susanne Moog
Susanne Moog [Sun, 10 Apr 2011 11:58:10 +0000 (13:58 +0200)]
[BUGFIX] List module - Fix pointer issue in pagebrowser
When you have more than 50 records in the list module, you can use
the page browser to display the next set of records.
When using this on the pages table, the pointer to page 2,3,.. is
added to the link of each page record. Selecting such a link causes
the result listing of that page to also start from that specific result
page. If there are not enough records, none will be shown.
Resolves: #20908
Change-Id: Ia112391533c40d4f2e56d41a6f893432480c8af1
Reviewed-on: http://review.typo3.org/1133
Reviewed-by: Susanne Moog
Tested-by: Susanne Moog
Philipp Gampe [Sat, 12 Mar 2011 16:26:37 +0000 (17:26 +0100)]
[Bugfix] getUrl creates malformed requests if no path is given
If you fetch an url like example.org without a '/' at the end,
you will get a 400 Bad request of only fetching headers (2).
If you get all content+headers (1) you will get a 404.
getUrl should be tolerant and add the slash if it is missing.
Change-Id: I86c7478d46d62f980735e5b80c6766e8bd811938
Resolves: #12225
Resolves: #13432
Resolves: #24483
Reviewed-on: http://review.typo3.org/1138
Reviewed-by: Susanne Moog
Tested-by: Susanne Moog
Steffen Ritter [Wed, 2 Mar 2011 12:46:27 +0000 (13:46 +0100)]
[BUGFIX] tcaTree Interface is not localized
The TCA tree introduced to 4.5 is not localized by now.
Change it
Change-Id: Ifb713ca7194c1ac0e37f939b4af267fad4acb107
Resolves: #M17818
Reviewed-on: http://review.typo3.org/971
Reviewed-by: Björn Pedersen
Reviewed-by: Steffen Kamper
Tested-by: Steffen Kamper
Reviewed-by: Jigal van Hemert
Tested-by: Jigal van Hemert
Helmut Hummel [Thu, 7 Apr 2011 21:52:43 +0000 (23:52 +0200)]
[CLEANUP] Replace spaces with tabs
By fixing #25347 spaces were used as indention instead of tabs.
Replace the spaces with tabs as a cleanup.
Change-Id: I7f0005e7a53339c59a393fc8254c9fdd9ba87f23
Reviewed-on: http://review.typo3.org/1508
Reviewed-by: Helmut Hummel
Tested-by: Helmut Hummel
Xavier Perseguers [Fri, 25 Mar 2011 11:55:13 +0000 (12:55 +0100)]
[TASK] Remove DEPRECATED static templates
There are still lot of DEPRECATED static templates for extensions. They are not
supported anymore by the corresponding extensions since years.
Change-Id: I0f5bf30f6e8bb7574ea4e8d281778dab0ee3ab64
Resolves: #M16123
Reviewed-on: http://review.typo3.org/1275
Reviewed-by: Fabien Udriot
Tested-by: Fabien Udriot
Reviewed-by: Susanne Moog
Reviewed-by: Markus Klein
Tested-by: Susanne Moog
Reviewed-by: Xavier Perseguers
Tested-by: Xavier Perseguers
Steffen Gebert [Sun, 3 Apr 2011 18:42:13 +0000 (20:42 +0200)]
[BUGFIX] Add missing parameter in "Page not found" handling
t3lib_message_ErrorpageMessage::__construct() requires two parameters,
however only one is given after change #24945, which modified the
handling of 404 errors.
This patch adds the second one.
Change-Id: I7c82c11e4d90c1127076370e72fa99cd2ccda03b
Resolves: #25727
Releases: 4.6, 4.5
Reviewed-on: http://review.typo3.org/1451
Reviewed-by: Steffen Kamper
Tested-by: Steffen Kamper
Reviewed-by: Xavier Perseguers
Tested-by: Xavier Perseguers
Steffen Kamper [Tue, 29 Mar 2011 10:52:27 +0000 (12:52 +0200)]
[BUGFIX] Disable of password field in setup should disable password2 too
When using this userTS for disable the password field:
setup.fields.password.disabled=1
Then the field password2 for repeat the password remains rendered.
Solution:
if password is disabled, disable password2 too.
Change-Id: I954343d287302a2b65f4d3a6abdf34e528377cbc
Resolves: #25423
Reviewed-on: http://review.typo3.org/1303
Reviewed-by: Björn Pedersen
Reviewed-by: BIDAU
Tested-by: BIDAU
Reviewed-by: Susanne Moog
Tested-by: Susanne Moog
Steffen Ritter [Wed, 2 Mar 2011 14:50:40 +0000 (15:50 +0100)]
[BUGFIX] tcaTree: make own record unselectable
Relations to an own record is not useful. You could exclude is via tca "andWhere"
This is not nice, because you wouldn't see the whole tree.
Just make the own record unselectable.
Change-Id: I90a40df315055c116bf4d2b688694120606b62b9
Resolves: #M16726
Reviewed-on: http://review.typo3.org/977
Reviewed-by: Philipp Gampe
Tested-by: Philipp Gampe
Reviewed-by: Kay Strobach
Reviewed-by: Tolleiv Nietsch
Reviewed-by: Jigal van Hemert
Tested-by: Jigal van Hemert
Stefano Kowalke [Sat, 5 Mar 2011 11:22:49 +0000 (12:22 +0100)]
[BUGFIX] Make backend layout dropdown in CE editform translateable
Change-Id: I755e70f1e82bc25515e879b9e166245a15d65b7e
Resolves: #M17873
Reviewed-on: http://review.typo3.org/1066
Reviewed-by: Steffen Kamper
Tested-by: Steffen Kamper
Reviewed-by: Christopher
Reviewed-by: Jigal van Hemert
Tested-by: Jigal van Hemert
sebastian.mueller [Tue, 29 Mar 2011 12:39:28 +0000 (14:39 +0200)]
[BUGFIX] Wrong display of csh icon and text in new record wizard
For each TCA-Table where you can add an element on a side, there is displayed
a csh icon with text (if text is configured). In db_new.php is a wrong
variable used so the csh text of the last tca-table is always used and for
each table. So it will also be displayed for tables which orginally got no csh text.
Additionally due to a missing break, the csh icon of a table is displayed
in the wrong line.
Change-Id: I39cee0f65d40626eb2c893fbaae4a9f25ae6c7d5
Resolves: #25371
Reviewed-on: http://review.typo3.org/1453
Reviewed-by: Francois Suter
Tested-by: Francois Suter
Xavier Perseguers [Sun, 3 Apr 2011 16:35:26 +0000 (18:35 +0200)]
[BUGFIX] BE list module fails to load after invalid inclusion removal
Class recordList is not included anymore due to a wrong require_once call deletion
Author: Xavier Perseguers <typo3@perseguers.ch>
Change-Id: I60722426dd36842ce4460a712b310ee8aef9b2e9
Fixes: #25408
Reviewed-on: http://review.typo3.org/1449
Reviewed-by: Helmut Hummel
Tested-by: Helmut Hummel
Steffen Müller [Fri, 1 Apr 2011 13:03:17 +0000 (15:03 +0200)]
[FEATURE][CONF] Option to hide BE search box in list mod
Change-Id: Ic93ccc8c736ad95952b97da82152f0621cff9195
Resolves: #25705
Reviewed-on: http://review.typo3.org/1405
Reviewed-by: Alexander Stehlik
Tested-by: Alexander Stehlik
Reviewed-by: Steffen Kamper
Tested-by: Steffen Kamper
Xavier Perseguers [Mon, 28 Mar 2011 16:09:09 +0000 (18:09 +0200)]
[BUGFIX] BE list module fails on some systems with PHP 5.3
When trying to load Web > List, the server answers with
Fatal error: Cannot redeclare class recordList
Change-Id: I4cc7e542568b5a528c97bc5b93afd909ca766f39
Resolves: #25408
Reviewed-on: http://review.typo3.org/1296
Reviewed-by: Xavier Perseguers
Tested-by: Xavier Perseguers
Xavier Perseguers [Fri, 25 Mar 2011 15:53:29 +0000 (16:53 +0100)]
[TASK] Remove explicit checks for PHP < 5.3
Make PHP 5.3 a requirement for TYPO3 4.6 and remove tests that check for older
versions of PHP.
Change-Id: Ib73e332a44f618e59f78c8e442d2bf27474c4964
Resolves: #25395
Reviewed-on: http://review.typo3.org/1277
Reviewed-by: Susanne Moog
Tested-by: Susanne Moog
Bjoern Pedersen [Fri, 1 Apr 2011 06:37:14 +0000 (08:37 +0200)]
[BUGFIX] Module -> List + "List only this table" search not working past page 1
Change-Id: I4edd9a020128246e4937dee8acb91ae7f393283d
Resolves: #25603
Reviewed-on: http://review.typo3.org/1390
Reviewed-by: Björn Pedersen
Tested-by: Björn Pedersen
Reviewed-by: Ernesto Baschny
Tested-by: Ernesto Baschny
Steffen Gebert [Thu, 31 Mar 2011 20:14:38 +0000 (22:14 +0200)]
[FEATURE] Add helpful error messages in case of shortcut misconfiguration
This change improves the output of wrongly misconfigured shortcut pages with
helpful error messages. Instead of "The requested page didn't have a proper
connection to the tree-root!", hints are given dependent on the configured
shortcut mode and target.
Thanks to Volker Graubaum (e-netconsulting) for sponsoring this fix with 80 beers
during the Bug Auction at T3BOARD11.
Change-Id: I09d88f907fc459b10e1598e117ea03733be8e0d0
Resolves: #24223
Reviewed-on: http://review.typo3.org/1366
Reviewed-by: Helmut Hummel
Tested-by: Helmut Hummel
Reviewed-by: Susanne Moog
Tested-by: Susanne Moog
Steffen Gebert [Sat, 26 Mar 2011 11:41:15 +0000 (12:41 +0100)]
[FEATURE] Link Exception messages to Wiki pages
Add a text and link to the Exception handler messages pointing to a page on typo3.org.
Change-Id: Ibbbd34657869cd9f0e5703124af4b4cc7673a679
Resolves: #M16271
Reviewed-on: http://review.typo3.org/1282
Reviewed-by: Helmut Hummel
Tested-by: Helmut Hummel
Reviewed-by: Steffen Kamper
Reviewed-by: Philipp Gampe
Reviewed-by: Susanne Moog
Tested-by: Susanne Moog
Rasto Birka [Thu, 31 Mar 2011 20:38:34 +0000 (22:38 +0200)]
[BUGFIX] unzip_path setting is used inconsistently
The setting $TYPO3_CONF_VARS']['BE']['unzip_path'] is used inconsistently in TYPO3.
In some cases, it expects the directory of the unzip binary. In other places, the
full path of the unzip binary is expected. This change adds a compatibilty layer
to cope with this.
Thanks to Tobias Liebig (networkteam GmbH) for sponsoring 60 beers during the
bug auction at T3BOARD11.
Change-Id: I67ef7b3544c2c2085f959229ff4343cbb736fde6
Resolves: #25470
Reviewed-on: http://review.typo3.org/1370
Reviewed-by: Michael Stucki
Tested-by: Michael Stucki
Sven Luzar [Mon, 28 Mar 2011 11:03:02 +0000 (13:03 +0200)]
[BUGFIX] Fix PHP Catchable Fatal Error in t3lib_iconWorks
The hook in t3lib_iconWorks::mapRecordTypeToSpriteIconName() passes an
array to call a user function, but the method t3lib_div::callUserFunction
expects a variable being passed by reference.
Assigning the array to a variable fixes this error.
Thanks to Haymo Meran (Gentics) for sponsoring this fix with 80 beers
at the bug auction at T3BOARD11.
Change-Id: I445ae05aa79913cb152da06cc66941db3fdf9227
Resolves: #25402
Reviewed-on: http://review.typo3.org/1290
Reviewed-by: Oliver Hader
Tested-by: Oliver Hader
Michael Stucki [Thu, 31 Mar 2011 21:33:39 +0000 (23:33 +0200)]
[BUGFIX] Use UTF-8 as default character set when creating new databases
When creating a new database in the Install Tool, UTF-8 should be
set as the default character set.
Thanks to Kian Gould (AOE Media) for sponsoring this fix with
250 beers during the bug auction at T3BOARD11.
Change-Id: I0584bfe1e6c50e4a7ea462d0196a45861d2c4d0e
Resolves: #22933
Reviewed-on: http://review.typo3.org/1375
Reviewed-by: Susanne Moog
Tested-by: Susanne Moog
Henrik Møller Rasmussen [Thu, 31 Mar 2011 20:14:05 +0000 (22:14 +0200)]
[BUGFIX] Added the storage field to the TCA for sysfolders.
The field disappeared in 4.5 and has now been added again.
Thanks to Rene Haflinger (Agentur Frontal) for sponsoring
this fix with 100 beers during the bug auction at T3BOARD11.
Change-Id: Id5419d1391e7952ce61d051100a8da7777741a17
Resolves: #25494
Reviewed-on: http://review.typo3.org/1368
Reviewed-by: Susanne Moog
Tested-by: Susanne Moog
Reviewed-by: Steffen Gebert
Tested-by: Steffen Gebert
Rasto Birka [Thu, 31 Mar 2011 19:41:13 +0000 (21:41 +0200)]
[TASK] Added nbproject to gitignore
Change-Id: I98f68764a4740fc017d73ebf9e32a3da0bbf58d6
Reviewed-on: http://review.typo3.org/1367
Reviewed-by: Michael Stucki
Tested-by: Michael Stucki
Sven Juergens [Wed, 9 Mar 2011 14:26:25 +0000 (15:26 +0100)]
[FEATURE] Add itemsProcFunc to wizard functionality
If you add a wizard to a BE input field, you can define items in configuration.
What is missing is the itemsProcFunc, to dynamically add items.
Change-Id: Ia9ab12e110b98af874748cd9e875a4d2d6cbcb2e
Resolves: #19898
Reviewed-on: http://review.typo3.org/1107
Reviewed-by: Susanne Moog
Tested-by: Susanne Moog
Ernesto Baschny [Thu, 31 Mar 2011 08:36:09 +0000 (10:36 +0200)]
[BUGFIX] Wrong 'target' generation in imagelinkwrap, making the generated page XHTML invalid
Change-Id: If3b34a0ca1be76d6f0fcef016858b2067e3efb1b
Resolves: #25311
Reviewed-on: http://review.typo3.org/1329
Reviewed-by: Philipp Kitzberger
Tested-by: Philipp Kitzberger
Reviewed-by: Ernesto Baschny
Tested-by: Ernesto Baschny
Steffen Gebert [Wed, 23 Mar 2011 22:03:21 +0000 (23:03 +0100)]
[BUGFIX] Send header Error 404 in case of page not found errors
If a page is not found, status 404 has to be sent.
Due to a recent change, an Exception is thrown instead of exiting.
As the Exception Handler overwrites the status 404 with a 503, this
code was sent to the client.
Change-Id: Ia6c7686c720863aa80e390c452527cabb338bde3
Resolves: #24945
Reviewed-on: http://review.typo3.org/1245
Reviewed-by: Susanne Moog
Tested-by: Susanne Moog
Steffen Gebert [Mon, 28 Mar 2011 14:25:18 +0000 (16:25 +0200)]
[BUGFIX] CSH tooltips in TCEforms lack cursor:pointer
Add a CSS class tipIsLinked to CSH tooltips, if there is more information
and thus the tooltip reacts on click to open detailed help.
Change-Id: I65c1312761b852f53826db5b477db3c645a9445f
Resolves: #M18051
Reviewed-on: http://review.typo3.org/1292
Reviewed-by: Steffen Kamper
Tested-by: Steffen Kamper
Reviewed-by: Susanne Moog
Tested-by: Susanne Moog
Stefano Kowalke [Tue, 29 Mar 2011 21:23:34 +0000 (23:23 +0200)]
[TASK] PHP closing tag don't exist in some files
Change-Id: I052ab0e98d036859cbf0222a4d23f37c1c2ec74c
Resolves: #25436
Reviewed-on: http://review.typo3.org/1317
Reviewed-by: Ernesto Baschny
Tested-by: Ernesto Baschny
Steffen Kamper [Tue, 29 Mar 2011 20:31:16 +0000 (22:31 +0200)]
[BUGFIX] Filtering extensions on remote repository does not respect repository selection
When searching for an extension in the remote repository with the "filter" text field, the currently selected repository is not taken into account and results from the main typo3 repository are returned.
Reason: the repository was still hardcoded to main repository
Change-Id: Ife22540867b58beab0bd7fd882b8a2ea9c02a624
Resolves: #25390
Reviewed-on: http://review.typo3.org/1315
Reviewed-by: Steffen Kamper
Tested-by: Steffen Kamper
Oliver Hader [Tue, 29 Mar 2011 16:30:33 +0000 (18:30 +0200)]
[BUGFIX] Pagetree uses wrong setting for limiting number of rendered page nodes
$GLOBALS['BE']['pageTree']['preloadLimit'] should be
$GLOBALS['TYPO3_CONF_VARS']['BE']['pageTree']['preloadLimit']
Change-Id: Iecefc92580a001267056a3748fb4423cb48767c7
Resolves: #25431
Reviewed-on: http://review.typo3.org/1310
Reviewed-by: Steffen Kamper
Tested-by: Steffen Kamper
Oliver Hader [Tue, 29 Mar 2011 16:03:31 +0000 (18:03 +0200)]
[TASK] Fix PHPdoc comment of dataProvider member variable
Change-Id: I02345d49af5e496796ec98a0cfe05b59ef0aedc0
Reviewed-on: http://review.typo3.org/1308
Reviewed-by: Oliver Hader
Tested-by: Oliver Hader
Tolleiv [Thu, 3 Mar 2011 10:13:15 +0000 (05:13 -0500)]
[TASK] Utilize hook_previewinfo to show the preview info
Change-Id: Ia926b145e9eb184d4244ec4f6552487e72392483
Related: #M17844
Fixes: #M17846
Reviewed-on: http://review.typo3.org/1005
Reviewed-by: Susanne Moog
Tested-by: Susanne Moog
Tolleiv [Thu, 3 Mar 2011 09:04:57 +0000 (04:04 -0500)]
[TASK] remove workspace previewInfo in frontend
The preview info should not contain workspace related code.
The proposed patch removes all output from tslib_fe::previewInfo
and adds the possibility to hook in it and render a specific
message. The followup implements this for the regular preview.
Change-Id: Ie3c05a1837bb2e11cdb66c72114218ae6e12c973
Related: #13571
Fixes: #M17844
Reviewed-on: http://review.typo3.org/1003
Reviewed-by: Susanne Moog
Tested-by: Susanne Moog
Steffen Gebert [Thu, 24 Mar 2011 21:22:10 +0000 (22:22 +0100)]
[BUGFIX] Fix Extension Download with compressionLevel enabled
To clear output buffers, the utility function t3lib_div::cleanOutputBuffers()
has to be used. Otherwise things break with compressionLevel enabled.
This change replaces the ob_end_clean() call in the Extension download process,
which did not work, as crap is sent to the user agent.
Change-Id: I078bab66c24d0667b3cd8c2397f26569ed2dbf8d
Resolves: #M18029
Also-Applies: TYPO3_4-5
Reviewed-on: http://review.typo3.org/1264
Reviewed-by: Susanne Moog
Tested-by: Susanne Moog
Steffen Kamper [Mon, 28 Mar 2011 08:53:07 +0000 (10:53 +0200)]
[BUGFIX] CSH not showing "more info" link icon anymore
With the missing link icon it's not clear that there is more info on click.
This cahnge brings back the icon to the tooltip.
Change-Id: I6a09c830772e3b128d31a400c5a8e59959f371a9
Resolves: #M18036
Reviewed-on: http://review.typo3.org/1288
Reviewed-by: Steffen Gebert
Tested-by: Steffen Gebert
Reviewed-by: Francois Suter
Tested-by: Francois Suter
Joh. Feustel [Mon, 28 Mar 2011 07:32:57 +0000 (09:32 +0200)]
[BUGFIX] BE ajax request fails on some systems with php 5.3
Because of a PHP bug on some systems the request to ajax.php fails because it
includes '../t3lib/class.t3lib_div.php' with relative path while init.php uses
PATH_t3lib . 'class.t3lib_div.php'.
On some systems PHP is not able to detect the already included file when a
different path is used.
Change-Id: I66e39690747c4f78b4571cf63777d98cb88b6b75
Resolves: #M17250
Reviewed-on: http://review.typo3.org/1286
Reviewed-by: Joh. Feustel
Tested-by: Joh. Feustel
Reviewed-by: Steffen Kamper
Reviewed-by: Xavier Perseguers
Tested-by: Xavier Perseguers
Alexander Stehlik [Sun, 13 Mar 2011 11:52:02 +0000 (12:52 +0100)]
[FEATURE] Improve usability of group fields
If only one value is allowed in a group field (maxitems==1), the
present value will be removed when the user selects a new value via
element browser or file upload field.
If the list is disabled (by disable_controls setting) the user doesn't
have the possibility to remove items. To prevent a "write once" field
the first item in the list will be removed if the maxitems limit has
been reached.
Change-Id: Iac48371f672b78e7c2ea3450af581759986015bc
Resolves: #M14430
Reviewed-on: http://review.typo3.org/1142
Reviewed-by: Steffen Gebert
Tested-by: Steffen Gebert
Reviewed-by: Steffen Kamper
Tested-by: Steffen Kamper
Steffen Gebert [Sat, 26 Mar 2011 09:37:45 +0000 (10:37 +0100)]
[BUGFIX] Wrong CSS definition for "submit"
Exchange the CSS style definition for the non-existant element "submit"
with "button", which is supposed to be meant.
Change-Id: Ib865addda8e3729970a4b72c1eb233e257ae9074
Resolves: #M18043
Reviewed-on: http://review.typo3.org/1279
Reviewed-by: Steffen Gebert
Tested-by: Steffen Gebert
Michael Buergi [Thu, 24 Mar 2011 10:29:44 +0000 (11:29 +0100)]
[BUGFIX] fix php fatal error of Upgrade wizard
The upgrade wizard of the install tool quits with a php fatal error if
php is configured to use registered globals.
When using registered globals, $GLOBALS['LANG'] may already be
initialized (with some string). The buggy line does not properly
initialize the language object if there is content in $GLOBALS['LANG'].
The solution changes this, so that $GLOBALS['LANG'] has to be an object
to prevent object creation. this line is also found in other places in
the typo3 source.
Change-Id: I1e553d233d6dac636d8a10a4ea5e60946e89d220
Resolves: #M17807
Reviewed-on: http://review.typo3.org/1254
Reviewed-by: Peter Niederlag
Reviewed-by: Philipp Gampe
Tested-by: Philipp Gampe
Reviewed-by: Xavier Perseguers
Reviewed-by: Jigal van Hemert
Tested-by: Jigal van Hemert
Rens Admiraal [Tue, 15 Mar 2011 17:15:45 +0000 (18:15 +0100)]
[BUGFIX] Fix path resolution in t3lib_compressor::compressCssFile()
This change makes compressCssFile() also use this helper, as it was forgotten in the
first place.
Change-Id: Id394f9a67e6222f7663e719ddce602d5ada2d10c
Resolves: #M18035
Reviewed-on: http://review.typo3.org/1176
Reviewed-by: Steffen Gebert
Tested-by: Steffen Gebert
Xavier Perseguers [Fri, 25 Mar 2011 08:59:06 +0000 (09:59 +0100)]
[TASK] Remove SVN auto properties $Id$
Remove SVN $Id$ auto-properties from TYPO3 source code
Change-Id: I26054ac32d77b212450b740178c8ce18be12e086
Resolves: #M18016
Reviewed-on: http://review.typo3.org/1272
Reviewed-by: Xavier Perseguers <typo3@perseguers.ch>
Tested-by: Xavier Perseguers <typo3@perseguers.ch>
Xavier Perseguers [Thu, 24 Mar 2011 08:57:11 +0000 (09:57 +0100)]
Fixed bug #18016
Change-Id: I9de4384637a2c35499d2f54e67b0924183b1bb9b
Reviewed-on: http://review.typo3.org/1266
Reviewed-by: Xavier Perseguers <typo3@perseguers.ch>
Tested-by: Xavier Perseguers <typo3@perseguers.ch>
Steffen Kamper [Thu, 17 Mar 2011 09:11:21 +0000 (10:11 +0100)]
[FEATURE] Add a time field to the Date Picker of datetime fields
This change adds a specialized Date Picker, which is used for datetime fields.
Previously, always 0:00 has been returned as time of the day.
Change-Id: I2be56f5ecb56edd9c5619ae3bbf6d8cb8a091768
Resolves: #M17596
Reviewed-on: http://review.typo3.org/1184
Reviewed-by: Steffen Kamper <info@sk-typo3.de>
Tested-by: Steffen Kamper <info@sk-typo3.de>
Steffen Kamper [Wed, 9 Mar 2011 23:37:56 +0000 (00:37 +0100)]
[BUGFIX] Remove superfluous code from Extension Manager
Remove code, which is not needed anymore and duplicates code.
Change-Id: I59491c8d194574e1ef80a461194fa054f32717ab
Resolves: #M17911
Reviewed-on: http://review.typo3.org/1112
Reviewed-by: Steffen Kamper <info@sk-typo3.de>
Tested-by: Steffen Kamper <info@sk-typo3.de>
Steffen Kamper [Sun, 13 Mar 2011 13:09:57 +0000 (14:09 +0100)]
[BUGFIX] Missing initialization of variables
Initialize variables, which are used, but not initialized in every case before.
Change-Id: Iccc6f2c31a4ade6485072f56afd7f1b227d1f516
Resolves: #M17934
Reviewed-on: http://review.typo3.org/1144
Reviewed-by: Steffen Gebert <steffen.gebert@typo3.org>
Tested-by: Steffen Gebert <steffen.gebert@typo3.org>
Reviewed-by: Sebastian Fischer <typo3@evoweb.de>
Reviewed-by: Steffen Kamper <info@sk-typo3.de>
Tested-by: Steffen Kamper <info@sk-typo3.de>
Steffen Kamper [Wed, 9 Mar 2011 17:30:10 +0000 (18:30 +0100)]
[BUGFIX] EM update module doesn't accept cache_extension table
You see only the message:
To get and update the list, go to the section "Import extensions" and use the button "Retrieve/Update".
even if table is filled.
The reason is that it looks for typo3temp/extensions.xml.hz instead checking if the table has records.
Solution:
Check table count instead.
Resolves: #M17907
Change-Id: I6c2905a0afce08f70c4743d66a1b521de30044fb
Reviewed-on: http://review.typo3.org/1108
Reviewed-by: Peter Beernink <p.beernink@drecomm.nl>
Tested-by: Peter Beernink <p.beernink@drecomm.nl>
Reviewed-by: Tolleiv Nietsch <info@tolleiv.de>
Reviewed-by: Steffen Kamper <info@sk-typo3.de>
Tested-by: Steffen Kamper <info@sk-typo3.de>
Steffen Kamper [Tue, 8 Mar 2011 11:44:07 +0000 (12:44 +0100)]
[BUGFIX] EM using filter in filter menu isn't clear enough
When checking one of the filter provided by the filter button menu, it gets printed bold.
However this seems not clear enough for many users, so display the "Clear filters" button would make it more clear.
Change-Id: I7345ab182fd5879ca18b2bf131c8c860f71def1f
Resolves: #M17892
Reviewed-on: http://review.typo3.org/1093
Reviewed-by: Peter Beernink <p.beernink@drecomm.nl>
Tested-by: Peter Beernink <p.beernink@drecomm.nl>
Reviewed-by: Markus Klein <klein.t3@mfc-linz.at>
Tested-by: Markus Klein <klein.t3@mfc-linz.at>
Reviewed-by: Steffen Kamper <info@sk-typo3.de>
Tested-by: Steffen Kamper <info@sk-typo3.de>
steffenk [Tue, 1 Mar 2011 14:07:05 +0000 (15:07 +0100)]
[BUGFIX] New EM, Date of last update not visible due to missing
In small views the text is not readable as it gets cut.
Solution:
Add the same text to a tooltip.
Resolves: #M17788
Change-Id: I8b963be0867e099383c15f367a9627b7eed0c5f0
Reviewed-on: http://review.typo3.org/960
Reviewed-by: Steffen Kamper <info@sk-typo3.de>
Tested-by: Steffen Kamper <info@sk-typo3.de>
Stefan Galinski [Fri, 11 Mar 2011 21:57:37 +0000 (22:57 +0100)]
[BUGFIX] Pagetree - Missing icons for expand/collapse whole branches
The expand/collapse context menu actions are missing icons.
Change-Id: I3ae86ff0a4de225854d173a485a0a8991adccd22
Resolves: #M17924
Reviewed-on: http://review.typo3.org/1130
Reviewed-by: Stefan Galinski <stefan.galinski@gmail.com>
Tested-by: Stefan Galinski <stefan.galinski@gmail.com>
Steffen Kamper [Thu, 10 Mar 2011 09:44:56 +0000 (10:44 +0100)]
[BUGFIX] TCEforms palette header is only indented on first tab
The first tab has different wrappers, so the CSS does not work on other tabs than the first one.
Change-Id: I980005f7dd981d7f3b27f7b1e1383129c369ae50
Resolves: #M17789
Reviewed-on: http://review.typo3.org/1117
Reviewed-by: Xavier Perseguers <typo3@perseguers.ch>
Tested-by: Xavier Perseguers <typo3@perseguers.ch>
Reviewed-by: Steffen Kamper <info@sk-typo3.de>
Tested-by: Steffen Kamper <info@sk-typo3.de>
Steffen Kamper [Mon, 7 Mar 2011 16:51:36 +0000 (17:51 +0100)]
[BUGFIX] EM locallist, grouping with author
When grouping with author you may see many groups with same author. This happens
if author of extensions is there with different email adresses.
Further author with email is clickable which is not useful in group header.
Solution:
use simple groupRenderer that returns author only.
Resolves: #M17887
Change-Id: Ia23082498fca61db975ff02eaf335578180315ea
Reviewed-on: http://review.typo3.org/1081
Reviewed-by: Björn Pedersen <bjoern.pedersen@frm2.tum.de>
Reviewed-by: Steffen Kamper <info@sk-typo3.de>
Tested-by: Steffen Kamper <info@sk-typo3.de>
Frederic Gaus, Flagbit GmbH & Co. KG [Mon, 21 Mar 2011 09:51:32 +0000 (10:51 +0100)]
[BUGFIX] link title with wrong encoded umlauts
Use Ext.urlEncode() to encode links in TBE_EDITOR. This ensures
correct generation of strings in conjunction with UTF-8 and fixes
wrongly encoded special chars (e.g. umlauts) in the browse-links
dialog's field link-title.
Change-Id: I003f7e64f8a10b0a49d6cbe0e2e82d67d614d21a
Resolves: #M17979
Reviewed-on: http://review.typo3.org/1227
Reviewed-by: Steffen Kamper <info@sk-typo3.de>
Tested-by: Steffen Kamper <info@sk-typo3.de>
Georg [Thu, 3 Mar 2011 14:04:31 +0000 (15:04 +0100)]
[TASK] Remove t3lib_sqlengine
t3lib_sqlengine is deprecated, remove it
Change-Id: I38563832cd0a549a84a0076cc855037cded58496
Reviewed-on: http://review.typo3.org/1013
Reviewed-by: Oliver Klee <typo3-coding@oliverklee.de>
Reviewed-by: Xavier Perseguers <typo3@perseguers.ch>
Reviewed-by: Björn Pedersen <bjoern.pedersen@frm2.tum.de>
Reviewed-by: Steffen Kamper <info@sk-typo3.de>
Tested-by: Steffen Kamper <info@sk-typo3.de>
Bjoern Pedersen [Mon, 21 Mar 2011 08:23:06 +0000 (09:23 +0100)]
[BUGFIX] Don't load ExtCore and ExtJS simultaneously
Reason: ExtJs fails if ExtCore is loaded.
ExtCore is a "small" compilation of ExtJS just for some little widgets and effects.
Resolves: #M17994
Change-Id: I5f676af89c981e0d6aea08ffa4c0db10324f0a0e
Reviewed-on: http://review.typo3.org/1212
Reviewed-by: Steffen Kamper <info@sk-typo3.de>
Tested-by: Steffen Kamper <info@sk-typo3.de>
Reviewed-by: Helmut Hummel <typo3@helmut-hummel.de>
Marco Huber [Thu, 17 Mar 2011 12:53:13 +0000 (13:53 +0100)]
[BUGFIX] TCA type="select", renderMode="tree", maxitems=1 doesn't work
Description
Problem:
I want to use the new TCA tree, but when I configure 1 as maxitems I
can't select a tree node.
This happens, because this.countSelectedNodes in
t3lib\js\extjs\tree\tree.js is 1 even when I create a new record.
And this.countSelectedNodes is 1 because $valueArray in
t3lib\tceforms\class.t3lib_tceforms_tree.php in the function
renderField always has at least one entry.
$valueArray is created by exploding $PA['itemFormElValue']. In fresh
records $PA['itemFormElValue'] is 0 and not NULL (or not set) and
has no other effects. But $valueArray has an entry and that's why
count($valueArray) is 1 and then this.countSelectedNodes is also 1.
To reproduce this issue you can use this TCA (Thanks to Steffen:
http://lists.typo3.org/pipermail/typo3-team-core/2010-November/044931.html):
$GLOBALS['TCA']['pages']['columns']['fe_group']['config']
['renderMode'] = 'tree';
$GLOBALS['TCA']['pages']['columns']['fe_group']['config']['maxitems'] = 1;
$GLOBALS['TCA']['pages']['columns']['fe_group']['config']['treeConfig']
= array(
'parentField' => 'subgroup',
'appearance' => array(
'expandAll' => TRUE,
'showHeader' => TRUE,
)
);
Solution:
To fix this issue $PA['itemFormElValue'] should be only exploded if it's
not 0. Like I wrote before, this is done in
t3lib\tceforms\class.t3lib_tceforms_tree.php and fixed in the appended
patch.
Fixing this issue causes a little follow up in t3lib\js\extjs\tree\tree.js.
This is also fixed in the patch.
Change-Id: I5d23ea1d63b1107925e265392f4d13f4db3d1063
Resolves: #M17976
Reviewed-on: http://review.typo3.org/1188
Reviewed-by: Moritz <moemos@gmail.com>
Tested-by: Moritz <moemos@gmail.com>
Reviewed-by: Steffen Ritter <info@rs-websystems.de>
Tested-by: Steffen Ritter <info@rs-websystems.de>
Franz G. Jahn [Mon, 14 Mar 2011 20:51:05 +0000 (21:51 +0100)]
[BUGFIX] Extension Manager Function "Backup/Delete => Dump Tables" doesn't work any more
If you try to dump Tables via Extension Manager, you will get a file which looks like this:
...
INSERT INTO static_languages VALUES (NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL);
INSERT INTO static_languages VALUES (NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL, NULL);
...
Reason:
tx_em_database::dumpTableContent uses associative Array $fieldStructure to access the database fields. Although $fieldStructure contains a mapping $field => $structure, the fields are read in a foreach-loop which uses the values of the array.
Change-Id: I6617cb5999d0ce4c184e6f15cbaf0d151bb4516d
Resolves: #M17938
Reviewed-on: http://review.typo3.org/1169
Reviewed-by: Björn Pedersen <bjoern.pedersen@frm2.tum.de>
Reviewed-by: Steffen Kamper <info@sk-typo3.de>
Tested-by: Steffen Kamper <info@sk-typo3.de>
Andreas Wolf [Sat, 12 Mar 2011 08:36:53 +0000 (09:36 +0100)]
[FEATURE] t3lib_DB: Turn off pconnect by default
By default, TYPO3 uses *_pconnect() to connect to the database. This
is however only required in case connecting to the DB server is slow. In
other cases there is no advantage in turning this on, but there may be
disadvantages if the configuration of database and web server are not
aligned.
See also http://www.php.net/manual/en/features.persistent-connections.php
This commit was made based on a discussion on typo3-dev, see message
<mailman.12316.
1298886389.648.typo3-dev@lists.typo3.org> and following.
Change-Id: Ibdc5f5c6045206039cbb23be47e1d2e648a94da5
Reviewed-on: http://review.typo3.org/1137
Tested-by: Mr. Jenkins <blueduck@gmx.net>
Reviewed-by: Xavier Perseguers <typo3@perseguers.ch>
Tested-by: Xavier Perseguers <typo3@perseguers.ch>
Reviewed-by: Oliver Klee <typo3-coding@oliverklee.de>
Reviewed-by: Andreas Wolf <andreas.wolf@ikt-werk.de>
Dmitry Dulepov [Wed, 2 Mar 2011 14:24:01 +0000 (15:24 +0100)]
[BUGFIX] preg_replace error at search for '/'
Change-Id: I7c11ed05d84e7706f479e21c3954288e99e5d982
Resolves: #M17468
Reviewed-on: http://review.typo3.org/974
Reviewed-by: Steffen Kamper <info@sk-typo3.de>
Reviewed-by: Philipp Gampe <forge.typo3.org@philippgampe.info>
Reviewed-by: Jigal van Hemert <jigal@xs4all.nl>
Tested-by: Jigal van Hemert <jigal@xs4all.nl>
Soren Malling [Mon, 7 Mar 2011 20:00:22 +0000 (21:00 +0100)]
[FEATURE] Use common classes for toolbar items
Create common CSS classes to be used for toolbar items so that it looks
the same, and extensions doesn't have to create the same css again and
again.
* Common "drop down" class
* Common ul/li definition
* Common layout, all looks the same
Change-Id: Iedad718ab4f26d87177e3d49ca93f58bf543d9f6
Resolves: #M17888
Reviewed-on: http://review.typo3.org/1088
Reviewed-by: Steffen Gebert <steffen.gebert@typo3.org>
Tested-by: Steffen Gebert <steffen.gebert@typo3.org>
Reviewed-by: Andreas Wolf <andreas.wolf@ikt-werk.de>
Tested-by: Andreas Wolf <andreas.wolf@ikt-werk.de>
Julian Hofmann [Thu, 10 Mar 2011 20:41:00 +0000 (21:41 +0100)]
[BUGFIX] Encryted SMTP does not work
Changes type of $TYPO3_CONF_VARS['MAIL']['transport_smtp_encrypt']
from checkbox to input field. So the user can enter a transport protocol
for connecting the SMTP server.
Change-Id: I1195d640320616234ba0be7cc0328151e44a8999
Resolves: #M17465
Reviewed-on: http://review.typo3.org/1127
Reviewed-by: Steffen Gebert <steffen.gebert@typo3.org>
Tested-by: Steffen Gebert <steffen.gebert@typo3.org>
Reviewed-by: Jigal van Hemert <jigal@xs4all.nl>
Tested-by: Jigal van Hemert <jigal@xs4all.nl>
Markus Klein [Fri, 4 Mar 2011 12:22:50 +0000 (13:22 +0100)]
[BUGFIX] Changing TCA via Tools->Configuration breaks TCA
Changing the TCA (especially the columns-part of a table) via
Tools->Configuration breaks eg. the list module.
Tools->Configuration writes strings like
$GLOBALS['TCA']['tt_news']['columns']['image']['config']['allowed'] = 'jpg';
to typo3conf/extTables.php.
However, a call to loadTCA($table) call is not added before, so
the TCA array is incomplete. Furthermore if you set the columns part
in extTables.php any proceeding call to loadTCA will not work, since
'columns' is already defined.
Change-Id: Ifc4103cdd04b6b063e3d099b5c5ccb47dfb7d442
Resolves: #M17754
Reviewed-on: http://review.typo3.org/1043
Reviewed-by: Steffen Gebert <steffen.gebert@typo3.org>
Tested-by: Steffen Gebert <steffen.gebert@typo3.org>
Reviewed-by: Stefan Galinski <stefan.galinski@gmail.com>
Reviewed-by: Susanne Moog <typo3@susannemoog.de>
Steffen Ritter [Tue, 1 Mar 2011 12:42:24 +0000 (13:42 +0100)]
[BUGFIX] Pages with Enforce Protocol (SSL) may generate wrong URL
If TYPO3 is installed in a subfolder, you won't need a absRefPrefix set because normally links are relative.
If a Protocol change will (SSL -> PLAIN or vice versa) happen, the URL has to be generated absolute.
This will fail in the described behaviour.
Notes:
This one has been reported by Michael Maier.
Attached patch is my "easier to read" version doing the same as _v2 of Jigal and the result of talking through all cases which may arise.
Change-Id: Ie5821361b578b6f1fa3612ac9bfa44dd476c9c76
Resolves: #M17118
Reviewed-on: http://review.typo3.org/953
Reviewed-by: Tobias Liebig <mail_typo3.org@etobi.de>
Reviewed-by: Andreas Wolf <andreas.wolf@ikt-werk.de>
Tested-by: Andreas Wolf <andreas.wolf@ikt-werk.de>
Reviewed-by: Sebastian Fischer <typo3@evoweb.de>
Reviewed-by: Helmut Hummel <typo3@helmut-hummel.de>
Tested-by: Helmut Hummel <typo3@helmut-hummel.de>
Reviewed-by: Jigal van Hemert <jigal@xs4all.nl>
Jo Hasenau [Fri, 4 Mar 2011 08:42:13 +0000 (09:42 +0100)]
[BUGFIX] List module - backend layout - moving content elements
When moving a content element in the List module by clicking on
the "Re-position content element" icon, an user is presented with
the old 4 columns instead of columns that would represent the newly
introduced backend layout as it is appearing in the Page module.
Change-Id: Ie8120784b0ccb9036d012437cae966de3484f1e6
Fixes: #M17529
Reviewed-on: http://review.typo3.org/1040
Reviewed-by: Andreas Wolf <andreas.wolf@ikt-werk.de>
Tested-by: Andreas Wolf <andreas.wolf@ikt-werk.de>
Bjoern Pedersen [Mon, 7 Mar 2011 10:12:08 +0000 (11:12 +0100)]
[BUGFIX] Missing initialization of DefaultBEMode in feedit
Formsonopage-mode raises error: Call to undefined method stdClass::getDynNestedStack.
Thus call initDefaultBEMode() during initialization.
Change-Id: If3ffc4b4fb3aff0dac72f3cbca405cb74d082783
Resolves: #M15553
Reviewed-on: http://review.typo3.org/1078
Reviewed-by: Björn Pedersen <bjoern.pedersen@frm2.tum.de>
Tested-by: Andreas Wolf <andreas.wolf@ikt-werk.de>
Reviewed-by: Andreas Wolf <andreas.wolf@ikt-werk.de>
Peter Niederlag(office) [Tue, 8 Mar 2011 10:16:18 +0000 (11:16 +0100)]
[FEATURE] add .gitignore for common file exclusions
porting what was in svn:ignore before
needs to be aplied to all branches separatly, once we agree
Change-Id: If03b9e4796604056acadf3bf662f7990c84ca8b5
Fixes: #13627
Reviewed-on: http://review.typo3.org/1096
Tested-by: Andreas Wolf <andreas.wolf@ikt-werk.de>
Reviewed-by: Steffen Ritter <info@rs-websystems.de>
Tested-by: Steffen Ritter <info@rs-websystems.de>
Reviewed-by: Andreas Wolf <andreas.wolf@ikt-werk.de>
steffenk [Thu, 3 Mar 2011 19:02:24 +0000 (20:02 +0100)]
[BUGFIX] Additions to fileDenyPattern give security warning in BE
The check is wrong. Instead of checking if the entry is equal to default entry, it should check if any parts of the default value are removed.
As it's a security warning, this should be stated in the warning, so change the wording.
Change-Id: If20d362bef39a4d8533a54e8124b4913b9dcb55e
Resolves: #M17817
Reviewed-on: http://review.typo3.org/1031
Tested-by: Andreas Wolf <andreas.wolf@ikt-werk.de>
Reviewed-by: Steffen Ritter <info@rs-websystems.de>
Tested-by: Steffen Ritter <info@rs-websystems.de>
Reviewed-by: Andreas Wolf <andreas.wolf@ikt-werk.de>
Steffen Gebert [Mon, 7 Mar 2011 16:13:25 +0000 (17:13 +0100)]
[BUGFIX] Throw correct Exceptions everywhere
Improve the way, TYPO3 is throwing Exceptions:
* Add a unique error code to all Exception($msg, $code) calls
* Replace die() statements with Exceptions, if possible
* Replace some general Exceptions with RuntimeException or ArgumentException.
Change-Id: Ic148f14b9a71136c917e9dc8b95513341b3c6ca0
Resolves: #M16953
Reviewed-on: http://review.typo3.org/1079
Reviewed-by: Andreas Wolf <andreas.wolf@ikt-werk.de>
Tested-by: Andreas Wolf <andreas.wolf@ikt-werk.de>
Reviewed-by: Steffen Ritter <info@rs-websystems.de>
Francois Suter [Thu, 3 Mar 2011 20:15:09 +0000 (21:15 +0100)]
[BUGFIX] Deleted overlay missing in overlay priorities
The "deleted" overlay type is missing from the list of overlay
priorities as defined in
$TBE_STYLES['spriteIconApi']['spriteIconRecordOverlayPriorities'].
This patch fixes this by adding it as the first item in the list,
since one can assume "deleted" to be the most important status.
NOTE: it's hard to view the result of this patch as deleted records
are not normally shown in the TYPO3 BE. You may have deleted
records when working in a workspace, but they will still not be
visible after this patch due to a further bug in t3lib_iconworks.
See http://bugs.typo3.org/view.php?id=17117 for a discussion of that
further bug.
To view the effect of this patch, apply the patch from #M17117,
go to a workspace, delete an unversioned record and see the icon
change.
Change-Id: Ie2c1f6ef672c5eb73fbc37066fd7596380c64932
Resolves: #M17815
Reviewed-on: http://review.typo3.org/1089
Reviewed-by: Steffen Ritter <info@rs-websystems.de>
Tested-by: Steffen Ritter <info@rs-websystems.de>
Steffen Ritter [Tue, 1 Mar 2011 12:21:24 +0000 (13:21 +0100)]
[TASK] Remove SpriteGenerator IE6 Fallback
The spriteGenerator has features to generate GIF variants (and according CSS) as a fallback for IE6.
Remove it as it is not used anymore (due to #M17805).
Resolves #M17808
Depends #M17805
Change-Id: I6fae8845d4c2ecae3ad56aa8c143dd2e1d76a8b3
Reviewed-on: http://review.typo3.org/952
Reviewed-by: Stefan Galinski <stefan.galinski@gmail.com>
Reviewed-by: Philipp Gampe <forge.typo3.org@philippgampe.info>
Reviewed-by: Andreas Wolf <andreas.wolf@ikt-werk.de>
Tested-by: Andreas Wolf <andreas.wolf@ikt-werk.de>
Tolleiv [Wed, 2 Mar 2011 19:56:44 +0000 (14:56 -0500)]
[TASK] Cleanup sys_workspace mentions in t3lib/stddb/tables.php
Change-Id: I358c131274c00efa60193f958c6f1e378eff9857
Resolves: #M17838
Resolves: #12616
Reviewed-on: http://review.typo3.org/991
Reviewed-by: Philipp Gampe <forge.typo3.org@philippgampe.info>
Reviewed-by: Susanne Moog <typo3@susannemoog.de>
Tested-by: Susanne Moog <typo3@susannemoog.de>
Reviewed-by: Ringer Georg <mail@ringerge.org>
Tested-by: Ringer Georg <mail@ringerge.org>
Reviewed-by: Andreas Wolf <andreas.wolf@ikt-werk.de>
Tested-by: Andreas Wolf <andreas.wolf@ikt-werk.de>
Steffen Kamper [Tue, 8 Mar 2011 12:42:25 +0000 (13:42 +0100)]
[BUGFIX] Wrong label in felogin redirectDisable
Wrong label in felogin redirectDisable.
The label says "Enable Redirect"
From functionality it should be "Disable Redirect" as it was before.
Change-Id: I5f7015e15eac905e5b85d1f3dcc008fec741fd54
Resolves: #M17466
Refs: master,TYPO3_4-5
Reviewed-on: http://review.typo3.org/1094
Reviewed-by: Oliver Hader <oliver@typo3.org>
Tested-by: Oliver Hader <oliver@typo3.org>
Dmitry Dulepov [Thu, 3 Mar 2011 13:38:03 +0000 (14:38 +0100)]
[FEATURE] Sort plugin names in BE form alphabetically
Change-Id: Ic470c0418008b601a14a1cc1bccec13226cb2ae0
Resolves: #M17856
Reviewed-on: http://review.typo3.org/1010
Reviewed-by: Andreas Wolf <andreas.wolf@ikt-werk.de>
Tested-by: Andreas Wolf <andreas.wolf@ikt-werk.de>
steffenk [Sun, 6 Mar 2011 11:19:05 +0000 (12:19 +0100)]
[BUGFIX] New EM - cannot save files with noEdit disabled
Due a typo the setting is not recognized correct.
Change-Id: Icb7cbf93e3dcc5ba4f22b1cad59c3c61ab62bcd3
Resolves: #M17811
Reviewed-on: http://review.typo3.org/1067
Reviewed-by: Steffen Kamper <info@sk-typo3.de>
Tested-by: Steffen Kamper <info@sk-typo3.de>
Steffen Gebert [Sun, 6 Mar 2011 13:57:22 +0000 (14:57 +0100)]
[BUGFIX] Typo in top.TYPO3.ModuleMenu
Change TYPO3.ModuleMenu.selecteModule to selectedModule.
Change-Id: Ie0faa9d2d72c4c51de8a92159323ee63afd620b4
Resolves: #M17875
Reviewed-on: http://review.typo3.org/1069
Reviewed-by: Steffen Kamper <info@sk-typo3.de>
Tested-by: Steffen Kamper <info@sk-typo3.de>
Reviewed-by: Tolleiv Nietsch <info@tolleiv.de>
Reviewed-by: Susanne Moog <typo3@susannemoog.de>
Tested-by: Susanne Moog <typo3@susannemoog.de>
steffenk [Fri, 4 Mar 2011 11:21:16 +0000 (12:21 +0100)]
[BUGFIX] EM interface for renderHook is missing
Seems this file was not added with latest merge, add it
Change-Id: I7a6cf6edc6ccdb8e39f38c702fedf4dd1c766c8b
Resolves: #M17865
Reviewed-on: http://review.typo3.org/1041
Reviewed-by: Christian Mueller <christian@kitsunet.de>
Reviewed-by: Steffen Kamper <info@sk-typo3.de>
Tested-by: Steffen Kamper <info@sk-typo3.de>
Steffen Ritter [Tue, 1 Mar 2011 10:05:35 +0000 (11:05 +0100)]
[TASK] Remove specials for IE6 from t3skin
IE6 Supports will be dropped in 4.6...
There are some special CSS files, 2 php classes and sprites as GIF Icons.
Remove the special IE6 handling in t3skin
Change-Id: I2151004f574fb2b3891ff28914814a94a3eb97e0
Resolves: #M17805
Reviewed-on: http://review.typo3.org/949
Reviewed-by: Tobias Liebig <mail_typo3.org@etobi.de>
Reviewed-by: Jigal van Hemert <jigal@xs4all.nl>
Tested-by: Jigal van Hemert <jigal@xs4all.nl>
steffenk [Fri, 4 Mar 2011 10:00:18 +0000 (11:00 +0100)]
[BUGFIX] Error 414 - "Request-URI Too Large" when saving file in extension manager (old version)
Due a missing closing " the form has invalid HTML that result in a very long GET request
Change-Id: I0e369dec5cb91845402daaece82b215e7a6f7dd0
Resolves: #M17812
Reviewed-on: http://review.typo3.org/1035
Reviewed-by: Christopher <chrissitopher@gmx.de>
Tested-by: Christopher <chrissitopher@gmx.de>
Reviewed-by: Steffen Kamper <info@sk-typo3.de>
Tobias Liebig [Wed, 2 Mar 2011 15:00:11 +0000 (16:00 +0100)]
[+FEATURE] t3editor: use the t3editor in TCA-Forms
Change-Id: Ib61434c6a959d492dcee42b2a38631482c93b713
Resolves: #M17863
Reviewed-on: http://review.typo3.org/1033
Reviewed-by: Markus Klein <klein.t3@mfc-linz.at>
Tested-by: Markus Klein <klein.t3@mfc-linz.at>
Reviewed-by: Jochen Rau <j.rau@web.de>
Tested-by: Jochen Rau <j.rau@web.de>
Stefan [Wed, 2 Mar 2011 15:18:09 +0000 (16:18 +0100)]
[BUGFIX] Pagetree - Selection state incorrect after a cancelled edit
Currently the selection state of a node is set to the wrong node if you cancel an inline edit.
Change-Id: I2d0af2a4e37c975ef7885ee3cc4ea872d5a405bc
Resolves: #M17828
Reviewed-on: http://review.typo3.org/979
Reviewed-by: Stefan Galinski <stefan.galinski@gmail.com>
Tested-by: Stefan Galinski <stefan.galinski@gmail.com>
Stefan [Wed, 2 Mar 2011 17:19:49 +0000 (18:19 +0100)]
[BUGFIX] pagetree->expand contextmenu action does not work as expected
When I click "expand branch" in the new pagetree once it simply expands the next level und the page where I opened the context menu.
When I click the same context menu link (at the same page) a second time it expands all pages on all levels below the page where I clicked.
I'd expect the tree either doing the first OR the second action but not 2 different actions on the same useraction.
Change-Id: I4f6d2b5342e0d8c5f4e89d71b2527f8c640ad30e
Resolves: #M17394
Reviewed-on: http://review.typo3.org/986
Reviewed-by: Stefan Galinski <stefan.galinski@gmail.com>
Tested-by: Stefan Galinski <stefan.galinski@gmail.com>
Oliver [Thu, 3 Mar 2011 13:52:16 +0000 (14:52 +0100)]
[BUGFIX] Drop the checks for the image file size from the install tool
The file sizes of the generated images in the install tool may somewhat differ in size while
the images are perfectly fine. Thus these checks do not provide helpful information, but
produce false warnings. This change removes this image file size check.
Change-Id: I60fb288945ed56e42940218c95a38f7202bf05bb
Resolves: #M17849
Reviewed-on: http://review.typo3.org/1012
Reviewed-by: Philipp Gampe <forge.typo3.org@philippgampe.info>
Tested-by: Philipp Gampe <forge.typo3.org@philippgampe.info>
Reviewed-by: Christian Mueller <christian@kitsunet.de>
Reviewed-by: Arvid Jakobsson <arvid.jakobsson@gmail.com>
Reviewed-by: Oliver Klee <typo3-coding@oliverklee.de>
Reviewed-by: Dmitry Dulepov <dmitry@typo3.org>
Tested-by: Dmitry Dulepov <dmitry@typo3.org>
Xavier Perseguers [Thu, 3 Mar 2011 14:58:51 +0000 (15:58 +0100)]
[FEATURE] Hook Request for t3lib_db
For the introduction of uuid within TYPO3, a few hooks need to be added to t3lib_DB.
Change-Id: I3822710055d8d735e615d80f1d979d0c87e94632
Resolves: #13579
Reviewed-on: http://review.typo3.org/1023
Reviewed-by: Thomas Maroschik <tmaroschik@dfau.de>
Tested-by: Thomas Maroschik <tmaroschik@dfau.de>
Reviewed-by: Xavier Perseguers <typo3@perseguers.ch>
Tested-by: Xavier Perseguers <typo3@perseguers.ch>
Oliver [Wed, 2 Mar 2011 19:24:08 +0000 (20:24 +0100)]
[TASK] Create a first proof-of-concept unit test for t3lib_TCEmain::process_datamap
Resolves: #M17836
Change-Id: I92fa2b9d080e819b81451de3571f101a5d39a235
Reviewed-on: http://review.typo3.org/990
Reviewed-by: Ringer Georg <mail@ringerge.org>
Tested-by: Ringer Georg <mail@ringerge.org>
Reviewed-by: Tolleiv Nietsch <info@tolleiv.de>
Tested-by: Tolleiv Nietsch <info@tolleiv.de>
Oliver Hader [Wed, 2 Mar 2011 18:39:26 +0000 (19:39 +0100)]
[BUGFIX] Import/Export module uses page id as tree depth
The parameter tx_impexp[pagetree][levels] must be set to zero in this case.
Change-Id: I3828cac7d38117e5a3045a1ac983ce538b629f86
Resolves: #M17814
Reviewed-on: http://review.typo3.org/988
Reviewed-by: Stefan Galinski <stefan.galinski@gmail.com>
Tested-by: Stefan Galinski <stefan.galinski@gmail.com>
Reviewed-by: Oliver Hader <oliver@typo3.org>
Sebastian [Wed, 2 Mar 2011 14:58:50 +0000 (15:58 +0100)]
[TASK] Changed copyright note to current year
Changed the copyright note in javascript files as they are not changed automaticaly
Change-Id: I6a275e0567c98b6b97603a9f2d87070bd8ad3737
Reviewed-on: http://review.typo3.org/978
Reviewed-by: Steffen Ritter <info@rs-websystems.de>
Tested-by: Steffen Ritter <info@rs-websystems.de>
Reviewed-by: Stefan Galinski <stefan.galinski@gmail.com>
Tested-by: Stefan Galinski <stefan.galinski@gmail.com>