WordPress.org
Get WordPress

Make WordPress Core

Opened 14 years ago

Closed 14 years ago

#20543 closed defect (bug) (fixed)

Removing a custom field via XML-RPC results in a PHP Notice

Reported by: scott.gonzalez Owned by: nacin
Priority: normal Milestone: 3.4
Component: XML-RPC Version:
Severity: normal Keywords:
Cc: Focuses:

Description

In order to remove a custom field, you pass the id without a key or value, but the code is currently checking for a value before determining if the field should be deleted.

PHP Notice: Undefined index: value in .../wp-includes/class-wp-xmlrpc-server.php on line 279

testing against HEAD.

Attachments (1)

20543.patch​ (905 bytes ) - added by scott.gonzalez 14 years ago.

Download all attachments as: .zip

Change History (4)

#1 @nacin
14 years ago

  • Milestone Awaiting Review → 3.4

@scott.gonzalez
14 years ago

#2 @nacin
14 years ago

​http://unit-tests.trac.wordpress.org/changeset/697

php wp-test.php -d -t TestXMLRPCServer_wp_editPost (-d forcing WP_DEBUG on) -- before patch, error due to this notice.

#3 @nacin
14 years ago

  • Owner set to nacin
  • Resolution → fixed
  • Status new → closed

In [20583]:

Do not check for a meta value in XML-RPC set_custom_fields unless we know we are editing a key. Prevents a notice when a custom field is being deleted. props scott.gonzalez. fixes #20543.

Note: See TracTickets for help on using tickets.