Bug #617

Warnings from PhpEclipse

Details

Submitted	2004-01-24 11:00 UTC
From	pear-01 at oliverklee dot de
Assigned	danielc
Status	Closed
Package	DB
PHP Version	Irrelevant
OS	Linux
Roadmaps	(Not assigned)

Comments

[2004-01-24 11:00 UTC] pear-01 at oliverklee dot de

Description:
------------
When opening the DB 1.5.0rc2 package in PhpEclipse, I get the following Warnings and Errors for DB.php:

Line 229: Error: ';' expected after expression (Found token: Variable($classname))

This can be fixed by changing the line
@include_once("DB/${type}.php");
to
if (is_array($options) && isset($options["debug"]) &&
$options["debug"] >= 2) {
// expose php errors with sufficient debug level
include_once "DB/${type}.php";
} else {
@include_once "DB/${type}.php";
}

Line 707: ':' expected after 'case' keyword (Found token: ;)

Solution: Change
case 'limit_count';
$this->limit_count = $value; break;
to
case 'limit_count':
$this->limit_count = $value; break;

At line 611: Too many closing '}'; end-of-file not reached.

Solution: Count the braces with a good editor (better than PhpEclipse).

[2004-01-24 11:07 UTC] pear-01 at oliverklee dot de

Looks like the fix for the first warning also fixes the third one.

[2004-01-25 18:40 UTC] pear-01 at oliverklee dot de

I just had a look at the CVS version. The first issue seems to have been fixed, but the issue with the case statements still exists. Reopening.