[wp-trac] [WordPress Trac] #51278: Update return types to reflect the real return types. Remove mixed.

WordPress Trac noreply at wordpress.org
Wed May 26 08:02:47 UTC 2021


#51278: Update return types to reflect the real return types. Remove mixed.
------------------------------------+-------------------------------------
 Reporter:  ReneHermi               |       Owner:  (none)
     Type:  enhancement             |      Status:  reviewing
 Priority:  normal                  |   Milestone:  Future Release
Component:  General                 |     Version:
 Severity:  major                   |  Resolution:
 Keywords:  has-patch dev-feedback  |     Focuses:  docs, coding-standards
------------------------------------+-------------------------------------
Changes (by ReneHermi):

 * severity:  normal => major


Comment:

 It's an important update to the docs. The lack of getting someone to
 review this quickly proves the necessity of why this docBlock update is so
 important. The one who reviews this might think she will have to go
 through a line-by-line check of `get_option()` because it does have so
 many exceptions instead of clear return values. I'd say it's not necessary
 to do this and this PR can be added into the core without such a deep code
 inspection.

 Very talented people have already looked into this like @johnbillion

 Even if we overlooked something in this PR (which I do not believe) it
 will still improve the documentation tremendously with less effort. At
 worse it will not cover and document all eventualities and return values
 but the current docBlock only covers a fraction of them.

 So instead of refining and delaying this PR more and more until so much
 time has passed that it is not going to be anywhere, I highly recommend
 bringing it into core asap. Then we can still fine-tune it with smaller
 PR's which are easier to review if it should be necessary.

 I've changed severe status from normal to major and hope someone will get
 on to this soon. We are nearing 10 months for such a "simple" PR. I do not
 expect someone to look into this if it takes more time.

-- 
Ticket URL: <https://core.trac.wordpress.org/ticket/51278#comment:24>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform


More information about the wp-trac mailing list