feat(favorites): frontend revisions to pray and pay project

cl/assets/templates/base.html

@@ -233,7 +233,7 @@ <h1>You did not supply the "private" variable to your template.
                 </li>
                 {% flag "pray-and-pay" %}
                   <li>
-                    <a href="{% url "top_prayers" %}#recap-alerts" tabindex="203">Pray and Pay Project</a>
+                    <a href="{% url "top_prayers" %}" tabindex="203">Pray and Pay Project</a>
                   </li>
                 {% endflag %}
                 <li>

cl/favorites/templates/top_prayers.html

@@ -12,28 +12,54 @@
 
 
 {% block content %}
+  <div class="col-xs-12">
+    <h1 class="text-center">Community's Most Requested Documents</h1>
+  </div>
+
+
+  <div class="col-xs-12">
+    <div class="well well-sm">
+      <p>There have been <b>{{ granted_stats.count }}</b> requests granted for <b>{{ granted_stats.num_distinct }}</b> unique documents that cost <b>${{ granted_stats.total_cost|floatformat:2 }}</b>.</p>
+      <br>
+      <p>There are <b>{{ waiting_stats.count }}</b> requests pending for <b>{{ waiting_stats.num_distinct }}</b> unique documents that cost at least <b>${{ waiting_stats.total_cost|floatformat:2 }}</b>.</p>
+      <br>
+      <p>{% if user.is_authenticated %}
+      <a href="{% url 'user_prayers' user.username %}">View your prayers</a>
+      {% else %}
+        <a href="{% url 'sign-in' %}?next={% url 'top_prayers' %}">Sign in to view your prayers</a>
+      {% endif %}</p>
+    </div>
+  </div>
 
   <div class="col-xs-12">
     <div class="table-responsive">
       <table class="settings-table table">
         <thead>
             <tr>
-              <th>User Preference</th>
-              <th>Document Description</th>
+              <th>Court</th>
+              <th>Case Name</th>
               <th>Document Number</th>
-              <th>PACER Doc ID</th>
-              <th>Document Court</th>
+              <th>Document Description</th>
               <th>Buy on Pacer</th>
             </tr>
         </thead>
         <tbody>
             {% for prayer in top_prayers %}
             <tr>
-              <td>{{ forloop.counter }}</td>
+              <td>{{ prayer.docket_entry.docket.court.citation_string }}</td>
+              {% with docket=prayer.docket_entry.docket %}
+              <td>
+                <a href="{% url "view_docket" docket.pk docket.slug %}">
+                  {{ prayer.docket_entry.docket|best_case_name|safe|v_wrapper }} ({{ prayer.docket_entry.docket.docket_number }})
+                </a>
+              </td>
+              <td>
+                <a href="{% url "view_docket" docket.pk docket.slug %}#entry-{{ prayer.docket_entry.entry_number }}">
+                  {{ prayer.document_number }}
+                </a>
+              </td>
+              {% endwith %}
               <td>{{ prayer.description }}</td>
-              <td>{{ prayer.document_number }}</td>
-              <td>{{ prayer.pacer_doc_id }}</td>
-              <td>{{ prayer.docket_entry.docket.court_id }}</td>
               <td><a href="{{ prayer.pacer_url }}"
                 {% if not request.COOKIES.buy_on_pacer_modal and not request.COOKIES.recap_install_plea %}
                   class="open_buy_pacer_modal btn btn-default btn-xs"

cl/favorites/templates/user_prayers.html

