#9614 closed (invalid)
contrib.comments requires Authentication middleware
| Reported by: | anonymous | Owned by: | nobody |
|---|---|---|---|
| Component: | Documentation | Version: | dev |
| Severity: | Keywords: | contrib.comments authentication middleware | |
| Cc: | Triage Stage: | Accepted | |
| Has patch: | yes | Needs documentation: | no |
| Needs tests: | no | Patch needs improvement: | no |
| Easy pickings: | no | UI/UX: | no |
Description
contrib.comments requires that the Authentication middleware is used. I suggest checking if request has attribute user could be nice for people that don't use the auth middleware and wishes to use the comments contrib
Attachments (2)
Change History (6)
by , 15 years ago
| Attachment: | comments.patch added |
|---|
comment:1 by , 15 years ago
| Version: | 1.0 → SVN |
|---|
I am tagging this a documentation request because this requires the django.core.context_processors.request to be enabled in the settings. Your patch is almost reimplementing the context processor.
comment:2 by , 15 years ago
| Component: | django.contrib.comments → Documentation |
|---|---|
| Triage Stage: | Unreviewed → Accepted |
comment:3 by , 15 years ago
| Resolution: | → invalid |
|---|---|
| Status: | new → closed |
On second thoughts, this ticket is invalid and I need to tell myself not to triage so late in the night.
Note:
See TracTickets
for help on using tickets.
patch checking if request has an attribute user