Project

General

Profile

Actions
Copy link

Bug #20576

closed

t3lib_div has several calls to debug without namespace which leads to php-errors

Added by Steffen Ritter over 15 years ago. Updated over 13 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
-
Target version:
-
Start date:
2009-06-05
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
4.3
PHP Version:
5.2
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

In t3lib_div there are many calls to t3lib_div::debug... Some of them lack the namespace/class and therefore only call debug() which actually leads to php error.

Simple solution: just replace such calls with correct ones.
(issue imported from #M11279)

Actions
Copy link
 #1

Updated by Oliver Hader about 15 years ago

Hi Steffen,
can you please give an example of these errors? What is shown? I never experienced a behaviour like this. Which PHP version are you using?

Actions
Copy link
 #2

Updated by Felix Oertel about 15 years ago

Hi,

ok, please kindly ignore my patch. ;)

The problem is not calling debug() WITHOUT namespace but WITH. In typo3/config_default.php a debug() without namespace is defined and I think this is gonna be used.

I started a discussion on the dev-list.

Regards

Actions
Copy link
 #3

Updated by Benni Mack almost 15 years ago

Hey Felix,

I just read through the dev-list thread. Basically this means we should consider making all calls to t3lib_div::debug(), but keep the old one for the sake of backwards-compatibility?

Actions
Copy link
 #4

Updated by Felix Oertel almost 15 years ago

Hi Benjamin,

as far as I can remember, we are supposed to call debug() without t3lib_div::. The debug() defined in typo3/config_default.php looks for alternative ways to handle debug-output and calls t3lib_div::debug() as a fallback.

But it would be a good idea to move debug() out of config_default.php because it's not that much a config-thing. ;)

Regards, Felix

Actions
Copy link
 #5

Updated by Benni Mack over 14 years ago

REMINDER. Felix, it would be great if you could fix this issue! Looking forward to that change.

Actions
Copy link
 #6

Updated by Benni Mack over 14 years ago

REMINDER #2: Felix, could you please let us know if we can close this issue? According to the core list, this change was a misunderstanding...? Or did I miss something.

Actions
Copy link
 #7

Updated by Tolleiv Nietsch over 13 years ago

  • Status changed from Needs Feedback to Closed
  • Target version deleted (-1)

Closing this for the sake of a clean bugtracker - feel free to open it again once you're able to provide new input.

Actions
Copy link

Also available in: Atom PDF