Opened 16 years ago
Closed 16 years ago
#14261 closed defect (bug) (fixed)
comment_text() needs to pass some IDs
| Reported by: | nacin | Owned by: | |
|---|---|---|---|
| Priority: | normal | Milestone: | 3.1 |
| Component: | Comments | Version: | |
| Severity: | normal | Keywords: | gsoc has-patch |
| Cc: | Focuses: |
Description
Bug: The comment_text() function doesn't pass the $comment_ID to get_comment_text().
Enhancement: Additionally, it'd be nice for the filters in both of these functions to be passed the comment ID.
Attachments (2)
Change History (7)
#3
@
16 years ago
I've incorporated this proposed param into #14856, let me know if this is unhelpful and I'll refactor the patches on that ticket.
#4
in reply to: ↑ 2
@
16 years ago
Replying to simonwheatley:
N.B. You can get the comment ID from the
$commentglobal.
We'd need to here. If $comment_ID = 0, we only set $comment via get_comment(), but don't update $comment_ID. So we'll have to change $comment_ID to $comment->ID.
I like the patch in #14856 but I don't really see a problem with getting this in on its own.
![(please configure the [header_logo] section in trac.ini)](/chrome/site/your_project_logo.png)
N.B. You can get the comment ID from the
$commentglobal.