Revision 3030 Overloading Alert

[GoogleCodeExporter]

In revision 3030, I have this alert:

[Runtime Notice] Declaration of TMysqlMetaData::quoteTableName() should be 
compatible with that of TDbMetaData::quoteTableName() (@line 385 in file 
D:\webs\prado\prado-3.2a\framework\Data\Common\Mysql\TMysqlMetaData.php).

I use php v5.3.5

I update for this temporal solution for now:

Line 47:    
public function quoteTableName($name)
change by:
public function quoteTableName($name, $lft=null, $rgt=null)

Line 57:
public function quoteColumnName($name)
change by:
public function quoteColumnName($name, $lft=null, $rgt=null)

Line 67:
public function quoteColumnAlias($name)
change by:
public function quoteColumnAlias($name, $lft=null, $rgt=null)

Original issue reported on code.google.com by mario...@hotmail.com on 21 Sep 2011 at 3:05

Attachments:

• TMysqlMetaData.php

[GoogleCodeExporter]

Original comment by GODZilla...@gmail.com on 21 Sep 2011 at 3:25

• Changed state: Accepted

[GoogleCodeExporter]

Hmm, difficult. This was done intentionally, not the "Runtime Notice", these 
were not triggered on my developer-machine, but i'll check my error_reporting 
configuration.

Since the driver-specific implementation knows the delimiters, it makes sense 
to remove delimiter-parameters in driver implementation to avoid confusion.

I'll think about a solution.

Original comment by GODZilla...@gmail.com on 21 Sep 2011 at 4:03

[GoogleCodeExporter]

This issue was closed by revision r3049.

Original comment by GODZilla...@gmail.com on 16 Oct 2011 at 5:25

• Changed state: Fixed

[GoogleCodeExporter]

OK my fault!
I configured error_reporting = "E_ALL | E_STRICT" in php.ini,
but conditionaly override setting depending on an ENV variable where E_STRICT 
was removed *argh*

But i committed a solution - Sorry for the inconvenience.

Original comment by GODZilla...@gmail.com on 16 Oct 2011 at 5:37