Bug #22037
closedmod.web_list.hideTables does not accept spaces
0%
Description
If you enter the following config in the page tsconfig, table tt_content will be hidden, but not tt_news :
mod.web_list.hideTables = tt_content, tt_news
This is due to the space after the comma. If you remove the space, both tables will be hidden.
(issue imported from #M13386)
Files
Updated by Chris topher almost 15 years ago
Since it works correctly without the space, I don't see a problem here.
This is a "comma seperated list" and seperation with comma works as expected.
Updated by Calderini Daniel almost 15 years ago
You're right, it works as expected. But it is a bit inconsistent with other options.
For example:
mod.web_list.allowedNewTables
mod.web_list.deniedNewTables
Both accept spaces between table names, even if the data type is the same as hideTables (list of tablenames).
So maybe it just needs to be written in doc_core_tsconfig that spaces are not allowed...
Updated by Marcel Greter almost 15 years ago
IMO whitespace should be allowed as one might want to write something like that:
mod.web_list.hideTables (
tx_extension_table1,
tx_extension_table2,
tx_extension_table3,
tx_extension_table4,
...
)
Updated by Georg Ringer almost 15 years ago
problem is not the hideTables but t3lib_div::inList which doesn't work with spaces in comma seperated lists
Updated by Marcel Greter over 14 years ago
Is there a reason not to use t3lib_div::trimExplode and in_array as it is with allowedNewTables and deniedNewTables? IMO I don't see any (actually I think it should have been that way from the start)?
I posted a patch as the necessary changes are very small and easy to do.
Updated by Benni Mack over 14 years ago
Hey Marcel.
Care to send this to the TYPO3 core list?
Updated by Marcel Greter over 14 years ago
Hi Benjamin,
I did submit it to the list on March 8, but did not receive any feedback at all.
Best wishes, Marcel
Updated by Christian Kuhn over 14 years ago
Hey Marcel, I just reviewed your patch in the core list. Thank for your patience.