Final iteration

lms/djangoapps/shoppingcart/tests/test_reports.py

@@ -14,7 +14,7 @@ from django.test.utils import override_settings
 from course_modes.models import CourseMode
 from courseware.tests.tests import TEST_DATA_MONGO_MODULESTORE
 from shoppingcart.models import (Order, CertificateItem, PaidCourseRegistration, PaidCourseRegistrationAnnotation)
-from shoppingcart.views import initialize_report, REPORT_TYPES
+from shoppingcart.views import initialize_report
 from student.tests.factories import UserFactory
 from student.models import CourseEnrollment
 from xmodule.modulestore.tests.django_utils import ModuleStoreTestCase

lms/djangoapps/shoppingcart/tests/test_views.py

@@ -398,7 +398,6 @@ class CSVReportViewsTest(ModuleStoreTestCase):
         self.assertEqual(response['Content-Type'], 'text/csv')
         report = initialize_report(report_type, start_date, end_date, start_letter, end_letter)
         self.assertIn(",".join(report.header()), response.content)
-        # TODO add another test here
 
 
 class UtilFnsTest(TestCase):

lms/djangoapps/shoppingcart/views.py

@@ -196,8 +196,6 @@ def csv_report(request):
     if not _can_download_report(request.user):
         return HttpResponseForbidden(_('You do not have permission to view this page.'))
 
-    # TODO temp filler for start letter, end letter
-
     if request.method == 'POST':
         start_date = request.POST.get('start_date', '')
         end_date = request.POST.get('end_date', '')

lms/envs/common.py

@@ -203,7 +203,7 @@ FEATURES = {
     # only edX superusers can perform the downloads)
     'ALLOW_COURSE_STAFF_GRADE_DOWNLOADS': False,
 
-    'ENABLED_PAYMENT_REPORTS': [ "refund_report", "itemized_purchase_report", "university_revenue_share",  "certificate_status"],
+    'ENABLED_PAYMENT_REPORTS': ["refund_report", "itemized_purchase_report", "university_revenue_share", "certificate_status"],
 }
 
 # Used for A/B testing

lms/templates/shoppingcart/download_report.html

@@ -14,6 +14,7 @@
     </section>
   % endif
   <form method="post">
+  %if ("itemized_purchase_report" or "refund_report") in settings.FEATURES['ENABLED_PAYMENT_REPORTS']:
     <p>${_("These reports are delimited by start and end dates.")}</p>
     <label for="start_date">${_("Start Date: ")}</label>
     <input id="start_date" type="text" value="${start_date}" name="start_date"/>
@@ -22,17 +23,19 @@
     <br/>
 
     %if "itemized_purchase_report" in settings.FEATURES['ENABLED_PAYMENT_REPORTS']:
-    <button type = "submit" name="requested_report" value="itemized_purchase_report">Itemized Purchase Report</button>
-    <br/>
+      <button type = "submit" name="requested_report" value="itemized_purchase_report">Itemized Purchase Report</button>
+      <br/>
     %endif
 
     %if "refund_report" in settings.FEATURES['ENABLED_PAYMENT_REPORTS']:
-    <button type = "submit" name="requested_report" value="refund_report">Refund Report</button>
-    <br/>
+      <button type = "submit" name="requested_report" value="refund_report">Refund Report</button>
+      <br/>
     %endif
 
     <br/>
-
+  %endif
+  
+  %if ("certificate_status" or "university_revenue_share") in settings.FEATURES['ENABLED_PAYMENT_REPORTS']:
     <p>${_("These reports are delimited alphabetically by university name.  i.e., generating a report with 'Start Letter' A and 'End Letter' C will generate reports for all universities starting with A, B, and C.")}</p>
     <label for="start_letter">${_("Start Letter: ")}</label>
     <input id="start_letter" type="text" value="${start_letter}" name="start_letter"/>
@@ -42,13 +45,14 @@
     <br/>
 
     %if "university_revenue_share" in settings.FEATURES['ENABLED_PAYMENT_REPORTS']:
-    <button type = "submit" name="requested_report" value="university_revenue_share">University Revenue Share</button>
-    <br/>
+      <button type = "submit" name="requested_report" value="university_revenue_share">University Revenue Share</button>
+      <br/>
     %endif
 
     %if "certificate_status" in settings.FEATURES['ENABLED_PAYMENT_REPORTS']:
-    <button type="submit" name="requested_report" value="certificate_status">Certiciate Status</button>
-    <br/>
+      <button type="submit" name="requested_report" value="certificate_status">Certiciate Status</button>
+      <br/>
     %endif
+  %endif
   </form>
 </section>