[TYPO3-core] RFC #8649: Bug: No check on memory_limit in class.tx_impexp.php

[Dmitry Dulepov [typo3]]

Hi!

Michael Stucki wrote:
> Which admin wants to block users from using a feature which the user has
> permission to use? If the memory limit is 32 MB (that is sufficient in many
> cases) we still know that impexp usually needs more - a lot more - memory.

Michael, did you read what I wrote actually? Admin can set memory_limit in Install tool exactly for this purpose! This is system wide setting and it must be the absolute maximum! Read the description of the setting! People will not expect TYPO3 to disobey this value.

If admin wants people to use import/export, he should set sufficient memory_limit in Install tool. This is my idea. And it is fully inline with current implementation, for example, Extension Manager. I do not see why import/export should behave differently from Extension Manager. Extension Manager uses system wide memory_limit, so should import/export. Otherwise we will end up setting memory_limit in every module in the system. Is this what you want really?

-- 
Dmitry Dulepov
TYPO3 Core team
More about TYPO3: http://typo3bloke.net/
Subscribe: http://typo3bloke.net/rss.xml