ECOM-528 refactor the code.

common/djangoapps/student/tests/test_recent_enrollments.py

@@ -162,8 +162,6 @@ class TestRecentEnrollments(ModuleStoreTestCase):
     )
     @ddt.unpack
     def test_donate_button(self, course_modes, enrollment_mode, show_donate):
-        from nose.tools import set_trace;
-        set_trace()
         # Enable the enrollment success message
         self._configure_message_timeout(10000)
 

common/djangoapps/student/views.py

@@ -758,16 +758,13 @@ def _allow_donation(course_modes, course_id, enrollment):
         True if the course is allowing donations.
 
     """
-    # from nose.tools import set_trace;
-    # set_trace()
     donations_enabled = DonationConfiguration.current().enabled
     is_verified_mode = CourseMode.has_verified_mode(course_modes[course_id])
     has_payment_option = CourseMode.has_payment_options(course_id)
     return_val = False
     if donations_enabled:
-        if not is_verified_mode:
-            if not has_payment_option:
-                return_val = True
+        if not (is_verified_mode and has_payment_option):
+            return_val = True
         else:
             if enrollment.mode in ['audit', 'honor']:
                 return_val = True