WordPress.org
Get WordPress

Make WordPress Core

Opened 12 years ago

Closed 12 years ago

Last modified 12 years ago

#29751 closed defect (bug) (fixed)

Merge results strings

Reported by: pavelevap Owned by: SergeyBiryukov
Priority: normal Milestone: 4.1
Component: I18N Version: 4.0
Severity: normal Keywords:
Cc: Focuses:

Description

We have 2 almost same strings for search results in menu metaboxes and link window in visual editor. It should be merged, I guess. See attached patch...

Attachments (1)

merge_result_strings.patch​ (1.3 KB ) - added by pavelevap 12 years ago.

Download all attachments as: .zip

Change History (6)

@pavelevap
12 years ago

#1 @SergeyBiryukov
12 years ago

  • Milestone Awaiting Review → 4.1

I think we can also remove the context (added in [14285]) to merge the string with the one used in wp-admin/includes/nav-menu.php​.

#2 @SergeyBiryukov
12 years ago

  • Owner set to SergeyBiryukov
  • Resolution → fixed
  • Status new → closed

In 29768:

Merge two similar strings.

props pavelevap.
fixes #29751.

#3 @TobiasBg
12 years ago

  • Resolution fixed
  • Status closed → reopened

_x( 'No results found.' ) in [29768] will break due to a missing parameter.
It should be __( 'No results found.' ).

#4 @SergeyBiryukov
12 years ago

  • Resolution → fixed
  • Status reopened → closed

In 29769:

Use correct i18n function after [29768].

props TobiasBg.
fixes #29751.

#5 @pavelevap
12 years ago

Thank you, Sergey. So there were two similar strings removed in the end, nice...

Note: See TracTickets for help on using tickets.