Use a with block rather than set to avoid leaking variables

This commit is contained in:
Gregoo 2025-01-31 11:05:19 +01:00
parent 4278e088a4
commit 73fa71e380

View File

@ -19,23 +19,24 @@
</thead>
<tbody>
{% for item in table_collection %}
{% set retirement_date = retired.get(item.fields.set) %}
<tr>
{{ table.image(item.url_for_image(), caption=item.fields.name, alt=item.fields.set) }}
<td>{{ item.fields.set }} {{ table.rebrickable(item) }}</td>
<td>{{ item.fields.name }}</td>
<td>{{ item.theme.name }}</td>
<td>{{ item.fields.year }}</td>
<td>{{ item.fields.number_of_parts }}</td>
<td>{% if retirement_date %}{{ retirement_date }}{% else %}<span class="badge rounded-pill text-bg-light border">Not found</span>{% endif %}</td>
{% if g.login.is_authenticated() %}
<td>
<form action="{{ item.url_for_delete() }}" method="post">
<button type="submit" class="btn btn-sm btn-danger"><i class="ri-delete-bin-2-line"></i> Delete</button>
</form>
</td>
{% endif %}
</tr>
{% with retirement_date = retired.get(item.fields.set) %}
<tr>
{{ table.image(item.url_for_image(), caption=item.fields.name, alt=item.fields.set) }}
<td>{{ item.fields.set }} {{ table.rebrickable(item) }}</td>
<td>{{ item.fields.name }}</td>
<td>{{ item.theme.name }}</td>
<td>{{ item.fields.year }}</td>
<td>{{ item.fields.number_of_parts }}</td>
<td>{% if retirement_date %}{{ retirement_date }}{% else %}<span class="badge rounded-pill text-bg-light border">Not found</span>{% endif %}</td>
{% if g.login.is_authenticated() %}
<td>
<form action="{{ item.url_for_delete() }}" method="post">
<button type="submit" class="btn btn-sm btn-danger"><i class="ri-delete-bin-2-line"></i> Delete</button>
</form>
</td>
{% endif %}
</tr>
{% endwith %}
{% endfor %}
</tbody>
</table>