Fix for #53727 (is_subclass_of resolution with interfaces) View as Markdown

Hi Ralph,

I reassigned it myself, and will take a look on Monday or Tuesday.

Thanks. Dmitry.

Hey Dmitry,

is_a_impl() is an implementation that is shared by is_a() and
is_subclass_of() calls. There is a flag called only_subclass that allows
there to be switching behavior in is_a_impl().

The issue has been updated with a new patch that now takes objects into
account. It also has a comment in there that better explains what the
patch is doing.

I cannot assign it to you as I am not the original reporter, nor have
access, can you assign it to yourself?

Thanks,
Ralph

Hi Ralph,

I agree with expected behavior, but didn't understand the patch and
which side effects it may have as well as the original implementation of
is_a_impl(). It looks for me like one huge bug :)

Please reassign the bug to me, if you like me to take a deeper look.
I can do it only on next week.

Thanks. Dmitry.

Hi all,

Concerning: https://bugs.php.net/bug.php?id=53727

I had put together a quick patch against PHP_5_3 to fix, for what I and
others might consider, an issue with is_subclass_of() returning false in
this situation:

interface A {}
class B implements A {}
var_dump(is_subclass_of('B', 'A')); // false, should be true

It returns false in situations where a class does not have a parent. If
a parent currently exists, the above works fine. For example:

interface A {}
class B implements A {}
class C extends B {}
var_dump(is_subclass_of('B', 'A')); // true

Attached is a patch against PHP_5_3, I can create a patch against
PHP_5_4 if need be.

I've added a test file and all the current Zend/tests run as expected.

Thanks,
Ralph

Hi Ralph,

Your patch was absolutely correct.
I've committed it with small improvements.

Thanks. Dmitry.

Hey Dmitry,

is_a_impl() is an implementation that is shared by is_a() and
is_subclass_of() calls. There is a flag called only_subclass that allows
there to be switching behavior in is_a_impl().

The issue has been updated with a new patch that now takes objects into
account. It also has a comment in there that better explains what the
patch is doing.

I cannot assign it to you as I am not the original reporter, nor have
access, can you assign it to yourself?

Thanks,
Ralph

Hi Ralph,

I agree with expected behavior, but didn't understand the patch and
which side effects it may have as well as the original implementation of
is_a_impl(). It looks for me like one huge bug :)

Please reassign the bug to me, if you like me to take a deeper look.
I can do it only on next week.

Thanks. Dmitry.

Hi all,

Concerning: https://bugs.php.net/bug.php?id=53727

I had put together a quick patch against PHP_5_3 to fix, for what I and
others might consider, an issue with is_subclass_of() returning false in
this situation:

interface A {}
class B implements A {}
var_dump(is_subclass_of('B', 'A')); // false, should be true

It returns false in situations where a class does not have a parent. If
a parent currently exists, the above works fine. For example:

interface A {}
class B implements A {}
class C extends B {}
var_dump(is_subclass_of('B', 'A')); // true

Attached is a patch against PHP_5_3, I can create a patch against
PHP_5_4 if need be.

I've added a test file and all the current Zend/tests run as expected.

Thanks,
Ralph

On Wed, Jun 29, 2011 at 9:22 PM, David Zülke
david.zuelke@bitextender.com wrote:

On Wed, Jun 29, 2011 at 8:49 PM, Ralph Schindler ralph@smashlabs.com
wrote:

Correct.

I was hasty in that example, the first was copied & tested (and is
reflected
in the test, as is that variation of what I wrote up.)

Either way, test and patch work in 5_3.

Doesn't this functionality confuse matters?

If this patch is added, is there now no difference between instanceof
and is_subclass_of(). If this is the case my question is then why do
we have two methods to do the same thing?

I thought instanceof was for parent classes + interfaces.. and
is_subclass_of() was just for parent classes.

instanceof is a language construct and only operates on object instances.

I didn't spot you could pass a string into is_subclass_of() I thought
it was only objects. With that in mind it would be good to have
subclass_of() include interfaces.

are you kidding, right?

$class = 'stdClass';
$instance = new $class;
var_dump( $instance instanceof $class );

David

Not like class_implements(), since that requires processing the returned
array... Rather something that would function similarly to is_subclass_of()
(in that it returns a bool, not an array), but for interfaces.
I seem to run into a need for that on a fairly regular basis. (when I first
found class_implements() this is what I thought it did before reading the
docs)

-matt

Oops. My bad. Missed the array_key_exists() part in the example.

David