From: Jane Sandberg <sandbej@linnbenton.edu>
Date: Fri, 14 Aug 2020 23:55:02 +0000 (-0700)
Subject: LP1849212: display course information in catalog for record-only acmcm entries
X-Git-Url: https://old-git.evergreen-ils.org/?a=commitdiff_plain;h=ac7469d19911a6ae48c660917ccf99ee21e4c518;p=working%2FEvergreen.git

LP1849212: display course information in catalog for record-only acmcm entries

Signed-off-by: Jane Sandberg <sandbej@linnbenton.edu>
---

diff --git a/Open-ILS/src/eg2/src/app/staff/catalog/result/record.component.html b/Open-ILS/src/eg2/src/app/staff/catalog/result/record.component.html
index 27a0d545ec..a585474281 100644
--- a/Open-ILS/src/eg2/src/app/staff/catalog/result/record.component.html
+++ b/Open-ILS/src/eg2/src/app/staff/catalog/result/record.component.html
@@ -42,14 +42,14 @@
               <ng-container *ngIf="hasMrConstituentRecords(summary)">
                   <a routerLink="/staff/catalog/search"
                     [queryParams]="appendFromMrParam(summary)">
-                    <eg-bib-display-field [summary]="summary" field="title" 
+                    <eg-bib-display-field [summary]="summary" field="title"
                       [usePlaceholder]="true"></eg-bib-display-field>
                   </a>
               </ng-container>
               <ng-container *ngIf="!hasMrConstituentRecords(summary)">
                 <a routerLink="/staff/catalog/record/{{summary.id}}"
                   [queryParams]="currentParams()">
-                  <eg-bib-display-field [summary]="summary" field="title" 
+                  <eg-bib-display-field [summary]="summary" field="title"
                     [usePlaceholder]="true"></eg-bib-display-field>
                 </a>
               </ng-container>
@@ -59,7 +59,7 @@
             <div class="col-lg-12">
               <a routerLink="/staff/catalog/search"
                 [queryParams]="getAuthorSearchParams(summary)">
-                <eg-bib-display-field [summary]="summary" field="author" 
+                <eg-bib-display-field [summary]="summary" field="author"
                   [usePlaceholder]="true"></eg-bib-display-field>
               </a>
             </div>
@@ -81,14 +81,14 @@
             <div class="col-lg-12">
               <ng-container *ngIf="summary.display.physical_description">
                 <!-- [].concat() to avoid modifying the summary arrays -->
-                <div class="pb-1" i18n>Phys. Desc.: 
-                  <eg-bib-display-field [summary]="summary" 
+                <div class="pb-1" i18n>Phys. Desc.:
+                  <eg-bib-display-field [summary]="summary"
                     field="physical_description" joiner=","></eg-bib-display-field>
                 </div>
               </ng-container>
               <ng-container *ngIf="summary.display.edition">
-                <div class="pb-1" i18n>Edition: 
-                  <eg-bib-display-field [summary]="summary" 
+                <div class="pb-1" i18n>Edition:
+                  <eg-bib-display-field [summary]="summary"
                     field="edition" joiner=","></eg-bib-display-field>
                 </div>
               </ng-container>
@@ -101,31 +101,31 @@
                   </div>
                 </ng-container>
                 <ng-template #pubDate>
-                  <div class="pb-1" i18n>Pub Date: 
+                  <div class="pb-1" i18n>Pub Date:
                     <eg-bib-display-field [summary]="summary" field="pubdate">
                     </eg-bib-display-field>
                   </div>
                 </ng-template>
               </ng-container>
               <ng-container *ngIf="summary.display.isbn">
-                <div class="pb-1" i18n>ISBN: 
-                  <eg-bib-display-field [summary]="summary" 
+                <div class="pb-1" i18n>ISBN:
+                  <eg-bib-display-field [summary]="summary"
                     field="isbn" joiner=","></eg-bib-display-field>
                 </div>
               </ng-container>
               <ng-container *ngIf="summary.display.upc">
