[TYPO3-core] RFC: Bug #12598: Use flash messages to show SQL debug errors in the backend

[Martin Kutschker]

Oliver Hader schrieb:
> Hi Masi,
> 
> Martin Kutschker schrieb:
>> Oliver Hader schrieb:
>>> This is an SVN patch request.
>>>
>>> Type: Bugfix
>>>
>>> Bugtracker references:
>>> http://bugs.typo3.org/view.php?id=12598
>>>
>>> Branches: Trunk
>>>
>>> Problem:
>>> If sqlDebug is enabled to visualize SQL errors this currently breaks the
>>> document and in TCEforms buttons cannot be clicked anymore.
>>>
>>> Solution:
>>> Flash messages should be used to solve this issue.
>> -1 Checking for the mode in t3lib is the wrong place to do it. Is there really no other way?
>>
>> How about adding a context property * that handles error conditions. The BE could provide a context
>> that uses the flash messages.
>>
>> * in v5 this would probably be handled by dependency injection.
> 
> I don't understand what you're talking about and what that means for v4.
> Can you give an example for the use-case of the sqlDebug thingy?

Forget the dependency injection. What I'm talking about are the comparisons like that "TYPO3_MODE
=== 'BE'" that creep into t3lib. That's IMHO horrible. t3lib shouldn't need to know about the mode.
It's like putting all MVC classes into one

Masi