Add option to filter reading stats by year read (#10679)

Co-authored-by: Drini Cami <[email protected]>

Author: RedJade26

openlibrary/plugins/upstream/mybooks.py

@@ -157,9 +157,9 @@ def GET(self, username):
 
 
 class readinglog_stats(delegate.page):
-    path = "/people/([^/]+)/books/(want-to-read|currently-reading|already-read)/stats"
+    path = "/people/([^/]+)/books/(want-to-read|currently-reading|already-read)(/year/\\d{4})?/stats"
 
-    def GET(self, username, key='want-to-read'):
+    def GET(self, username, key='want-to-read', year=None):
         user = web.ctx.site.get('/people/%s' % username)
         if not user:
             return render.notfound("User %s" % username, create=False)
@@ -168,8 +168,13 @@ def GET(self, username, key='want-to-read'):
         if not cur_user or cur_user.key.split('/')[-1] != username:
             return render.permission_denied(web.ctx.path, 'Permission Denied')
 
+        yearly_reads = BookshelvesEvents.get_user_yearly_read_counts(username)
+        if year:
+            # eg '/year/2025'; skip the '/year/'
+            year = int(year.split('/')[-1])
+
         readlog = ReadingLog(user=user)
-        works = readlog.get_works(key, page=1, limit=2000).docs
+        works = readlog.get_works(key, page=1, limit=2000, year=year).docs
         works_json = [
             {
                 # Fallback to key if it is a redirect
@@ -203,6 +208,8 @@ def GET(self, username, key='want-to-read'):
             web.ctx.path.rsplit('/', 1)[0],
             key,
             lang=web.ctx.lang,
+            year=year,
+            yearly_reads=yearly_reads,
         )
 
 

openlibrary/templates/account/readinglog_stats.html

@@ -1,4 +1,4 @@
-$def with (works_json, authors_json, works_count, owner_key, owner_name, shelf_path, shelf_key, lang='en')
+$def with (works_json, authors_json, works_count, owner_key, owner_name, shelf_path, shelf_key, year,yearly_reads, lang='en')
 
 $ reading_log_stats_config = {
 $    'works': works_json,
@@ -104,6 +104,19 @@
         »
     </div>
     <h1>$_('My %(shelf_name)s Stats', shelf_name=shelf_name)</h1>
+    $if shelf_key == 'already-read' and yearly_reads:
+    <div class="breadcrumb-wrapper">
+        <div class="sansserif grey account-settings-menu navigation-breadcrumbs">
+            $ all_url = owner_key + '/books/already-read/stats'
+            $ year_url = owner_key + '/books/already-read/year/%d/stats'
+            $if year:
+                $:render_template("books/year_breadcrumb_select", yearly_reads, all_url, year_url, selected= str(year))
+            $else:
+
+                $:render_template("books/year_breadcrumb_select", yearly_reads, all_url, year_url )
+
+        </div>
+    </div>
 
     <p>$:_('Displaying stats about <strong>%(works_count)d</strong> books. Note all charts show only the top 20 bars. Note reading log stats are private.', works_count=works_count)</p>
 </div>