Skip to:
Content

BuddyPress.org

Opened 13 years ago

Closed 13 years ago

Last modified 13 years ago

#3779 closed defect (bug) (fixed)

Audit trailingslash links

Reported by: boonebgorges's profile boonebgorges Owned by: boonebgorges's profile boonebgorges
Milestone: 1.6 Priority: major
Severity: normal Version:
Component: Core Keywords: needs-patch
Cc:

Description

In r5412 (see #2681) trailing slashes were enforced before BP content resolves. This should be accompanied by a careful look at our own interior redirects and links, to ensure that we are minimizing necessary redirects by always pointing users to the trailingslashed version of a URL. (Note: for URLs that don't have an associated screen function - they are actions - it doesn't matter, since they never resolve, and canonicalization only happens at template loading time.)

I will probably work on this a component at a time. Patches welcome.

Change History (7)

#1 @boonebgorges
13 years ago

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

#2 @boonebgorges
13 years ago

(In [5414]) Ensures links are constructed with trailing slash in activity component. See #3779

#3 @boonebgorges
13 years ago

(In [5415]) Ensures links are constructed with trailing slash in blogs and core component. See #3779

#4 @boonebgorges
13 years ago

(In [5416]) Ensures links are constructed with trailing slash in forums and friends components. See #3779

#5 @boonebgorges
13 years ago

(In [5417]) Ensures links are constructed with trailing slash in groups, members, and messages components. See #3779

#6 @boonebgorges
13 years ago

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

(In [5418]) Ensures links are constructed with trailing slash in settings and xprofile components. Fixes #3779

#7 @djpaul
13 years ago

(In [5574]) Fix a bunch of URLs containing double forward slashes. See #3779

Note: See TracTickets for help on using tickets.