Make WordPress Core

Opened 7 years ago

Closed 7 years ago

#42147 closed defect (bug) (fixed)

Change mention of URI to URL in /wp-admin/options-permalink.php

Reported by: tobifjellner's profile tobifjellner Owned by: sergeybiryukov's profile SergeyBiryukov
Milestone: 4.9 Priority: normal
Severity: normal Version: 4.9
Component: Permalinks Keywords: has-patch needs-testing dev-feedback
Focuses: Cc:

Description

/wp-admin/options-permalink.php:L228

Mentions the term URI. Although this abbreviation is (also) correct, I suggest we use URL here, in order not to introduce a new term that's not used anywhere else in WordPress core.

Snippet of current code:

'category' => __( '%s (Category slug. Nested sub-categories appear as nested directories in the URI.)' ),

Attachments (1)

42147.diff (746 bytes) - added by danieltj 7 years ago.
Changes URI to URL

Download all attachments as: .zip

Change History (8)

#1 @subrataemfluence
7 years ago

Hence URI is a super-set of URL and they are different in nature, could there be a specific reason of using URI instead of URL?

However, this is true that the abbreviation URI is not used anywhere else in WordPress core.

Last edited 7 years ago by subrataemfluence (previous) (diff)

@danieltj
7 years ago

Changes URI to URL

#2 @danieltj
7 years ago

  • Keywords has-patch needs-testing added

Smallest patch I've ever submitted.

In 42147.diff

To remain consistent throughout, changed URI to URL.

Last edited 7 years ago by danieltj (previous) (diff)

#3 @rinkuyadav999
7 years ago

  • Keywords dev-feedback added

#4 @SergeyBiryukov
7 years ago

URI is used in quite a few inline docs and comments, but it looks like this is indeed the only instance in a translatable string. Let's change it to URL for consistency with other strings.

#5 @SergeyBiryukov
7 years ago

  • Milestone changed from Awaiting Review to 4.9

#6 @SergeyBiryukov
7 years ago

Introduced in [41182].

#7 @SergeyBiryukov
7 years ago

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

In 42023:

Permalinks: Change mention of URI to URL in the description of %category% tag.

Props danieltj, tobifjellner.
Fixes #42147.

Note: See TracTickets for help on using tickets.