Fix for #1813: Refactor error messages for rewards

evap/rewards/tools.py

@@ -32,26 +32,22 @@ def save_redemptions(request, redemptions: Dict[int, int], previous_redeemed_poi
     # check consistent previous redeemed points
     # do not validate reward points, to allow receiving points after page load
     if previous_redeemed_points != redeemed_points_of_user(request.user):
-        raise OutdatedRedemptionData(
-            _(
-                "It appears that your browser sent multiple redemption requests. You can see all successful redemptions below."
-            )
-        )
+        raise OutdatedRedemptionData()
 
     total_points_available = reward_points_of_user(request.user)
     total_points_redeemed = sum(redemptions.values())
 
     if total_points_redeemed <= 0:
-        raise NoPointsSelected(_("You cannot redeem 0 points."))
+        raise NoPointsSelected()
 
     if total_points_redeemed > total_points_available:
-        raise NotEnoughPoints(_("You don't have enough reward points."))
+        raise NotEnoughPoints()
 
     for event_id in redemptions:
         if redemptions[event_id] > 0:
             event = get_object_or_404(RewardPointRedemptionEvent, pk=event_id)
             if event.redeem_end_date < date.today():
-                raise RedemptionEventExpired(_("Sorry, the deadline for this event expired already."))
+                raise RedemptionEventExpired()
 
             RewardPointRedemption.objects.create(user_profile=request.user, value=redemptions[event_id], event=event)
 

evap/rewards/views.py

@@ -41,12 +41,20 @@ def redeem_reward_points(request):
     try:
         save_redemptions(request, redemptions, previous_redeemed_points)
         messages.success(request, _("You successfully redeemed your points."))
-    except (NoPointsSelected, NotEnoughPoints, RedemptionEventExpired) as error:
-        messages.warning(request, error)
-        return 400
-    except OutdatedRedemptionData as error:
-        messages.error(request, error)
-        return 409
+    except (NoPointsSelected, NotEnoughPoints, RedemptionEventExpired, OutdatedRedemptionData) as error:
+        error_string = ""
+        status_code = 400
+        if isinstance(error, NoPointsSelected):
+            error_string = "You cannot redeem 0 points."
+        elif isinstance(error, NotEnoughPoints):
+            error_string = "You don't have enough reward points."
+        elif isinstance(error, RedemptionEventExpired):
+            error_string = "Sorry, the deadline for this event expired already."
+        elif isinstance(error, OutdatedRedemptionData):
+            status_code = 409
+            error_string = "It appears that your browser sent multiple redemption requests. You can see all successful redemptions below."
+        messages.warning(request, _(error_string))
+        return status_code
     return 200