Make WordPress Core

Opened 14 years ago

Closed 14 years ago

Last modified 12 years ago

#15517 closed defect (bug) (fixed)

'postid' field in metaWeblog.getRecentPosts should be a string

Reported by: mrroundhill's profile mrroundhill Owned by: westi's profile westi
Milestone: 3.1 Priority: normal
Severity: critical Version: 3.1
Component: XML-RPC Keywords: xml-rpc
Focuses: Cc:


In 3.1, the xml-rpc metaWeblog.getRecentPosts call is returning the 'postid' field as an integer when it should be a string:

Clients such as the WordPress for iPhone app can't handle the different type and force closes when refreshing the posts tab.

Attachments (1)

15517.diff (475 bytes) - added by barry 14 years ago.

Download all attachments as: .zip

Change History (10)

#1 @westi
14 years ago

  • Milestone changed from Awaiting Review to 3.1
  • Owner set to westi
  • Status changed from new to accepted

#2 @westi
14 years ago

(In [16508]) Ensure that we return post IDs as strings to honour API defintions. See #15517

#3 @westi
14 years ago

We need to check the other API definitions to ensure we return the correct datatypes.

#4 @josephscott
14 years ago

I'd recommend being very cautious about this types of changes, for things that have been one way for a very long time. This might be the right thing to do, but that doesn't rule out the potential for causing problems with a large number of existing clients.

#5 @josephscott
14 years ago

  • Cc josephscott added
  • Severity changed from critical to minor

#6 @barry
14 years ago

blog_id needs to be a string too.

14 years ago

#7 @westi
14 years ago

  • Severity changed from minor to critical

#8 @westi
14 years ago

  • Resolution set to fixed
  • Status changed from accepted to closed

We haven't seen any more reports of broken apis since the last change which was committed in [16572]

Closing as fixed - open a new ticket for any newly discovered issues.

#9 @kevbo
12 years ago

I think a mistake was made here. The Blogger API actually says that the post ID should return a long, not a string. Also, getPost should also have been changed but is not. See ticket #20548.

Last edited 12 years ago by SergeyBiryukov (previous) (diff)
Note: See TracTickets for help on using tickets.