Get WordPress

Make WordPress Core

Opened 9 months ago

Closed 7 months ago

#59840 closed defect (bug) (fixed)

Twenty Twenty-Four: register_block_pattern_category()'s proprieties are missing the theme's textdomain

Reported by: acosmin's profile acosmin Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 6.4.3 Priority: normal
Severity: normal Version: 6.4
Component: Bundled Theme Keywords: good-first-bug has-patch commit fixed-major dev-reviewed
Focuses: Cc:

Description

​https://themes.trac.wordpress.org/browser/twentytwentyfour/1.0/functions.php#L199
​https://themes.trac.wordpress.org/browser/twentytwentyfour/1.0/functions.php#L200

Change History (15)

#1 @acosmin
9 months ago

  • Component changed from General to Bundled Theme

#2 @acosmin
9 months ago

The label can probably use the Core translation, but the description doesn't seem to be included.

#3 @afercia
9 months ago

  • Keywords needs-patch added
  • Milestone changed from Awaiting Review to 6.4.2
  • Severity changed from major to normal

#4 @afercia
9 months ago

  • Keywords good-first-bug added

Milestoning for next minor release, as this is a small, self-contained fix.

This ticket was mentioned in ​PR #5659 on ​WordPress/wordpress-develop by ​gavande1.
9 months ago #5

  • Keywords has-patch added; needs-patch removed

Trac ticket: #59840

#6 @huzaifaalmesbah
9 months ago

Thanks For adding Patch @gavande1. I saw your patch ​5659.diff Everything is ok.

#7 @ironprogrammer
9 months ago

  • Keywords commit added

I've also reviewed the patch, LGTM 👍🏻

Marking for commit consideration.

#8 @SergeyBiryukov
9 months ago

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

In 57127:

Twenty Twenty-Four: Add missing textdomain in pattern category description.

Follow-up to [56951].

Props gavande1, acosmin, afercia, huzaifaalmesbah, ironprogrammer.
Fixes #59840.

​@SergeyBiryukov commented on ​PR #5659:
9 months ago #9

Thanks for the PR! Merged in r57127.

#10 @SergeyBiryukov
9 months ago

  • Keywords fixed-major added
  • Resolution fixed deleted
  • Status changed from closed to reopened

Reopening for backporting to the 6.4 branch.

#11 @SergeyBiryukov
8 months ago

  • Milestone changed from 6.4.2 to 6.4.3

This ticket was mentioned in ​Slack in #core by jorbin. ​View the logs.
8 months ago

#13 @jorbin
8 months ago

  • Keywords dev-feedback added

#14 @jorbin
8 months ago

  • Keywords dev-reviewed added; dev-feedback removed

I've reviewed [57127] and it is good for backport.

#15 @peterwilsoncc
7 months ago

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

In 57307:

Twenty Twenty-Four: Add missing textdomain in pattern category description.

Follow-up to [56951].

Reviewed by jorbin.
Merges [57127] to the 6.4 branch.

Props gavande1, acosmin, afercia, huzaifaalmesbah, ironprogrammer, SergeyBiryukov.
Fixes #59840.

Note: See TracTickets for help on using tickets.