Opened 11 years ago
Closed 11 years ago
#2625 closed defect (bug) (fixed)
New reply legend shouldn't say Create New Topic in "" if forum title is blank
Reported by: |
|
Owned by: |
|
---|---|---|---|
Milestone: | 2.6 | Priority: | normal |
Severity: | trivial | Version: | |
Component: | General | Keywords: | |
Cc: |
Description
I'm using bbPress with Buddypress, and making group forums that have the same names as their groups. On a group forum topic listing, the name of the group is at the top of the page, and then the name of the forum (which is the same thing) is below it. This was redundant, so I filtered the forum title out by filtering bbp_get_forum_title()
and having it return nothing. Admittedly, not the cleanest hack. But this makes the new reply legend say Create New Topic in ""
, i.e., with empty quotation marks. This shouldn't happen, so I added a check to the line that creates this output, to make sure that the forum title isn't empty before trying to print it.
This is my first time submitting a patch on Trac (I'm more of a github guy), so let me know if I did this right!
Looks good to me. Let's get this into 2.6. Thanks for the patch!