Make WordPress Core

Opened 6 years ago

Closed 6 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 6 years ago.
Changes URI to URL

Download all attachments as: .zip

Change History (8)

#1 @subrataemfluence
6 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 6 years ago by subrataemfluence (previous) (diff)

@danieltj
6 years ago

Changes URI to URL

#2 @danieltj
6 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.

Version 0, edited 6 years ago by danieltj (next)

#3 @rinkuyadav999
6 years ago

  • Keywords dev-feedback added

#4 @SergeyBiryukov
6 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
6 years ago

  • Milestone changed from Awaiting Review to 4.9

#6 @SergeyBiryukov
6 years ago

Introduced in [41182].

#7 @SergeyBiryukov
6 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.