-                <div class="pb-1" i18n>UPC: 
-                  <eg-bib-display-field [summary]="summary" 
+                <div class="pb-1" i18n>UPC:
+                  <eg-bib-display-field [summary]="summary"
                     field="upc" joiner=","></eg-bib-display-field>
                 </div>
               </ng-container>
               <ng-container *ngIf="summary.display.issn">
-                <div i18n>ISSN: 
-                  <eg-bib-display-field [summary]="summary" 
+                <div i18n>ISSN:
+                  <eg-bib-display-field [summary]="summary"
                     field="issn" joiner=","></eg-bib-display-field>
                 </div>
               </ng-container>
-              <ng-container *ngIf="has_course">
+              <ng-container *ngIf="hasCourse">
                 <div i18n>Associated Courses:
                   <span *ngFor="let course of courses; let isLast=last">
                     <a routerLink="/staff/admin/local/asset/course_list/{{course.id()}}">
@@ -139,7 +139,7 @@
         </div>
       </div>
       <div class="col-lg-2">
-        <div class="row" [ngClass]="{'pt-2':copyIndex > 0}" 
+        <div class="row" [ngClass]="{'pt-2':copyIndex > 0}"
           *ngFor="let copyCount of summary.holdingsSummary; let copyIdx = index">
           <div class="w-100" *ngIf="copyCount.type == 'staff'">
             <div class="float-left text-left w-50">
@@ -171,7 +171,7 @@
             <div class="float-right small-text-1">
               Created {{summary.record.create_date() | date:'shortDate'}} by
               <!-- creator if fleshed after the initial data set is loaded -->
-              <a *ngIf="summary.record.creator().usrname" target="_self" 
+              <a *ngIf="summary.record.creator().usrname" target="_self"
                 href="/eg/staff/circ/patron/{{summary.record.creator().id()}}/checkout">
                   {{summary.record.creator().usrname()}}
               </a>
@@ -184,7 +184,7 @@
           <div class="col-lg-12">
             <div class="float-right small-text-1" i18n>
               Edited {{summary.record.edit_date() | date:'shortDate'}} by
-              <a *ngIf="summary.record.editor().usrname" target="_self" 
+              <a *ngIf="summary.record.editor().usrname" target="_self"
                 href="/eg/staff/circ/patron/{{summary.record.editor().id()}}/checkout">
                   {{summary.record.editor().usrname()}}
               </a>
