Making WordPress.org

Opened 10 years ago

Closed 8 years ago

Last modified 5 years ago

#577 closed defect (bug) (fixed)

credits API should return the correct version

Reported by: markoheijnen's profile markoheijnen Owned by: coffee2code's profile coffee2code
Milestone: Priority: normal
Component: API Keywords: has-patch commit
Cc:

Description

You could request data for a version that the API doesn't support. It then fallsback to WP_CORE_STABLE_BRANCH except the requested version is still displayed. We should then return the actual version so you can that validate your request.

Also any way we could provide data for versions older then 3.2?

Attachments (1)

577.diff (594 bytes) - added by markoheijnen 10 years ago.

Download all attachments as: .zip

Change History (8)

@markoheijnen
10 years ago

#1 @markoheijnen
9 years ago

Any feedback on this?

#2 in reply to: ↑ description @ramiy
9 years ago

Replying to markoheijnen:

Also any way we could provide data for versions older then 3.2?

Just installed wp3.2, wp31 and wp30 to create the WP_31_Credits and WP_30_Credits classes for the API.

But is seems like credits.php was added only in 3.2, this is why the API supports only 3.2+

If you have this data for older versions, I can write the code.

I need:

  • a list of users:
    • Project Leaders
    • Extended Core Team
    • Recent Rockstars
    • Core Contributors
  • a list of external libraries
Version 0, edited 9 years ago by ramiy (next)

#3 @ramiy
9 years ago

The full list of "Core Contributors" can be found in blog release post:

3.2: https://wordpress.org/news/2011/07/gershwin/
3.1: https://wordpress.org/news/2011/02/threeone/
3.0: https://wordpress.org/news/2010/06/thelonious/

We can retrieve the "external libraries" using the change log of the next version.

#4 @SergeyBiryukov
8 years ago

  • Keywords commit added

577.diff looks good to me.

#5 @coffee2code
8 years ago

  • Owner set to coffee2code
  • Resolution set to fixed
  • Status changed from new to closed

In 5133:

Credits API: Return 'version' value that matches the data being returned.

For invalid version requests (e.g. something pre-3.2 or post-WP_CORE_STABLE_BRANCH), the Credits API returns data for WP_CORE_STABLE_BRANCH. In such cases the 'version' value in the response was the invalid version being requested.

The returned 'version' should instead reflect the version that matches the data. API consumers can compare their requested version to the response version to see if the fallback (current stable) is being returned.

Props markoheijnen.
Fixes #577.

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


5 years ago

#7 in reply to: ↑ description @SergeyBiryukov
5 years ago

Replying to markoheijnen:

Also any way we could provide data for versions older then 3.2?

Follow-up: #4697

Note: See TracTickets for help on using tickets.