[wp-trac] [WordPress Trac] #27881: Compatibility with Hack
WordPress Trac
noreply at wordpress.org
Mon May 19 15:12:26 UTC 2014
#27881: Compatibility with Hack
----------------------------+--------------------
Reporter: wonderboymusic | Owner:
Type: enhancement | Status: new
Priority: normal | Milestone: 4.0
Component: General | Version: trunk
Severity: normal | Resolution:
Keywords: needs-patch | Focuses:
----------------------------+--------------------
Comment (by wonderboymusic):
Here are some issues to look at, might not all be valid, but is the
remaining actionable output from `hackificator`:
{{{
wp-admin/includes/class-wp-filesystem-base.php:357:19,26: The method
getchmod is undefined in an object of type WP_Filesystem_Base ... wp-
admin/includes/class-wp-filesystem-base.php:14:7,24: Check this out
wp-admin/includes/class-wp-filesystem-direct.php:24:18,28: You are
extending a class that needs to be initialized
Make sure you call parent::__construct.
wp-admin/includes/class-wp-filesystem-ftpsockets.php:22:18,28: You are
extending a class that needs to be initialized
Make sure you call parent::__construct.
wp-admin/includes/class-wp-filesystem-ssh2.php:44:18,28: You are extending
a class that needs to be initialized
Make sure you call parent::__construct.
wp-admin/includes/class-wp-upgrader-skins.php:28:31,31: Don't use
references!
wp-admin/includes/class-wp-upgrader-skins.php:702:11,17: The member
options is undefined in an object of type Automatic_Upgrader_Skin
wp-admin/includes/class-wp-upgrader-skins.php:697:7,29: Check this out
wp-admin/includes/class-wp-users-list-table.php:388:5,8: Expected modifier
wp-admin/includes/deprecated.php:341:2,4: Expected modifier
wp-admin/includes/list-table.php:90:18,38: This is a dangerous method
name, if you want to define a constructor, use __construct
wp-admin/includes/revision.php:176:35,47: The variable $restore_link is
defined
wp-admin/includes/revision.php:135:4,16: But in a different scope)
wp-admin/includes/template.php:38:29,29: Don't use references!
wp-admin/options.php:235:12,12: Expected attribute value
wp-admin/plugin-editor.php:235:224,224: Expected ul
wp-admin/update-core.php:613:20,20: Expected attribute value
wp-includes/bookmark-template.php:246:21,25: The variable $cats is defined
wp-includes/bookmark-template.php:231:3,7: But in a different scope
wp-includes/capabilities.php:307:2,4: Expected modifier
wp-includes/category.php:344:28,28: Don't use references!
wp-includes/class-oembed.php:181:66,71: Undefined variable: $links
wp-includes/class-wp-customize-setting.php:392:45,45: Don't use
references!
wp-includes/class-wp-image-editor.php:277:22,34: The method get_extension
is undefined in an object of type WP_Image_Editor
wp-includes/class-wp-image-editor.php:14:16,30: Check this out
wp-includes/class-wp-walker.php:100:29,29: Don't use references!
wp-includes/class-wp-walker.php:333:12,21: The variable $total_top is
defined
wp-includes/class-wp-walker.php:327:4,13: But in a different scope
wp-includes/class-wp-xmlrpc-server.php:265:26,26: Don't use references!
wp-includes/class.wp-dependencies.php:122:10,42: Too many arguments
wp-includes/class.wp-scripts.php:172:15,20: The variable $after is defined
wp-includes/class.wp-scripts.php:159:4,9: But in a different scope
wp-includes/cron.php:327:10,10: Yeah...we're not going to support
continue/break N. It makes static analysis tricky and it's not really
essential)
wp-includes/date.php:108:2,9: Expected modifier
wp-includes/default-widgets.php:16:2,9: Expected modifier
wp-includes/class.wp-styles.php:87:10,146: Too many arguments
wp-includes/plugin.php:156:10,22: Definition is here
wp-includes/query.php:3425:2,9: Expected modifier
wp-includes/revision.php:289:31,31: Don't use references!
wp-includes/rewrite.php:428:2,4: Expected modifier
wp-includes/script-loader.php:50:30,30: Don't use references!
wp-includes/user.php:452:2,4: Expected modifier
wp/wp-includes/widgets.php:26:2,4: Expected modifier
wp-includes/wp-db.php:64:2,4: Expected modifier
wp-includes/wp-diff.php:34:2,4: Expected modifier
}}}
--
Ticket URL: <https://core.trac.wordpress.org/ticket/27881#comment:54>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list