Opened 9 years ago

Closed 9 years ago

#5107 closed (fixed)

Replace Context with RequestContext in django.contrib.syndication.feeds

Reported by: anonymous Owned by: Jacob
Component: Uncategorized Version:
Severity: Keywords: syndication
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: no
Easy pickings: UI/UX:

Description

Replace Context with RequestContext.

Attachments (1)

j1.diff (584 bytes) - added by peter@… 9 years ago.
adjust feed to use RequestContext for Title and Description Template Render

Download all attachments as: .zip

Change History (7)

Changed 9 years ago by peter@…

Attachment: j1.diff added

adjust feed to use RequestContext for Title and Description Template Render

comment:1 Changed 9 years ago by Simon G. <dev@…>

Resolution: invalid
Status: newclosed

Why? what benefits does this have?

comment:2 Changed 9 years ago by Simon G. <dev@…>

Summary: Haut Simple! django.contrib.syndication.feedsReplace Context with RequestContext in django.contrib.syndication.feeds

comment:3 Changed 9 years ago by anonymous

Resolution: invalid
Status: closedreopened
Triage Stage: UnreviewedDesign decision needed
Version: 0.96

There might be another solution, but the reporter wants to run context processors on the templates (so they can access {{ MEDIA_URL }} in this case).

I'm not sure an overall replacement is in order, but an option seems fair.

comment:4 Changed 9 years ago by Chris Beaven

(that was me)

comment:5 Changed 9 years ago by Jacob

Triage Stage: Design decision neededAccepted

comment:6 Changed 9 years ago by Jacob

Resolution: fixed
Status: reopenedclosed

(In [7324]) Fixed #5107: swapped in RequestContext for Context in syndication feeds. Thanks, peter@…

Note: See TracTickets for help on using tickets.
Back to Top