Skip to:

Opened 14 years ago

Closed 13 years ago

#2349 closed defect (bug) (fixed)

Activity Permalinks (#) on activity page are broken for long post titles

Reported by: 3sixty's profile 3sixty Owned by:
Milestone: 1.5 Priority: normal
Severity: Version:
Component: Activity Keywords: has-patch


This URL looks ridiculous but illustrates 2 points:

  1. For the new activity permalinks on the activity page of, there is a truncation problem that is causing the URL to break and point to the first record in the _bb table
  2. Larger issue: Malformed/nonexistent slugs SHOULD trigger a 404 error, but instead, they point to the first record in the _bb table

See the ridiculous URL for more details...

Attachments (1)

2349-1.patch (526 bytes) - added by boonebgorges 14 years ago.

Download all attachments as: .zip

Change History (10)

#1 @3sixty
14 years ago

  • Milestone changed from 1.2.4 to 1.3

#2 @boonebgorges
14 years ago

The problem is this. By default, bbPress limits slug length to 255 characters. But the primary_link column in the BP activity table is set to varchar(150). So in most cases (by the time you subtract the length of the domain, etc) you might be left with a max of 100 of those characters being stored in primary_link.

I think the answer is to bump the maximum size of primary_link. See 2349-1.patch. I increased it to 300, but even that might not be enough for long domain names, etc. Anyone have a sense of the best way to handle it?

#3 @boonebgorges
14 years ago

  • Component changed from Core to Activity

#4 @cnorris23
14 years ago

  • Keywords has-patch added

#5 @DJPaul
14 years ago

Suggest we punt and review after bbPress plugin release/integration.

#6 @boonebgorges
13 years ago

  • Milestone changed from 1.3 to 1.4


#7 @dwenaus
13 years ago

my suggestion: don't fix the underlying problem, but for versino 1.3 just put on a bigger bandage that requires no testing: Set the varchar be 255 in length. Later in version 1.4 address the issue properly by reducing the slug length of bbpress, or extending beyond 255.

FYI, this is affecting the website as well. It's my second time in two days I've hit this. :)

#8 @DJPaul
13 years ago

  • Milestone changed from 1.4 to 1.3

#9 @djpaul
13 years ago

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

(In [4124]) Correctly handle invalid forum permalinks. Also fixes a couple of PHP notices. Fixes #2349.

Note: See TracTickets for help on using tickets.