Don't show table unless there are items to show

This commit is contained in:
Scot Hacker 2018-03-12 23:20:39 -07:00
parent b361abeaf6
commit 982864e1ba
2 changed files with 59 additions and 57 deletions

View file

@ -34,6 +34,7 @@
</form> </form>
{% endif %} {% endif %}
{% if items %}
{% if list_slug == "mine" %} {% if list_slug == "mine" %}
<h1>Tasks assigned to me (in all groups)</h1> <h1>Tasks assigned to me (in all groups)</h1>
{% else %} {% else %}
@ -92,6 +93,9 @@
</table> </table>
<p><input type="submit" name="process_tasks" value="Continue..." class="todo-button"></p> <p><input type="submit" name="process_tasks" value="Continue..." class="todo-button"></p>
</form> </form>
{% else %}
<h4>No items on this list yet (add one!)</h4>
{% endif %}
{% if list_slug != "mine" %} {% if list_slug != "mine" %}
{% if view_completed %} {% if view_completed %}
@ -104,7 +108,6 @@
{% endif %} {% endif %}
{% endblock %} {% endblock %}
{% block extra_js %} {% block extra_js %}

View file

@ -22,7 +22,7 @@ from todo.utils import toggle_done, toggle_deleted, send_notify_mail
def check_user_allowed(user): def check_user_allowed(user):
""" """
Verifies user is logged in, and in staff if that setting is enabled. Verifies user is logged in, and in staff if that setting is enabled.
Per-object permission checks (e.g. to view a particular list) must be in the views that handle those objects. Per-object permission checks (e.g. to view a particular list) are in the views that handle those objects.
""" """
if settings.STAFF_ONLY: if settings.STAFF_ONLY:
@ -106,7 +106,6 @@ def list_detail(request, list_id=None, list_slug=None, view_completed=False):
task_list = get_object_or_404(TaskList, id=list_id) task_list = get_object_or_404(TaskList, id=list_id)
items = Item.objects.filter(task_list=task_list.id) items = Item.objects.filter(task_list=task_list.id)
# Apply filters to base queryset
if view_completed: if view_completed:
items = items.filter(completed=True) items = items.filter(completed=True)
else: else: