[wp-trac] [WordPress Trac] #16980: Empty Values are converted to null by class-ixr.php

WordPress Trac wp-trac at lists.automattic.com
Tue Mar 29 11:09:48 UTC 2011


#16980: Empty Values are converted to null by class-ixr.php
------------------------------------+------------------------------
 Reporter:  nprasath002             |       Owner:
     Type:  defect (bug)            |      Status:  new
 Priority:  normal                  |   Milestone:  Awaiting Review
Component:  XML-RPC                 |     Version:  3.1
 Severity:  major                   |  Resolution:
 Keywords:  has-patch dev-feedback  |
------------------------------------+------------------------------
Changes (by solarissmoke):

 * keywords:   => has-patch dev-feedback


Comment:

 Patch fixes is for me. Logic is as follows:

 - According to the spec, the `<value>` tag can either contain a subtag
 which specifies the data type, or can contain a string. As it was, the
 parser would only handle non-empty strings so supplying `<value></value>`
 would cause it to just ignore the whole node.

 - My patch adds a tracking variable so we know when we reach the end of a
 `<value>` element whether it contained a sub-tag. If it didn't, we just
 use the contents of the `<value>` tag itself - empty or otherwise. Now the
 struct provided in comment:7 works.

 PS Devs - what is the policy on modifying external libraries? I've
 reported the bug with Incutio in the hope they might address it in a
 future version, no response yet.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/16980#comment:10>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list