@@ -0,0 +1,117 @@
+{% extends "base.html" %}
+{% load extras %}
+{% load text_filters %}
+{% load static %}
+{% load pacer %}
+{% load tz %}
+
+{% block title %}{% if is_page_owner %}Your RECAP Requests {% else %}RECAP Requests for: {{ requested_user }}{% endif %} – CourtListener.com{% endblock %}
+{% block og_title %}{% if is_page_owner %}Your RECAP Requests {% else %}RECAP Requests for: {{ requested_user }}{% endif %} – CourtListener.com{% endblock %}
+{% block description %}CourtListener lets you request purchase of legal documents. View the document requests for {{ requested_user }}.{% endblock %}
+{% block og_description %}CourtListener lets you request purchase of legal documents. View the document requests for {{ requested_user }}.{% endblock %}
+
+{% block content %}
+  <div class="col-xs-12">
+    <h1 class="text-center">{% if is_page_owner %}Your RECAP Requests {% else %}RECAP Requests for: {{ requested_user }}{% endif %}</h1>
+  </div>
+
+  <div class="col-xs-12">
+    <div class="well well-sm">
+      {% if is_page_owner %}
+        <p>You have had <b>{{ count }}</b> requests fulfilled for documents that cost <b>${{ total_cost|floatformat:2 }}</b>.<p>
+        <p>{% if is_eligible %}You are eligible to make document requests.{% else %}You have reached your daily limit; wait 24 hours to make new requests.{% endif %}</p>
+      {% endif %}
+    </div>
+  </div>
+
+  <div class="col-xs-12">
+    <div class="table-responsive">
+      <table class="settings-table table">
+        <thead>
+            <tr>
+              <th>Court</th>
+              <th>Case Name</th>
+              <th>Document Number</th>
+              <th>Document Description</th>
+              <th>Requested On</th>
+              <th>Status</th>
+              {% if is_page_owner %}<th>Delete</th>{% endif %}
+            </tr>
+        </thead>
+        <tbody>
+            {% for prayer in prayers %}
+            <tr>
+              <td>{{ prayer.docket_entry.docket.court.citation_string }}</td>
+              {% with docket=prayer.docket_entry.docket %}
+              <td>
+                <a href="{% url "view_docket" docket.pk docket.slug %}">
+                  {{ prayer.docket_entry.docket|best_case_name|safe|v_wrapper }} ({{ prayer.docket_entry.docket.docket_number }})
+                </a>
+              </td>
+              <td>
+                <a href="{% url "view_docket" docket.pk docket.slug %}#entry-{{ prayer.docket_entry.entry_number }}">
+                  {{ prayer.document_number }}
+                </a>
+              </td>
+              {% endwith %}
+              <td>{{ prayer.description }}</td>
+              <td>{{ prayer.date_created|date:"M j, Y"  }}</td>
+              <td>{% if prayer.status == WAITING %} Pending {% elif prayer.status == GRANTED %}
+                <div class="btn-group hidden-xs col-sm-4 col-md-3 hidden-print flex">
+                                  {% if rd.filepath_local %}
+                                    <a href="{{ rd.filepath_local.url }}"
+                                       rel="nofollow"
+                                       role="button"
+                                       class="btn btn-primary btn-xs"
+                                            {% if rd.date_upload %}
+                                       title="Uploaded {{ rd.date_upload|timezone:timezone }}"
+                                            {% endif %}>
+                                      Download PDF
+                                    </a>
+                                    <button type="button"
+                                            class="btn btn-primary btn-xs dropdown-toggle"
+                                            data-toggle="dropdown"
+                                            aria-haspopup="true"
+                                            aria-expanded="false">
+                                      <span class="caret"></span>
+                                      <span class="sr-only">Toggle Dropdown</span>
+                                    </button>
+                                    <ul class="dropdown-menu">
+                                      <li>
+                                        <a href="{{ rd.filepath_local.url }}" rel="nofollow">From
+                                          CourtListener</a>
+                                      </li>
+                                      {% if rd.filepath_ia %}
+                                        <li>
+                                          <a href="{{ rd.filepath_ia }}"
+                                             rel="nofollow">From
+                                            Internet Archive</a>
+                                        </li>
+                                      {% endif %} {% endif %} {% endif %}</td>
+              {% if is_page_owner %}<td><a href="{% url 'delete_prayer' recap_document=prayer.pk %}">Delete</a></td>{% endif %}
+              {% comment %} <td><a href="{{ prayer.pacer_url }}"
+                {% if not request.COOKIES.buy_on_pacer_modal and not request.COOKIES.recap_install_plea %}
+                  class="open_buy_pacer_modal btn btn-default btn-xs"
+                  data-toggle="modal" data-target="#modal-buy-pacer"
+                {% else%}
+                  class="btn btn-default btn-xs"
+                {% endif %}
+                  target="_blank"
+                  rel="nofollow">Buy on PACER {% if prayer.page_count %}(${{ prayer|price }}){% endif %}</td> {% endcomment %}
+            </tr>
+            {% empty %}
+            <tr>
+                <td colspan="2">No document requests made. Consider making one!</td>
+            </tr>
+            {% endfor %}
+        </tbody>
+      </table>
+    </div>
+  </div>
+{% endblock %}
+
+{% block footer-scripts %}
+  <script defer type="text/javascript"
+            src="{% static "js/buy_pacer_modal.js" %}"></script>
+  {% include "includes/buy_pacer_modal.html" %}
+{% endblock %}

cl/favorites/urls.py

@@ -6,6 +6,7 @@
     delete_prayer_view,
     open_prayers,
     save_or_update_note,
+    user_prayers_view,
     view_tag,
     view_tags,
 )
@@ -25,6 +26,7 @@
         name="view_tag",
     ),
     path("tags/<str:username>/", view_tags, name="tag_list"),
