From 41fdbce89ba0141c5ad703b5f91cd700299c0f8b Mon Sep 17 00:00:00 2001
From: Brian Simpson <bsimpson63@gmail.com>
Date: Thu, 6 Mar 2014 15:01:15 -0500
Subject: [PATCH] POST_request_promo: skip hidden items.

---
 r2/r2/controllers/api.py | 2 +-
 r2/r2/lib/promote.py     | 4 ++++
 2 files changed, 5 insertions(+), 1 deletion(-)

diff --git a/r2/r2/controllers/api.py b/r2/r2/controllers/api.py
index ad9c24f6d..7a9f26409 100755
--- a/r2/r2/controllers/api.py
+++ b/r2/r2/controllers/api.py
@@ -3832,7 +3832,7 @@ class ApiController(RedditController):
         promo_tuples = promote.lottery_promoted_links(srnames, n=10)
         builder = CampaignBuilder(promo_tuples,
                                   wrap=default_thing_wrapper(),
-                                  keep_fn=promote.is_promoted,
+                                  keep_fn=promote.promo_keep_fn,
                                   num=1,
                                   skip=True)
         listing = LinkListing(builder, nextprev=False).listing()
diff --git a/r2/r2/lib/promote.py b/r2/r2/lib/promote.py
index 5290ad41b..55bda78d3 100644
--- a/r2/r2/lib/promote.py
+++ b/r2/r2/lib/promote.py
@@ -93,6 +93,10 @@ def cost_per_click(spend, clicks):
         return 0
 
 
+def promo_keep_fn(item):
+    return is_promoted(item) and not item.hidden
+
+
 # attrs
 
 def promo_traffic_url(l): # old traffic url