Project

General

Profile

Actions

Bug #15154

closed

There are errors in the code

Added by Tomas Matousek over 18 years ago. Updated over 15 years ago.

Status:
Closed
Priority:
Should have
Assignee:
-
Category:
-
Target version:
-
Start date:
2005-10-26
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
3.8.0
PHP Version:
5
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

We are working on a PHP compiler (Phalanger, http://php-compiler.net) and testing it against various PHP applications. We have tested also Type3 and our compiler reported the following warnings and errors. We think these are really issues of the code and not the compiler itself. If you find some of these reports false, please, report.

C:\Web\Typo3\misc\phpcheck\incfile.php(228,15): error: Class 't3lib_div' does not contain definition of method 'view_array'
C:\Web\Typo3\t3lib\class.t3lib_iconworks.php(100,44): warning: Mandatory formal parametr 'backPath' declared behind optional formal parameter
C:\Web\Typo3\t3lib\class.t3lib_iconworks.php(444,92): error: Parameter 'w' in function/method declaration specified twice
C:\Web\Typo3\t3lib\class.t3lib_stdgraphic.php(487,92): error: Parameter 'w' in function/method declaration specified twice
C:\Web\Typo3\t3lib\class.t3lib_tsfebeuserauth.php(805,46): warning: Mandatory formal parametr 'perms_clause' declared behind optional formal parameter
C:\Web\Typo3\tslib\media\scripts\gmenu_foldout.php(176,21): error: Cannot use [] for reading
C:\Web\Typo3\typo3\ext\kickstarter\class.tx_kickstarter_sectionbase.php(127,5): warning: The 'case' label with value fields has already been declared
C:\Web\Typo3\typo3\ext\kickstarter\class.tx_kickstarter_sectionbase.php(141,5): warning: The 'case' label with value fields has already been declared
C:\Web\Typo3\typo3\ext\phpmyadmin\modsub\phpmyadmin-2.6.2\libraries\common.lib.php(765,40): warning: Mandatory formal parametr 'in' declared behind optional formal parameter
C:\Web\Typo3\typo3\ext\tt_board\pi\class.tx_ttboard.php(764,48): warning: Mandatory formal parametr 'searchWord' declared behind optional formal parameter
C:\Web\Typo3\typo3\sysext\cms\tslib\media\scripts\gmenu_foldout.php(176,21): error: Cannot use [] for reading
C:\Web\Typo3\typo3\sysext\indexed_search\modfunc1\class.tx_indexedsearch_modfunc1.php(398,42): warning: Mandatory formal parametr 'extraGrListRows' declared behind optional formal parameter
C:\Web\Typo3\typo3\sysext\indexed_search\modfunc2\class.tx_indexedsearch_modfunc2.php(186,48): warning: Mandatory formal parametr 'perms_clause' declared behind optional formal parameter
C:\Web\Typo3\typo3\sysext\indexed_search\pi\class.tx_indexedsearch.php(1582,40): warning: Mandatory formal parametr 'specRowConf' declared behind optional formal parameter
C:\Web\Typo3\typo3\t3lib\class.t3lib_iconworks.php(100,44): warning: Mandatory formal parametr 'backPath' declared behind optional formal parameter
C:\Web\Typo3\typo3\t3lib\class.t3lib_iconworks.php(444,92): error: Parameter 'w' in function/method declaration specified twice
C:\Web\Typo3\typo3\t3lib\class.t3lib_stdgraphic.php(487,92): error: Parameter 'w' in function/method declaration specified twice
C:\Web\Typo3\typo3\t3lib\class.t3lib_tsfebeuserauth.php(805,46): warning: Mandatory formal parametr 'perms_clause' declared behind optional formal parameter

(issue imported from #M1731)

Actions #1

Updated by Michael Scharkow over 18 years ago

I can verify some but not all warnings and errors. Will have a look at it, although it does not seem critical. I will probably split this into different bugs, be patient.

Actions #2

Updated by Christian Kuhn over 15 years ago

Closing this very old bug:

Please open new entries for still valid warnings / errors and split into separate bugs.

Actions

Also available in: Atom PDF