PHP IDE :: NuSphere Forums :: View topic

Posted: Sun Mar 09, 2008 12:12 pm

Hello.

This is just a little bug.

Code:

function change_status($old, $new)
{
echo "status change: {$old}->{$new}\n";
}

If I try to code complete $new variable error shows up: Code completion is not available due to the following error(s): Failed to identify type of "$new" local variable.

Posted: Sun Mar 09, 2008 12:25 pm

That is not a bug. How do you expect PHPEd to know the type of the $new variable?
Just add a phpdoc statement before the function to help:

Code:

/**
* @param DataTypeOld [$old] [description]
* @param DataTypeNew [$new] [description]
*/
function change_status($old, $new)
{
echo "status change: {$old}->{$new}\n";
}

Posted: Sun Mar 09, 2008 12:31 pm

Blizz,

How about now:

Code:

function change_status($old, $new)
{
echo "status change: {$old}->{$ne}\n";
}

Cursor is after "$ne" and if I press ctrl+space I have this error, but should code complete into $new (with $old works).

Posted: Sun Mar 09, 2008 6:11 pm

Well -> is an operator, so I assume PHPEd attempts to locate a member in the $old variable. Just add spaces between them and try again:

Code:

echo "status change: {$old} -> {$ne}\n";

Posted: Mon Mar 10, 2008 4:33 am

Blizz,

I know that I can add spaces, but I don't want spaces there and it is legal PHP code. It is a bug.

PS. Why somebody changed topic? Is this wrong place to submit bug reports?

Posted: Mon Mar 10, 2008 4:38 am

I'm trying to explain to you that it isn't a bug.
PHPEd interprets your code as $old->$ne| (| is the cursor).
So it tries to locate a member in the "object" $old.
You can have it complete the $new by adding spaces in between as I suggested (or by using => instead of -> for example) so that it isn't interpreted as an object anymore.
PS: My reference to "valid code" was meant for the {$old}->{$new} part, as this can be interpreted as a member of an object by php (I think it could even give you problems when you use it like that, since the php interpretor will probably do the same)

Posted: Mon Mar 10, 2008 4:46 am

Blizz,

I understand you, but I'm trying to explain you, that PHPED shouldn't interpret this like $old->$new because of {}. It's just small bug, I can live with that, it's not big deal. Just wanted to report a bug.

Regards

Posted: Mon Mar 10, 2008 4:58 am

Just tested it in PHP and it seems that you are right, it does output correctly. My apologies, I thought it would be interpreted differently.
It is a bug then.