#48252 closed defect (bug) (fixed)
Consistent deprecation of PHP 4 constructors
Reported by: | jrf | Owned by: | SergeyBiryukov |
---|---|---|---|
Milestone: | 5.4 | Priority: | normal |
Severity: | minor | Version: | 5.3 |
Component: | General | Keywords: | has-patch |
Focuses: | docs, coding-standards | Cc: |
Description
While testing a new sniff for WPCS, I ran into an number of inconsistencies in the use of the `_deprecated_constructor()` function.
In particular:
- Missing
@deprecated
tags in the function docblock of some PHP4 constructors. - Some PHP4 constructors in (external) dependencies not being marked as deprecated.
The attached patches would fix these inconsistencies.
Previous ticket: #31982
Related: #41121
Attachments (3)
Change History (13)
@
5 years ago
Twenty Eleven: mark PHP-4 constructor as deprecated As per [33086], no call to _deprecated_constructor()
is added as the theme can be used by WordPress versions from before the function was introduced.
#2
@
5 years ago
- Milestone changed from Awaiting Review to 5.4
- Owner set to SergeyBiryukov
- Status changed from new to reviewing
#7
follow-up:
↓ 8
@
5 years ago
Thanks @SergeyBiryukov for getting these committed!
I just noticed that in [46629], the version numbers still refer to 5.3.0
(when I created the patches), while trunk
is currently 5.4.0
. Should that still be fixed ?
Note: See
TracTickets for help on using
tickets.
WP_Widget(_Factory): PHP4 constructor / missing @deprecated tags