Opened 8 years ago

Closed 7 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@… 8 years ago.
adjust feed to use RequestContext for Title and Description Template Render

Download all attachments as: .zip

Change History (7)

Changed 8 years ago by peter@…

adjust feed to use RequestContext for Title and Description Template Render

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

  • Needs documentation unset
  • Needs tests unset
  • Patch needs improvement unset
  • Resolution set to invalid
  • Status changed from new to closed

Why? what benefits does this have?

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

  • Summary changed from Haut Simple! django.contrib.syndication.feeds to Replace Context with RequestContext in django.contrib.syndication.feeds

comment:3 Changed 8 years ago by anonymous

  • Resolution invalid deleted
  • Status changed from closed to reopened
  • Triage Stage changed from Unreviewed to Design decision needed
  • Version 0.96 deleted

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 8 years ago by SmileyChris

(that was me)

comment:5 Changed 8 years ago by jacob

  • Triage Stage changed from Design decision needed to Accepted

comment:6 Changed 7 years ago by jacob

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

(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