TYPO3 Core

After having a look at the current source code, I can assure you, that there is no difference in handling of core tables and ext tables.

Please upload your extTables.php so I can test with this one.

Thank you for your response.

I rechecked this with a quite simple setup, so no conflicts ocour.

What I did:

• Typo3 4.5.1 installed dummy (tgz)
• Import default database (1-2-3 installer)
• NO further setup.
• log into backend.
• install tt_news (latest from TER).
• create a test-page and create one record with propper title set.

  -> BE listview of the Record is OK.

• Goto Tools>Configuration and Edit the TCA of tt_news:
  e..g
  $GLOBALS['TCA']['tt_news']['columns']['image']['config']['allowed'] = 'gif,jpg,jpeg,tif,tiff,bmp,pcx,tga,png,pdf,ai';

(this does not change anything to the configuration exept this line is written into extTables.php, cause this is the default value)

• Recheck the Listview -> Broken, Title is "No Title" and you cannot edit the Record (blank page).

next try: maybe ist a tt_-Problem (I remember tt_ is like user some exeption in TYPO3 due to historical reasons).
Install Some EXT with different name e.g. ve_guestbook

• Create a Record of this EXT
• Add e.g.
  $GLOBALS['TCA']['tx_veguestbook_entries']['columns']['firstname']['config']['max'] = 25;

-> and you see: No Title.

If you remove the $TCA lines, is works like expected. titles are shown again.

And finally try this with some EXT without even a underscroe in its key: irfaq
Guess -> braken list view, no edit...

But still:
tt_content is not affected if you set
$GLOBALS['TCA']['tt_content']['columns']['date']['config']['default'] = 0;

And last not least, switch to source back to V4.4.4
OK, now the view is broken as well...... maybe not well tested before Sorry for this.

But this is a Bug, isn't it?

Since I have used a relly minimalistic setup, I hope you can repeat this steps easyly.

again checked with 4.5.0,4.5.1,4.5.2, 4.3.9, 4.2.16 (halleluja to the symlinks!)

So you're updating this directly with Tools>Configuration?
Did you check what the content of extTables.php is, after you changed a value there? Or can you post it here?

Thx.

So you're updating this directly with Tools>Configuration?

Yes

Did you check what the content of extTables.php is, after you changed a value there? Or can you post it here?

The example lines $GLOBALS['TCA']... are directly taken from extTables.php.
in extTables.php you only find this lines and the default examples (commented out, you know this).

Does your extTables.php contain the lines:
?> ??

Remove them!

This should actually be fixed automatically by some some lowlevel config script.. No idea why it's not.

Does your extTables.php contain the lines:

Yes.

Removing stops brekink the BE, but the definitions in this file are not set.

I tried this with some die(); command.

With the die() is executed, without not.
And the definitions are not taking effect (not even wrongly) if is missing.

This should actually be fixed automatically by some some lowlevel config script.. No idea why it's not.

So you can reproduce this behavoir?

Yes I tried exactly with your News lines above.

You can search for TYPO3_extTableDef_script in the core. (See extTablesUsage.jpg)
This is a definition from config_default.php and contains the reference to extTables.php.

Whenever this file is used it is only included like:
include (PATH_typo3conf.TYPO3_extTableDef_script);

And in typo3/sysext/lowlevel/config/index.php there's this code:
$extTables = t3lib_div::getURL(PATH_typo3conf . TYPO3_extTableDef_script);
$extTables = '/is', '', $extTables) . $changedLine . LF . '?>';
$success = t3lib_div::writeFile(PATH_typo3conf . TYPO3_extTableDef_script, $extTables);

I'm off for today. I'll checkout 4.4.4 tomorrow and compare things, maybe we find a difference.

Hmmm....

Since I know this script/features did work, I will wait until someone of the core-Team-Gods will take a look on this strange thing.

Thank you Markus for your help so far.

Got it!

TCA is not fully loaded at time of inclusion.

Doing this works out of the box:
t3lib_div::loadTCA('tt_news');
$GLOBALS['TCA']['tt_news']['columns']['image']['config']['allowed'] = 'gif,jpg,jpeg,tif,tiff,bmp,pcx,tga,png,pdf';

Thanks Markus for analysing the problem. Thus, the behaviour was just obvious that modifying values that are not loaded yet does not work.
So we could add the loadTCA() call to the Tools>Configuration module if anything else than 'ctrl' gets modified here.

So this means, that for tt_content it's only working without loadTCA because it is loaded by some extension extTables.php?

And here is the patch.
Please test and I'll post it to core list.

And v2 beautifys the flash message.

Thank you a lot, Markus - you made my day!

Thus, the behaviour was just obvious

Now as you say this, I remember this, too. I read this, somewhere some ages ago (insideTypo3-manual?).

But i am not one of this hard-core-TCA-Junkies, so i forgott about.

So we could add the loadTCA()

I dont think it should be added there, cause: The suggested call in Tools>Configuration is "wrong".
for inclusion in extTables.php you have to change it from $TCA... (usable for tca.php, where loadTCA is called) to
$GLOBALS['TCA']...

it should be enough to include in the extTables.php file some commented hint to this "Problem"/behaviour with none tt_content-tables.

From my point of view this case may be closed as solved.

I dont think it should be added there, cause: The suggested call in Tools>Configuration is "wrong".
for inclusion in extTables.php you have to change it from $TCA... (usable for tca.php, where loadTCA is called) to
$GLOBALS['TCA']...

Sorry I don't get your point here. Can you explain in more detail please?

quote from default extTables.php:

// Raise upload limit for images in 'image' content-elements to 10*1024 bytes = 1MB
$GLOBALS['TCA']['tt_content']['columns']['image']['config']['max_size'] = 10*1024;

But if you do this with Tools>Configuration, the code to add says:

$TCA['tt_content']['columns']['image']['config']['max_size'] = 20480;

so this is the reason why I belive this must be changed to $GLOBALS['TCA'].

Short: is there a reson for the examples given in extTables.php differ from suggestion in Tools>Configuration?

confused....???

is $GLOBALS['TCA'] == $TCA ?
Does your patch still require the loadTCA() statement?

$TCA is the same as $GLOBALS['TCA'] if you import it from global scope by writing something like "global ...".

Yes Tools>Configuration generates this:
$TCA['tt_conten....
But
When you click on the button this is replaced by
$GLOBALS['TCA']['tt_cont.....
and then written to extTables.php

I see... tryed it and: yes, it's the same.
Thank your for your help, again

:-)

Did you test my patch?

Please vote for it on core list!

Just load TCA for the table before trying to modify it... t3lib_div::loadTCA() is all you need and must do actually.

No change necessary in TYPO3.

Hi Dmitry,

but that's exactly what my patch does.
If the user uses Tools->Configuration for making changes to TCA he will not modify extTables.php manually, so TYPO3 should take care of added the required loadTCA.

Please logon to Gerrit and review the patch there.

http://review.typo3.org/

User/Pwd same as for bugtracker here.
Thx.

Merged to master 246e57d484123fcf2617d758118ade456bb124a5

Review for 4-5: http://review.typo3.org/1128

Merged to 4-5 in 42da5edaab8d29e99bc69475e5aa4eb74b205691