diff --git a/Open-ILS/src/eg2/src/app/staff/catalog/result/record.component.ts b/Open-ILS/src/eg2/src/app/staff/catalog/result/record.component.ts
index d587f5a0ba..31f7d02588 100644
--- a/Open-ILS/src/eg2/src/app/staff/catalog/result/record.component.ts
+++ b/Open-ILS/src/eg2/src/app/staff/catalog/result/record.component.ts
@@ -30,7 +30,7 @@ export class ResultRecordComponent implements OnInit, OnDestroy {
     searchContext: CatalogSearchContext;
     isRecordSelected: boolean;
     basketSub: Subscription;
-    has_course: boolean;
+    hasCourse = false;
     courses: any[] = [];
 
     constructor(
@@ -62,17 +62,14 @@ export class ResultRecordComponent implements OnInit, OnDestroy {
     }
 
     loadCourseInformation(recordId) {
-        console.log("Entering loadCourseInformation");
         this.course.isOptedIn().then(res => {
             if (res) {
-                this.course.fetchCopiesInCourseFromRecord(recordId).then(course_list => {
+                this.course.fetchCoursesForRecord(recordId).then(course_list => {
                     Object.keys(course_list).forEach(key => {
                         this.courses.push(course_list[key]);
                     });
-                    this.has_course = true;
+                    this.hasCourse = true;
                 });
-            } else {
-                this.has_course = false;
             }
         });
     }
diff --git a/Open-ILS/src/eg2/src/app/staff/share/bib-summary/bib-summary.component.html b/Open-ILS/src/eg2/src/app/staff/share/bib-summary/bib-summary.component.html
index a8e39532d3..16426c22ec 100644
--- a/Open-ILS/src/eg2/src/app/staff/share/bib-summary/bib-summary.component.html
+++ b/Open-ILS/src/eg2/src/app/staff/share/bib-summary/bib-summary.component.html
@@ -15,12 +15,12 @@
     </div>
     <div class="flex-1"></div>
     <div>
-      <a class="with-material-icon no-href text-primary" 
+      <a class="with-material-icon no-href text-primary"
         title="Show More" i18n-title
         *ngIf="!expand" (click)="expand=true">
         <span class="material-icons">expand_more</span>
       </a>
-      <a class="with-material-icon no-href text-primary" 
+      <a class="with-material-icon no-href text-primary"
         title="Show Less" i18n-title
         *ngIf="expand" (click)="expand=false">
         <span class="material-icons">expand_less</span>
@@ -39,7 +39,7 @@
         </a>
       </div>
     </ng-container>
-    
+
     <!-- make room for the jacket image when expanded -->
     <div [ngClass]="{'col-lg-11 pl-0': expand, 'col-lg-12': !expand}">
       <div class="card-body">
@@ -91,7 +91,7 @@
               <div class="flex-1">{{summary.record.edit_date() | date:'short'}}</div>
             </div>
           </li>
-          <ng-container *ngIf="expand && has_course">
+          <ng-container *ngIf="expand && hasCourse">
             <li class="list-group-item">
               <div class="d-flex">
                 <div class="flex-1 font-weight-bold" i18n>Associated Courses</div>
diff --git a/Open-ILS/src/eg2/src/app/staff/share/bib-summary/bib-summary.component.ts b/Open-ILS/src/eg2/src/app/staff/share/bib-summary/bib-summary.component.ts
index edd73bf57f..9103dea1ee 100644
--- a/Open-ILS/src/eg2/src/app/staff/share/bib-summary/bib-summary.component.ts
+++ b/Open-ILS/src/eg2/src/app/staff/share/bib-summary/bib-summary.component.ts
@@ -14,7 +14,7 @@ import {CatalogService} from '@eg/share/catalog/catalog.service';
 export class BibSummaryComponent implements OnInit {
 
     initDone = false;
-    has_course = false;
+    hasCourse = false;
     courses: any;
 
     // True / false if the display is vertically expanded
@@ -78,15 +78,13 @@ export class BibSummaryComponent implements OnInit {
         });
     }
 
-    loadCourseInformation(record_id) {
+    loadCourseInformation(recordId) {
         this.org.settings('circ.course_materials_opt_in').then(setting => {
             if (setting['circ.course_materials_opt_in']) {
-                this.course.fetchCopiesInCourseFromRecord(record_id).then(course_list => {
-                    this.courses = course_list;
-                    this.has_course = true;
+                this.course.fetchCoursesForRecord(recordId).then(courseList => {
+                    this.courses = courseList;
+                    this.hasCourse = true;
                 });
-            } else {
-                this.has_course = false;
             }
         });
     }
diff --git a/Open-ILS/src/eg2/src/app/staff/share/course.service.ts b/Open-ILS/src/eg2/src/app/staff/share/course.service.ts
index 2aa18b7823..f8d464df6d 100644
--- a/Open-ILS/src/eg2/src/app/staff/share/course.service.ts
+++ b/Open-ILS/src/eg2/src/app/staff/share/course.service.ts
@@ -82,23 +82,12 @@ export class CourseService {
         });
     }
 
-    fetchCopiesInCourseFromRecord(record_id) {
-        const cp_list = [];
-        return new Promise((resolve, reject) => {
-            this.net.request(
-                'open-ils.cat',
-                'open-ils.cat.asset.copy_tree.global.retrieve',
-                this.auth.token(), record_id
-            ).subscribe(copy_tree => {
-                copy_tree.forEach(cn => {
-                    cn.copies().forEach(cp => {
-                        cp_list.push(cp.id());
-                    });
-                });
-            }, err => reject(err),
-            () => {
-                resolve(this.getCoursesFromMaterial(cp_list));
-            });
+    fetchCoursesForRecord(recordId) {
+        return this.pcrud.search(
+            'acmcm', {record: recordId}, {atomic: true}
+        ).toPromise().then((materials) => {
+            const courseIds = materials.map((material) => material.course());
+            return this.getCourses(Array.from(new Set(courseIds)));
         });
     }