Bug->Doc #75449 [Asn]: Ignored return type for abstract methods, defined in traits

From: Date: Wed, 28 Nov 2018 12:20:48 +0000
Subject: Bug->Doc #75449 [Asn]: Ignored return type for abstract methods, defined in traits
References: 1  Groups: php.doc.bugs 
Request: Send a blank email to doc-bugs+get-16178@lists.php.net to get a copy of this message
Edit report at https://bugs.php.net/bug.php?id=75449&edit=1

 ID:                 75449
 Updated by:         cmb@php.net
 Reported by:        alex dot rill at yandex dot ru
 Summary:            Ignored return type for abstract methods, defined in
                     traits
 Status:             Assigned
-Type:               Bug
+Type:               Documentation Problem
 Package:            Class/Object related
 Operating System:   Windows
 PHP Version:        7.1.11
-Assigned To:        pmmaga
+Assigned To:        cmb
 Block user comment: N
 Private report:     N

 New Comment:

There has been some discussion about this on the internals@
mailing list[1], and it seem to me that Levi[2] is right.
Therefore I'm changing this to a documentation problem.

[1] <https://externals.io/message/103032>
[2] <https://externals.io/message/103032#103039>


Previous Comments:
------------------------------------------------------------------------
[2018-04-29 21:25:02] eric at ericstern dot com

I encountered this as well (discovered this report when filing my own bug, so I'm commenting
here rather than filing a dupe). Note that this is in no way specific to static methods.

A similar reproduce case: https://3v4l.org/AVSXN

And more importantly, a test case where the issue that should have been detected at compile-time
instead was left until runtime: https://3v4l.org/02d4a This is
the main use-case of abstract trait methods.

------------------------------------------------------------------------
[2017-11-22 23:28:08] pmmaga@php.net

Actually, after re-reading the page on traits, the usage of abstract methods for this purpose is
mentioned:

> Traits support the use of abstract methods in order to impose requirements upon the exhibiting
> class.

Given that, I am re-opening this bug.

------------------------------------------------------------------------
[2017-11-21 23:17:46] pmmaga@php.net

Thank you for taking the time to write to us, but this is not
a bug. Please double-check the documentation available at
http://www.php.net/manual/ and the instructions on how to
report
a bug at http://bugs.php.net/how-to-report.php

This is expected behavior. In your first example, the abstract function from the trait is overriden
by the function declared on your class. No inheritance rules are applied at all. It's not only
about the return value type.

In your second example, you are importing the abstract function to an abstract class. In that case,
the inheritance rules are applied when you are extending from that abstract class.

------------------------------------------------------------------------
[2017-10-31 02:12:55] saintatgod at gmail dot com

<?php
trait T {
        abstract static function values(): array;
}

//class B {
//      use T;
//      static function values() {
//              return 'Worked!'; // string, not array
//      }
//}

//echo B::valuesi(); // print: Worked!

abstract class B {
        use T;
        self::values(); // if inheritance trait, must implement abstract function in trait
}

class C extends B {
        static function values() {
                return 'Don\'t worked';
        }
}

echo C::values();

------------------------------------------------------------------------
[2017-10-31 01:43:27] saintatgod at gmail dot com

where is Class D ?

------------------------------------------------------------------------


The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at

    https://bugs.php.net/bug.php?id=75449


--
Edit this bug report at https://bugs.php.net/bug.php?id=75449&edit=1


Thread (2 messages)

« previous php.doc.bugs (#16178) next »