Opened 13 years ago
Closed 12 years ago
#23406 closed defect (bug) (fixed)
Some notices messages which can maybe fixed
| Reported by: |
|
Owned by: | |
|---|---|---|---|
| Milestone: | 3.7 | Priority: | normal |
| Severity: | trivial | Version: | 3.5.1 |
| Component: | Inline Docs | Keywords: | has-patch dev-feedback |
| Focuses: | Cc: |
Description
Hi,
I use wordpress multisite on my own cms and I notice there are many notice messages when I load WP (include($wppath."includes/wp.inc.php") :
1) Declaration of Walker_Category::start_el() should be compatible with that of Walker::start_el()) in /wp-includes/category-template.php line 902
2) Declaration of WP_Scripts::set_group() should be compatible with that of WP_Dependencies::set_group()) in wp-includes/class.wp-scripts.php
3) Declaration of Walker_Nav_Menu::start_el() should be compatible with that of Walker::start_el()) in wp-includes/nav-menu-template.php
4) is_a(): Deprecated. Please use the instanceof operator) dans le fichier wp-includes/capabilities.php
Hope it helps
Regards,
Alexandre
Attachments (5)
Change History (16)
#2
@
12 years ago
- Keywords needs-docs added
- Milestone changed from Awaiting Review to 3.7
These have since been rehabilitated, but the doc blocks on the methods are out of sync
#3
@
12 years ago
- Component changed from Warnings/Notices to Inline Docs
- Keywords has-patch dev-feedback added; needs-docs removed
@wonderboymusic: Did I miss any in 23406.2.diff?
#9
@
12 years ago
- Resolution fixed deleted
- Status changed from closed to reopened
As far as I understand, 5.2.4 is still the "minimum" version for WordPress. In 5.2.4, is_a is deprecated. Perhaps either is_a shouldn't be used, or the "minimum" version should be bumped to 5.3, where it was reintroduced.
Please update to PHP 5.3, where is_a() is no longer deprecated.
The others appear to be valid issues:
wp-includes/class.wp-dependencies.php 223: function set_group( $handle, $recursion, $group ) { wp-includes/class.wp-scripts.php 161: function set_group( $handle, $recursion, $group = false ) {wp-admin/includes/nav-menu.php 37: function start_el(&$output, $item, $depth, $args) { 238: function start_el(&$output, $item, $depth, $args) { wp-includes/category-template.php 823: function start_el( &$output, $category, $depth = 0, $args = array(), $id = 0 ) { wp-includes/nav-menu-template.php 67: function start_el( &$output, $item, $depth = 0, $args = array(), $id = 0 ) {