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

[Michael Stucki]

Hi guys,

> 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?

Sorry but I think you're loosing track here. The discussion is now about
overriding sysadmin settings although this is exactly how it has been done
for ages!

The real problem is that if you really need _more_ than 256M, the value is
still hardcoded (currently) and there is no way to change this except
patching the code.

So please let's fix this problem first and discuss about any way of
improving the impexp part later (in fact, I would very much welcome this!)

- michael
-- 
Use a newsreader! Check out
http://typo3.org/community/mailing-lists/use-a-news-reader/