+    # Prayer pages
     path("prayers/top/", open_prayers, name="top_prayers"),
     path(
         "prayer/create/<int:recap_document>/",
@@ -36,4 +38,5 @@
         delete_prayer_view,
         name="delete_prayer",
     ),
+    path("prayers/<str:username>/", user_prayers_view, name="user_prayers"),
 ]

cl/favorites/utils.py

@@ -1,7 +1,9 @@
 from datetime import timedelta
 
+from asgiref.sync import sync_to_async
 from django.conf import settings
 from django.contrib.auth.models import User
+from django.core.cache import cache
 from django.core.mail import EmailMultiAlternatives, get_connection
 from django.db.models import (
     Avg,
@@ -117,11 +119,16 @@ async def get_top_prayers() -> list[RECAPDocument]:
             "attachment_number",
             "pacer_doc_id",
             "page_count",
+            "is_free_on_pacer",
             "description",
+            "docket_entry__entry_number",
             "docket_entry__docket_id",
             "docket_entry__docket__slug",
+            "docket_entry__docket__case_name",
+            "docket_entry__docket__docket_number",
             "docket_entry__docket__pacer_case_id",
             "docket_entry__docket__court__jurisdiction",
+            "docket_entry__docket__court__citation_string",
             "docket_entry__docket__court_id",
         )
         .annotate(
@@ -147,6 +154,46 @@ async def get_top_prayers() -> list[RECAPDocument]:
     return [doc async for doc in documents.aiterator()]
 
 
+async def get_user_prayers(user: User) -> list[Prayer]:
+    user_prayers = Prayer.objects.filter(user=user).values("recap_document_id")
+
+    documents = (
+        RECAPDocument.objects.filter(id__in=Subquery(user_prayers))
+        .select_related(
+            "docket_entry",
+            "docket_entry__docket",
+            "docket_entry__docket__court",
+        )
+        .only(
+            "pk",
+            "document_type",
+            "document_number",
+            "attachment_number",
+            "pacer_doc_id",
+            "page_count",
+            "is_free_on_pacer",
+            "description",
+            "date_created",
+            "docket_entry__entry_number",
+            "docket_entry__docket_id",
+            "docket_entry__docket__slug",
+            "docket_entry__docket__case_name",
+            "docket_entry__docket__docket_number",
+            "docket_entry__docket__pacer_case_id",
+            "docket_entry__docket__court__jurisdiction",
+            "docket_entry__docket__court__citation_string",
+            "docket_entry__docket__court_id",
+        )
+        .annotate(
+            prayer_status=F("prayers__status"),
+            prayer_date_created=F("prayers__date_created"),
+        )
+        .order_by("prayers__date_created")
+    )
+
+    return [document async for document in documents.aiterator()]
+
+
 def send_prayer_emails(instance: RECAPDocument) -> None:
     open_prayers = Prayer.objects.filter(
         recap_document=instance, status=Prayer.WAITING
@@ -211,19 +258,46 @@ async def get_user_prayer_history(user: User) -> tuple[int, float]:
     return count, total_cost
 
 
-async def get_lifetime_prayer_stats() -> tuple[int, int, float]:
+async def get_lifetime_prayer_stats(
+    status: int,
+) -> tuple[
+    int, int, float
+]:  # status can be only 1 (WAITING) or 2 (GRANTED) based on the Prayer model
 
-    filtered_list = Prayer.objects.filter(status=Prayer.GRANTED)
+    cache_key = f"prayer-stats-{status}"
+
+    data = await cache.aget(cache_key)
+
+    if data is not None:
+        return (
+            data["count"],
+            data["num_distinct_purchases"],
+            data["total_cost"],
+        )
+
+    filtered_list = Prayer.objects.filter(status=status)
 
     count = await filtered_list.acount()
 
     total_cost = 0
     distinct_documents = set()
 
     async for prayer in filtered_list:
-        distinct_documents.add(prayer.recap_document)
-        total_cost += float(await price(prayer.recap_document))
+        recap_document = await sync_to_async(lambda: prayer.recap_document)()
 
-    num_distinct_purchases = len(distinct_documents)
+        distinct_documents.add(recap_document)
+
+        document_price = price(recap_document)
+        total_cost += float(document_price) if document_price else 0.0
+
+    num_distinct = len(distinct_documents)
+
+    data = {
+        "count": count,
+        "num_distinct_purchases": num_distinct,
+        "total_cost": total_cost,
+    }
+    one_day = 60 * 60 * 24
+    await cache.aset(cache_key, data, one_day)
 
-    return count, num_distinct_purchases, total_cost
+    return count, num_distinct, total_cost