From fe449abb47f144d2a2f6773cbb2fcbb89b063c15 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?V=C3=ADctor=20Losada=20Hern=C3=A1ndez?=
 <losada.hernandez.victor@gmail.com>
Date: Sat, 10 Feb 2024 16:41:39 +0100
Subject: [PATCH] trying to figure out pagination

---
 .../pages/archivePage/archivePage.jsx         | 126 ++++++++++--------
 server/archive.api.js                         |  50 ++++---
 2 files changed, 96 insertions(+), 80 deletions(-)

diff --git a/client/homebrew/pages/archivePage/archivePage.jsx b/client/homebrew/pages/archivePage/archivePage.jsx
index a644ff449..bf6446f3c 100644
--- a/client/homebrew/pages/archivePage/archivePage.jsx
+++ b/client/homebrew/pages/archivePage/archivePage.jsx
@@ -24,85 +24,95 @@ const ArchivePage = createClass({
     return {
       title      : this.props.query.title || '',
       brewCollection : null,
+      page       : 1,
+      totalPages : 1,
       searching  : false,
       error      : null,
-      limit      : '',
     };
   },
   componentDidMount : function() {
-    this.lookup();
+    this.loadPage(1);
   },
-  
-
   handleChange(e) {
     this.setState({ title: e.target.value });
   },
-  lookup() {
-    this.setState({ searching: true, error: null });
-    request
-      .get(`/archive/${this.state.title}`)
-      .then((res) => this.setState({ brewCollection: res.body.brews }, this.setState({ limit: res.body.message}, this.setState({ error: null}))))
-      .catch((err) => this.setState({ error: err }))
-      .finally(() => this.setState({ searching: false }));
+
+  updateStateWithBrews : (brews, page, totalPages) => {
+    this.setState({
+      brewCollection: brews,
+      page: page,
+      totalPages: totalPages,
+      searching: false
+    });
+  },
+  loadPage: async function(pageNumber) {
+    try {
+      this.updateUrl();
+      this.setState({ searching: true, error: null });
+      const title = encodeURIComponent(this.state.title);
+      const response = await fetch(`/archive?title=${title}&page=${pageNumber}`);
+
+  
+      if (response.ok) {
+        const res = await response.json();
+        this.updateStateWithBrews(res.brews, pageNumber, res.totalPages);
+      }
+
+    } catch (error) {
+      console.log("LoadPage error: " + error);
+    }
   },
   updateUrl: function() {
     const url = new URL(window.location.href);
     const urlParams = new URLSearchParams(url.search);
-    // Clear existing parameters
-    urlParams.delete('sort');
-    urlParams.delete('dir');
-    urlParams.delete('filter');
-
-    // Set the pathname to '/archive/?query'
+  
+    // Set the title and page parameters
     urlParams.set('title', this.state.title);
-
-    url.search = urlParams;
+    urlParams.set('page', this.state.page);
+  
+    url.search = urlParams.toString(); // Convert URLSearchParams to string
     window.history.replaceState(null, null, url);
-},
+  },
+  
   renderFoundBrews() {
-    const brews = this.state.brewCollection;
-    console.log('brews: ',brews);
+    const { title, brewCollection, page, totalPages, error } = this.state;
 
-    if (this.state.title == '') {
-      return(
-        <div className="foundBrews noBrews">
-          <h3>Whenever you want, just start typing...</h3>
-      </div>
-      );
-    }
+    if (title === '') {return (<div className="foundBrews noBrews"><h3>Whenever you want, just start typing...</h3></div>);}
 
-    if (this.state.error !== null) {
-      return(
+    if (error !== null) { return (
         <div className="foundBrews noBrews">
-          <div>
-            <h3>I'm sorry, your request didn't work</h3>
-            <br />
-            <p>Your search is not enough specific, too many brews meet this criteria for us to forward them.</p>
+          <div><h3>I'm sorry, your request didn't work</h3>
+            <br /><p>Your search is not specific enough. Too many brews meet this criteria for us to display them.</p>
+          </div></div>
+    );}
+
+    if (!brewCollection || brewCollection.length === 0) { return (
+        <div className="foundBrews noBrews">
+          <h3>We haven't found brews meeting your request.</h3>
+        </div>
+      );}
+
+    return (
+      <div className="foundBrews">
+        <span className="brewCount">{`Brews Found: ${brewCollection.length}`}</span>
+
+        {brewCollection.map((brew, index) => (
+          <BrewItem brew={brew} key={index} reportError={this.props.reportError} />
+        ))}
+        <div className="paginationControls">
+          {page > 1 && (
+            <button onClick={() => this.loadPage(page - 1)}>Previous Page</button>
+          )}
+          <span className="currentPage">Page {page}</span>
+          {page < totalPages && (
+            <button onClick={() => this.loadPage(page + 1)}>Next Page</button>
+          )}
         </div>
       </div>
-      );
-    }
-
-    if (!brews || brews.length === 0) {
-      return(
-      <div className="foundBrews noBrews">
-        <h3>We haven't found brews meeting your request.</h3>
-      </div>
-       
-      );
-    }
-    this.updateUrl();
-      return <div className="foundBrews">
-        <span className="brewCount">{brews.length} Brews Found</span>
-        <span className="limit">{this.state.limit}</span>
-        {brews.map((brew, index) => (
-        <BrewItem brew={brew} key={index} reportError={this.props.reportError}/>
-        ))}
-      </div>
-       
-    
+    );
   },
 
+
   renderForm: function () {
     return (
       <div className='brewLookup'>
@@ -115,7 +125,7 @@ const ArchivePage = createClass({
           onKeyDown={(e) => {
             if (e.key === 'Enter') {
               this.handleChange(e);
-              this.lookup();
+              this.loadPage(1);
             }
           }}
           placeholder='v3 Reference Document'
@@ -125,7 +135,7 @@ const ArchivePage = createClass({
             <input type="checkbox" id="v3" /><label>v3 only</label>
             */}
             
-            <button onClick={() => { this.handleChange({ target: { value: this.state.title } }); this.lookup(); }}>
+            <button onClick={() => { this.handleChange({ target: { value: this.state.title } }); this.loadPage(1); }}>
           <i
             className={cx('fas', {
               'fa-search': !this.state.searching,
diff --git a/server/archive.api.js b/server/archive.api.js
index 8642f0538..18161aecf 100644
--- a/server/archive.api.js
+++ b/server/archive.api.js
@@ -7,40 +7,46 @@ const archive = {
     /* Searches for matching title, also attempts to partial match */
     findBrews: async (req, res, next) => {
         try {
-            const limit = 1000;
-            const brews = await HomebrewModel.find({
-                title: { $regex: req.params.query, $options: 'i' },
+            const page = parseInt(req.params.page) || 1;
+            console.log('try:',page);
+            const pageSize = 10; // Set a default page size
+            const skip = (page - 1) * pageSize;
+
+            const title = {
+                title: { $regex: decodeURIComponent(req.params.title), $options: 'i' },
                 published: true
-            },
-            {
-                editId:0,
-                googleId:0,
-                text:0,
-                textBin:0,
-            })
-            .limit(limit)
-            .maxTimeMS(10000)
-            .exec();
+            };
+
+            const projection = {
+                editId: 0,
+                googleId: 0,
+                text: 0,
+                textBin: 0,
+            };
+
+            const brews = await HomebrewModel.find(title, projection)
+                .skip(skip)
+                .limit(pageSize)
+                .maxTimeMS(5000)
+                .exec();
 
             if (!brews || brews.length === 0) {
                 // No published documents found with the given title
                 return res.status(404).json({ error: 'Published documents not found' });
             }
 
-            let message = '';
-            if (brews.length === limit) {
-                // If the limit has been reached, include a message in the response
-                message = `You've reached the limit of ${limit} documents, you can try being more specific in your search.`;
-            }
+            const totalDocuments = await HomebrewModel.countDocuments(title);
 
-            return res.json({ brews, message });
+            const totalPages = Math.ceil(totalDocuments / pageSize);
+
+            return res.json({ brews, page, totalPages});
         } catch (error) {
             console.error(error);
             return res.status(500).json({ error: 'Internal Server Error' });
         }
     }
-}
+};
 
-router.get('/archive/:query', asyncHandler(archive.findBrews));
+router.get('/archive/:title/:page', asyncHandler(archive.findBrews));
 
-module.exports = router;
+module.exports = router;
\ No newline at end of file