Opened 12 years ago
Closed 12 years ago
#26679 closed defect (bug) (fixed)
metaWeblog.getRecentPosts missing `sticky`
| Reported by: |
|
Owned by: |
|
|---|---|---|---|
| Milestone: | 3.9 | Priority: | normal |
| Severity: | minor | Version: | |
| Component: | XML-RPC | Keywords: | has-patch |
| Focuses: | Cc: |
Description
http://codex.wordpress.org/XML-RPC_MetaWeblog_API#Return_Values_2 states "struct: see #metaWeblog.getPost for fields", but sticky is not returned, although valuable.
See: https://github.com/wordpress-mobile/WordPress-Android/issues/349, for example
Attachments (2)
Change History (11)
#2
@
12 years ago
As you seem to have discovered on your own in that GitHub issue, a better approach would be to just use the wp.getPosts method instead. It was added in 3.4, so compatibility should be fine for the mobile apps; certainly better than waiting for 3.9 or later. You should consider the Movable Type API deprecated at this point.
#3
@
12 years ago
- Severity changed from normal to minor
wp.getPosts is indeed a superior API and hopefully the lead developers will want to move WordPress for Android over to it instead. But overall, this is a trivial miss that wasn't covered in the Codex until now, and it's trivial to fix, too.
But wherever this ticket ends up - I'm fine with it.
#5
@
12 years ago
I'm fine that this patch gets committed but it's not a fix for the mobile apps. Since for them it's better to switch to wp.getPosts to have better support for it.
#6
@
12 years ago
I can't but agree. I'm going to push for wp.getPosts there for sure, since it's far more superior. This ticket should be unrelated to the mobile app and not dependent on it; it's a mere trivial bugfix, where the Codex promised sticky but it was not delivered.
#8
@
12 years ago
- Summary changed from mt.getRecentPosts missing `sticky` to metaWeblog.getRecentPosts missing `sticky`
Updated the ticket title since this is actually a MetaWeblog method, not a MoveableType one. Ideally this field would actually be called wp_sticky to make it clear that it's a WordPress-specific extension, but it appears we already called it sticky in metaWeblog.getPost and metaWeblog.editPost.
I'd prefer not to encourage continued use of these old methods by adding new features to them, but I'm okay with doing this for the sake of consistency.
Should be $entry, not $post.