From b2b4ae930ec92c48a3c1fea2c7d961c1ab5f225e Mon Sep 17 00:00:00 2001
From: Bertrand Zuchuat <bertrand.zuchuat@rero.ch>
Date: Wed, 10 Apr 2024 13:39:42 +0200
Subject: [PATCH] fix(document): import with many identifiers

If the size of the query exceeds 1024 characters, we truncate
the identifiers and display a warning message to the user.

* Closes rero/rero-ils#3118.

Co-Authored-by: Bertrand Zuchuat <bertrand.zuchuat@rero.ch>
---
 .../dialog-import/dialog-import.component.html   | 12 ++++++++++++
 .../dialog-import/dialog-import.component.ts     |  3 +++
 .../document-detail/document-detail.component.ts | 16 ++++++++++++++--
 3 files changed, 29 insertions(+), 2 deletions(-)

diff --git a/projects/admin/src/app/record/detail-view/document-detail-view/dialog-import/dialog-import.component.html b/projects/admin/src/app/record/detail-view/document-detail-view/dialog-import/dialog-import.component.html
index 7c8b6e3e5..1c50f3908 100644
--- a/projects/admin/src/app/record/detail-view/document-detail-view/dialog-import/dialog-import.component.html
+++ b/projects/admin/src/app/record/detail-view/document-detail-view/dialog-import/dialog-import.component.html
@@ -36,6 +36,18 @@ <h5 translate>Documents are available</h5>
       </li>
     }
   </ul>
+  @if (warningMessages.length > 0) {
+    <div class="mt-3 row">
+      <div class="col-1">
+        <i class="fa fa-exclamation-circle fa-2x text-danger" aria-hidden="true"></i>
+      </div>
+      <div class="col-11">
+        @for (message of warningMessages; track message) {
+          <p class="mb-2 font-weight-bold">{{ message }}</p>
+        }
+      </div>
+    </div>
+  }
   <div class="font-weight-bold mt-3" translate>Do you want to import this document?</div>
 </div>
 <div class="modal-footer">
diff --git a/projects/admin/src/app/record/detail-view/document-detail-view/dialog-import/dialog-import.component.ts b/projects/admin/src/app/record/detail-view/document-detail-view/dialog-import/dialog-import.component.ts
index a6ce58d08..48d54d5ec 100644
--- a/projects/admin/src/app/record/detail-view/document-detail-view/dialog-import/dialog-import.component.ts
+++ b/projects/admin/src/app/record/detail-view/document-detail-view/dialog-import/dialog-import.component.ts
@@ -27,6 +27,9 @@ export class DialogImportComponent {
   /** Available record */
   records: any[];
 
+  /** Message */
+  warningMessages: string[] = [];
+
   /** Observable for action */
   confirmation$: Subject<boolean> = new Subject<boolean>();
 
diff --git a/projects/admin/src/app/record/detail-view/document-detail-view/document-detail/document-detail.component.ts b/projects/admin/src/app/record/detail-view/document-detail-view/document-detail/document-detail.component.ts
index 03943a5fd..3c7f58087 100644
--- a/projects/admin/src/app/record/detail-view/document-detail-view/document-detail/document-detail.component.ts
+++ b/projects/admin/src/app/record/detail-view/document-detail-view/document-detail/document-detail.component.ts
@@ -154,7 +154,12 @@ export class DocumentDetailComponent extends DetailComponent {
       this.router.navigate(route, { queryParams: data });
     } else {
       // Find documents(s) with query params
-      const query = queryParams.join(' OR ');
+      let warning = false;
+      let query = queryParams.join(' OR ');
+      if (query.length > 1024) {
+        query = query.substring(0, query.substring(0, 1024).lastIndexOf('OR') - 1);
+        warning = true;
+      }
       this.recordService.getRecords(
         'documents', query, 1, undefined, undefined, undefined, { accept: 'application/rero+json' }
       ).subscribe((response: Record) => {
@@ -163,9 +168,16 @@ export class DocumentDetailComponent extends DetailComponent {
         } else {
           const config = {
             initialState: {
-              records: response.hits.hits
+              records: response.hits.hits,
+              warningMessages: []
             }
           };
+          if (warning) {
+            config.initialState.warningMessages = [
+              this.translate.instant('All the identifiers have not been checked.'),
+              this.translate.instant('Please, check manually if a record is already present in the catalog.')
+            ];
+          }
           const bsModalRef = this.bsModalService.show(DialogImportComponent, config);
           bsModalRef.content.confirmation$.subscribe((confirmation: boolean) => {
             if (confirmation) {