Make WordPress Core

Opened 17 months ago

Closed 17 months ago

Last modified 17 months ago

#22608 closed defect (bug) (fixed)

When editing a [gallery], menu_order sorting should be used

Reported by: nacin Owned by: koopersmith
Milestone: 3.5 Priority: normal
Severity: normal Version: 3.5
Component: Media Keywords: has-patch commit dev-reviewed
Focuses: Cc:


Initially reported by devinreams via #22607. When editing a gallery created with simply the [gallery] shortcode, menu_order sorting is not used, which means they end up in a different order than they show on the frontend.

The default "orderby" for the gallery shortcode is menu_order ID, both ascending.

Attachments (1)

22608.diff (5.5 KB) - added by koopersmith 17 months ago.

Download all attachments as: .zip

Change History (9)

comment:1 nacin17 months ago

  • Owner set to koopersmith
  • Status changed from new to assigned

comment:2 devinreams17 months ago

  • Cc devin@… added

koopersmith17 months ago

comment:3 koopersmith17 months ago

  • Keywords has-patch needs-testing added

Patch adds capability to sort queries by menu_order ID by adding the menuOrder property to Attachment models. Any gallery that does not specify any ids and does not specify a different orderby attribute will use menu order sorting by default. Also, the "uploaded to this post" filter is now sorted by menu order.

comment:4 devinreams17 months ago

I confirmed the following:

  1. Editing an existing gallery with a manual sort order shows those in the proper order
  2. Changing the sort order, aka updating the gallery from "Visual" view, still updates the text shortcode to include "ids" parameter
  3. The "Uploaded to this post" filter shows the proper order (the manual sort order)

comment:5 ryan17 months ago

Same here. Looks good.

comment:6 nacin17 months ago

  • Keywords commit dev-reviewed added; needs-testing removed

comment:7 nacin17 months ago

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

In 22882:

When editing a plain, legacy [gallery] shortcode, ensure we are sorting its contents using menu_order.

When attachments are sorted, we convert the shortcode to [gallery ids=""], as before.

props koopersmith.
fixes #22608.

comment:8 nacin17 months ago

#22758 was marked as a duplicate.

Note: See TracTickets for help on using tickets.