From b1000dd0f816bdb007fd96506ac4236466edc214 Mon Sep 17 00:00:00 2001 From: Ethan Dalool Date: Fri, 1 Jan 2021 13:22:42 -0800 Subject: [PATCH] Move orderby direction check above column expansion. --- etiquette/searchhelpers.py | 24 ++++++++++++------------ 1 file changed, 12 insertions(+), 12 deletions(-) diff --git a/etiquette/searchhelpers.py b/etiquette/searchhelpers.py index 6751da7..f79943e 100644 --- a/etiquette/searchhelpers.py +++ b/etiquette/searchhelpers.py @@ -324,6 +324,18 @@ def normalize_orderby(orderby, warning_bag=None): raise exc continue + if direction not in ('asc', 'desc'): + message = constants.WARNING_ORDERBY_BADDIRECTION.format( + column=column, + direction=direction, + ) + exc = ValueError(message) + if warning_bag: + warning_bag.add(exc) + else: + raise exc + direction = 'desc' + if column not in constants.ALLOWED_ORDERBY_COLUMNS: exc = ValueError(constants.WARNING_ORDERBY_BADCOL.format(column=column)) if warning_bag: @@ -347,18 +359,6 @@ def normalize_orderby(orderby, warning_bag=None): elif column == 'ratio': column = '(width / height)' - if direction not in ('asc', 'desc'): - message = constants.WARNING_ORDERBY_BADDIRECTION.format( - column=column, - direction=direction, - ) - exc = ValueError(message) - if warning_bag: - warning_bag.add(exc) - else: - raise exc - direction = 'desc' - requested_order = (column, direction) final_orderby.append(requested_order)