From 9b5ef063a1bfdafdad5fbf3c0a380e34c6b2e840 Mon Sep 17 00:00:00 2001
From: Bill Erickson <berickxx@gmail.com>
Date: Tue, 15 Mar 2022 10:15:30 -0400
Subject: [PATCH] LP1952931 LI# link goes to LI items; show title in summary

Signed-off-by: Bill Erickson <berickxx@gmail.com>
---
 Open-ILS/src/eg2/src/app/staff/acq/asn/receive.component.html | 9 +++++++--
 Open-ILS/src/eg2/src/app/staff/acq/asn/receive.component.ts   | 8 +++++++-
 2 files changed, 14 insertions(+), 3 deletions(-)

diff --git a/Open-ILS/src/eg2/src/app/staff/acq/asn/receive.component.html b/Open-ILS/src/eg2/src/app/staff/acq/asn/receive.component.html
index c59d16b33e..0009e5ea34 100644
--- a/Open-ILS/src/eg2/src/app/staff/acq/asn/receive.component.html
+++ b/Open-ILS/src/eg2/src/app/staff/acq/asn/receive.component.html
@@ -113,7 +113,7 @@ across different vendors to match a container code.
 <ng-template #liIdTmpl let-row="row">
   <a target="_blank" 
     fragment="{{row.lineitem.id()}}"
-    routerLink="/staff/acq/po/{{row.lineitem.purchase_order().id()}}">
+    routerLink="/staff/acq/po/{{row.lineitem.purchase_order().id()}}/lineitem/{{row.lineitem.id()}}">
     {{row.lineitem.id()}}
   </a>
 </ng-template>
@@ -125,7 +125,7 @@ across different vendors to match a container code.
 </ng-template>
 
 <div class="row" *ngIf="receiving">
-  <div class="col-lg-8 offset-lg-2">
+  <div class="col-lg-10 offset-lg-1">
     <div class="card">
       <div class="card-header" i18n>Receiving Items <span *ngIf="dryRun"> (Dry Run)</span></div>
       <div class="card-body">
@@ -135,12 +135,17 @@ across different vendors to match a container code.
           </li>
 
           <li class="list-group-item d-flex font-weight-bold">
+            <div class="flex-3" i18n>Title</div>
             <div class="flex-1" i18n>Lineitem</div>
             <div class="flex-1" i18n>Notified</div>
             <div class="flex-1" i18n>Received</div>
           </li>
 
           <li class="list-group-item d-flex" *ngFor="let li of receiveResponse.lineitems">
+            <div class="flex-3">
+              <a routerLink="/staff/catalog/record/{{liCache[li.id].lineitem.eg_bib_id()}}"
+                target="_blank">{{liCache[li.id].title}}</a>
+            </div>
             <div class="flex-1">
               <a routerLink="/staff/acq/po/{{li.po}}/lineitem/{{li.id}}/items"
                 target="_blank">#{{li.id}}</a>
diff --git a/Open-ILS/src/eg2/src/app/staff/acq/asn/receive.component.ts b/Open-ILS/src/eg2/src/app/staff/acq/asn/receive.component.ts
index 7ddfd4411f..07ceddeacf 100644
--- a/Open-ILS/src/eg2/src/app/staff/acq/asn/receive.component.ts
+++ b/Open-ILS/src/eg2/src/app/staff/acq/asn/receive.component.ts
@@ -30,6 +30,7 @@ export class AsnReceiveComponent implements OnInit {
     dryRun = true;
     receiveOnScan = false;
     notFound = false;
+    liCache: {[id: number]: any} = {};
 
     // Technically possible for one container code to match across providers.
     container: IdlObject;
@@ -66,7 +67,7 @@ export class AsnReceiveComponent implements OnInit {
     gridifyEntry(entry: IdlObject): any {
         const li = entry.lineitem();
         const sum = li.order_summary();
-        return {
+        const display = {
             entry: entry,
             lineitem: li,
             title: this.li.getFirstAttributeValue(li, 'title'),
@@ -78,6 +79,10 @@ export class AsnReceiveComponent implements OnInit {
                 sum.recv_count() + sum.cancel_count()
             )
         };
+
+        this.liCache[li.id()] = display;
+
+        return display;
     }
 
     findContainer() {
@@ -86,6 +91,7 @@ export class AsnReceiveComponent implements OnInit {
         this.container = null;
         this.containers = [];
         this.entries = [];
+        this.liCache = {};
 
         this.pcrud.search('acqsn',
             {container_code: this.barcode},
-- 
2.11.0