From 10f4261eebba392616bbeb5b41ff3fbc947a08fc Mon Sep 17 00:00:00 2001
From: Ihor Brazhnichenko <ihor.bra@gmail.com>
Date: Wed, 9 Nov 2022 13:23:33 +0200
Subject: [PATCH] #23 Add pagination for operations

---
 app/sections/credit_offers/credit_offer.html  |  2 ++
 app/sections/dashboard/dashboard.html         |  5 +++-
 .../operations/operations-table.directive.js  | 25 ++++++++++++++-----
 app/sections/operations/operations.html       |  5 +++-
 app/sections/operations/operations.service.js |  2 +-
 app/sections/pools/pool.html                  |  2 ++
 6 files changed, 32 insertions(+), 9 deletions(-)

diff --git a/app/sections/credit_offers/credit_offer.html b/app/sections/credit_offers/credit_offer.html
index 5635d7b..2ce881f 100644
--- a/app/sections/credit_offers/credit_offer.html
+++ b/app/sections/credit_offers/credit_offer.html
@@ -40,6 +40,8 @@
                           group-by-credit-offer-id="credit_offer_id"
                           filter-by-asset-id-enabled="false"
                           show-filters="showFilters"
+                          default-date-from="now-1y"
+                          filters-date-from="now-1y"
         />
     </div>
 </div>
diff --git a/app/sections/dashboard/dashboard.html b/app/sections/dashboard/dashboard.html
index 0ee287a..c03d7fc 100644
--- a/app/sections/dashboard/dashboard.html
+++ b/app/sections/dashboard/dashboard.html
@@ -143,7 +143,10 @@
             <img src="images/filter.svg" ng-click="showFilters = !showFilters" class="operations-filter"/>
         </div>
 
-        <operations-table total-items="total_ops" show-filters="showFilters"/>
+        <operations-table total-items="total_ops"
+                          show-filters="showFilters"
+                          default-date-from="now-1M"
+                          filters-date-from="now-1y"/>
 
     </div>
 </div>
diff --git a/app/sections/operations/operations-table.directive.js b/app/sections/operations/operations-table.directive.js
index ba0f2d6..f38cc66 100644
--- a/app/sections/operations/operations-table.directive.js
+++ b/app/sections/operations/operations-table.directive.js
@@ -31,12 +31,13 @@ This show/hide filtering fields for the user to filter all operations by account
  _ _ _ _ _ _ _ _
 |   Date From  |
  - - - - - - - -
-Use this field to optimize ES search by date limits
+Use this fields to optimize ES search by date limits
 
 // By default: now-1M
 
 (Example)
-<operations-table date-from="now-1y"/>
+<operations-table default-date-from="now-1y"/> this is a date-from which applies when no filters
+<operations-table filters-date-from="now-1y"/> this is a date-from which applies when any filter defined by the user
 
  _ _ _ _ _ _ _ _ _
 |   Show filters  |
@@ -63,7 +64,8 @@ Use this field show / hide user filters
                 groupByAccountId: '=',
                 groupByCreditOfferId: '=',
                 groupByPoolId: '=',
-                dateFrom: "@",
+                defaultDateFrom: "@",
+                filtersDateFrom: "@",
                 filterByAccountIdEnabled: '=',
                 filterByAssetIdEnabled: '=',
                 filterByOperationTypeEnabled: '=',
@@ -74,6 +76,10 @@ Use this field show / hide user filters
         
         function operationsTableController($scope, $filter, Notify, OperationsService, utilities) {
             
+            function filtersDefined() {
+                return $scope.filters.operationType !== '-1' || $scope.filters.accountIdOrName || $scope.filters.assetIdOrName;
+            }
+            
             // list of values for filter by operation type <select>
             $scope.operationTypes = new Array(75).fill('#').map((item, key) => {
                 const name = utilities.operationType(key)[0]
@@ -85,7 +91,10 @@ Use this field show / hide user filters
                 }
             }).filter((item) => !!item);
             
-            $scope.dateFrom = $scope.dateFrom || 'now-1M'
+            $scope.totalItems = 0;
+            
+            $scope.defaultDateFrom = $scope.defaultDateFrom || 'now-1M'
+            $scope.filtersDateFrom = $scope.filtersDateFrom || 'now-1y'
             
             // default values for filtering fields
             $scope.filters = {
@@ -113,6 +122,9 @@ Use this field show / hide user filters
             ];
             
             $scope.select = function (page_operations) {
+                if(page_operations === 1)
+                    $scope.totalItems = 0;
+                
                 const page = page_operations - 1;
                 const limit = 20;
                 const offset = page * limit;
@@ -129,7 +141,7 @@ Use this field show / hide user filters
                     limit,
                     offset,
                     date_to,
-                    date_from: $scope.dateFrom || undefined,
+                    date_from: filtersDefined() ? $scope.filtersDateFrom : $scope.defaultDateFrom || undefined,
                     assetId: $scope.filters.assetIdOrName,
                     accountId: $scope.groupByAccountId || $scope.filters.accountIdOrName,
                     creditOfferId: $scope.groupByCreditOfferId,
@@ -158,7 +170,8 @@ Use this field show / hide user filters
                             allowMultiple: true
                         });
                     } else {
-                        $scope.operations = response;
+                        $scope.totalItems += response.length;
+                        $scope.operations = response.slice(0, limit); // remove the additional item which defines the existence of the next page
                         $scope.currentPage = page_operations;
                         if (page_operations == 1) {
                             if (response.length > 0) {
diff --git a/app/sections/operations/operations.html b/app/sections/operations/operations.html
index 3ec35c2..079e133 100644
--- a/app/sections/operations/operations.html
+++ b/app/sections/operations/operations.html
@@ -36,6 +36,9 @@
             <img src="images/filter.svg" ng-click="showFilters = !showFilters" class="operations-filter"/>
         </div>
 
-        <operations-table show-filters="showFilters"/>
+        <operations-table show-filters="showFilters"
+                          default-date-from="now-1M"
+                          filters-date-from="now-1y"
+        />
     </div>
 </div>
diff --git a/app/sections/operations/operations.service.js b/app/sections/operations/operations.service.js
index b1d67b7..0441fe8 100644
--- a/app/sections/operations/operations.service.js
+++ b/app/sections/operations/operations.service.js
@@ -53,7 +53,7 @@
             
             return $http.get(appConfig.urls.elasticsearch_wrapper() + "/history", {
                     params: {
-                        "limit": limit,
+                        "limit": limit + 1, // fetch a one more item to define is there a next page
                         "offset": offset,
                         "from_date": date_from ? date_from : (offset < MAGIC_NUMBER ? "now-1M" : "2015-10-10"),
                         "to_date": date_to,
diff --git a/app/sections/pools/pool.html b/app/sections/pools/pool.html
index a739384..0397bb2 100644
--- a/app/sections/pools/pool.html
+++ b/app/sections/pools/pool.html
@@ -164,6 +164,8 @@
                               filter-by-asset-id-enabled="false"
                               filter-by-operation-type-enabled="false"
                               show-filters="showFilters"
+                              default-date-from="now-1y"
+                              filters-date-from="now-1y"
             />
         </div>
     </div>