diff --git a/app/views/general.py b/app/views/general.py index 9946fe6..8fd5b4f 100644 --- a/app/views/general.py +++ b/app/views/general.py @@ -31,11 +31,11 @@ def home() -> str: "Generate the home view" prev_day = datetime.now() - timedelta(days=1) recently_closed = get_orders( - (Order.stoptime > prev_day) & (Order.stoptime < datetime.now()) & Order.association.in_(current_user.association_list()) + (Order.stoptime > prev_day) & (Order.stoptime < datetime.now()) ) return render_template( "home.html", orders=get_orders( - ((datetime.now() > Order.starttime) & (Order.stoptime > datetime.now()) | (Order.stoptime == None)) & Order.association.in_(current_user.association_list()) + ((datetime.now() > Order.starttime) & (Order.stoptime > datetime.now()) | (Order.stoptime == None)) ), recently_closed=recently_closed ) diff --git a/app/views/order.py b/app/views/order.py index 80d930a..840fefd 100644 --- a/app/views/order.py +++ b/app/views/order.py @@ -27,7 +27,7 @@ def orders(form: OrderForm = None) -> str: form.location_id.default = location_id form.process() form.populate() - return render_template("orders.html", orders=get_orders(expression=Order.association.in_(current_user.association_list())), form=form) + return render_template("orders.html", orders=get_orders(), form=form) @order_bp.route("/create", methods=["POST"]) @@ -396,16 +396,17 @@ def get_orders(expression=None) -> typing.List[Order]: """Give the list of all currently open and public Orders""" order_list: typing.List[OrderForm] = [] if expression is None: - expression = (datetime.now() > Order.starttime) & ( - Order.stoptime - > datetime.now() - # pylint: disable=C0121 - ) | (Order.stoptime == None) + expression = ((datetime.now() > Order.starttime) & ( + Order.stoptime + > datetime.now() + # pylint: disable=C0121 + ) | (Order.stoptime == None) + ) & (Order.association.in_(current_user.association_list())) if not current_user.is_anonymous(): order_list = Order.query.filter(expression).all() else: order_list = Order.query.filter( # pylint: disable=C0121 - expression & (Order.public == True) + expression & (Order.public == True) & (Order.association.in_(current_user.association_list())) ).all() return order_list