From 24324b642a16efbb175e624dbbfb4e499afbfcc6 Mon Sep 17 00:00:00 2001 From: Brian Simpson Date: Fri, 8 Nov 2013 08:12:58 -0500 Subject: [PATCH] promote.py: Delete unneeded wrap_promoted and sponsor_wrapper. --- r2/r2/controllers/promotecontroller.py | 2 +- r2/r2/lib/pages/pages.py | 7 +++---- r2/r2/lib/promote.py | 11 ----------- 3 files changed, 4 insertions(+), 16 deletions(-) diff --git a/r2/r2/controllers/promotecontroller.py b/r2/r2/controllers/promotecontroller.py index 4ba14dad7..6674faf0b 100644 --- a/r2/r2/controllers/promotecontroller.py +++ b/r2/r2/controllers/promotecontroller.py @@ -272,7 +272,7 @@ class PromoteController(ListingController): def GET_edit_promo(self, link): if not link or link.promoted is None: return self.abort404() - rendered = wrap_links(link, wrapper=promote.sponsor_wrapper, skip=False) + rendered = wrap_links(link, skip=False) form = PromoteLinkForm(link, rendered) page = PromotePage('new_promo', content=form) return page.render() diff --git a/r2/r2/lib/pages/pages.py b/r2/r2/lib/pages/pages.py index 3ebfff551..4c7e2f409 100644 --- a/r2/r2/lib/pages/pages.py +++ b/r2/r2/lib/pages/pages.py @@ -3563,7 +3563,7 @@ class PromoteLinkForm(Templated): self.subreddit_selector = SubredditSelector() - self.link = promote.wrap_promoted(link) + self.link = link self.listing = listing campaigns = PromoCampaign._by_link(link._id) self.campaigns = promote.get_renderable_campaigns(link, campaigns) @@ -3598,8 +3598,7 @@ class RefundPage(Reddit): def __init__(self, link, campaign): self.link = link self.campaign = campaign - self.listing = wrap_links(link, wrapper=promote.sponsor_wrapper, - skip=False) + self.listing = wrap_links(link, skip=False) billable_impressions = promote.get_billable_impressions(campaign) billable_amount = promote.get_billable_amount(campaign, billable_impressions) @@ -3783,7 +3782,7 @@ class MediaEmbedBody(CachedTemplate): class PaymentForm(Templated): def __init__(self, link, campaign, **kw): - self.link = promote.wrap_promoted(link) + self.link = link self.campaign = promote.get_renderable_campaigns(link, campaign) Templated.__init__(self, **kw) diff --git a/r2/r2/lib/promote.py b/r2/r2/lib/promote.py index 4818717b9..40f03c5c6 100644 --- a/r2/r2/lib/promote.py +++ b/r2/r2/lib/promote.py @@ -30,7 +30,6 @@ import time from pylons import g, c from pylons.i18n import ungettext -from r2.lib.wrapped import Wrapped from r2.lib import ( amqp, authorize, @@ -235,10 +234,6 @@ def get_renderable_campaigns(link, campaigns): r = r[0] return r -def wrap_promoted(link): - if not isinstance(link, Wrapped): - link = Wrapped(link) - return link # These could be done with relationships, but that seeks overkill as # we never query based on user and only check per-thing @@ -307,12 +302,6 @@ def new_promotion(title, url, selftext, user, ip): emailer.new_promo(l) return l -def sponsor_wrapper(link): - w = Wrapped(link) - w.render_class = PromotedLink - w.rowstyle = "promoted link" - return w - def get_transactions(link, campaigns): """Return Bids for specified campaigns on the link.