WordPress.org

Make WordPress Core

Opened 5 months ago

Closed 4 weeks ago

Last modified 4 weeks ago

#53303 closed enhancement (fixed)

Widget type REST API endpoint should be sorted by id

Reported by: spacedmonkey Owned by: TimothyBlynJacobs
Milestone: 5.9 Priority: normal
Severity: normal Version: 5.8
Component: REST API Keywords: has-patch has-unit-tests early
Focuses: rest-api Cc:

Description

The results of Widget type REST API endpoint should be sorted by widget type id to make the output more readable.

Change History (17)

This ticket was mentioned in ​PR #1321 on ​WordPress/wordpress-develop by ​spacedmonkey.
5 months ago

  • Keywords has-patch added; needs-patch removed

Trac ticket: https://core.trac.wordpress.org/ticket/53303

#2 @spacedmonkey
5 months ago

@TimothyBlynJacobs Any chance of a review?

#3 @prbot
5 months ago

​TimothyBJacobs commented on ​PR #1321:

lgtm, can you add a test?

#4 @prbot
5 months ago

​spacedmonkey commented on ​PR #1321:

lgtm, can you add a test?

You know I looked into a test, but couldn't think of a good one. Any thoughts?

#5 @hellofromTonya
5 months ago

  • Keywords needs-unit-tests added
  • Milestone changed from 5.8 to 5.9

Today is 5.8 Beta 1. As work continues, punting to 5.9.

#6 @prbot
5 months ago

​spacedmonkey commented on ​PR #1321:

@TimothyBJacobs What tests should I add?

#7 @spacedmonkey
5 months ago

@hellofromTonya I personally think this not being sorted correctly is a bug. It is a one liner that doesn't really need test IMO.

#8 @prbot
5 months ago

​TimothyBJacobs commented on ​PR #1321:

Sorry for not getting back to you earlier. I'd probably just ksort the result in the tests and assert that it is the same as the returned, non-sorted version.

#9 @prbot
4 months ago

​spacedmonkey commented on ​PR #1321:

@TimothyBJacobs I have added a unit test.

#10 @spacedmonkey
4 months ago

  • Keywords has-unit-tests added; needs-unit-tests removed

#11 @spacedmonkey
4 months ago

  • Keywords early added

#12 @spacedmonkey
3 months ago

@TimothyBlynJacobs Any chance of review / merge.

This ticket was mentioned in ​Slack in #core-restapi by spacedmonkey. ​View the logs.
3 months ago

This ticket was mentioned in ​Slack in #core by audrasjb. ​View the logs.
5 weeks ago

#15 @audrasjb
5 weeks ago

This ticket was reviewed during today's #core bug scrub.

FWIW, just mentioning that a comment was added in the PR @spacedmonkey :)

This test feels very fragile, it will require updating whenever we introduce or remove a new widget. Can we instead assert that the response is sorted?

#16 @TimothyBlynJacobs
4 weeks ago

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

In 51882:

REST API: Sort widget types by their id.

Fixes #53303.
Props spacedmonkey.

Note: See TracTickets for help on using tickets.