From f32c30607b9141adff493b601b585571d3554f7c Mon Sep 17 00:00:00 2001 From: hemant10yadav Date: Mon, 9 Dec 2024 11:36:40 +0530 Subject: [PATCH] refactor table --- commcare_connect/opportunity/views.py | 71 ++++++++++++------ commcare_connect/templates/base.html | 4 - .../opportunity/all_opportunities_view.html | 43 ++++++++--- .../opportunity/opportunity_list.html | 75 ++----------------- .../partial_opportunity_table.html | 74 ++++++++++++++++++ commcare_connect/templates/pagination.html | 72 ++++++++++++++---- 6 files changed, 223 insertions(+), 116 deletions(-) create mode 100644 commcare_connect/templates/opportunity/partial_opportunity_table.html diff --git a/commcare_connect/opportunity/views.py b/commcare_connect/opportunity/views.py index 081ac0a0..f69baaa5 100644 --- a/commcare_connect/opportunity/views.py +++ b/commcare_connect/opportunity/views.py @@ -12,6 +12,7 @@ from django.forms import modelformset_factory from django.http import FileResponse, Http404, HttpResponse from django.shortcuts import get_object_or_404, redirect, render +from django.template.loader import render_to_string from django.urls import reverse from django.utils.html import format_html from django.utils.safestring import mark_safe @@ -170,28 +171,6 @@ def get_context_data(self, **kwargs): return context -class AllOpportunitiesView(SuperUserMixin, SingleTableView): - model = Opportunity - paginate_by = 15 - table_class = OpportunityTable - template_name = "opportunity/all_opportunities_view.html" - - def get_queryset(self): - ordering = self.request.GET.get("sort", "name") - if ordering not in [ - "name", - "-name", - "start_date", - "-start_date", - "end_date", - "-end_date", - "active", - "-active", - ]: - ordering = "name" - return Opportunity.objects.all().order_by(ordering) - - class OpportunityCreate(OrganizationUserMemberRoleMixin, CreateView): template_name = "opportunity/opportunity_create.html" form_class = OpportunityCreationForm @@ -1250,3 +1229,51 @@ def invoice_approve(request, org_slug, pk): ) payment.save() return HttpResponse(headers={"HX-Trigger": "newInvoice"}) + + +class AllOpportunitiesView(SuperUserMixin, SingleTableView): + model = Opportunity + paginate_by = 10 + template_name = "opportunity/all_opportunities_view.html" + table_class = OpportunityTable + + def get_queryset(self): + search_query = self.request.GET.get("search", "") + status_filter = self.request.GET.get("status", "") + ordering = self.request.GET.get("sort", "name") + + valid_ordering = [ + "name", + "-name", + "start_date", + "-start_date", + "end_date", + "-end_date", + ] + + if ordering not in valid_ordering: + ordering = "name" + + opportunities = Opportunity.objects.all() + + if search_query: + opportunities = opportunities.filter(name__icontains=search_query) + + if status_filter: + if status_filter == "active": + opportunities = opportunities.filter(active=True, end_date__gte=now().date()) + elif status_filter == "inactive": + opportunities = opportunities.filter(end_date__lt=now().date()) + + return opportunities.order_by(ordering) + + def get_context_data(self, **kwargs): + context = super().get_context_data(**kwargs) + context["table"] = self.table_class(self.get_queryset()) + return context + + def render_to_response(self, context, **response_kwargs): + if self.request.htmx: + html = render_to_string("opportunity/partial_opportunity_table.html", context, request=self.request) + return HttpResponse(html) + return super().render_to_response(context, **response_kwargs) diff --git a/commcare_connect/templates/base.html b/commcare_connect/templates/base.html index 947c260b..f417ddab 100644 --- a/commcare_connect/templates/base.html +++ b/commcare_connect/templates/base.html @@ -65,10 +65,6 @@ {% translate "Opportunities" %} -
  • - {% translate "All opportunities" %} -
    • {% endif %} {% endif %} diff --git a/commcare_connect/templates/opportunity/all_opportunities_view.html b/commcare_connect/templates/opportunity/all_opportunities_view.html index 05f696cf..89875d8a 100644 --- a/commcare_connect/templates/opportunity/all_opportunities_view.html +++ b/commcare_connect/templates/opportunity/all_opportunities_view.html @@ -1,20 +1,43 @@ -{% extends "base.html" %} +{% extends "opportunity/base.html" %} {% load static %} -{% load sort_link %} {% load i18n %} -{% load django_tables2 %} +{% block title %}{{ request.org }} - All Opportunities{% endblock %} -{% block title %}{{ request.org }} - Opportunities{% endblock %} +{% block breadcrumbs_inner %} + {{ block.super }} +
  • All-Org View
    • +{% endblock %} {% block content %} -
    -
    -
    +
    +

    All Opportunities

    -
    - {% render_table table %} +
    +
    +
    +
    +
    + +
    +
    + +
    +
    + +
    +
    +
    +
    + {% include 'opportunity/partial_opportunity_table.html' %}
    -
    {% endblock content %} diff --git a/commcare_connect/templates/opportunity/opportunity_list.html b/commcare_connect/templates/opportunity/opportunity_list.html index 04be5a47..a373d88e 100644 --- a/commcare_connect/templates/opportunity/opportunity_list.html +++ b/commcare_connect/templates/opportunity/opportunity_list.html @@ -19,6 +19,13 @@

    Opportunities + {% if request.user.is_superuser %} + + All-Org View + + {% endif %} {% if request.org_membership.is_viewer %}

    - - - - - - - - - - - - - {% for opportunity in page_obj %} - - - - - - - - - - {% empty %} - - - - {% endfor %} - -
    {% sort_link 'name' 'Name' %}{% sort_link 'start_date' 'Start Date' %}{% sort_link 'end_date' 'End Date' %}StatusProgramManage
    {{ opportunity.name }}{{ opportunity.start_date|default:"Not Set" }}{{ opportunity.end_date|default:"Not Set" }} - {% if opportunity.is_setup_complete %} - {% if opportunity.is_active %} - Active - {% else %} - Inactive - {% endif %} - {% else %} - Pending Setup - {% endif %} - {% if opportunity.managed %} {{ opportunity.managedopportunity.program.name }} {% else %} - {% endif %} -
    - -  View - - {% if request.org_membership.is_viewer %} - - {% if not opportunity.managed %} - - {% endif %} - {% else %} -  Edit - {% if not opportunity.managed %} -  Add Budget - - {% endif %} - {% endif %} -
    -
    {% translate "No opportunities yet." %}
    + {% include 'opportunity/partial_opportunity_table.html' %}
    - {% include 'pagination.html' %}
    {% if program_invitation_table and program_invitation_table.data %} diff --git a/commcare_connect/templates/opportunity/partial_opportunity_table.html b/commcare_connect/templates/opportunity/partial_opportunity_table.html new file mode 100644 index 00000000..3cc64ca4 --- /dev/null +++ b/commcare_connect/templates/opportunity/partial_opportunity_table.html @@ -0,0 +1,74 @@ +{% load sort_link %} +{% load django_tables2 %} +{% load i18n %} +
    + + + + + + + + + + + + + {% for opportunity in page_obj %} + + + + + + + + + + {% empty %} + + + + {% endfor %} + +
    {% sort_link 'name' 'Name' %}{% sort_link 'start_date' 'Start Date' %}{% sort_link 'end_date' 'End Date' %}StatusProgramManage
    {{ opportunity.name }}{{ opportunity.start_date|default:"Not Set" }}{{ opportunity.end_date|default:"Not Set" }} + {% if opportunity.is_setup_complete %} + {% if opportunity.is_active %} + Active + {% else %} + Inactive + {% endif %} + {% else %} + Pending Setup + {% endif %} + {% if opportunity.managed %} {{ opportunity.managedopportunity.program.name }} {% else %} - {% endif %} +
    + +  View + + {% if request.org_membership.is_viewer %} + + {% if not opportunity.managed %} + + {% endif %} + {% else %} +  Edit + {% if not opportunity.managed %} +  Add Budget + + {% endif %} + {% endif %} +
    +
    {% translate "No opportunities yet." %}
    +
    + {% include 'pagination.html' %} +
    +
    diff --git a/commcare_connect/templates/pagination.html b/commcare_connect/templates/pagination.html index 0d0a1c8e..8684cc1d 100644 --- a/commcare_connect/templates/pagination.html +++ b/commcare_connect/templates/pagination.html @@ -16,21 +16,67 @@ {% endif %} - {% for page_number in page_obj.paginator.page_range %} - {% if page_obj.number == page_number %} -
  • - - {{ page_number }} - -
    • + {% with total_pages=page_obj.paginator.num_pages current_page=page_obj.number %} + {% if total_pages > 10 %} + {# First page #} + {% if current_page > 4 %} +
  • + 1 +
    • +
  • + ... +
    • + {% endif %} + + {# Intelligent page range #} + {% with start=current_page|add:"-2" end=current_page|add:"2" %} + {% for page_number in page_obj.paginator.page_range %} + {% if page_number >= start and page_number <= end %} + {% if current_page == page_number %} +
  • + {{ page_number }} +
    • + {% else %} +
  • + + {{ page_number }} + +
    • + {% endif %} + {% endif %} + {% endfor %} + {% endwith %} + + {# Last page #} + {% if current_page < total_pages|add:"-3" %} +
  • + ... +
    • +
  • + + {{ total_pages }} + +
    • + {% endif %} + {% else %} -
  • - - {{ page_number }} - -
    • + {# If total pages are 10 or less, show all pages normally #} + {% for page_number in page_obj.paginator.page_range %} + {% if page_obj.number == page_number %} +
  • + {{ page_number }} +
    • + {% else %} +
  • + + {{ page_number }} + +
    • + {% endif %} + {% endfor %} {% endif %} - {% endfor %} + {% endwith %} + {% if page_obj.has_next %}