Context Navigation

← Previous Ticket
Next Ticket →

#15517 closed defect (bug) (fixed)

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

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

Description

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

http://msdn.microsoft.com/en-us/library/aa905674.aspx

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 16 years ago.

Download all attachments as: .zip

Change History (10)

#1 @westi
16 years ago

Milestone Awaiting Review → 3.1
Owner set to westi
Status new → accepted

#2 @westi
16 years ago

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

#3 @westi
16 years ago

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

#4 @josephscott
16 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
16 years ago

Cc josephscott added
Severity critical → minor

#6 @barry
16 years ago

blog_id needs to be a string too.

@barry
16 years ago

Attachment 15517.diff added

#7 @westi
16 years ago

Severity minor → critical

#8 @westi
16 years ago

Resolution → fixed
Status accepted → 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
14 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 14 years ago by SergeyBiryukov (previous) (diff)

Note: See TracTickets for help on using tickets.

Download in other formats:

Make WordPress Core

Context Navigation

#15517 closed defect (bug) (fixed)

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

Description

Attachments (1)

Change History (10)

#1 @westi 16 years ago

#2 @westi 16 years ago

#3 @westi 16 years ago

#4 @josephscott 16 years ago

#5 @josephscott 16 years ago

#6 @barry 16 years ago

@barry 16 years ago

#7 @westi 16 years ago

#8 @westi 16 years ago

#9 @kevbo 14 years ago