Use order.location_name instead of order.location.name

app/models/order.py

@@ -22,9 +22,7 @@ class Order(db.Model):
 
     def __getattr__(self, name):
         if name == "location":
-            location = first(filter(lambda l: l.id == self.location_id, location_definitions))
+            return first(filter(lambda l: l.id == self.location_id, location_definitions))
-            if location:
-                return location
         raise AttributeError()
 
     def __repr__(self) -> str:

app/templates/order.html

@@ -25,7 +25,11 @@
         </form>
         {% endif %}
         <br/>
-        location: <a href="{{ order.location.website }}">{{ order.location.name }}</a><br/>
+        location: {% if order.location %}
+            <a href="{{ url_for('general_bp.location', location_id=order.location_id) }}">{{ order.location_name }}</a>
+        {% else %}
+            {{ order.location_name }}
+        {% endif %}<br/>
         {% if order.location.telephone != None %}
             telephone: <a href="tel://{{ order.location.telephone }}">{{ order.location.telephone }}</a><br/>
         {% endif %}

app/templates/utils.html

@@ -1,7 +1,7 @@
 {% macro render_order(order) -%}
 <div class="row darke order_row">
     <div class="col-md-8 col-lg-9 order_data">
-        <h5>{{ order.location.name }}</h5>
+        <h5>{{ order.location_name }}</h5>
         <b class="amount_of_orders">{{ order.items.count() }} orders</b></p>
         <p class="time_data">
         {% if order.stoptime %}

app/views/order.py

@@ -57,6 +57,7 @@ def order_from_id(order_id: int, form: OrderForm = None) -> str:
     if form is None:
         form = AnonOrderItemForm() if current_user.is_anonymous() \
             else OrderItemForm()
+        if order.location:
             form.populate(order.location)
     if order.is_closed():
         form = None