Making WordPress.org

Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#2585 closed defect (bug) (fixed)

API is returning a different author name than the plugin directory

Reported by: ocean90's profile ocean90 Owned by: dd32's profile dd32
Milestone: Plugin Directory v3.0 Priority: high
Component: Plugin Directory Keywords: has-patch
Cc:

Description

The API seems to return a different author name as the previous API and the front end of the new plugin directory.

https://wordpress.org/plugins-wp/public-post-preview/

Attachments (5)

2585.png (1.1 MB) - added by ocean90 8 years ago.
plugins.png (760.5 KB) - added by hallsofmontezuma 8 years ago.
plugins.php
plugin api.png (415.1 KB) - added by hallsofmontezuma 8 years ago.
plugin api
plugin install.png (246.3 KB) - added by hallsofmontezuma 8 years ago.
plugin install
2585.diff (1.3 KB) - added by obenland 8 years ago.

Change History (16)

@ocean90
8 years ago

#1 @ocean90
8 years ago

  • Owner set to dd32
  • Status changed from new to reviewing

Already chatted with @dd32 about this. He'll take a look at this.

#2 @dd32
8 years ago

  • Status changed from reviewing to accepted

#3 @hallsofmontezuma
8 years ago

On the "View details" link in plugins.php, in addition to the author being wrong (it's the original submitter of the plugin rather than the current author) the link also goes to the wrong website.

It's also incorrect on plugin-install.php, but presumably that info comes from the same place.

Interestingly, at https://api.wordpress.org/plugins/info/1.0/simplemap/, the author URL is correct, and the author name, but not the author profile.

@hallsofmontezuma
8 years ago

plugins.php

@hallsofmontezuma
8 years ago

plugin api

@hallsofmontezuma
8 years ago

plugin install

#4 @Otto42
8 years ago

  • Priority changed from normal to high

Related: #2630

Version 0, edited 8 years ago by Otto42 (next)

#5 @Otto42
8 years ago

#2640 was marked as a duplicate.

@obenland
8 years ago

#6 @obenland
8 years ago

  • Keywords has-patch added; needs-patch removed

@dd32 Is this the right place to fix it?

#7 @dd32
8 years ago

@obenland Looks good to me - I haven't tested it though, verify that it looks good in https://wordpress.org/plugins/wp-json/plugins/v1/plugin/$slug and commit - the caches in the API are ~24hrs though, but any commit or change to an affected plugin will cause that to be cleared.

#8 @Otto42
8 years ago

Might want to add esc_url on it if core doesn't, but other than that, patch looks good.

#9 @dd32
8 years ago

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

In 5204:

Plugin Directory: Use the Author and Author URI plugin headers in the API response instead of the original plugin submitters details.
This doesn't change author_profile which still references the original uploaders profile, as it always has. This field doesn't appear to be used by WordPress though.

Props obenland for initial patch.
Fixes #2585.

This ticket was mentioned in Slack in #meta by sergey. View the logs.


8 years ago

Note: See TracTickets for help on using tickets.