do not show Request Support links for old purchases (bug 710789)

Author: cvan

apps/addons/templates/addons/listing/items_mobile.html

@@ -56,7 +56,7 @@ <h3>{{ addon.name }}</h3>
             <span>
           {% endif %}
           {% if contribs %}
-            {{ refund_info(amo, addon, contributions, is_refunded, is_chargeback) }}
+            {{ refund_info(amo, addon, contributions) }}
           {% endif %}
         </div>
       </details>

apps/addons/templates/addons/macros.html

@@ -58,12 +58,16 @@
   {% endif %}
 {% endmacro %}
 
-{% macro refund_info(amo, addon, contributions, is_refunded, is_chargeback) %}
+{% macro refund_info(amo, addon, contributions) %}
   {% set contributions = contributions.get(addon.pk, []) %}
   {% if contributions %}
-    {% set get_support = not is_refunded and not is_chargeback %}
     <ul class="contributions">
+      {% set last_contrib = contributions[-1].type == amo.CONTRIB_PURCHASE
+                            and contributions[-1].id %}
       {% for contribution in contributions %}
+        {# Show "Request Support" link if this is the last contribution
+           and is a purchase. #}
+        {% set get_support = contribution.id == last_contrib %}
         <li>
             {% if contribution.type == amo.CONTRIB_PURCHASE %}
               <span class="purchase{{ ' supportable' if get_support }}">

apps/users/templates/users/purchases.html

@@ -33,7 +33,7 @@ <h3>
                   {{ impala_reviews_link(addon) }}
                   {{ price(addon) }}
                   {{ sort_vital(addon) }}
-                  {{ refund_info(amo, addon, contributions, is_refunded, is_chargeback) }}
+                  {{ refund_info(amo, addon, contributions) }}
                 </div>
               </div>
               <div class="action">

apps/users/tests/test_views.py

@@ -1579,7 +1579,9 @@ def _test_repurchased(self):
         assert not item.hasClass('reversed'), (
             "Unexpected 'refunded' class on '.item'")
         assert not item.find('.refund-notice'), 'Unexpected refund message'
-        eq_(item.find('.purchase').eq(2).siblings('a').attr('href'),
+        purchases = item.find('.contributions')
+        eq_(purchases.find('.request-support').length, 1)
+        eq_(purchases.find('li').eq(2).find('.request-support').attr('href'),
             reverse('users.support', args=[c[1].id]))
 
     def test_repurchased(self):